Skip to content

chore: migrate build-script allowlist to pnpm-workspace.yaml for pnpm 11#627

Open
keturahaivision wants to merge 1 commit into
opensource-construction:mainfrom
keturahaivision:chore/pnpm11-build-allowlist
Open

chore: migrate build-script allowlist to pnpm-workspace.yaml for pnpm 11#627
keturahaivision wants to merge 1 commit into
opensource-construction:mainfrom
keturahaivision:chore/pnpm11-build-allowlist

Conversation

@keturahaivision

Copy link
Copy Markdown

What

Adds pnpm-workspace.yaml to frontend/ and backend/, declaring the trusted native build dependencies (esbuild, @tailwindcss/oxide) via allowBuilds.

Why

pnpm 11 stopped reading the build-script allowlist from the package.json pnpm.onlyBuiltDependencies field — it now reads it from pnpm-workspace.yaml. On pnpm 11, pnpm install prints:

[WARN] The "pnpm" field in package.json is no longer read by pnpm. The following keys were ignored: "pnpm.onlyBuiltDependencies".

and defers the native build scripts (esbuild, @tailwindcss/oxide), so pnpm run build fails (Vite / Tailwind can't find their binaries) until a user manually runs pnpm approve-builds.

Fix

Declaring the same trusted builds in pnpm-workspace.yaml (the location pnpm 11 reads) restores a clean, non-interactive pnpm install + pnpm run build. Verified locally on pnpm 11.9 / Node 25 — pnpm install runs the esbuild + oxide postinstalls and pnpm run build completes and writes to public/.

Config only — no runtime or dependency-version changes.

🤖 Generated with Claude Code

pnpm 11 no longer reads the build-script allowlist from the package.json
"pnpm.onlyBuiltDependencies" field; it reads it from pnpm-workspace.yaml.
Without this, pnpm install on pnpm 11 defers the native build scripts
(esbuild, @tailwindcss/oxide) and the app fails to build until the user
runs pnpm approve-builds.

Add pnpm-workspace.yaml to frontend and backend declaring the trusted
build dependencies via allowBuilds so installs stay green on pnpm 11.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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