Skip to content

chore: raise the pnpm pin to ^12.4.1 - #14

Merged
dinwwwh merged 1 commit into
mainfrom
chore/pnpm-12-4-1
Sep 12, 2026
Merged

chore: raise the pnpm pin to ^12.4.1#14
dinwwwh merged 1 commit into
mainfrom
chore/pnpm-12-4-1

Conversation

@dinwwwh

@dinwwwh dinwwwh commented Sep 12, 2026

Copy link
Copy Markdown
Member

Raises the pnpm pin in devEngines.packageManager from ^12.0.0 to ^12.4.1 and records 12.4.1 in pnpm-lock.yaml (previously 12.0.0), the same change as middleapi/orpc#2012. With a range pin, any environment that launches pnpm through Corepack resolves the range to the newest 12.x on npm and cannot switch to the version recorded in the lockfile, so an older lockfile pin fails pnpm install --frozen-lockfile with ERR_PNPM_FROZEN_LOCKFILE_WITH_OUTDATED_LOCKFILE. Keeping the pin at the current release avoids that.

Behavior

  • The lockfile diff is limited to the packageManagerDependencies block and the @pnpm/exe.* platform packages for 12.4.1.
  • The next pnpm 12.x release needs another refresh (pnpm self-update <version> followed by pnpm install).

Testing

  • pnpm install --frozen-lockfile passes with pnpm 12.4.1: the supply-chain check passes and the resolution step is skipped.
  • eslint package.json passes.

Keeps the devEngines range and the pnpm-lock.yaml pin at the current
pnpm 12.x release. Environments that launch pnpm through Corepack
resolve the range to the newest 12.x and cannot switch to the lockfile's
version, so an older lockfile pin fails frozen installs.
@pullfrog

pullfrog Bot commented Sep 12, 2026

Copy link
Copy Markdown

This run was cancelled 🛑

The workflow was cancelled before completion. Please check the link below for details.

Pullfrog  | View workflow run | via Pullfrog𝕏

@codecov

codecov Bot commented Sep 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@dinwwwh
dinwwwh merged commit 22624bf into main Sep 12, 2026
6 of 7 checks passed
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.

1 participant