From c5036a6681e6ec2afdbc8634e55322344bf5befc Mon Sep 17 00:00:00 2001 From: garrytan Date: Sat, 31 Oct 2026 19:07:03 +0000 Subject: [PATCH] test(review-pass): the GST-482 worktree-brief case runs the Red Team through --critical-found (GSTA-167) On a first pass at full depth with no review log, PR A cuts the standalone Red Team, so the case created two readers and its '> 2' guard failed. The count only stood for 'the Red Team stage renders too'. --critical-found keeps that stage in the fixture, and the case now names all three readers; every brief must still end with the worktree tail. Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> --- test/review-pass.test.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/review-pass.test.ts b/test/review-pass.test.ts index 9a391431d..80829bc30 100644 --- a/test/review-pass.test.ts +++ b/test/review-pass.test.ts @@ -540,11 +540,12 @@ describe('readers on a shared machine review the checkout the pass ships (GST-48 fs.mkdirSync(out, { recursive: true }); fs.writeFileSync(path.join(out, 'coverage.prompt.md'), 'Run Step 7 as written.\n'); const h = host(dir, env); - const res = await runPass(h, { base: 'main', out, pass: 'w3p1', cwd: wt, planFlags: ['--depth-floor', 'full'], + const res = await runPass(h, { base: 'main', out, pass: 'w3p1', cwd: wt, planFlags: ['--depth-floor', 'full', '--critical-found'], extras: [{ name: 'coverage', prompt_file: path.join(out, 'coverage.prompt.md'), first: true }] }); expect(res).not.toContain('STOP:'); expect(res).toContain('NEXT: judge (Phase 4)'); expect(h.creates.length).toBeGreaterThan(2); + expect(h.creates.map(c => c.title)).toEqual(['ship w3p1 audit/coverage', 'ship w3p1 specialist-batched', 'ship w3p1 red-team']); for (const c of h.creates) expect(c.prompt.endsWith('\n\n' + withWorkspaceTail(wt)), c.title).toBe(true); expect(bin(wt, env, ['verify', '--ledger', path.join(out, 'review-ledger.jsonl')]).stdout).not.toContain('wrong_tree'); }, 240_000);