Visitar URL original
fix: update pnpm to v11.19.0 by jamietanna · Pull Request #283 · pnpm/action-setup · GitHub
Skip to content

fix: update pnpm to v11.19.0 - #283

Merged
zkochan merged 2 commits into
pnpm:masterfrom
jamietanna:fix/bump
Aug 3, 2026
Merged

zkochan merged 2 commits into
pnpm:masterfrom
jamietanna:fix/bump

Conversation

@jamietanna

@jamietanna jamietanna commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • node scripts/update-bootstrap.mjs 11.19.0 regenerated lockfiles with matching versions/integrity hashes
  • pnpm run build rebuilt dist/index.js and confirmed it embeds @pnpm/exe":"11.19.0"

Summary by CodeRabbit

Chores

  • Updated the bundled pnpm tooling to version 11.19.0.
  • Refreshed platform-specific package references and verification metadata included with the installation process.
  • Updated package registry information to ensure the bundled tooling can be retrieved and validated consistently across supported platforms.
  • Improved consistency and reliability when installing pnpm across supported environments.

@jamietanna
jamietanna requested a review from zkochan as a code owner August 3, 2026 07:41
@coderabbitai

coderabbitai Bot commented Aug 3, 2026 •

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@jamietanna, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 7 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2ec07a22-4ad2-4de6-a0bd-98d1a9d5931b

📥 Commits

Reviewing files that changed from the base of the PR and between ddaf3cc and dc8abb5.

⛔ Files ignored due to path filters (1)
  • dist/index.js is excluded by !**/dist/**
📒 Files selected for processing (2)
  • src/install-pnpm/bootstrap/exe-lock.json
  • src/install-pnpm/bootstrap/pnpm-lock.json
📝 Walkthrough

Walkthrough

The bootstrap lockfiles update pnpm, @pnpm/exe, and platform-specific packages from version 11.7.0 to 11.19.0. Registry URLs and integrity hashes are refreshed.

Changes

Bootstrap lockfile update

Layer / File(s) Summary
Bootstrap version resolution
src/install-pnpm/bootstrap/exe-lock.json, src/install-pnpm/bootstrap/pnpm-lock.json
The root @pnpm/exe and pnpm entries now resolve version 11.19.0 with updated metadata.
Platform package resolution
src/install-pnpm/bootstrap/exe-lock.json
The Linux, macOS, and Windows optional packages now resolve version 11.19.0 with updated archive URLs and integrity hashes.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: zkochan

Poem

A rabbit checks each hash in line,
pnpm hops to version nineteen.
Platform packages follow through,
With refreshed links and checksums too.
Bootstrap lockfiles now align.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive The lockfiles show the pnpm update, but the required dist/index.js rebuild is excluded from review by !/dist/. Verify the excluded dist/index.js rebuild and confirm it embeds @pnpm/exe version 11.19.0 and resolves the standalone-mode failure.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the primary change: updating the bundled pnpm version to 11.19.0.
Out of Scope Changes check ✅ Passed The reviewed changes only update pnpm bootstrap lockfiles, which directly supports the linked issue and stated pull request objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Bump bundled/bootstrap pnpm to 11.13.1

🐞 Bug fix ⚙️ Configuration changes 🕐 10-20 Minutes

Grey Divider

AI Description

• Update bootstrap lockfiles to reference pnpm/@pnpm/exe v11.13.1.
• Refresh resolved tarball URLs and integrity hashes for all bundled platform binaries.
Diagram

graph TD
  Dev["Maintainer"] --> Script["update-bootstrap.mjs"] --> Locks["bootstrap lockfiles"] --> Installer["install-pnpm"] --> Registry{{"npm registry"}} --> Tool["pnpm/exe 11.13.1"]
Loading
High-Level Assessment

Updating the pinned bootstrap lockfiles via the existing regeneration workflow is the right approach here (keeps bootstrap installs deterministic and verifiable). Alternatives like relying on a system pnpm or floating versions would reduce determinism and increase install variability.

Files changed (2) +36 / -36

Other (2) +36 / -36
exe-lock.jsonBump @pnpm/exe and platform binaries to 11.13.1 +32/-32

Bump @pnpm/exe and platform binaries to 11.13.1

• Updates the pinned @pnpm/exe version from 11.7.0 to 11.13.1. Refreshes resolved tarball URLs and integrity hashes for all optional platform-specific @pnpm/* packages.

src/install-pnpm/bootstrap/exe-lock.json

pnpm-lock.jsonBump pnpm to 11.13.1 in bootstrap lockfile +4/-4

Bump pnpm to 11.13.1 in bootstrap lockfile

• Updates the pinned pnpm package version from 11.7.0 to 11.13.1. Refreshes the resolved tarball URL and integrity hash to match the new version.

src/install-pnpm/bootstrap/pnpm-lock.json

@zkochan

zkochan commented Aug 3, 2026

Copy link
Copy Markdown
Member

Why not to the latest?

@jamietanna

Copy link
Copy Markdown
Contributor Author

Wasn't sure if we had a policy around that :D Happy bumping to the literal latest, or should I do a version or two behind?

@zkochan

zkochan commented Aug 3, 2026

Copy link
Copy Markdown
Member

The literal latest

@jamietanna jamietanna changed the title fix: update pnpm to v11.13.1 fix: update pnpm to v11.19.0 Aug 3, 2026
@jamietanna
jamietanna force-pushed the fix/bump branch 2 times, most recently from ec8eefb to ddaf3cc Compare August 3, 2026 08:49
@jamietanna

jamietanna commented Aug 3, 2026 •

Copy link
Copy Markdown
Contributor Author

Not sure why CI is failing - I couldn't seem to repro locally

#284 might be related? Nope

zkochan pushed a commit that referenced this pull request Aug 3, 2026
Noticed while working on #283.
@jamietanna

Copy link
Copy Markdown
Contributor Author

Claude Sonnet 5 believes:

- Root cause: check-dist rebuilds dist/index.js and diffs it against what's committed. esbuild's minified output isn't byte-identical across platforms (macOS arm64, where I built it, vs Linux x64, where CI builds it) — same esbuild version, same locked dependencies, just different bytes. Bumping CI's pnpm to v11 (#284) didn't touch this; it was solving a different problem.

jamietanna and others added 2 commits August 3, 2026 10:27
Via `scripts/update-bootstrap.mjs 11.19.0`

Co-authored-by: Claude Sonnet 5 <jamie.tanna+claude-code@mend.io>
As it needs to be rebuilt on Linux.
@jamietanna

Copy link
Copy Markdown
Contributor Author

I've re-pushed from my Linux machine, which seems to work - that seemed to have been the issue!

@zkochan
zkochan merged commit 48261ac into pnpm:master Aug 3, 2026
21 checks passed
dbmikus pushed a commit to gofixpoint/amika that referenced this pull request Oct 23, 2026
Bumps [pnpm/action-setup](https://github.com/pnpm/action-setup) from 5.0.0 to 6.1.0.

## Release notes

Sourced from pnpm/action-setup's releases.

### v6.1.0

### What's Changed
- feat: support pnpm v12 by `@zkochan` in pnpm/action-setup#288

Full Changelog: pnpm/action-setup@v6.0.10...v6.1.0

### v6.0.10

### What's Changed
- docs(README): point users to the successor pnpm/setup action by `@BlackHole1` in pnpm/action-setup#282
- refactor: introduce restore keys for cache by `@SukkaW` in pnpm/action-setup#280
- ci: use pnpm 11 for `pr-check` by `@jamietanna` in pnpm/action-setup#284
- fix: update pnpm to v11.19.0 by `@jamietanna` in pnpm/action-setup#283
- docs: update README to include devEngines.packageManager by `@nemchik` in pnpm/action-setup#273

### New Contributors
- `@jamietanna` made their first contribution in pnpm/action-setup#284
- `@nemchik` made their first contribution in pnpm/action-setup#273

Full Changelog: pnpm/action-setup@v6...v6.0.10

### v6.0.9

### What's Changed
- fix: update pnpm to v11.7.0 by `@zkochan` in pnpm/action-setup#267

Full Changelog: pnpm/action-setup@v6...v6.0.9

### v6.0.8

### What's Changed
- docs(README): fix `cache_dependency_path` type by `@haines` in pnpm/action-setup#257
- fix: drop patchPnpmEnv so standalone+self-update works on Windows by `@zkochan` in pnpm/action-setup#258
- fix: update pnpm to 11.1.1 by `@mungodewar` in pnpm/action-setup#248

### New Contributors
- `@mungodewar` made their first contribution in pnpm/action-setup#248

Full Changelog: pnpm/action-setup@v6.0.7...v6.0.8

### v6.0.7

### What's Changed
- fix: honor devEngines.packageManager.onFail=error (#252) by `@zkochan` in pnpm/action-setup#254
- fix: restore inputs from state in post by `@haines` in pnpm/action-setup#255
- fix: self-update bootstrap to packageManager-pinned version (#233) by `@zkochan` in pnpm/action-setup#256

### New Contributors
- `@haines` made their first contribution in pnpm/action-setup#255

Full Changelog: pnpm/action-setup@v6.0.6...v6.0.7

... (truncated)

## Commits
- `ea17c68` feat: support pnpm v12 (#288)
- `0977fd9` docs: Update README to include devEngines.packageManager (#273)
- `48261ac` fix: update pnpm to v11.19.0 (#283)
- `75677f7` ci: use pnpm 11 for `pr-check` (#284)
- `769ae71` refactor: introduce restore keys for cache (#280)
- `6fed91f` docs(README): point users to the successor pnpm/setup action (#282)
- `0ebf471` fix: update pnpm to v11.7.0 (#267)
- `0e279bb` fix: update pnpm to 11.1.1 (#248)
- `3e83581` fix: drop patchPnpmEnv so standalone+self-update works on Windows (#258)
- `551b42e` docs(README): fix `cache_dependency_path` type (#257)
- Additional commits viewable in compare view
theinditector pushed a commit to IndicoDataSolutions/pnpm-action-setup that referenced this pull request Oct 28, 2026
* fix: update pnpm to v11.7.0 (pnpm#267)

* fix: update pnpm to v11.7.0

* fix: update bundle

* docs(README): point users to the successor pnpm/setup action (pnpm#282)

* docs(README): point users to the successor pnpm/setup action

The pnpm setup action has moved to `pnpm/setup`, which installs
pnpm v11+ as a self-contained native executable and can install a
JavaScript runtime (Node.js, Bun, or Deno) in the same step,
replacing `actions/setup-node`.

Add a notice at the top of the README, a migration section with a
before/after workflow example and an input/output mapping table,
and mention the successor in the Notes section. The rest of the
docs stay intact since `pnpm/action-setup` remains the way to
install pnpm v10 and older.

Signed-off-by: Kevin Cui <bh@bugs.cc>

* docs(README): address review feedback on migration section

Add an explicit version: 11 to the migration example and note when the
input can be omitted, since pnpm/setup requires pnpm v11+ and a repo
migrating from version: 10 may have no packageManager field or one that
pins v10.

Move the v2 upgrade warning below the page title so the two callouts
are no longer adjacent blockquotes (markdownlint MD028), keeping the
successor notice and the legacy warning as separate blocks.

Fix the comma splice and use "set up" as the verb in the Notes
section.

Signed-off-by: Kevin Cui <bh@bugs.cc>

---------

Signed-off-by: Kevin Cui <bh@bugs.cc>

* refactor: introduce restore keys for cache (pnpm#280)

* ci: use pnpm 11 for `pr-check` (pnpm#284)

Noticed while working on pnpm#283.

* fix: update pnpm to v11.19.0 (pnpm#283)

* fix: update pnpm to v11.19.0

Via `scripts/update-bootstrap.mjs 11.19.0`

Co-authored-by: Claude Sonnet 5 <jamie.tanna+claude-code@mend.io>

* fixup! fix: update pnpm to v11.19.0

As it needs to be rebuilt on Linux.

---------

Co-authored-by: Claude Sonnet 5 <jamie.tanna+claude-code@mend.io>

* docs: Update README to include devEngines.packageManager (pnpm#273)

Support added in pnpm#211 and pnpm#256

* feat: support pnpm v12 (pnpm#288)

* feat: support pnpm v12

Approve the pinned @pnpm/exe install script for npm 12, add pnpm 12 coverage across supported runner platforms, and clarify that action-setup remains supported alongside pnpm/setup.

* refactor: use pnpm 12 native bootstrap

Install the plain pnpm v12 package as the single native bootstrap, remove the legacy @pnpm/exe lockfile and runtime path, and retain the standalone input as a no-op for workflow compatibility.

* fix: keep bootstrap updates on pnpm 12

Handle npm's array-shaped view output and resolve only pnpm 12 distribution tags.

* fix: use native bootstrap only for pnpm 12

Route pnpm 12-only versions and ranges through the plain pnpm native package while preserving the existing Node and @pnpm/exe bootstrap paths for older releases.

* style: remove updater narrative comments

---------

Signed-off-by: Kevin Cui <bh@bugs.cc>
Co-authored-by: Zoltan Kochan <z@kochan.io>
Co-authored-by: Kevin Cui <github@bugs.cc>
Co-authored-by: Sukka <isukkaw@gmail.com>
Co-authored-by: Jamie Tanna <github@jamietanna.co.uk>
Co-authored-by: Jamie Tanna <jamie@jamietanna.co.uk>
Co-authored-by: Claude Sonnet 5 <jamie.tanna+claude-code@mend.io>
Co-authored-by: Eric Nemchik <eric@nemchik.com>
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.

Error after installing in standalone mode when trying to install packages with a CLI globally - This: not found

2 participants