Skip to content

Split ChatGPT and Codex onboarding; fix ChatGPT setup path - #803

Merged
kody-bot merged 2 commits into
mainfrom
cursor/chatgpt-onboarding-instructions-7897
Jul 22, 2026
Merged

kody-bot merged 2 commits into
mainfrom
cursor/chatgpt-onboarding-instructions-7897

Conversation

@kentcdodds

@kentcdodds kentcdodds commented Jul 20, 2026 •

Copy link
Copy Markdown
Owner

Summary

From Daniel Leal’s onboarding recording: ChatGPT setup instructions pointed people at the wrong create-app path and mixed Codex config.toml into the same tab. This splits ChatGPT and Codex into separate Get started tabs, updates ChatGPT to Settings → Plugins → Browse plugins → Create app, and adds a copyable site favicon URL (/apple-touch-icon.png).

The ChatGPT guidance now links OpenAI’s official developer-mode setup, identifies eligible paid plans and managed-workspace admin enablement, and explains that owners can edit a developer-mode app’s name and logo later from Manage in Apps settings. docs/use/connect-your-agent.md matches the route copy.

Test plan

  • Focused onboarding client unit test
  • npm run validate
  • Local /onboarding spot-check: corrected ChatGPT copy/link/cards and separate Codex TOML tab
  • CI Validate, Cursor Bugbot, and CodeRabbit checks

Walkthrough

Corrected ChatGPT onboarding guidance

System recap — composes existing primitives (low risk)

Mode: recap · Base: main @ 218e7684 · Head: 8d12791a

Classification: composes — onboarding copy, client-tab wiring, and documentation only; no primitive behavior or storage/API contracts change.

Primitives touched

Primitive Group Impact
app-ui surfaces composes — separate ChatGPT/Codex setup and accurate developer-mode guidance

System map

The onboarding browser surface directs users to the appropriate ChatGPT or Codex configuration path without changing MCP authentication or runtime behavior.

Legend: green = composes (wiring only) · amber = extended by this PR · red = new primitive · gray = context (unchanged, included only when an edge crosses it).

flowchart LR
  appUi["app-ui<br/>Browser app (Remix 3)"]:::touched
  openAi["OpenAI developer-mode guide"]:::untouched
  mcp["Kody /mcp endpoint"]:::untouched
  appUi -->|"eligibility and admin troubleshooting link"| openAi
  appUi -->|"ChatGPT app URL or Codex TOML"| mcp
  classDef touched fill:#1a7f37,color:#fff
  classDef extended fill:#9a6700,color:#fff
  classDef added fill:#cf222e,color:#fff
  classDef untouched fill:#57606a,color:#fff
Loading

Before / after

Before After
ChatGPT and Codex shared one tab Each client has its own setup tab
Missing Plugins UI suggested a reload Copy identifies eligible plans and managed-workspace admin enablement
Icon warning said it could never be changed Copy accurately says owners can edit developer-mode app name/logo from Manage
Open in Web Open in Cursor 

Summary by CodeRabbit

  • New Features
    • Updated onboarding to split the combined ChatGPT/Codex experience into separate “ChatGPT” and “Codex” tabs with tailored setup steps.
    • ChatGPT setup now includes the MCP server URL, a generated app icon link, and a developer-mode guide.
    • Codex setup now shows only the dedicated config.toml snippet entry.
  • Documentation
    • Refreshed “Connect your agent” wording to describe ChatGPT and Codex separately with clearer client notes.
  • Tests
    • Updated onboarding tests to validate the new tab list and the app icon/developer-mode URL outputs.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The onboarding MCP client catalog now separates ChatGPT and Codex tabs. ChatGPT renders developer-mode, MCP URL, and favicon setup content, while Codex renders TOML configuration instructions. Documentation and tests reflect the new client split.

Changes

MCP onboarding clients

Layer / File(s) Summary
Split client catalog
packages/worker/client/routes/onboarding-mcp-clients.ts
Replaces codex-chatgpt with separate chatgpt and codex kinds and tabs, and adds ChatGPT setup helpers.
Render client-specific setup
packages/worker/client/routes/onboarding-mcp-client-tabs.tsx
Adds ChatGPT developer-mode, MCP URL, and favicon cards while restricting the Codex branch to TOML configuration instructions.
Update coverage and instructions
packages/worker/client/routes/onboarding-mcp-clients.node.test.ts, docs/use/connect-your-agent.md
Updates tab and filtering assertions, tests the new helper outputs, and documents separate ChatGPT and Codex setup steps.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant OnboardingTabs
  participant IconUrlBuilder
  User->>OnboardingTabs: Select ChatGPT
  OnboardingTabs->>IconUrlBuilder: Build icon URL from MCP server URL
  IconUrlBuilder-->>OnboardingTabs: Return apple-touch-icon URL
  OnboardingTabs-->>User: Display MCP URL and favicon setup cards
Loading

Possibly related PRs

  • kentcdodds/kody#761: Earlier MCP onboarding client tab work that this change extends by splitting ChatGPT and Codex setup.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: separating ChatGPT and Codex onboarding and correcting the ChatGPT setup flow.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/chatgpt-onboarding-instructions-7897

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@kody-bot
kody-bot marked this pull request as ready for review July 20, 2026 17:39
Update Get started instructions from Daniel Leal feedback: separate
ChatGPT web from Codex config, point ChatGPT at Plugins → Create app,
and link the site favicon for the immutable plugin icon.

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor
cursor Bot force-pushed the cursor/chatgpt-onboarding-instructions-7897 branch from f1a05e6 to 18a7676 Compare July 20, 2026 19:15
@github-actions

github-actions Bot commented Jul 20, 2026 •

Copy link
Copy Markdown
Contributor

🔎 Preview deployed: https://kody-pr-803.kody-a99.workers.dev

Worker: kody-pr-803
D1: kody-pr-803-db
KV: kody-pr-803-oauth-kv

Mocks:

@kody-bot
kody-bot merged commit 17e3270 into main Jul 22, 2026
5 checks passed
@kody-bot
kody-bot deleted the cursor/chatgpt-onboarding-instructions-7897 branch July 22, 2026 22:06
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.

3 participants