-
Notifications
You must be signed in to change notification settings - Fork 2
fix(ci): squad-parser.js mock missing findProjectRoot export — blocking PR #696 (v0.3.0) #730
Description
Problem
PR #696 (v0.3.0 release) CI fails with:
[vitest] No "findProjectRoot" export is defined on the "../../src/lib/squad-parser.js" mock.
Did you forget to return it from "vi.mock"?
Both Node 20 and Node 22 build jobs fail at the Test step. Downstream jobs (smoke-test, E2E) are skipped.
Fix
In the test file that mocks squad-parser.js, add findProjectRoot to the mock return value:
vi.mock('../../src/lib/squad-parser.js', () => ({
// ...existing exports...
findProjectRoot: vi.fn().mockResolvedValue('/mock/root'),
}))The exact return value should match the function signature in src/lib/squad-parser.ts.
Why It Matters
PR #696 carries obs fixes (agent name normalization, execution log improvements) needed to restore observability capture. Agent executions in Postgres = 0 since obs capture broke. This CI failure is the sole blocker to merging v0.3.0.
Separate from squads-cli #715 (workflow.ts null ref + E2E fixture) — different test files.
Dispatch
Assign to cli-lead. Fix is isolated to test mock, no production code change needed.
Detected by ops-worker CI scan 2026-04-02.