Skip to content

Commit 7675bb0

Browse files
committed
ci(docker): drop unused pnpm setup that broke setup-node@v5 caching
The Build <function>-fn matrix jobs were failing in "Post Setup Node.js" with: Path Validation Error: Path(s) specified in the action for caching do(es) not exist, hence no cache is being saved. Root cause: pnpm/action-setup@v6 ran before actions/setup-node@v5, exposing PNPM_HOME. setup-node@v5 then auto-detected pnpm via the packageManager field in root package.json and enabled pnpm-store caching — but these jobs never run pnpm install (they only invoke node --experimental-strip-types scripts/generate.ts). The store path never gets created, and v5 promotes the missing-path warning to an error during cache save. Fix: remove pnpm/action-setup from docker.yaml since the workflow doesn't use pnpm on the runner. The actual function builds run inside Docker, which installs pnpm itself via "npm install -g pnpm@<v>". simple-email had been passing only by accident (cache hit from a previous warm run); the other three were deterministic failures.
1 parent 070c727 commit 7675bb0

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

.github/workflows/docker.yaml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,12 @@ jobs:
5656
- name: Checkout
5757
uses: actions/checkout@v5
5858

59-
- name: Setup pnpm
60-
uses: pnpm/action-setup@v6
61-
59+
# No pnpm/action-setup here: this workflow only runs
60+
# `node --experimental-strip-types scripts/generate.ts` on the
61+
# runner (zero deps); the actual builds happen inside Docker.
62+
# Including pnpm makes setup-node@v5 auto-detect it via
63+
# packageManager + PNPM_HOME and try to save a cache for an
64+
# empty store, which fails with "Path Validation Error".
6265
- name: Setup Node.js
6366
uses: actions/setup-node@v5
6467
with:

0 commit comments

Comments
 (0)