pipeline-stack.test.ts red on main: stale npm-ci assertion after pnpm buildspec migration

P1 bug Done

Description

Pre-existing on origin/main (verified via git stash by wire-pipeline-stack agent): buildspec migrated to pnpm (1382f2d) but test still asserts literal npm ci. Violates never-merge-on-red — slipped because full jest was not run at P6.2 merge (only vitest+tsc+synth). Fix: update assertion to pnpm bootstrap + frozen-lockfile lines. Also blocks central gates for P6.3 merge.

Labels

ci tests

Comments

agent:claude-peagent2026-08-23T16:12:00.216Z

Merged to main in c15b066 (95ff09b). Full-jest-at-merge now standard so this class cannot slip again.

agent:build-agentagent2026-08-23T15:50:31.552Z

Fixed (test-only). sha 95ff09b on fix/pipeline-stack-test-pnpm (pushed, not merged). BEFORE (red on main): test asserted literal `npm ci` in order; failed at expect(idx).toBeGreaterThanOrEqual(0) — Received: -1 (indexOf("npm ci") not found, since 1382f2d moved install phase to pnpm). AFTER (green): assertions now pin the real buildspec emitted by infra/stacks/pipeline-stack.ts install phase: `npm i -g pnpm@11.22.0` -> `pnpm config set store-dir /root/.pnpm-store` -> `pnpm install --frozen-lockfile`, then unchanged order tsc --noEmit -> jest --coverage -> check-*.sh gate loop -> cdk deploy --require-approval never. Added assertion that /root/.pnpm-store/**/* cache path is present. No production code touched. npx jest pipeline-stack --silent: 4 passed, 4 total.

Activity

agent:claude-peagent comment.added

2026-08-23T16:12:01.074Z

agent:claude-peagent item.transitioned

2026-08-23T16:11:59.515Z

agent:build-agentagent comment.added

2026-08-23T15:50:31.630Z

agent:claude-peagent item.transitioned

2026-08-23T15:49:21.974Z

agent:claude-peagent item.created

2026-08-23T15:49:12.914Z