Skip to content

chore(devfile): add command to install claude#359

Merged
svor merged 1 commit intomainfrom
sv-add-install-claude-command
Apr 10, 2026
Merged

chore(devfile): add command to install claude#359
svor merged 1 commit intomainfrom
sv-add-install-claude-command

Conversation

@svor
Copy link
Copy Markdown
Collaborator

@svor svor commented Apr 10, 2026

What does this PR do?

  • Add anthropic.claude-code extension into the list of recommended
  • Add command to install claude code

What issues does this PR fix or reference?

eclipse-che/che#23806

Is it tested? How?

Summary by CodeRabbit

Release Notes

  • New Features

    • Added Claude Code extension support to the development environment.
  • Chores

    • Updated development tooling configuration to include Claude Code installation during environment setup.

Signed-off-by: Valerii Svydenko <vsvydenk@redhat.com>
@svor svor requested review from ibuziuk and tolusha as code owners April 10, 2026 10:02
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

📝 Walkthrough

Walkthrough

Configuration updates to integrate Claude Code support into the development environment. A new devfile command installs the Claude Code npm package globally during development, and the VS Code extension is added to the recommended extensions list.

Changes

Cohort / File(s) Summary
Claude Code Integration
.devfile.yaml, .vscode/extensions.json
Added global npm installation command for @anthropic-ai/claude-code in devfile and added anthropic.claude-code VS Code extension to recommendations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A whisker of code, so clever and keen,
Claude hops in to join the dev scene,
Extensions installed, commands all set,
The finest AI tooling rabbit could get! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: adding a command to install Claude Code to the devfile configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sv-add-install-claude-command

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
.devfile.yaml (1)

57-62: Pin Claude Code to a tested version to keep workspace setup reproducible.

Line 62 installs a mutable latest version (npm install -g @anthropic-ai/claude-code``). This can introduce unpredictable breakages across developer workspaces. The package is actively maintained (latest: 2.1.100, Node >=18.0.0), so pinning to a specific tested version ensures consistent setup across the team.

🔧 Suggested change
-      commandLine: "npm install -g `@anthropic-ai/claude-code`"
+      commandLine: "npm install -g `@anthropic-ai/claude-code`@2.1.100"

Alternatively, use an environment variable for flexibility:

-      commandLine: "npm install -g `@anthropic-ai/claude-code`"
+      commandLine: "npm install -g `@anthropic-ai/claude-code`@${CLAUDE_CODE_VERSION:-2.1.100}"
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.devfile.yaml around lines 57 - 62, The install step with id
"install-claude-code" currently runs "commandLine: npm install -g
`@anthropic-ai/claude-code`" which installs a moving "latest" release; change this
to pin a tested version (e.g., "npm install -g
`@anthropic-ai/claude-code`@2.1.100") or reference an environment variable (e.g.,
use ${CLAUDE_CODE_VERSION} in the commandLine) so workspace setup is
reproducible; update the commandLine under the install-claude-code exec block
accordingly and ensure a default version is set in devfile env or CI where
applicable.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.devfile.yaml:
- Around line 57-62: The install step with id "install-claude-code" currently
runs "commandLine: npm install -g `@anthropic-ai/claude-code`" which installs a
moving "latest" release; change this to pin a tested version (e.g., "npm install
-g `@anthropic-ai/claude-code`@2.1.100") or reference an environment variable
(e.g., use ${CLAUDE_CODE_VERSION} in the commandLine) so workspace setup is
reproducible; update the commandLine under the install-claude-code exec block
accordingly and ensure a default version is set in devfile env or CI where
applicable.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4be2d0ea-0704-477c-aef0-a728a30f8c56

📥 Commits

Reviewing files that changed from the base of the PR and between 1126161 and d784d1d.

📒 Files selected for processing (2)
  • .devfile.yaml
  • .vscode/extensions.json

@svor svor requested a review from olexii4 April 10, 2026 10:09
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Apr 10, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: olexii4, svor

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@svor svor merged commit b2d7051 into main Apr 10, 2026
6 checks passed
@svor svor deleted the sv-add-install-claude-command branch April 10, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants