docs(cloud): Comfy Cloud MCP setup guide — Connectors deep link, OAuth, retire installer#1176
Open
mattmillerai wants to merge 12 commits into
Open
docs(cloud): Comfy Cloud MCP setup guide — Connectors deep link, OAuth, retire installer#1176mattmillerai wants to merge 12 commits into
mattmillerai wants to merge 12 commits into
Conversation
New cloud/mcp.mdx documenting how to connect Claude Code and Claude Desktop to Comfy Cloud via the hosted MCP server (OAuth), including the install one-liners, the Claude Desktop custom-connector walkthrough (Customize -> Connectors -> + -> Add custom connector), slash commands, and a headless/CI API-key fallback. Adds the page to the Cloud Exclusive nav group. Claude Desktop screenshots are placeholders; images/cloud/mcp/README.md lists the exact files to drop in.
Adds auto-generated labeled placeholder PNGs for the four Claude Desktop steps so cloud/mcp.mdx renders and `mint broken-links` passes. Replace each in place with a real capture (same filenames); see images/cloud/mcp/README.md.
- Replace the four auto-generated placeholders with real captures: Connectors, Add custom connector, the connector modal (name + URL), and the OAuth sign-in. - Add a CardGroup after Install linking to the Claude Code / Claude Desktop sections. - Drop the now-obsolete placeholder README and the "screenshots pending" note.
Restructure the MCP page around per-client installs: Claude Code now leads with the explicit `claude mcp add` one-liner (no longer assumes the installer wired it), Claude Desktop keeps its connector walkthrough, and the curl/irm installer moves to an optional "Slash commands" helper. Adds a client picker up top.
Remove contrast-with-the-old-local-installer phrasing (no API key to copy/paste, nothing to run locally, no Node.js, the 'run anything locally?' FAQ) that only makes sense to returning users, not first-time readers.
…ntions - Promote a single Install (H2) with Claude Code, Claude Desktop, Slash commands, and Headless/CI as sub-sections (H3), so the on-page TOC nests them. - Remove the named Cursor/Amp references (kept a neutral 'more clients coming'), since those clients land later.
Inline Icon on the Claude Code (terminal) and Claude Desktop (monitor) section headers, matching the Pick-your-client cards. Anchors (#claude-code, #claude-desktop) are preserved, so the card jump-links still work.
Lead the Claude Code section with the plugin install (/plugin marketplace add Comfy-Org/comfy-skills + /plugin install comfy-cloud@comfy-skills) — one step for the MCP connection and the namespaced slash commands. Drop the deprecated curl-installer section; keep a manual 'claude mcp add' as a fallback. Standardize the server name on comfy-cloud and add an FAQ noting Desktop has no slash commands.
Adds a pre-filled "Add custom connector" link (connectorName + connectorUrl) to the OAuth sign-in section so Claude web/Desktop users can add cloud.comfy.org/mcp without editing a config file. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…anual setup The comfy-cloud-mcp thin-client repo (install.sh / install.ps1) is being archived. Remove the one-click installer section from the MCP setup page in all three languages and repoint API-key users at manual setup. Add the Connectors deep link to ja/zh for parity with en. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (5)
📝 WalkthroughWalkthroughA new ChangesClaude MCP Connection Documentation
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
✨ Simplify code
Comment |
Member
|
@mattmillerai I think we have removed the mcp-server.mdx in your PR https://github.com/Comfy-Org/docs/pull/1094/changes |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Builds out the Comfy Cloud MCP setup page (
development/cloud/mcp-server.mdx) into a real "Connect to Claude" guide, and retires the deprecated installer.Highlights:
comfy-cloudplugin; each platform's install is self-contained.connectorName+connectorUrl), so it's click → Add → Connect (OAuth), no config file to edit.curl|bashinstaller.Comfy-Org/comfy-cloud-mcpwas archived (2026-06-22); the page no longer referencesinstall.sh/install.ps1and leads with Connectors + manual setup instead.Note for the reviewer
The ja/zh Connectors card text (incl. the Add custom connector dialog label —
カスタムコネクタを追加/添加自定义连接器) was written by me, not a native pass. A check on that phrasing against Claude's actual localized UI strings would be the most useful thing to look at.Separately flagged, not changed here: the Claude Desktop manual tab still documents hand-editing
claude_desktop_config.jsonwith aurl+headersblock. Current Claude Desktop routes remote MCP servers through Connectors, so that tab may be stale — worth a follow-up.