Skip to content

Fix npm update prefix for symlinked bins#67

Open
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/resolve-npm-prefix-symlink
Open

Fix npm update prefix for symlinked bins#67
w287346141 wants to merge 1 commit into
Yapie0:mainfrom
w287346141:codex/resolve-npm-prefix-symlink

Conversation

@w287346141
Copy link
Copy Markdown

Summary

  • resolve symlinked npm bin paths before extracting the global install prefix
  • keep synthetic/nonexistent test paths working by falling back to the literal path
  • add a regression test covering a real bin symlink layout

Validation

  • npm.cmd test -- --run tests/version.test.ts tests/startup-update-check.test.ts
  • npm.cmd run lint
  • npm.cmd run typecheck
  • git diff --check
  • push hook ran npm.cmd run verify

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