@zhixuan92/multi-model-agent-core 5.0.3 → 5.2.0

package/dist/lifecycle/findings-parser.js

@@ -1,143 +0,0 @@
-// v4.4.x — extracts structured findings from one read-route criterion
-// turn. Worker emits `## Finding N:` blocks per the format spec; this
-// parser converts them into StructuredReport.findings[] entries.
-//
-// Tolerant by design: LLMs vary heading level (## vs ###), heading noun
-// (Finding/Issue/Concern), terminator (colon/period/none), bullet marker
-// (- vs *), and bold wrapping (**Severity:**). Strict canonical format is
-// preferred, but the parser recovers from common drift so a worker mistake
-// becomes a degraded-but-usable result rather than silent data loss.
-// `## Finding 1:` (canonical) plus tolerated drift:
-//   - heading level 2–4 (##, ###, ####)
-//   - noun: Finding | Issue | Concern (singular only; reviewers use "Concern")
-//   - optional space, optional bold (**Finding 1:**)
-//   - terminator: `:`, `.`, `)`, or end-of-line
-// Uses [ \t]* (horizontal whitespace) so the regex doesn't cross newlines —
-// `\s*` matches newlines and would let `(.*)` consume the following bullet line.
-const FINDING_HEADING_RE = /^#{2,4}[ \t]*\**[ \t]*(?:Finding|Issue|Concern)[ \t]+(\d+)\**[ \t]*[:.)]?[ \t]*(.*)$/im;
-// Same shape but per-line for the block-splitter (no /m needed when matched line-by-line).
-const FINDING_HEADING_LINE_RE = /^#{2,4}[ \t]*\**[ \t]*(?:Finding|Issue|Concern)[ \t]+\d+\b/i;
-// `- Severity: high` plus tolerated drift: * bullet, **bold:**, no bullet at all.
-const bulletRe = (label) => new RegExp(`^(?:[-*][ \\t]+)?\\**[ \\t]*${label}[ \\t]*\\**[ \\t]*:[ \\t]*(.+)$`, 'im');
-const SEVERITY_VALUES = new Set(['critical', 'high', 'medium', 'low']);
-export function parseFindings(text, criterionId, legalOutcomes = ['found', 'clean', 'not_applicable'], warnSink = () => { }) {
-    if (!text || text.trim().length === 0) {
-        return { findings: [], outcome: 'clean' };
-    }
-    const blocks = [];
-    const lines = text.split('\n');
-    let current = [];
-    for (const line of lines) {
-        if (FINDING_HEADING_LINE_RE.test(line)) {
-            if (current.length > 0)
-                blocks.push(current.join('\n'));
-            current = [line];
-        }
-        else if (current.length > 0) {
-            current.push(line);
-        }
-    }
-    if (current.length > 0)
-        blocks.push(current.join('\n'));
-    const findings = [];
-    for (const block of blocks) {
-        const headingMatch = block.match(FINDING_HEADING_RE);
-        const headingNum = headingMatch?.[1] ?? '?';
-        const headingInline = headingMatch?.[2]?.trim() ?? '';
-        const headingText = `Finding ${headingNum}: ${headingInline}`;
-        // Claim can come from a `- Claim:` bullet (reviewer format) OR the inline
-        // text on the heading line (legacy worker format).
-        let claim = block.match(bulletRe('Claim'))?.[1]?.trim() ?? '';
-        if (!claim)
-            claim = headingInline;
-        if (claim.startsWith('[N/A]'))
-            continue;
-        // A finding with no claim text at all is useless downstream — drop it
-        // and surface a warning so the operator can spot worker-emission drift.
-        if (!claim || claim.trim().length === 0) {
-            warnSink('findings_parser_drop', {
-                route: criterionId,
-                droppedFindingHeading: headingText,
-                reasonCode: 'empty_claim',
-            });
-            continue;
-        }
-        // Emit warning if claim is empty, but continue processing with empty claim
-        if (!claim || claim.trim().length === 0) {
-            warnSink('findings_parser_drop', {
-                route: criterionId,
-                droppedFindingHeading: headingText,
-                reasonCode: 'empty_claim',
-            });
-            continue;
-        }
-        const sevRaw = block.match(bulletRe('Severity'))?.[1]?.trim().split(/\s+/)[0]?.toLowerCase();
-        // Emit warning if Severity is missing, but continue with default
-        if (!sevRaw) {
-            warnSink('findings_parser_drop', {
-                route: criterionId,
-                droppedFindingHeading: headingText,
-                reasonCode: 'missing_core_bullet',
-            });
-        }
-        // Emit warning if Severity is invalid, but continue with default
-        const severity = (sevRaw && SEVERITY_VALUES.has(sevRaw))
-            ? sevRaw
-            : 'medium';
-        if (sevRaw && !SEVERITY_VALUES.has(sevRaw)) {
-            warnSink('findings_parser_drop', {
-                route: criterionId,
-                droppedFindingHeading: headingText,
-                reasonCode: 'invalid_severity',
-            });
-        }
-        const category = block.match(bulletRe('Category'))?.[1]?.trim().split(/\s+/)[0] ?? criterionId;
-        const evidence = (block.match(bulletRe('Evidence'))?.[1] ?? block.match(bulletRe('Issue'))?.[1])?.trim();
-        const suggestion = (block.match(bulletRe('Suggestion'))?.[1] ?? block.match(bulletRe('Fix'))?.[1])?.trim();
-        // Drop for investigate routes if Evidence contains no file:line citation
-        // ANYWHERE. Workers naturally write `In [src/foo.ts:42] the function …` or
-        // wrap citations in markdown links — both forms now pass; only evidence
-        // that is pure prose with no path:line gets dropped.
-        if (criterionId.startsWith('investigate-') && evidence && !/[\w./-]+:\d+/.test(evidence)) {
-            warnSink('findings_parser_drop', {
-                route: criterionId,
-                droppedFindingHeading: headingText,
-                reasonCode: 'invalid_evidence_format',
-            });
-            continue;
-        }
-        const f = { severity, category, claim };
-        if (evidence)
-            f.evidence = evidence;
-        if (suggestion)
-            f.suggestion = suggestion;
-        findings.push(f);
-    }
-    // Extract outcome. Canonical: `## Outcome\nfound`. Tolerated drift:
-    //   `## Outcome: found` (inline), `### Outcome\nfound` (heading level),
-    //   `**Outcome:** found` (bold bullet), `Outcome: found` (no heading).
-    // Whichever form, value must be one of the legal-outcome enum literals.
-    let outcome = findings.length > 0 ? 'found' : 'clean';
-    // Horizontal-whitespace only on the heading line; newline before the value
-    // on the next line is explicit. Three alternatives covered:
-    //   1. `## Outcome\nfound` (heading + next-line value)
-    //   2. `## Outcome: found` (heading + inline value)
-    //   3. `Outcome: found` (bullet/bold/plain, no heading)
-    const OUTCOME_LINE_RE = /^(?:#{2,4}[ \t]*\**[ \t]*Outcome\**[ \t]*[:.]?[ \t]*(\w*)[ \t]*\n[ \t]*(\w*)|(?:[-*][ \t]+)?\**[ \t]*Outcome[ \t]*\**[ \t]*:[ \t]*(\w+))/im;
-    const m = text.match(OUTCOME_LINE_RE);
-    if (m) {
-        const raw = (m[1] || m[2] || m[3] || '').trim().toLowerCase();
-        if (raw === 'found' || raw === 'clean' || raw === 'not_applicable') {
-            outcome = raw;
-        }
-        // empty / unrecognized → keep the inferred outcome from findings.length
-    }
-    // Honor the route's legal-outcome set: if the worker declared a value that's
-    // illegal for this criterion (e.g. 'not_applicable' on an issue-hunting
-    // route), fall back to the inferred outcome rather than emit garbage.
-    if (!legalOutcomes.includes(outcome)) {
-        outcome = findings.length > 0 ? 'found' : (legalOutcomes[0] ?? 'clean');
-    }
-    return { findings, outcome };
-}
-//# sourceMappingURL=findings-parser.js.map

package/dist/lifecycle/findings-parser.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"findings-parser.js","sourceRoot":"","sources":["../../src/lifecycle/findings-parser.ts"],"names":[],"mappings":"AAAA,sEAAsE;AACtE,sEAAsE;AACtE,iEAAiE;AACjE,EAAE;AACF,wEAAwE;AACxE,yEAAyE;AACzE,0EAA0E;AAC1E,2EAA2E;AAC3E,qEAAqE;AAIrE,oDAAoD;AACpD,wCAAwC;AACxC,+EAA+E;AAC/E,qDAAqD;AACrD,gDAAgD;AAChD,4EAA4E;AAC5E,iFAAiF;AACjF,MAAM,kBAAkB,GAAG,wFAAwF,CAAC;AACpH,2FAA2F;AAC3F,MAAM,uBAAuB,GAAG,6DAA6D,CAAC;AAC9F,kFAAkF;AAClF,MAAM,QAAQ,GAAG,CAAC,KAAa,EAAE,EAAE,CACjC,IAAI,MAAM,CAAC,+BAA+B,KAAK,iCAAiC,EAAE,IAAI,CAAC,CAAC;AAmB1F,MAAM,eAAe,GAAG,IAAI,GAAG,CAAC,CAAC,UAAU,EAAE,MAAM,EAAE,QAAQ,EAAE,KAAK,CAAC,CAAC,CAAC;AAEvE,MAAM,UAAU,aAAa,CAC3B,IAAY,EACZ,WAAmB,EACnB,gBAAgD,CAAC,OAAO,EAAE,OAAO,EAAE,gBAAgB,CAAC,EACpF,WAAqB,GAAG,EAAE,GAAE,CAAC;IAE7B,IAAI,CAAC,IAAI,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACtC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC;IAC5C,CAAC;IAED,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,OAAO,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,uBAAuB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACvC,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC;gBAAE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;YACxD,OAAO,GAAG,CAAC,IAAI,CAAC,CAAC;QACnB,CAAC;aAAM,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC9B,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACrB,CAAC;IACH,CAAC;IACD,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC;QAAE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IAExD,MAAM,QAAQ,GAAc,EAAE,CAAC;IAC/B,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,MAAM,YAAY,GAAG,KAAK,CAAC,KAAK,CAAC,kBAAkB,CAAC,CAAC;QACrD,MAAM,UAAU,GAAG,YAAY,EAAE,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC;QAC5C,MAAM,aAAa,GAAG,YAAY,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;QACtD,MAAM,WAAW,GAAG,WAAW,UAAU,KAAK,aAAa,EAAE,CAAC;QAE9D,0EAA0E;QAC1E,mDAAmD;QACnD,IAAI,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;QAC9D,IAAI,CAAC,KAAK;YAAE,KAAK,GAAG,aAAa,CAAC;QAClC,IAAI,KAAK,CAAC,UAAU,CAAC,OAAO,CAAC;YAAE,SAAS;QAExC,sEAAsE;QACtE,wEAAwE;QACxE,IAAI,CAAC,KAAK,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACxC,QAAQ,CAAC,sBAAsB,EAAE;gBAC/B,KAAK,EAAE,WAAW;gBAClB,qBAAqB,EAAE,WAAW;gBAClC,UAAU,EAAE,aAAa;aAC1B,CAAC,CAAC;YACH,SAAS;QACX,CAAC;QAED,2EAA2E;QAC3E,IAAI,CAAC,KAAK,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACxC,QAAQ,CAAC,sBAAsB,EAAE;gBAC/B,KAAK,EAAE,WAAW;gBAClB,qBAAqB,EAAE,WAAW;gBAClC,UAAU,EAAE,aAAa;aAC1B,CAAC,CAAC;YACH,SAAS;QACX,CAAC;QAED,MAAM,MAAM,GAAG,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,CAAC;QAE7F,iEAAiE;QACjE,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,QAAQ,CAAC,sBAAsB,EAAE;gBAC/B,KAAK,EAAE,WAAW;gBAClB,qBAAqB,EAAE,WAAW;gBAClC,UAAU,EAAE,qBAAqB;aAClC,CAAC,CAAC;QACL,CAAC;QAED,iEAAiE;QACjE,MAAM,QAAQ,GAAwB,CAAC,MAAM,IAAI,eAAe,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;YAC3E,CAAC,CAAE,MAA8B;YACjC,CAAC,CAAC,QAAQ,CAAC;QACb,IAAI,MAAM,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;YAC3C,QAAQ,CAAC,sBAAsB,EAAE;gBAC/B,KAAK,EAAE,WAAW;gBAClB,qBAAqB,EAAE,WAAW;gBAClC,UAAU,EAAE,kBAAkB;aAC/B,CAAC,CAAC;QACL,CAAC;QAED,MAAM,QAAQ,GAAG,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,WAAW,CAAC;QAC/F,MAAM,QAAQ,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC;QACzG,MAAM,UAAU,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC;QAE3G,yEAAyE;QACzE,2EAA2E;QAC3E,wEAAwE;QACxE,qDAAqD;QACrD,IAAI,WAAW,CAAC,UAAU,CAAC,cAAc,CAAC,IAAI,QAAQ,IAAI,CAAC,cAAc,CAAC,IAAI,CAAC,QAAQ,CAAC,EAAE,CAAC;YACzF,QAAQ,CAAC,sBAAsB,EAAE;gBAC/B,KAAK,EAAE,WAAW;gBAClB,qBAAqB,EAAE,WAAW;gBAClC,UAAU,EAAE,yBAAyB;aACtC,CAAC,CAAC;YACH,SAAS;QACX,CAAC;QAED,MAAM,CAAC,GAAY,EAAE,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,CAAC;QACjD,IAAI,QAAQ;YAAE,CAAC,CAAC,QAAQ,GAAG,QAAQ,CAAC;QACpC,IAAI,UAAU;YAAE,CAAC,CAAC,UAAU,GAAG,UAAU,CAAC;QAC1C,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACnB,CAAC;IAED,oEAAoE;IACpE,wEAAwE;IACxE,uEAAuE;IACvE,wEAAwE;IACxE,IAAI,OAAO,GAAwB,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC;IAC3E,2EAA2E;IAC3E,4DAA4D;IAC5D,uDAAuD;IACvD,oDAAoD;IACpD,wDAAwD;IACxD,MAAM,eAAe,GAAG,4IAA4I,CAAC;IACrK,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;IACtC,IAAI,CAAC,EAAE,CAAC;QACN,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QAC9D,IAAI,GAAG,KAAK,OAAO,IAAI,GAAG,KAAK,OAAO,IAAI,GAAG,KAAK,gBAAgB,EAAE,CAAC;YACnE,OAAO,GAAG,GAA0B,CAAC;QACvC,CAAC;QACD,wEAAwE;IAC1E,CAAC;IAED,6EAA6E;IAC7E,wEAAwE;IACxE,sEAAsE;IACtE,IAAI,CAAC,aAAa,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;QACrC,OAAO,GAAG,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,OAAO,CAAC,CAAC;IAC1E,CAAC;IAED,OAAO,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;AAC/B,CAAC"}

package/dist/lifecycle/git-toplevel.d.ts

@@ -1,12 +0,0 @@
-export interface ResolveOptions {
-    timeoutMs?: number;
-}
-/**
- * Returns the canonical absolute path of the git toplevel for `cwd`, or
- * null if `cwd` is not in a git repo, git is unavailable, the directory
- * does not exist, or the spawn times out (default 5 s).
- *
- * Pure function of (cwd, filesystem state). Never throws.
- */
-export declare function resolveGitToplevel(cwd: string, opts?: ResolveOptions): Promise<string | null>;
-//# sourceMappingURL=git-toplevel.d.ts.map

package/dist/lifecycle/git-toplevel.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"git-toplevel.d.ts","sourceRoot":"","sources":["../../src/lifecycle/git-toplevel.ts"],"names":[],"mappings":"AAEA,MAAM,WAAW,cAAc;IAC7B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;;GAMG;AACH,wBAAsB,kBAAkB,CACtC,GAAG,EAAE,MAAM,EACX,IAAI,GAAE,cAAmB,GACxB,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAmCxB"}

package/dist/lifecycle/git-toplevel.js

@@ -1,52 +0,0 @@
-import { spawn } from 'node:child_process';
-/**
- * Returns the canonical absolute path of the git toplevel for `cwd`, or
- * null if `cwd` is not in a git repo, git is unavailable, the directory
- * does not exist, or the spawn times out (default 5 s).
- *
- * Pure function of (cwd, filesystem state). Never throws.
- */
-export async function resolveGitToplevel(cwd, opts = {}) {
-    const timeoutMs = opts.timeoutMs ?? 5000;
-    return new Promise((resolve) => {
-        let settled = false;
-        const settle = (value) => {
-            if (settled)
-                return;
-            settled = true;
-            resolve(value);
-        };
-        let child;
-        try {
-            child = spawn('git', ['-C', cwd, 'rev-parse', '--show-toplevel'], {
-                stdio: ['ignore', 'pipe', 'pipe'],
-                // windowsHide: suppress the console window Windows opens per spawned
-                // console binary when the daemon has no attached console. No-op on POSIX.
-                windowsHide: true,
-            });
-        }
-        catch {
-            settle(null);
-            return;
-        }
-        let stdout = '';
-        child.stdout?.on('data', (b) => { stdout += b.toString('utf8'); });
-        child.stderr?.on('data', () => { });
-        child.on('error', () => settle(null));
-        child.on('exit', (code) => {
-            if (code === 0)
-                settle(stdout.trim() || null);
-            else
-                settle(null);
-        });
-        const t = setTimeout(() => {
-            try {
-                child.kill('SIGKILL');
-            }
-            catch { /* ignore */ }
-            settle(null);
-        }, timeoutMs);
-        child.on('exit', () => clearTimeout(t));
-    });
-}
-//# sourceMappingURL=git-toplevel.js.map

package/dist/lifecycle/git-toplevel.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"git-toplevel.js","sourceRoot":"","sources":["../../src/lifecycle/git-toplevel.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAM3C;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,GAAW,EACX,OAAuB,EAAE;IAEzB,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,IAAI,IAAI,CAAC;IACzC,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE;QAC7B,IAAI,OAAO,GAAG,KAAK,CAAC;QACpB,MAAM,MAAM,GAAG,CAAC,KAAoB,EAAE,EAAE;YACtC,IAAI,OAAO;gBAAE,OAAO;YACpB,OAAO,GAAG,IAAI,CAAC;YACf,OAAO,CAAC,KAAK,CAAC,CAAC;QACjB,CAAC,CAAC;QACF,IAAI,KAAK,CAAC;QACV,IAAI,CAAC;YACH,KAAK,GAAG,KAAK,CAAC,KAAK,EAAE,CAAC,IAAI,EAAE,GAAG,EAAE,WAAW,EAAE,iBAAiB,CAAC,EAAE;gBAChE,KAAK,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,CAAC;gBACjC,qEAAqE;gBACrE,0EAA0E;gBAC1E,WAAW,EAAE,IAAI;aAClB,CAAC,CAAC;QACL,CAAC;QAAC,MAAM,CAAC;YACP,MAAM,CAAC,IAAI,CAAC,CAAC;YACb,OAAO;QACT,CAAC;QACD,IAAI,MAAM,GAAG,EAAE,CAAC;QAChB,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,MAAM,IAAI,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3E,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,GAAiB,CAAC,CAAC,CAAC;QAClD,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC;QACtC,KAAK,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE;YACxB,IAAI,IAAI,KAAK,CAAC;gBAAE,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,IAAI,IAAI,CAAC,CAAC;;gBACzC,MAAM,CAAC,IAAI,CAAC,CAAC;QACpB,CAAC,CAAC,CAAC;QACH,MAAM,CAAC,GAAG,UAAU,CAAC,GAAG,EAAE;YACxB,IAAI,CAAC;gBAAC,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YAAC,CAAC;YAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;YACrD,MAAM,CAAC,IAAI,CAAC,CAAC;QACf,CAAC,EAAE,SAAS,CAAC,CAAC;QACd,KAAK,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;AACL,CAAC"}

package/dist/lifecycle/handlers/annotate-stage.d.ts

@@ -1,35 +0,0 @@
-import type { LifecycleState } from '../stage-plan-types.js';
-import { type ResearchSourcesUsedEntry } from '../../reporting/report-parser-slots/research-report.js';
-import type { AnnotatePayload, StageGate } from '../stage-io.js';
-export interface StructuredReport {
-    summary: string;
-    workerStatus: 'done' | 'done_with_concerns' | 'blocked' | 'failed';
-    unresolved: string[];
-    filesChanged: string[];
-    reviewVerdict: 'approved' | 'changes_required' | null;
-    reviewConcerns: string[];
-    reworkApplied: boolean;
-    commitSha: string | null;
-    commitMessage: string | null;
-    commitSkipReason: string | null;
-    findings: {
-        severity: string;
-        category: string;
-        claim: string;
-        evidence?: string;
-        suggestion?: string;
-    }[];
-    criteriaErrors: {
-        criterionId: string;
-        error: string;
-    }[];
-    /** Research-only: parsed `## Sources used` markdown table. Absent on
-     *  every other route (audit/review/debug/investigate/write routes). */
-    sourcesUsed?: ResearchSourcesUsedEntry[];
-    findingsOutcome?: 'found' | 'clean' | 'not_applicable';
-    findingsOutcomeReason?: string | null;
-    outcomeInferred?: boolean;
-    outcomeMalformed?: boolean;
-}
-export declare function annotator(state: LifecycleState): Promise<StageGate<AnnotatePayload>>;
-//# sourceMappingURL=annotate-stage.d.ts.map

package/dist/lifecycle/handlers/annotate-stage.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"annotate-stage.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/annotate-stage.ts"],"names":[],"mappings":"AAmBA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAG7D,OAAO,EAEL,KAAK,wBAAwB,EAC9B,MAAM,wDAAwD,CAAC;AAGhE,OAAO,KAAK,EAAE,eAAe,EAAE,SAAS,EAAE,MAAM,gBAAgB,CAAC;AAIjE,MAAM,WAAW,gBAAgB;IAC/B,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,GAAG,oBAAoB,GAAG,SAAS,GAAG,QAAQ,CAAC;IACnE,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,aAAa,EAAE,UAAU,GAAG,kBAAkB,GAAG,IAAI,CAAC;IACtD,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,aAAa,EAAE,OAAO,CAAC;IACvB,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,gBAAgB,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,QAAQ,EAAE;QAAE,QAAQ,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC1G,cAAc,EAAE;QAAE,WAAW,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IACzD;2EACuE;IACvE,WAAW,CAAC,EAAE,wBAAwB,EAAE,CAAC;IACzC,eAAe,CAAC,EAAE,OAAO,GAAG,OAAO,GAAG,gBAAgB,CAAC;IACvD,qBAAqB,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtC,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,gBAAgB,CAAC,EAAE,OAAO,CAAC;CAC5B;AAED,wBAAsB,SAAS,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC,CA0T1F"}

package/dist/lifecycle/handlers/annotate-stage.js

@@ -1,360 +0,0 @@
-// v4.4.x — unified Annotating stage.
-//
-// LLM judge layer (spec §5.7.2), NOT a pure transform: this stage fires a
-// real LLM turn on the standard tier (see getSession('standard') below) for
-// both read and write routes. The LLM is the PROPOSER — it may set
-// completed/message/summary — while the deterministic parser
-// (applyAnnotatePreconditions) is the ENFORCER and the mechanical fields
-// (findings, filesChanged, commitSha) are always overridden from upstream
-// gates. It judges/summarizes the worker's emitted report; it does NOT
-// independently re-read the codebase to find defects (that is the Review
-// stage, which is write-routes-only). Falls back to mechanical synthesis only
-// on tier-3 prompt-budget truncation or after all transport retries fail.
-//
-// It assembles the canonical StructuredReport from state.lastRunResult, the
-// Review stage's verdict + concerns, the Rework flag, and the Committing
-// stage's outcome. Same handler for read and write routes — route-specific
-// fields hold empty arrays / nulls on the other side so the orchestrator
-// parses one shape.
-import { mergeStageStats } from '../merge-stage-stats.js';
-import { HUMAN_LABEL } from '../stage-labels.js';
-import { parseSourcesUsed, } from '../../reporting/report-parser-slots/research-report.js';
-import { applyAnnotatePreconditions } from '../annotate-parser.js';
-import { annotatePromptWrite, annotatePromptRead } from '../annotate-prompts.js';
-const READ_ROUTES = new Set(['audit', 'review', 'debug', 'investigate', 'research', 'journal-recall']);
-export async function annotator(state) {
-    const t0 = Date.now();
-    const last = (state.lastRunResult ?? {});
-    const route = state.route;
-    const isRead = !!route && READ_ROUTES.has(route);
-    const findings = last.findings ?? [];
-    const summary = last.summary
-        ?? (isRead ? `produced ${findings.length} findings` : (last.output ?? '').slice(0, 200));
-    // Commit fields are authoritative from the commit GATE payload — the commit
-    // handler writes the SHA/message/files THERE, not into state.lastRunResult
-    // (see git-commit-handler.ts header). Sourcing them from `last` left them
-    // perpetually null on the user-facing report, and `last.filesChanged` (a
-    // repo-wide diff) leaked sibling workers' files under concurrent dispatch.
-    // The committed gate's filesChanged is this worker's own pathspec.
-    const commitGate = state.gates?.['commit'];
-    const commitPayload = (commitGate?.payload ?? null);
-    const committed = commitGate?.outcome === 'advance' && commitPayload?.kind === 'committed';
-    const report = {
-        summary,
-        workerStatus: last.workerStatus ?? (isRead ? 'done' : 'failed'),
-        unresolved: last.unresolved ?? [],
-        filesChanged: isRead ? [] : (committed ? (commitPayload?.filesChanged ?? []) : (last.filesChanged ?? [])),
-        reviewVerdict: isRead ? null : ((state.reviewVerdict) ?? null),
-        reviewConcerns: isRead ? [] : ((state.reviewConcerns) ?? []),
-        reworkApplied: isRead ? false : Boolean(state.reworkApplied),
-        commitSha: isRead ? null : (committed ? (commitPayload?.commitSha ?? null) : null),
-        commitMessage: isRead ? null : (committed ? (commitPayload?.commitMessage ?? null) : null),
-        commitSkipReason: isRead || committed ? null : (commitPayload?.kind === 'no_op' ? (commitPayload?.reason ?? null) : null),
-        findings: isRead ? findings : [],
-        criteriaErrors: isRead ? (last.criteriaErrors ?? []) : [],
-    };
-    if (route === 'research') {
-        // Prefer the deterministic table derived from the EvidencePack (threaded
-        // onto lastRunResult by perform-implementation). The worker-emitted
-        // `## Sources used` markdown is an unreliable fallback — the per-criterion
-        // synthesis loop has no designated turn that emits it.
-        const fromPack = last.sourcesUsed;
-        report.sourcesUsed = (Array.isArray(fromPack) && fromPack.length > 0)
-            ? fromPack
-            : parseSourcesUsed(last.output ?? '');
-    }
-    report.findingsOutcome = last.findingsOutcome;
-    report.findingsOutcomeReason = last.findingsOutcomeReason;
-    report.outcomeInferred = last.outcomeInferred;
-    report.outcomeMalformed = last.outcomeMalformed;
-    state.structuredReport = report;
-    // v5: prompt-budget truncation per spec §14 assumption 7 + AC-30 to AC-33.
-    // Three tiers, applied progressively when config.truncateAnnotatePromptTier
-    // signals overflow:
-    //   tier 1 → strip Finding.evidence
-    //   tier 2 → also strip summary fields (set to '')
-    //   tier 3 → fallback mode: emit deterministic AnnotatePayload with verbatim
-    //            spec-text message; completed=false; findings passthrough.
-    const cfg = state.config ?? {};
-    const tier = cfg.truncateAnnotatePromptTier ?? 0;
-    const envelope = state.executionContext?.envelope;
-    const reviewGate = state.gates?.['review'];
-    const implementGate = state.gates?.['implement'];
-    const aggregatedFindings = [
-        ...(last.findings ?? []),
-        ...((reviewGate?.outcome === 'advance' ? reviewGate.payload?.findings : undefined) ?? []),
-        ...((implementGate?.outcome === 'advance' ? implementGate.payload?.findings : undefined) ?? []),
-    ];
-    // Dedupe by claim+evidence (preserve first occurrence)
-    const seen = new Set();
-    const dedupedFindings = [];
-    for (const f of aggregatedFindings) {
-        const key = `${f.claim ?? ''}|${f.evidence ?? ''}`;
-        if (seen.has(key))
-            continue;
-        seen.add(key);
-        dedupedFindings.push(f);
-    }
-    // Mechanical filesChanged + commitSha derivation (spec §5.7 rule 4).
-    // Reuses the `commitGate` resolved above.
-    const mechanicalFilesChanged = commitGate?.outcome === 'advance' &&
-        commitGate.payload?.kind === 'committed'
-        ? commitGate.payload.filesChanged ?? []
-        : (isRead ? [] : report.filesChanged);
-    const mechanicalCommitSha = commitGate?.outcome === 'advance' &&
-        commitGate.payload?.kind === 'committed'
-        ? (commitGate.payload.commitSha ?? null)
-        : (isRead ? null : (report.commitSha ?? null));
-    let truncatedFindings = dedupedFindings.slice();
-    let truncatedSummary = report.summary;
-    let fallbackMode = false;
-    if (tier >= 1) {
-        const droppedEvidenceCount = truncatedFindings.filter(f => f.evidence !== undefined).length;
-        truncatedFindings = truncatedFindings.map(f => {
-            const cp = { ...f };
-            delete cp.evidence;
-            return cp;
-        });
-        // Annotation truncation is now recorded as a validation warning in the envelope
-        const env = envelope;
-        env?.recordValidationWarning?.({ rule: 'AnnotateTruncationTier1', path: 'annotatePrompt' });
-    }
-    if (tier >= 2) {
-        const dropped = truncatedSummary ? 1 : 0;
-        truncatedSummary = '';
-        const env = envelope;
-        env?.recordValidationWarning?.({ rule: 'AnnotateTruncationTier2', path: 'annotatePrompt' });
-    }
-    if (tier >= 3) {
-        // Tier 3 = fallback mode per spec §5.7.3
-        fallbackMode = true;
-        const env = envelope;
-        env?.recordValidationWarning?.({ rule: 'AnnotateTruncationTier3', path: 'annotatePrompt' });
-    }
-    let annotated;
-    // LLM judge layer (spec §5.7.2). The annotator stage runs an LLM turn
-    // unless tier-3 truncation kicked us into fallback mode. The LLM is the
-    // proposer; the deterministic parser is the enforcer — applyAnnotatePreconditions
-    // always runs on the LLM-proposed payload and may flip completed=false. On
-    // transport error we fall back to mechanical synthesis (same path that ran
-    // before this layer was added).
-    let llmCostUSD = 0;
-    let llmDurationMs = 0;
-    let llmTurnsUsed = 0;
-    let llmTransportFailed = false;
-    let llmModel = null;
-    let llmInputTokens = 0;
-    let llmOutputTokens = 0;
-    let llmCachedReadTokens = 0;
-    let llmCachedNonReadTokens = 0;
-    if (fallbackMode) {
-        // Fallback findings: only gate-sourced findings flow through (gates.review,
-        // gates.implement). lastRunResult.findings are dropped as part of tier-3
-        // citation/non-gate-evidence clearing (spec §5.7.3 / AC-32).
-        const fallbackFindings = [];
-        const fbSeen = new Set();
-        const reviewFindings = (reviewGate?.outcome === 'advance' ? reviewGate.payload?.findings : undefined) ?? [];
-        const implementFindings = (implementGate?.outcome === 'advance' ? implementGate.payload?.findings : undefined) ?? [];
-        for (const f of [...reviewFindings, ...implementFindings]) {
-            const k = `${f.claim ?? ''}|${f.evidence ?? ''}`;
-            if (fbSeen.has(k))
-                continue;
-            fbSeen.add(k);
-            const cp = { ...f };
-            delete cp.evidence;
-            fallbackFindings.push(cp);
-        }
-        annotated = {
-            completed: false,
-            message: 'annotator prompt budget exceeded after tier-3 truncation; verdict computed mechanically from upstream gates',
-            findings: fallbackFindings,
-            summary: '',
-            filesChanged: mechanicalFilesChanged,
-            commitSha: mechanicalCommitSha,
-        };
-    }
-    else {
-        // Default mechanical proposal — used directly when LLM fails or returns
-        // unparseable output. The parser still gets the final say either way.
-        const mechanical = {
-            completed: true,
-            message: truncatedSummary || (isRead ? 'investigation completed' : 'task completed'),
-            findings: truncatedFindings,
-            summary: truncatedSummary,
-            filesChanged: mechanicalFilesChanged,
-            commitSha: mechanicalCommitSha,
-        };
-        let proposed = mechanical;
-        const ctx = state.executionContext;
-        // Annotator runs on the standard tier; capture its model from the provider
-        // config so the wire row attributes annotate cost+tokens to the real model.
-        // TurnResult does not carry model, so (r as any).model below is undefined
-        // for every provider — without this lookup the annotate stage's model was
-        // always null on the wire even when an LLM call actually fired.
-        const annotateModelFromConfig = ctx?.providers?.['standard']?.config?.model ?? null;
-        if (ctx && typeof ctx.getSession === 'function') {
-            const prompt = isRead ? annotatePromptRead(state) : annotatePromptWrite(state);
-            const session = ctx.getSession('standard');
-            let tres;
-            const t0 = Date.now();
-            // Retry on transport errors: 3 attempts with 0s, 1s, 2s backoff (AC-19, AC-20)
-            const retryDelays = [0, 1000, 2000];
-            let lastErr;
-            // Initialize to error; will be overwritten on success or last failure
-            tres = { kind: 'transport_error', message: 'annotator transport failed', ms: 0 };
-            for (let attempt = 0; attempt < retryDelays.length; attempt++) {
-                if (attempt > 0) {
-                    await new Promise(resolve => setTimeout(resolve, retryDelays[attempt]));
-                }
-                try {
-                    const r = await session.send(prompt, { stageLabel: HUMAN_LABEL.annotating });
-                    tres = {
-                        kind: 'ok',
-                        text: r.output ?? '',
-                        costUSD: typeof r.costUSD === 'number' ? r.costUSD : null,
-                        turnsUsed: r.turns ?? 1,
-                        ms: Date.now() - t0,
-                        model: r.model ?? null,
-                        inputTokens: r.usage?.inputTokens ?? 0,
-                        outputTokens: r.usage?.outputTokens ?? 0,
-                        cachedReadTokens: r.usage?.cachedReadTokens ?? 0,
-                        cachedNonReadTokens: r.usage?.cachedNonReadTokens ?? 0,
-                    };
-                    break; // Success, exit retry loop
-                }
-                catch (err) {
-                    lastErr = err instanceof Error ? err : new Error(String(err));
-                    // Update error message for next retry or final failure
-                    tres = { kind: 'transport_error', message: lastErr.message, ms: Date.now() - t0 };
-                }
-            }
-            if (tres.kind === 'ok') {
-                llmCostUSD = tres.costUSD ?? 0;
-                // TurnResult does not carry model — fall back to the provider config
-                // we captured above (tres.model is undefined for every provider).
-                llmModel = tres.model ?? annotateModelFromConfig;
-                llmDurationMs = tres.ms;
-                llmTurnsUsed = tres.turnsUsed;
-                llmInputTokens = tres.inputTokens;
-                llmOutputTokens = tres.outputTokens;
-                llmCachedReadTokens = tres.cachedReadTokens;
-                llmCachedNonReadTokens = tres.cachedNonReadTokens;
-                const parsed = extractAnnotateJson(tres.text);
-                if (parsed) {
-                    // Authoritative fields come from upstream gates (mechanicalFilesChanged,
-                    // mechanicalCommitSha, dedupedFindings). The LLM gets to propose
-                    // completed + message + summary; everything mechanical we override.
-                    proposed = {
-                        completed: typeof parsed.completed === 'boolean' ? parsed.completed : mechanical.completed,
-                        message: typeof parsed.message === 'string' && parsed.message.length > 0 ? parsed.message : mechanical.message,
-                        summary: typeof parsed.summary === 'string' ? parsed.summary : mechanical.summary,
-                        findings: truncatedFindings,
-                        filesChanged: mechanicalFilesChanged,
-                        commitSha: mechanicalCommitSha,
-                    };
-                }
-                // parsed === null → keep mechanical (parser unaffected)
-            }
-            else {
-                // transport_error — fall through to mechanical synthesis
-                llmTransportFailed = true;
-                llmDurationMs = tres.ms;
-                // Transport errors are now recorded as validation warnings
-                const env = envelope;
-                env?.recordValidationWarning?.({ rule: 'AnnotateLLMTransportError', path: 'annotatePrompt' });
-            }
-        }
-        annotated = applyAnnotatePreconditions(proposed, state);
-    }
-    state.annotatePayload = annotated;
-    mergeStageStats(state, 'annotating', {
-        inputTokens: llmInputTokens,
-        outputTokens: llmOutputTokens,
-        cachedReadTokens: llmCachedReadTokens,
-        cachedNonReadTokens: llmCachedNonReadTokens,
-        turnCount: llmTurnsUsed,
-        costUSD: llmCostUSD,
-        durationMs: Date.now() - t0,
-        filesWrittenCount: 0,
-    }, {
-        tier: llmTurnsUsed > 0 ? 'standard' : null,
-        model: llmModel,
-        // Mirror the outcome the implementer (or reviewer) computed so the
-        // annotating stage row carries the same value downstream consumers see.
-        findingsOutcome: last.findingsOutcome,
-        findingsOutcomeReason: last.findingsOutcomeReason,
-        outcomeInferred: last.outcomeInferred,
-        outcomeMalformed: last.outcomeMalformed,
-    });
-    // suppress unused-variable warning while keeping the durationMs field for
-    // diagnostics callers; mergeStageStats already captured the value above.
-    void llmDurationMs;
-    void llmTransportFailed;
-    const annotatingStats = state.lastRunResult
-        ?.stageStats?.annotating;
-    if (annotatingStats) {
-        annotatingStats.outcome = 'transformed';
-        annotatingStats.maxIdleMs = 0;
-        annotatingStats.totalIdleMs = 0;
-    }
-    return {
-        outcome: 'advance',
-        payload: annotated,
-        telemetry: {
-            stageLabel: 'annotate',
-            durationMs: Date.now() - t0,
-            costUSD: llmCostUSD,
-            turnsUsed: llmTurnsUsed,
-            stopReason: 'normal',
-        },
-    };
-}
-/**
- * Extract a single JSON object from the LLM's response.
- *
- * Accepts:
- *   - A fenced ```json …``` block (preferred form per annotate prompts)
- *   - A fenced ``` …``` block (no language tag)
- *   - The first balanced `{…}` substring in the text
- *
- * Returns `null` if no parseable JSON is found, signalling the caller to keep
- * the mechanical proposal. This is intentional: a malformed LLM response
- * should not cause stage failure — the parser already has a deterministic
- * answer it can fall back on.
- */
-function extractAnnotateJson(text) {
-    if (!text)
-        return null;
-    const fencedJson = text.match(/```(?:json)?\s*\n([\s\S]*?)\n```/i);
-    const candidate = fencedJson ? fencedJson[1] : firstBalancedBraces(text);
-    if (!candidate)
-        return null;
-    try {
-        const parsed = JSON.parse(candidate);
-        if (parsed && typeof parsed === 'object' && !Array.isArray(parsed)) {
-            return parsed;
-        }
-    }
-    catch {
-        /* fall through */
-    }
-    return null;
-}
-function firstBalancedBraces(text) {
-    const start = text.indexOf('{');
-    if (start < 0)
-        return null;
-    let depth = 0;
-    for (let i = start; i < text.length; i++) {
-        const ch = text[i];
-        if (ch === '{')
-            depth++;
-        else if (ch === '}') {
-            depth--;
-            if (depth === 0)
-                return text.slice(start, i + 1);
-        }
-    }
-    return null;
-}
-//# sourceMappingURL=annotate-stage.js.map