Skip to content

chore: retire xmcp runtime surface - #5994

Open
disintegrator wants to merge 1 commit into
mainfrom
drop-xmcp
Open

chore: retire xmcp runtime surface#5994
disintegrator wants to merge 1 commit into
mainfrom
drop-xmcp

Conversation

@disintegrator

@disintegrator disintegrator commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Linear: AIM-10

Summary

  • delete the experimental server/internal/xmcp package and remove its server registration
  • retire the /x/mcp runtime, OAuth, callback, install, and well-known routes without redirects
  • make /mcp/{slug} the sole hosted surface in route telemetry, security checks, OAuth state, generated API descriptions, and remote-session registration
  • reject cached endpoint and remote-login state that references the retired route

Motivation

/mcp/{slug} now serves every MCP backend through mcp_endpoints, so the experimental parallel runtime is redundant. Removing it also eliminates the legacy toolset fallback exposed through its OAuth handlers and prevents new clients or cached flows from continuing on the retired surface.


Summary by cubic

Retires the experimental /x/mcp runtime surface for AIM-10, making /mcp/{slug} the sole hosted MCP path. The old runtime, OAuth, callback, install, and well-known routes now return 404 without redirects.

  • Deletes the server/internal/xmcp package and removes its server registration.
  • Updates route telemetry, security checks, OAuth state, generated API descriptions, and remote-session registration to use the canonical path.
  • Rejects cached endpoint and remote-login state that references the retired route.

Written for commit 53e3899. Summary will update on new commits.

Review in cubic

@disintegrator
disintegrator requested a review from a team as a code owner September 2, 2026 21:57
@changeset-bot

changeset-bot Bot commented Sep 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 53e3899

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@disintegrator disintegrator added go Pull requests that update go code linear labels Sep 2, 2026
@cubic-dev-ai

cubic-dev-ai Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Running ultrareview automatically — This cross-cutting 62-file refactor retires the /x/mcp runtime surface and touches OAuth, consent, token, and authn-challenge flows plus cached-state validation; a missed route reference or stale-state edge case could break MCP authentication and hosted serving.. I'll post findings when complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ultrareview completed in 9m 47s

All reported issues were addressed

Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread server/internal/mcp/resolved_mcp_endpoint.go Outdated
Comment thread server/internal/mcp/mcpmetrics/metrics.go Outdated
Comment thread server/internal/oauth/wellknown/wellknown.go Outdated
Comment thread server/internal/mcp/authnchallenge.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update go code linear

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants