chore: bump pnpm/action-setup to v6 - #96
Merged
Merged
Conversation
The repo never used corepack — CI already installs pnpm through pnpm/action-setup and pins it with `packageManager`. v4 → v6 is a drop-in: inputs and outputs are unchanged (only the action's own runtime moved from node20 to node24), and v6 is the first version that reads `devEngines.packageManager`, so switching to that declaration later needs no further CI change. `packageManager: pnpm@11.0.8` stays as the pin — action-setup, pnpm 10 and pnpm 11 all consume it, and pnpm refuses to honour both fields at once. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Buu8ZHjnksQQdzS6vgn5ez
Contributor
📦 Snapshot releasePublished |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
This started as "remove corepack" — but there is no corepack in this repo. No
corepack enable, nocorepack prepare, nothing in the workflows, docs, or scripts. CI already installs pnpm viapnpm/action-setupand pins it withpackageManager: pnpm@11.0.8. Companion to tenphi/tasty#265.Changes
pnpm/action-setupv4 → v6 in all four call sites (ci.yml,deploy-playground.yml, both jobs ofrelease.yml).Drop-in despite skipping a major: v4 and v6 declare identical inputs and outputs — the only
action.ymldeltas are a description wording change and the action's own runtime movingnode20→node24. v6 is also the first version that readsdevEngines.packageManager, so adopting that declaration later needs no further CI change.Why
packageManagerstays (anddevEngines.packageManagerdoes not land here)The forward-looking
devEngines.packageManagerdeclaration isn't viable for this repo yet. Verified locally against pnpm 11.22.0 / pnpm 11.0.8 / npm 10.9.7:Cannot use both "packageManager" and "devEngines.packageManager" in package.json. "packageManager" will be ignored.packageManageris consumed here — by pnpm's own version management and bypnpm/action-setupwhenversionis omitted. v4 does not readdevEnginesat all, so dropping the field before this bump would have broken CI.devEnginesbreaks npm-based publishing.release.ymlrunsnpm pkg set versionandnpm publish. WithdevEngines.packageManager: {name: pnpm, …}andonFail: "download"(or"error"), every npm command exits non-zero:onFail: "warn"/"ignore"lets npm proceed, andwarnprints that block on each npm invocation.devEngines.packageManagerlanded in pnpm 11.0, so a contributor on pnpm 10 would get no pin and no error — todaypackageManagerauto-switches them.onFail: "download", pnpm writespackageManagerDependenciesplus@pnpm/exe/pnpm/@reflinkentries — ~199 extra lines of lockfile.Verification
Workflow-only change — no manifest or lockfile edits. All three workflow files parse as YAML, and the v4↔v6 input/output comparison above was made against the actions' own
action.yml.Follow-up worth considering (not in this PR)
pnpm/action-setup's README now names a successor:pnpm/setup@v1, recommended for pnpm 11+. It installs pnpm as a standalone native binary and can provision the runtime, replacingactions/setup-node. Bigger change, own PR — and it needs care aroundrelease.yml'sregistry-urlsetup for publishing.Generated by Claude Code