@kernlang/agon 0.1.3 → 0.1.5

package/dist/{chunk-52VTWOLH.js → chunk-46WNYE4R.js}

@@ -1,6 +1,9 @@
 #!/usr/bin/env node
 import {
-  ENGINE_COLORS,
+  stripReasoning,
+  stripTuiChrome
+} from "./chunk-HSPQEDHX.js";
+import {
   filterDefaultOrchestrationEngines,
   getSessionAllowList,
   handleCesarBrain,
@@ -8,8 +11,13 @@ import {
   runCampfire,
   runDelegate,
   runForge,
-  runTribunal
-} from "./chunk-XOJPAFCJ.js";
+  runTribunal,
+  sessionResultStore
+} from "./chunk-4NTH3EAR.js";
+import {
+  ENGINE_COLORS,
+  icons
+} from "./chunk-DGTU4UWQ.js";
 import {
   AgentSession,
   AgentTeam,
@@ -49,7 +57,7 @@ import {
   spawnWithTimeout,
   tracker,
   worktreeChangedDiff
-} from "./chunk-PFHGKBQT.js";
+} from "./chunk-HAJIKZGU.js";
 
 // src/generated/handlers/plan-mode.ts
 import { writeFileSync as writeFileSync2, mkdirSync as mkdirSync3 } from "fs";
@@ -194,98 +202,6 @@ function buildConsensus(outcomes, minVerified, minPair) {
   };
 }
 
-// src/generated/blocks/engine-helpers.ts
-function parseToolInputPayload(input) {
-  const rawInput = String(input ?? "");
-  let parsed = {};
-  try {
-    if (rawInput.trim().startsWith("{")) {
-      parsed = JSON.parse(rawInput);
-    }
-  } catch (e) {
-    parsed = {};
-  }
-  return { rawInput, parsed };
-}
-function extractPatchText(rawInput, parsed) {
-  const values = [parsed?.patch, parsed?.content, parsed?.diff, parsed?.input].filter((value) => typeof value === "string" && value.trim().length > 0);
-  const fromParsed = values.find((value) => value.includes("*** Begin Patch") || value.includes("diff --git") || value.split("\n").some((line) => line.startsWith("@@")));
-  if (fromParsed) {
-    return fromParsed;
-  }
-  if (rawInput.includes("*** Begin Patch") || rawInput.includes("diff --git") || rawInput.split("\n").some((line) => line.startsWith("@@"))) {
-    return rawInput;
-  }
-  return values[0] ?? "";
-}
-function parsePatchPreview(rawInput, parsed) {
-  const patchText = extractPatchText(rawInput, parsed);
-  const files = [];
-  const lines = [];
-  let additions = 0;
-  let deletions = 0;
-  const addFile = (filePath) => {
-    const clean = filePath.trim().replace(/^["']|["']$/g, "");
-    if (clean && !files.includes(clean)) files.push(clean);
-  };
-  for (const line of patchText.split("\n")) {
-    const customFile = line.match(/^\*\*\* (?:Update|Add|Delete) File: (.+)$/);
-    if (customFile) {
-      addFile(customFile[1]);
-      continue;
-    }
-    const diffFile = line.match(/^diff --git a\/(.+?) b\/(.+)$/);
-    if (diffFile) {
-      addFile(diffFile[2]);
-      continue;
-    }
-    if (line.startsWith("+++ b/")) {
-      addFile(line.slice("+++ b/".length));
-      continue;
-    }
-    if (line.startsWith("--- a/")) {
-      addFile(line.slice("--- a/".length));
-      continue;
-    }
-    if (line.startsWith("@@")) {
-      lines.push(line);
-      continue;
-    }
-    if (line.startsWith("+") && !line.startsWith("+++")) {
-      additions += 1;
-      lines.push(line);
-      continue;
-    }
-    if (line.startsWith("-") && !line.startsWith("---")) {
-      deletions += 1;
-      lines.push(line);
-    }
-  }
-  return { files, lines, additions, deletions };
-}
-function extractSummary(text, maxLen) {
-  let s = text.replace(/<think>[\s\S]*?<\/think>\s*/gi, "");
-  s = s.replace(/^#+\s+.+\n/gm, "");
-  s = s.replace(/^\s*[-*]\s+/gm, "");
-  s = s.replace(/\*\*/g, "");
-  s = s.trim();
-  const firstSentence = s.match(/^[^.!?\n]{10,}[.!?]/);
-  const summary = firstSentence ? firstSentence[0] : s.slice(0, maxLen);
-  return summary.length > maxLen ? summary.slice(0, maxLen - 1) + "\u2026" : summary;
-}
-function stripReasoning(text) {
-  return text.replace(/<(think|thinking|reasoning)>[\s\S]*?<\/\1>\s*/gi, "").trim();
-}
-function stripTuiChrome(text) {
-  const hadChrome = /[·✢✳✶✻✽⎿]/.test(text);
-  let s = text.replace(/[·✢✳✴✵✶✷✸✹✺✻✼✽✾⎿]/g, "").replace(/❯/g, "");
-  if (hadChrome) {
-    s = s.replace(/[A-Z][^\n…]{0,40}…/g, "");
-    s = s.replace(/^[\s\d%]+/, "");
-  }
-  return s.replace(/[ \t]{2,}/g, " ").replace(/\n{3,}/g, "\n\n").trim();
-}
-
 // src/generated/handlers/review.ts
 function resolveReviewTarget(target, cwd) {
   const t = (target ?? "uncommitted").trim();
@@ -727,6 +643,50 @@ ${repairBlock}`;
   const severityCounts = summarizeReviewFindings(response);
   return { response, blocking, parseFailed, unstructured, severityCounts, usage };
 }
+function stripMachineBlock(response) {
+  const idx = response.lastIndexOf(REVIEW_SENTINEL);
+  if (idx < 0) return response;
+  return response.slice(0, idx).trimEnd();
+}
+function reviewOutcome(engineId, response, status, note) {
+  if (status !== "ok") return { engine: engineId, status, findings: [], note };
+  const raw = (extractReviewFindings(response) || []).filter((x) => x && typeof x === "object");
+  const findings = raw.map((x) => ({
+    engine: engineId,
+    severity: typeof x.severity === "string" ? x.severity : x.blocking ? "blocking" : "nit",
+    blocking: x.blocking,
+    confidence: x.confidence,
+    file: x.file,
+    lines: x.lines,
+    problem: x.problem,
+    minimalFix: x.minimalFix
+  }));
+  return { engine: engineId, status: "ok", findings };
+}
+function buildReviewConsensusLines(consensus) {
+  const fmt = (f) => `  \u2022 [${f.severity} ${f.maxConfidence.toFixed(2)} \xD7${f.engines.length}${f.pairVotes >= 2 ? " pair" : ""}] ${f.problem}${f.file ? ` (${f.file}${f.lines ? ":" + f.lines : ""})` : ""}`;
+  const lines = [`Consensus \u2014 ${consensus.summary}`];
+  if (consensus.verified.length) {
+    lines.push("VERIFIED (actionable):");
+    for (const f of consensus.verified) lines.push(fmt(f));
+  }
+  if (consensus.needsCheck.length) {
+    lines.push("NEEDS-CHECK (want a second opinion):");
+    for (const f of consensus.needsCheck) lines.push(fmt(f));
+  }
+  if (consensus.speculative.length) lines.push(`SPECULATIVE: ${consensus.speculative.length} low-confidence finding(s) \u2014 likely noise.`);
+  if (consensus.nits.length) lines.push(`NITS: ${consensus.nits.length}.`);
+  if (consensus.engineFailures.length) lines.push(`FAILED (no machine verdict): ${consensus.engineFailures.map((e) => `${e.engine} (${e.status})`).join(", ")}.`);
+  return lines;
+}
+function formatReviewCounts(c) {
+  if (!c || c.total === 0) return "no findings";
+  const parts = [];
+  if (c.blocking) parts.push(`${c.blocking} blocking`);
+  if (c.important) parts.push(`${c.important} important`);
+  if (c.nit) parts.push(`${c.nit} ${c.nit === 1 ? "nit" : "nits"}`);
+  return parts.join(", ");
+}
 async function handleReview(dispatch, ctx, target, requestedEngine) {
   const abort = new AbortController();
   try {
@@ -756,15 +716,8 @@ async function handleReview(dispatch, ctx, target, requestedEngine) {
     dispatch({ type: "spinner-start", message: `${engineId} reviewing ${label}\u2026`, color });
     let response = "";
     let unstructured = false;
-    let streaming = false;
     try {
-      const result = await runReviewCore(diff, label, engineId, ctx, abort.signal, (chunk) => {
-        if (!streaming) {
-          dispatch({ type: "spinner-stop" });
-          streaming = true;
-        }
-        dispatch({ type: "streaming-chunk", engineId, chunk });
-      });
+      const result = await runReviewCore(diff, label, engineId, ctx, abort.signal);
       response = result.response;
       unstructured = result.unstructured;
     } catch (err) {
@@ -772,32 +725,51 @@ async function handleReview(dispatch, ctx, target, requestedEngine) {
       dispatch({ type: "error", message: `${engineId}: ${err instanceof Error ? err.message : String(err)}` });
       return;
     }
+    dispatch({ type: "spinner-stop" });
     if (abort.signal.aborted) {
-      dispatch({ type: "spinner-stop" });
       return;
     }
-    if (!streaming && response) {
-      dispatch({ type: "engine-block", engineId, color, content: response });
-    }
-    if (streaming) {
-      dispatch({ type: "streaming-end", engineId });
-    }
-    if (response) {
-      appendMessage(ctx.chatSession, { role: "user", content: `[review ${label}]`, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
-      appendMessage(ctx.chatSession, { role: "engine", engineId, content: response, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
-      tracker.record(engineId, { prompt: `[review ${label}]`, response });
-      ctx.lastReviewResult = {
-        engineId,
-        target: target ?? "uncommitted",
-        label,
-        diff,
-        reviewOutput: response,
-        timestamp: Date.now()
-      };
-      dispatch({ type: "info", message: unstructured ? `Review complete (unstructured \u2014 findings weren't machine-parseable, but the review above is valid). Say "fix it" or "fix it with <engine>" to address it.` : `Review complete. Say "fix it" or "fix it with <engine>" to address the findings.` });
-    } else {
+    if (!response) {
       dispatch({ type: "warning", message: `${engineId} returned no review output.` });
+      return;
     }
+    appendMessage(ctx.chatSession, { role: "user", content: `[review ${label}]`, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
+    appendMessage(ctx.chatSession, { role: "engine", engineId, content: response, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
+    tracker.record(engineId, { prompt: `[review ${label}]`, response });
+    ctx.lastReviewResult = {
+      engineId,
+      target: target ?? "uncommitted",
+      label,
+      diff,
+      reviewOutput: response,
+      timestamp: Date.now()
+    };
+    const status = unstructured ? "unstructured" : "ok";
+    let consensusSummary;
+    let blocking = false;
+    if (status === "ok") {
+      const consensus = buildConsensus([reviewOutcome(engineId, response, status)]);
+      consensusSummary = buildReviewConsensusLines(consensus).join("\n");
+      blocking = consensus.autoBlock;
+      dispatch({ type: blocking ? "warning" : "info", message: consensusSummary });
+    } else {
+      consensusSummary = `${engineId}: unstructured review \u2014 no machine-parseable findings (the prose is valid).`;
+      dispatch({ type: "info", message: consensusSummary });
+    }
+    sessionResultStore.add({
+      type: "review",
+      timestamp: (/* @__PURE__ */ new Date()).toISOString(),
+      question: label,
+      engines: [engineId],
+      winner: null,
+      data: {
+        label,
+        consensusSummary,
+        blocking,
+        reviews: [{ engineId, status, reviewOutput: stripMachineBlock(response) }]
+      }
+    });
+    dispatch({ type: "info", message: unstructured ? `Review complete (unstructured \u2014 no machine verdict, but the review is valid). Ctrl+R for the full review \xB7 say "fix it" to address it.` : `Review complete. Ctrl+R for the full review \xB7 say "fix it" or "fix it with <engine>" to address the findings.` });
   } finally {
     dispatch({ type: "spinner-stop" });
     ctx.setActiveAbort(null);
@@ -845,7 +817,6 @@ async function handleReviewMany(dispatch, ctx, target, requestedEngines) {
         timedOut = true;
         controller.abort();
       }, timeoutSec * 1e3);
-      const color = ENGINE_COLORS[engineId] ?? 124;
       try {
         const result = await runReviewCore(diff, label, engineId, ctx, controller.signal);
         const response = (result.response ?? "").trim();
@@ -857,10 +828,11 @@ async function handleReviewMany(dispatch, ctx, target, requestedEngines) {
           dispatch({ type: "warning", message: `${engineId} returned no review output.` });
           return { engineId, reviewOutput: "", unstructured: false, status: "error", note: "no output" };
         }
-        dispatch({ type: "engine-block", engineId, color, content: response });
+        const status = result.unstructured ? "unstructured" : "ok";
+        dispatch({ type: "info", message: result.unstructured ? `${icons().success} ${engineId}: unstructured (no machine verdict)` : `${icons().success} ${engineId}: ${formatReviewCounts(result.severityCounts)}` });
         appendMessage(ctx.chatSession, { role: "engine", engineId, content: response, timestamp: (/* @__PURE__ */ new Date()).toISOString() });
         tracker.record(engineId, { prompt: `[review ${label}]`, response });
-        return { engineId, reviewOutput: response, unstructured: result.unstructured, status: result.unstructured ? "parse-failed" : "ok" };
+        return { engineId, reviewOutput: response, unstructured: result.unstructured, status };
       } catch (err) {
         if (timedOut) {
           dispatch({ type: "warning", message: `${engineId}: timed out after ${timeoutSec}s \u2014 skipped.` });
@@ -880,36 +852,10 @@ async function handleReviewMany(dispatch, ctx, target, requestedEngines) {
       dispatch({ type: "warning", message: `No review output returned from ${engineIds.join(", ")}.` });
       return;
     }
-    const outcomes = all.map((c) => {
-      if (c.status !== "ok") return { engine: c.engineId, status: c.status, findings: [], note: c.note };
-      const raw = extractReviewFindings(c.reviewOutput) || [];
-      const findings = raw.map((x) => ({
-        engine: c.engineId,
-        severity: typeof x.severity === "string" ? x.severity : x.blocking ? "blocking" : "nit",
-        blocking: x.blocking,
-        confidence: x.confidence,
-        file: x.file,
-        lines: x.lines,
-        problem: x.problem,
-        minimalFix: x.minimalFix
-      }));
-      return { engine: c.engineId, status: "ok", findings };
-    });
+    const outcomes = all.map((c) => reviewOutcome(c.engineId, c.reviewOutput, c.status, c.note));
     const consensus = buildConsensus(outcomes);
-    const fmt = (f) => `  \u2022 [${f.severity} ${f.maxConfidence.toFixed(2)} \xD7${f.engines.length}${f.pairVotes >= 2 ? " pair" : ""}] ${f.problem}${f.file ? ` (${f.file}${f.lines ? ":" + f.lines : ""})` : ""}`;
-    const lines = [`Consensus \u2014 ${consensus.summary}`];
-    if (consensus.verified.length) {
-      lines.push("VERIFIED (actionable):");
-      for (const f of consensus.verified) lines.push(fmt(f));
-    }
-    if (consensus.needsCheck.length) {
-      lines.push("NEEDS-CHECK (want a second opinion):");
-      for (const f of consensus.needsCheck) lines.push(fmt(f));
-    }
-    if (consensus.speculative.length) lines.push(`SPECULATIVE: ${consensus.speculative.length} low-confidence finding(s) \u2014 likely noise.`);
-    if (consensus.nits.length) lines.push(`NITS: ${consensus.nits.length}.`);
-    if (consensus.engineFailures.length) lines.push(`FAILED (no machine verdict): ${consensus.engineFailures.map((e) => `${e.engine} (${e.status})`).join(", ")}.`);
-    dispatch({ type: consensus.autoBlock ? "warning" : "info", message: lines.join("\n") });
+    const consensusSummary = buildReviewConsensusLines(consensus).join("\n");
+    dispatch({ type: consensus.autoBlock ? "warning" : "info", message: consensusSummary });
     const anyUnstructured = collected.some((c) => c.unstructured);
     ctx.lastReviewResult = {
       engineId: collected.map((r) => r.engineId).join(", "),
@@ -921,7 +867,20 @@ async function handleReviewMany(dispatch, ctx, target, requestedEngines) {
 ${r.reviewOutput}`).join("\n\n---\n\n"),
       timestamp: Date.now()
     };
-    dispatch({ type: "info", message: `Multi-review complete (${collected.map((r) => r.engineId).join(", ")}).${anyUnstructured ? " Some reviews were unstructured (no machine verdict) but valid." : ""} Say "fix it" or "fix it with <engine>" to address the findings.` });
+    sessionResultStore.add({
+      type: "review",
+      timestamp: (/* @__PURE__ */ new Date()).toISOString(),
+      question: label,
+      engines: collected.map((r) => r.engineId),
+      winner: null,
+      data: {
+        label,
+        consensusSummary,
+        blocking: consensus.autoBlock,
+        reviews: collected.map((r) => ({ engineId: r.engineId, status: r.status, reviewOutput: stripMachineBlock(r.reviewOutput) }))
+      }
+    });
+    dispatch({ type: "info", message: `Multi-review complete (${collected.map((r) => r.engineId).join(", ")}).${anyUnstructured ? " Some reviews were unstructured (no machine verdict) but valid." : ""} Ctrl+R for the full reviews \xB7 say "fix it" or "fix it with <engine>" to address the findings.` });
   } finally {
     ctx.setActiveAbort(null);
   }
@@ -2709,9 +2668,6 @@ ${tResult.summary}` },
 
 export {
   buildConsensus,
-  parseToolInputPayload,
-  parsePatchPreview,
-  extractSummary,
   resolveReviewTarget,
   selectReviewEngine,
   extractReviewFindings,
@@ -2724,4 +2680,4 @@ export {
   handleExitPlanMode,
   buildStepExecutors
 };
-//# sourceMappingURL=chunk-52VTWOLH.js.map
+//# sourceMappingURL=chunk-46WNYE4R.js.map