Skip to content

fix(install): prefer active nvm node after installation#146

Merged
duguwanglong merged 1 commit intomainfrom
fix/install-nvm-node-path-resolution
Apr 20, 2026
Merged

fix(install): prefer active nvm node after installation#146
duguwanglong merged 1 commit intomainfrom
fix/install-nvm-node-path-resolution

Conversation

@xiami762
Copy link
Copy Markdown
Contributor

@xiami762 xiami762 commented Apr 20, 2026 •
edited
Loading

Summary

  • keep the installer aligned with the Node.js version just activated by nvm
  • clear the shell command cache after PATH updates so follow-up checks do not resolve a stale node binary
  • fail with a clearer warning when the current shell still resolves a different Node.js executable after nvm use

fix(install): prefer active nvm node after installation
9e2b5c0 
Keep the installer aligned with the Node.js version just activated by nvm so follow-up checks do not fall back to an older cached or higher-priority binary.

Made-with: Cursor
@xiami762 xiami762 requested a review from duguwanglong April 20, 2026 07:59
@duguwanglong duguwanglong merged commit 0ecb1cb into main Apr 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@duguwanglong duguwanglong duguwanglong approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xiami762 @duguwanglong