fix: update outdated Claude Code CLI install instructions#1661
fix: update outdated Claude Code CLI install instructions#1661withsivram wants to merge 1 commit intoeyaltoledano:mainfrom
Conversation
…no#1634) Replace `npm install -g @anthropic-ai/claude-code` with the current native binary install method `curl -fsSL https://claude.ai/install.sh | sh` in warning/error messages and documentation. Affected files: - src/ai-providers/claude-code.js: warn message when CLI not detected - packages/tm-core/src/modules/loop/services/loop.service.ts: ENOENT error message - docs/examples/claude-code-usage.md: requirements section Fixes eyaltoledano#1634 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughUpdated installation instructions for Claude Code CLI across documentation and error messages. Replaced Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Pull request overview
Updates user-facing Claude Code CLI installation guidance to reflect the current primary install method rather than the deprecated npm install -g @anthropic-ai/claude-code, aligning runtime messages and docs with the latest distribution approach.
Changes:
- Updated Claude Code provider “CLI not detected” warning to show the new install command.
- Updated Loop service ENOENT error message to show the new install command.
- Updated the Claude Code usage docs requirements section to show the new install command and renumbered steps.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/ai-providers/claude-code.js | Updates the warning shown when the Claude Code CLI is not found. |
| packages/tm-core/src/modules/loop/services/loop.service.ts | Updates the ENOENT error message when the CLI executable is missing. |
| docs/examples/claude-code-usage.md | Updates installation instructions in the example documentation requirements section. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 1. Claude Code CLI must be installed and authenticated on your system: | ||
| ```bash | ||
| npm install @anthropic-ai/claude-code | ||
| curl -fsSL https://claude.ai/install.sh | sh | ||
| ``` |
There was a problem hiding this comment.
The Requirements section now hard-codes a macOS/Linux-style curl ... | sh install command. Since installation is OS-dependent (and many users won’t have curl/sh available, e.g., Windows), consider changing this to a link to the official Claude Code installation docs and optionally listing the install script as one platform-specific option instead of the only instruction.
| log( | ||
| 'warn', | ||
| 'Claude Code CLI not detected. Install it with: npm install -g @anthropic-ai/claude-code' | ||
| 'Claude Code CLI not detected. Install it with: curl -fsSL https://claude.ai/install.sh | sh' |
There was a problem hiding this comment.
The warning message now tells users to run curl ... | sh, which is (1) not cross-platform (won’t work on Windows by default) and (2) a risky pattern to recommend in runtime logs. Consider pointing to the official Claude Code install docs URL (and optionally mentioning the install script as a macOS/Linux example) instead of embedding a pipe-to-shell command directly in the warning.
| 'Claude Code CLI not detected. Install it with: curl -fsSL https://claude.ai/install.sh | sh' | |
| 'Claude Code CLI not detected. Install it by following the instructions at https://docs.anthropic.com/claude-code/install (for macOS/Linux, you can use the install script described there).' |
| return sandbox | ||
| ? 'Docker is not installed. Install Docker Desktop to use --sandbox mode.' | ||
| : 'Claude CLI is not installed. Install with: npm install -g @anthropic-ai/claude-code'; | ||
| : 'Claude CLI is not installed. Install with: curl -fsSL https://claude.ai/install.sh | sh'; |
There was a problem hiding this comment.
This error message embeds a curl ... | sh install command, which is not cross-platform and is a risky instruction to show in errors. Also, the wording says "Claude CLI" while other parts of the codebase refer to "Claude Code CLI"; using the more specific name would avoid confusion with other Claude-related CLIs. Prefer linking to the official install docs (and optionally include platform-specific install hints).
| : 'Claude CLI is not installed. Install with: curl -fsSL https://claude.ai/install.sh | sh'; | |
| : 'Claude Code CLI is not installed. See https://docs.anthropic.com/claude/docs/claude-code-cli for installation instructions.'; |
Summary
npm install -g @anthropic-ai/claude-codewith the current official installer:curl -fsSL https://claude.ai/install.sh | shsrc/ai-providers/claude-code.js— warning message when CLI not detectedpackages/tm-core/src/modules/loop/services/loop.service.ts— ENOENT error messagedocs/examples/claude-code-usage.md— requirements/prerequisites sectionTest plan
Fixes #1634
🤖 Generated with Claude Code
Summary by CodeRabbit