Skip to content

[STG-1927] Add local MCP model gateway TDD coverage#181

Draft
monadoid wants to merge 2 commits into
mainfrom
stg-1927
Draft

[STG-1927] Add local MCP model gateway TDD coverage#181
monadoid wants to merge 2 commits into
mainfrom
stg-1927

Conversation

@monadoid
Copy link
Copy Markdown

@monadoid monadoid commented May 7, 2026

Summary

Adds failing TDD coverage for local Browserbase MCP model-gateway behavior.

The tests describe the expected behavior before implementation:

  • config accepts provider/model modelName without modelApiKey
  • config resolution preserves an omitted model key instead of injecting a dummy provider key
  • Stagehand construction uses Browserbase model gateway when only a Browserbase API key and model name are provided
  • explicit provider modelApiKey remains the highest-priority BYOK path

Validation

  • pnpm test runs and fails on the new intended assertions:
    • schema still requires modelApiKey for custom models
    • config still injects dummy-api-key
    • Stagehand construction still passes the raw model string instead of gateway-backed model config
  • pnpm install and commit hooks completed; unrelated lockfile formatting from install was reverted.

@monadoid monadoid marked this pull request as ready for review May 7, 2026 18:43
@monadoid monadoid changed the title [codex] Add local MCP model gateway TDD coverage [STG-1927] Add local MCP model gateway TDD coverage May 7, 2026
@monadoid monadoid marked this pull request as draft May 11, 2026 17: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.

1 participant