Skip to content

fix(deps): approve esbuild + sharp build scripts for pnpm 11 - #8

Merged
MikeBengtson merged 1 commit into
mainfrom
fix/pnpm-onlybuilt-deps
May 13, 2026
Merged

MikeBengtson merged 1 commit into
mainfrom
fix/pnpm-onlybuilt-deps

Conversation

@MikeBengtson

Copy link
Copy Markdown
Collaborator

Summary

pnpm 11.1.1 hardened ignored-build-script handling — install now fails with ERR_PNPM_IGNORED_BUILDS unless build scripts are explicitly approved. This was breaking the e2e webServer startup (pnpm install exit 1).

  • `pnpm-workspace.yaml`: drop the placeholder `allowBuilds:` block (had literal "set this to true or false" strings) in favor of `onlyBuiltDependencies`.
  • `package.json`: add `pnpm.onlyBuiltDependencies` for redundancy across pnpm config-key migrations.

Test plan

  • `pnpm install` clean (no ERR_PNPM_IGNORED_BUILDS)
  • `cd testing/e2e && npm test` → 275 passed, 66 skipped

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

pnpm 11.1.1 hardened ignored-build-script handling — install now fails
with ERR_PNPM_IGNORED_BUILDS unless build scripts are explicitly approved.
This was breaking the e2e webServer startup (pnpm install exit 1) and any
fresh clone trying to build the SPA.

- pnpm-workspace.yaml: drop the placeholder allowBuilds: block (had literal
  "set this to true or false" strings) in favor of onlyBuiltDependencies.
- package.json: add pnpm.onlyBuiltDependencies for redundancy across pnpm
  config-key migrations.

Verified: pnpm install clean, testing/e2e npm test → 275 passed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@MikeBengtson
MikeBengtson merged commit 39bba70 into main May 13, 2026
3 of 6 checks passed
@MikeBengtson
MikeBengtson deleted the fix/pnpm-onlybuilt-deps branch May 13, 2026 23:33
@MikeBengtson
MikeBengtson restored the fix/pnpm-onlybuilt-deps branch June 10, 2026 02:02
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