Comfy-Org/comfy-cloud-mcp
1
0
Fork 0
mirror of https://github.com/Comfy-Org/comfy-cloud-mcp.git synced 2026-11-04 23:37:25 +00:00
Code Issues 2 Projects Packages Wiki Activity

[PR #2] [MERGED] fix: Claude Desktop requires OAuth, fix Amp config, add slash commands #6

New issue
Closed
opened 2026-10-26 22:27:47 +00:00 by bsakaguchi · 0 comments
bsakaguchi commented 2026-10-26 22:27:47 +00:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-cloud-mcp/pull/2
Author: @mattmillerai
Created: 3/30/2026
Status: ✅ Merged
Merged: 3/30/2026
Merged by: @mattmillerai

Base: main ← Head: fix/stdio-transport

📝 Commits (5)

  • 8778756 fix: switch all clients to stdio transport via npx
  • d3dd498 fix: guide Claude Desktop users to Connectors UI, keep remote HTTP
  • 930367a fix: skip Claude Desktop chat mode (needs OAuth), point to Code mode
  • 23f8c37 fix: exit early if only Claude Desktop detected (needs OAuth)
  • cd2f78d fix: simplify Claude Desktop-only message

📊 Changes

1 file changed (+55 additions, -58 deletions)

View changed files

📝 install.sh (+55 -58)

📄 Description

Summary

Fixes installer issues with Claude Desktop and Amp, adds slash command support for Claude Desktop users.

Changes

  • Claude Desktop (chat mode): Requires OAuth for remote MCP servers, which isn't supported yet. If it's the only client detected, the installer now exits with a clear message pointing users to install Claude Code.
  • Amp: Fixed broken config command — now uses --transport http like Claude Code.
  • Slash commands: Now installed for Claude Desktop users too — Code mode within the Desktop app reads from ~/.claude/commands/.
  • Claude Code / Cursor: Unchanged — remote HTTP.

Test plan

  • Run installer with only Claude Desktop detected — verify "install Claude Code" message and clean exit
  • Run installer with Claude Code detected — verify HTTP config is written
  • Run installer with Claude Code + Claude Desktop — verify slash commands are offered
  • Run installer with Cursor — verify HTTP config is written
  • Run installer with Amp — verify --transport http config

🤖 Generated with Claude Code

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfy-cloud-mcp/pull/2 **Author:** [@mattmillerai](https://github.com/mattmillerai) **Created:** 3/30/2026 **Status:** ✅ Merged **Merged:** 3/30/2026 **Merged by:** [@mattmillerai](https://github.com/mattmillerai) **Base:** `main` ← **Head:** `fix/stdio-transport` --- ### 📝 Commits (5) - [`8778756`](https://github.com/Comfy-Org/comfy-cloud-mcp/commit/8778756c9dc0ffd2895385b06f963c5213e80e8f) fix: switch all clients to stdio transport via npx - [`d3dd498`](https://github.com/Comfy-Org/comfy-cloud-mcp/commit/d3dd498fa3b3c97098a1e258625cfa5921bd6342) fix: guide Claude Desktop users to Connectors UI, keep remote HTTP - [`930367a`](https://github.com/Comfy-Org/comfy-cloud-mcp/commit/930367a95b89ccffb0de3086e658cd6b5b13ead0) fix: skip Claude Desktop chat mode (needs OAuth), point to Code mode - [`23f8c37`](https://github.com/Comfy-Org/comfy-cloud-mcp/commit/23f8c376c12d9f9a27dd02c4767ba4e5b4debb86) fix: exit early if only Claude Desktop detected (needs OAuth) - [`cd2f78d`](https://github.com/Comfy-Org/comfy-cloud-mcp/commit/cd2f78ddbf27d0a4680d83957ee23cd02c3e59f1) fix: simplify Claude Desktop-only message ### 📊 Changes **1 file changed** (+55 additions, -58 deletions) <details> <summary>View changed files</summary> 📝 `install.sh` (+55 -58) </details> ### 📄 Description ## Summary Fixes installer issues with Claude Desktop and Amp, adds slash command support for Claude Desktop users. ## Changes - **Claude Desktop (chat mode)**: Requires OAuth for remote MCP servers, which isn't supported yet. If it's the only client detected, the installer now exits with a clear message pointing users to install Claude Code. - **Amp**: Fixed broken config command — now uses `--transport http` like Claude Code. - **Slash commands**: Now installed for Claude Desktop users too — Code mode within the Desktop app reads from `~/.claude/commands/`. - **Claude Code / Cursor**: Unchanged — remote HTTP. ## Test plan - [ ] Run installer with only Claude Desktop detected — verify "install Claude Code" message and clean exit - [ ] Run installer with Claude Code detected — verify HTTP config is written - [ ] Run installer with Claude Code + Claude Desktop — verify slash commands are offered - [ ] Run installer with Cursor — verify HTTP config is written - [ ] Run installer with Amp — verify `--transport http` config 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-26 22:27:47 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
add-cla-workflow
jojodecayz-patch-1
chore/onboard-unreviewed-merge-detector
matt/be-1228-readme-purple-logo
chore/skills-from-comfy-skills-repo
mattmiller/oauth-default-claude-code-desktop
update-readme-beta
sync/skills-signed-urls-and-media-generation
No results found.
Labels
Clear labels   
pull-request
Mirrored from GitHub Pull Request

No labels
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Comfy-Org/comfy-cloud-mcp#6
No description provided.