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

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

@@ -1,191 +0,0 @@
-// v4.4.x — Rework stage.
-//
-// Fires only when reviewVerdict === 'changes_required'. Runs on the
-// same standard session that did Implementing (full conversation
-// continuity). Worker is asked to address the reviewer's concerns.
-// Rework's WorkerOutput merges onto Implementing's per the spec's
-// "Rework → Implementing field merge rules": summary/workerStatus/unresolved/commitMessage take Rework's
-// values; filesChanged is the union of both phases.
-import { reviewPayload } from '../stage-plan-types.js';
-import { replaceLastRunResultPreservingTrackers, mergeStageStats } from '../merge-stage-stats.js';
-import { reworkPrompt } from './rework-prompt.js';
-import { buildWarmFollowupMessage } from '../warm-followup.js';
-import { assembleRunResult } from '../../providers/assemble-run-result.js';
-import { parseWorkerOutput } from '../worker-output-contract.js';
-import { HUMAN_LABEL } from '../stage-labels.js';
-import { startProgressWatchdog, recordPostHocSignals } from '../../bounded-execution/progress-watchdog.js';
-function reworkSkip(comment, t0) {
-    return {
-        outcome: 'skip',
-        comment,
-        payload: null,
-        telemetry: { stageLabel: 'rework', durationMs: Date.now() - t0, costUSD: 0, turnsUsed: 0, stopReason: 'normal' },
-    };
-}
-function reworkHalt(comment, t0) {
-    return {
-        outcome: 'halt',
-        comment,
-        payload: null,
-        telemetry: { stageLabel: 'rework', durationMs: Date.now() - t0, costUSD: 0, turnsUsed: 0, stopReason: 'transport_error' },
-    };
-}
-export async function reworkHandler(state) {
-    const t0 = Date.now();
-    if (state.terminal)
-        return reworkSkip('rework skipped: terminal', t0);
-    if (state.reworkApplied !== undefined || state.reworkError !== undefined) {
-        return reworkSkip('rework already applied', t0);
-    }
-    if (reviewPayload(state).verdict !== 'changes_required') {
-        return reworkSkip('rework skipped: review verdict is not changes_required', t0);
-    }
-    const ctx = state.executionContext;
-    const task = state.task;
-    const last = state.lastRunResult;
-    if (!ctx || !task || !last)
-        return reworkSkip('rework skipped: missing context', t0);
-    const findings = reviewPayload(state).findings;
-    if (findings.length === 0) {
-        state.reworkApplied = false;
-        return reworkSkip('rework skipped: review produced no findings', t0);
-    }
-    let cumulativeDiff = '';
-    if (state.diffTracker) {
-        try {
-            cumulativeDiff = await state.diffTracker.cumulativeDiff();
-        }
-        catch { /* tolerated */ }
-    }
-    // Rework matches the implementer's tier — rework's job is to FIX the
-    // implementer's work, so it needs the same capability. Read implementer
-    // tier from executionContext.assignedTier; fall back to the implementing
-    // stage's gate payload, then 'standard' as a defensive last resort.
-    // Final fallback: if the matched tier has no provider configured, use
-    // whichever tier does (parity with reviewer's fallback behavior).
-    const desiredReworkTier = ctx.assignedTier
-        ?? (state.gates?.['implement']?.payload?.agentTier)
-        ?? 'standard';
-    const reworkTier = ctx.providers[desiredReworkTier]
-        ? desiredReworkTier
-        : (ctx.providers['standard'] ? 'standard' : 'complex');
-    const provider = ctx.providers[reworkTier];
-    if (!provider) {
-        state.reworkError = `no provider available for tier ${reworkTier}`;
-        return reworkHalt(`no provider available for tier ${reworkTier}`, t0);
-    }
-    const concerns = findings.map((f) => `[${f.source}] ${f.text}`);
-    const promptCtx = {
-        brief: task.prompt ?? '',
-        workerOutput: last.output ?? '',
-        diff: cumulativeDiff,
-        planContext: task.planContext,
-        priorConcerns: concerns,
-    };
-    // Rework always resumes the implementer's thread — the systemPrompt,
-    // brief, prior output, and cumulative diff are already in conversation
-    // history. We send only the new instruction (reviewer deviations +
-    // fix action) wrapped in the standard warm-followup preamble.
-    const fullPrompt = buildWarmFollowupMessage(reworkPrompt(promptCtx))
-        + '\n\nDo NOT run git history-mutating commands (commit / add / push / reset / rebase / etc.) — the Committing stage will handle persistence at the end.';
-    let result;
-    // Wire progress watchdog around the rework session.send.
-    const wdConfig = {
-        enabled: ctx.config?.defaults?.progressWatchdogEnabled ?? true,
-        thrashTurns: ctx.config?.defaults?.thrashTurns ?? 25,
-        thrashWallClockMs: ctx.config?.defaults?.thrashWallClockMs ?? 1_200_000,
-        thrashSoftWallClockMs: 600_000,
-    };
-    const wdState2 = { fired: false };
-    const wdController = ctx.stall.controller;
-    let disposeWd;
-    if (wdConfig.enabled) {
-        disposeWd = startProgressWatchdog({
-            state,
-            controller: wdController,
-            emit: (_event) => { },
-            config: wdConfig,
-            taskIndex: ctx.taskIndex,
-            batchId: ctx.batchId,
-            state2: wdState2,
-        });
-    }
-    let turn;
-    try {
-        const session = ctx.getSession(reworkTier);
-        turn = await session.send(fullPrompt, { stageLabel: HUMAN_LABEL.rework });
-        result = assembleRunResult(turn);
-    }
-    catch (err) {
-        state.reworkError = err instanceof Error ? err.message : String(err);
-        disposeWd?.();
-        return reworkHalt(`rework session.send failed: ${state.reworkError}`, t0);
-    }
-    finally {
-        disposeWd?.();
-    }
-    if (wdConfig.enabled && turn !== undefined) {
-        await recordPostHocSignals(state, turn.turns ?? 0, wdConfig, (_event) => { }, ctx.taskIndex, ctx.batchId);
-    }
-    if (result.status !== 'ok') {
-        state.reworkError = `rework returned status: ${result.status}`;
-        return reworkHalt(state.reworkError, t0);
-    }
-    // Parse the Rework worker's WorkerOutput JSON block.
-    const reworked = parseWorkerOutput(result.output ?? '');
-    state.reworkApplied = true;
-    state.reworkOutput = result.output;
-    replaceLastRunResultPreservingTrackers(state, result);
-    // Apply merge rules: Rework owns summary/workerStatus/unresolved/commitMessage;
-    // filesChanged is the union of Implementing's + Rework's.
-    const merged = state.lastRunResult;
-    if (merged) {
-        const priorFilesChanged = (last.filesChanged) ?? [];
-        merged.summary = reworked.summary;
-        merged.workerStatus = reworked.workerSelfAssessment;
-        merged.filesChanged = Array.from(new Set([...priorFilesChanged, ...reworked.filesChanged]));
-        // v5: unresolved, commitMessage removed from worker output schema
-        merged.unresolved = reworked.unresolved ?? [];
-        if (reworked.commitMessage)
-            merged.commitMessage = reworked.commitMessage;
-        // Persist parsedCleanly from rework so enrichRuntimeResult can access it
-        merged.parsedCleanly = reworked.parsedCleanly;
-    }
-    mergeStageStats(state, 'rework', {
-        inputTokens: result.usage?.inputTokens ?? 0,
-        outputTokens: result.usage?.outputTokens ?? 0,
-        cachedReadTokens: result.usage?.cachedReadTokens ?? 0,
-        cachedNonReadTokens: result.usage?.cachedNonReadTokens ?? 0,
-        turnCount: result.turns ?? 1,
-        // `result` comes from assembleRunResult which writes the turn cost to
-        // top-level `actualCostUSD` (not a `costUSD` field). Reading the wrong
-        // field name was the historical cause of rework stages recording
-        // cost=null/0 in telemetry; canonical lookup is `actualCostUSD` with
-        // legacy `costUSD` as a safety fallback.
-        costUSD: result.actualCostUSD ?? result.costUSD ?? null,
-        durationMs: result.durationMs ?? null,
-        filesWrittenCount: Array.isArray(result.filesWritten) ? result.filesWritten.length : 0,
-    }, {
-        tier: reworkTier,
-        model: ctx.providers[reworkTier]?.config?.model ?? null,
-    });
-    const payload = {
-        workerSelfAssessment: reworked.workerSelfAssessment === 'done' ? 'done' : 'failed',
-        summary: reworked.summary ?? '',
-        filesChanged: state.lastRunResult?.filesChanged ?? [],
-        unaddressedFindingIds: [],
-        parsedCleanly: reworked.parsedCleanly,
-    };
-    return {
-        outcome: 'advance',
-        payload,
-        telemetry: {
-            stageLabel: 'rework',
-            durationMs: Date.now() - t0,
-            costUSD: result.actualCostUSD ?? null,
-            turnsUsed: result.turns ?? 1,
-            stopReason: 'normal',
-        },
-    };
-}
-//# sourceMappingURL=rework-stage.js.map

package/dist/lifecycle/handlers/rework-stage.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"rework-stage.js","sourceRoot":"","sources":["../../../src/lifecycle/handlers/rework-stage.ts"],"names":[],"mappings":"AAAA,yBAAyB;AACzB,EAAE;AACF,oEAAoE;AACpE,iEAAiE;AACjE,mEAAmE;AACnE,kEAAkE;AAClE,yGAAyG;AACzG,oDAAoD;AAGpD,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AAKvD,OAAO,EAAE,sCAAsC,EAAE,eAAe,EAAE,MAAM,yBAAyB,CAAC;AAClG,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,EAAE,wBAAwB,EAAE,MAAM,qBAAqB,CAAC;AAC/D,OAAO,EAAE,iBAAiB,EAAE,MAAM,wCAAwC,CAAC;AAC3E,OAAO,EAAE,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACjE,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AACjD,OAAO,EAAE,qBAAqB,EAAE,oBAAoB,EAAE,MAAM,8CAA8C,CAAC;AAG3G,SAAS,UAAU,CAAC,OAAe,EAAE,EAAU;IAC7C,OAAO;QACL,OAAO,EAAE,MAAM;QACf,OAAO;QACP,OAAO,EAAE,IAAI;QACb,SAAS,EAAE,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,UAAU,EAAE,QAAQ,EAAE;KACjH,CAAC;AACJ,CAAC;AACD,SAAS,UAAU,CAAC,OAAe,EAAE,EAAU;IAC7C,OAAO;QACL,OAAO,EAAE,MAAM;QACf,OAAO;QACP,OAAO,EAAE,IAAI;QACb,SAAS,EAAE,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,OAAO,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,UAAU,EAAE,iBAAiB,EAAE;KAC1H,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,aAAa,CAAC,KAAqB;IACvD,MAAM,EAAE,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACtB,IAAI,KAAK,CAAC,QAAQ;QAAE,OAAO,UAAU,CAAC,0BAA0B,EAAE,EAAE,CAAC,CAAC;IACtE,IAAI,KAAK,CAAC,aAAa,KAAK,SAAS,IAAI,KAAK,CAAC,WAAW,KAAK,SAAS,EAAE,CAAC;QACzE,OAAO,UAAU,CAAC,wBAAwB,EAAE,EAAE,CAAC,CAAC;IAClD,CAAC;IACD,IAAI,aAAa,CAAC,KAAK,CAAC,CAAC,OAAO,KAAK,kBAAkB,EAAE,CAAC;QACxD,OAAO,UAAU,CAAC,wDAAwD,EAAE,EAAE,CAAC,CAAC;IAClF,CAAC;IAED,MAAM,GAAG,GAAG,KAAK,CAAC,gBAAgD,CAAC;IACnE,MAAM,IAAI,GAAG,KAAK,CAAC,IAA4B,CAAC;IAChD,MAAM,IAAI,GAAG,KAAK,CAAC,aAA6C,CAAC;IACjE,IAAI,CAAC,GAAG,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI;QAAE,OAAO,UAAU,CAAC,iCAAiC,EAAE,EAAE,CAAC,CAAC;IAErF,MAAM,QAAQ,GAAG,aAAa,CAAC,KAAK,CAAC,CAAC,QAAQ,CAAC;IAC/C,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC1B,KAAK,CAAC,aAAa,GAAG,KAAK,CAAC;QAC5B,OAAO,UAAU,CAAC,6CAA6C,EAAE,EAAE,CAAC,CAAC;IACvE,CAAC;IAED,IAAI,cAAc,GAAG,EAAE,CAAC;IACxB,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;QACtB,IAAI,CAAC;YAAC,cAAc,GAAG,MAAM,KAAK,CAAC,WAAW,CAAC,cAAc,EAAE,CAAC;QAAC,CAAC;QAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC;IAC9F,CAAC;IAED,qEAAqE;IACrE,wEAAwE;IACxE,yEAAyE;IACzE,oEAAoE;IACpE,sEAAsE;IACtE,kEAAkE;IAClE,MAAM,iBAAiB,GACpB,GAAoC,CAAC,YAAY;WAC/C,CAAE,KAAK,CAAC,KAAK,EAAE,CAAC,WAAW,CAAC,EAAE,OAA4C,EAAE,SAAS,CAAC;WACtF,UAAU,CAAC;IAChB,MAAM,UAAU,GAAc,GAAG,CAAC,SAAS,CAAC,iBAAiB,CAAC;QAC5D,CAAC,CAAC,iBAAiB;QACnB,CAAC,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC;IACzD,MAAM,QAAQ,GAAG,GAAG,CAAC,SAAS,CAAC,UAAU,CAAyB,CAAC;IACnE,IAAI,CAAC,QAAQ,EAAE,CAAC;QACd,KAAK,CAAC,WAAW,GAAG,kCAAkC,UAAU,EAAE,CAAC;QACnE,OAAO,UAAU,CAAC,kCAAkC,UAAU,EAAE,EAAE,EAAE,CAAC,CAAC;IACxE,CAAC;IAED,MAAM,QAAQ,GAAG,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;IAChE,MAAM,SAAS,GAAG;QAChB,KAAK,EAAE,IAAI,CAAC,MAAM,IAAI,EAAE;QACxB,YAAY,EAAE,IAAI,CAAC,MAAM,IAAI,EAAE;QAC/B,IAAI,EAAE,cAAc;QACpB,WAAW,EAAG,IAAiC,CAAC,WAAW;QAC3D,aAAa,EAAE,QAAQ;KACxB,CAAC;IACF,qEAAqE;IACrE,uEAAuE;IACvE,mEAAmE;IACnE,8DAA8D;IAC9D,MAAM,UAAU,GACd,wBAAwB,CAAC,YAAY,CAAC,SAAS,CAAC,CAAC;UAC/C,uJAAuJ,CAAC;IAE5J,IAAI,MAAwB,CAAC;IAC7B,yDAAyD;IACzD,MAAM,QAAQ,GAA2B;QACvC,OAAO,EAAG,GAAG,CAAC,MAAM,EAAE,QAAkD,EAAE,uBAAuB,IAAI,IAAI;QACzG,WAAW,EAAG,GAAG,CAAC,MAAM,EAAE,QAAqC,EAAE,WAAW,IAAI,EAAE;QAClF,iBAAiB,EAAG,GAAG,CAAC,MAAM,EAAE,QAA2C,EAAE,iBAAiB,IAAI,SAAS;QAC3G,qBAAqB,EAAE,OAAO;KAC/B,CAAC;IACF,MAAM,QAAQ,GAAG,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC;IAClC,MAAM,YAAY,GAAG,GAAG,CAAC,KAAK,CAAC,UAAU,CAAC;IAC1C,IAAI,SAAmC,CAAC;IACxC,IAAI,QAAQ,CAAC,OAAO,EAAE,CAAC;QACrB,SAAS,GAAG,qBAAqB,CAAC;YAChC,KAAK;YACL,UAAU,EAAE,YAAY;YACxB,IAAI,EAAE,CAAC,MAAM,EAAE,EAAE,GAAsF,CAAC;YACxG,MAAM,EAAE,QAAQ;YAChB,SAAS,EAAE,GAAG,CAAC,SAAS;YACxB,OAAO,EAAE,GAAG,CAAC,OAAO;YACpB,MAAM,EAAE,QAAQ;SACjB,CAAC,CAAC;IACL,CAAC;IACD,IAAI,IAAsD,CAAC;IAC3D,IAAI,CAAC;QACH,MAAM,OAAO,GAAG,GAAG,CAAC,UAAU,CAAC,UAAU,CAAC,CAAC;QAC3C,IAAI,GAAG,MAAM,OAAO,CAAC,IAAI,CAAC,UAAU,EAAE,EAAE,UAAU,EAAE,WAAW,CAAC,MAAM,EAAE,CAAC,CAAC;QAC1E,MAAM,GAAG,iBAAiB,CAAC,IAAI,CAAC,CAAC;IACnC,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,KAAK,CAAC,WAAW,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;QACrE,SAAS,EAAE,EAAE,CAAC;QACd,OAAO,UAAU,CAAC,+BAA+B,KAAK,CAAC,WAAW,EAAE,EAAE,EAAE,CAAC,CAAC;IAC5E,CAAC;YAAS,CAAC;QACT,SAAS,EAAE,EAAE,CAAC;IAChB,CAAC;IACD,IAAI,QAAQ,CAAC,OAAO,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;QAC3C,MAAM,oBAAoB,CACxB,KAAK,EACJ,IAA2B,CAAC,KAAK,IAAI,CAAC,EACvC,QAAQ,EACR,CAAC,MAAM,EAAE,EAAE,GAA0E,CAAC,EACtF,GAAG,CAAC,SAAS,EACb,GAAG,CAAC,OAAO,CACZ,CAAC;IACJ,CAAC;IAED,IAAI,MAAM,CAAC,MAAM,KAAK,IAAI,EAAE,CAAC;QAC3B,KAAK,CAAC,WAAW,GAAG,2BAA2B,MAAM,CAAC,MAAM,EAAE,CAAC;QAC/D,OAAO,UAAU,CAAC,KAAK,CAAC,WAAW,EAAE,EAAE,CAAC,CAAC;IAC3C,CAAC;IAED,qDAAqD;IACrD,MAAM,QAAQ,GAAG,iBAAiB,CAAC,MAAM,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC;IAExD,KAAK,CAAC,aAAa,GAAG,IAAI,CAAC;IAC3B,KAAK,CAAC,YAAY,GAAG,MAAM,CAAC,MAAM,CAAC;IACnC,sCAAsC,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;IAEtD,gFAAgF;IAChF,0DAA0D;IAC1D,MAAM,MAAM,GAAG,KAAK,CAAC,aAAoD,CAAC;IAC1E,IAAI,MAAM,EAAE,CAAC;QACX,MAAM,iBAAiB,GAAG,CAAE,IAAoC,CAAC,YAAY,CAAC,IAAI,EAAE,CAAC;QACrF,MAAM,CAAC,OAAO,GAAG,QAAQ,CAAC,OAAO,CAAC;QAClC,MAAM,CAAC,YAAY,GAAG,QAAQ,CAAC,oBAAoB,CAAC;QACpD,MAAM,CAAC,YAAY,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,GAAG,CAAC,CAAC,GAAG,iBAAiB,EAAE,GAAG,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAC5F,kEAAkE;QAClE,MAAM,CAAC,UAAU,GAAI,QAAgB,CAAC,UAAU,IAAI,EAAE,CAAC;QACvD,IAAK,QAAgB,CAAC,aAAa;YAAE,MAAM,CAAC,aAAa,GAAI,QAAgB,CAAC,aAAa,CAAC;QAC5F,yEAAyE;QACzE,MAAM,CAAC,aAAa,GAAG,QAAQ,CAAC,aAAa,CAAC;IAChD,CAAC;IAED,eAAe,CAAC,KAAK,EAAE,QAAQ,EAAE;QAC/B,WAAW,EAAE,MAAM,CAAC,KAAK,EAAE,WAAW,IAAI,CAAC;QAC3C,YAAY,EAAE,MAAM,CAAC,KAAK,EAAE,YAAY,IAAI,CAAC;QAC7C,gBAAgB,EAAE,MAAM,CAAC,KAAK,EAAE,gBAAgB,IAAI,CAAC;QACrD,mBAAmB,EAAE,MAAM,CAAC,KAAK,EAAE,mBAAmB,IAAI,CAAC;QAC3D,SAAS,EAAE,MAAM,CAAC,KAAK,IAAI,CAAC;QAC5B,sEAAsE;QACtE,uEAAuE;QACvE,iEAAiE;QACjE,qEAAqE;QACrE,yCAAyC;QACzC,OAAO,EAAG,MAA4C,CAAC,aAAa,IAAK,MAAsC,CAAC,OAAO,IAAI,IAAI;QAC/H,UAAU,EAAG,MAAkC,CAAC,UAAU,IAAI,IAAI;QAClE,iBAAiB,EAAE,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;KACvF,EAAE;QACD,IAAI,EAAE,UAAU;QAChB,KAAK,EAAG,GAAG,CAAC,SAAS,CAAC,UAAU,CAAC,EAAE,MAAyC,EAAE,KAAK,IAAI,IAAI;KAC5F,CAAC,CAAC;IAEH,MAAM,OAAO,GAAkB;QAC7B,oBAAoB,EAAE,QAAQ,CAAC,oBAAoB,KAAK,MAAM,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ;QAClF,OAAO,EAAE,QAAQ,CAAC,OAAO,IAAI,EAAE;QAC/B,YAAY,EAAG,KAAK,CAAC,aAAyD,EAAE,YAAY,IAAI,EAAE;QAClG,qBAAqB,EAAE,EAAE;QACzB,aAAa,EAAE,QAAQ,CAAC,aAAa;KACtC,CAAC;IACF,OAAO;QACL,OAAO,EAAE,SAAS;QAClB,OAAO;QACP,SAAS,EAAE;YACT,UAAU,EAAE,QAAQ;YACpB,UAAU,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE;YAC3B,OAAO,EAAG,MAA4C,CAAC,aAAa,IAAI,IAAI;YAC5E,SAAS,EAAE,MAAM,CAAC,KAAK,IAAI,CAAC;YAC5B,UAAU,EAAE,QAAQ;SACrB;KACF,CAAC;AACJ,CAAC"}

package/dist/lifecycle/handlers/spec-review-prompt.d.ts

@@ -1,8 +0,0 @@
-export declare const OUTPUT_FORMAT: string;
-export declare function specReviewPrompt(ctx: {
-    brief: string;
-    workerSummary: string;
-    filesChanged: string[];
-    diff?: string;
-}): string;
-//# sourceMappingURL=spec-review-prompt.d.ts.map

package/dist/lifecycle/handlers/spec-review-prompt.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"spec-review-prompt.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/spec-review-prompt.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,aAAa,QA0BlB,CAAC;AAET,wBAAgB,gBAAgB,CAAC,GAAG,EAAE;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,aAAa,EAAE,MAAM,CAAC;IAAC,YAAY,EAAE,MAAM,EAAE,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,MAAM,CAkB7H"}

package/dist/lifecycle/handlers/spec-review-prompt.js

@@ -1,47 +0,0 @@
-export const OUTPUT_FORMAT = `
-## Verdict
-approved | changes_required
-
-## Findings
-Emit zero or more findings using EXACTLY this block format. Each finding is its own block.
-
-## Finding N: <one-line claim>
-- Severity: critical | high | medium | low
-- Category: <one word — e.g. missing-step, wrong-file, broken-contract>
-- Evidence: <verbatim excerpt from source, ≥20 chars — or (none) if inferable>
-- Suggestion: <one sentence — how to fix it>
-
-## Finding N+1:
-...
-
-If no findings, write "## Findings\n(none)".
-
-## Outcome
-found | clean
-
-**Severity definitions (per spec-review):**
-- **critical:** Plan step missed/wrong such that feature won't work
-- **high:** Plan step partially implemented
-- **medium:** Diverges in non-essential ways
-- **low:** Cosmetic drift
-`.trim();
-export function specReviewPrompt(ctx) {
-    const diffContent = ctx.diff && ctx.diff.trim() ? ctx.diff : '(no diff available)';
-    return `You are the spec reviewer for this task.
-
-Brief: ${ctx.brief}
-
-Worker said: ${ctx.workerSummary}
-
-Files changed: ${ctx.filesChanged.join(', ') || '(none)'}
-
-Diff (authoritative — what actually changed on disk):
-${diffContent}
-
-Guardrails:
-- do NOT claim files missing/untracked — the diff is authoritative
-- test status is in Worker said; don't infer skipped from diff
-
-${OUTPUT_FORMAT}`;
-}
-//# sourceMappingURL=spec-review-prompt.js.map

package/dist/lifecycle/handlers/spec-review-prompt.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"spec-review-prompt.js","sourceRoot":"","sources":["../../../src/lifecycle/handlers/spec-review-prompt.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,MAAM,aAAa,GAAG;;;;;;;;;;;;;;;;;;;;;;;;;;CA0B5B,CAAC,IAAI,EAAE,CAAC;AAET,MAAM,UAAU,gBAAgB,CAAC,GAAoF;IACnH,MAAM,WAAW,GAAG,GAAG,CAAC,IAAI,IAAI,GAAG,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,qBAAqB,CAAC;IACnF,OAAO;;SAEA,GAAG,CAAC,KAAK;;eAEH,GAAG,CAAC,aAAa;;iBAEf,GAAG,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,QAAQ;;;EAGtD,WAAW;;;;;;EAMX,aAAa,EAAE,CAAC;AAClB,CAAC"}

package/dist/lifecycle/handlers/terminal-handlers.d.ts

@@ -1,61 +0,0 @@
-import type { LifecycleState } from '../stage-plan-types.js';
-import type { StageGate, TerminalPayload } from '../stage-io.js';
-/**
- * Terminal-stage handlers (#45 Step 6).
- *
- * Four StagePlan rows:
- *   - 5.3.5 register_terminal_block — registers a context block carrying
- *     the terminal RuntimeRunResult so /retry can reference the prior task's
- *     output.
- *   - 5.4   emit_task_terminal     — emits the per-task terminal event
- *     (task_done_summary) through ctx.bus.
- *   - 5.5   persist_to_batch_registry — marks the per-task state in the
- *     batch registry.
- *   - 6.1   flush_telemetry — drains the telemetry queue so failure
- *     events are persisted before the per-task terminal stage exits.
- *
- * Each handler is idempotent on its state-slot guard:
- *   - state.contextBlockId
- *   - state.taskTerminalEmitted
- *   - state.batchRegistryPersisted
- *   - state.telemetryFlushed
- *
- * Defensive no-ops on missing state.executionContext or
- * state.lastRunResult. All four rows are marked runOnTerminal in the
- * StagePlan, so they fire even on hard-fail paths.
- */
-export declare function registerTerminalBlockHandler(state: LifecycleState): void;
-export declare function emitTaskTerminalHandler(state: LifecycleState): void;
-export declare function persistToBatchRegistryHandler(state: LifecycleState): void;
-/**
- * Row 5.6 — record_task_completed → envelope.seal.
- *
- * Seals the per-task envelope to finalize all accumulated state and
- * trigger telemetry upload via TelemetryUploader subscriber. Idempotent on
- * state.taskCompletedRecorded. No-op when the server hasn't supplied an
- * envelope (CLI/test paths).
- */
-export declare function recordTaskCompletedHandler(state: LifecycleState): Promise<void>;
-export declare function flushTelemetryHandler(state: LifecycleState): Promise<void>;
-/**
- * v5 unified `terminalHandler` — single entry point that runs the five
- * terminal side effects in idempotency-safe order and returns the
- * `StageGate<TerminalPayload>` the v5 driver expects.
- *
- * The five sub-handlers above are this function's building blocks: one per
- * side effect, kept individually exported so each can be unit-tested in
- * isolation (tests/lifecycle/handlers/terminal-handlers.test.ts) and so
- * observability AC tests can exercise per-side-effect failures
- * (tests/acceptance/stage-io-observability.test.ts). The unified
- * `terminalHandler` is what STAGE_PLAN's `terminal` stage actually invokes;
- * it sequences the five sub-handlers in idempotency-safe order and folds
- * their state-slot guards into the TerminalPayload shape. Per spec §4.7 the side-effect map carries one
- * boolean per side effect — true = succeeded at least once for this state
- * instance; false = attempted-and-failed or never-attempted-because-context-
- * missing.
- *
- * Idempotency: re-invocation MUST be safe and return the same payload (each
- * sub-handler short-circuits on its state-slot guard, e.g. `state.contextBlockId`).
- */
-export declare function terminalHandler(state: LifecycleState): Promise<StageGate<TerminalPayload>>;
-//# sourceMappingURL=terminal-handlers.d.ts.map

package/dist/lifecycle/handlers/terminal-handlers.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"terminal-handlers.d.ts","sourceRoot":"","sources":["../../../src/lifecycle/handlers/terminal-handlers.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAG7D,OAAO,KAAK,EAAE,SAAS,EAAE,eAAe,EAAE,MAAM,gBAAgB,CAAC;AAUjE;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AAEH,wBAAgB,4BAA4B,CAAC,KAAK,EAAE,cAAc,GAAG,IAAI,CAyBxE;AAED,wBAAgB,uBAAuB,CAAC,KAAK,EAAE,cAAc,GAAG,IAAI,CAUnE;AAMD,wBAAgB,6BAA6B,CAAC,KAAK,EAAE,cAAc,GAAG,IAAI,CAkBzE;AAED;;;;;;;GAOG;AACH,wBAAsB,0BAA0B,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CA2ErF;AAoED,wBAAsB,qBAAqB,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,IAAI,CAAC,CAkBhF;AAED;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAsB,eAAe,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,SAAS,CAAC,eAAe,CAAC,CAAC,CAwDhG"}

package/dist/lifecycle/handlers/terminal-handlers.js

@@ -1,327 +0,0 @@
-import { findModelProfile } from '../../config/model-profile-registry.js';
-import { getRealFilesChanged } from '../../bounded-execution/real-diff.js';
-import { deriveCompletion, extractCompletionInputs } from '../derive-completion.js';
-import { TerminalBlockRegistrar } from '../../reporting/terminal-block-registrar.js';
-import { renderTerminalReportMarkdown } from '../../reporting/terminal-report-markdown.js';
-import { WRITE_ROUTES } from '../stage-io.js';
-import { findEscapedWrites } from '../file-confinement-check.js';
-/**
- * Terminal-stage handlers (#45 Step 6).
- *
- * Four StagePlan rows:
- *   - 5.3.5 register_terminal_block — registers a context block carrying
- *     the terminal RuntimeRunResult so /retry can reference the prior task's
- *     output.
- *   - 5.4   emit_task_terminal     — emits the per-task terminal event
- *     (task_done_summary) through ctx.bus.
- *   - 5.5   persist_to_batch_registry — marks the per-task state in the
- *     batch registry.
- *   - 6.1   flush_telemetry — drains the telemetry queue so failure
- *     events are persisted before the per-task terminal stage exits.
- *
- * Each handler is idempotent on its state-slot guard:
- *   - state.contextBlockId
- *   - state.taskTerminalEmitted
- *   - state.batchRegistryPersisted
- *   - state.telemetryFlushed
- *
- * Defensive no-ops on missing state.executionContext or
- * state.lastRunResult. All four rows are marked runOnTerminal in the
- * StagePlan, so they fire even on hard-fail paths.
- */
-export function registerTerminalBlockHandler(state) {
-    if (state.contextBlockId)
-        return;
-    const ctx = state.executionContext;
-    const envelope = ctx?.envelope;
-    const store = ctx?.contextBlockStore;
-    const registry = ctx?.batchRegistry;
-    if (!envelope || !store || !registry)
-        return;
-    const snap = envelope.snapshot();
-    // Write routes (delegate / execute-plan / retry) produce code + a commit;
-    // their durable record is the diff, not a prose block. No registration.
-    if (WRITE_ROUTES.includes(snap.route))
-        return;
-    try {
-        const markdown = renderTerminalReportMarkdown(snap);
-        const registrar = new TerminalBlockRegistrar(store, registry);
-        const id = registrar.register({
-            batchId: snap.batchId, taskIndex: snap.taskIndex, route: snap.route, markdown,
-        });
-        if (id)
-            state.contextBlockId = id;
-    }
-    catch {
-        // Best-effort: registration is advisory and must not block seal/flush/persist.
-        envelope.recordValidationWarning({
-            rule: 'TerminalBlockRegisterFailed',
-            path: `${snap.route}:${snap.batchId}:${snap.taskIndex}`,
-        });
-    }
-}
-export function emitTaskTerminalHandler(state) {
-    if (state.taskTerminalEmitted)
-        return;
-    const ctx = state.executionContext;
-    if (!ctx)
-        return;
-    // The old `task_completed` named event was replaced by the sealed-envelope
-    // snapshot push inside envelope.seal() (see recordTaskCompletedHandler); the
-    // TelemetryUploader subscriber picks up that snapshot and runs toWireRecord.
-    // No bus.emit happens here — the envelope IS the event. This handler now only
-    // marks the idempotency flag so re-runs are no-ops.
-    state.taskTerminalEmitted = true;
-}
-export function persistToBatchRegistryHandler(state) {
-    if (state.batchRegistryPersisted)
-        return;
-    const ctx = state.executionContext;
-    if (!ctx)
-        return;
-    const registry = ctx.batchRegistry;
-    if (!registry || typeof registry.complete !== 'function') {
-        state.batchRegistryPersisted = true; // structural ack
-        return;
-    }
-    try {
-        registry.complete(ctx.taskIndex, state.lastRunResult);
-    }
-    catch {
-        // Persistence failure is non-fatal at the per-task boundary; retention
-        // sweep (row 6.3) reconciles via timer.
-    }
-    state.batchRegistryPersisted = true;
-}
-/**
- * Row 5.6 — record_task_completed → envelope.seal.
- *
- * Seals the per-task envelope to finalize all accumulated state and
- * trigger telemetry upload via TelemetryUploader subscriber. Idempotent on
- * state.taskCompletedRecorded. No-op when the server hasn't supplied an
- * envelope (CLI/test paths).
- */
-export async function recordTaskCompletedHandler(state) {
-    if (state.taskCompletedRecorded)
-        return;
-    const ctx = state.executionContext;
-    if (!ctx)
-        return;
-    const envelope = ctx.envelope;
-    if (!envelope) {
-        state.taskCompletedRecorded = true;
-        return;
-    }
-    const real = await getRealFilesChanged(state);
-    const last = state.lastRunResult;
-    if (real.source === 'git_error') {
-        envelope.recordValidationWarning({ rule: 'GitDiffUnavailable', path: 'realFilesChanged' });
-    }
-    const completionInputs = extractCompletionInputs(state);
-    // Use deriveCompletion when implement gate is populated. commitKind being
-    // undefined is legitimate (read routes have no commit stage) — deriveCompletion
-    // handles those branches internally. Only fall back to workerStatus when the
-    // implement gate itself is missing (lifecycle never started, brief_too_vague paths).
-    const hasGateInputs = completionInputs.implementOutcome !== undefined;
-    let sealStatus;
-    if (hasGateInputs) {
-        const { completed } = deriveCompletion(completionInputs);
-        const hasConcerns = (state.executionContext?.envelope?.snapshot()?.findings ?? []).length > 0;
-        sealStatus = completed ? (hasConcerns ? 'done_with_concerns' : 'done') : 'failed';
-    }
-    else {
-        // Fallback only when implement gate is missing entirely (brief_too_vague etc.)
-        const ws = (state.workerStatus ?? last?.workerStatus);
-        sealStatus = ws === 'done' ? 'done' : ws === 'done_with_concerns' ? 'done_with_concerns' : 'failed';
-    }
-    // Commit outcome (authoritative, from the commit gate) — computed BEFORE the
-    // confinement guard so it can corroborate the worker's self-reported writes.
-    // Carried onto the envelope below so the response's structuredReport surfaces
-    // the real SHA/message (the response is built from envelope snapshots).
-    const commitGate = state.gates?.['commit'];
-    const commitPayload = (commitGate?.outcome === 'advance' ? commitGate.payload : null);
-    const didCommit = commitPayload?.kind === 'committed';
-    // Confinement guard: a worker must only write under its dispatched cwd. A
-    // reported write that escaped (e.g. into a sibling git worktree / the daemon's
-    // startup cwd) is a real mislocation. BUT the worker-reported filesWritten
-    // string is unreliable — LLM workers routinely report normalized/relative/
-    // hallucinated absolute paths (e.g. "/repo/src/x.ts" or "/workspace/src/x.ts")
-    // for a file actually written under the dispatched cwd. So hard-fail ONLY when
-    // the worker claims an escaped write AND no commit landed in cwd (didCommit =
-    // false → the write is not in this repo → it genuinely wrote elsewhere). When a
-    // commit DID land in cwd, the self-reported path is a hallucination — record an
-    // advisory warning rather than failing a legitimate, committed task on an
-    // unreliable self-report string. (The git commit is the authoritative signal.)
-    let escapeErrorCode = null;
-    const escaped = findEscapedWrites(last?.filesWritten ?? [], ctx.cwd);
-    if (escaped.length > 0) {
-        envelope.recordValidationWarning({ rule: 'WorkerWriteEscapedCwd', path: escaped.join(', ') });
-        if (!didCommit) {
-            sealStatus = 'failed';
-            escapeErrorCode = 'tool_sandbox_cwd_violation';
-        }
-    }
-    envelope.seal({
-        status: sealStatus,
-        terminalAt: new Date().toISOString(),
-        stopReason: last?.terminationReason?.cause ?? null,
-        structuredError: last?.structuredError ?? null,
-        errorCode: escapeErrorCode ?? last?.errorCode ?? null,
-        realFilesChanged: real.files,
-        commitSha: didCommit ? (commitPayload?.commitSha ?? null) : null,
-        commitMessage: didCommit ? (commitPayload?.commitMessage ?? null) : null,
-        commitSkipReason: commitPayload?.kind === 'no_op' ? (commitPayload?.reason ?? null) : null,
-        contextBlockId: state.contextBlockId ?? null,
-    });
-    state.taskCompletedRecorded = true;
-}
-/**
- * Synthesize an `implementing` stage entry from top-level RuntimeRunResult fields
- * when the per-stage tracker hasn't populated it. Without this, the wire
- * event ships with `stages: []` for every task, which violates the backend's
- * R2.1 invariant ("empty stages only allowed for brief_too_vague|error")
- * for any task that succeeded — every upload would 400.
- *
- * This is a fallback; it does not replace stats already populated by the
- * runner-shell or lifecycle stage tracker.
- */
-function ensureImplementingStage(rr, ctx) {
-    // Even when no LLM call ever fires (runner_crash, all_tiers_unavailable,
-    // dispatcher-no-result), the configured implementer model is known up
-    // front via ctx.implementerProvider.config. Stamp it into the synthesized
-    // stage and the top-level rr.models so the wire row reports the *intended*
-    // model instead of the literal 'custom' fallback in event-builder.
-    const fallbackModel = ctx.implementerProvider?.config?.model ?? null;
-    const fallbackFamily = fallbackModel ? findModelProfile(fallbackModel).family : null;
-    if (rr.models === undefined && fallbackModel !== null) {
-        rr.models = {
-            implementer: fallbackModel,
-            specReviewer: undefined,
-            qualityReviewer: undefined,
-        };
-    }
-    const existing = (rr.stageStats?.implementing);
-    if (existing?.entered)
-        return;
-    const usage = rr.usage ?? { inputTokens: 0, outputTokens: 0, cachedReadTokens: 0, cachedNonReadTokens: 0 };
-    const synthesized = {
-        stage: 'implementing',
-        entered: true,
-        durationMs: rr.durationMs ?? 0,
-        costUSD: rr.cost?.costUSD ?? null,
-        agentTier: ctx.assignedTier ?? 'standard',
-        modelFamily: fallbackFamily,
-        model: fallbackModel,
-        maxIdleMs: 0,
-        totalIdleMs: 0,
-        activityEvents: 0,
-        inputTokens: usage.inputTokens ?? 0,
-        outputTokens: usage.outputTokens ?? 0,
-        cachedReadTokens: usage.cachedReadTokens ?? 0,
-        cachedNonReadTokens: usage.cachedNonReadTokens ?? 0,
-        turnCount: rr.turns ?? 0,
-        filesWrittenCount: Array.isArray(rr.filesWritten) ? rr.filesWritten.length : 0,
-        directoriesListed: 0,
-    };
-    rr.stageStats = {
-        ...(rr.stageStats ?? {}),
-        implementing: synthesized,
-    };
-}
-export async function flushTelemetryHandler(state) {
-    if (state.telemetryFlushed)
-        return;
-    const ctx = state.executionContext;
-    if (!ctx)
-        return;
-    const recorder = ctx.recorder;
-    if (!recorder || typeof recorder.flush !== 'function') {
-        state.telemetryFlushed = true;
-        return;
-    }
-    try {
-        await recorder.flush();
-    }
-    catch {
-        // Telemetry flush is best-effort; the recorder retains in-memory queue
-        // for the next opportunity.
-    }
-    state.telemetryFlushed = true;
-}
-/**
- * v5 unified `terminalHandler` — single entry point that runs the five
- * terminal side effects in idempotency-safe order and returns the
- * `StageGate<TerminalPayload>` the v5 driver expects.
- *
- * The five sub-handlers above are this function's building blocks: one per
- * side effect, kept individually exported so each can be unit-tested in
- * isolation (tests/lifecycle/handlers/terminal-handlers.test.ts) and so
- * observability AC tests can exercise per-side-effect failures
- * (tests/acceptance/stage-io-observability.test.ts). The unified
- * `terminalHandler` is what STAGE_PLAN's `terminal` stage actually invokes;
- * it sequences the five sub-handlers in idempotency-safe order and folds
- * their state-slot guards into the TerminalPayload shape. Per spec §4.7 the side-effect map carries one
- * boolean per side effect — true = succeeded at least once for this state
- * instance; false = attempted-and-failed or never-attempted-because-context-
- * missing.
- *
- * Idempotency: re-invocation MUST be safe and return the same payload (each
- * sub-handler short-circuits on its state-slot guard, e.g. `state.contextBlockId`).
- */
-export async function terminalHandler(state) {
-    const t0 = Date.now();
-    const flags = {
-        contextBlockId: null,
-        telemetryFlushed: false,
-        batchRegistryPersisted: false,
-        taskTerminalEmitted: false,
-        projectCleanupTicked: false,
-    };
-    try {
-        registerTerminalBlockHandler(state);
-        flags.contextBlockId = state.contextBlockId ?? null;
-    }
-    catch {
-        /* leave null on failure */
-    }
-    try {
-        await flushTelemetryHandler(state);
-        flags.telemetryFlushed = state.telemetryFlushed === true;
-    }
-    catch {
-        /* leave false */
-    }
-    try {
-        persistToBatchRegistryHandler(state);
-        flags.batchRegistryPersisted = state.batchRegistryPersisted === true;
-    }
-    catch {
-        /* leave false */
-    }
-    try {
-        emitTaskTerminalHandler(state);
-        flags.taskTerminalEmitted = state.taskTerminalEmitted === true;
-    }
-    catch {
-        /* leave false */
-    }
-    try {
-        await recordTaskCompletedHandler(state);
-        flags.projectCleanupTicked = true; // record-completed doubles as project activity tick
-    }
-    catch {
-        /* leave false */
-    }
-    return {
-        outcome: 'advance',
-        payload: flags,
-        telemetry: {
-            stageLabel: 'terminal',
-            durationMs: Date.now() - t0,
-            costUSD: 0,
-            turnsUsed: 0,
-            stopReason: 'normal',
-        },
-    };
-}
-//# sourceMappingURL=terminal-handlers.js.map