Skip to content

feat(claudecode): make CLI command configurable#418

Open
hushicai wants to merge 1 commit intochenhg5:mainfrom
hushicai:feature/claudecode-cmd-configurable
Open

feat(claudecode): make CLI command configurable#418
hushicai wants to merge 1 commit intochenhg5:mainfrom
hushicai:feature/claudecode-cmd-configurable

Conversation

@hushicai
Copy link
Copy Markdown
Contributor

@hushicai hushicai commented Apr 2, 2026

Background

Some users build custom Claude Code binaries with different executable names (e.g., claude-js from leaked source). The claudecode agent was hardcoding "claude" as the CLI command.

Change

Add cmd option in agent config to allow custom CLI binary names. Default is "claude" for backward compatibility.

Usage

[projects.agent]
type = "claudecode"

[projects.agent.options]
cmd = "claude-js"  # custom command name, defaults to "claude"

Background: Some users build custom Claude Code binaries with
different executable names (e.g., claude-js from leaked source).
The claudecode agent was hardcoding 'claude' as the CLI command.

Change: Add 'cmd' option in agent config to allow custom CLI
binary names. Default is 'claude' for backward compatibility.

Usage in config.toml:
  [projects.agent]
  type = "claudecode"
  [projects.agent.options]
  cmd = "claude-js"  # custom command name
Copy link
Copy Markdown
Owner

@chenhg5 chenhg5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Clean implementation for configurable CLI command.

✅ Feature correctness:

  • Adds cmd option to claudecode agent config
  • Defaults to "claude" for backward compatibility
  • Properly validates CLI exists in PATH

✅ Code quality: Minimal changes, follows existing patterns.

✅ Tests: CI passes.

Useful for users with custom Claude Code binaries.

@chumicat
Copy link
Copy Markdown

chumicat commented Apr 2, 2026

Seems PR duplicate with #373

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