Skip to content

fix(cli): report missing npm during install#12268

Open
cyphercodes wants to merge 1 commit intocontinuedev:mainfrom
cyphercodes:fix/cli-install-npm-diagnostics
Open

fix(cli): report missing npm during install#12268
cyphercodes wants to merge 1 commit intocontinuedev:mainfrom
cyphercodes:fix/cli-install-npm-diagnostics

Conversation

@cyphercodes
Copy link
Copy Markdown

Description

Fixes #12192.

Adds explicit npm validation before the CLI installer uses npm config get prefix. If Node.js is present but npm is missing or its global prefix cannot be read / resolves to an unsafe value, the installer now prints a targeted Node.js was found but npm is missing or misconfigured error instead of only exiting through the generic cleanup message.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

Not applicable; this is installer error handling.

Tests

  • bash -n extensions/cli/scripts/install.sh
  • git diff --check
  • Temporary fake-PATH harness for setup_npm_path with no npm available: verified it exits non-zero with the targeted npm diagnostic.
  • Temporary installer main-flow harness with fake Node.js v22.22.2 and no npm available: verified it exits non-zero after reporting the targeted npm diagnostic.

@cyphercodes cyphercodes requested a review from a team as a code owner April 30, 2026 23:39
@cyphercodes cyphercodes requested review from sestinj and removed request for a team April 30, 2026 23:39
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@cyphercodes
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

CLI install can fail with no error message

1 participant