From 5d24e2e7cd9af457a5dd03d693bfea06aa728f44 Mon Sep 17 00:00:00 2001 From: Zoltan Kochan Date: Thu, 21 May 2026 21:28:56 +0200 Subject: [PATCH] ci(pacquet): drop --filter so pn -r can call the hidden .test script Adding --filter=pnpm re-enabled pnpm's hidden-script guard and broke the invocation with ERR_PNPM_HIDDEN_SCRIPT. Recursive mode bypasses that guard on its own; --if-present silently skips workspace packages that don't define .test, so the unfiltered form does the right thing. Written by an agent (Claude Code, claude-opus-4-7). --- .github/workflows/pacquet-ci.yml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/pacquet-ci.yml b/.github/workflows/pacquet-ci.yml index 09bccb0e8c..d3b9c21417 100644 --- a/.github/workflows/pacquet-ci.yml +++ b/.github/workflows/pacquet-ci.yml @@ -169,11 +169,11 @@ jobs: - name: Run pnpm e2e against pacquet env: PNPM_E2E_BIN: ${{ github.workspace }}/target/release/pacquet - # Invoke pnpm's hidden `.test` script via `pn -r` exactly the way - # test.yml does for the standard e2e job — that's the call shape the - # suite is known to pass on. Filtering to pnpm keeps us scoped to - # this package's tests only. - run: pnpm --filter=pnpm -r .test + # `pn -r .test` is the same call shape test.yml uses for the standard + # e2e job. Recursive mode bypasses the hidden-script check; explicit + # `--filter=pnpm` re-enables it, so leave it off and let `--if-present` + # silently skip the workspace packages that don't define `.test`. + run: pnpm --no-sort --workspace-concurrency=1 --if-present -r .test doc: name: Doc