Skip to content

Test the moved org's v2 MCP URL - #2283

Draft
RhysSullivan wants to merge 3 commits into
v2-moved-mcp-urlfrom
v2-moved-mcp-url-tests
Draft

RhysSullivan wants to merge 3 commits into
v2-moved-mcp-urlfrom
v2-moved-mcp-url-tests

Conversation

@RhysSullivan

@RhysSullivan RhysSullivan commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

Tests for the PR below.

  • e2e (cloud/v2-moved-notice.test.ts): the move answer's MCP URL is exactly v2's <origin>/mcp (not an org-pinned path). The notice shows that URL and "Connect your agents to v2 instead, and choose when you sign in:". The Connect card and /setup-mcp show the v2 command, "Choose when you sign in.", and the old v1 URL labelled. The long-name check includes the URL row. An MCP client's initialize result has no instructions before the move and the exact move instructions (URL and org to choose) after it, and the tool list stays the same.
  • e2e: an MCP client connected before the move gets HTTP 404 on its next request, both while its session is still in memory (after the re-read window, which e2e sets to 1s via MCP_V2_MOVED_RECHECK_MS) and after the session's runtime was released and restored. Its new session's initialize carries the notice, and that session keeps working past the re-read window.
  • e2e: at 320px wide, a 45-character unbroken v2 slug stays inside the notice and the Connect card.
  • Unit: the session DO ends a session whose client must re-initialize (also after a restore), keeps it while an execution is paused or a request is running, never ends it for a caller that doesn't own it, and reports a live initialize to the host. The decision rule: re-initialize only when the org has moved and the client got a different notice or none.
  • Unit: the v2McpUrl parsing, V2_MCP_ORIGIN validation, the shipped config, and the instructions text and read (unset var, failure, timeout).

@RhysSullivan
RhysSullivan added this pull request to stack #2284 October 12, 2026 22:17
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
executor-cloud 0f37340 Oct 12 2026, 11:29 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing 0f37340 Commit Preview URL

Branch Preview URL
Oct 12 2026, 11:28 PM

@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Cloudflare preview

Console https://executor-preview-pr-2283.executor-e2e.workers.dev
MCP https://executor-preview-pr-2283.executor-e2e.workers.dev/mcp
Deployed commit 0f37340

Sign-in is Cloudflare Access (one-time PIN to an allowed email). The preview has its own database and encryption key; it is destroyed when this PR closes.

@pkg-pr-new

pkg-pr-new Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@2283

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@2283

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@2283

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@2283

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@2283

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@2283

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@2283

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@2283

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@2283

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@2283

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@2283

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@2283

executor

npm i https://pkg.pr.new/executor@2283

commit: 0f37340

@RhysSullivan
RhysSullivan force-pushed the v2-moved-mcp-url-tests branch from 7f15d4c to 6fa259c Compare October 12, 2026 22:37
@RhysSullivan
RhysSullivan force-pushed the v2-moved-mcp-url-tests branch from 6fa259c to 0f37340 Compare October 12, 2026 23:26

This branch has not been deployed

No deployments
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