chorus-codes 0.7.0 → 0.7.2

package/dist/daemon/runner/doer.js

@@ -0,0 +1,336 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.runDoerHeadless = runDoerHeadless;
+/**
+ * Doer streaming execution.
+ *
+ * Drives one doer subprocess via its AgentShim's runHeadless iterator,
+ * forwarding text_delta events to a buffered StreamFileWriter and to the
+ * SSE bus, and producing the final `{content, full}` shape the phase
+ * loop consumes.
+ *
+ * Critical invariants (covered by tests/runner-doer.test.ts):
+ *   - text_delta events accumulate AND get flushed to answer.md
+ *   - message_done with non-empty finalText overwrites authoritatively +
+ *     appends ## DONE sentinel when missing
+ *   - message_done with EMPTY finalText keeps accumulated deltas
+ *     (the Gemini "result-line-only" failure mode)
+ *   - error event flips errored AND preserves accumulated content
+ *   - StreamFileWriter buffer is flushed in the finally block on every path
+ */
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const template_schema_js_1 = require("../../lib/template-schema.js");
+const permissions_js_1 = require("../../lib/settings/permissions.js");
+const cli_health_js_1 = require("../../lib/cli-health.js");
+const stream_file_writer_js_1 = require("./stream-file-writer.js");
+async function runDoerHeadless(args) {
+    const { shim, chatId, phase, round, agentName, askContent, answerFile, doerCwd, abortSignal, onEvent, modelOverride, } = args;
+    if (!shim.runHeadless) {
+        // Defensive — caller should have checked. Fail closed.
+        return null;
+    }
+    const perms = await (0, permissions_js_1.getPermissions)();
+    let accumulated = '';
+    let finalText;
+    let errored = false;
+    // Captured from the first error event so we can write it to
+    // answer.md when the subprocess dies before producing any content.
+    // Mirrors the reviewer-side handling so chat dirs are self-
+    // explanatory after a silent-failure crash.
+    let errorSummary;
+    let capturedUsage;
+    const startedAt = Date.now();
+    // Initialize answer.md so the artifacts endpoint sees the file mid-stream.
+    fs.writeFileSync(answerFile, '');
+    const writer = new stream_file_writer_js_1.StreamFileWriter(answerFile);
+    const stream = shim.runHeadless({
+        cwd: doerCwd,
+        promptText: askContent,
+        model: modelOverride ?? phase.doer.models?.[0],
+        sandbox: perms.sandboxProfile,
+        autoApprove: perms.autoApprovePrompts,
+        networkAccess: perms.networkAccess,
+        abortSignal,
+        timeoutMs: phase.timeoutMs ?? template_schema_js_1.DEFAULT_PHASE_TIMEOUT_MS,
+    });
+    try {
+        for await (const event of stream) {
+            if (event.type === 'text_delta') {
+                accumulated += event.text;
+                writer.write(event.text);
+                onEvent({
+                    chatId,
+                    type: 'phase_progress',
+                    payload: {
+                        phaseId: phase.id,
+                        round,
+                        role: 'doer',
+                        agent: agentName,
+                        output: accumulated.slice(-500),
+                    },
+                    ts: Date.now(),
+                });
+            }
+            else if (event.type === 'tool_call_start') {
+                onEvent({
+                    chatId,
+                    type: 'phase_progress',
+                    payload: {
+                        phaseId: phase.id,
+                        round,
+                        role: 'doer',
+                        agent: agentName,
+                        tool: event.tool,
+                    },
+                    ts: Date.now(),
+                });
+            }
+            else if (event.type === 'progress') {
+                onEvent({
+                    chatId,
+                    type: 'phase_progress',
+                    payload: {
+                        phaseId: phase.id,
+                        round,
+                        role: 'doer',
+                        agent: agentName,
+                        elapsedMs: event.elapsedMs,
+                    },
+                    ts: Date.now(),
+                });
+            }
+            else if (event.type === 'message_done') {
+                finalText = event.finalText;
+                if (event.usage)
+                    capturedUsage = event.usage;
+                writer.flushNow();
+                if (event.finalText.trim().length === 0) {
+                    const existing = fs.existsSync(answerFile)
+                        ? fs.readFileSync(answerFile, 'utf-8')
+                        : '';
+                    if (!/\n##\s*DONE\s*\n?$/i.test(existing.trimEnd())) {
+                        fs.appendFileSync(answerFile, existing.endsWith('\n') ? '\n## DONE\n' : '\n\n## DONE\n');
+                    }
+                }
+                else {
+                    const needsSentinel = !/\n##\s*DONE\s*\n?$/i.test(event.finalText.trimEnd());
+                    const finalContent = needsSentinel
+                        ? `${event.finalText}\n\n## DONE\n`
+                        : event.finalText.endsWith('\n')
+                            ? event.finalText
+                            : `${event.finalText}\n`;
+                    fs.writeFileSync(answerFile, finalContent);
+                }
+                // Tell the cockpit this participant is fully on disk so it can
+                // flip the card immediately rather than wait for the 8s polling
+                // tick. Mirrored in reviewer.ts.
+                // Persist runtime stats next to answer.md — see reviewer.ts for
+                // rationale. Use path.dirname + path.join (clearer than the
+                // anchored regex; flagged in retroactive PR #16 review).
+                try {
+                    const statsPath = path.join(path.dirname(answerFile), '_stats.json');
+                    fs.writeFileSync(statsPath, JSON.stringify({
+                        durationMs: Date.now() - startedAt,
+                        ...(capturedUsage ? { usage: capturedUsage } : {}),
+                    }), 'utf-8');
+                }
+                catch {
+                    /* sidecar is informational; ignore write errors */
+                }
+                // participant_done payload carries identity only — the cockpit
+                // refetches /api/run-artifacts on this event to pick up the
+                // sidecar-backed stats. Earlier diff also embedded durationMs
+                // and usage in the SSE payload; retroactive PR #16 review by
+                // both opencode reviewers flagged those as dead bytes never
+                // consumed on the cockpit side.
+                onEvent({
+                    chatId,
+                    type: 'participant_done',
+                    payload: {
+                        phaseId: phase.id,
+                        round,
+                        role: 'doer',
+                        agent: agentName,
+                    },
+                    ts: Date.now(),
+                });
+            }
+            else if (event.type === 'error') {
+                errored = true;
+                // Mirror reviewer.ts: persist OpenRouter HTTP-error state into
+                // cli-health so the home-page card flips to quota/auth/rate-limit
+                // and so the run-page error event carries a remediation CTA.
+                const classified = (0, cli_health_js_1.classifyOpenRouterError)(event.kind, event.message);
+                if (classified) {
+                    (0, cli_health_js_1.recordHealth)({
+                        lineage: 'openrouter',
+                        status: classified.status,
+                        message: classified.message,
+                    }).catch((healthErr) => {
+                        console.error('[chorus] recordHealth failed for openrouter:', healthErr);
+                    });
+                }
+                if (!errorSummary) {
+                    errorSummary = {
+                        kind: event.kind,
+                        message: classified?.message ?? event.message,
+                    };
+                }
+                onEvent({
+                    chatId,
+                    type: 'cli_error',
+                    payload: {
+                        phaseId: phase.id,
+                        phaseKind: phase.kind,
+                        phaseIdx: 0,
+                        round,
+                        role: 'doer',
+                        agent: agentName,
+                        error: {
+                            kind: event.kind,
+                            message: classified?.message ?? event.message,
+                            ...(classified?.cta ? { cta: classified.cta } : {}),
+                            lineage: phase.doer.lineage,
+                        },
+                    },
+                    ts: Date.now(),
+                });
+            }
+        }
+    }
+    catch (err) {
+        errored = true;
+        const message = err instanceof Error ? err.message : String(err);
+        if (!errorSummary) {
+            errorSummary = { kind: 'stream_failure', message };
+        }
+        onEvent({
+            chatId,
+            type: 'cli_error',
+            payload: {
+                phaseId: phase.id,
+                phaseKind: phase.kind,
+                phaseIdx: 0,
+                round,
+                role: 'doer',
+                agent: agentName,
+                error: {
+                    kind: 'stream_failure',
+                    message,
+                    lineage: phase.doer.lineage,
+                },
+            },
+            ts: Date.now(),
+        });
+    }
+    finally {
+        writer.flushNow();
+        // When the subprocess died without producing any content, write the
+        // error summary to answer.md so the chat dir is self-explanatory.
+        // Same pattern as runReviewerHeadless — see comment there.
+        if (errored && accumulated.length === 0 && (!finalText || finalText.length === 0) && errorSummary) {
+            try {
+                // Mirror reviewer.ts: include cli-health resetAt for quota /
+                // rate-limit failures so the cockpit can render a countdown.
+                let resetAt;
+                try {
+                    const h = await (0, cli_health_js_1.getHealth)(phase.doer.lineage);
+                    if (typeof h.resetAt === 'number' && h.resetAt > Date.now()) {
+                        resetAt = h.resetAt;
+                    }
+                }
+                catch {
+                    /* health lookup is informational */
+                }
+                fs.writeFileSync(answerFile, `## DOER FAILED\n\n` +
+                    `**Kind:** ${errorSummary.kind}\n` +
+                    `**Lineage:** ${phase.doer.lineage}\n` +
+                    `**Model:** ${modelOverride ?? phase.doer.models?.[0] ?? '(default)'}\n` +
+                    (resetAt ? `**Resets:** ${new Date(resetAt).toISOString()}\n` : '') +
+                    `\n${errorSummary.message}\n`);
+            }
+            catch {
+                /* best-effort — don't fail the runner because of a write error */
+            }
+        }
+        // If the StreamFileWriter went dead (FS ENOSPC, EACCES, etc.) it
+        // dropped the failing chunk to avoid retrying the same sync write
+        // forever. Surface this so the user sees "stream stopped writing"
+        // rather than silently truncated answer.md.
+        if (writer.isDead()) {
+            const err = writer.lastError();
+            onEvent({
+                chatId,
+                type: 'cli_warning',
+                payload: {
+                    phaseId: phase.id,
+                    round,
+                    role: 'doer',
+                    agent: agentName,
+                    reason: 'stream_writer_dead',
+                    message: `answer.md write failed; subsequent deltas dropped: ${err ? err.message : 'unknown'}`,
+                    cta: 'Check disk space + permissions on ~/.chorus/chats. Re-run when fixed.',
+                },
+                ts: Date.now(),
+            });
+        }
+    }
+    if (errored && finalText === undefined && accumulated.length === 0) {
+        return null;
+    }
+    const content = finalText && finalText.length > 0 ? finalText : accumulated;
+    if (finalText === undefined && content.trim().length === 0) {
+        return null;
+    }
+    // `full` means "the doer's answer is complete" — a truthful signal to
+    // downstream reviewers that they can review the artifact as-is. An
+    // errored stream that wrote partial bytes before the subprocess crashed
+    // must NOT be marked full: reviewers receiving truncated mid-paragraph
+    // text will silently produce nonsense verdicts on a half-written answer.
+    // Only treat as full when EITHER the parser saw a final message_done
+    // (finalText set) AND no error, OR we accumulated text without any
+    // error mid-stream. The launch-eve gemini review of runner orchestration
+    // flagged this — earlier code returned full=true whenever
+    // `accumulated.length > 0`, regardless of `errored` state.
+    const isFull = !errored && (finalText !== undefined || accumulated.length > 0);
+    return {
+        content,
+        full: isFull,
+        ...(capturedUsage ? { usage: capturedUsage } : {}),
+    };
+}
+//# sourceMappingURL=doer.js.map

package/dist/daemon/runner/doer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"doer.js","sourceRoot":"","sources":["../../../src/daemon/runner/doer.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAgCA,0CA8UC;AA9WD;;;;;;;;;;;;;;;;GAgBG;AACH,uCAAyB;AACzB,2CAA6B;AAE7B,qEAAwE;AAExE,sEAAmE;AACnE,2DAKiC;AACjC,mEAA2D;AAGpD,KAAK,UAAU,eAAe,CAAC,IAgBrC;IAgBC,MAAM,EACJ,IAAI,EACJ,MAAM,EACN,KAAK,EACL,KAAK,EACL,SAAS,EACT,UAAU,EACV,UAAU,EACV,OAAO,EACP,WAAW,EACX,OAAO,EACP,aAAa,GACd,GAAG,IAAI,CAAC;IAET,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC;QACtB,uDAAuD;QACvD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,KAAK,GAAG,MAAM,IAAA,+BAAc,GAAE,CAAC;IACrC,IAAI,WAAW,GAAG,EAAE,CAAC;IACrB,IAAI,SAA6B,CAAC;IAClC,IAAI,OAAO,GAAG,KAAK,CAAC;IACpB,4DAA4D;IAC5D,mEAAmE;IACnE,4DAA4D;IAC5D,4CAA4C;IAC5C,IAAI,YAA2D,CAAC;IAChE,IAAI,aAKS,CAAC;IACd,MAAM,SAAS,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IAE7B,2EAA2E;IAC3E,EAAE,CAAC,aAAa,CAAC,UAAU,EAAE,EAAE,CAAC,CAAC;IACjC,MAAM,MAAM,GAAG,IAAI,wCAAgB,CAAC,UAAU,CAAC,CAAC;IAEhD,MAAM,MAAM,GAAG,IAAI,CAAC,WAAW,CAAC;QAC9B,GAAG,EAAE,OAAO;QACZ,UAAU,EAAE,UAAU;QACtB,KAAK,EAAE,aAAa,IAAI,KAAK,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC;QAC9C,OAAO,EAAE,KAAK,CAAC,cAAc;QAC7B,WAAW,EAAE,KAAK,CAAC,kBAAkB;QACrC,aAAa,EAAE,KAAK,CAAC,aAAa;QAClC,WAAW;QACX,SAAS,EAAE,KAAK,CAAC,SAAS,IAAI,6CAAwB;KACvD,CAAC,CAAC;IAEH,IAAI,CAAC;QACH,IAAI,KAAK,EAAE,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;YACjC,IAAI,KAAK,CAAC,IAAI,KAAK,YAAY,EAAE,CAAC;gBAChC,WAAW,IAAI,KAAK,CAAC,IAAI,CAAC;gBAC1B,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;gBACzB,OAAO,CAAC;oBACN,MAAM;oBACN,IAAI,EAAE,gBAAgB;oBACtB,OAAO,EAAE;wBACP,OAAO,EAAE,KAAK,CAAC,EAAE;wBACjB,KAAK;wBACL,IAAI,EAAE,MAAM;wBACZ,KAAK,EAAE,SAAS;wBAChB,MAAM,EAAE,WAAW,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC;qBAChC;oBACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;iBACf,CAAC,CAAC;YACL,CAAC;iBAAM,IAAI,KAAK,CAAC,IAAI,KAAK,iBAAiB,EAAE,CAAC;gBAC5C,OAAO,CAAC;oBACN,MAAM;oBACN,IAAI,EAAE,gBAAgB;oBACtB,OAAO,EAAE;wBACP,OAAO,EAAE,KAAK,CAAC,EAAE;wBACjB,KAAK;wBACL,IAAI,EAAE,MAAM;wBACZ,KAAK,EAAE,SAAS;wBAChB,IAAI,EAAE,KAAK,CAAC,IAAI;qBACjB;oBACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;iBACf,CAAC,CAAC;YACL,CAAC;iBAAM,IAAI,KAAK,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;gBACrC,OAAO,CAAC;oBACN,MAAM;oBACN,IAAI,EAAE,gBAAgB;oBACtB,OAAO,EAAE;wBACP,OAAO,EAAE,KAAK,CAAC,EAAE;wBACjB,KAAK;wBACL,IAAI,EAAE,MAAM;wBACZ,KAAK,EAAE,SAAS;wBAChB,SAAS,EAAE,KAAK,CAAC,SAAS;qBAC3B;oBACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;iBACf,CAAC,CAAC;YACL,CAAC;iBAAM,IAAI,KAAK,CAAC,IAAI,KAAK,cAAc,EAAE,CAAC;gBACzC,SAAS,GAAG,KAAK,CAAC,SAAS,CAAC;gBAC5B,IAAI,KAAK,CAAC,KAAK;oBAAE,aAAa,GAAG,KAAK,CAAC,KAAK,CAAC;gBAC7C,MAAM,CAAC,QAAQ,EAAE,CAAC;gBAClB,IAAI,KAAK,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBACxC,MAAM,QAAQ,GAAG,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC;wBACxC,CAAC,CAAC,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC;wBACtC,CAAC,CAAC,EAAE,CAAC;oBACP,IAAI,CAAC,qBAAqB,CAAC,IAAI,CAAC,QAAQ,CAAC,OAAO,EAAE,CAAC,EAAE,CAAC;wBACpD,EAAE,CAAC,cAAc,CACf,UAAU,EACV,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,eAAe,CAC1D,CAAC;oBACJ,CAAC;gBACH,CAAC;qBAAM,CAAC;oBACN,MAAM,aAAa,GAAG,CAAC,qBAAqB,CAAC,IAAI,CAC/C,KAAK,CAAC,SAAS,CAAC,OAAO,EAAE,CAC1B,CAAC;oBACF,MAAM,YAAY,GAAG,aAAa;wBAChC,CAAC,CAAC,GAAG,KAAK,CAAC,SAAS,eAAe;wBACnC,CAAC,CAAC,KAAK,CAAC,SAAS,CAAC,QAAQ,CAAC,IAAI,CAAC;4BAC9B,CAAC,CAAC,KAAK,CAAC,SAAS;4BACjB,CAAC,CAAC,GAAG,KAAK,CAAC,SAAS,IAAI,CAAC;oBAC7B,EAAE,CAAC,aAAa,CAAC,UAAU,EAAE,YAAY,CAAC,CAAC;gBAC7C,CAAC;gBACD,+DAA+D;gBAC/D,gEAAgE;gBAChE,iCAAiC;gBACjC,gEAAgE;gBAChE,4DAA4D;gBAC5D,yDAAyD;gBACzD,IAAI,CAAC;oBACH,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,aAAa,CAAC,CAAC;oBACrE,EAAE,CAAC,aAAa,CACd,SAAS,EACT,IAAI,CAAC,SAAS,CAAC;wBACb,UAAU,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,SAAS;wBAClC,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,aAAa,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;qBACnD,CAAC,EACF,OAAO,CACR,CAAC;gBACJ,CAAC;gBAAC,MAAM,CAAC;oBACP,mDAAmD;gBACrD,CAAC;gBACD,+DAA+D;gBAC/D,4DAA4D;gBAC5D,8DAA8D;gBAC9D,6DAA6D;gBAC7D,4DAA4D;gBAC5D,gCAAgC;gBAChC,OAAO,CAAC;oBACN,MAAM;oBACN,IAAI,EAAE,kBAAkB;oBACxB,OAAO,EAAE;wBACP,OAAO,EAAE,KAAK,CAAC,EAAE;wBACjB,KAAK;wBACL,IAAI,EAAE,MAAM;wBACZ,KAAK,EAAE,SAAS;qBACjB;oBACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;iBACf,CAAC,CAAC;YACL,CAAC;iBAAM,IAAI,KAAK,CAAC,IAAI,KAAK,OAAO,EAAE,CAAC;gBAClC,OAAO,GAAG,IAAI,CAAC;gBACf,+DAA+D;gBAC/D,kEAAkE;gBAClE,6DAA6D;gBAC7D,MAAM,UAAU,GAAG,IAAA,uCAAuB,EAAC,KAAK,CAAC,IAAI,EAAE,KAAK,CAAC,OAAO,CAAC,CAAC;gBACtE,IAAI,UAAU,EAAE,CAAC;oBACf,IAAA,4BAAY,EAAC;wBACX,OAAO,EAAE,YAAY;wBACrB,MAAM,EAAE,UAAU,CAAC,MAAM;wBACzB,OAAO,EAAE,UAAU,CAAC,OAAO;qBAC5B,CAAC,CAAC,KAAK,CAAC,CAAC,SAAkB,EAAE,EAAE;wBAC9B,OAAO,CAAC,KAAK,CAAC,8CAA8C,EAAE,SAAS,CAAC,CAAC;oBAC3E,CAAC,CAAC,CAAC;gBACL,CAAC;gBACD,IAAI,CAAC,YAAY,EAAE,CAAC;oBAClB,YAAY,GAAG;wBACb,IAAI,EAAE,KAAK,CAAC,IAAI;wBAChB,OAAO,EAAE,UAAU,EAAE,OAAO,IAAI,KAAK,CAAC,OAAO;qBAC9C,CAAC;gBACJ,CAAC;gBACD,OAAO,CAAC;oBACN,MAAM;oBACN,IAAI,EAAE,WAAW;oBACjB,OAAO,EAAE;wBACP,OAAO,EAAE,KAAK,CAAC,EAAE;wBACjB,SAAS,EAAE,KAAK,CAAC,IAAI;wBACrB,QAAQ,EAAE,CAAC;wBACX,KAAK;wBACL,IAAI,EAAE,MAAM;wBACZ,KAAK,EAAE,SAAS;wBAChB,KAAK,EAAE;4BACL,IAAI,EAAE,KAAK,CAAC,IAAI;4BAChB,OAAO,EAAE,UAAU,EAAE,OAAO,IAAI,KAAK,CAAC,OAAO;4BAC7C,GAAG,CAAC,UAAU,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,UAAU,CAAC,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;4BACnD,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,OAAO;yBAC5B;qBACF;oBACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;iBACf,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,OAAO,GAAG,IAAI,CAAC;QACf,MAAM,OAAO,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;QACjE,IAAI,CAAC,YAAY,EAAE,CAAC;YAClB,YAAY,GAAG,EAAE,IAAI,EAAE,gBAAgB,EAAE,OAAO,EAAE,CAAC;QACrD,CAAC;QACD,OAAO,CAAC;YACN,MAAM;YACN,IAAI,EAAE,WAAW;YACjB,OAAO,EAAE;gBACP,OAAO,EAAE,KAAK,CAAC,EAAE;gBACjB,SAAS,EAAE,KAAK,CAAC,IAAI;gBACrB,QAAQ,EAAE,CAAC;gBACX,KAAK;gBACL,IAAI,EAAE,MAAM;gBACZ,KAAK,EAAE,SAAS;gBAChB,KAAK,EAAE;oBACL,IAAI,EAAE,gBAAgB;oBACtB,OAAO;oBACP,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,OAAO;iBAC5B;aACF;YACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;SACf,CAAC,CAAC;IACL,CAAC;YAAS,CAAC;QACT,MAAM,CAAC,QAAQ,EAAE,CAAC;QAClB,oEAAoE;QACpE,kEAAkE;QAClE,2DAA2D;QAC3D,IAAI,OAAO,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,CAAC,SAAS,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,CAAC,IAAI,YAAY,EAAE,CAAC;YAClG,IAAI,CAAC;gBACH,6DAA6D;gBAC7D,6DAA6D;gBAC7D,IAAI,OAA2B,CAAC;gBAChC,IAAI,CAAC;oBACH,MAAM,CAAC,GAAG,MAAM,IAAA,yBAAS,EAAC,KAAK,CAAC,IAAI,CAAC,OAAqB,CAAC,CAAC;oBAC5D,IAAI,OAAO,CAAC,CAAC,OAAO,KAAK,QAAQ,IAAI,CAAC,CAAC,OAAO,GAAG,IAAI,CAAC,GAAG,EAAE,EAAE,CAAC;wBAC5D,OAAO,GAAG,CAAC,CAAC,OAAO,CAAC;oBACtB,CAAC;gBACH,CAAC;gBAAC,MAAM,CAAC;oBACP,oCAAoC;gBACtC,CAAC;gBACD,EAAE,CAAC,aAAa,CACd,UAAU,EACV,oBAAoB;oBAClB,aAAa,YAAY,CAAC,IAAI,IAAI;oBAClC,gBAAgB,KAAK,CAAC,IAAI,CAAC,OAAO,IAAI;oBACtC,cAAc,aAAa,IAAI,KAAK,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC,IAAI,WAAW,IAAI;oBACxE,CAAC,OAAO,CAAC,CAAC,CAAC,eAAe,IAAI,IAAI,CAAC,OAAO,CAAC,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC;oBACnE,KAAK,YAAY,CAAC,OAAO,IAAI,CAChC,CAAC;YACJ,CAAC;YAAC,MAAM,CAAC;gBACP,kEAAkE;YACpE,CAAC;QACH,CAAC;QACD,iEAAiE;QACjE,kEAAkE;QAClE,kEAAkE;QAClE,4CAA4C;QAC5C,IAAI,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC;YACpB,MAAM,GAAG,GAAG,MAAM,CAAC,SAAS,EAAE,CAAC;YAC/B,OAAO,CAAC;gBACN,MAAM;gBACN,IAAI,EAAE,aAAa;gBACnB,OAAO,EAAE;oBACP,OAAO,EAAE,KAAK,CAAC,EAAE;oBACjB,KAAK;oBACL,IAAI,EAAE,MAAM;oBACZ,KAAK,EAAE,SAAS;oBAChB,MAAM,EAAE,oBAAoB;oBAC5B,OAAO,EAAE,sDAAsD,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS,EAAE;oBAC9F,GAAG,EAAE,uEAAuE;iBAC7E;gBACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;aACf,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,IAAI,OAAO,IAAI,SAAS,KAAK,SAAS,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACnE,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,OAAO,GAAG,SAAS,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC;IAE5E,IAAI,SAAS,KAAK,SAAS,IAAI,OAAO,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC3D,OAAO,IAAI,CAAC;IACd,CAAC;IAED,sEAAsE;IACtE,mEAAmE;IACnE,wEAAwE;IACxE,uEAAuE;IACvE,yEAAyE;IACzE,qEAAqE;IACrE,mEAAmE;IACnE,yEAAyE;IACzE,0DAA0D;IAC1D,2DAA2D;IAC3D,MAAM,MAAM,GAAG,CAAC,OAAO,IAAI,CAAC,SAAS,KAAK,SAAS,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IAE/E,OAAO;QACL,OAAO;QACP,IAAI,EAAE,MAAM;QACZ,GAAG,CAAC,aAAa,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,aAAa,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KACnD,CAAC;AACJ,CAAC"}

package/dist/daemon/runner/prior-round.js

@@ -0,0 +1,140 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.readPriorRoundFeedback = readPriorRoundFeedback;
+/**
+ * Prior-round feedback reader.
+ *
+ * On round ≥ 2 the doer was rerunning the SAME prompt as round 1 — only
+ * the round number in the heading changed. Reviewer findings from round
+ * N-1 were never threaded back, so disagreement → retry was effectively
+ * "ask again, hope the LLM rolls a better die" instead of a real revision
+ * loop.
+ *
+ * This helper assembles a "## Prior round feedback" markdown block from
+ * the previous round's reviewer answer.md files. The doer prompt builder
+ * inlines it before the "## How to respond" footer.
+ *
+ * Caps:
+ *   - per-reviewer slice: 16KB (truncated answers still mark the cut)
+ *   - total block:        64KB (additional reviewers replaced with marker)
+ *
+ * Numbers chosen to leave the 256KB doer prompt budget mostly intact for
+ * attached files + persona + task framing, while still surfacing enough
+ * reviewer text to be actionable.
+ */
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const PER_REVIEWER_MAX_BYTES = 16 * 1024;
+const TOTAL_FEEDBACK_MAX_BYTES = 64 * 1024;
+/**
+ * Returns a markdown block summarizing the previous round's reviewer
+ * findings, or empty string when there's nothing to feed back.
+ *
+ * Empty cases:
+ *   - round <= 1 (no prior round exists)
+ *   - prior round dir missing (e.g. crash mid-round)
+ *   - no reviewer-* dirs found
+ *   - all reviewer answer.md files unreadable / empty
+ */
+function readPriorRoundFeedback(chatDir, round) {
+    if (round <= 1)
+        return '';
+    const priorRoundDir = path.join(chatDir, `round-${round - 1}`);
+    if (!fs.existsSync(priorRoundDir))
+        return '';
+    let entries;
+    try {
+        entries = fs.readdirSync(priorRoundDir);
+    }
+    catch {
+        return '';
+    }
+    // Stable sort by dir name so reviewer order in the prompt matches the
+    // run page rendering. dir name embeds the reviewer index already.
+    const reviewerDirs = entries
+        .filter((n) => n.startsWith('reviewer-'))
+        .sort();
+    if (reviewerDirs.length === 0)
+        return '';
+    const chunks = [];
+    let totalBytes = 0;
+    for (const dir of reviewerDirs) {
+        const answerPath = path.join(priorRoundDir, dir, 'answer.md');
+        if (!fs.existsSync(answerPath))
+            continue;
+        let body;
+        try {
+            body = fs.readFileSync(answerPath, 'utf-8');
+        }
+        catch {
+            continue;
+        }
+        if (body.trim().length === 0)
+            continue;
+        // Reviewer dir name shape: `reviewer-<agentName>-<idx>`. Strip the
+        // prefix + trailing index so the heading reads cleanly. Falls back
+        // to the raw dir name if the regex misses (defensive — agent names
+        // with internal dashes still match because we anchor on -<digits>$).
+        const labelMatch = dir.match(/^reviewer-(.+)-(\d+)$/);
+        const label = labelMatch
+            ? `${labelMatch[1]} (#${labelMatch[2]})`
+            : dir;
+        const truncated = body.length > PER_REVIEWER_MAX_BYTES;
+        const slice = truncated ? body.slice(0, PER_REVIEWER_MAX_BYTES) : body;
+        const block = `### Reviewer: ${label}\n` +
+            slice +
+            (truncated ? `\n\n_(truncated — full answer was ${body.length} bytes)_\n` : '');
+        const blockBytes = Buffer.byteLength(block, 'utf-8');
+        if (totalBytes + blockBytes > TOTAL_FEEDBACK_MAX_BYTES) {
+            const remaining = reviewerDirs.length - chunks.length;
+            chunks.push(`### _(${remaining} more reviewer${remaining === 1 ? '' : 's'} omitted — feedback block exceeded ${TOTAL_FEEDBACK_MAX_BYTES}-byte cap)_`);
+            break;
+        }
+        chunks.push(block);
+        totalBytes += blockBytes;
+    }
+    if (chunks.length === 0)
+        return '';
+    return [
+        '## Prior round feedback',
+        '',
+        'The previous round did not reach reviewer consensus. Read each reviewer\'s findings below and revise your answer to address them — do not just repeat what you wrote before. Where reviewers disagree with each other, use your judgment.',
+        '',
+        ...chunks,
+        '',
+    ].join('\n');
+}
+//# sourceMappingURL=prior-round.js.map

package/dist/daemon/runner/prior-round.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prior-round.js","sourceRoot":"","sources":["../../../src/daemon/runner/prior-round.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAqCA,wDA6EC;AAlHD;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,uCAAyB;AACzB,2CAA6B;AAE7B,MAAM,sBAAsB,GAAG,EAAE,GAAG,IAAI,CAAC;AACzC,MAAM,wBAAwB,GAAG,EAAE,GAAG,IAAI,CAAC;AAE3C;;;;;;;;;GASG;AACH,SAAgB,sBAAsB,CAAC,OAAe,EAAE,KAAa;IACnE,IAAI,KAAK,IAAI,CAAC;QAAE,OAAO,EAAE,CAAC;IAE1B,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,KAAK,GAAG,CAAC,EAAE,CAAC,CAAC;IAC/D,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,aAAa,CAAC;QAAE,OAAO,EAAE,CAAC;IAE7C,IAAI,OAAiB,CAAC;IACtB,IAAI,CAAC;QACH,OAAO,GAAG,EAAE,CAAC,WAAW,CAAC,aAAa,CAAC,CAAC;IAC1C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,sEAAsE;IACtE,kEAAkE;IAClE,MAAM,YAAY,GAAG,OAAO;SACzB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;SACxC,IAAI,EAAE,CAAC;IAEV,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAEzC,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,IAAI,UAAU,GAAG,CAAC,CAAC;IAEnB,KAAK,MAAM,GAAG,IAAI,YAAY,EAAE,CAAC;QAC/B,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,GAAG,EAAE,WAAW,CAAC,CAAC;QAC9D,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC;YAAE,SAAS;QAEzC,IAAI,IAAY,CAAC;QACjB,IAAI,CAAC;YACH,IAAI,GAAG,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;QAC9C,CAAC;QAAC,MAAM,CAAC;YACP,SAAS;QACX,CAAC;QAED,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC;YAAE,SAAS;QAEvC,mEAAmE;QACnE,mEAAmE;QACnE,mEAAmE;QACnE,qEAAqE;QACrE,MAAM,UAAU,GAAG,GAAG,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC;QACtD,MAAM,KAAK,GAAG,UAAU;YACtB,CAAC,CAAC,GAAG,UAAU,CAAC,CAAC,CAAC,MAAM,UAAU,CAAC,CAAC,CAAC,GAAG;YACxC,CAAC,CAAC,GAAG,CAAC;QAER,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,GAAG,sBAAsB,CAAC;QACvD,MAAM,KAAK,GAAG,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,sBAAsB,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QAEvE,MAAM,KAAK,GACT,iBAAiB,KAAK,IAAI;YAC1B,KAAK;YACL,CAAC,SAAS,CAAC,CAAC,CAAC,qCAAqC,IAAI,CAAC,MAAM,YAAY,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QAElF,MAAM,UAAU,GAAG,MAAM,CAAC,UAAU,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC;QACrD,IAAI,UAAU,GAAG,UAAU,GAAG,wBAAwB,EAAE,CAAC;YACvD,MAAM,SAAS,GAAG,YAAY,CAAC,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC;YACtD,MAAM,CAAC,IAAI,CACT,SAAS,SAAS,iBAAiB,SAAS,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,sCAAsC,wBAAwB,aAAa,CACzI,CAAC;YACF,MAAM;QACR,CAAC;QAED,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACnB,UAAU,IAAI,UAAU,CAAC;IAC3B,CAAC;IAED,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAEnC,OAAO;QACL,yBAAyB;QACzB,EAAE;QACF,2OAA2O;QAC3O,EAAE;QACF,GAAG,MAAM;QACT,EAAE;KACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC"}