chore: migrate to pnpm and enable minimum release age - #534
Merged
Merged
Conversation
Migrates the repository's package manager from npm to pnpm as part of an org-wide supply-chain hardening effort. Enables a 24h minimum release age for third-party packages (via pnpm-workspace.yaml and renovate.json) while excluding @apify/* and @crawlee/* so internal releases can still be consumed immediately. - Replace package-lock.json with pnpm-lock.yaml - Move workspaces config from package.json to pnpm-workspace.yaml - Use workspace:* protocol for internal package cross-deps - Add .npmrc with hoisted node-linker - Add reusable .github/actions/pnpm-install composite action - Update all CI workflows to use pnpm - Bump Node engine floor to >=18 and CI test matrix to [18, 20, 22, 24] - Bump packageManager to pnpm@10.24.0 - pnpm publish --no-git-checks in release script - Add renovate rule: @apify/* and @crawlee/* bypass minimumReleaseAge Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
the
t-tooling
Replaces the local .github/actions/pnpm-install composite copy with the shared one from apify/workflows@main. Identical behavior, less duplication.
github-actions
Bot
added
the
tested
B4nan
marked this pull request as ready for review
April 15, 2026 15:55
Silences npm warnings about unknown options like node-linker; pnpm reads the same keys from pnpm-workspace.yaml in camelCase form.
Block accidental npm/yarn install — npm 10.5+ and pnpm 10.x both honor devEngines.packageManager and refuse to run when it doesn't match.
… with pnpm-workspace exclusions Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
devEngines.packageManager breaks pnpm commands that delegate to npm (pnpm version, pnpm pkg set, etc). Use the battle-tested only-allow approach instead (same as Vite, Vue, Astro).
barjin
approved these changes
Apr 22, 2026
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
3 participants
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.
Summary
Migrates fingerprint-suite from npm to pnpm as part of an org-wide supply-chain hardening effort, aligning with the patterns established in apify/apify-shared-js#616, apify/actor-scraper#273 and apify/got-scraping#175.
Enables a 24-hour
minimumReleaseAgefor third-party packages both at install time (viapnpm-workspace.yaml) and at dependency-update time (viarenovate.json).@apify/*and@crawlee/*are excluded so our own releases can still be consumed immediately.Changes
package-lock.jsonwithpnpm-lock.yaml.workspacesconfig frompackage.jsontopnpm-workspace.yamland addminimumReleaseAgesettings with an exclude list for@apify/*and@crawlee/*.generative-bayesian-network,header-generator,fingerprint-generator) to theworkspace:*protocol so pnpm links the local copies rather than fetching from the registry..npmrcwithnode-linker=hoisted,link-workspace-packages=true,prefer-workspace-packages=true,public-hoist-pattern[]=*; drop.npmrcfrom.gitignore..github/actions/pnpm-installand use it from every workflow that installs deps (test-and-sync,publish-to-npm,e2e-benchmark,model-updater).npm run X/npxwithpnpm X/pnpm execin root and package scripts and in workflows.engines.nodeto>=18.0.0in every workspace package and update the CI test matrix to[18, 20, 22, 24].packageManagertopnpm@10.24.0.scripts/publish.shusespnpm publish --no-git-checks.renovate.json: add"minimumReleaseAge": "1 day"and"internalChecksFilter": "strict", remove the legacy"constraints": { "npm": "^7.0.0" }, and add a package rule that setsminimumReleaseAge: 0 daysfor@apify/*and@crawlee/*.🤖 Generated with Claude Code