Upgrade pnpm to 11.25.0 - #429
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Thermo-nuclear review: APPROVE WITH PROCESS NOTEThe diff is appropriately boring: one package-manager pin is updated consistently across Process note: this PR still targets |
Thermo-nuclear code quality review: CHANGESP1: the current PR diff is structurally contaminated by fourteen unrelated porting commitsThe PR is titled as a pnpm upgrade, but its current main-base diff is 35 files and about 2.3k additions. Before the pnpm commit, the branch carries fourteen upstream-port/review commits touching Antigravity history/quota, Cursor token costs, Codex API/reset handling, Grok, OpenRouter, localization, and other provider code. Examples include a new 570-line This cannot be reviewed or merged as an atomic package-manager change. Do not rearrange this history in place. Use the clean replacement branch/PR that contains only the intended approved tips, then close this contaminated PR. The existing clean replacement is #457. |
Upgrade pnpm to 11.25.0 from #429
|
Superseded by #457, which carries the intended pnpm/toolchain change on clean history, includes the thermo review fixes, and is now merged. Closing this original instead of resolving/importing its unrelated porting commits. |
Summary
pnpm-lock.yamlunchanged because the lockfile format/content is compatible and frozen install is stableValidation
pnpm --version->11.25.0pnpm install --frozen-lockfile-> pass;pnpm-lock.yamlSHA-256 unchangedscripts/release-pipeline.tests.ps1-> pass.github/scripts/interaction-guard.test.mjs-> 9/9 passinstall-release-prerequisites.ps1 -AssertOnlyaccepts Node 24 and pnpm 11.25.0, then stops later because this workstation does not have Inno Setup 6Local CI mirror limitation
local-check.ps1 -Slice cireaches Rust clippy but cannot complete on this workstation becauselink.exeresolves toC:\Program Files\Git\usr\bin\link.exerather than the MSVC linker. No Rust code changed in this PR.No UI changes, so CUA proof is not applicable.