chorus-codes 0.7.0 → 0.7.1

package/dist/daemon/runner/prompt-builder.js

@@ -0,0 +1,292 @@
+"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.packAttachedFiles = packAttachedFiles;
+exports.buildAsk = buildAsk;
+exports.buildReviewerAsk = buildReviewerAsk;
+/**
+ * Pure prompt-construction helpers.
+ *
+ * Three functions that take phase config + user inputs and return the
+ * ask.md text the runner pastes into the doer/reviewer CLIs. No fs writes,
+ * no subprocess — just string assembly + (for packAttachedFiles) read-only
+ * filesystem inspection that's exercised through tests against tmp dirs.
+ *
+ * Extracted out of runner.ts so the streaming hot paths can be split later
+ * without breaking these contracts.
+ */
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+// Per-file cap and total cap when inlining attached files into a prompt.
+// Numbers chosen to keep prompts comfortably within Anthropic / OpenAI / Google
+// 1M-token budgets while still surfacing realistic source files. Hardcoded
+// for now; if template authors need larger payloads we'd lift these into
+// template config (template.inputs.maxFileBytes / maxTotalBytes).
+const ATTACHED_FILE_MAX_BYTES = 64 * 1024;
+const ATTACHED_FILES_TOTAL_BYTES = 256 * 1024;
+/**
+ * Inline the contents of user-attached files into a single markdown block
+ * the doer/reviewer can read directly. Drops files that:
+ *   - traverse out of repoPath/cwd via `..` (security)
+ *   - are symlinks (TOCTOU defence)
+ *   - aren't regular files (sockets, fifos, etc.)
+ *   - don't exist
+ *   - would blow past the 256KB total cap
+ *
+ * Each surviving file is fenced as a markdown code block with its
+ * extension as the language hint.
+ */
+function packAttachedFiles(paths, repoPath) {
+    if (!paths || paths.length === 0)
+        return '';
+    const chunks = [];
+    let totalBytes = 0;
+    const cwdRoot = path.resolve(repoPath ?? process.cwd());
+    for (const rel of paths) {
+        if (path.isAbsolute(rel)) {
+            chunks.push(`### \`${rel}\` — _absolute path rejected, skipping_`);
+            continue;
+        }
+        const abs = path.resolve(path.join(cwdRoot, rel));
+        const display = rel;
+        if (!abs.startsWith(cwdRoot + path.sep) && abs !== cwdRoot) {
+            chunks.push(`### \`${display}\` — _path traversal rejected, skipping_`);
+            continue;
+        }
+        if (!fs.existsSync(abs)) {
+            chunks.push(`### \`${display}\` — _file not found, skipping_`);
+            continue;
+        }
+        let body;
+        try {
+            let fd = -1;
+            try {
+                // O_NOFOLLOW on Linux/macOS fails with ELOOP if path is a symlink.
+                // On Windows, O_NOFOLLOW is unsupported; fall back to lstat+read.
+                if (process.platform !== 'win32') {
+                    try {
+                        fd = fs.openSync(abs, fs.constants.O_RDONLY | fs.constants.O_NOFOLLOW);
+                    }
+                    catch (openErr) {
+                        // ELOOP = symlink detected via O_NOFOLLOW
+                        if (openErr instanceof Error && openErr.message.includes('ELOOP')) {
+                            chunks.push(`### \`${display}\` — _symlink rejected, skipping_`);
+                            continue;
+                        }
+                        throw openErr;
+                    }
+                    const stat = fs.fstatSync(fd);
+                    if (!stat.isFile()) {
+                        chunks.push(`### \`${display}\` — _not a regular file, skipping_`);
+                        continue;
+                    }
+                    body = fs.readFileSync(abs, 'utf-8');
+                }
+                else {
+                    // Windows fallback: lstat + read (not race-proof but best effort)
+                    const lstat = fs.lstatSync(abs);
+                    if (lstat.isSymbolicLink()) {
+                        chunks.push(`### \`${display}\` — _symlink rejected, skipping_`);
+                        continue;
+                    }
+                    if (!lstat.isFile()) {
+                        chunks.push(`### \`${display}\` — _not a regular file, skipping_`);
+                        continue;
+                    }
+                    body = fs.readFileSync(abs, 'utf-8');
+                }
+            }
+            finally {
+                if (fd >= 0)
+                    fs.closeSync(fd);
+            }
+        }
+        catch (err) {
+            const msg = err instanceof Error ? err.message : String(err);
+            chunks.push(`### \`${display}\` — _read error: ${msg}_`);
+            continue;
+        }
+        const truncated = body.length > ATTACHED_FILE_MAX_BYTES;
+        const slice = truncated ? body.slice(0, ATTACHED_FILE_MAX_BYTES) : body;
+        const remainingBudget = ATTACHED_FILES_TOTAL_BYTES - totalBytes;
+        if (slice.length > remainingBudget) {
+            chunks.push(`### \`${display}\` — _skipped: would exceed ${ATTACHED_FILES_TOTAL_BYTES}-byte total cap_`);
+            continue;
+        }
+        totalBytes += slice.length;
+        const ext = path.extname(display).slice(1) || '';
+        chunks.push(`### \`${display}\`${truncated ? ` (truncated to ${ATTACHED_FILE_MAX_BYTES} bytes)` : ''}\n\`\`\`${ext}\n${slice}\n\`\`\``);
+    }
+    if (chunks.length === 0)
+        return '';
+    return ['## Attached files', '', ...chunks, ''].join('\n');
+}
+/**
+ * Wrap a persona system_prompt in a fenced block for ask.md. Returns
+ * empty string when no persona prompt is provided so call sites can do
+ * `[personaBlock(...), ...rest].join('\n')` without conditionals.
+ *
+ * **Fence rationale (retroactive PR #17 review, all 3 reviewers flagged):**
+ * persona.system_prompt is user-editable text from the personas table.
+ * Without delimiters a malicious or just typo'd persona can ship markdown
+ * structure (`# heading`, `---` HR, ``` code fences) that bleeds into the
+ * task framing — at best confusing the LLM about heading hierarchy, at
+ * worst overriding "Your role" / "How to respond" sections downstream.
+ *
+ * We use HTML/XML-style tags rather than markdown fences because:
+ *   - Markdown fences would force the LLM to read the persona AS code,
+ *     not instructions.
+ *   - HTML tags are widely understood by current LLMs as semantic
+ *     boundaries (Anthropic + OpenAI both document this pattern).
+ *   - A persona that contains the literal `</persona_instructions>` to
+ *     try to break out is the only escape vector, and we strip it.
+ */
+function personaPromptBlock(systemPrompt) {
+    if (!systemPrompt || systemPrompt.trim().length === 0)
+        return '';
+    // Defensive escape: strip any closing tag that would break out of our
+    // fence. Keeps the worst case (a malicious persona) from rewriting the
+    // task framing. Open tags are harmless; only the closer matters.
+    const sanitized = systemPrompt.trim().replace(/<\/persona_instructions>/gi, '');
+    return [
+        '<persona_instructions>',
+        sanitized,
+        '</persona_instructions>',
+        '',
+    ].join('\n');
+}
+/** Build the doer ask.md prompt for one phase iteration. */
+function buildAsk(phase, _phaseIdx, round, work, inputs, filesBlock, personaSystemPrompt, priorRoundFeedback) {
+    const lines = [];
+    const personaBlock = personaPromptBlock(personaSystemPrompt);
+    if (personaBlock) {
+        lines.push(personaBlock);
+    }
+    lines.push(`# Chorus task — round ${round}, phase ${phase.id}`);
+    lines.push('');
+    lines.push('## Your role');
+    lines.push('doer');
+    lines.push('');
+    lines.push('## What to do');
+    lines.push(phase.title);
+    if (phase.description) {
+        lines.push('');
+        lines.push(phase.description);
+    }
+    lines.push('');
+    lines.push("## The user's request");
+    lines.push(work);
+    lines.push('');
+    if (filesBlock) {
+        lines.push(filesBlock);
+    }
+    if (inputs.include && inputs.include.length > 0) {
+        lines.push('## Inputs (from prior phases)');
+        for (const includePhaseId of inputs.include) {
+            lines.push(`- Phase ${includePhaseId}: (link to answer.md)`);
+        }
+        lines.push('');
+    }
+    if (inputs.exclude && inputs.exclude.length > 0) {
+        lines.push('## Excluded (do NOT read)');
+        for (const excludePhaseId of inputs.exclude) {
+            lines.push(`- Phase ${excludePhaseId}: explicitly blocked`);
+        }
+        lines.push('');
+    }
+    if (priorRoundFeedback && priorRoundFeedback.trim().length > 0) {
+        lines.push(priorRoundFeedback);
+    }
+    lines.push('## How to respond');
+    lines.push('Write your full answer and end with: ## DONE');
+    return lines.join('\n');
+}
+/** Build the reviewer ask.md prompt for one phase iteration. */
+function buildReviewerAsk(phase, _phaseIdx, round, work, doerOutput, filesBlock, personaSystemPrompt) {
+    const lines = [];
+    const personaBlock = personaPromptBlock(personaSystemPrompt);
+    if (personaBlock) {
+        lines.push(personaBlock);
+    }
+    lines.push(`# Chorus review — round ${round}, phase ${phase.id}`);
+    lines.push('');
+    lines.push('## Your role');
+    lines.push('reviewer');
+    lines.push('');
+    lines.push('## What to review');
+    lines.push(phase.title);
+    if (phase.description) {
+        lines.push('');
+        lines.push(phase.description);
+    }
+    lines.push('');
+    lines.push("## The user's request");
+    lines.push(work);
+    lines.push('');
+    if (filesBlock) {
+        lines.push(filesBlock);
+    }
+    lines.push('## Artifact to review');
+    lines.push('```');
+    // Truncation cap: 256 KB matches MAX_PHASE_OUTPUT_BYTES in lib/db. The
+    // prior 2000-char cap silently amputated any diff or draft over ~50
+    // lines, which made review-only mode useless and degraded standard
+    // review mode whenever the doer wrote a real implementation. 256 KB
+    // covers ~5000 lines of typical code; bigger artifacts truncate with a
+    // visible marker so reviewers can still flag the gap.
+    const ARTIFACT_PROMPT_CAP_BYTES = 256 * 1024;
+    const byteLen = Buffer.byteLength(doerOutput, 'utf-8');
+    if (byteLen <= ARTIFACT_PROMPT_CAP_BYTES) {
+        lines.push(doerOutput);
+    }
+    else {
+        // Slice on bytes, then walk back to the last valid UTF-8 start byte so
+        // we don't hand the LLM a U+FFFD-laden tail. UTF-8 continuation bytes
+        // start with 0b10xxxxxx — walk left while the cut byte is a
+        // continuation byte; landing on a start byte (or ASCII) is safe.
+        const buf = Buffer.from(doerOutput, 'utf-8');
+        let cut = ARTIFACT_PROMPT_CAP_BYTES;
+        while (cut > 0 && (buf[cut] & 0b1100_0000) === 0b1000_0000)
+            cut--;
+        lines.push(buf.subarray(0, cut).toString('utf-8'));
+        lines.push(`... (truncated — full artifact was ${byteLen} bytes, cap is ${ARTIFACT_PROMPT_CAP_BYTES} bytes)`);
+    }
+    lines.push('```');
+    lines.push('');
+    lines.push('## Your verdict');
+    lines.push('Do you approve? Answer: approve or request changes, end with: ## DONE');
+    return lines.join('\n');
+}
+//# sourceMappingURL=prompt-builder.js.map

package/dist/daemon/runner/prompt-builder.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"prompt-builder.js","sourceRoot":"","sources":["../../../src/daemon/runner/prompt-builder.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAmCA,8CA+FC;AAqCD,4BA4DC;AAGD,4CAkEC;AAxSD;;;;;;;;;;GAUG;AACH,uCAAyB;AACzB,2CAA6B;AAG7B,yEAAyE;AACzE,gFAAgF;AAChF,2EAA2E;AAC3E,yEAAyE;AACzE,kEAAkE;AAClE,MAAM,uBAAuB,GAAG,EAAE,GAAG,IAAI,CAAC;AAC1C,MAAM,0BAA0B,GAAG,GAAG,GAAG,IAAI,CAAC;AAE9C;;;;;;;;;;;GAWG;AACH,SAAgB,iBAAiB,CAC/B,KAA2B,EAC3B,QAA4B;IAE5B,IAAI,CAAC,KAAK,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAE5C,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,IAAI,UAAU,GAAG,CAAC,CAAC;IAEnB,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAExD,KAAK,MAAM,GAAG,IAAI,KAAK,EAAE,CAAC;QACxB,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YACzB,MAAM,CAAC,IAAI,CAAC,SAAS,GAAG,yCAAyC,CAAC,CAAC;YACnE,SAAS;QACX,CAAC;QAED,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,CAAC;QAClD,MAAM,OAAO,GAAG,GAAG,CAAC;QAEpB,IAAI,CAAC,GAAG,CAAC,UAAU,CAAC,OAAO,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,GAAG,KAAK,OAAO,EAAE,CAAC;YAC3D,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,0CAA0C,CAAC,CAAC;YACxE,SAAS;QACX,CAAC;QAED,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YACxB,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,iCAAiC,CAAC,CAAC;YAC/D,SAAS;QACX,CAAC;QAED,IAAI,IAAY,CAAC;QACjB,IAAI,CAAC;YACH,IAAI,EAAE,GAAG,CAAC,CAAC,CAAC;YACZ,IAAI,CAAC;gBACH,mEAAmE;gBACnE,kEAAkE;gBAClE,IAAI,OAAO,CAAC,QAAQ,KAAK,OAAO,EAAE,CAAC;oBACjC,IAAI,CAAC;wBACH,EAAE,GAAG,EAAE,CAAC,QAAQ,CAAC,GAAG,EAAE,EAAE,CAAC,SAAS,CAAC,QAAQ,GAAG,EAAE,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC;oBACzE,CAAC;oBAAC,OAAO,OAAO,EAAE,CAAC;wBACjB,0CAA0C;wBAC1C,IAAI,OAAO,YAAY,KAAK,IAAI,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;4BAClE,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,mCAAmC,CAAC,CAAC;4BACjE,SAAS;wBACX,CAAC;wBACD,MAAM,OAAO,CAAC;oBAChB,CAAC;oBACD,MAAM,IAAI,GAAG,EAAE,CAAC,SAAS,CAAC,EAAE,CAAC,CAAC;oBAC9B,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,CAAC;wBACnB,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,qCAAqC,CAAC,CAAC;wBACnE,SAAS;oBACX,CAAC;oBACD,IAAI,GAAG,EAAE,CAAC,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;gBACvC,CAAC;qBAAM,CAAC;oBACN,kEAAkE;oBAClE,MAAM,KAAK,GAAG,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC;oBAChC,IAAI,KAAK,CAAC,cAAc,EAAE,EAAE,CAAC;wBAC3B,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,mCAAmC,CAAC,CAAC;wBACjE,SAAS;oBACX,CAAC;oBACD,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC;wBACpB,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,qCAAqC,CAAC,CAAC;wBACnE,SAAS;oBACX,CAAC;oBACD,IAAI,GAAG,EAAE,CAAC,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;gBACvC,CAAC;YACH,CAAC;oBAAS,CAAC;gBACT,IAAI,EAAE,IAAI,CAAC;oBAAE,EAAE,CAAC,SAAS,CAAC,EAAE,CAAC,CAAC;YAChC,CAAC;QACH,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;YAC7D,MAAM,CAAC,IAAI,CAAC,SAAS,OAAO,qBAAqB,GAAG,GAAG,CAAC,CAAC;YACzD,SAAS;QACX,CAAC;QAED,MAAM,SAAS,GAAG,IAAI,CAAC,MAAM,GAAG,uBAAuB,CAAC;QACxD,MAAM,KAAK,GAAG,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,uBAAuB,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;QACxE,MAAM,eAAe,GAAG,0BAA0B,GAAG,UAAU,CAAC;QAEhE,IAAI,KAAK,CAAC,MAAM,GAAG,eAAe,EAAE,CAAC;YACnC,MAAM,CAAC,IAAI,CACT,SAAS,OAAO,+BAA+B,0BAA0B,kBAAkB,CAC5F,CAAC;YACF,SAAS;QACX,CAAC;QAED,UAAU,IAAI,KAAK,CAAC,MAAM,CAAC;QAC3B,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACjD,MAAM,CAAC,IAAI,CACT,SAAS,OAAO,KAAK,SAAS,CAAC,CAAC,CAAC,kBAAkB,uBAAuB,SAAS,CAAC,CAAC,CAAC,EAAE,WAAW,GAAG,KAAK,KAAK,UAAU,CAC3H,CAAC;IACJ,CAAC;IAED,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IACnC,OAAO,CAAC,mBAAmB,EAAE,EAAE,EAAE,GAAG,MAAM,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC7D,CAAC;AAED;;;;;;;;;;;;;;;;;;;GAmBG;AACH,SAAS,kBAAkB,CAAC,YAAgC;IAC1D,IAAI,CAAC,YAAY,IAAI,YAAY,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IACjE,sEAAsE;IACtE,uEAAuE;IACvE,iEAAiE;IACjE,MAAM,SAAS,GAAG,YAAY,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,4BAA4B,EAAE,EAAE,CAAC,CAAC;IAChF,OAAO;QACL,wBAAwB;QACxB,SAAS;QACT,yBAAyB;QACzB,EAAE;KACH,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC;AAED,4DAA4D;AAC5D,SAAgB,QAAQ,CACtB,KAAY,EACZ,SAAiB,EACjB,KAAa,EACb,IAAY,EACZ,MAAuB,EACvB,UAAkB,EAClB,mBAA4B,EAC5B,kBAA2B;IAE3B,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,MAAM,YAAY,GAAG,kBAAkB,CAAC,mBAAmB,CAAC,CAAC;IAC7D,IAAI,YAAY,EAAE,CAAC;QACjB,KAAK,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC3B,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,yBAAyB,KAAK,WAAW,KAAK,CAAC,EAAE,EAAE,CAAC,CAAC;IAChE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;IAC3B,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACnB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;IAC5B,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IACxB,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;QACtB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;IAChC,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;IACpC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACjB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,IAAI,UAAU,EAAE,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACzB,CAAC;IAED,IAAI,MAAM,CAAC,OAAO,IAAI,MAAM,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChD,KAAK,CAAC,IAAI,CAAC,+BAA+B,CAAC,CAAC;QAC5C,KAAK,MAAM,cAAc,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;YAC5C,KAAK,CAAC,IAAI,CAAC,WAAW,cAAc,uBAAuB,CAAC,CAAC;QAC/D,CAAC;QACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IAED,IAAI,MAAM,CAAC,OAAO,IAAI,MAAM,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChD,KAAK,CAAC,IAAI,CAAC,2BAA2B,CAAC,CAAC;QACxC,KAAK,MAAM,cAAc,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;YAC5C,KAAK,CAAC,IAAI,CAAC,WAAW,cAAc,sBAAsB,CAAC,CAAC;QAC9D,CAAC;QACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IAED,IAAI,kBAAkB,IAAI,kBAAkB,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC/D,KAAK,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;IACjC,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IAChC,KAAK,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAC;IAE3D,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED,gEAAgE;AAChE,SAAgB,gBAAgB,CAC9B,KAAY,EACZ,SAAiB,EACjB,KAAa,EACb,IAAY,EACZ,UAAkB,EAClB,UAAkB,EAClB,mBAA4B;IAE5B,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,MAAM,YAAY,GAAG,kBAAkB,CAAC,mBAAmB,CAAC,CAAC;IAC7D,IAAI,YAAY,EAAE,CAAC;QACjB,KAAK,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC;IAC3B,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,2BAA2B,KAAK,WAAW,KAAK,CAAC,EAAE,EAAE,CAAC,CAAC;IAClE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;IAC3B,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACvB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IAChC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IACxB,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;QACtB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;IAChC,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;IACpC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACjB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,IAAI,UAAU,EAAE,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACzB,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;IACpC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClB,uEAAuE;IACvE,oEAAoE;IACpE,mEAAmE;IACnE,oEAAoE;IACpE,uEAAuE;IACvE,sDAAsD;IACtD,MAAM,yBAAyB,GAAG,GAAG,GAAG,IAAI,CAAC;IAC7C,MAAM,OAAO,GAAG,MAAM,CAAC,UAAU,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IACvD,IAAI,OAAO,IAAI,yBAAyB,EAAE,CAAC;QACzC,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IACzB,CAAC;SAAM,CAAC;QACN,uEAAuE;QACvE,sEAAsE;QACtE,4DAA4D;QAC5D,iEAAiE;QACjE,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;QAC7C,IAAI,GAAG,GAAG,yBAAyB,CAAC;QACpC,OAAO,GAAG,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,GAAG,WAAW,CAAC,KAAK,WAAW;YAAE,GAAG,EAAE,CAAC;QAClE,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC;QACnD,KAAK,CAAC,IAAI,CAAC,sCAAsC,OAAO,kBAAkB,yBAAyB,SAAS,CAAC,CAAC;IAChH,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;IAC9B,KAAK,CAAC,IAAI,CACR,uEAAuE,CACxE,CAAC;IAEF,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/dist/daemon/runner/review-only-phase.js

@@ -0,0 +1,103 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.runReviewOnlyPhase = runReviewOnlyPhase;
+const fs_1 = __importDefault(require("fs"));
+const path_1 = __importDefault(require("path"));
+const reviewer_driver_js_1 = require("./reviewer-driver.js");
+/**
+ * Run a review-only phase. The artifact (supplied at chat-create time) is
+ * written to a synthetic doer answer slot; reviewers then critique it
+ * just like a real doer's answer. Single pass — no iterate, no retry.
+ * Reviewer agreement/disagreement is reported via outcome.allReviewersFailed;
+ * the verdict itself doesn't gate further phases (review-only is the whole
+ * point of the chat).
+ *
+ * Synthetic doer events make the cockpit + replay code paths Just Work
+ * without special-casing the missing doer card.
+ */
+async function runReviewOnlyPhase(args) {
+    const { chatDir, chatId, phase, phaseIdx, artifact, work, filesBlock, tmuxMgr, errorDetector, onEvent, abortSignal, } = args;
+    if (abortSignal.aborted) {
+        return { completed: false, allReviewersFailed: false, agreed: false, summary: 'aborted' };
+    }
+    const round = 1; // review-only is always single-pass
+    const roundDir = path_1.default.join(chatDir, `round-${round}`);
+    if (!fs_1.default.existsSync(roundDir)) {
+        fs_1.default.mkdirSync(roundDir, { recursive: true });
+    }
+    // Synthetic doer dir holds the artifact as answer.md so the cockpit's
+    // existing replay code finds it under the same path shape it expects.
+    const syntheticDoerDir = path_1.default.join(roundDir, 'doer-artifact');
+    if (!fs_1.default.existsSync(syntheticDoerDir)) {
+        fs_1.default.mkdirSync(syntheticDoerDir, { recursive: true });
+    }
+    const answerFile = path_1.default.join(syntheticDoerDir, 'answer.md');
+    // Strip trailing whitespace before the sentinel check so an artifact
+    // ending with "## DONE\n" or "## DONE  " doesn't produce a duplicate
+    // sentinel after we append. Idempotent: artifacts without the sentinel
+    // get a clean "\n\n## DONE\n" tail.
+    const trimmed = artifact.replace(/\s+$/, '');
+    const artifactWithSentinel = /##\s*DONE$/i.test(trimmed)
+        ? `${trimmed}\n`
+        : `${trimmed}\n\n## DONE\n`;
+    fs_1.default.writeFileSync(answerFile, artifactWithSentinel);
+    // Synthetic doer phase events. agent='artifact' is a sentinel value the
+    // cockpit can render as "user-supplied" rather than as a real CLI run.
+    onEvent({
+        chatId,
+        type: 'phase_start',
+        payload: {
+            phaseId: phase.id,
+            phaseIdx,
+            kind: phase.kind,
+            round,
+            role: 'doer',
+            agent: 'artifact',
+            synthetic: true,
+        },
+        ts: Date.now(),
+    });
+    onEvent({
+        chatId,
+        type: 'phase_progress',
+        payload: {
+            phaseId: phase.id,
+            round,
+            role: 'doer',
+            agent: 'artifact',
+            output: artifact.slice(0, 500),
+            synthetic: true,
+        },
+        ts: Date.now(),
+    });
+    // The synthetic StandardPhase shape carries the same reviewer block
+    // plus a no-op iterate config so runReviewers can consume it. Local
+    // to this call — never escapes back into the template.
+    const syntheticStandardPhase = {
+        id: phase.id,
+        kind: 'review',
+        title: phase.title,
+        description: phase.description,
+        doer: { lineage: 'any' },
+        reviewer: phase.reviewer,
+        inputs: phase.inputs,
+        iterate: {
+            maxRounds: 1,
+            onDisagreement: 'continue',
+            shareSessionAcrossRounds: false,
+            shareSessionAcrossPhases: false,
+        },
+        timeoutMs: phase.timeoutMs,
+    };
+    const consensus = await (0, reviewer_driver_js_1.runReviewers)(chatDir, chatId, syntheticStandardPhase, phaseIdx, round, artifact, work, filesBlock, tmuxMgr, errorDetector, onEvent, abortSignal, args.templateFallbackReviewer);
+    return {
+        completed: !abortSignal.aborted,
+        allReviewersFailed: consensus.allFailed,
+        agreed: consensus.agreed,
+        summary: consensus.summary,
+    };
+}
+//# sourceMappingURL=review-only-phase.js.map

package/dist/daemon/runner/review-only-phase.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-only-phase.js","sourceRoot":"","sources":["../../../src/daemon/runner/review-only-phase.ts"],"names":[],"mappings":";;;;;AAmBA,gDAqIC;AAxJD,4CAAoB;AACpB,gDAAwB;AAIxB,6DAAoD;AAGpD;;;;;;;;;;GAUG;AACI,KAAK,UAAU,kBAAkB,CAAC,IAaxC;IAQC,MAAM,EACJ,OAAO,EACP,MAAM,EACN,KAAK,EACL,QAAQ,EACR,QAAQ,EACR,IAAI,EACJ,UAAU,EACV,OAAO,EACP,aAAa,EACb,OAAO,EACP,WAAW,GACZ,GAAG,IAAI,CAAC;IAET,IAAI,WAAW,CAAC,OAAO,EAAE,CAAC;QACxB,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,kBAAkB,EAAE,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAC5F,CAAC;IAED,MAAM,KAAK,GAAG,CAAC,CAAC,CAAC,oCAAoC;IACrD,MAAM,QAAQ,GAAG,cAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,KAAK,EAAE,CAAC,CAAC;IACtD,IAAI,CAAC,YAAE,CAAC,UAAU,CAAC,QAAQ,CAAC,EAAE,CAAC;QAC7B,YAAE,CAAC,SAAS,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IAC9C,CAAC;IACD,sEAAsE;IACtE,sEAAsE;IACtE,MAAM,gBAAgB,GAAG,cAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAC;IAC9D,IAAI,CAAC,YAAE,CAAC,UAAU,CAAC,gBAAgB,CAAC,EAAE,CAAC;QACrC,YAAE,CAAC,SAAS,CAAC,gBAAgB,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;IACtD,CAAC;IACD,MAAM,UAAU,GAAG,cAAI,CAAC,IAAI,CAAC,gBAAgB,EAAE,WAAW,CAAC,CAAC;IAC5D,qEAAqE;IACrE,qEAAqE;IACrE,uEAAuE;IACvE,oCAAoC;IACpC,MAAM,OAAO,GAAG,QAAQ,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IAC7C,MAAM,oBAAoB,GAAG,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC;QACtD,CAAC,CAAC,GAAG,OAAO,IAAI;QAChB,CAAC,CAAC,GAAG,OAAO,eAAe,CAAC;IAC9B,YAAE,CAAC,aAAa,CAAC,UAAU,EAAE,oBAAoB,CAAC,CAAC;IAEnD,wEAAwE;IACxE,uEAAuE;IACvE,OAAO,CAAC;QACN,MAAM;QACN,IAAI,EAAE,aAAa;QACnB,OAAO,EAAE;YACP,OAAO,EAAE,KAAK,CAAC,EAAE;YACjB,QAAQ;YACR,IAAI,EAAE,KAAK,CAAC,IAAI;YAChB,KAAK;YACL,IAAI,EAAE,MAAM;YACZ,KAAK,EAAE,UAAU;YACjB,SAAS,EAAE,IAAI;SAChB;QACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;KACf,CAAC,CAAC;IACH,OAAO,CAAC;QACN,MAAM;QACN,IAAI,EAAE,gBAAgB;QACtB,OAAO,EAAE;YACP,OAAO,EAAE,KAAK,CAAC,EAAE;YACjB,KAAK;YACL,IAAI,EAAE,MAAM;YACZ,KAAK,EAAE,UAAU;YACjB,MAAM,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC;YAC9B,SAAS,EAAE,IAAI;SAChB;QACD,EAAE,EAAE,IAAI,CAAC,GAAG,EAAE;KACf,CAAC,CAAC;IAEH,oEAAoE;IACpE,oEAAoE;IACpE,uDAAuD;IACvD,MAAM,sBAAsB,GAAkB;QAC5C,EAAE,EAAE,KAAK,CAAC,EAAE;QACZ,IAAI,EAAE,QAAQ;QACd,KAAK,EAAE,KAAK,CAAC,KAAK;QAClB,WAAW,EAAE,KAAK,CAAC,WAAW;QAC9B,IAAI,EAAE,EAAE,OAAO,EAAE,KAAK,EAAE;QACxB,QAAQ,EAAE,KAAK,CAAC,QAAQ;QACxB,MAAM,EAAE,KAAK,CAAC,MAAM;QACpB,OAAO,EAAE;YACP,SAAS,EAAE,CAAC;YACZ,cAAc,EAAE,UAAU;YAC1B,wBAAwB,EAAE,KAAK;YAC/B,wBAAwB,EAAE,KAAK;SAChC;QACD,SAAS,EAAE,KAAK,CAAC,SAAS;KAC3B,CAAC;IAEF,MAAM,SAAS,GAAG,MAAM,IAAA,iCAAY,EAClC,OAAO,EACP,MAAM,EACN,sBAAsB,EACtB,QAAQ,EACR,KAAK,EACL,QAAQ,EACR,IAAI,EACJ,UAAU,EACV,OAAO,EACP,aAAa,EACb,OAAO,EACP,WAAW,EACX,IAAI,CAAC,wBAAwB,CAC9B,CAAC;IAEF,OAAO;QACL,SAAS,EAAE,CAAC,WAAW,CAAC,OAAO;QAC/B,kBAAkB,EAAE,SAAS,CAAC,SAAS;QACvC,MAAM,EAAE,SAAS,CAAC,MAAM;QACxB,OAAO,EAAE,SAAS,CAAC,OAAO;KAC3B,CAAC;AACJ,CAAC"}