@zhixuan92/multi-model-agent-core 4.0.3 → 4.0.4

package/README.md

@@ -229,15 +229,17 @@ mmagent logs --follow --batch=<id>   # tail + filter
 
 As of 3.4.0 every task-execution event the worker emits to the verbose stderr stream is also written to the JSONL log via a single `emit(TaskEvent)` writer — schema parity across both sinks. Crash/disconnect events (`startup`, `request_start`, `request_complete`, `shutdown`, `error`) are written unconditionally; per-task events (`heartbeat`, `stage_change`, `tool_call`, `turn_complete`, etc.) flow through the same writer.
 
-## What's new in 4.0.3
-
-- **Wire field rename:** `parentModel*` → `mainModel*`. `costDeltaVsParentUSD` → `costDeltaVsMainUSD`, `parentEquivalentCostUSD` → `mainEquivalentCostUSD`. Internal record now matches wire shape 1:1 (no more rename shim in `buildWirePayload`).
-- **`config.defaults.mainModel` removed.** Library callers should pass `mainModel` per-task on the `TaskSpec`. The standalone daemon reads it from the `X-MMA-Main-Model` header.
-- **Canonical model-name preservation.** `extractCanonicalModelName('claude-opus-4-7')` now returns `'claude-opus-4-7'` (was: `'claude-opus'`). Best-effort extraction handles arbitrary wrappers.
-- **`HeadlineTemplate.compose` signature** added optional `runResult: RunResult` and `task: TaskSpec` params (backwards-compatible). Audit/review/delegate composers use them to fall back to `runResult.annotatedFindings` / `runResult.filesWritten` when the structured report doesn't carry them.
-- **`ContextBlockStore` interface** extended with `size`, `pin`/`unpin`/`refcount`, `clear`, `ttlMs`. New `FileBackedContextBlockStore` implementation persists to `<projectCwd>/.mma/context-blocks/` (the daemon's default; tests use `createInMemoryProjectContext`).
-- **`totalDurationMs` reflects real wall-clock.** `event-builder.ts` now uses `max(runResult.durationMs, Σ stageDurations)`. The proportional scale-down was masking the implementer-only bug — gone.
-- **New shared modules** under `core/`: `lifecycle/stage-progression`, `reporting/severity`, `reporting/headline-text`, `providers/tool-name-sets`. Each centralizes a previously-duplicated concern.
+## What's new in 4.0.4
+
+- **`DiffTracker` (NEW: `lifecycle/diff-tracker.ts`).** Snapshot-based unified-diff helper used by spec / quality / diff reviewers. Captures pre-task baseline of declared `task.filePaths` (works in non-git directories), produces a Myers-style line-LCS unified diff at any review point, capped at 50KB with truncation marker. Reviewer templates and engine extended to receive the diff plus `priorConcerns` so verdicts must point to specific diff lines.
+- **`finding-criteria.ts` (NEW shared module).** Single source of truth for `SEVERITY_LADDER`, `EVIDENCE_GROUNDING`, `SCOPE_DISCIPLINE`, `ANNOTATOR_CHECK_AWARENESS_RO` (read-only), `REVIEWER_AWARENESS_AP` (artifact-producing). Per-tool implementer prompts include the criteria so workers self-align with what each reviewer will validate.
+- **Lenient JSON output parsers.** Both `reviewer-output-parser` and `annotator-output-parser` now accept ` ```json ` fenced, ` ``` ` (no language tag), bare JSON, and embedded objects/arrays via balanced-brace counting (string-literal aware).
+- **`replaceLastRunResultPreservingTrackers` (`merge-stage-stats.ts`).** Spec/quality chain handlers now union `filesRead` / `filesWritten` / `toolCalls` arrays across rework rounds rather than wiping them when a no-op rework round overwrites `lastRunResult`.
+- **`HeadlineTemplate.compose`** ergonomics: every artifact-producing tool's headline now follows `[<status>] <route>: <summary>` (delegate / execute-plan / retry / debug). Verify, audit, review composers now also propagate `task.filePaths[0]` for clean-run path display.
+- **Verify report parser accepts narrative + JSON.** `verifyReportSchema.parse` accepts both ` ```json ` blocks and `## Finding N:` narrative — matches the implementer prompt's "Do NOT emit JSON" instruction.
+- **Investigate prompt rewritten to match parser.** `## Summary` / `## Citations` / `## Confidence` sections — pre-fix the prompt asked for numbered narrative which the parser couldn't extract (every investigation reported `0 citations, confidence unparseable`).
+- **`priorSpecConcerns` / `priorQualityConcerns` slots on `LifecycleState`** carry concerns from earlier rework rounds into later ones, so round N+1's reviewer can verify prior issues are addressed.
+- **`firstSentenceOrTruncate`** hardened (off-by-one regex fix, newline collapse before sentence detection, defense against invalid `max` values).
 
 Full history: [CHANGELOG](https://github.com/zhixuan312/multi-model-agent/blob/master/CHANGELOG.md).
 

package/dist/intake/brief-compiler-slots/delegate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delegate.d.ts","sourceRoot":"","sources":["../../../src/intake/brief-compiler-slots/delegate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAkB,MAAM,aAAa,CAAC;AAG7D,MAAM,MAAM,YAAY,GAAG,MAAM,GAAG,cAAc,GAAG,WAAW,GAAG,MAAM,CAAC;AAE1E,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;IACrB,SAAS,CAAC,EAAE,UAAU,GAAG,SAAS,CAAC;IACnC,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;IAC3B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;CAC1B;AAID,wBAAgB,qBAAqB,CAAC,KAAK,EAAE;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,SAAS,CAAC,EAAE,MAAM,EAAE,CAAA;CAAE,GAAG,MAAM,CAU7F;AAED,wBAAgB,oBAAoB,CAClC,KAAK,EAAE,iBAAiB,EAAE,GAAG,IAAI,GAAG,SAAS,EAC7C,SAAS,EAAE,MAAM,GAChB,SAAS,EAAE,CA0Bb;AAID,MAAM,WAAW,aAAa;IAC5B,KAAK,EAAE,iBAAiB,EAAE,CAAC;IAC3B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,EAAE,MAAM,CAAC;IACZ,SAAS,EAAE,UAAU,GAAG,SAAS,CAAC;IAClC,YAAY,EAAE,YAAY,CAAC;IAC3B,eAAe,EAAE,MAAM,EAAE,CAAC;CAC3B;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,EAAE,CAWlE"}
+{"version":3,"file":"delegate.d.ts","sourceRoot":"","sources":["../../../src/intake/brief-compiler-slots/delegate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAkB,MAAM,aAAa,CAAC;AAI7D,MAAM,MAAM,YAAY,GAAG,MAAM,GAAG,cAAc,GAAG,WAAW,GAAG,MAAM,CAAC;AAE1E,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;IACrB,SAAS,CAAC,EAAE,UAAU,GAAG,SAAS,CAAC;IACnC,eAAe,CAAC,EAAE,MAAM,EAAE,CAAC;IAC3B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,aAAa,CAAC,EAAE,MAAM,EAAE,CAAC;CAC1B;AAID,wBAAgB,qBAAqB,CAAC,KAAK,EAAE;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,SAAS,CAAC,EAAE,MAAM,EAAE,CAAA;CAAE,GAAG,MAAM,CAgB7F;AAED,wBAAgB,oBAAoB,CAClC,KAAK,EAAE,iBAAiB,EAAE,GAAG,IAAI,GAAG,SAAS,EAC7C,SAAS,EAAE,MAAM,GAChB,SAAS,EAAE,CA0Bb;AAID,MAAM,WAAW,aAAa;IAC5B,KAAK,EAAE,iBAAiB,EAAE,CAAC;IAC3B,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,aAAa;IAC5B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,EAAE,MAAM,CAAC;IACZ,SAAS,EAAE,UAAU,GAAG,SAAS,CAAC;IAClC,YAAY,EAAE,YAAY,CAAC;IAC3B,eAAe,EAAE,MAAM,EAAE,CAAC;CAC3B;AAED,wBAAgB,YAAY,CAAC,KAAK,EAAE,aAAa,GAAG,aAAa,EAAE,CAWlE"}

package/dist/intake/brief-compiler-slots/delegate.js

@@ -1,4 +1,5 @@
 import { createDraftId } from '../draft-id.js';
+import { REVIEWER_AWARENESS_AP } from '../../review/templates/finding-criteria.js';
 const SCOPE_CONTRACT = `Stay scoped to the explicit task description. Do NOT enlarge the task. If the task references files, read those files first; do not enumerate adjacent ones.`;
 export function compileDelegatePrompt(input) {
     // When the caller pinned filePaths, surface them as a hard constraint
@@ -9,7 +10,13 @@ export function compileDelegatePrompt(input) {
     const filePathsClause = input.filePaths && input.filePaths.length > 0
         ? `\n\nFILE CONSTRAINT: write your code to exactly these file path(s), no others, no renames: ${input.filePaths.map((p) => `\`${p}\``).join(', ')}.`
         : '';
-    return `${input.prompt}\n\n${SCOPE_CONTRACT}${filePathsClause}`;
+    // Tool sweep #12: tell the worker what the spec + quality reviewers
+    // will judge AGAINST so the worker self-aligns. Pre-fix the worker
+    // wrote code blind to the rubric, the reviewer flagged spec mismatches
+    // the worker could have caught itself, and the rework loop spiraled.
+    // Now: worker calibrates against the same criteria → first-round
+    // approvals are the norm, rework triggers only on real mistakes.
+    return `${input.prompt}\n\n${SCOPE_CONTRACT}${filePathsClause}\n\n${REVIEWER_AWARENESS_AP}`;
 }
 export function compileDelegateTasks(tasks, requestId) {
     if (tasks == null) {

package/dist/intake/brief-compiler-slots/delegate.js.map

@@ -1 +1 @@
-{"version":3,"file":"delegate.js","sourceRoot":"","sources":["../../../src/intake/brief-compiler-slots/delegate.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAc/C,MAAM,cAAc,GAAG,8JAA8J,CAAC;AAEtL,MAAM,UAAU,qBAAqB,CAAC,KAA+C;IACnF,sEAAsE;IACtE,sEAAsE;IACtE,iEAAiE;IACjE,qEAAqE;IACrE,uEAAuE;IACvE,MAAM,eAAe,GAAG,KAAK,CAAC,SAAS,IAAI,KAAK,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC;QACnE,CAAC,CAAC,8FAA8F,KAAK,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;QACpJ,CAAC,CAAC,EAAE,CAAC;IACP,OAAO,GAAG,KAAK,CAAC,MAAM,OAAO,cAAc,GAAG,eAAe,EAAE,CAAC;AAClE,CAAC;AAED,MAAM,UAAU,oBAAoB,CAClC,KAA6C,EAC7C,SAAiB;IAEjB,IAAI,KAAK,IAAI,IAAI,EAAE,CAAC;QAClB,OAAO,CAAC,IAAI,CAAC,oEAAoE,CAAC,CAAC;QACnF,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACvB,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,OAAO,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;QAC/B,MAAM,aAAa,GAA4B,eAAe,CAAC,IAAI,CAAuC,CAAC;QAC3G,OAAO;YACL,OAAO,EAAE,aAAa,CAAC,SAAS,EAAE,KAAK,EAAE,MAAM,CAAC;YAChD,MAAM,EAAE;gBACN,KAAK,EAAE,gBAAgB;gBACvB,aAAa;aACI;YACnB,MAAM,EAAE,qBAAqB,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,EAAE,SAAS,EAAE,IAAI,CAAC,SAAS,EAAE,CAAC;YACjF,IAAI,EAAE,IAAI,CAAC,IAAI;YACf,SAAS,EAAE,IAAI,CAAC,SAAS;YACzB,SAAS,EAAE,IAAI,CAAC,SAAS;YACzB,eAAe,EAAE,IAAI,CAAC,eAAe;YACrC,YAAY,EAAE,IAAI,CAAC,YAAY;YAC/B,aAAa,EAAE,IAAI,CAAC,aAAa;SAClC,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC;AAkBD,MAAM,UAAU,YAAY,CAAC,KAAoB;IAC/C,MAAM,SAAS,GAAG,KAAK,CAAC,SAAS,IAAI,MAAM,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC;IACtE,MAAM,MAAM,GAAG,oBAAoB,CAAC,KAAK,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;IAC5D,OAAO,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC;QACnC,SAAS,EAAE,KAAK;QAChB,KAAK,EAAE,KAAK,CAAC,MAAM;QACnB,GAAG,EAAE,OAAO,CAAC,GAAG,EAAE;QAClB,SAAS,EAAE,KAAK,CAAC,SAAS,IAAI,UAAU;QACxC,YAAY,EAAE,KAAK,CAAC,YAAY,IAAI,MAAM;QAC1C,eAAe,EAAE,KAAK,CAAC,eAAe,IAAI,EAAE;KAC7C,CAAC,CAAC,CAAC;AACN,CAAC"}
+{"version":3,"file":"delegate.js","sourceRoot":"","sources":["../../../src/intake/brief-compiler-slots/delegate.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAC;AAC/C,OAAO,EAAE,qBAAqB,EAAE,MAAM,4CAA4C,CAAC;AAcnF,MAAM,cAAc,GAAG,8JAA8J,CAAC;AAEtL,MAAM,UAAU,qBAAqB,CAAC,KAA+C;IACnF,sEAAsE;IACtE,sEAAsE;IACtE,iEAAiE;IACjE,qEAAqE;IACrE,uEAAuE;IACvE,MAAM,eAAe,GAAG,KAAK,CAAC,SAAS,IAAI,KAAK,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC;QACnE,CAAC,CAAC,8FAA8F,KAAK,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;QACpJ,CAAC,CAAC,EAAE,CAAC;IACP,oEAAoE;IACpE,mEAAmE;IACnE,uEAAuE;IACvE,qEAAqE;IACrE,iEAAiE;IACjE,iEAAiE;IACjE,OAAO,GAAG,KAAK,CAAC,MAAM,OAAO,cAAc,GAAG,eAAe,OAAO,qBAAqB,EAAE,CAAC;AAC9F,CAAC;AAED,MAAM,UAAU,oBAAoB,CAClC,KAA6C,EAC7C,SAAiB;IAEjB,IAAI,KAAK,IAAI,IAAI,EAAE,CAAC;QAClB,OAAO,CAAC,IAAI,CAAC,oEAAoE,CAAC,CAAC;QACnF,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACvB,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,OAAO,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE;QAC/B,MAAM,aAAa,GAA4B,eAAe,CAAC,IAAI,CAAuC,CAAC;QAC3G,OAAO;YACL,OAAO,EAAE,aAAa,CAAC,SAAS,EAAE,KAAK,EAAE,MAAM,CAAC;YAChD,MAAM,EAAE;gBACN,KAAK,EAAE,gBAAgB;gBACvB,aAAa;aACI;YACnB,MAAM,EAAE,qBAAqB,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,MAAM,EAAE,SAAS,EAAE,IAAI,CAAC,SAAS,EAAE,CAAC;YACjF,IAAI,EAAE,IAAI,CAAC,IAAI;YACf,SAAS,EAAE,IAAI,CAAC,SAAS;YACzB,SAAS,EAAE,IAAI,CAAC,SAAS;YACzB,eAAe,EAAE,IAAI,CAAC,eAAe;YACrC,YAAY,EAAE,IAAI,CAAC,YAAY;YAC/B,aAAa,EAAE,IAAI,CAAC,aAAa;SAClC,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC;AAkBD,MAAM,UAAU,YAAY,CAAC,KAAoB;IAC/C,MAAM,SAAS,GAAG,KAAK,CAAC,SAAS,IAAI,MAAM,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC;IACtE,MAAM,MAAM,GAAG,oBAAoB,CAAC,KAAK,CAAC,KAAK,EAAE,SAAS,CAAC,CAAC;IAC5D,OAAO,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,KAAK,EAAE,EAAE,CAAC,CAAC;QACnC,SAAS,EAAE,KAAK;QAChB,KAAK,EAAE,KAAK,CAAC,MAAM;QACnB,GAAG,EAAE,OAAO,CAAC,GAAG,EAAE;QAClB,SAAS,EAAE,KAAK,CAAC,SAAS,IAAI,UAAU;QACxC,YAAY,EAAE,KAAK,CAAC,YAAY,IAAI,MAAM;QAC1C,eAAe,EAAE,KAAK,CAAC,eAAe,IAAI,EAAE;KAC7C,CAAC,CAAC,CAAC;AACN,CAAC"}

package/dist/lifecycle/diff-tracker.d.ts

@@ -0,0 +1,72 @@
+/**
+ * DiffTracker — snapshot files at task start, produce a cumulative
+ * unified diff at any point during the lifecycle.
+ *
+ * Tool sweep #6 motivation: every reviewer template (spec / quality /
+ * diff) was reviewing the worker's TEXT CLAIM about its work, not the
+ * actual change on disk. Result:
+ *   - Spec reviewers defaulted to "changes_required" because they
+ *     could not verify a claim they couldn't see → endless rework
+ *     spirals on already-correct work.
+ *   - Quality findings were rooted in the worker's prose summary
+ *     rather than diff lines → false positives, missed regressions.
+ *   - The "diff reviewer" did not actually receive a diff (a misnomer).
+ *
+ * Fix: every reviewer prompt now receives `cumulativeDiff` — the
+ * unified diff of every change made since task start, regardless of
+ * which rework round produced it. With evidence in hand the reviewer
+ * can be precise: "diff matches brief? approve" — single round, no
+ * spiral, no waste.
+ *
+ * "Cumulative" matters: across spec_rework rounds 1..N, the reviewer
+ * needs to see the totality of edits so it can confirm prior reworks'
+ * additions are still present. A latest-round-only diff would make
+ * the round-2 reviewer say "you didn't add Y" when round-1 actually
+ * did.
+ *
+ * Implementation choice: snapshot-based, NOT git-based.
+ *   - Works in non-git directories (test fixtures, sandboxes,
+ *     fresh-scaffold projects).
+ *   - No assumption about HEAD state.
+ *   - No mutation of `.git/` (no `add -N`, no temporary index entries).
+ *   - Captures new files cleanly (baseline=null → full new-file diff).
+ *
+ * Diff format: unified diff (`@@ -before,count +after,count @@`)
+ * generated via Myers-style line LCS. The LLM is the consumer; the
+ * format is what every model has been trained on.
+ *
+ * Output cap: 50KB total. Excess is truncated with a "[diff truncated
+ * at 50KB — N more bytes elided]" marker so the LLM knows the diff is
+ * incomplete and can react accordingly (rather than silently judging
+ * a partial picture).
+ */
+export declare class DiffTracker {
+    private cwd;
+    private baselines;
+    constructor(cwd: string);
+    /**
+     * Capture current file content as the baseline. Idempotent: if a
+     * path was already snapshotted, the existing baseline wins (the
+     * earliest-known content is the canonical pre-task state).
+     *
+     * Call once at task start with all `task.filePaths`; call again
+     * lazily (via `ensureSnapshotted`) if a worker writes a path that
+     * wasn't pre-declared.
+     */
+    snapshot(relativePaths: ReadonlyArray<string>): Promise<void>;
+    /**
+     * Defensive: if the worker writes a file that wasn't pre-declared
+     * in `task.filePaths`, we don't have a baseline for it. Capture
+     * one NOW (whatever the file's current state) so subsequent diffs
+     * show only what THIS rework round changed for it. This is a
+     * best-effort fallback; the operator should declare filePaths.
+     */
+    ensureSnapshotted(relativePath: string): Promise<void>;
+    /** Return the unified diff of every snapshotted path against its
+     *  current on-disk content. Empty string when nothing changed.
+     *  Capped at MAX_DIFF_BYTES with a truncation marker. */
+    cumulativeDiff(): Promise<string>;
+    /** Test hook: how many baselines have been captured. */
+    size(): number;
+}
+//# sourceMappingURL=diff-tracker.d.ts.map

package/dist/lifecycle/diff-tracker.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-tracker.d.ts","sourceRoot":"","sources":["../../src/lifecycle/diff-tracker.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAyCG;AAYH,qBAAa,WAAW;IAGV,OAAO,CAAC,GAAG;IAFvB,OAAO,CAAC,SAAS,CAA+B;gBAE5B,GAAG,EAAE,MAAM;IAE/B;;;;;;;;OAQG;IACG,QAAQ,CAAC,aAAa,EAAE,aAAa,CAAC,MAAM,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAmBnE;;;;;;OAMG;IACG,iBAAiB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAK5D;;6DAEyD;IACnD,cAAc,IAAI,OAAO,CAAC,MAAM,CAAC;IAkBvC,wDAAwD;IACxD,IAAI,IAAI,MAAM;CAGf"}

package/dist/lifecycle/diff-tracker.js

@@ -0,0 +1,316 @@
+/**
+ * DiffTracker — snapshot files at task start, produce a cumulative
+ * unified diff at any point during the lifecycle.
+ *
+ * Tool sweep #6 motivation: every reviewer template (spec / quality /
+ * diff) was reviewing the worker's TEXT CLAIM about its work, not the
+ * actual change on disk. Result:
+ *   - Spec reviewers defaulted to "changes_required" because they
+ *     could not verify a claim they couldn't see → endless rework
+ *     spirals on already-correct work.
+ *   - Quality findings were rooted in the worker's prose summary
+ *     rather than diff lines → false positives, missed regressions.
+ *   - The "diff reviewer" did not actually receive a diff (a misnomer).
+ *
+ * Fix: every reviewer prompt now receives `cumulativeDiff` — the
+ * unified diff of every change made since task start, regardless of
+ * which rework round produced it. With evidence in hand the reviewer
+ * can be precise: "diff matches brief? approve" — single round, no
+ * spiral, no waste.
+ *
+ * "Cumulative" matters: across spec_rework rounds 1..N, the reviewer
+ * needs to see the totality of edits so it can confirm prior reworks'
+ * additions are still present. A latest-round-only diff would make
+ * the round-2 reviewer say "you didn't add Y" when round-1 actually
+ * did.
+ *
+ * Implementation choice: snapshot-based, NOT git-based.
+ *   - Works in non-git directories (test fixtures, sandboxes,
+ *     fresh-scaffold projects).
+ *   - No assumption about HEAD state.
+ *   - No mutation of `.git/` (no `add -N`, no temporary index entries).
+ *   - Captures new files cleanly (baseline=null → full new-file diff).
+ *
+ * Diff format: unified diff (`@@ -before,count +after,count @@`)
+ * generated via Myers-style line LCS. The LLM is the consumer; the
+ * format is what every model has been trained on.
+ *
+ * Output cap: 50KB total. Excess is truncated with a "[diff truncated
+ * at 50KB — N more bytes elided]" marker so the LLM knows the diff is
+ * incomplete and can react accordingly (rather than silently judging
+ * a partial picture).
+ */
+import * as fs from 'node:fs/promises';
+import * as path from 'node:path';
+const MAX_DIFF_BYTES = 50 * 1024; // 50KB cap to protect reviewer context window
+const CONTEXT_LINES = 3; // unified-diff convention
+export class DiffTracker {
+    cwd;
+    baselines = new Map();
+    constructor(cwd) {
+        this.cwd = cwd;
+    }
+    /**
+     * Capture current file content as the baseline. Idempotent: if a
+     * path was already snapshotted, the existing baseline wins (the
+     * earliest-known content is the canonical pre-task state).
+     *
+     * Call once at task start with all `task.filePaths`; call again
+     * lazily (via `ensureSnapshotted`) if a worker writes a path that
+     * wasn't pre-declared.
+     */
+    async snapshot(relativePaths) {
+        for (const rel of relativePaths) {
+            if (this.baselines.has(rel))
+                continue;
+            const abs = path.resolve(this.cwd, rel);
+            try {
+                const content = await fs.readFile(abs, 'utf-8');
+                this.baselines.set(rel, content);
+            }
+            catch (e) {
+                if (e.code === 'ENOENT') {
+                    // File didn't exist at task start. Worker may create it →
+                    // diff later will show as a full new-file diff.
+                    this.baselines.set(rel, null);
+                }
+                else {
+                    throw e;
+                }
+            }
+        }
+    }
+    /**
+     * Defensive: if the worker writes a file that wasn't pre-declared
+     * in `task.filePaths`, we don't have a baseline for it. Capture
+     * one NOW (whatever the file's current state) so subsequent diffs
+     * show only what THIS rework round changed for it. This is a
+     * best-effort fallback; the operator should declare filePaths.
+     */
+    async ensureSnapshotted(relativePath) {
+        if (this.baselines.has(relativePath))
+            return;
+        await this.snapshot([relativePath]);
+    }
+    /** Return the unified diff of every snapshotted path against its
+     *  current on-disk content. Empty string when nothing changed.
+     *  Capped at MAX_DIFF_BYTES with a truncation marker. */
+    async cumulativeDiff() {
+        const segments = [];
+        for (const [rel, before] of this.baselines.entries()) {
+            const abs = path.resolve(this.cwd, rel);
+            let after;
+            try {
+                after = await fs.readFile(abs, 'utf-8');
+            }
+            catch (e) {
+                if (e.code === 'ENOENT')
+                    after = null;
+                else
+                    throw e;
+            }
+            if (before === after)
+                continue;
+            const seg = formatUnifiedDiff(rel, before, after);
+            if (seg)
+                segments.push(seg);
+        }
+        return capWithMarker(segments.join('\n\n'));
+    }
+    /** Test hook: how many baselines have been captured. */
+    size() {
+        return this.baselines.size;
+    }
+}
+function capWithMarker(s) {
+    if (s.length <= MAX_DIFF_BYTES)
+        return s;
+    const elided = s.length - MAX_DIFF_BYTES;
+    return s.slice(0, MAX_DIFF_BYTES) + `\n\n[diff truncated at ${MAX_DIFF_BYTES} bytes — ${elided} more bytes elided]`;
+}
+/** Unified-diff for one file. */
+function formatUnifiedDiff(relPath, before, after) {
+    if (before === after)
+        return '';
+    const beforeLines = before === null ? [] : splitKeepNewlines(before);
+    const afterLines = after === null ? [] : splitKeepNewlines(after);
+    const aLabel = before === null ? '/dev/null' : `a/${relPath}`;
+    const bLabel = after === null ? '/dev/null' : `b/${relPath}`;
+    const ops = lineDiff(beforeLines, afterLines);
+    const hunks = collectHunks(ops, CONTEXT_LINES);
+    if (hunks.length === 0)
+        return '';
+    const out = [];
+    out.push(`--- ${aLabel}`);
+    out.push(`+++ ${bLabel}`);
+    for (const h of hunks) {
+        out.push(`@@ -${h.beforeStart + 1},${h.beforeCount} +${h.afterStart + 1},${h.afterCount} @@`);
+        for (const line of h.lines)
+            out.push(line);
+    }
+    return out.join('\n');
+}
+function splitKeepNewlines(s) {
+    // Split into logical lines. Trailing newline is preserved on each
+    // line except the last (which may or may not have one, like POSIX).
+    if (s === '')
+        return [];
+    const out = s.split('\n');
+    // If original ended with \n, the split produces a trailing '' which
+    // represents "the final newline" — drop it (unified diff lines DON'T
+    // include the trailing newline; it's implicit).
+    if (out.length > 0 && out[out.length - 1] === '')
+        out.pop();
+    return out;
+}
+/**
+ * Line-level diff using a simple LCS (Hirschberg / dynamic-programming).
+ * Returns ops in document order. For typical edits (a few lines changed
+ * in a file of hundreds of lines) this is O(N×M) but with N×M small.
+ * For huge files it's still bounded — the result is line-count
+ * sensitive, not byte-count sensitive.
+ */
+function lineDiff(a, b) {
+    const n = a.length;
+    const m = b.length;
+    // Build LCS length matrix.
+    const dp = Array.from({ length: n + 1 }, () => new Array(m + 1).fill(0));
+    for (let i = n - 1; i >= 0; i--) {
+        for (let j = m - 1; j >= 0; j--) {
+            if (a[i] === b[j])
+                dp[i][j] = dp[i + 1][j + 1] + 1;
+            else
+                dp[i][j] = Math.max(dp[i + 1][j], dp[i][j + 1]);
+        }
+    }
+    // Backtrack to produce ops.
+    const ops = [];
+    let i = 0, j = 0;
+    while (i < n && j < m) {
+        if (a[i] === b[j]) {
+            ops.push({ kind: 'equal', text: a[i] });
+            i++;
+            j++;
+        }
+        else if (dp[i + 1][j] >= dp[i][j + 1]) {
+            ops.push({ kind: 'delete', text: a[i] });
+            i++;
+        }
+        else {
+            ops.push({ kind: 'insert', text: b[j] });
+            j++;
+        }
+    }
+    while (i < n)
+        ops.push({ kind: 'delete', text: a[i++] });
+    while (j < m)
+        ops.push({ kind: 'insert', text: b[j++] });
+    return ops;
+}
+/**
+ * Group ops into unified-diff hunks with `context` lines of unchanged
+ * surrounding code. Skip pure-equal stretches longer than 2×context.
+ */
+function collectHunks(ops, context) {
+    const hunks = [];
+    let i = 0;
+    let beforeIdx = 0; // 0-based line index into the "before" file
+    let afterIdx = 0;
+    while (i < ops.length) {
+        // Skip purely-equal runs up to first change.
+        while (i < ops.length && ops[i].kind === 'equal') {
+            // Tentatively advance — but if a change is within `context`
+            // ahead, we'll back off to include it as preceding-context.
+            let lookahead = i;
+            while (lookahead < ops.length && lookahead < i + context && ops[lookahead].kind === 'equal')
+                lookahead++;
+            if (lookahead < ops.length && ops[lookahead].kind !== 'equal')
+                break;
+            beforeIdx++;
+            afterIdx++;
+            i++;
+        }
+        if (i >= ops.length)
+            break;
+        // Start a hunk. Include up to `context` preceding equal lines.
+        const hunkLines = [];
+        let preStart = Math.max(0, i - context);
+        // Walk back to preStart, but only include 'equal' ops on the way.
+        // Use bookkeeping: how many equal lines we step back.
+        const preLines = [];
+        let walkBack = i - 1;
+        while (walkBack >= preStart && ops[walkBack].kind === 'equal' && preLines.length < context) {
+            preLines.unshift(' ' + ops[walkBack].text);
+            walkBack--;
+        }
+        const hunkBeforeStart = beforeIdx - preLines.length;
+        const hunkAfterStart = afterIdx - preLines.length;
+        hunkLines.push(...preLines);
+        // Walk forward, collecting changes + interleaved equals (up to
+        // 2×context equals before splitting into a new hunk).
+        let beforeCount = preLines.length;
+        let afterCount = preLines.length;
+        let trailingEqualCount = 0;
+        while (i < ops.length) {
+            const op = ops[i];
+            if (op.kind === 'equal') {
+                if (trailingEqualCount >= 2 * context) {
+                    // Decide whether to split: peek to see if more changes coming.
+                    let look = i;
+                    while (look < ops.length && ops[look].kind === 'equal')
+                        look++;
+                    if (look >= ops.length || look - i > context) {
+                        // No more changes within range — close the hunk with `context` trailing.
+                        break;
+                    }
+                }
+                hunkLines.push(' ' + op.text);
+                beforeCount++;
+                afterCount++;
+                beforeIdx++;
+                afterIdx++;
+                trailingEqualCount++;
+                i++;
+            }
+            else if (op.kind === 'delete') {
+                hunkLines.push('-' + op.text);
+                beforeCount++;
+                beforeIdx++;
+                trailingEqualCount = 0;
+                i++;
+            }
+            else {
+                // insert
+                hunkLines.push('+' + op.text);
+                afterCount++;
+                afterIdx++;
+                trailingEqualCount = 0;
+                i++;
+            }
+        }
+        // Trim trailing context to exactly `context` lines if we accumulated more.
+        while (trailingEqualCount > context) {
+            const last = hunkLines[hunkLines.length - 1];
+            if (last.startsWith(' ')) {
+                hunkLines.pop();
+                beforeCount--;
+                afterCount--;
+                trailingEqualCount--;
+                beforeIdx--;
+                afterIdx--;
+                i--;
+            }
+            else
+                break;
+        }
+        hunks.push({
+            beforeStart: hunkBeforeStart,
+            beforeCount,
+            afterStart: hunkAfterStart,
+            afterCount,
+            lines: hunkLines,
+        });
+    }
+    return hunks;
+}
+//# sourceMappingURL=diff-tracker.js.map

package/dist/lifecycle/diff-tracker.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-tracker.js","sourceRoot":"","sources":["../../src/lifecycle/diff-tracker.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAyCG;AAEH,OAAO,KAAK,EAAE,MAAM,kBAAkB,CAAC;AACvC,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAElC,MAAM,cAAc,GAAG,EAAE,GAAG,IAAI,CAAC,CAAC,8CAA8C;AAChF,MAAM,aAAa,GAAG,CAAC,CAAC,CAAU,0BAA0B;AAM5D,MAAM,OAAO,WAAW;IAGF;IAFZ,SAAS,GAAG,IAAI,GAAG,EAAoB,CAAC;IAEhD,YAAoB,GAAW;QAAX,QAAG,GAAH,GAAG,CAAQ;IAAG,CAAC;IAEnC;;;;;;;;OAQG;IACH,KAAK,CAAC,QAAQ,CAAC,aAAoC;QACjD,KAAK,MAAM,GAAG,IAAI,aAAa,EAAE,CAAC;YAChC,IAAI,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,SAAS;YACtC,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;YACxC,IAAI,CAAC;gBACH,MAAM,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;gBAChD,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;YACnC,CAAC;YAAC,OAAO,CAAC,EAAE,CAAC;gBACX,IAAK,CAA2B,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;oBACnD,0DAA0D;oBAC1D,gDAAgD;oBAChD,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;gBAChC,CAAC;qBAAM,CAAC;oBACN,MAAM,CAAC,CAAC;gBACV,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;IAED;;;;;;OAMG;IACH,KAAK,CAAC,iBAAiB,CAAC,YAAoB;QAC1C,IAAI,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,YAAY,CAAC;YAAE,OAAO;QAC7C,MAAM,IAAI,CAAC,QAAQ,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC;IACtC,CAAC;IAED;;6DAEyD;IACzD,KAAK,CAAC,cAAc;QAClB,MAAM,QAAQ,GAAa,EAAE,CAAC;QAC9B,KAAK,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,EAAE,CAAC;YACrD,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC;YACxC,IAAI,KAAe,CAAC;YACpB,IAAI,CAAC;gBACH,KAAK,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;YAC1C,CAAC;YAAC,OAAO,CAAC,EAAE,CAAC;gBACX,IAAK,CAA2B,CAAC,IAAI,KAAK,QAAQ;oBAAE,KAAK,GAAG,IAAI,CAAC;;oBAC5D,MAAM,CAAC,CAAC;YACf,CAAC;YACD,IAAI,MAAM,KAAK,KAAK;gBAAE,SAAS;YAC/B,MAAM,GAAG,GAAG,iBAAiB,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,CAAC,CAAC;YAClD,IAAI,GAAG;gBAAE,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC9B,CAAC;QACD,OAAO,aAAa,CAAC,QAAQ,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC;IAC9C,CAAC;IAED,wDAAwD;IACxD,IAAI;QACF,OAAO,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC;IAC7B,CAAC;CACF;AAED,SAAS,aAAa,CAAC,CAAS;IAC9B,IAAI,CAAC,CAAC,MAAM,IAAI,cAAc;QAAE,OAAO,CAAC,CAAC;IACzC,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,GAAG,cAAc,CAAC;IACzC,OAAO,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,cAAc,CAAC,GAAG,0BAA0B,cAAc,YAAY,MAAM,qBAAqB,CAAC;AACtH,CAAC;AAED,iCAAiC;AACjC,SAAS,iBAAiB,CAAC,OAAe,EAAE,MAAgB,EAAE,KAAe;IAC3E,IAAI,MAAM,KAAK,KAAK;QAAE,OAAO,EAAE,CAAC;IAChC,MAAM,WAAW,GAAG,MAAM,KAAK,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,iBAAiB,CAAC,MAAM,CAAC,CAAC;IACrE,MAAM,UAAU,GAAG,KAAK,KAAK,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;IAElE,MAAM,MAAM,GAAG,MAAM,KAAK,IAAI,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,OAAO,EAAE,CAAC;IAC9D,MAAM,MAAM,GAAG,KAAK,KAAK,IAAI,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,OAAO,EAAE,CAAC;IAE7D,MAAM,GAAG,GAAG,QAAQ,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;IAC9C,MAAM,KAAK,GAAG,YAAY,CAAC,GAAG,EAAE,aAAa,CAAC,CAAC;IAC/C,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAElC,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,GAAG,CAAC,IAAI,CAAC,OAAO,MAAM,EAAE,CAAC,CAAC;IAC1B,GAAG,CAAC,IAAI,CAAC,OAAO,MAAM,EAAE,CAAC,CAAC;IAC1B,KAAK,MAAM,CAAC,IAAI,KAAK,EAAE,CAAC;QACtB,GAAG,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,WAAW,GAAG,CAAC,IAAI,CAAC,CAAC,WAAW,KAAK,CAAC,CAAC,UAAU,GAAG,CAAC,IAAI,CAAC,CAAC,UAAU,KAAK,CAAC,CAAC;QAC9F,KAAK,MAAM,IAAI,IAAI,CAAC,CAAC,KAAK;YAAE,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC7C,CAAC;IACD,OAAO,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACxB,CAAC;AAED,SAAS,iBAAiB,CAAC,CAAS;IAClC,kEAAkE;IAClE,oEAAoE;IACpE,IAAI,CAAC,KAAK,EAAE;QAAE,OAAO,EAAE,CAAC;IACxB,MAAM,GAAG,GAAG,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC1B,oEAAoE;IACpE,qEAAqE;IACrE,gDAAgD;IAChD,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC,IAAI,GAAG,CAAC,GAAG,CAAC,MAAM,GAAG,CAAC,CAAC,KAAK,EAAE;QAAE,GAAG,CAAC,GAAG,EAAE,CAAC;IAC5D,OAAO,GAAG,CAAC;AACb,CAAC;AAOD;;;;;;GAMG;AACH,SAAS,QAAQ,CAAC,CAAW,EAAE,CAAW;IACxC,MAAM,CAAC,GAAG,CAAC,CAAC,MAAM,CAAC;IACnB,MAAM,CAAC,GAAG,CAAC,CAAC,MAAM,CAAC;IACnB,2BAA2B;IAC3B,MAAM,EAAE,GAAe,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC,GAAG,CAAC,EAAE,EAAE,GAAG,EAAE,CAAC,IAAI,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;IACrF,KAAK,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;QAChC,KAAK,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAChC,IAAI,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;gBAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC;;gBAC9C,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;QACvD,CAAC;IACH,CAAC;IACD,4BAA4B;IAC5B,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC;IACjB,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC;QACtB,IAAI,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;YAClB,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YACxC,CAAC,EAAE,CAAC;YAAC,CAAC,EAAE,CAAC;QACX,CAAC;aAAM,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC;YACxC,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YACzC,CAAC,EAAE,CAAC;QACN,CAAC;aAAM,CAAC;YACN,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YACzC,CAAC,EAAE,CAAC;QACN,CAAC;IACH,CAAC;IACD,OAAO,CAAC,GAAG,CAAC;QAAE,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;IACzD,OAAO,CAAC,GAAG,CAAC;QAAE,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;IACzD,OAAO,GAAG,CAAC;AACb,CAAC;AAUD;;;GAGG;AACH,SAAS,YAAY,CAAC,GAAa,EAAE,OAAe;IAClD,MAAM,KAAK,GAAW,EAAE,CAAC;IACzB,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,IAAI,SAAS,GAAG,CAAC,CAAC,CAAC,4CAA4C;IAC/D,IAAI,QAAQ,GAAG,CAAC,CAAC;IAEjB,OAAO,CAAC,GAAG,GAAG,CAAC,MAAM,EAAE,CAAC;QACtB,6CAA6C;QAC7C,OAAO,CAAC,GAAG,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;YACjD,4DAA4D;YAC5D,4DAA4D;YAC5D,IAAI,SAAS,GAAG,CAAC,CAAC;YAClB,OAAO,SAAS,GAAG,GAAG,CAAC,MAAM,IAAI,SAAS,GAAG,CAAC,GAAG,OAAO,IAAI,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,KAAK,OAAO;gBAAE,SAAS,EAAE,CAAC;YACzG,IAAI,SAAS,GAAG,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,SAAS,CAAC,CAAC,IAAI,KAAK,OAAO;gBAAE,MAAM;YACrE,SAAS,EAAE,CAAC;YAAC,QAAQ,EAAE,CAAC;YAAC,CAAC,EAAE,CAAC;QAC/B,CAAC;QACD,IAAI,CAAC,IAAI,GAAG,CAAC,MAAM;YAAE,MAAM;QAE3B,+DAA+D;QAC/D,MAAM,SAAS,GAAa,EAAE,CAAC;QAC/B,IAAI,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC,GAAG,OAAO,CAAC,CAAC;QACxC,kEAAkE;QAClE,sDAAsD;QACtD,MAAM,QAAQ,GAAa,EAAE,CAAC;QAC9B,IAAI,QAAQ,GAAG,CAAC,GAAG,CAAC,CAAC;QACrB,OAAO,QAAQ,IAAI,QAAQ,IAAI,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,KAAK,OAAO,IAAI,QAAQ,CAAC,MAAM,GAAG,OAAO,EAAE,CAAC;YAC3F,QAAQ,CAAC,OAAO,CAAC,GAAG,GAAG,GAAG,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC;YAC3C,QAAQ,EAAE,CAAC;QACb,CAAC;QACD,MAAM,eAAe,GAAG,SAAS,GAAG,QAAQ,CAAC,MAAM,CAAC;QACpD,MAAM,cAAc,GAAG,QAAQ,GAAG,QAAQ,CAAC,MAAM,CAAC;QAClD,SAAS,CAAC,IAAI,CAAC,GAAG,QAAQ,CAAC,CAAC;QAE5B,+DAA+D;QAC/D,sDAAsD;QACtD,IAAI,WAAW,GAAG,QAAQ,CAAC,MAAM,CAAC;QAClC,IAAI,UAAU,GAAG,QAAQ,CAAC,MAAM,CAAC;QACjC,IAAI,kBAAkB,GAAG,CAAC,CAAC;QAC3B,OAAO,CAAC,GAAG,GAAG,CAAC,MAAM,EAAE,CAAC;YACtB,MAAM,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC;YAClB,IAAI,EAAE,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;gBACxB,IAAI,kBAAkB,IAAI,CAAC,GAAG,OAAO,EAAE,CAAC;oBACtC,+DAA+D;oBAC/D,IAAI,IAAI,GAAG,CAAC,CAAC;oBACb,OAAO,IAAI,GAAG,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,IAAI,CAAC,CAAC,IAAI,KAAK,OAAO;wBAAE,IAAI,EAAE,CAAC;oBAC/D,IAAI,IAAI,IAAI,GAAG,CAAC,MAAM,IAAI,IAAI,GAAG,CAAC,GAAG,OAAO,EAAE,CAAC;wBAC7C,yEAAyE;wBACzE,MAAM;oBACR,CAAC;gBACH,CAAC;gBACD,SAAS,CAAC,IAAI,CAAC,GAAG,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;gBAC9B,WAAW,EAAE,CAAC;gBAAC,UAAU,EAAE,CAAC;gBAC5B,SAAS,EAAE,CAAC;gBAAC,QAAQ,EAAE,CAAC;gBACxB,kBAAkB,EAAE,CAAC;gBACrB,CAAC,EAAE,CAAC;YACN,CAAC;iBAAM,IAAI,EAAE,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;gBAChC,SAAS,CAAC,IAAI,CAAC,GAAG,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;gBAC9B,WAAW,EAAE,CAAC;gBACd,SAAS,EAAE,CAAC;gBACZ,kBAAkB,GAAG,CAAC,CAAC;gBACvB,CAAC,EAAE,CAAC;YACN,CAAC;iBAAM,CAAC;gBACN,SAAS;gBACT,SAAS,CAAC,IAAI,CAAC,GAAG,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;gBAC9B,UAAU,EAAE,CAAC;gBACb,QAAQ,EAAE,CAAC;gBACX,kBAAkB,GAAG,CAAC,CAAC;gBACvB,CAAC,EAAE,CAAC;YACN,CAAC;QACH,CAAC;QAED,2EAA2E;QAC3E,OAAO,kBAAkB,GAAG,OAAO,EAAE,CAAC;YACpC,MAAM,IAAI,GAAG,SAAS,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;YAC7C,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;gBACzB,SAAS,CAAC,GAAG,EAAE,CAAC;gBAChB,WAAW,EAAE,CAAC;gBAAC,UAAU,EAAE,CAAC;gBAAC,kBAAkB,EAAE,CAAC;gBAClD,SAAS,EAAE,CAAC;gBAAC,QAAQ,EAAE,CAAC;gBACxB,CAAC,EAAE,CAAC;YACN,CAAC;;gBAAM,MAAM;QACf,CAAC;QAED,KAAK,CAAC,IAAI,CAAC;YACT,WAAW,EAAE,eAAe;YAC5B,WAAW;YACX,UAAU,EAAE,cAAc;YAC1B,UAAU;YACV,KAAK,EAAE,SAAS;SACjB,CAAC,CAAC;IACL,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC"}

package/dist/lifecycle/handlers/prepare-execution-context-handler.d.ts

@@ -15,5 +15,5 @@ import type { LifecycleState } from '../stage-plan-types.js';
  *     so state.task is the per-dispatch task. This handler surfaces task[0]
  *     as a fallback for downstream handlers that need a TaskSpec.
  */
-export declare function prepareExecutionContextHandler(state: LifecycleState): void;
+export declare function prepareExecutionContextHandler(state: LifecycleState): Promise<void>;
 //# sourceMappingURL=prepare-execution-context-handler.d.ts.map

package/dist/lifecycle/handlers/prepare-execution-context-handler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prepare-execution-context-handler.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/prepare-execution-context-handler.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAG7D;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,8BAA8B,CAAC,KAAK,EAAE,cAAc,GAAG,IAAI,CAqB1E"}
+{"version":3,"file":"prepare-execution-context-handler.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/prepare-execution-context-handler.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAI7D;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,8BAA8B,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAyCzF"}

package/dist/lifecycle/handlers/prepare-execution-context-handler.js

@@ -1,3 +1,4 @@
+import { DiffTracker } from '../diff-tracker.js';
 /**
  * StageHandler for row 2.5 (prepare_execution_context).
  *
@@ -14,7 +15,7 @@
  *     so state.task is the per-dispatch task. This handler surfaces task[0]
  *     as a fallback for downstream handlers that need a TaskSpec.
  */
-export function prepareExecutionContextHandler(state) {
+export async function prepareExecutionContextHandler(state) {
     // Fallback: if rawRequest carries a TaskSpec[] and state.task is empty,
     // surface the first task so per-task handlers have something to read.
     if (!state.task) {
@@ -34,5 +35,26 @@ export function prepareExecutionContextHandler(state) {
     if (task && task.reviewPolicy) {
         state.reviewPolicy = task.reviewPolicy;
     }
+    // Tool sweep #6: snapshot the worker's declared filePaths BEFORE the
+    // implementer runs so reviewer stages can produce a cumulative diff
+    // against the pre-task baseline. Skip read-only routes (audit / review
+    // / verify / debug / investigate / explore) — they don't write files
+    // by sandbox policy, so a tracker would just be empty noise.
+    if (!state.diffTracker && task && state.toolCategory !== 'read_only') {
+        const cwd = task.cwd;
+        const filePaths = Array.isArray(task.filePaths) ? task.filePaths : [];
+        if (cwd && filePaths.length > 0) {
+            const tracker = new DiffTracker(cwd);
+            try {
+                await tracker.snapshot(filePaths);
+                state.diffTracker = tracker;
+            }
+            catch {
+                // Snapshot failures (permission, unreadable) shouldn't block the
+                // task. Reviewer just sees an empty diff and falls back to the
+                // worker-output-only path — degraded but not broken.
+            }
+        }
+    }
 }
 //# sourceMappingURL=prepare-execution-context-handler.js.map

package/dist/lifecycle/handlers/prepare-execution-context-handler.js.map

@@ -1 +1 @@
-{"version":3,"file":"prepare-execution-context-handler.js","sourceRoot":"","sources":["../../../src/lifecycle/handlers/prepare-execution-context-handler.ts"],"names":[],"mappings":"AAGA;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,8BAA8B,CAAC,KAAqB;IAClE,wEAAwE;IACxE,sEAAsE;IACtE,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QAChB,MAAM,GAAG,GAAG,KAAK,CAAC,OAA6C,CAAC;QAChE,MAAM,KAAK,GAAG,GAAG,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,CAAC;QAC9B,IAAI,KAAK;YAAE,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC;IAChC,CAAC;IAED,4EAA4E;IAC5E,yEAAyE;IACzE,sEAAsE;IACtE,2DAA2D;IAC3D,4EAA4E;IAC5E,4EAA4E;IAC5E,mEAAmE;IACnE,MAAM,IAAI,GAAG,KAAK,CAAC,IAA4B,CAAC;IAChD,IAAI,IAAI,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;QAC9B,KAAK,CAAC,YAAY,GAAG,IAAI,CAAC,YAAY,CAAC;IACzC,CAAC;AAEH,CAAC"}
+{"version":3,"file":"prepare-execution-context-handler.js","sourceRoot":"","sources":["../../../src/lifecycle/handlers/prepare-execution-context-handler.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAEjD;;;;;;;;;;;;;;;GAeG;AACH,MAAM,CAAC,KAAK,UAAU,8BAA8B,CAAC,KAAqB;IACxE,wEAAwE;IACxE,sEAAsE;IACtE,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QAChB,MAAM,GAAG,GAAG,KAAK,CAAC,OAA6C,CAAC;QAChE,MAAM,KAAK,GAAG,GAAG,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,CAAC;QAC9B,IAAI,KAAK;YAAE,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC;IAChC,CAAC;IAED,4EAA4E;IAC5E,yEAAyE;IACzE,sEAAsE;IACtE,2DAA2D;IAC3D,4EAA4E;IAC5E,4EAA4E;IAC5E,mEAAmE;IACnE,MAAM,IAAI,GAAG,KAAK,CAAC,IAA4B,CAAC;IAChD,IAAI,IAAI,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;QAC9B,KAAK,CAAC,YAAY,GAAG,IAAI,CAAC,YAAY,CAAC;IACzC,CAAC;IAED,qEAAqE;IACrE,oEAAoE;IACpE,uEAAuE;IACvE,qEAAqE;IACrE,6DAA6D;IAC7D,IAAI,CAAC,KAAK,CAAC,WAAW,IAAI,IAAI,IAAI,KAAK,CAAC,YAAY,KAAK,WAAW,EAAE,CAAC;QACrE,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,CAAC;QACrB,MAAM,SAAS,GAAG,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC;QACtE,IAAI,GAAG,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAChC,MAAM,OAAO,GAAG,IAAI,WAAW,CAAC,GAAG,CAAC,CAAC;YACrC,IAAI,CAAC;gBACH,MAAM,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,CAAC;gBAClC,KAAK,CAAC,WAAW,GAAG,OAAO,CAAC;YAC9B,CAAC;YAAC,MAAM,CAAC;gBACP,iEAAiE;gBACjE,+DAA+D;gBAC/D,qDAAqD;YACvD,CAAC;QACH,CAAC;IACH,CAAC;AACH,CAAC"}

package/dist/lifecycle/handlers/quality-chain-handlers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"quality-chain-handlers.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/quality-chain-handlers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAqU7D,eAAO,MAAM,0BAA0B,UAtIkB,cAAc,KAAG,OAAO,CAAC,IAAI,CAsIjB,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAvIkB,cAAc,KAAG,OAAO,CAAC,IAAI,CAuIjB,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAxIkB,cAAc,KAAG,OAAO,CAAC,IAAI,CAwIjB,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAjDa,cAAc,KAAG,OAAO,CAAC,IAAI,CAiDZ,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAlDa,cAAc,KAAG,OAAO,CAAC,IAAI,CAkDZ,CAAC;AAEtE;;;;;;;;;;;;;GAaG;AACH,wBAAgB,yBAAyB,CAAC,KAAK,EAAE,cAAc,GAAG,IAAI,CAmBrE"}
+{"version":3,"file":"quality-chain-handlers.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/quality-chain-handlers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAmW7D,eAAO,MAAM,0BAA0B,UApJkB,cAAc,KAAG,OAAO,CAAC,IAAI,CAoJjB,CAAC;AACtE,eAAO,MAAM,0BAA0B,UArJkB,cAAc,KAAG,OAAO,CAAC,IAAI,CAqJjB,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAtJkB,cAAc,KAAG,OAAO,CAAC,IAAI,CAsJjB,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAhDa,cAAc,KAAG,OAAO,CAAC,IAAI,CAgDZ,CAAC;AACtE,eAAO,MAAM,0BAA0B,UAjDa,cAAc,KAAG,OAAO,CAAC,IAAI,CAiDZ,CAAC;AAEtE;;;;;;;;;;;;;GAaG;AACH,wBAAgB,yBAAyB,CAAC,KAAK,EAAE,cAAc,GAAG,IAAI,CAyCrE"}