Browser: preparation for tools

[Kyle Cutler (kycutler)]

No description provided.

[Copilot]

Pull request overview

This PR prepares the integrated browser for “tools” by introducing a shared-process Playwright automation service, wiring browser views into page tracking (“shared with agent”), and tightening CDP/session cleanup.

Changes:

• Expanded IPlaywrightService from a TODO stub into a concrete automation + page-tracking API (summary, screenshots, dialog/file chooser handling, arbitrary function invocation).
• Added “tracked/shared with agent” state to BrowserViewModel, backed by Playwright page tracking and an update event.
• Improved lifecycle handling in CDP/browser-view group code (session disposal on target removal; context bookkeeping on view removal), and updated Playwright dependency.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/vs/workbench/contrib/browserView/common/browserView.ts	Adds sharedWithAgent state + event, and routes share/unshare via IPlaywrightService.
src/vs/platform/browserView/node/playwrightTab.ts	New Playwright page wrapper for summaries, logs, and dialog/file chooser state.
src/vs/platform/browserView/node/playwrightService.ts	Implements shared-process Playwright service, tracking, lazy init, page manager, screenshots, summaries, and function invocation.
src/vs/platform/browserView/electron-main/browserViewGroup.ts	Updates context bookkeeping when views are removed from a group.
src/vs/platform/browserView/common/playwrightService.ts	Defines the new IPlaywrightService API contract (tracking + automation).
src/vs/platform/browserView/common/cdp/proxy.ts	Disposes sessions when a target is unregistered to avoid stale sessions.
package.json	Pins playwright-core to an alpha build.
package-lock.json	Lockfile updates to reflect the Playwright dependency change.

[Joaquín Ruales (jruales)]

Overview of changes in the first commit (828c530) of this PR

graph TD
    subgraph Workbench
        BVM["BrowserViewModel<br/>sharedWithAgent toggle"]
    end

    subgraph Shared Process
        PS["PlaywrightService<br/>lazy init + API facade"]
        PM["PlaywrightPageManager<br/>tracking + FIFO matching"]
        PT["PlaywrightTab<br/>dialog/log/summary per page"]
        PW["Playwright-core<br/>(browser automation)"]
    end

    subgraph Electron Main
        BVG["BrowserViewGroup<br/>context bookkeeping fix"]
        CDP["CDPBrowserProxy<br/>session cleanup fix"]
    end

    subgraph Browser
        Pages["Browser Pages"]
    end

    BVM -->|"start/stopTrackingPage"| PS
    PS -->|"delegates tracking"| PM
    PS -->|"lazy init: createGroup + connectOverCDP"| BVG
    PM -->|"FIFO pairs viewId ↔ Playwright page"| PW
    PW -->|"controls"| Pages
    PM -->|"wraps each page"| PT
    BVG -->|"CDP WebSocket"| CDP
    CDP -->|"session lifecycle"| Pages
    PS -.->|"onDidChangeTrackedPages"| BVM

Loading