peaks-loop 4.0.53 → 4.1.0

package/dist/services/qa/bdd-test-style-verifier.d.ts

@@ -39,30 +39,7 @@
  *   malformed test files. A parse error is a structural problem; the
  *   caller must surface it.
  */
-/** Structured failure reasons the verifier can return. */
-export type BddStyleFailureReason = 'missing-given-when-then' | 'description-no-should-when';
-/** Successful verdict — includes the count of inspected `it`/`test` calls. */
-export interface BddStyleOk {
-    readonly ok: true;
-    readonly scanned: number;
-}
-/** Structured failure verdict — the file/line makes the rejection actionable. */
-export interface BddStyleFail {
-    readonly ok: false;
-    readonly reason: BddStyleFailureReason;
-    readonly file: string;
-    readonly line: number;
-    /** For `description-no-should-when`: the original description. */
-    readonly description?: string;
-    /** For `missing-given-when-then`: a stable string the caller can compare. */
-    readonly expected?: string;
-}
-export type BddStyleVerdict = BddStyleOk | BddStyleFail;
-/** Public input surface — keep small so the contract is hard to misuse. */
-export interface VerifyBddStyleInput {
-    readonly projectRoot: string;
-    readonly testFiles: readonly string[];
-}
+import { type BddStyleVerdict, type VerifyBddStyleInput } from './bdd-test-style-contract.js';
 /**
  * Verify that every `it(...)` / `test(...)` call in the given test
  * files follows the BDD given-when-then contract.
@@ -86,3 +63,4 @@ export interface VerifyBddStyleInput {
  * caller maps to `qa-handoff` rejection.
  */
 export declare function verifyBddStyle(input: VerifyBddStyleInput): BddStyleVerdict;
+export type { BddStyleFailureReason, BddStyleOk, BddStyleFail, BddStyleVerdict, VerifyBddStyleInput } from './bdd-test-style-contract.js';

package/dist/services/qa/bdd-test-style-verifier.js

@@ -42,6 +42,7 @@
 import { readFileSync } from 'node:fs';
 import { resolve } from 'node:path';
 import ts from 'typescript';
+import { matchesBddTriple, hasWhenOrShould } from './bdd-test-style-contract.js';
 /** Test runners whose first string-arg is the test description. */
 const TEST_NAMES = new Set(['it', 'test']);
 /**
@@ -129,7 +130,7 @@ function checkDescription(call, sourceFile, relPath) {
             file: relPath,
             line: line + 1,
             description: '<non-literal first argument>',
-            expected: 'first argument must be a string literal containing "when" or "should"',
+            expected: 'first argument must be a string literal containing "when" or "should"'
         };
     }
     const description = firstArg.text;
@@ -142,7 +143,7 @@ function checkDescription(call, sourceFile, relPath) {
             file: relPath,
             line: line + 1,
             description,
-            expected: 'description must contain the word "when" or "should" (BDD style)',
+            expected: 'description must contain the word "when" or "should" (BDD style)'
         };
     }
     return null;
@@ -182,7 +183,7 @@ function checkBody(call, sourceFile, relPath) {
         reason: 'missing-given-when-then',
         file: relPath,
         line: line + 1,
-        expected: 'block-body callback with // given: / // when: / // then: comments at the top',
+        expected: 'block-body callback with // given: / // when: / // then: comments at the top'
     };
 }
 function getCallbackBlock(call) {
@@ -236,33 +237,6 @@ function makeMissingCommentFailure(block, sourceFile, relPath) {
         reason: 'missing-given-when-then',
         file: relPath,
         line: line + 1,
-        expected: '// given: / // when: / // then: triple at the top of the block body',
+        expected: '// given: / // when: / // then: triple at the top of the block body'
     };
 }
-/**
- * Match the three leading comments against the BDD triple. Each
- * entry must be a `// <keyword>:` line (with optional trailing
- * whitespace); the keywords must appear in `given`, `when`, `then`
- * order, case-insensitive.
- */
-function matchesBddTriple(triple) {
-    if (triple.length !== 3)
-        return false;
-    // Each entry must be a `// <keyword>:` line, optionally followed
-    // by descriptive text. The Slice A migrator's `buildCommentBlock`
-    // produces `// given: the test setup` / `// when:  the function
-    // under test is invoked` / `// then:  the result matches the
-    // expectation` — the `when` line uses two spaces after the colon
-    // for visual alignment with `given:` and `then:`, so the regex
-    // is intentionally permissive about trailing text.
-    const patterns = [
-        /^\s*\/\/\s*given\s*:/i,
-        /^\s*\/\/\s*when\s*:/i,
-        /^\s*\/\/\s*then\s*:/i,
-    ];
-    return patterns.every((pat, i) => pat.test(triple[i] ?? ''));
-}
-/** True when `text` contains `when` or `should` as a whole word. */
-function hasWhenOrShould(text) {
-    return /(\bwhen\b|\bshould\b)/i.test(text);
-}

package/dist/services/qa/browser-event-logger.js

@@ -47,7 +47,8 @@ export class BrowserEventLogger {
         try {
             appendFileSync(this.filePath, line + '\n');
         }
-        catch { // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
+        catch {
+            // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
             /* best-effort; never block the CLI on a log write */
         }
     }

package/dist/services/qa/qa-business-review-state.js

@@ -20,11 +20,26 @@ import { isUnsafePathInput } from '../../shared/path-safety.js';
 import { REQUEST_ID_PATTERN } from '../artifacts/request-artifact-service.js';
 /** 6-item business checklist (12 Gaps QA perspective). */
 export const QA_BUSINESS_ITEMS = [
-    { id: 'business-flow', question: '这个功能"用起来"对吗?(业务流程顺不顺,操作路径是否反人类,跟现有系统交互有没有断层)' },
-    { id: 'req-coverage', question: '老板提的需求都覆盖了吗?(大需求点 vs 小需求点,显式需求 vs 隐式需求)' },
-    { id: 'boundary-cases', question: '边界 case 跟业务预期一致吗?(异常输入 / 错误提示用户语言 / 空加载失败状态)' },
-    { id: 'ui-assembly', question: '页面模式 / 关键交互 / 信息密度 跟产品预期一致吗?(装配验收,非视觉设计)' },
-    { id: 'exception-tone', question: '异常态 / 边界态视觉跟产品语调一致吗?(空状态有引导 / 加载态不闪烁 / 失败态有重试入口)' },
+    {
+        id: 'business-flow',
+        question: '这个功能"用起来"对吗?(业务流程顺不顺,操作路径是否反人类,跟现有系统交互有没有断层)'
+    },
+    {
+        id: 'req-coverage',
+        question: '老板提的需求都覆盖了吗?(大需求点 vs 小需求点,显式需求 vs 隐式需求)'
+    },
+    {
+        id: 'boundary-cases',
+        question: '边界 case 跟业务预期一致吗?(异常输入 / 错误提示用户语言 / 空加载失败状态)'
+    },
+    {
+        id: 'ui-assembly',
+        question: '页面模式 / 关键交互 / 信息密度 跟产品预期一致吗?(装配验收,非视觉设计)'
+    },
+    {
+        id: 'exception-tone',
+        question: '异常态 / 边界态视觉跟产品语调一致吗?(空状态有引导 / 加载态不闪烁 / 失败态有重试入口)'
+    },
     { id: 'mergeable', question: '能合入下个小版本吗?(业务风险 / 用户体验风险 / 集成断点)' }
 ];
 export function buildEmptyQaReview(requestId, sessionId, now = new Date()) {

package/dist/services/qa/screenshot-archive-service.js

@@ -92,9 +92,7 @@ export function archiveScreenshots(options) {
             archivedAt: now.toISOString()
         };
     }
-    const targetContentsBefore = existsSync(targetDir)
-        ? readdirSync(targetDir)
-        : [];
+    const targetContentsBefore = existsSync(targetDir) ? readdirSync(targetDir) : [];
     mkdirSync(targetDir, { recursive: true });
     const candidates = listScreenshotFiles(sourceDir, targetDir);
     const moved = [];

package/dist/services/rd/ast-gate.js

@@ -59,13 +59,13 @@ export async function runAstGate(input) {
                     api: name,
                     expectedVersion: depVersion,
                     actualVersion: 'unknown', // could resolve via npm view if needed
-                    severity: 'error',
+                    severity: 'error'
                 });
             }
         }
     }
     return {
         passed: violations.length === 0,
-        violations,
+        violations
     };
 }

package/dist/services/rd/impl.js

@@ -39,7 +39,7 @@ export async function writeImpl(input) {
         inputSig: input.inputSig,
         changedFiles: [...input.changedFiles],
         externalApiCalls: [...input.externalApiCalls],
-        astGateResult: input.astGate,
+        astGateResult: input.astGate
     };
     const sha256 = sha256Of(partial);
     const final = { ...partial, sha256 };

package/dist/services/rd/rd-service.js

@@ -53,7 +53,7 @@ function normalizeGoal(goal) {
     return normalized;
 }
 function isClearLowRiskGoal(goal) {
-    return /^fix\b/i.test(goal) && /\b(typo|spelling|comment|docs?|test|lint|format|copy)\b/i.test(goal);
+    return (/^fix\b/i.test(goal) && /\b(typo|spelling|comment|docs?|test|lint|format|copy)\b/i.test(goal));
 }
 const MIN_SAFE_SWARM_WORKERS = 25;
 const MAX_SAFE_SWARM_WORKERS = 80;
@@ -70,35 +70,71 @@ function resolveWorkerTarget(maxWorkers) {
     return { workerTarget: maxWorkers, blockedReasons: [] };
 }
 function resolveArtifactWorkspacePath(request) {
-    return request.artifactWorkspacePath ?? (request.workspace ? getLocalArtifactPath(request.workspace) : undefined);
+    return (request.artifactWorkspacePath ??
+        (request.workspace ? getLocalArtifactPath(request.workspace) : undefined));
 }
 function hasPlannerArtifactWorkspace(request, artifactWorkspacePath) {
-    return !!request.workspace && !!artifactWorkspacePath && hasValidArtifactWorkspace(request.workspace, artifactWorkspacePath);
+    return (!!request.workspace &&
+        !!artifactWorkspacePath &&
+        hasValidArtifactWorkspace(request.workspace, artifactWorkspacePath));
 }
 function buildTaskIds(workerTarget) {
     const fixed = [
-        'rd-discovery-1', 'rd-discovery-2', 'rd-discovery-3', 'rd-discovery-4', 'rd-discovery-5', 'rd-discovery-6', 'rd-discovery-7', 'rd-discovery-8',
-        'rd-planning-1', 'rd-planning-2', 'rd-planning-3', 'rd-planning-4', 'rd-planning-5', 'rd-planning-6', 'rd-planning-7', 'rd-planning-8',
-        'rd-test-1', 'rd-test-2', 'rd-test-3',
-        'peaks-qa-1', 'peaks-qa-2', 'peaks-qa-3', 'peaks-qa-4',
-        'rd-reducer-1',
+        'rd-discovery-1',
+        'rd-discovery-2',
+        'rd-discovery-3',
+        'rd-discovery-4',
+        'rd-discovery-5',
+        'rd-discovery-6',
+        'rd-discovery-7',
+        'rd-discovery-8',
+        'rd-planning-1',
+        'rd-planning-2',
+        'rd-planning-3',
+        'rd-planning-4',
+        'rd-planning-5',
+        'rd-planning-6',
+        'rd-planning-7',
+        'rd-planning-8',
+        'rd-test-1',
+        'rd-test-2',
+        'rd-test-3',
+        'peaks-qa-1',
+        'peaks-qa-2',
+        'peaks-qa-3',
+        'peaks-qa-4',
+        'rd-reducer-1'
     ];
     const implementationCount = Math.max(workerTarget - fixed.length, 1);
     const implementation = Array.from({ length: implementationCount }, (_, index) => `rd-impl-${String(index + 1).padStart(RD_TASK_ID_PAD_WIDTH, '0')}`);
-    return [...fixed.slice(0, RD_FIXED_TASK_BOUNDARY), ...implementation, ...fixed.slice(RD_FIXED_TASK_BOUNDARY)];
+    return [
+        ...fixed.slice(0, RD_FIXED_TASK_BOUNDARY),
+        ...implementation,
+        ...fixed.slice(RD_FIXED_TASK_BOUNDARY)
+    ];
 }
 function isInsidePath(childPath, parentPath) {
     const relativePath = relative(parentPath, childPath);
     return relativePath === '' || (!relativePath.startsWith('..') && !isAbsolute(relativePath));
 }
 function isSafeRepoRelativePath(candidate) {
-    if (!candidate || candidate.includes('\\') || candidate.startsWith('/') || /^[A-Za-z]:/.test(candidate) || candidate.includes(':') || isAbsolute(candidate)) {
+    if (!candidate ||
+        candidate.includes('\\') ||
+        candidate.startsWith('/') ||
+        /^[A-Za-z]:/.test(candidate) ||
+        candidate.includes(':') ||
+        isAbsolute(candidate)) {
         return false;
     }
-    return candidate.split('/').every((segment) => segment.length > 0 && segment !== '.' && segment !== '..');
+    return candidate
+        .split('/')
+        .every((segment) => segment.length > 0 && segment !== '.' && segment !== '..');
 }
 function extractCandidatePath(line) {
-    const trimmed = line.trim().replace(/^[-*+]\s+/, '').trim();
+    const trimmed = line
+        .trim()
+        .replace(/^[-*+]\s+/, '')
+        .trim();
     const codeSpan = trimmed.match(/^`([^`]+)`(?:\s|$)/);
     const firstToken = trimmed.split(/\s+/)[0];
     const candidate = codeSpan === null ? firstToken : codeSpan[1];
@@ -126,7 +162,9 @@ function selectConcreteTargetArea(targetAreas, index) {
     return targetAreas[index % targetAreas.length];
 }
 function getTaskModelRole(wave) {
-    return wave === 'implementation candidates' || wave === 'unit-test execution' ? 'execution' : 'strongest';
+    return wave === 'implementation candidates' || wave === 'unit-test execution'
+        ? 'execution'
+        : 'strongest';
 }
 function getTaskModelId(modelRole, executionModelId, strongestModelId) {
     return modelRole === 'execution' ? executionModelId : strongestModelId;
@@ -139,11 +177,16 @@ function readArtifactFile(rootPath, artifactWorkspacePath, artifact) {
         const rootRealPath = stableRealPath(rootPath);
         const rdRootPath = resolve(rootPath, '..');
         const sessionRootPath = resolve(rdRootPath, '..');
-        if (lstatSync(sessionRootPath).isSymbolicLink() || lstatSync(rdRootPath).isSymbolicLink() || lstatSync(rootPath).isSymbolicLink() || !isInsidePath(rootRealPath, artifactWorkspaceRealPath)) {
+        if (lstatSync(sessionRootPath).isSymbolicLink() ||
+            lstatSync(rdRootPath).isSymbolicLink() ||
+            lstatSync(rootPath).isSymbolicLink() ||
+            !isInsidePath(rootRealPath, artifactWorkspaceRealPath)) {
             return null;
         }
         const artifactStat = lstatSync(artifactPath);
-        if (artifactStat.isSymbolicLink() || !artifactStat.isFile() || artifactStat.size > MAX_ARTIFACT_BYTES) {
+        if (artifactStat.isSymbolicLink() ||
+            !artifactStat.isFile() ||
+            artifactStat.size > MAX_ARTIFACT_BYTES) {
             return null;
         }
         if (!isInsidePath(stableRealPath(artifactPath), rootRealPath)) {
@@ -153,7 +196,12 @@ function readArtifactFile(rootPath, artifactWorkspacePath, artifact) {
         try {
             const openedStat = fstatSync(fd);
             const currentStat = statSync(artifactPath);
-            if (!openedStat.isFile() || openedStat.size > MAX_ARTIFACT_BYTES || openedStat.dev !== artifactStat.dev || openedStat.ino !== artifactStat.ino || openedStat.dev !== currentStat.dev || openedStat.ino !== currentStat.ino) {
+            if (!openedStat.isFile() ||
+                openedStat.size > MAX_ARTIFACT_BYTES ||
+                openedStat.dev !== artifactStat.dev ||
+                openedStat.ino !== artifactStat.ino ||
+                openedStat.dev !== currentStat.dev ||
+                openedStat.ino !== currentStat.ino) {
                 return null;
             }
             const buffer = Buffer.alloc(MAX_ARTIFACT_BYTES + 1);
@@ -171,7 +219,8 @@ function readArtifactFile(rootPath, artifactWorkspacePath, artifact) {
             closeSync(fd);
         }
     }
-    catch { // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
+    catch {
+        // TODO(g2): legacy silent catch — grace: 1 minor release (v2.14.0)
         return null;
     }
 }
@@ -188,7 +237,9 @@ function resolveExecutionModelId() {
     }
 }
 function getConcreteTargetAreas(request, artifactWorkspacePath, hasApprovedTechArtifacts) {
-    if (!artifactWorkspacePath || !hasApprovedTechArtifacts || !hasPlannerArtifactWorkspace(request, artifactWorkspacePath)) {
+    if (!artifactWorkspacePath ||
+        !hasApprovedTechArtifacts ||
+        !hasPlannerArtifactWorkspace(request, artifactWorkspacePath)) {
         return [];
     }
     // Slice 2026-06-29-change-id-root-removal: read path now resolves via
@@ -213,15 +264,16 @@ function buildStandardsOverlay(request) {
     if (!request.projectRoot) {
         return {};
     }
-    const check = resolveRdStartupStandardsCheck({ projectRoot: request.projectRoot, strict: request.strictStandards === true });
+    const check = resolveRdStartupStandardsCheck({
+        projectRoot: request.projectRoot,
+        strict: request.strictStandards === true
+    });
     // Slice 2026-06-16-peaks-rd-no-gates — Repair cycle 1:
     // thread BOTH diagnostic and errorCode so strict-mode callers still
     // surface `EPEAKS_NO_STANDARDS`. The pre-fix version early-returned
     // on `diagnostic !== null` and dropped `errorCode` (QA#4 AC3 violation).
     const overlay = { standardsGates: check.gates };
-    const withDiagnostic = check.diagnostic !== null
-        ? { ...overlay, standardsDiagnostic: check.diagnostic }
-        : overlay;
+    const withDiagnostic = check.diagnostic !== null ? { ...overlay, standardsDiagnostic: check.diagnostic } : overlay;
     return check.errorCode !== null
         ? { ...withDiagnostic, standardsErrorCode: check.errorCode }
         : withDiagnostic;
@@ -241,7 +293,7 @@ function buildPlan(request, standardsOverlay) {
     const techStatus = getTechStatus({
         sessionId: request.sessionId,
         ...(artifactWorkspacePath ? { artifactWorkspacePath } : {}),
-        ...(request.workspace ? { workspace: request.workspace } : {}),
+        ...(request.workspace ? { workspace: request.workspace } : {})
     });
     const requiresTechApproval = request.requiresTechApproval ?? !isClearLowRiskGoal(goal);
     const techGateSkipped = !requiresTechApproval;
@@ -259,15 +311,17 @@ function buildPlan(request, standardsOverlay) {
                 taskGraph: 'rd/swarm/task-graph.json',
                 waveManifests: [],
                 workerBriefs: [],
-                reducerReport: 'rd/swarm/reducer-report.md',
+                reducerReport: 'rd/swarm/reducer-report.md'
             },
             gateStatus: withStandardsOverlay({
                 techApprovalRequired: requiresTechApproval,
                 techStatus: techStatus.status,
-                ...(techGateSkipped ? { skipReason: 'tech-gate-skipped-clear-implementation-path' } : {}),
+                ...(techGateSkipped
+                    ? { skipReason: 'tech-gate-skipped-clear-implementation-path' }
+                    : {})
             }, standardsOverlay),
             blockedReasons,
-            nextActions: [],
+            nextActions: []
         };
     }
     if (requiresTechApproval && techStatus.status !== 'approved') {
@@ -284,14 +338,16 @@ function buildPlan(request, standardsOverlay) {
                 taskGraph: 'rd/swarm/task-graph.json',
                 waveManifests: [],
                 workerBriefs: [],
-                reducerReport: 'rd/swarm/reducer-report.md',
+                reducerReport: 'rd/swarm/reducer-report.md'
             },
             gateStatus: withStandardsOverlay({
                 techApprovalRequired: true,
-                techStatus: techStatus.status,
+                techStatus: techStatus.status
             }, standardsOverlay),
             blockedReasons: ['tech-approval-required', ...blockedReasons],
-            nextActions: ['Run peaks tech plan --dry-run and approve the tech plan before running peaks swarm plan.'],
+            nextActions: [
+                'Run peaks tech plan --dry-run and approve the tech plan before running peaks swarm plan.'
+            ]
         };
     }
     if (blockedReasons.includes('worker-count-below-target')) {
@@ -308,15 +364,19 @@ function buildPlan(request, standardsOverlay) {
                 taskGraph: 'rd/swarm/task-graph.json',
                 waveManifests: [],
                 workerBriefs: [],
-                reducerReport: 'rd/swarm/reducer-report.md',
+                reducerReport: 'rd/swarm/reducer-report.md'
             },
             gateStatus: withStandardsOverlay({
                 techApprovalRequired: requiresTechApproval,
                 techStatus: techStatus.status,
-                ...(techGateSkipped ? { skipReason: 'tech-gate-skipped-clear-implementation-path' } : {}),
+                ...(techGateSkipped
+                    ? { skipReason: 'tech-gate-skipped-clear-implementation-path' }
+                    : {})
             }, standardsOverlay),
             blockedReasons,
-            nextActions: ['Lower max-workers to match the current change scope or accept the capped target.'],
+            nextActions: [
+                'Lower max-workers to match the current change scope or accept the capped target.'
+            ]
         };
     }
     const taskIds = buildTaskIds(workerTarget);
@@ -333,7 +393,7 @@ function buildPlan(request, standardsOverlay) {
         { name: 'implementation candidates', taskIds: [...implementationTaskIds] },
         { name: 'unit-test execution', taskIds: [...unitTestTaskIds] },
         { name: 'quality gates', taskIds: [...qualityTaskIds] },
-        { name: 'reducer', taskIds: [...reducerTaskIds] },
+        { name: 'reducer', taskIds: [...reducerTaskIds] }
     ];
     const waveDependencies = {
         discovery: [],
@@ -341,10 +401,20 @@ function buildPlan(request, standardsOverlay) {
         'implementation candidates': planningTaskIds,
         'unit-test execution': implementationTaskIds,
         'quality gates': unitTestTaskIds,
-        reducer: qualityTaskIds,
+        reducer: qualityTaskIds
     };
     const tasks = taskIds.map((taskId, index) => {
-        const wave = index < RD_DISCOVERY_PLANNING_WAVE_SIZE ? 'discovery' : index < RD_IMPLEMENTATION_BAND_START ? 'planning' : index < taskIds.length - RD_DISCOVERY_PLANNING_WAVE_SIZE ? 'implementation candidates' : index < taskIds.length - RD_QUALITY_GATE_BAND_SIZE ? 'unit-test execution' : index < taskIds.length - 1 ? 'quality gates' : 'reducer';
+        const wave = index < RD_DISCOVERY_PLANNING_WAVE_SIZE
+            ? 'discovery'
+            : index < RD_IMPLEMENTATION_BAND_START
+                ? 'planning'
+                : index < taskIds.length - RD_DISCOVERY_PLANNING_WAVE_SIZE
+                    ? 'implementation candidates'
+                    : index < taskIds.length - RD_QUALITY_GATE_BAND_SIZE
+                        ? 'unit-test execution'
+                        : index < taskIds.length - 1
+                            ? 'quality gates'
+                            : 'reducer';
         const briefPath = `rd/swarm/workers/${taskId}/brief.md`;
         const implementationIndex = index - RD_IMPLEMENTATION_BAND_START;
         const targetArea = wave === 'implementation candidates' && hasConcreteTargetAreas(concreteTargetAreas)
@@ -371,14 +441,14 @@ function buildPlan(request, standardsOverlay) {
                         ? `${taskId}-test-command-result.md`
                         : wave === 'quality gates'
                             ? `${taskId}-qa-review.md`
-                            : `${taskId}.md`,
+                            : `${taskId}.md`
         };
     });
     const conflictGroups = waves.map((wave) => ({
         groupId: `group-${wave.name.replace(/\s+/g, '-')}`,
         ownedPaths: wave.taskIds.map((taskId) => `rd/swarm/workers/${taskId}/brief.md`),
         parallelismPolicy: wave.taskIds.length > 1 ? 'parallel' : 'sequential',
-        reason: `${wave.name} work is isolated by worker output path`,
+        reason: `${wave.name} work is isolated by worker output path`
     }));
     return {
         sessionId: request.sessionId,
@@ -393,15 +463,19 @@ function buildPlan(request, standardsOverlay) {
             taskGraph: 'rd/swarm/task-graph.json',
             waveManifests: waves.map((_wave, index) => `rd/swarm/waves/wave-${index + 1}-${_wave.name}.json`),
             workerBriefs: tasks.map((task) => task.outputs[0]),
-            reducerReport: 'rd/swarm/reducer-report.md',
+            reducerReport: 'rd/swarm/reducer-report.md'
         },
         gateStatus: withStandardsOverlay({
             techApprovalRequired: requiresTechApproval,
             techStatus: techStatus.status,
-            ...(techGateSkipped ? { skipReason: 'tech-gate-skipped-clear-implementation-path' } : {}),
+            ...(techGateSkipped
+                ? { skipReason: 'tech-gate-skipped-clear-implementation-path' }
+                : {})
         }, standardsOverlay),
         blockedReasons,
-        nextActions: blockedReasons.length > 0 ? ['Lower max-workers to match the current change scope or accept the capped target.'] : [],
+        nextActions: blockedReasons.length > 0
+            ? ['Lower max-workers to match the current change scope or accept the capped target.']
+            : []
     };
 }
 export function createRdSwarmPlan(request) {
@@ -425,7 +499,7 @@ export function createRdSwarmPlan(request) {
             outputs: result.outputs,
             gateStatus: result.gateStatus,
             blockedReasons: result.blockedReasons,
-            nextActions: [...WORKSPACE_UNAVAILABLE_NEXT_ACTIONS],
+            nextActions: [...WORKSPACE_UNAVAILABLE_NEXT_ACTIONS]
         };
     }
     const blockedReason = result.blockedReasons[0];
@@ -443,11 +517,11 @@ export function createRdSwarmPlan(request) {
             outputs: result.outputs,
             gateStatus: result.gateStatus,
             blockedReasons: result.blockedReasons,
-            nextActions: result.nextActions,
+            nextActions: result.nextActions
         };
     }
     return {
         available: true,
-        ...result,
+        ...result
     };
 }

package/dist/services/rd/reviewer-dispatch-policy.d.ts

@@ -26,12 +26,33 @@
  * PRD handoff + the project-scoped audit templates (slice v2.12.0
  * Tier 1+2+3 = Group A).
  *
- * The dispatch policy here is the **canonical decision table** consumed
- * by the LLM-side runner and pinned by tests:
- *   - `tests/unit/rd/karpathy-skip-on-config-docs-chore.test.ts`
- *     (5 → 3 element pinning)
- *   - `tests/unit/rd/deprecated-reviewer-back-compat.test.ts`
- *     (NEW in v2.12.0 — 8 cases; back-compat for the 2 removed slots)
+ * This module holds the machine-readable decision table for that collapse;
+ * the prose the LLM runner actually reads is
+ * `skills/bee/peaks-rd/references/parallel-review-fanout.md`. It used to
+ * describe itself as "consumed by the LLM-side runner", which named no
+ * mechanism — an LLM reads prose, not TypeScript exports — and that is the
+ * reason every export below sat at zero importers until slice F2.
+ *
+ * Pinning is half-done. The two test files this block used to cite —
+ * `tests/unit/rd/karpathy-skip-on-config-docs-chore.test.ts` (the 5 → 3
+ * element pinning) and
+ * `tests/unit/rd/deprecated-reviewer-back-compat.test.ts` (the 8
+ * back-compat cases) — were both deleted in `f17aa377`. The **predicate
+ * half** was re-pinned by `tests/unit/rd/reviewer-dispatch-policy.test.ts`
+ * (`7191140f`, 6 cases: `RD_DEPRECATED_REVIEWERS` + `isDeprecatedReviewer`).
+ * The **decision-table half** — `RD_FANOUT_REVIEWERS`' 3-element
+ * membership, `reviewerListFor`, `karpathySlotIndex`,
+ * `shouldDispatchKarpathy` — still has no pin.
+ *
+ * Slice F2 (rid-f2-ac1-wiring) gave this module its FIRST caller in
+ * `src/cli/commands/sub-agent-shared.ts` (`deprecatedReviewerWarnings`),
+ * invoked from the dispatch chokepoint in
+ * `src/cli/commands/dispatch-commands.ts` and pinned by
+ * `tests/unit/cli/sub-agent-dispatch-deprecated-reviewer.test.ts`. Before
+ * that, all 13 exports had zero importers across src/ + packages/ +
+ * scripts/, so nothing rejected or rerouted `security-reviewer` /
+ * `perf-baseline-reviewer` on the way in. That import is still the only
+ * one: the other 12 exports below remain unreferenced in this repo.
  *
  * For `config | docs | chore` request types, the slice already skips
  * the entire fanout (SKILL.md line 132 says "Config / docs / chore: no
@@ -47,8 +68,15 @@
  * back-compat window. The `isDeprecatedReviewer(name)` predicate lets
  * dispatchers (or legacy on-disk rd/{security-review,perf-baseline}.md
  * readers) detect a removed slot and route to the new audit skill
- * instead of failing the gate. See Tier 5 (`artifact-prerequisites.ts`)
- * for the matching prereq-side back-compat (`mustContainAny` form).
+ * instead of failing the gate. Tier 5 (`artifact-prerequisites.ts`)
+ * holds the matching prereq-side back-compat: `AUDIT_SECURITY` /
+ * `AUDIT_PERF` accept the legacy `rd/security-review.md` /
+ * `rd/perf-baseline.md` artifacts via `legacyRelativePaths`.
+ *
+ * Both halves are wired as of slice F2 and both ACCEPT the legacy slot.
+ * The dispatch side emits the reroute notice as a warning
+ * (`deprecatedReviewerWarnings`) rather than a refusal, precisely so it
+ * does not disagree with the prereq side about the same deprecation.
  */
 export declare const RD_REVIEW_REQUEST_TYPES: readonly ["feat", "bugfix", "refactor", "config", "docs", "chore"];
 export type RdReviewRequestType = (typeof RD_REVIEW_REQUEST_TYPES)[number];