Skip to content

fix: ensure nemoclaw CLI is on PATH after nvm-based install#8

Closed
ericksoa wants to merge 1 commit into
mainfrom
fix/nemoclaw-path-after-install
Closed

fix: ensure nemoclaw CLI is on PATH after nvm-based install#8
ericksoa wants to merge 1 commit into
mainfrom
fix/nemoclaw-path-after-install

Conversation

@ericksoa

Copy link
Copy Markdown
Contributor

Summary

  • After install.sh installs Node.js via nvm, npm link / npm install -g puts the nemoclaw binary in nvm's non-standard bin directory which isn't on PATH
  • Users see nemoclaw: command not found after a successful install
  • This fix adds the npm global bin dir to PATH immediately during install, persists it to .bashrc/.zshrc, and verifies nemoclaw is callable before continuing to onboard

Test plan

  • Run install.sh on a clean Ubuntu 22.04 container with no Node.js
  • Verify nemoclaw is callable immediately after install completes
  • Open a new shell and verify nemoclaw is still callable

🤖 Generated with Claude Code

After install.sh installs Node via nvm, npm global bins land in a
non-standard directory that isn't on PATH. This adds the npm global
bin dir to PATH immediately and persists it to .bashrc/.zshrc so
nemoclaw works in the current session and future shells.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Aaron Erickson <aerickson@nvidia.com>
@ericksoa

Copy link
Copy Markdown
Contributor Author

Superseded by #228, which takes a more comprehensive approach (refresh_path, nvm fallback detection, soft verification, dedicated install-openshell.sh, post-install shell reload message).

@ericksoa ericksoa closed this Mar 17, 2026
jessesanford pushed a commit to jessesanford/NemoClaw that referenced this pull request Mar 24, 2026
jyaunches added a commit that referenced this pull request May 6, 2026
- Fix recovery scripts in agent-runtime.ts that still used curl -sf
  on / instead of the new HTTP status code pattern on /health (#3)
- Add device-auth-health-e2e to scorecard.needs (#8)
- Use openshell-${SANDBOX_NAME} SSH host alias in E2E test (#7)
@cv cv mentioned this pull request Jun 5, 2026
12 tasks
@wscurran wscurran added the bug-fix PR fixes a bug or regression label Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix PR fixes a bug or regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants