JMR-dev/LibreMail-Bug-Report-Ingest
1
0
Fork 0
Code Issues 1 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

#7 Approve protobufjs build script so pnpm install exits 0

Browse Source 
@usebruno/cli (the API-test runner added in this PR) pulls in protobufjs,
whose postinstall build script pnpm 11 leaves un-approved by default. That
makes `pnpm install` exit non-zero (ERR_PNPM_IGNORED_BUILDS), which also
aborts `pnpm exec` / `pnpm run` via their verify-deps-before-run precheck
and would break CI's pnpm install once this lands on main.

Add protobufjs to the existing allowBuilds allowlist in pnpm-workspace.yaml
(same mechanism already used for esbuild/sharp/workerd). With it, pnpm
install exits 0 and the Bruno OpenCollection YAML suite runs green through
the pnpm wrapper (pnpm exec bru run / pnpm run test:api), 8/8 tests passing
against `go run ./cmd/devserver`.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
JMR-dev
2026-07-02 14:02:04 -05:00
co-authored by Claude Opus 4.8
parent 4c22056e66
commit b31455f1c3
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pnpm-workspace.yaml
+7 -3
View File
@@ -1,7 +1,11 @@
# Approve build (postinstall) scripts for wrangler's toolchain deps.
# These are first-party Cloudflare / well-known packages required by
# `wrangler dev`/`deploy`; they are not needed for `go test` or the dev server.
# Approve build (postinstall) scripts for dev-tooling deps.
# esbuild/sharp/workerd are first-party Cloudflare / well-known packages required
# by `wrangler dev`/`deploy`; protobufjs is a transitive dependency of
# `@usebruno/cli` (the API-test runner). None are needed for `go test` or the dev
# server. Without approving protobufjs, pnpm 11 exits non-zero on the ignored
# build script, which would break CI's `pnpm install` and `pnpm exec`/`pnpm run`.
allowBuilds:
esbuild: true
protobufjs: true
sharp: true
workerd: true