chore(mcp): remove support for extension protocol v1#41857
Conversation
This comment has been minimized.
This comment has been minimized.
All clients use protocol v2 by now. Remove the v1 handlers on both the relay and the extension side and collapse the abstractions that only existed to support two protocol versions: - Relay: drop ExtensionProtocolV1 and the ExtensionProtocolHandler interface, always use ExtensionProtocolV2. - Extension: fold the v2 protocol handler into RelayConnection, drop the eager (v1) pending connection path and the protocolVersion plumbing. The connect page now rejects clients requesting protocol version < 2.
Also drop the dead error path after connectionRequested — the background records the relay URL synchronously and always succeeds.
575db11 to
4ca6a9b
Compare
Test results for "MCP"4 failed 7756 passed, 1249 skipped Merge workflow run. |
|
Hi, I'm the Playwright bot and I took a first look at the failing CI. 🟢 The failures are all pre-existing flakes — this PR looks clearThe 4 failures are in DetailsThis PR removes extension protocol v1 (relay Pre-existing flake / infra
All four spread the failures across browsers/OSes and PRs the change can't be responsible for, and none exercise the removed v1 relay/protocol path. I'm a first pass, but the signal here is clean. Triaged by the Playwright bot - agent run |
All clients use protocol v2 by now. Remove the v1 handlers on both the relay and the extension side and collapse the abstractions that only existed to support two protocol versions: