feat: record tunneled MCP resource identifiers from the dashboard - #5906
feat: record tunneled MCP resource identifiers from the dashboard#5906daviddanialy wants to merge 3 commits into
Conversation
🦋 Changeset detectedLatest commit: 5646559 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
All reported issues were addressed across 6 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
c22d227 to
4bc3042
Compare
108992e to
17adc3b
Compare
4bc3042 to
bc79380
Compare
This comment has been minimized.
This comment has been minimized.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015gm43HhrAUR6poQA5xKK9W
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015gm43HhrAUR6poQA5xKK9W
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015gm43HhrAUR6poQA5xKK9W
bc79380 to
5646559
Compare
|
Superseded by 20df268 on #5905 — the dashboard work was folded into the server PR after a review pass reshaped it: the settings section is now a shared EditableFieldSection with per-section mutations, and the project-scoped RequireScope gate from 5646559 was dropped as ineffective (the check carries no projectId dimension, and it falsely denies server-scoped grants). The ui changeset rides along on #5905. |
AIM-151. Stacked on #5905 (server).
Summary
Adds a Resource Identifier section to the tunneled MCP source Settings tab, following the existing Display Name section pattern (shared update mutation, same invalidation,
mcp:writegate). The field records the server's RFC 9728 protected resource identifier; saving an empty value clears it. Includes the regenerated TypeScript SDK for the newresource_identifierfield on the tunneled MCP create/update forms and server view.Motivation
The server side (#5905) routes tunneled per-user credentials by this recorded identifier; this is the surface where an operator sets it. The copy states the safety contract — Gram never connects to the address — and that blank is correct for servers without their own OAuth.
🤖 Generated with Claude Code
https://claude.ai/code/session_015gm43HhrAUR6poQA5xKK9W
Summary by cubic
Adds a Resource Identifier section to the tunneled MCP source Settings tab so operators can record the server's RFC 9728 protected resource identifier, which routes user credentials to the server by exact match. Saving an empty value clears it.
Bug Fixes
mcp:writegate is scoped to the source's project, so a grant held on another project can't enable Save.Written for commit 5646559. Summary will update on new commits.