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)
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.

1 participant