Repository navigation
feat(integrations): add Grok Build CLI harness - #2876
antonvishal wants to merge 22 commits into
Conversation
Resolve overview.mdx conflicts to include both Grok Build and fx entries. Keep the screenshot base64 budget from browserbase#2791 alongside the ACP facade launcher changes. Also pass --max-screenshot-base64-bytes=60000 to the facade spawned by the ACP launcher so ACP transports get the same frame-cap protection as fx, mark BROWSERBASE_PROJECT_ID optional in the Grok Build README per browserbase#2777, and update the affected ACP arg assertions.
|
|
This PR is from an external contributor and must be approved by a stagehand team member with write access before CI can run. |
# Conflicts: # packages/docs/docs.json # packages/docs/v4/integrations/overview.mdx
…2-pr-2876 # Conflicts: # turbo.json
Bring PR browserbase#2876 onto Miguel's facade/eval stack: native `--rules` policy, turn budget, session-loss, keyed observations, and reported usage/cost from the grok CLI stream. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The latest approval by @filip-michalsky could not refresh the mirrored PR automatically (push-failed). The external PR stays open, and the mirrored PR should be updated manually before work continues. |
|
@antonvishal CI is failing |
The short dependency-wait timeout can abort after fetch has already been sent. That late hit was counted in the next phase and failed TypeScript unit under CI load. Co-authored-by: Cursor <cursoragent@cursor.com>
…sertion" This reverts commit 215a91a.
|
@filip-michalsky the CI failure is not related to this PR. So created a separated PR to fix CI. |
93b9cba to
623416c
Compare
# Conflicts: # packages/evals/package.json # pnpm-lock.yaml
623416c to
58464db
Compare
|
The latest approval by @miguelg719 could not refresh the mirrored PR automatically (push-failed). The external PR stays open, and the mirrored PR should be updated manually before work continues. |
| packages/integrations/deepagents-sdk/dist/** | ||
| packages/integrations/fx-sdk/dist/** | ||
| packages/integrations/cursor-sdk/dist/** | ||
| packages/integrations/grok-build-sdk/dist/** |
There was a problem hiding this comment.
this blocks the mirror pr @filip-michalsky @antonvishal. closing this one, please reopen without touching .github workflows, we will include this in the mirror pr
What
Adds the
grok_buildeval harness using the native Grok Build CLI. Each run uses an isolated Grok home and trusted MCP configuration, with Stagehand browser tools and normalized tool, usage, and completion events.Partial tool updates remain pending until completion. Wrapped MCP calls retain their actual tool names, arguments, results, and browser evidence. Browser guidance starts with main-frame snapshots and checks every task constraint before the final answer.
Testing
evals run b:webvoyager --harness grok_build --tool stagehand_facade -m grok-build/grok-4.6 -l 1 -t 1 -e browserbaseruns and passes.Summary by cubic
Adds a Grok Build integration and eval harness via a new
@browserbasehq/stagehand-integrations-grok-build-sdksession adapter that drives the nativegrokCLI in headless streaming mode over MCP/stdio.grok_buildharness with thestagehand_facade,playwright_mcp, andchrome_devtools_mcptool surfaces.XAI_API_KEYis unset and writing the MCP mount into the isolated home's trusted config.--rulespolicy, turn budget, and session-loss handling, and reports usage and cost from the grok CLI stream.--always-approvefor unattended runs, configurable viaEVAL_GROK_BUILD_ALWAYS_APPROVE.end_turnstops from max-turn stops and process failures so only genuine failures surface as SDK errors.--max-screenshot-base64-bytes=60000to the facade spawned by the launcher and marksBROWSERBASE_PROJECT_IDoptional..grok/config.toml,AGENTS.md, docs, and unit tests for the CLI session, runner, and trajectory adapter.Written for commit 58464db. Summary will update on new commits.