fix(ci): pin pnpm@9 to unblock publish - #556
Merged
parthlambdatest merged 1 commit intoAug 30, 2026
Merged
Conversation
The v4.1.80 publish failed with ERR_PNPM_IGNORED_BUILDS. release.yaml installs pnpm unpinned (npm install -g pnpm), so it picked up pnpm 10, which turned ignored build scripts from a warning into a hard error. v4.1.79 published fine on Aug 10; pnpm 10 landed in between, so the next publish was going to fail whoever cut it. Pin pnpm@9: the version that generated the committed lockfile (lockfileVersion 9.0) and produced every successful publish to date. The five blocked packages (sharp, esbuild, the three @playwright/browser-*) all need their postinstall scripts, and pnpm 9 runs them by default. Deliberately not adding pnpm.onlyBuiltDependencies: once that allowlist exists, pnpm 10 silently skips scripts for any package not listed, so a future dependency needing one would break quietly instead of failing loudly as it did here. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
shrinishLT
force-pushed
the
fix-pnpm-ignored-builds
branch
from
August 30, 2026 06:41
1f2b3ec to
66635f3
Compare
parthlambdatest
approved these changes
Aug 30, 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
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.
Problem
The
v4.1.80publish failed:Cause
release.yamlinstalls pnpm unpinned:pnpm 10 turned ignored build scripts from a warning into a hard error.
v4.1.79published successfully on Aug 10; pnpm 10 landed between then and now, so the next publish was going to fail regardless of its contents —v4.1.80was simply the first one cut. (Thev4.1.72/v4.1.73failures on Jul 22 are unrelated; a greenv4.1.74followed the same day.)Fix
Pin
pnpm@9— the version that generated the committed lockfile (lockfileVersion: '9.0') and produced every successful publish to date. pnpm 9 runs build scripts by default, which all five blocked packages require:sharpesbuild@playwright/browser-{chromium,firefox,webkit}One line;
package.jsonis untouched.Why not
pnpm.onlyBuiltDependenciesConsidered and deliberately left out. Once that allowlist exists, pnpm 10 silently skips build scripts for any package not in the list — so a future dependency needing one would break quietly, instead of failing loudly as it did here. The loud error is worth keeping as a signal; the unpinned install is the actual root cause.
Release state
4.1.80is not on npm (latest published is4.1.79) — the failed run never reached the publish step. So no version bump is needed:stagestays on4.1.80and that version is still free.Order: merge this → re-run / re-cut
v4.1.80→ then #554 (stage -> prod).🤖 Generated with Claude Code