@kernlang/agon 0.1.4 → 0.1.5

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

@@ -2,7 +2,7 @@
 import {
   stripReasoning,
   stripTuiChrome
-} from "./chunk-6ANHPXGZ.js";
+} from "./chunk-HSPQEDHX.js";
 import {
   filterDefaultOrchestrationEngines,
   getSessionAllowList,
@@ -11,11 +11,13 @@ import {
   runCampfire,
   runDelegate,
   runForge,
-  runTribunal
-} from "./chunk-FCCH7IPJ.js";
+  runTribunal,
+  sessionResultStore
+} from "./chunk-4NTH3EAR.js";
 import {
-  ENGINE_COLORS
-} from "./chunk-WE32YJKT.js";
+  ENGINE_COLORS,
+  icons
+} from "./chunk-DGTU4UWQ.js";
 import {
   AgentSession,
   AgentTeam,
@@ -55,7 +57,7 @@ import {
   spawnWithTimeout,
   tracker,
   worktreeChangedDiff
-} from "./chunk-C22VTCS6.js";
+} from "./chunk-HAJIKZGU.js";
 
 // src/generated/handlers/plan-mode.ts
 import { writeFileSync as writeFileSync2, mkdirSync as mkdirSync3 } from "fs";
@@ -641,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 {
@@ -670,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) {
@@ -686,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);
@@ -759,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();
@@ -771,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.` });
@@ -794,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(", "),
@@ -835,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);
   }
@@ -2635,4 +2680,4 @@ export {
   handleExitPlanMode,
   buildStepExecutors
 };
-//# sourceMappingURL=chunk-O6YP55RV.js.map
+//# sourceMappingURL=chunk-46WNYE4R.js.map