Skip to content

Fix devcontainer smoke test failure from incomplete Claude CLI install - #51

Merged
idvoretskyi merged 4 commits into
mainfrom
copilot/fix-test-devcontainer-job
Aug 20, 2026
Merged

Fix devcontainer smoke test failure from incomplete Claude CLI install#51
idvoretskyi merged 4 commits into
mainfrom
copilot/fix-test-devcontainer-job

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The Test Devcontainer Actions job was failing in the smoke test because claude --version resolved to the wrapper binary, but the native Claude CLI binary had not been installed. This change makes the devcontainer setup complete that install deterministically during updateContentCommand.

  • Root cause

    • @anthropic-ai/claude-code was present, but its native binary placement step had not completed before the smoke test ran.
    • The failure surfaced only when CI executed claude --version inside the built devcontainer.
  • Change

    • Update .devcontainer/devcontainer.json to install both CLIs in updateContentCommand.
    • Run Claude’s packaged install.cjs explicitly after the global install so the native claude binary is placed regardless of npm script execution behavior.
  • Behavioral impact

    • Keeps the existing devcontainer shape and toolset unchanged.
    • Makes Claude CLI availability explicit and reproducible for CI and local devcontainer builds.
{
  "updateContentCommand": "npm install -g @anthropic-ai/claude-code opencode-ai && node \"$(npm root -g)/@anthropic-ai/claude-code/install.cjs\""
}

Copilot AI and others added 2 commits August 20, 2026 04:52
Co-authored-by: idvoretskyi <118459+idvoretskyi@users.noreply.github.com>
Co-authored-by: idvoretskyi <118459+idvoretskyi@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'Test Devcontainer' Fix devcontainer smoke test failure from incomplete Claude CLI install Aug 20, 2026
Copilot AI requested a review from idvoretskyi August 20, 2026 04:54
@idvoretskyi
idvoretskyi marked this pull request as ready for review August 20, 2026 04:57
Co-authored-by: idvoretskyi <118459+idvoretskyi@users.noreply.github.com>
@idvoretskyi
idvoretskyi merged commit d2c45ce into main Aug 20, 2026
8 checks passed
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.

2 participants