chore: add input for path, and use coder_env for model in claude-code module #617
+102
−38
Conversation
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
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the claude-code module to use environment variables for model configuration instead of CLI flags, and adds a new input variable for custom PATH configuration.
- Model configuration now uses
ANTHROPIC_MODELenvironment variable viacoder_envresource instead of--modelCLI flag - Added
claude_pathvariable to allow custom PATH configuration for the Claude Code binary - Removed ARG_MODEL parameter and related CLI flag handling from start script
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| registry/coder/modules/claude-code/main.tf | Added claude_path variable and anthropic_model coder_env resource; removed ARG_MODEL parameter from start script invocation; updated model variable description |
| registry/coder/modules/claude-code/scripts/start.sh | Removed ARG_MODEL variable declaration, debug print, and --model CLI flag handling |
| registry/coder/modules/claude-code/main.test.ts | Updated test to verify ANTHROPIC_MODEL environment variable is set via coder_env instead of checking for --model CLI flag |
| registry/coder/modules/claude-code/README.md | Updated all version references from 4.2.7 to 4.3.0 |
matifali
reviewed
Dec 19, 2025
Comment on lines
201
to
205
| variable "claude_path" { | ||
| type = string | ||
| description = "Path to prepend to PATH for Claude Code binary." | ||
| default = "$HOME/.local/bin" | ||
| } |
Member
There was a problem hiding this comment.
How are you ensuring this is the same path used in the install script? There is a chance these two can drift.
…on for this since official installer does not support versions or install path flags
…ring claude is in path
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.
Description
Adds a
claude_pathvariable to override the path that thecoder_envsets for claude-code.Model now uses
coder_envto set theANTHROPIC_MODELenv instead of using the--modelcli flag which did not set the anthropic model globally in the workspace.Type of Change
Module Information
Path:
registry/coder/modules/claude-codeNew version:
v4.3.0Breaking change: [ ] Yes [X] No
Testing & Validation
bun test)bun fmt)Related Issues