chore: upgrade pnpm to 11.1.2#49
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9b9432f636
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR updates repository tooling metadata to use pnpm 11.1.2, and also bumps the Node.js version used in CI workflows.
Changes:
- Update
packageManagermetadata topnpm@11.1.2. - Bump GitHub Actions CI Node.js version from 20.19.0 to 22.20.0 for lint/test jobs.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | Bumps the packageManager field to pnpm 11.1.2. |
| .github/workflows/nodejs.yml | Updates CI matrices to run lint/tests on Node 22.20.0. |
Comments suppressed due to low confidence (1)
.github/workflows/nodejs.yml:62
package.jsondeclaresengines.node >= 20.19.0, but this test matrix now only runs on 22.20.0. Consider including 20.19.0 in the matrix as well so Windows/Linux coverage includes the minimum supported Node version.
strategy:
matrix:
os: [ubuntu-latest, windows-latest]
node-version: [22.20.0]
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
This PR upgrades the repository package manager metadata to pnpm 11.1.2 and updates pnpm 11 install configuration where dependency build scripts need an explicit allowBuilds decision. Lockfile or small validation fixes are included only where pnpm 11/local checks required them.
Related Links