Skip to content

chore: fix CI + release 0.5.0#13

Merged
triuzzi merged 1 commit into
mainfrom
fix/ci-everything-and-0.5.0
May 7, 2026
Merged

chore: fix CI + release 0.5.0#13
triuzzi merged 1 commit into
mainfrom
fix/ci-everything-and-0.5.0

Conversation

@triuzzi
Copy link
Copy Markdown
Owner

@triuzzi triuzzi commented May 7, 2026

Summary

Fixes all blocking CI failures and bumps to 0.5.0.

CI fixes

  • `run-tests.yml`: install Brave Browser before running e2e tests on ubuntu/macos/windows. The chrome-devtools test suite `runCli(['start'])` spawns Brave; runners come with Chrome pre-installed but not Brave, so every e2e `list_pages`/`take_screenshot` test failed with "Could not find Brave Browser".
  • `eslint.config.mjs` + `brave-integration.test.mjs`: add the integration test to `allowDefaultProject` (so the TS project service can parse it), rename unused `label`/`submitUid` to underscore-prefixed, and auto-format. Resolves the `[Required] Check correct format` blocker.
  • `scripts/generate-cli.ts`: write the generated CLI to `brave-devtools-cli-options.ts` (was still chrome-named) and use `BRAVE_DEVTOOLS_MCP_NO_USAGE_STATISTICS` in the spawn env. Resolves the `[Required] Check docs updated` blocker (`npm run gen` would otherwise leave a stray chrome-named file).
  • Remove `.github/workflows/release-please.yml`: it referenced `BROWSER_AUTOMATION_BOT_TOKEN`, an upstream-only secret. We cut releases manually via npm publish + git tag + gh release create, so this workflow is dead code in our fork.
  • `npm run gen` regenerated: `brave-devtools-mcp-cli-options.ts`, `flag_usage_metrics.json`. Prettier reformatted: `check-latest-version.ts`, `src/browser.ts`, `README.md`, `brave-integration.test.html`.

Release

Bumps `package.json`, `package-lock.json`, `server.json` (top-level + `packages[0]`), `.claude-plugin/plugin.json`, `src/version.ts` to 0.5.0.

Test plan

  • `npm run clean && npm run build` — clean
  • `npm run check-format` — clean (eslint + prettier)
  • `npm run gen` — no diff after re-run
  • `node scripts/test.mjs` — all passing locally
  • CI all green on this PR
  • After merge: tag `v0.5.0`, GitHub release, `npm publish brave-mcp@0.5.0`

🤖 Generated with Claude Code

CI fixes:
- run-tests.yml: install Brave Browser before running e2e tests on
  ubuntu/macos/windows runners (the chrome-devtools tests RunCli need
  a real Brave binary at the expected path).
- eslint.config.mjs + tests/e2e/brave-integration.test.mjs: include
  the integration test file in allowDefaultProject so it gets
  type-checked and linted; rename two unused identifiers (label,
  submitUid) to underscore-prefixed; auto-format.
- scripts/generate-cli.ts: write generated CLI to brave-devtools-cli-options.ts
  (was still pointing at the chrome-named path); use BRAVE_DEVTOOLS_MCP_NO_USAGE_STATISTICS
  in the spawn env.
- Remove .github/workflows/release-please.yml — uses upstream-only
  BROWSER_AUTOMATION_BOT_TOKEN secret and we cut releases manually.
- npm run gen ran cleanly: brave-devtools-mcp-cli-options.ts and
  flag_usage_metrics.json regenerated; check-latest-version.ts,
  src/browser.ts, README.md, brave-integration.test.html prettified.

Release:
- Bump package.json, package-lock.json, server.json (top-level +
  packages[0]), .claude-plugin/plugin.json, src/version.ts to 0.5.0.

Co-Authored-By: Claude <noreply@anthropic.com>
@triuzzi triuzzi self-assigned this May 7, 2026
@triuzzi triuzzi merged commit 2aaadf0 into main May 7, 2026
16 checks passed
@triuzzi triuzzi deleted the fix/ci-everything-and-0.5.0 branch May 7, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant