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

package/dist/tools/retry/tool-config.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-config.d.ts","sourceRoot":"","sources":["../../../src/tools/retry/tool-config.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,6CAA6C,CAAC;AAElF,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,EAAkB,KAAK,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAClE,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,sCAAsC,CAAC;AAoBvE,wBAAgB,aAAa,CAAC,QAAQ,EAAE,mBAAmB,GAAG,IAAI,CAYjE;AAED,eAAO,MAAM,UAAU,EAAE,UAAU,CAAC,KAAK,EAAE,UAAU,EAAE,OAAO,CA0C7D,CAAC"}

package/dist/tools/retry/tool-config.js

@@ -1,79 +0,0 @@
-import { inputSchema } from './schema.js';
-import { retryBriefSlot } from './brief-slot.js';
-import { noStructuredReportSchema } from '../../reporting/report-parser-slots/no-structured-report.js';
-import { makeFindingsHeadlineTemplate } from '../../reporting/findings-headline.js';
-import { notApplicable } from '../../reporting/not-applicable.js';
-import { implementGoalPrompt } from '../../lifecycle/goal-prompts.js';
-const RETRY_PREAMBLE = [
-    'RETRY: the plan below was partially executed in a prior run. The commits since the run',
-    'start (see `git log`) show what is already done. Do NOT redo committed, correct work —',
-    'complete what is missing and fix what is wrong, committing with the same `[task N]` convention.',
-    '',
-].join('\n');
-/** Thrown when the prior batch's goal has been evicted from the cache. */
-class GoalNotFoundError extends Error {
-    code = 'goal_not_found';
-    constructor(batchId) { super(`no stored goal for batch ${batchId}`); this.name = 'GoalNotFoundError'; }
-}
-export function registerRetry(registry) {
-    registry.register({
-        routeName: 'retry_tasks',
-        httpMethod: 'POST',
-        httpPath: '/retry',
-        surface: 'tool',
-        schema: inputSchema,
-        toolCategory: 'assist',
-        agentTypeDefault: 'standard',
-        agentTypeOverridable: false,
-        responseShapeName: 'BatchResponse',
-    });
-}
-export const toolConfig = {
-    name: 'retry',
-    category: 'assist',
-    dispatchMode: 'serial',
-    dispatchModeOverridable: false,
-    agentType: 'standard',
-    briefSlot: retryBriefSlot,
-    buildTaskSpec: (brief, ctx) => {
-        const batch = ctx.projectContext?.batchCache?.get(brief.batchId);
-        const origTask = batch?.tasks[0];
-        if (!origTask?.goal)
-            throw new GoalNotFoundError(brief.batchId);
-        // Re-fire the same goal-set; the prepare stage captures a fresh baseSha
-        // (current HEAD) so the re-run continues from where the prior run left off.
-        return {
-            ...origTask,
-            prompt: RETRY_PREAMBLE + implementGoalPrompt(origTask.goal),
-        };
-    },
-    reportSchema: noStructuredReportSchema,
-    headlineTemplate: makeFindingsHeadlineTemplate('retry', 'high'),
-    postProcessEnvelope: (envelope, ctx) => {
-        const results = (Array.isArray(envelope.results) ? envelope.results : []);
-        const total = results.length;
-        let ok = 0, incomplete = 0, error = 0;
-        for (const r of results) {
-            const s = r?.status;
-            if (s === 'ok')
-                ok++;
-            else if (s === 'error')
-                error++;
-            else
-                incomplete++;
-        }
-        const aggregate = error > 0 ? 'error' : incomplete > 0 ? 'incomplete' : 'ok';
-        let detail = `${ok}/${total} tasks complete`;
-        if (incomplete > 0)
-            detail += `, ${incomplete} incomplete`;
-        if (error > 0)
-            detail += `, ${error} error`;
-        envelope.headline = `[${aggregate}] retry: ${detail}`;
-        envelope.structuredReport = notApplicable('no structured report emitted by this executor');
-        if (ctx?.batchId) {
-            envelope.retryBatchId = ctx.batchId;
-        }
-        return envelope;
-    },
-};
-//# sourceMappingURL=tool-config.js.map

package/dist/tools/retry/tool-config.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-config.js","sourceRoot":"","sources":["../../../src/tools/retry/tool-config.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAE1C,OAAO,EAAE,cAAc,EAAmB,MAAM,iBAAiB,CAAC;AAGlE,OAAO,EAAE,wBAAwB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,EAAE,4BAA4B,EAAE,MAAM,sCAAsC,CAAC;AACpF,OAAO,EAAE,aAAa,EAAE,MAAM,mCAAmC,CAAC;AAClE,OAAO,EAAE,mBAAmB,EAAE,MAAM,iCAAiC,CAAC;AAEtE,MAAM,cAAc,GAAG;IACrB,wFAAwF;IACxF,wFAAwF;IACxF,iGAAiG;IACjG,EAAE;CACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,0EAA0E;AAC1E,MAAM,iBAAkB,SAAQ,KAAK;IAC1B,IAAI,GAAG,gBAAgB,CAAC;IACjC,YAAY,OAAe,IAAI,KAAK,CAAC,4BAA4B,OAAO,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,GAAG,mBAAmB,CAAC,CAAC,CAAC;CAChH;AAED,MAAM,UAAU,aAAa,CAAC,QAA6B;IACzD,QAAQ,CAAC,QAAQ,CAAC;QAChB,SAAS,EAAE,aAAa;QACxB,UAAU,EAAE,MAAM;QAClB,QAAQ,EAAE,QAAQ;QAClB,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,WAAW;QACnB,YAAY,EAAE,QAAQ;QACtB,gBAAgB,EAAE,UAAU;QAC5B,oBAAoB,EAAE,KAAK;QAC3B,iBAAiB,EAAE,eAAe;KACnC,CAAC,CAAC;AACL,CAAC;AAED,MAAM,CAAC,MAAM,UAAU,GAA2C;IAChE,IAAI,EAAE,OAAO;IACb,QAAQ,EAAE,QAAQ;IAClB,YAAY,EAAE,QAAQ;IACtB,uBAAuB,EAAE,KAAK;IAC9B,SAAS,EAAE,UAAU;IACrB,SAAS,EAAE,cAAc;IACzB,aAAa,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE;QAC5B,MAAM,KAAK,GAAG,GAAG,CAAC,cAAc,EAAE,UAAU,EAAE,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QACjE,MAAM,QAAQ,GAAG,KAAK,EAAE,KAAK,CAAC,CAAC,CAAyB,CAAC;QACzD,IAAI,CAAC,QAAQ,EAAE,IAAI;YAAE,MAAM,IAAI,iBAAiB,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAChE,wEAAwE;QACxE,4EAA4E;QAC5E,OAAO;YACL,GAAG,QAAQ;YACX,MAAM,EAAE,cAAc,GAAG,mBAAmB,CAAC,QAAQ,CAAC,IAAI,CAAC;SAC5D,CAAC;IACJ,CAAC;IACD,YAAY,EAAE,wBAAwB;IACtC,gBAAgB,EAAE,4BAA4B,CAAC,OAAO,EAAE,MAAM,CAAC;IAC/D,mBAAmB,EAAE,CAAC,QAAQ,EAAE,GAAG,EAAE,EAAE;QACrC,MAAM,OAAO,GAAG,CAAC,KAAK,CAAC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAuB,CAAC;QAChG,MAAM,KAAK,GAAG,OAAO,CAAC,MAAM,CAAC;QAC7B,IAAI,EAAE,GAAG,CAAC,EAAE,UAAU,GAAG,CAAC,EAAE,KAAK,GAAG,CAAC,CAAC;QACtC,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;YACxB,MAAM,CAAC,GAAG,CAAC,EAAE,MAAM,CAAC;YACpB,IAAI,CAAC,KAAK,IAAI;gBAAE,EAAE,EAAE,CAAC;iBAChB,IAAI,CAAC,KAAK,OAAO;gBAAE,KAAK,EAAE,CAAC;;gBAC3B,UAAU,EAAE,CAAC;QACpB,CAAC;QACD,MAAM,SAAS,GACb,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,UAAU,GAAG,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC;QAC7D,IAAI,MAAM,GAAG,GAAG,EAAE,IAAI,KAAK,iBAAiB,CAAC;QAC7C,IAAI,UAAU,GAAG,CAAC;YAAE,MAAM,IAAI,KAAK,UAAU,aAAa,CAAC;QAC3D,IAAI,KAAK,GAAG,CAAC;YAAE,MAAM,IAAI,KAAK,KAAK,QAAQ,CAAC;QAC5C,QAAQ,CAAC,QAAQ,GAAG,IAAI,SAAS,YAAY,MAAM,EAAE,CAAC;QACtD,QAAQ,CAAC,gBAAgB,GAAG,aAAa,CAAC,+CAA+C,CAAC,CAAC;QAC3F,IAAI,GAAG,EAAE,OAAO,EAAE,CAAC;YACjB,QAAQ,CAAC,YAAY,GAAG,GAAG,CAAC,OAAO,CAAC;QACtC,CAAC;QACD,OAAO,QAAQ,CAAC;IAClB,CAAC;CACF,CAAC"}

package/dist/tools/review/brief-slot.d.ts

@@ -1,11 +0,0 @@
-import type { Input } from '../../tools/review/schema.js';
-export interface ReviewBrief {
-    filePath?: string;
-    code?: string;
-    filePaths?: string[];
-    focus?: string[];
-    hasContextBlocks: boolean;
-    contextBlockIds: string[];
-}
-export declare function reviewBriefSlot(input: Input): ReviewBrief[];
-//# sourceMappingURL=brief-slot.d.ts.map

package/dist/tools/review/brief-slot.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"brief-slot.d.ts","sourceRoot":"","sources":["../../../src/tools/review/brief-slot.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,8BAA8B,CAAC;AAE1D,MAAM,WAAW,WAAW;IAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;IACrB,KAAK,CAAC,EAAE,MAAM,EAAE,CAAC;IACjB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,eAAe,EAAE,MAAM,EAAE,CAAC;CAC3B;AAMD,wBAAgB,eAAe,CAAC,KAAK,EAAE,KAAK,GAAG,WAAW,EAAE,CAoB3D"}

package/dist/tools/review/brief-slot.js

@@ -1,23 +0,0 @@
-function hasContent(value) {
-    return value !== undefined && value.trim().length > 0;
-}
-export function reviewBriefSlot(input) {
-    const hasContextBlocks = Array.isArray(input.contextBlockIds) && input.contextBlockIds.length > 0;
-    const validPaths = (input.filePaths ?? []).filter(p => p.trim().length > 0);
-    if (hasContent(input.code) || validPaths.length <= 1) {
-        return [{
-                code: input.code,
-                filePaths: input.filePaths,
-                focus: input.focus,
-                hasContextBlocks,
-                contextBlockIds: input.contextBlockIds ?? [],
-            }];
-    }
-    return validPaths.map(fp => ({
-        filePath: fp,
-        focus: input.focus,
-        hasContextBlocks,
-        contextBlockIds: input.contextBlockIds ?? [],
-    }));
-}
-//# sourceMappingURL=brief-slot.js.map

package/dist/tools/review/brief-slot.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"brief-slot.js","sourceRoot":"","sources":["../../../src/tools/review/brief-slot.ts"],"names":[],"mappings":"AAWA,SAAS,UAAU,CAAC,KAAyB;IAC3C,OAAO,KAAK,KAAK,SAAS,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,CAAC;AACxD,CAAC;AAED,MAAM,UAAU,eAAe,CAAC,KAAY;IAC1C,MAAM,gBAAgB,GAAG,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,eAAe,CAAC,IAAI,KAAK,CAAC,eAAe,CAAC,MAAM,GAAG,CAAC,CAAC;IAClG,MAAM,UAAU,GAAG,CAAC,KAAK,CAAC,SAAS,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IAE5E,IAAI,UAAU,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,UAAU,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;QACrD,OAAO,CAAC;gBACN,IAAI,EAAE,KAAK,CAAC,IAAI;gBAChB,SAAS,EAAE,KAAK,CAAC,SAAS;gBAC1B,KAAK,EAAE,KAAK,CAAC,KAAK;gBAClB,gBAAgB;gBAChB,eAAe,EAAE,KAAK,CAAC,eAAe,IAAI,EAAE;aAC7C,CAAC,CAAC;IACL,CAAC;IAED,OAAO,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;QAC3B,QAAQ,EAAE,EAAE;QACZ,KAAK,EAAE,KAAK,CAAC,KAAK;QAClB,gBAAgB;QAChB,eAAe,EAAE,KAAK,CAAC,eAAe,IAAI,EAAE;KAC7C,CAAC,CAAC,CAAC;AACN,CAAC"}

package/dist/tools/review/implementer-criteria.d.ts

@@ -1,48 +0,0 @@
-/**
- * Review-specific implementer criteria.
- *
- * REVIEW'S PURPOSE — read this before adding categories.
- * mma-review is the pre-merge gate. The maintainer accepting your verdict
- * will NOT re-investigate before pressing merge — your output is treated
- * as authoritative. The success criterion is:
- *
- *   "After fixes, will the merge be safe, correct, and maintainable —
- *    such that a regression is unlikely to ship?"
- *
- * That criterion makes a finding load-bearing. A nit that doesn't change
- * whether the merge is safe is low-priority no matter how clean the
- * suggested rewrite reads. A cross-file ripple that breaks a caller
- * not in the diff is the audit-equivalent of an unimplementable fix —
- * load-bearing even though the named file looks fine in isolation.
- *
- * Review examines source code in named files against a focus area
- * (security/correctness/performance/style). Findings should be
- * line-quotable — that's the natural shape of code defects — but
- * cross-file findings backed by call-site references are also valid.
- */
-/**
- * The orientation block. Goes at the TOP of every review prompt.
- *
- * This is the load-bearing addition. Without an explicit purpose
- * statement, workers default to "find issues in this file" — which
- * produces line-by-line proofreading and misses the cross-file rippe,
- * test-gap, and implicit-contract findings that actually block merges.
- */
-export declare const REVIEW_PURPOSE_ORIENTATION: string;
-export declare const EVIDENCE_RULE_REVIEW: string;
-export declare const SCOPE_RULE_REVIEW: string;
-/**
- * The failure-mode taxonomy for code reviews.
- *
- * Without this block, workers default to line-by-line proofreading of
- * the named file and miss cross-file ripples, test gaps, and
- * implicit-contract regressions — the findings that actually block
- * merges. The 10 categories below are what a careful maintainer would
- * scan for before pressing merge.
- */
-export declare const CODE_REVIEW_FAILURE_MODES: string;
-export declare const ANNOTATOR_AWARENESS_REVIEW: string;
-import { type CriterionEntry } from '../criteria-types.js';
-/** Structured per-criterion array for parallel-criteria fan-out. */
-export declare const REVIEW_CRITERIA: readonly CriterionEntry[];
-//# sourceMappingURL=implementer-criteria.d.ts.map

package/dist/tools/review/implementer-criteria.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"implementer-criteria.d.ts","sourceRoot":"","sources":["../../../src/tools/review/implementer-criteria.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH;;;;;;;GAOG;AACH,eAAO,MAAM,0BAA0B,QAmB3B,CAAC;AAEb,eAAO,MAAM,oBAAoB,QAQrB,CAAC;AAEb,eAAO,MAAM,iBAAiB,QAOlB,CAAC;AAEb;;;;;;;;GAQG;AACH,eAAO,MAAM,yBAAyB,QAmB1B,CAAC;AAEb,eAAO,MAAM,0BAA0B,QAO3B,CAAC;AAEb,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAE1E,oEAAoE;AACpE,eAAO,MAAM,eAAe,EAAE,SAAS,cAAc,EAA6C,CAAC"}

package/dist/tools/review/implementer-criteria.js

@@ -1,108 +0,0 @@
-/**
- * Review-specific implementer criteria.
- *
- * REVIEW'S PURPOSE — read this before adding categories.
- * mma-review is the pre-merge gate. The maintainer accepting your verdict
- * will NOT re-investigate before pressing merge — your output is treated
- * as authoritative. The success criterion is:
- *
- *   "After fixes, will the merge be safe, correct, and maintainable —
- *    such that a regression is unlikely to ship?"
- *
- * That criterion makes a finding load-bearing. A nit that doesn't change
- * whether the merge is safe is low-priority no matter how clean the
- * suggested rewrite reads. A cross-file ripple that breaks a caller
- * not in the diff is the audit-equivalent of an unimplementable fix —
- * load-bearing even though the named file looks fine in isolation.
- *
- * Review examines source code in named files against a focus area
- * (security/correctness/performance/style). Findings should be
- * line-quotable — that's the natural shape of code defects — but
- * cross-file findings backed by call-site references are also valid.
- */
-/**
- * The orientation block. Goes at the TOP of every review prompt.
- *
- * This is the load-bearing addition. Without an explicit purpose
- * statement, workers default to "find issues in this file" — which
- * produces line-by-line proofreading and misses the cross-file rippe,
- * test-gap, and implicit-contract findings that actually block merges.
- */
-export const REVIEW_PURPOSE_ORIENTATION = [
-    'Why this review exists:',
-    'mma-review is the pre-merge gate. The maintainer accepting your verdict will NOT re-investigate before merging — your verdict is treated as authoritative. A miss here ships to production.',
-    '',
-    'Your job is to find anything that would make the merge unsafe, including issues that look fine in the named files in isolation:',
-    '- a changed function with no test (or with a test that does not exercise the change)',
-    '- a changed signature whose direct callers (visible in the named files or via grep on the symbol) were not updated',
-    '- a change that introduces a new edge case (null/empty/timeout/error path) the code does not handle',
-    '- a race or concurrency hazard the change exposes (shared state mutation, missing lock, await-after-check pattern)',
-    '- a resource leak the change introduces (unclosed handle, untracked promise, file descriptor not freed)',
-    '- a backward-compatibility break in a public API or wire schema',
-    '- a security regression (auth bypass, injection, untrusted input flowing to a sink, data exposure)',
-    '- a performance regression (N+1 query, unbounded loop, blocking I/O on a hot path, unnecessary deep clone)',
-    '- an implicit-contract assumption — the change relies on the caller doing X but the contract does not state X',
-    '- a pre-existing bug entangled with the change (NOT a finding against this diff — separate cleanly)',
-    '',
-    'A finding that points at any of these is high-value EVEN IF the prose of the change reads cleanly. Conversely, a stylistic nit that does not change merge safety is low-priority no matter how clean the suggested rewrite reads.',
-    '',
-    'The completion test: would a maintainer who reads only your review and the diff (not the surrounding code) understand which changes are required, why each is required, and where each lives — well enough to apply the fix and re-merge?',
-].join('\n');
-export const EVIDENCE_RULE_REVIEW = [
-    'Evidence grounding (REQUIRED for every finding):',
-    '- Cite `file:line` (or `file:line-line` for a span) where the issue lives.',
-    '- Quote the exact code excerpt or command output that demonstrates the issue. Do not paraphrase — quote.',
-    '- For CROSS-FILE findings (a change in named file A breaks a caller B), cite both: the line in A that triggers the break, AND the call site in B that breaks. If B is not in the named files but is reachable via grep on the changed symbol, name it explicitly. Cross-file findings backed by call-site references are FULLY VALID — do not drop them as out-of-scope.',
-    '- For TEST-GAP findings, name the test file you would expect to cover the change AND quote the diff line that has no test coverage. If no test file exists for the changed area, that itself is the finding.',
-    '- For IMPLICIT-CONTRACT findings, quote the line in the named file that depends on the assumption AND name the contract source (the public docstring, the type, the README) that does not state the assumption.',
-    '- If you cannot quote evidence in one of these forms, do NOT raise the finding. Note "investigation needed" in your summary instead.',
-].join('\n');
-export const SCOPE_RULE_REVIEW = [
-    'Scope:',
-    '- The named files. Behavior of direct callers/callees can be referenced when visible in those files.',
-    '- Cross-file ripples ARE in scope when the changed symbol is searchable: if the named files change a public function, look for its call sites in the rest of the repo and flag any caller that would break. This is the highest-value cross-file work for a code review.',
-    '- Test gaps ARE in scope: if the named files change behavior and a test file is the natural sibling (e.g. `foo.ts` → `tests/foo.test.ts`), check whether the test exercises the change.',
-    '- Out of scope: speculation about untouched files unrelated to the diff; doc/spec issues (those belong in an audit, not a review); style nits when the focus area is security/correctness/performance.',
-    '- Pre-existing bugs (the diff did not introduce them) belong in their own backlog item, not in this review. Note them in a "Pre-existing — out of scope" section if you spot them, but DO NOT mix them into the merge-blocking findings.',
-].join('\n');
-/**
- * The failure-mode taxonomy for code reviews.
- *
- * Without this block, workers default to line-by-line proofreading of
- * the named file and miss cross-file ripples, test gaps, and
- * implicit-contract regressions — the findings that actually block
- * merges. The 10 categories below are what a careful maintainer would
- * scan for before pressing merge.
- */
-export const CODE_REVIEW_FAILURE_MODES = [
-    'Look for these kinds of issues — applicable to ALL code reviews regardless of focus. The focus area (security/correctness/performance/style) tells you which lens to weight, but every code review should sweep the full taxonomy:',
-    '',
-    '1. TEST GAP — the diff changes behavior, but no test exercises the change. Either: no test file exists, OR the test file exists but the changed branch is not covered. **Always check for the natural sibling test file when reviewing source-code changes.**',
-    '2. CROSS-FILE RIPPLE — a changed signature, return shape, public type, or wire schema is referenced from another file that was not updated. **If the named files change a public symbol, grep for the symbol and flag any unupdated caller.**',
-    '3. PRE-EXISTING-BUG-VS-NEW-REGRESSION — a defect exists in the named files but the diff did not introduce it. Do NOT blame the diff for prior bugs; note them in a separate "Pre-existing — out of scope" section. Conversely, if the diff DID introduce or worsen a defect, flag it as a regression.',
-    '4. MISSING EDGE CASE — the change adds a code path but does not handle null/undefined/empty/timeout/error/zero/negative inputs the path could see. Walk the change against each natural boundary value.',
-    '5. RACE / CONCURRENCY — the change introduces shared state mutation, removes a lock, splits a previously-atomic operation, or adds an await between a check and an action (TOCTOU). Flag these even when no test reproduces.',
-    '6. RESOURCE LEAK — the change opens a handle (file, socket, lock, transaction, AbortController) without a guaranteed close path; or introduces an untracked promise that may reject silently.',
-    '7. BACKWARD-COMPAT BREAK — the change modifies a public API, exported type, wire schema, environment variable, or CLI flag in a way that breaks existing callers. Flag and require a migration note.',
-    '8. SECURITY REGRESSION — the change introduces or worsens auth bypass, injection (SQL/command/prompt), untrusted input flowing to a sink (eval/exec/HTML/SQL), data exposure, or weakened sandboxing. Apply the security lens to every change, not just security-flagged ones.',
-    '9. PERFORMANCE REGRESSION — the change adds N+1 queries, unbounded loops, blocking I/O on a hot path, unnecessary deep clones, or shifts work from build/init time to request time. Apply the performance lens to every change, not just performance-flagged ones.',
-    '10. IMPLICIT-CONTRACT ASSUMPTION — the changed code relies on the caller (or environment) doing X but the contract (docstring, type, README) does not state X. The change works for in-repo callers but will silently break when the contract is read literally.',
-    '',
-    'Severity calibration for code reviews:',
-    '- critical: the merge would corrupt data, expose credentials, allow auth bypass, break a public API in production, or cause production outage. A reader who applied the fix incorrectly could ship the regression.',
-    '- high: the merge would introduce a real bug, security gap, or substantial regression that blocks release. Cross-file ripple where a caller is broken. Missing edge case in a code path that production traffic will hit.',
-    '- medium: a real issue worth fixing soon: test gap on a non-trivial change, race condition with low contention, performance regression on a non-hot path, missing edge case on an unlikely input.',
-    '- low: stylistic / naming / dead-code / minor-refactor opportunity. Does not change merge safety.',
-].join('\n');
-export const ANNOTATOR_AWARENESS_REVIEW = [
-    'After your output, an annotator validates each finding against this code-review rubric:',
-    '- Is the finding within the requested focus area (or universally applicable: security, performance, correctness apply to every review)?',
-    '- Does the evidence quote real code from the named files OR cite a real call site reachable via grep on the changed symbol?',
-    '- Is the severity calibrated to actual merge-safety impact (would a reader who applied the fix incorrectly ship a regression)?',
-    '- Is the finding within scope (named files + cross-file ripples on changed symbols + sibling test files), or is it speculation about unrelated code?',
-    'Self-check before emitting. Findings that fail any check are downgraded or dropped — but cross-file ripple findings backed by call-site references and test-gap findings backed by sibling-test-file references are FULLY VALID, do NOT downgrade them as "speculation about untouched files."',
-].join('\n');
-import { parseCriteria } from '../criteria-types.js';
-/** Structured per-criterion array for parallel-criteria fan-out. */
-export const REVIEW_CRITERIA = parseCriteria(CODE_REVIEW_FAILURE_MODES);
-//# sourceMappingURL=implementer-criteria.js.map

package/dist/tools/review/implementer-criteria.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"implementer-criteria.js","sourceRoot":"","sources":["../../../src/tools/review/implementer-criteria.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,0BAA0B,GAAG;IACxC,yBAAyB;IACzB,6LAA6L;IAC7L,EAAE;IACF,iIAAiI;IACjI,sFAAsF;IACtF,oHAAoH;IACpH,qGAAqG;IACrG,oHAAoH;IACpH,yGAAyG;IACzG,iEAAiE;IACjE,oGAAoG;IACpG,4GAA4G;IAC5G,+GAA+G;IAC/G,qGAAqG;IACrG,EAAE;IACF,mOAAmO;IACnO,EAAE;IACF,2OAA2O;CAC5O,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,oBAAoB,GAAG;IAClC,kDAAkD;IAClD,4EAA4E;IAC5E,0GAA0G;IAC1G,0WAA0W;IAC1W,8MAA8M;IAC9M,iNAAiN;IACjN,sIAAsI;CACvI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,iBAAiB,GAAG;IAC/B,QAAQ;IACR,sGAAsG;IACtG,0QAA0Q;IAC1Q,yLAAyL;IACzL,wMAAwM;IACxM,0OAA0O;CAC3O,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb;;;;;;;;GAQG;AACH,MAAM,CAAC,MAAM,yBAAyB,GAAG;IACvC,oOAAoO;IACpO,EAAE;IACF,+PAA+P;IAC/P,+OAA+O;IAC/O,uSAAuS;IACvS,yMAAyM;IACzM,8NAA8N;IAC9N,+LAA+L;IAC/L,sMAAsM;IACtM,gRAAgR;IAChR,oQAAoQ;IACpQ,kQAAkQ;IAClQ,EAAE;IACF,wCAAwC;IACxC,oNAAoN;IACpN,2NAA2N;IAC3N,mMAAmM;IACnM,mGAAmG;CACpG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,0BAA0B,GAAG;IACxC,yFAAyF;IACzF,yIAAyI;IACzI,6HAA6H;IAC7H,gIAAgI;IAChI,sJAAsJ;IACtJ,gSAAgS;CACjS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,OAAO,EAAE,aAAa,EAAuB,MAAM,sBAAsB,CAAC;AAE1E,oEAAoE;AACpE,MAAM,CAAC,MAAM,eAAe,GAA8B,aAAa,CAAC,yBAAyB,CAAC,CAAC"}

package/dist/tools/review/schema.d.ts

@@ -1,64 +0,0 @@
-import { z } from 'zod';
-export declare const inputSchema: z.ZodObject<{
-    code: z.ZodOptional<z.ZodString>;
-    focus: z.ZodOptional<z.ZodArray<z.ZodEnum<{
-        style: "style";
-        security: "security";
-        performance: "performance";
-        correctness: "correctness";
-    }>>>;
-    subtype: z.ZodDefault<z.ZodEnum<{
-        default: "default";
-    }>>;
-    filePaths: z.ZodOptional<z.ZodArray<z.ZodString>>;
-    contextBlockIds: z.ZodOptional<z.ZodArray<z.ZodString>>;
-}, z.core.$strip>;
-export type Input = z.infer<typeof inputSchema>;
-export declare const outputSchema: z.ZodObject<{
-    headline: z.ZodString;
-    results: z.ZodUnion<readonly [z.ZodArray<z.ZodType<unknown, unknown, z.core.$ZodTypeInternals<unknown, unknown>>>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    batchTimings: z.ZodUnion<readonly [z.ZodObject<{}, z.core.$loose>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    costSummary: z.ZodUnion<readonly [z.ZodObject<{}, z.core.$loose>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    structuredReport: z.ZodUnion<readonly [z.ZodObject<{}, z.core.$loose>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    error: z.ZodUnion<readonly [z.ZodObject<{
-        code: z.ZodString;
-        message: z.ZodString;
-        details: z.ZodOptional<z.ZodUnknown>;
-    }, z.core.$strip>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    specReviewVerdict: z.ZodOptional<z.ZodEnum<{
-        error: "error";
-        approved: "approved";
-        concerns: "concerns";
-        changes_required: "changes_required";
-        annotated: "annotated";
-        skipped: "skipped";
-        not_applicable: "not_applicable";
-    }>>;
-    qualityReviewVerdict: z.ZodOptional<z.ZodEnum<{
-        error: "error";
-        approved: "approved";
-        concerns: "concerns";
-        changes_required: "changes_required";
-        annotated: "annotated";
-        skipped: "skipped";
-        not_applicable: "not_applicable";
-    }>>;
-    roundsUsed: z.ZodOptional<z.ZodNumber>;
-}, z.core.$loose>;
-export type Output = z.infer<typeof outputSchema>;
-//# sourceMappingURL=schema.d.ts.map

package/dist/tools/review/schema.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"schema.d.ts","sourceRoot":"","sources":["../../../src/tools/review/schema.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAMxB,eAAO,MAAM,WAAW;;;;;;;;;;;;;iBAQtB,CAAC;AAEH,MAAM,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,WAAW,CAAC,CAAC;AAEhD,eAAO,MAAM,YAAY;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;iBAA8B,CAAC;AAExD,MAAM,MAAM,MAAM,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,YAAY,CAAC,CAAC"}

package/dist/tools/review/schema.js

@@ -1,17 +0,0 @@
-// packages/core/src/tool-schemas/review.ts
-import { z } from 'zod';
-import { buildOutputEnvelopeSchema } from '../shared-output.js';
-// Ported verbatim from packages/mcp/src/tools/review-code.ts (reviewCodeSchema).
-// commonToolFields (filePaths + contextBlockIds) are inlined here to avoid
-// cross-package coupling.
-export const inputSchema = z.object({
-    code: z.string().optional().describe('Inline code to review'),
-    focus: z.array(z.enum(['security', 'performance', 'correctness', 'style'])).optional(),
-    subtype: z.enum(['default']).default('default').describe('Criteria-set selector. Only `default` is wired today; reserved for future per-language / per-risk-class subtypes.'),
-    filePaths: z.array(z.string()).optional()
-        .describe('Files the sub-agent should focus on. Multiple files are processed in parallel.'),
-    contextBlockIds: z.array(z.string()).optional()
-        .describe('IDs from register_context_block to prepend to prompt. Use for delta audits, diff-scoped reviews, or shared specs.'),
-});
-export const outputSchema = buildOutputEnvelopeSchema();
-//# sourceMappingURL=schema.js.map

package/dist/tools/review/schema.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"schema.js","sourceRoot":"","sources":["../../../src/tools/review/schema.ts"],"names":[],"mappings":"AAAA,2CAA2C;AAC3C,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,yBAAyB,EAAE,MAAM,qBAAqB,CAAC;AAEhE,iFAAiF;AACjF,2EAA2E;AAC3E,0BAA0B;AAC1B,MAAM,CAAC,MAAM,WAAW,GAAG,CAAC,CAAC,MAAM,CAAC;IAClC,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,uBAAuB,CAAC;IAC7D,KAAK,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,aAAa,EAAE,aAAa,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC,QAAQ,EAAE;IACtF,OAAO,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,mHAAmH,CAAC;IAC7K,SAAS,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;SACtC,QAAQ,CAAC,gFAAgF,CAAC;IAC7F,eAAe,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC,QAAQ,EAAE;SAC5C,QAAQ,CAAC,mHAAmH,CAAC;CACjI,CAAC,CAAC;AAIH,MAAM,CAAC,MAAM,YAAY,GAAG,yBAAyB,EAAE,CAAC"}

package/dist/tools/review/subtypes.d.ts

@@ -1,4 +0,0 @@
-import type { ReadOnlySubtypeSpec } from '../../lifecycle/read-only-subtype-spec.js';
-export type ReviewSubtype = 'default';
-export declare const REVIEW_SUBTYPES: Record<ReviewSubtype, ReadOnlySubtypeSpec>;
-//# sourceMappingURL=subtypes.d.ts.map

package/dist/tools/review/subtypes.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"subtypes.d.ts","sourceRoot":"","sources":["../../../src/tools/review/subtypes.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,2CAA2C,CAAC;AAOrF,MAAM,MAAM,aAAa,GAAG,SAAS,CAAC;AAkBtC,eAAO,MAAM,eAAe,EAAE,MAAM,CAAC,aAAa,EAAE,mBAAmB,CAStE,CAAC"}

package/dist/tools/review/subtypes.js

@@ -1,27 +0,0 @@
-import { REVIEW_PURPOSE_ORIENTATION, EVIDENCE_RULE_REVIEW, SCOPE_RULE_REVIEW, ANNOTATOR_AWARENESS_REVIEW, REVIEW_CRITERIA, } from './implementer-criteria.js';
-// Copied verbatim from ROUTE_SEMANTICS.review in
-// packages/core/src/routing/read-route-criteria.ts (removed in Task 8).
-const SEMANTICS_DEFAULT = {
-    goalLine: 'Find ALL issues of THIS specific kind in the diff / source above.',
-    emptyOutcomeLine: 'If none exist, respond with the literal text "No findings for this criterion." — that is a fully valid outcome. Do NOT pad to avoid returning empty.',
-    findingMeaningParagraph: 'A finding is an ISSUE introduced or worsened by the change under review (correctness bug, missing test, race, security regression, contract break, etc.). Severity reflects how much the issue blocks merge / ships a regression.',
-    severityMeanings: {
-        critical: 'Production-breaking on merge',
-        high: 'Correctness gap surfacing in normal use',
-        medium: 'Maintainability/fragility',
-        low: 'Style',
-    },
-    mustEmitAtLeastOne: false,
-    legalOutcomes: ['found', 'clean'],
-};
-export const REVIEW_SUBTYPES = {
-    default: {
-        criteria: REVIEW_CRITERIA,
-        orientation: REVIEW_PURPOSE_ORIENTATION,
-        evidenceRule: EVIDENCE_RULE_REVIEW,
-        scopeRule: SCOPE_RULE_REVIEW,
-        annotatorAwareness: ANNOTATOR_AWARENESS_REVIEW,
-        semantics: SEMANTICS_DEFAULT,
-    },
-};
-//# sourceMappingURL=subtypes.js.map

package/dist/tools/review/subtypes.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"subtypes.js","sourceRoot":"","sources":["../../../src/tools/review/subtypes.ts"],"names":[],"mappings":"AAEA,OAAO,EACL,0BAA0B,EAAE,oBAAoB,EAAE,iBAAiB,EACnE,0BAA0B,EAAE,eAAe,GAC5C,MAAM,2BAA2B,CAAC;AAInC,iDAAiD;AACjD,wEAAwE;AACxE,MAAM,iBAAiB,GAAmB;IACxC,QAAQ,EAAE,mEAAmE;IAC7E,gBAAgB,EAAE,sJAAsJ;IACxK,uBAAuB,EAAE,mOAAmO;IAC5P,gBAAgB,EAAE;QAChB,QAAQ,EAAE,8BAA8B;QACxC,IAAI,EAAE,yCAAyC;QAC/C,MAAM,EAAE,2BAA2B;QACnC,GAAG,EAAE,OAAO;KACb;IACD,kBAAkB,EAAE,KAAK;IACzB,aAAa,EAAE,CAAC,OAAO,EAAE,OAAO,CAAU;CAC3C,CAAC;AAEF,MAAM,CAAC,MAAM,eAAe,GAA+C;IACzE,OAAO,EAAE;QACP,QAAQ,EAAE,eAAe;QACzB,WAAW,EAAE,0BAA0B;QACvC,YAAY,EAAE,oBAAoB;QAClC,SAAS,EAAE,iBAAiB;QAC5B,kBAAkB,EAAE,0BAA0B;QAC9C,SAAS,EAAE,iBAAiB;KAC7B;CACF,CAAC"}

package/dist/tools/review/tool-config.d.ts

@@ -1,7 +0,0 @@
-import { ToolSurfaceRegistry } from '../../tool-surface/tool-surface-registry.js';
-import type { Input } from './schema.js';
-import type { ToolConfig } from '../../lifecycle/tool-config-types.js';
-import { type ReviewBrief } from './brief-slot.js';
-export declare function registerReview(registry: ToolSurfaceRegistry): void;
-export declare const toolConfig: ToolConfig<Input, ReviewBrief, unknown>;
-//# sourceMappingURL=tool-config.d.ts.map

package/dist/tools/review/tool-config.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-config.d.ts","sourceRoot":"","sources":["../../../src/tools/review/tool-config.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,6CAA6C,CAAC;AAElF,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,sCAAsC,CAAC;AAEvE,OAAO,EAAmB,KAAK,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAKpE,wBAAgB,cAAc,CAAC,QAAQ,EAAE,mBAAmB,GAAG,IAAI,CAYlE;AAsCD,eAAO,MAAM,UAAU,EAAE,UAAU,CAAC,KAAK,EAAE,WAAW,EAAE,OAAO,CA0C9D,CAAC"}

package/dist/tools/review/tool-config.js

@@ -1,94 +0,0 @@
-import { inputSchema } from './schema.js';
-import { reviewBriefSlot } from './brief-slot.js';
-import { noStructuredReportSchema } from '../../reporting/report-parser-slots/no-structured-report.js';
-import { makeFindingsHeadlineTemplate } from '../../reporting/findings-headline.js';
-import { DEFAULT_TASK_TIMEOUT_MS } from '../../config/schema.js';
-export function registerReview(registry) {
-    registry.register({
-        routeName: 'review',
-        httpMethod: 'POST',
-        httpPath: '/review',
-        surface: 'tool',
-        schema: inputSchema,
-        toolCategory: 'read_only',
-        agentTypeDefault: 'complex',
-        agentTypeOverridable: false,
-        responseShapeName: 'BatchResponse',
-    });
-}
-/**
- * Per-focus "done" conditions.
- *
- * The full failure-mode taxonomy in CODE_REVIEW_FAILURE_MODES applies to
- * all reviews regardless of focus. These per-focus conditions tell the
- * worker which lens to weight, not which categories to skip. Security,
- * performance, and correctness lenses are universally applicable to
- * every code change — the focus array picks emphasis, not gating.
- *
- * When focus is empty/missing, the worker performs a comprehensive sweep
- * applying all four lenses with the executability/merge-safety
- * orientation block at the top of the prompt.
- */
-const REVIEW_DONE_CONDITIONS = {
-    security: 'Lens emphasis: security. Apply the full failure-mode taxonomy through the security lens: auth bypass, injection (SQL/command/prompt), untrusted input flowing to a sink (eval/exec/HTML), data exposure, weakened sandboxing, and hardcoded secrets. Each finding has severity, location, and remediation.',
-    performance: 'Lens emphasis: performance. Apply the full failure-mode taxonomy through the performance lens: N+1 queries, unbounded loops, blocking I/O on hot paths, unnecessary deep clones, work shifted from build/init time to request time, and missing caching where the same value is recomputed. Each finding has impact level, location, and fix recommendation.',
-    correctness: 'Lens emphasis: correctness. Apply the full failure-mode taxonomy through the correctness lens: logic errors, off-by-one, unhandled edge cases (null/undefined/empty/timeout/error/zero/negative), type mismatches, contract violations, race conditions, and resource leaks. Each finding has severity, location, and correct behavior.',
-    style: 'Lens emphasis: style. Apply the full failure-mode taxonomy through the style lens: naming, formatting, dead code, inconsistent patterns, deprecated APIs, and missing types. Note: style is rarely the highest-value review lens for a non-trivial diff — sweep the correctness, security, and performance categories too.',
-};
-const DELTA_REVIEW_SUFFIX = ' Perform a full review (do not reduce thoroughness). Verify each prior finding as addressed or unaddressed. Omit addressed prior findings. Include unaddressed prior findings and new findings. End with a summary of which prior findings were resolved.';
-function resolveReviewDoneCondition(focus, hasContextBlocks) {
-    let base;
-    if (!focus || focus.length === 0) {
-        base = 'Comprehensive code review. Apply the full failure-mode taxonomy (the orientation block above) through all four lenses (correctness, security, performance, style). Emphasize TEST GAP, CROSS-FILE RIPPLE, MISSING EDGE CASE, and IMPLICIT-CONTRACT ASSUMPTION — these are the categories most often missed and most likely to ship regressions. Each finding has category, severity, location, and recommendation.';
-    }
-    else {
-        base = focus.map(f => REVIEW_DONE_CONDITIONS[f] ?? '').filter(Boolean).join(' ');
-    }
-    return hasContextBlocks ? base + DELTA_REVIEW_SUFFIX : base;
-}
-export const toolConfig = {
-    name: 'review',
-    category: 'read_only',
-    dispatchMode: 'parallel',
-    dispatchModeOverridable: false,
-    agentType: 'complex',
-    briefSlot: reviewBriefSlot,
-    buildTaskSpec: (brief, ctx) => {
-        // Propagate filePaths + mainModel onto the TaskSpec so the headline
-        // composer can name the file in clean-review headlines and so the
-        // wire telemetry carries main_model attribution.
-        const filePaths = brief.filePath
-            ? [brief.filePath]
-            : (brief.filePaths && brief.filePaths.length > 0 ? brief.filePaths : undefined);
-        const targetParts = ['Review this code:'];
-        if (brief.code)
-            targetParts.push('```\n' + brief.code + '\n```');
-        if (filePaths && filePaths.length > 0) {
-            targetParts.push(`Target files:\n${filePaths.map(p => `- ${p}`).join('\n')}`);
-        }
-        if (brief.focus)
-            targetParts.push(`Focus: ${Array.isArray(brief.focus) ? brief.focus.join(', ') : brief.focus}`);
-        // The read-route dispatcher builds the worker prefix from this pure target
-        // + FINDING_FORMAT_SHARED + review RouteSemantics; `prompt` mirrors it
-        // (required field / telemetry, not the read-route worker input).
-        const target = targetParts.join('\n\n');
-        return {
-            prompt: target,
-            readTarget: target,
-            agentType: 'complex',
-            reviewPolicy: 'none',
-            briefQualityPolicy: 'off',
-            done: resolveReviewDoneCondition(brief.focus, brief.hasContextBlocks),
-            tools: ctx.config.defaults?.tools ?? 'full',
-            timeoutMs: ctx.config.defaults?.timeoutMs ?? DEFAULT_TASK_TIMEOUT_MS,
-            sandboxPolicy: ctx.config.defaults?.sandboxPolicy ?? 'cwd-only',
-            cwd: ctx.projectContext?.cwd ?? ctx.cwd,
-            contextBlockIds: brief.contextBlockIds,
-            filePaths,
-            mainModel: ctx.mainModel ?? undefined,
-        };
-    },
-    reportSchema: noStructuredReportSchema,
-    headlineTemplate: makeFindingsHeadlineTemplate('review', 'blocking'),
-};
-//# sourceMappingURL=tool-config.js.map

package/dist/tools/review/tool-config.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-config.js","sourceRoot":"","sources":["../../../src/tools/review/tool-config.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAI1C,OAAO,EAAE,eAAe,EAAoB,MAAM,iBAAiB,CAAC;AACpE,OAAO,EAAE,wBAAwB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,EAAE,4BAA4B,EAAE,MAAM,sCAAsC,CAAC;AACpF,OAAO,EAAE,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AAEjE,MAAM,UAAU,cAAc,CAAC,QAA6B;IAC1D,QAAQ,CAAC,QAAQ,CAAC;QAChB,SAAS,EAAE,QAAQ;QACnB,UAAU,EAAE,MAAM;QAClB,QAAQ,EAAE,SAAS;QACnB,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,WAAW;QACnB,YAAY,EAAE,WAAW;QACzB,gBAAgB,EAAE,SAAS;QAC3B,oBAAoB,EAAE,KAAK;QAC3B,iBAAiB,EAAE,eAAe;KACnC,CAAC,CAAC;AACL,CAAC;AAED;;;;;;;;;;;;GAYG;AACH,MAAM,sBAAsB,GAA2B;IACrD,QAAQ,EACN,4SAA4S;IAC9S,WAAW,EACT,8VAA8V;IAChW,WAAW,EACT,yUAAyU;IAC3U,KAAK,EACH,4TAA4T;CAC/T,CAAC;AAEF,MAAM,mBAAmB,GAAG,2PAA2P,CAAC;AAExR,SAAS,0BAA0B,CAAC,KAA2B,EAAE,gBAAyB;IACxF,IAAI,IAAY,CAAC;IACjB,IAAI,CAAC,KAAK,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACjC,IAAI,GAAG,oZAAoZ,CAAC;IAC9Z,CAAC;SAAM,CAAC;QACN,IAAI,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,sBAAsB,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACnF,CAAC;IACD,OAAO,gBAAgB,CAAC,CAAC,CAAC,IAAI,GAAG,mBAAmB,CAAC,CAAC,CAAC,IAAI,CAAC;AAC9D,CAAC;AAED,MAAM,CAAC,MAAM,UAAU,GAA4C;IACjE,IAAI,EAAE,QAAQ;IACd,QAAQ,EAAE,WAAW;IACrB,YAAY,EAAE,UAAU;IACxB,uBAAuB,EAAE,KAAK;IAC9B,SAAS,EAAE,SAAS;IACpB,SAAS,EAAE,eAAe;IAC1B,aAAa,EAAE,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE;QAC5B,oEAAoE;QACpE,kEAAkE;QAClE,iDAAiD;QACjD,MAAM,SAAS,GAAG,KAAK,CAAC,QAAQ;YAC9B,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC;YAClB,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,IAAI,KAAK,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC;QAClF,MAAM,WAAW,GAAa,CAAC,mBAAmB,CAAC,CAAC;QACpD,IAAI,KAAK,CAAC,IAAI;YAAE,WAAW,CAAC,IAAI,CAAC,OAAO,GAAG,KAAK,CAAC,IAAI,GAAG,OAAO,CAAC,CAAC;QACjE,IAAI,SAAS,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACtC,WAAW,CAAC,IAAI,CAAC,kBAAkB,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAChF,CAAC;QACD,IAAI,KAAK,CAAC,KAAK;YAAE,WAAW,CAAC,IAAI,CAAC,UAAU,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,EAAE,CAAC,CAAC;QACjH,2EAA2E;QAC3E,uEAAuE;QACvE,iEAAiE;QACjE,MAAM,MAAM,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACxC,OAAO;YACL,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,MAAM;YAClB,SAAS,EAAE,SAAS;YACpB,YAAY,EAAE,MAAM;YACpB,kBAAkB,EAAE,KAAK;YACzB,IAAI,EAAE,0BAA0B,CAAC,KAAK,CAAC,KAAK,EAAE,KAAK,CAAC,gBAAgB,CAAC;YACrE,KAAK,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,KAAK,IAAI,MAAM;YAC3C,SAAS,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,SAAS,IAAI,uBAAuB;YACpE,aAAa,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,aAAa,IAAI,UAAU;YAC/D,GAAG,EAAE,GAAG,CAAC,cAAc,EAAE,GAAG,IAAI,GAAG,CAAC,GAAG;YACvC,eAAe,EAAE,KAAK,CAAC,eAAe;YACtC,SAAS;YACT,SAAS,EAAE,GAAG,CAAC,SAAS,IAAI,SAAS;SACtC,CAAC;IACJ,CAAC;IACD,YAAY,EAAE,wBAAwB;IACtC,gBAAgB,EAAE,4BAA4B,CAAC,QAAQ,EAAE,UAAU,CAAC;CACrE,CAAC"}

package/dist/tools/shared-output.d.ts

@@ -1,56 +0,0 @@
-import { z } from 'zod';
-export declare const errorSchema: z.ZodObject<{
-    code: z.ZodString;
-    message: z.ZodString;
-    details: z.ZodOptional<z.ZodUnknown>;
-}, z.core.$strip>;
-export declare const batchTimingsSchema: z.ZodObject<{}, z.core.$loose>;
-export declare const costSummarySchema: z.ZodObject<{}, z.core.$loose>;
-export declare const structuredReportSchema: z.ZodObject<{}, z.core.$loose>;
-export declare function buildOutputEnvelopeSchema(resultItemSchema?: z.ZodTypeAny): z.ZodObject<{
-    headline: z.ZodString;
-    results: z.ZodUnion<readonly [z.ZodArray<z.ZodType<unknown, unknown, z.core.$ZodTypeInternals<unknown, unknown>>>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    batchTimings: z.ZodUnion<readonly [z.ZodObject<{}, z.core.$loose>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    costSummary: z.ZodUnion<readonly [z.ZodObject<{}, z.core.$loose>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    structuredReport: z.ZodUnion<readonly [z.ZodObject<{}, z.core.$loose>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    error: z.ZodUnion<readonly [z.ZodObject<{
-        code: z.ZodString;
-        message: z.ZodString;
-        details: z.ZodOptional<z.ZodUnknown>;
-    }, z.core.$strip>, z.ZodObject<{
-        kind: z.ZodLiteral<"not_applicable">;
-        reason: z.ZodString;
-    }, z.core.$strip>]>;
-    specReviewVerdict: z.ZodOptional<z.ZodEnum<{
-        error: "error";
-        approved: "approved";
-        concerns: "concerns";
-        changes_required: "changes_required";
-        annotated: "annotated";
-        skipped: "skipped";
-        not_applicable: "not_applicable";
-    }>>;
-    qualityReviewVerdict: z.ZodOptional<z.ZodEnum<{
-        error: "error";
-        approved: "approved";
-        concerns: "concerns";
-        changes_required: "changes_required";
-        annotated: "annotated";
-        skipped: "skipped";
-        not_applicable: "not_applicable";
-    }>>;
-    roundsUsed: z.ZodOptional<z.ZodNumber>;
-}, z.core.$loose>;
-//# sourceMappingURL=shared-output.d.ts.map

package/dist/tools/shared-output.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"shared-output.d.ts","sourceRoot":"","sources":["../../src/tools/shared-output.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAKxB,eAAO,MAAM,WAAW;;;;iBAItB,CAAC;AAGH,eAAO,MAAM,kBAAkB,gCAA6B,CAAC;AAC7D,eAAO,MAAM,iBAAiB,gCAA6B,CAAC;AAC5D,eAAO,MAAM,sBAAsB,gCAA6B,CAAC;AAGjE,wBAAgB,yBAAyB,CAAC,gBAAgB,GAAE,CAAC,CAAC,UAAwB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;kBAYrF"}