Skip to content

Resolve resident browser proxy CDP - #2463

Open
miguelg719 wants to merge 5 commits into
feat/stagehand-init-first-rpcfrom
feat/resident-transport
Open

Resolve resident browser proxy CDP#2463
miguelg719 wants to merge 5 commits into
feat/stagehand-init-first-rpcfrom
feat/resident-transport

Conversation

@miguelg719

@miguelg719 miguelg719 commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • resolve Chromium through a /json/version endpoint
  • validate the configured proxy origin and returned loopback browser WebSocket URL
  • rewrite only scheme and authority while preserving the exact browser target path and query
  • use a bounded fetch timeout and the existing raw browser WebSocket CDP transport

Scope

Stack 2/6, based on the init-first protocol PR. This PR adds a separately testable transport resolver only; worker startup wiring lands later in the stack. Public builds remain unconfigured, with no compatibility negotiation, fallback, or public-connect behavior.

Validation

  • success, malformed JSON, missing/invalid URL, HTTP failure, and timeout tests
  • default-port, secure-WebSocket, escaped-path, and repeated-query preservation
  • complete server suite on this branch
  • complete TypeScript/package/rule suite at the stack tip

@changeset-bot

changeset-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: d59ecef

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread packages/server/service-worker-lifecycle/resident-browser-proxy.ts
Comment thread packages/server/service-worker-lifecycle/resident-browser-proxy.ts Outdated
Comment thread packages/server/tests/resident-browser-proxy.test.ts
@miguelg719
miguelg719 changed the base branch from feat/browserbase-resident-opt-in to feat/stagehand-init-first-rpc July 27, 2026 23:07
@miguelg719
miguelg719 force-pushed the feat/resident-transport branch from ea6badf to f70bdfd Compare July 27, 2026 23:08
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