From 39e670035ada65ad292aef17c6abdc576a51f663 Mon Sep 17 00:00:00 2001 From: garrytan Date: Sat, 3 Oct 2026 02:05:39 +0000 Subject: [PATCH] test: integration repairs after F2 (calibration test renders the AUQ format; census counts and release floors without the reverted overlay case) --- test/eng-finding-retry-budget.test.ts | 2 +- test/paid-retry-supervision.test.ts | 4 ++-- test/plan-review-calibration.test.ts | 4 +++- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/test/eng-finding-retry-budget.test.ts b/test/eng-finding-retry-budget.test.ts index f3966fcde..7ee125f7f 100644 --- a/test/eng-finding-retry-budget.test.ts +++ b/test/eng-finding-retry-budget.test.ts @@ -139,7 +139,7 @@ test('live periodic census fits the declared CI wall including setup', () => { test('registered allocation is deterministic and preserves every discovered file', () => { const files = collectPaidTestFiles(); - expect(files).toHaveLength(107); + expect(files).toHaveLength(106); expect(files).toContain('test/skill-e2e-ship-skip.test.ts'); const m = livePlan(files); expect(livePlan([...files].reverse())).toEqual(m); diff --git a/test/paid-retry-supervision.test.ts b/test/paid-retry-supervision.test.ts index d512cd329..b5de034b3 100644 --- a/test/paid-retry-supervision.test.ts +++ b/test/paid-retry-supervision.test.ts @@ -229,8 +229,8 @@ test('detached PR fallback and release commands cover their actual default worke )) / 1000 * 1.05)); } const detachedReleaseWall = Number(scripts['eval:bg:release'].match(/--timeout (\d+)/)?.[1]) * 1000; - expect(releaseFloors).toEqual([21_725, 33_852]); - expect(releaseFloors.reduce((total, floor) => total + floor, 0)).toBe(55_577); + expect(releaseFloors).toEqual([21_725, 33_821]); + expect(releaseFloors.reduce((total, floor) => total + floor, 0)).toBe(55_546); expect(detachedReleaseWall).toBe(116_700_000); expect(detachedReleaseWall).toBeGreaterThanOrEqual(releaseWall + 120_000); }); diff --git a/test/plan-review-calibration.test.ts b/test/plan-review-calibration.test.ts index e151b1a6d..e4337b482 100644 --- a/test/plan-review-calibration.test.ts +++ b/test/plan-review-calibration.test.ts @@ -5,9 +5,11 @@ import * as path from 'node:path'; import { planDecisionCalibrations } from './fixtures/plan-decision-classification'; import { buildPlanReviewDecisionPrompt } from './helpers/plan-review-decisions'; import { ENG_BATCHING_FINDINGS } from './helpers/plan-review-cases'; +import { generateAskUserFormat } from '../scripts/resolvers/preamble/generate-ask-user-format'; +import { HOST_PATHS, type TemplateContext } from '../scripts/resolvers/types'; const ROOT = path.resolve(import.meta.dir, '..'); test('calibration briefs preserve source-required structure and actual choices without phase/qid reliance', () => { - const format = fs.readFileSync(path.join(ROOT, 'scripts/resolvers/preamble/generate-ask-user-format.ts'), 'utf8'); + const format = generateAskUserFormat({ skillName: 'plan-ceo-review', host: 'claude', paths: HOST_PATHS.claude } as TemplateContext); const split = fs.readFileSync(path.join(ROOT, 'docs/askuserquestion-split.md'), 'utf8'); for (const token of ['Project/branch/task:', 'ELI10:', 'Stakes if we pick wrong:', 'Recommendation:', 'Pros / cons:', 'Net:']) expect(format).toContain(token); for (const token of ['Include', 'Defer', 'Cut', 'Hold']) expect(split).toContain(token);