From 9731f75c96e953f943d5754ad4dcd10be87d388c Mon Sep 17 00:00:00 2001 From: Steven Obiajulu Date: Mon, 16 Mar 2026 23:16:28 -0400 Subject: [PATCH] fix(install): align Node version, use HTTPS, fix recovery hint - Use $RECOMMENDED_NODE_MAJOR (22) instead of hardcoded 24. Aligns with Dockerfile (node:22-slim), brev-setup.sh (setup_22.x), test/Dockerfile.sandbox, and the variable defined on line 20. - Switch fallback npm install from git+ssh:// to git+https://. Users without GitHub SSH keys configured would fail to install. Matches the HTTPS URL used in scripts/install.sh (PR #89). - Update recovery hint to use GitHub URL instead of `npm install -g nemoclaw`, which fails until the package is published (#71). Signed-off-by: Steven Obiajulu --- install.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/install.sh b/install.sh index c7e37cad1d..4622205c07 100755 --- a/install.sh +++ b/install.sh @@ -139,7 +139,7 @@ install_nodejs() { bash "$nvm_tmp" rm -f "$nvm_tmp" ensure_nvm_loaded - nvm install 22 + nvm install "$RECOMMENDED_NODE_MAJOR" info "Node.js installed: $(node --version)" } @@ -319,7 +319,7 @@ verify_nemoclaw() { return 0 else warn "Could not locate the nemoclaw executable." - warn "Try running: npm install -g git+https://github.com/NVIDIA/NemoClaw.git" + warn "Try running: npm install -g \"git+https://github.com/NVIDIA/NemoClaw.git\"" fi error "Installation failed: nemoclaw binary not found."