From b31455f1c33fceb8ae384aa7aa1f25cb86f9b5c1 Mon Sep 17 00:00:00 2001 From: Jason Ross Date: Thu, 2 Jul 2026 14:02:04 -0500 Subject: [PATCH] #7 Approve protobufjs build script so pnpm install exits 0 @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 --- pnpm-workspace.yaml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/pnpm-workspace.yaml b/pnpm-workspace.yaml index c14b867..6f2ce73 100644 --- a/pnpm-workspace.yaml +++ b/pnpm-workspace.yaml @@ -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