fix: skip missing agent CLIs during uploads install - #723
Conversation
uploads install mcp used to fail the whole step when claude was not on PATH, which blocked MCP setup for Codex and Grok users. Register the hosted server with each CLI independently, and skip any that are not installed.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
uploads-auth | 795d6a4 | Commit Preview URL Branch Preview URL |
Aug 20 2026, 11:05 PM |
🦋 Changeset detectedLatest commit: 795d6a4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (2)
🚫 Excluded labels (none allowed) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
In plain terms
uploads installused to treat a missing Claude Code CLI as a hard failure, which blocked MCP setup even when Codex or Grok were available. It now registers the hosted MCP server with each of those CLIs independently, and skips any that are not on PATH.What it does / what it is not
mcp: skipped), not failed.uploads install(skills, hooks) still completes with exit 0.already configured. A realmcp adderror still fails that client, but the others still run.uploads login; Codex uses OAuth on first use, matching the plugin.mcp addCLI, so it is unchanged (hooks only).How to try it
On a machine without
claude, you should see that client skipped and Codex/Grok still registered if they are installed. With none of them on PATH, MCP reports a skip instead ofmcp: failed.Technical notes
--jsonnow reports per-client steps (mcp:claude,mcp:codex,mcp:grok) instead of a singlemcpkey.Test plan
pnpm --filter @buildinternet/uploads test test/install.test.ts(24 tests)pnpm --filter @buildinternet/uploads test test/commands-update.test.tspnpm --filter @buildinternet/uploads typecheck