Skip to content

ci: upgrade pnpm to 11#6367

Merged
metalmatze merged 1 commit into
mainfrom
ui/pnpm-11
Jun 9, 2026
Merged

ci: upgrade pnpm to 11#6367
metalmatze merged 1 commit into
mainfrom
ui/pnpm-11

Conversation

@metalmatze

@metalmatze metalmatze commented Jun 8, 2026

Copy link
Copy Markdown
Member

NixOS is dropping the older pnpm packages, so this moves us to pnpm 11. The non-obvious part is that pnpm 11 stopped reading the pnpm field in package.json, which meant our security overrides were silently ignored and pnpm install --frozen-lockfile failed on a config mismatch. They now live in pnpm-workspace.yaml alongside the allowBuilds allowlist, with @parca/* excluded from pnpm 11's 24h release-age delay, and CI reads the version from packageManager via action-setup@v6 instead of a pinned env var.

pnpm 11 no longer reads the "pnpm" field in package.json, so add the
security overrides to pnpm-workspace.yaml (where pnpm 10+ reads them)
and switch the build-script allowlist to allowBuilds. The overrides are
kept in package.json too because Vercel deploys still run pnpm 9 (the
default for a lockfileVersion 9.0 lockfile when Corepack is off), which
only reads them from there. Exclude our own @parca/* packages from
pnpm 11's 24h release-age delay.

CI reads the version from packageManager via action-setup@v6 instead of
a pinned PNPM_VERSION env var.
@alwaysmeticulous

alwaysmeticulous Bot commented Jun 8, 2026

Copy link
Copy Markdown

✅ Meticulous spotted 0 visual differences across 267 screens tested: view results.

Meticulous evaluated ~4 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit 5f91a0f ci: upgrade pnpm to 11. This comment will update as new commits are pushed.

@metalmatze metalmatze merged commit cce673d into main Jun 9, 2026
38 checks passed
@metalmatze metalmatze deleted the ui/pnpm-11 branch June 9, 2026 08:53
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.

2 participants