Remove the embedded Environment Browser - #67
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
playwright-clidetection and version-matched Agent Skill materialization without downloading or starting a browser.Why
The current Environment Browser combines two different product responsibilities: previewing services inside the Sandbox and sharing an interactive browser between a human and an agent. That coupling also pulls the Playwright Dashboard, headed-browser/VNC takeover, browser ownership, and profile recovery into one lifecycle. Removing this path establishes a clean boundary before Preview and Browser return as independent tabs.
Impact
sandpi-browser-dashboardAppService./api/v1/environments/{environmentId}/browser...surface is removed.Validation
npm ci --ignore-scriptsnpm run typechecknpm test(716 passed)npm run lintnpm run openapi:check(121 operations)npm run test:e2e -- --list(56 tests discovered)npm run build