Skip to content

build: pin pnpm 10 via packageManager so Dependabot and CI agree - #127

Merged
PAMulligan merged 2 commits into
mainfrom
chore/pin-pnpm
Sep 12, 2026
Merged

PAMulligan merged 2 commits into
mainfrom
chore/pin-pnpm

Conversation

@PAMulligan

Copy link
Copy Markdown
Collaborator

Why

Dependabot picks its own pnpm version when none is pinned and has started regenerating lockfiles with pnpm 11, which no longer reads pnpm.overrides from package.json. That drops the qs override section from pnpm-lock.yaml, and CI's pnpm 9 then fails every frozen install with ERR_PNPM_LOCKFILE_CONFIG_MISMATCH. This hit #120 and #126 and will hit every future Dependabot PR until the version is pinned.

What

  • Add "packageManager": "pnpm@10.34.5" to the root package.json. Both Dependabot and pnpm/action-setup honor it.
  • Remove the hardcoded version: 9 input from every workflow, since pnpm/action-setup@v4 errors when both a version input and packageManager are present.

pnpm 10 is the version that wrote the current lockfile (it carries the libc fields pnpm 9 does not emit) and it still reads the package.json overrides.

Verification

  • corepack pnpm install --frozen-lockfile with pnpm 10.34.5 succeeds against the unchanged lockfile, with no ignored build scripts.
  • Pipeline package: 133 tests pass. Script suite: 512 tests pass.

🤖 Generated with Claude Code

Dependabot picks its own pnpm version when none is pinned and has
started regenerating lockfiles with pnpm 11, which no longer reads
pnpm.overrides from package.json. That drops the qs override from
pnpm-lock.yaml, and CI's pnpm 9 then fails every frozen install with
ERR_PNPM_LOCKFILE_CONFIG_MISMATCH (seen on PRs #120 and #126).

Pin pnpm@10.34.5 in packageManager, which both Dependabot and
pnpm/action-setup honor, and drop the hardcoded version input from the
workflows since action-setup refuses to have both. pnpm 10 is the
version that wrote the current lockfile and still reads the
package.json overrides.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@PAMulligan
PAMulligan merged commit 6922c1b into main Sep 12, 2026
11 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