Skip to content

[STG-1681] Align tools with hosted MCP server#161

Open
shrey150 wants to merge 3 commits intomainfrom
align-with-hosted-mcp
Open

[STG-1681] Align tools with hosted MCP server#161
shrey150 wants to merge 3 commits intomainfrom
align-with-hosted-mcp

Conversation

@shrey150
Copy link
Contributor

@shrey150 shrey150 commented Mar 25, 2026

Summary

  • Rename all 6 tools to match hosted MCP at mcp.browserbase.com: start, end, navigate, act, observe, extract
  • Remove tools not in hosted version: screenshot, get_url, agent
  • Align tool input schemas (remove variables from act, remove sessionId from start, make extract instruction optional)
  • Update default model from gemini-2.0-flash to google/gemini-2.5-flash-lite
  • Align response format to { success: true, data: ... } wrapper
  • Remove all Smithery references (badge, image, yaml, SDK dep, CLI script)
  • Remove unused deps (sharp, @mcp-ui/server, @smithery/cli)
  • Update README to lead with hosted MCP, document new tool names
  • Add vitest unit tests (11 tests) + STDIO smoke test
  • Update eval configs to match new tool names
  • Add pnpm test step to CI

Test plan

  • pnpm build passes
  • pnpm test passes — 11 unit tests (tool names, schemas, capabilities) + 1 STDIO smoke test
  • CI green — unit tests, smoke test, and evals all pass
  • E2E verified against live Browserbase session — connected via STDIO as MCP client, called all 6 tools in sequence:
    • start → created session ad7b310b-d780-4905-b8ef-ff7d5e282256
    • navigate → navigated to https://example.com
    • observe → found "Learn more link" with xpath selector
    • extract → extracted "Example Domain"
    • act → clicked "More information..." link successfully
    • end → session closed
    • All responses returned { success: true, data: ... } format ✅

🤖 Generated with Claude Code

Closes STG-1681

shrey150 and others added 2 commits March 25, 2026 14:58
BREAKING CHANGE: Tool names, schemas, and default model updated to match
the hosted Browserbase MCP server. Removed screenshot, get_url, and agent
tools. Removed all Smithery references and dependencies.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add vitest with 11 unit tests verifying tool names, schemas, and capabilities
- Add STDIO smoke test that spawns server and verifies tools/list response
- Update all 3 eval configs to use new tool names (start, end, navigate, act, observe, extract)
- Remove eval workflows referencing deleted tools (screenshot, url, agent)
- Add pnpm test step to CI pipeline (runs before evals, needs no secrets)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

**Breaking changes:**

- Tool `browserbase_session_create` renamed to `start`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we keep the prefix? might get confusing but if this is exact 1-1 with hosted it makes sense

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think making it line up 1-1 is the goal

The end tool now returns {"success": true} instead of
"closed successfully via Stagehand", so expectedState checks
need to use "success" instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

2 participants