Skip to content

feat: record tunneled MCP resource identifiers from the dashboard - #5906

Open
daviddanialy wants to merge 3 commits into
daviddanialy/aim-151-serverfrom
daviddanialy/aim-151-dashboard
Open

feat: record tunneled MCP resource identifiers from the dashboard#5906
daviddanialy wants to merge 3 commits into
daviddanialy/aim-151-serverfrom
daviddanialy/aim-151-dashboard

Conversation

@daviddanialy

@daviddanialy daviddanialy commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

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:write gate). The field records the server's RFC 9728 protected resource identifier; saving an empty value clears it. Includes the regenerated TypeScript SDK for the new resource_identifier field 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

  • Since Settings sections share one update mutation, a failed save surfaced its error under whichever section the user did not touch; each section now keeps its own error and draft state.
  • The resource identifier's mcp:write gate 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.

Review in cubic

@daviddanialy
daviddanialy requested review from a team as code owners August 31, 2026 17:33
@linear-code

linear-code Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

AIM-151

@changeset-bot

changeset-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5646559

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
dashboard Minor

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

@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.

All reported issues were addressed across 6 files

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

Fix all with cubic | Re-trigger cubic

Comment thread client/dashboard/src/pages/sources/tunneled-mcp/TunneledMCPDetails.tsx Outdated
@daviddanialy
daviddanialy force-pushed the daviddanialy/aim-151-dashboard branch 3 times, most recently from c22d227 to 4bc3042 Compare August 31, 2026 18:57
@daviddanialy
daviddanialy force-pushed the daviddanialy/aim-151-server branch from 108992e to 17adc3b Compare August 31, 2026 19:26
@daviddanialy
daviddanialy force-pushed the daviddanialy/aim-151-dashboard branch from 4bc3042 to bc79380 Compare August 31, 2026 19:28
@blacksmith-sh

This comment has been minimized.

daviddanialy and others added 3 commits August 31, 2026 12:39
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
@daviddanialy
daviddanialy force-pushed the daviddanialy/aim-151-dashboard branch from bc79380 to 5646559 Compare August 31, 2026 19:39
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