Skip to content

Add pnpm-workspace.yaml to approve dependency build scripts#4

Merged
marawan206 merged 1 commit into
mainfrom
fix/pnpm-allow-builds
Jun 9, 2026
Merged

Add pnpm-workspace.yaml to approve dependency build scripts#4
marawan206 merged 1 commit into
mainfrom
fix/pnpm-allow-builds

Conversation

@marawan206

Copy link
Copy Markdown
Contributor

Summary

pnpm 11 blocks dependency install/build lifecycle scripts by default as a supply-chain safeguard. Two dependencies legitimately need theirs:

  • esbuild — its postinstall downloads/links the platform-specific native binary
  • pi-decimals — its install script generates required data files

Without approval, pnpm install exits with code 1 (ERR_PNPM_IGNORED_BUILDS), and since pnpm 11 verifies dependencies before running a script, pnpm dev fails to start on a fresh clone.

Change

Adds pnpm-workspace.yaml approving both packages via allowBuilds, so install completes cleanly and pnpm dev boots.

Testing

  • pnpm install → exits 0, both build scripts run
  • pnpm dev → Next.js dev server ready on http://localhost:3000

pnpm 11 blocks dependency install/build scripts by default as a
supply-chain safeguard. esbuild (downloads its platform binary) and
pi-decimals (generates data files) legitimately need theirs, so
without approval `pnpm install` exits 1 and `pnpm dev` fails to start.

Approve both via allowBuilds so install completes and dev runs.
@vercel

vercel Bot commented Jun 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
comfyci Error Error Jun 4, 2026 12:03pm

Request Review

@marawan206 marawan206 merged commit bf45eb4 into main Jun 9, 2026
6 of 7 checks passed
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.

1 participant