Skip to content

chore: split protocol.yml - #40645

Merged
Pavel Feldman (pavelfeldman) merged 1 commit into
microsoft:mainfrom
pavelfeldman:split_yml
May 5, 2026
Merged

chore: split protocol.yml#40645
Pavel Feldman (pavelfeldman) merged 1 commit into
microsoft:mainfrom
pavelfeldman:split_yml

Conversation

@pavelfeldman

Copy link
Copy Markdown
Member

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

17 failed
❌ [chrome] › mcp/autowait.spec.ts:19 › racy navigation destroys context @mcp-windows-latest-chrome
❌ [firefox] › mcp/cli-devtools.spec.ts:217 › video-start-stop @mcp-windows-latest-firefox
❌ [firefox] › mcp/cli-devtools.spec.ts:231 › video-chapter @mcp-windows-latest-firefox
❌ [firefox] › mcp/cli-json.spec.ts:171 › close after open returns closed status @mcp-windows-latest-firefox
❌ [firefox] › mcp/cli-session.spec.ts:70 › delete-data @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:124 › daemon show: closing page exits the process @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:164 › should capture annotations via show --annotate @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:185 › should start dashboard and annotate when no dashboard is running @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:207 › should enter annotate mode on fresh dashboard.tsx mount with -s --annotate @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:231 › should annotate via direct browser_annotate MCP call @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:264 › should annotate when context has no fixed viewport @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:301 › should cancel browser_annotate when the MCP request is aborted @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:332 › should cancel browser_annotate when the MCP client disconnects @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:361 › should switch screencast to -s session on show --annotate @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:410 › should disengage annotate mode when --annotate client disconnects @mcp-windows-latest-firefox
❌ [firefox] › mcp/dashboard.spec.ts:499 › save recording streams WebM bytes to the chosen file @mcp-windows-latest-firefox
❌ [msedge] › mcp/cli-test.spec.ts:76 › debug test with custom fixture using browser.newContext @mcp-windows-latest-msedge

1 flaky⚠️ [chrome] › mcp/cdp.spec.ts:24 › cdp server `@mcp-windows-latest-chrome`

6925 passed, 1052 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

2 flaky⚠️ [chromium-library] › library/video.spec.ts:647 › screencast › should capture full viewport `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:719 › screencast › should work with video+trace `@chromium-ubuntu-22.04-node22`

41639 passed, 851 skipped


Merge workflow run.

Yury Semikhatsky (yury-s) added a commit to yury-s/playwright that referenced this pull request May 8, 2026
Follow-up to the revert of PR microsoft#40184 that brings the tree to a working
state on top of the main-branch changes that landed after the PR:
- Re-add Electron protocol entries to packages/protocol/spec/electron.yml
and the Electron initializer to packages/protocol/spec/playwright.yml
(protocol.yml was split into per-interface spec/*.yml files in microsoft#40645).
- Keep Electron docs in docs/src/electron-api/ but restore their pre-PR
full content (the PR had truncated docs to match the trimmed-down
CDP-only API). doclint cli.js merges electron-api docs.
- Drop the leftover docs/src/electron/loader bundling step from
utils/build/build.js (loader is back at server/electron/loader.ts).
- Update the restored server/electron/electron.ts import after the
toConsoleMessageLocation -> stackTraceToLocation rename in microsoft#39767.
- Drop the explicit _electron registration from inprocess.ts; with the
server-side dispatcher restored, _electron is wired through the
Playwright initializer again.
- Remove duplicate _electron exports in index.mjs and overrides.d.ts.
- Restore the pre-PR ElectronApplication evaluate/evaluateHandle
override signatures so the generated types are consistent.
Also re-applies the timeout option from microsoft#40613 (electronApp.close({
timeout })) on top of the restored server-side architecture: a new
ElectronApplication.close protocol command takes an optional timeout
and force-kills the underlying browser process via
browserProcess.kill() when graceful close exceeds the deadline.
Yury Semikhatsky (yury-s) added a commit to yury-s/playwright that referenced this pull request May 8, 2026
Follow-up to the revert of PR microsoft#40184 that brings the tree to a working
state on top of the main-branch changes that landed after the PR:
- Re-add Electron protocol entries to packages/protocol/spec/electron.yml
and the Electron initializer to packages/protocol/spec/playwright.yml
(protocol.yml was split into per-interface spec/*.yml files in microsoft#40645).
- Keep Electron docs in docs/src/electron-api/ but restore their pre-PR
full content (the PR had truncated docs to match the trimmed-down
CDP-only API). doclint cli.js merges electron-api docs.
- Drop the leftover docs/src/electron/loader bundling step from
utils/build/build.js (loader is back at server/electron/loader.ts).
- Update the restored server/electron/electron.ts import after the
toConsoleMessageLocation -> stackTraceToLocation rename in microsoft#39767.
- Drop the explicit _electron registration from inprocess.ts; with the
server-side dispatcher restored, _electron is wired through the
Playwright initializer again.
- Remove duplicate _electron exports in index.mjs and overrides.d.ts.
- Restore the pre-PR ElectronApplication evaluate/evaluateHandle
override signatures so the generated types are consistent.
Yury Semikhatsky (yury-s) added a commit to yury-s/playwright that referenced this pull request May 8, 2026
Follow-up to the revert of PR microsoft#40184 that brings the tree to a working
state on top of the main-branch changes that landed after the PR:
- Re-add Electron protocol entries to packages/protocol/spec/electron.yml
and the Electron initializer to packages/protocol/spec/playwright.yml
(protocol.yml was split into per-interface spec/*.yml files in microsoft#40645).
- Keep Electron docs in docs/src/electron-api/ but restore their pre-PR
full content (the PR had truncated docs to match the trimmed-down
CDP-only API). doclint cli.js merges electron-api docs.
- Drop the leftover docs/src/electron/loader bundling step from
utils/build/build.js (loader is back at server/electron/loader.ts).
- Update the restored server/electron/electron.ts import after the
toConsoleMessageLocation -> stackTraceToLocation rename in microsoft#39767.
- Drop the explicit _electron registration from inprocess.ts; with the
server-side dispatcher restored, _electron is wired through the
Playwright initializer again.
- Remove duplicate _electron exports in index.mjs and overrides.d.ts.
- Restore the pre-PR ElectronApplication evaluate/evaluateHandle
override signatures so the generated types are consistent.
Sign up for freeto 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.

2 participants

@pavelfeldman@yury-s