Skip to content

docs: note that a wrong in-repo npm version means a missing volta inventory entry - #526

Merged
sarahentzel merged 3 commits into
developfrom
volta-npm-install-note
Aug 22, 2026
Merged

docs: note that a wrong in-repo npm version means a missing volta inventory entry#526
sarahentzel merged 3 commits into
developfrom
volta-npm-install-note

Conversation

@nabalone

@nabalone nabalone commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

The repo pins both node and npm via Volta in package.json ([email protected], [email protected]), but Volta does not always download a pinned npm on demand. When [email protected] is absent from Volta's local inventory it silently falls back to the npm bundled with Node 22 (10.9.x) — so npm --version reports 10.9.8 in the repo while the pin looks correct in package.json.

Adds a short note to the README Install section, phrased as a conditional: if the in-repo npm --version isn't the pinned version, that's the cause, and volta install [email protected] fixes it once. Contributors who already have it in their inventory can read straight past.

Docs only — no code or dependency changes.

Test plan

  • npx prettier --check README.md passes
  • Verified the underlying behavior locally: with only npm-11.5.1.tgz in Volta's inventory, in-repo npm --version reported 10.9.8; after volta install [email protected] it reports 12.0.2, and it keeps reporting 12.0.2 even with the user-level default set back to 11.5.1 — confirming the repo pin resolves from the inventory rather than from the user default.

🤖 Generated with Claude Code

nabalone and others added 2 commits August 21, 2026 19:33
Volta does not always fetch a pinned npm on demand. When [email protected] is
missing from its local inventory it silently falls back to the npm bundled
with Node 22 (10.9.x), so the pin in package.json looks like it is working
when it is not. Document the one-time `volta install [email protected]` and the
version check in the README Install section.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
Most contributors will already have the pinned npm in Volta's inventory, so
lead with the version check as the trigger rather than prescribing the install
for everyone.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@nabalone
nabalone force-pushed the volta-npm-install-note branch from 1e1e9ea to d40d288 Compare August 21, 2026 23:37
Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@nabalone nabalone changed the title docs: note the one-time volta install for the pinned npm docs: note that a wrong in-repo npm version means a missing volta inventory entry Aug 21, 2026
@nabalone
nabalone marked this pull request as ready for review August 21, 2026 23:38
@sarahentzel
sarahentzel merged commit 6c67ead into develop Aug 22, 2026
2 checks passed
@sarahentzel
sarahentzel deleted the volta-npm-install-note branch August 22, 2026 15:04
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.

2 participants