Repository navigation
mono - chore: upgrade package manager tooling - #82
Conversation
Move packageManager from pnpm 12.3.0 to 12.9.1, the newest stable release at least 7 days old, and record the corepack integrity hash. Co-authored-by: Jared Wray <me@jaredwray.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #82 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 3 3
Lines 19 19
Branches 1 1
=========================================
Hits 19 19 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1f26b0f5d9
ℹ️ 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".
| "node": ">=22.12.0" | ||
| }, | ||
| "packageManager": "pnpm@12.3.0", | ||
| "packageManager": "pnpm@12.9.1+sha512.06b055fff5cd20dc12781de173ced3310ce0951bf2ec620211a16045d5444d5c93a663210764ef29a653a610459ba0368f0e74f84d80c54723cf95aae7bc0d6d", |
There was a problem hiding this comment.
After this change, SECURITY.md:29 still says that packageManager pins pnpm@12.3.0. The security checklist now reports the wrong version for this control. Update the checklist to pnpm@12.9.1 in this change.
Useful? React with 👍 / 👎.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the package manager pnpm from version 12.3.0 to 12.9.1 in both package.json and pnpm-lock.yaml, along with updating its platform-specific executable dependencies. There are no review comments to address, and I have no additional feedback to provide.
2 participants
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Chore. Upgrades the package manager pin.
Summary
packageManagernamedpnpm@12.3.0with no integrity hash.corepack use pnpm@12.9.1rewrites it to that release plus its hash.12.9.1is the newest stable pnpm published at least 7 days ago.pnpmis not a devDependency, so there is nopnpm outdatedrow to follow.Changes
packageManagerfrom pnpm 12.3.0 to 12.9.1 and record the corepack integrity hashVerification
pnpm build && pnpm testpasses on Node 22.23.3 with pnpm 12.9.1 (15 tests, build succeeded)Artifact diffs