Skip to content

feat(connect): ChatGPT recipe on the MCP page - #378

Merged
jfrench9 merged 1 commit into
mainfrom
feature/chatgpt-connect-recipe
Sep 8, 2026
Merged

feat(connect): ChatGPT recipe on the MCP page#378
jfrench9 merged 1 commit into
mainfrom
feature/chatgpt-connect-recipe

Conversation

@jfrench9

@jfrench9 jfrench9 commented Sep 8, 2026

Copy link
Copy Markdown
Member

Summary

The RoboSystems plugin is live in the ChatGPT plugin directory (v1.0.0, the SEC filings read surface). This adds ChatGPT to the Connect page alongside Claude, Claude Code and Cursor.

  • McpSnippets.tsx: a ChatGPT recipe in the shared sign-in set — the OAuth URL as a developer-mode custom connector (Settings → Connectors → Create), which serves every tool of the graph picked at consent, RoboLedger included; the directory listing is linked as the no-setup path.
  • Connect page subtitle and the universal-section doc comment name ChatGPT.

Renders on both the universal address and the workspace addresses, and on the repository getting-started page, since all three use the same component.

Test plan

  • vitest — connect page (21) + repository getting-started (3) pass
  • tsc --noEmit, eslint, prettier --check on the changed files
  • Visual check of /connect after deploy

🤖 Generated with Claude Code

https://claude.ai/code/session_01DND6a4d6FNAGveRxViZiQE

The RoboSystems plugin is live in the ChatGPT plugin directory (v1.0.0,
the SEC read surface). The Connect page now carries a ChatGPT recipe next
to Claude's: the OAuth URL as a developer-mode custom connector, which
serves every tool of the graph picked at consent, with the directory
listing linked as the no-setup path. Page subtitle names ChatGPT.

Claude-Session: https://claude.ai/code/session_01DND6a4d6FNAGveRxViZiQE
@github-actions github-actions Bot added the change:normal Normal change: automated review documented on the pull request before merge label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Change classification: normal — platform or application code changed.

A normal change carries a documented automated review on this pull request before merge, in place of a second human approver. The change-classification job requests that review and passes once it has been posted. Recorded by the change-classification job.

@jfrench9

jfrench9 commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

@claude please review this PR

@claude

claude Bot commented Sep 8, 2026

Copy link
Copy Markdown

Claude finished @jfrench9's task in 1m 53s —— View job


Review complete

  • Gather context (PR diff, changed files)
  • Review McpSnippets.tsx changes
  • Review connect/content.tsx changes
  • Post review feedback

This is a small, well-scoped addition — a fourth recipe in the shared McpSignInSnippets set, so it automatically renders everywhere that set is used (universal address, workspace address, repository getting-started page) without touching those call sites.

Looks good:

  • The ChatGPT snippet reuses the same OAuth URL as the Claude recipe (src/components/mcp/McpSnippets.tsx:62-81), consistent with the "one URL, pick the graph at consent" model the rest of the page establishes.
  • External link follows the existing convention (target="_blank", rel="noopener noreferrer", same text-primary-600 dark:text-primary-400 hover:underline classes used elsewhere in content.tsx and the repository getting-started page).
  • Correctly omitted from the API-key section (content.tsx:426-441) — ChatGPT's custom connector is sign-in only, so it wouldn't make sense next to the header-based Claude Code / Cursor key snippets.
  • Doc comment (McpSnippets.tsx:40) and subtitle (content.tsx:262) updated to match; the connect/__tests__/content.test.tsx suite asserts on specific substrings/testids rather than an exhaustive snippet count, so this addition doesn't conflict with any existing assertion.

Minor nit:

  • McpSnippets.tsx:56 and :63 — the Claude and ChatGPT snippets both use copyLabel="Connector URL", so their copy buttons both render aria-label="Copy Connector URL" in the same section. Sighted users have the heading above each to disambiguate, but a screen-reader user tabbing between the two copy buttons hears the same label twice. Consider "Claude connector URL" / "ChatGPT connector URL" for the two labels. Low severity — not a functional bug.
    Fix this →

Not verified:

  • Could not execute vitest, tsc, eslint, or prettier in this sandbox — the tool run required approval that isn't available here, so I reviewed statically against the diff and read through content.test.tsx by hand to confirm no assertions conflict with the new snippet. The PR description states these already pass locally.
  • The hardcoded ChatGPT plugin directory URL (McpSnippets.tsx:70) — assumed correct per the PR description's claim that the plugin is live; not independently verified.

No blocking issues found.

@jfrench9
jfrench9 merged commit d1e19e1 into main Sep 8, 2026
7 checks passed
@jfrench9
jfrench9 deleted the feature/chatgpt-connect-recipe branch September 8, 2026 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:normal Normal change: automated review documented on the pull request before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant