Skip to content

Expose region param on manage_browsers create and list - #158

Merged
tnsardesai merged 1 commit into
mainfrom
hypeship/mcp-browser-region
Aug 14, 2026
Merged

Expose region param on manage_browsers create and list#158
tnsardesai merged 1 commit into
mainfrom
hypeship/mcp-browser-region

Conversation

@tnsardesai

@tnsardesai tnsardesai commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

POST /browsers accepts a region param (us-east | eu-west, fixed at create, plan-gated to Start-Up/Enterprise, defaults to us-east), and GET /browsers can filter by it. The pinned @onkernel/sdk@0.90.0 already types both, but the manage_browsers tool schema never exposed the field — so MCP clients couldn't select or filter by region.

Changes

  • Add region to the manage_browsers input schema, described for both (create) and (list).
  • Pass region through on create and as a filter on list.
  • Test covering both pass-throughs.

Testing

bun test src/lib/mcp — 177 pass, 0 fail. bunx tsc --noEmit clean. bun run format:check flags only the pre-existing AGENTS.md formatting issue on main.


Note

Low Risk
Thin schema and pass-through wiring to an already-typed SDK; no auth or data-model changes.

Overview
Exposes region (us-east | eu-west) on the manage_browsers MCP tool so clients can match the REST API: set region at session create (fixed afterward, plan-gated) and filter list by region.

The tool input schema documents both uses; create forwards region into browsers.create, and list passes it as a list filter. A unit test asserts both pass-throughs against a mocked Kernel client.

Reviewed by Cursor Bugbot for commit 6f4fa57. Bugbot is set up for automated code reviews on this repo. Configure here.

@vercel

vercel Bot commented Aug 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mcp Ready Ready Preview Aug 14, 2026 1:48am

@masnwilliams masnwilliams left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

reviewed for maintainability, abstraction quality, boundary cleanliness, and structural complexity. no actionable findings.

@tnsardesai
tnsardesai merged commit c01afa0 into main Aug 14, 2026
10 checks passed
@tnsardesai
tnsardesai deleted the hypeship/mcp-browser-region branch August 14, 2026 15:55
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