Skip to content

chore: declare pnpm via devEngines.packageManager - #3539

Merged
setchy merged 2 commits into
mainfrom
chore/devengines-package-manager
Oct 28, 2026
Merged

setchy merged 2 commits into
mainfrom
chore/devengines-package-manager

Conversation

@setchy

@setchy setchy commented Oct 28, 2026

Copy link
Copy Markdown
Member

Summary

Moves the pnpm pin from the legacy top-level packageManager field into devEngines.packageManager:

  • before: "packageManager": "pnpm@12.15.0"
  • after: "devEngines": { ..., "packageManager": { "name": "pnpm", "version": "12.15.0", "onFail": "download" } }

Same exact version, so there is no change to the resolved manager. pnpm 12 records it under packageManagerDependencies and can self-provision; Corepack ≥0.36 and both pnpm/action-setup and pnpm/setup read the field.

Notes:

  • corepack use pnpm@… would re-add the top-level field, so use corepack install or edit devEngines instead.
  • Node 25+ no longer bundles Corepack, so the devEngines path is the forward-looking one.

Verification

  • package.json parses; no top-level packageManager remains
  • pnpm install --lockfile-only resolves the same pnpm version
  • Lockfile unchanged (except the specifier hash where noted)

@setchy
setchy requested a review from afonsojramos as a code owner October 28, 2026 12:29
@github-actions github-actions Bot added the chore Miscellaneous tasks label Oct 28, 2026
@sonarqubecloud

Copy link
Copy Markdown

@setchy
setchy merged commit acd80eb into main Oct 28, 2026
19 checks passed
@setchy
setchy deleted the chore/devengines-package-manager branch October 28, 2026 12:44
@github-actions github-actions Bot added this to the Release 7.11.0 milestone Oct 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Miscellaneous tasks

Development

Successfully merging this pull request may close these issues.

1 participant