@mrrlin-dev/mcp 0.4.2 → 0.4.4

package/dist/bin.cjs

@@ -4273,7 +4273,7 @@ var require_websocket = __commonJS({
     var http2 = require("http");
     var net = require("net");
     var tls = require("tls");
-    var { randomBytes: randomBytes4, createHash: createHash5 } = require("crypto");
+    var { randomBytes: randomBytes4, createHash: createHash4 } = require("crypto");
     var { Duplex, Readable } = require("stream");
     var { URL: URL2 } = require("url");
     var PerMessageDeflate2 = require_permessage_deflate();
@@ -4941,7 +4941,7 @@ var require_websocket = __commonJS({
           abortHandshake(websocket, socket, "Invalid Upgrade header");
           return;
         }
-        const digest = createHash5("sha1").update(key + GUID).digest("base64");
+        const digest = createHash4("sha1").update(key + GUID).digest("base64");
         if (res.headers["sec-websocket-accept"] !== digest) {
           abortHandshake(websocket, socket, "Invalid Sec-WebSocket-Accept header");
           return;
@@ -5310,7 +5310,7 @@ var require_websocket_server = __commonJS({
     var EventEmitter3 = require("events");
     var http2 = require("http");
     var { Duplex } = require("stream");
-    var { createHash: createHash5 } = require("crypto");
+    var { createHash: createHash4 } = require("crypto");
     var extension2 = require_extension();
     var PerMessageDeflate2 = require_permessage_deflate();
     var subprotocol2 = require_subprotocol();
@@ -5617,7 +5617,7 @@ var require_websocket_server = __commonJS({
           );
         }
         if (this._state > RUNNING) return abortHandshake(socket, 503);
-        const digest = createHash5("sha1").update(key + GUID).digest("base64");
+        const digest = createHash4("sha1").update(key + GUID).digest("base64");
         const headers = [
           "HTTP/1.1 101 Switching Protocols",
           "Upgrade: websocket",
@@ -21174,7 +21174,7 @@ var import_node_http = __toESM(require("node:http"), 1);
 var import_node_os9 = __toESM(require("node:os"), 1);
 var import_node_path18 = __toESM(require("node:path"), 1);
 var import_node_child_process11 = require("node:child_process");
-var import_node_crypto8 = __toESM(require("node:crypto"), 1);
+var import_node_crypto7 = __toESM(require("node:crypto"), 1);
 
 // src/turn-replay-buffer.ts
 var DEFAULT_MAX_EVENTS = 2e3;
@@ -36574,6 +36574,8 @@ var agentPoolResponseSchema = external_exports.object({
 
 // ../../packages/schemas/dist/specConsensusReview.js
 var mrrlinSpecConsensusSuppressionEvidenceSchema = external_exports.record(external_exports.string(), external_exports.unknown()).nullable();
+var mrrlinSpecConsensusHistorySchema = external_exports.unknown();
+var mrrlinSpecConsensusHighlightsSchema = external_exports.object({ findings: external_exports.array(external_exports.string()) });
 var mrrlinSpecConsensusReviewSchema = external_exports.object({
   projectSlug: external_exports.string().min(1),
   specPageId: external_exports.string().min(1),
@@ -36581,6 +36583,8 @@ var mrrlinSpecConsensusReviewSchema = external_exports.object({
   outcome: external_exports.string().min(1),
   rounds: external_exports.number().int().nonnegative(),
   suppressionEvidence: mrrlinSpecConsensusSuppressionEvidenceSchema.default(null),
+  history: mrrlinSpecConsensusHistorySchema.nullable().default(null),
+  highlights: mrrlinSpecConsensusHighlightsSchema.nullable().default(null),
   createdAt: external_exports.string().datetime()
 });
 var mrrlinSpecConsensusReviewCreateSchema = external_exports.object({
@@ -36588,10 +36592,26 @@ var mrrlinSpecConsensusReviewCreateSchema = external_exports.object({
   contentHash: external_exports.string().min(1),
   outcome: external_exports.string().min(1),
   rounds: external_exports.number().int().nonnegative(),
-  suppressionEvidence: mrrlinSpecConsensusSuppressionEvidenceSchema.optional()
+  suppressionEvidence: mrrlinSpecConsensusSuppressionEvidenceSchema.optional(),
+  history: mrrlinSpecConsensusHistorySchema.optional(),
+  highlights: mrrlinSpecConsensusHighlightsSchema.optional()
 }).strict();
 var specConsensusReviewResponseSchema = external_exports.object({ data: mrrlinSpecConsensusReviewSchema });
 var specConsensusReviewMaybeResponseSchema = external_exports.object({ data: mrrlinSpecConsensusReviewSchema.nullable() });
+var PERSONA_ROLE_MAP = {
+  architect: "design",
+  "security-analyst": "security",
+  "seo-analyst": "SEO",
+  "compliance-reviewer": "compliance",
+  debugger: "root cause",
+  researcher: "research",
+  "content-editor": "wording",
+  "code-reviewer": "code",
+  "content-author": "content",
+  "seo-writer": "SEO writing",
+  "spec-author": "spec"
+};
+var UNNAMED_PERSONAS = ["plan-reviewer", "scope-analyst"];
 
 // ../../packages/schemas/dist/asyncJob.js
 var ASYNC_JOB_STATUSES = ["queued", "running", "done", "failed", "superseded"];
@@ -40190,7 +40210,7 @@ function startCodexProbe(deps = {}) {
 }
 
 // src/_generated/version.ts
-var PKG_VERSION = "0.4.2";
+var PKG_VERSION = "0.4.4";
 
 // src/api-base-url-migrate.ts
 var import_node_fs2 = __toESM(require("node:fs"), 1);
@@ -40992,7 +41012,7 @@ function startAgentPoolLoop(deps) {
 
 // src/execution-consumer.ts
 var import_node_child_process9 = require("node:child_process");
-var import_node_crypto5 = __toESM(require("node:crypto"), 1);
+var import_node_crypto4 = __toESM(require("node:crypto"), 1);
 var import_node_path10 = __toESM(require("node:path"), 1);
 var import_node_util = require("node:util");
 
@@ -41684,7 +41704,7 @@ async function runSpecGate(_target, deps, opts) {
   });
   if (result.outcome === "CONVERGED") {
     await deps.promoteSpecReady(currentHash);
-    return { outcome: "CONVERGED" };
+    return { outcome: "CONVERGED", headOutcomes: result.history.at(-1)?.headOutcomes ?? [] };
   }
   if (result.outcome === "CONSENSUS_BLOCKED") {
     return {
@@ -41734,6 +41754,63 @@ async function resolveSpecDomain(client, projectSlug, page) {
   }
 }
 
+// src/consensus/findings.ts
+function shouldCurateFindings(history) {
+  return history.some((entry) => entry.accepted.length > 0);
+}
+var SYNTHESIZED_PLACEHOLDER = /^head .* returned .* but no parseable critical issues/;
+function selectFindingsCandidates(history) {
+  const result = [];
+  for (const entry of history) {
+    for (const issue2 of entry.accepted) {
+      if (SYNTHESIZED_PLACEHOLDER.test(issue2.text)) {
+        continue;
+      }
+      result.push(issue2);
+    }
+  }
+  if (result.length > 3) {
+    return result.slice(0, 3);
+  }
+  return result;
+}
+function isFindingsHighlights(value) {
+  return typeof value === "object" && value !== null && Array.isArray(value.findings) && value.findings.every((f) => typeof f === "string");
+}
+async function curateFindings(history, deps) {
+  if (!shouldCurateFindings(history)) return null;
+  try {
+    const candidates = selectFindingsCandidates(history);
+    const issuesBlock = candidates.length > 0 ? candidates.map((issue2) => `- (${issue2.category}) ${issue2.text}`).join("\n") : "(no issues survived the deterministic pre-filter)";
+    const result = await deps.runCodexExec({
+      prompt: `--- ACCEPTED ISSUES ---
+${issuesBlock}`,
+      developerInstructions: deps.loadPersona("findings-writer"),
+      cwd: deps.cwd,
+      timeoutMs: deps.timeoutMs,
+      sandbox: "read-only",
+      codexExecutable: deps.codexExecutable
+    });
+    if (!result.ok) {
+      process.stderr.write(`[mrrlin-mcp consensus] findings curation failed: ${result.error}
+`);
+      return null;
+    }
+    const parsed = JSON.parse(result.text);
+    if (!isFindingsHighlights(parsed)) {
+      process.stderr.write("[mrrlin-mcp consensus] findings curation returned unparseable output\n");
+      return null;
+    }
+    return { findings: parsed.findings.slice(0, 3) };
+  } catch (err) {
+    process.stderr.write(
+      `[mrrlin-mcp consensus] findings curation threw: ${err instanceof Error ? err.message : String(err)}
+`
+    );
+    return null;
+  }
+}
+
 // src/consensus/code-gate-git.ts
 var import_node_child_process7 = require("node:child_process");
 var DIFF_MAX_BYTES = 2e5;
@@ -42173,12 +42250,22 @@ ${artifact}`,
       }
       const outcome = r["outcome"];
       const rounds2 = r["rounds"];
+      const history = r["history"];
+      const highlights = await curateFindings(Array.isArray(history) ? history : [], {
+        runCodexExec,
+        loadPersona,
+        cwd,
+        timeoutMs,
+        codexExecutable
+      });
       const { domain: domain2 } = await loadIntent();
       try {
         await client.recordSpecConsensusReview(projectSlug, specPageId, {
           contentHash: domainScopedReviewHash2(contentHash, domain2),
           outcome: String(outcome),
-          rounds: typeof rounds2 === "number" ? rounds2 : 0
+          rounds: typeof rounds2 === "number" ? rounds2 : 0,
+          ...history !== void 0 ? { history } : {},
+          ...highlights ? { highlights } : {}
         });
       } catch (err) {
         process.stderr.write(
@@ -42386,11 +42473,21 @@ You revise a deliverable to address the accepted issues so it satisfies the acce
       if (typeof contentHash !== "string" || contentHash.length === 0) return;
       const outcome = String(r["outcome"]);
       const rounds2 = typeof r["rounds"] === "number" ? r["rounds"] : 0;
+      const history = r["history"];
+      const highlights = await curateFindings(Array.isArray(history) ? history : [], {
+        runCodexExec,
+        loadPersona,
+        cwd,
+        timeoutMs,
+        codexExecutable
+      });
       try {
         await client.recordSpecConsensusReview(projectSlug, deliverablePageId, {
           contentHash: domainScopedReviewHash2(contentHash, domain2),
           outcome,
-          rounds: rounds2
+          rounds: rounds2,
+          ...history !== void 0 ? { history } : {},
+          ...highlights ? { highlights } : {}
         });
       } catch (err) {
         process.stderr.write(
@@ -42400,6 +42497,7 @@ You revise a deliverable to address the accepted issues so it satisfies the acce
       }
       if (runId && taskId) {
         try {
+          const diagnostics = Array.isArray(r["diagnostics"]) ? r["diagnostics"] : void 0;
           await client.appendExecutionArtifact(projectSlug, runId, {
             id: `EA-${import_node_crypto2.default.randomUUID()}`,
             kind: "verification",
@@ -42409,7 +42507,8 @@ You revise a deliverable to address the accepted issues so it satisfies the acce
               runId,
               taskId,
               deliverablePageId,
-              contentHash
+              contentHash,
+              ...diagnostics ? { diagnostics } : {}
             }
           });
         } catch {
@@ -42420,6 +42519,35 @@ You revise a deliverable to address the accepted issues so it satisfies the acce
 }
 
 // src/consensus/deliverable-gate-outcome.ts
+var MAX_DELIVERABLE_DIAGNOSTICS = 5;
+var MAX_DELIVERABLE_DIAGNOSTIC_LENGTH = 240;
+var MAX_DELIVERABLE_SUMMARY_LENGTH = 1200;
+function normalizeDiagnostic(text) {
+  return text.replace(/\s+/g, " ").trim();
+}
+function renderSummary(total, items, omitted) {
+  const label = `${total} finding${total === 1 ? "" : "s"} (${omitted} omitted)`;
+  return items.length > 0 ? `${label}: ${items.join("; ")}` : label;
+}
+function formatDeliverableGateDiagnostics(diagnostics) {
+  const items = [];
+  for (const diagnostic of diagnostics) {
+    if (items.length >= MAX_DELIVERABLE_DIAGNOSTICS) break;
+    const category = normalizeDiagnostic(diagnostic.category);
+    const text = normalizeDiagnostic(diagnostic.text);
+    const item = `${category}: ${text}`.slice(0, MAX_DELIVERABLE_DIAGNOSTIC_LENGTH);
+    const candidate = renderSummary(diagnostics.length, [...items, item], diagnostics.length - items.length - 1);
+    if (candidate.length <= MAX_DELIVERABLE_SUMMARY_LENGTH) items.push(item);
+  }
+  let omitted = diagnostics.length - items.length;
+  let summary = renderSummary(diagnostics.length, items, omitted);
+  while (summary.length > MAX_DELIVERABLE_SUMMARY_LENGTH && items.length > 0) {
+    items.pop();
+    omitted = diagnostics.length - items.length;
+    summary = renderSummary(diagnostics.length, items, omitted);
+  }
+  return summary;
+}
 function mapDeliverableGateResult(result) {
   switch (result.outcome) {
     case "DELIVERABLE_CHANGED_DURING_REVIEW":
@@ -42433,17 +42561,9 @@ function mapDeliverableGateResult(result) {
         reason: "reviewers were unavailable (provider/quota) during the deliverable review (CONSENSUS_UNAVAILABLE)"
       };
     case "CONSENSUS_BLOCKED": {
-      const clarifications = result.clarifications ?? [];
-      if (clarifications.length > 0) {
-        return {
-          action: "questions",
-          clarifications,
-          reason: "reviewers did not converge; unresolved scope/ambiguity/correctness issues were filed as operator questions (CONSENSUS_BLOCKED)"
-        };
-      }
       return {
         action: "handoff",
-        reason: "reviewers did not converge and raised no clarification-eligible issues (CONSENSUS_BLOCKED)"
+        reason: `reviewers did not converge; ${formatDeliverableGateDiagnostics(result.diagnostics ?? [])} (CONSENSUS_BLOCKED)`
       };
     }
     case "MISSING_ACCEPTANCE_CRITERIA":
@@ -42462,74 +42582,8 @@ function mapDeliverableGateResult(result) {
   }
 }
 
-// src/consensus/deliverable-gate-questions.ts
-var import_node_crypto3 = require("node:crypto");
-function buildDeliverableQuestionOptions(category) {
-  if (category === "correctness") {
-    return [
-      {
-        label: "Deliverable is correct",
-        description: "The flagged claim is accurate as written; no change needed.",
-        answer: "Confirm the deliverable's content is correct as written; the reviewer's concern does not require a change."
-      },
-      {
-        label: "Deliverable needs correction",
-        description: "The flagged claim is wrong or unsupported; the deliverable needs revision.",
-        answer: "The deliverable is incorrect here and must be revised to fix the flagged claim before it can be accepted."
-      }
-    ];
-  }
-  if (category === "scope") {
-    return [
-      {
-        label: "Coverage is sufficient",
-        description: "The deliverable's current coverage of the acceptance criteria is sufficient as written.",
-        answer: "Accept the deliverable's current scope/coverage as sufficient against the acceptance criteria."
-      },
-      {
-        label: "Coverage is missing",
-        description: "The deliverable is missing required coverage and needs revision to address it.",
-        answer: "The deliverable is missing required coverage and must be revised to address the flagged gap."
-      }
-    ];
-  }
-  return [
-    {
-      label: "Interpretation is fine",
-      description: "The deliverable's current interpretation of the ambiguous point is acceptable as written.",
-      answer: "Accept the deliverable's current interpretation of the ambiguous point as written."
-    },
-    {
-      label: "Needs clarification",
-      description: "The ambiguous point needs to be resolved and the deliverable revised accordingly.",
-      answer: "The deliverable must be revised to resolve the flagged ambiguity explicitly."
-    }
-  ];
-}
-async function fileDeliverableGateQuestions(client, projectSlug, taskId, clarifications) {
-  let filed = 0;
-  for (const clarification of clarifications) {
-    const id = `II-${taskId}-deliverable-gate-${(0, import_node_crypto3.createHash)("sha256").update(clarification.question).digest("hex").slice(0, 10)}`;
-    try {
-      await client.createInboxItem(projectSlug, {
-        id,
-        kind: "question",
-        taskId,
-        body: clarification.question,
-        origin: "gate",
-        questionOptions: buildDeliverableQuestionOptions(clarification.category)
-      });
-      filed += 1;
-    } catch (error51) {
-      const message = error51 instanceof Error ? error51.message : String(error51);
-      if (/inbox_items_pkey/.test(message)) filed += 1;
-    }
-  }
-  return filed;
-}
-
 // src/async-tools/draft-spec.ts
-var import_node_crypto4 = require("node:crypto");
+var import_node_crypto3 = require("node:crypto");
 var DRAFT_SPEC_TOOL_NAME = "draft_task_spec";
 var SPECS_FOLDER = "Specs";
 var DRAFT_SPEC_TLDR_MAX_CHARS = 300;
@@ -43008,7 +43062,7 @@ function onComplete(outcome, input) {
     };
   }
   const summary = outcome.error;
-  const hash3 = (0, import_node_crypto4.createHash)("sha256").update(summary).digest("hex").slice(0, 10);
+  const hash3 = (0, import_node_crypto3.createHash)("sha256").update(summary).digest("hex").slice(0, 10);
   return {
     kind: "info",
     id: `II-sched-draft-failed-${input.taskId}-${hash3}`,
@@ -43040,9 +43094,9 @@ function renderMilestone(m, ctx) {
       return s;
     }
     case "consensus_converged":
-      return "Spec review passed \u2014 no issues found";
+      return "The reviewers agreed on the spec";
     case "consensus_converged_needs_approval": {
-      let s = "Spec review passed. Please review and approve";
+      let s = "The reviewers agreed on the spec. Please review and approve";
       if (ctx.specUrl) s += ` \u2014 ${ctx.specUrl}`;
       if (ctx.tldr) s += `. Summary: ${ctx.tldr}`;
       return s;
@@ -44310,7 +44364,7 @@ function isBridgeGeneratedArtifact(artifact) {
   return typeof payload === "object" && payload !== null && "event" in payload;
 }
 function newArtifactId() {
-  return `EA-${import_node_crypto5.default.randomUUID()}`;
+  return `EA-${import_node_crypto4.default.randomUUID()}`;
 }
 var BLAST_RADIUS_FLOOR_PATTERNS = [/^supabase\/migrations\//, /^packages\/db\/src\/schema\.ts$/];
 function isBlastRadiusFloorPath(path19) {
@@ -44565,7 +44619,7 @@ async function driveRun(deps, projectSlug, runId) {
   if (!claimed) return;
   if (!claimed.leaseId) {
     await deps.client.createInboxItem(projectSlug, {
-      id: `II-${import_node_crypto5.default.randomUUID()}`,
+      id: `II-${import_node_crypto4.default.randomUUID()}`,
       kind: "handoff",
       taskId: claimed.taskId,
       body: `Execution run ${runId} was claimed but returned no leaseId. This indicates DB/API corruption and requires operator attention.`
@@ -45355,10 +45409,6 @@ async function driveNonCodeRun(deps, projectSlug, runId, leaseId, task, domain2)
   }
   if (gateResult.outcome !== "CONVERGED") {
     const mapped = mapDeliverableGateResult(gateResult);
-    if (mapped.action === "questions") {
-      await fileDeliverableGateQuestions(deps.client, projectSlug, task.id, mapped.clarifications).catch(() => {
-      });
-    }
     await failRun(
       deps,
       projectSlug,
@@ -45488,7 +45538,7 @@ async function reconcileTaskAwaitingMerge(deps, projectSlug, task, runId, reason
   }
   try {
     await deps.client.createInboxItem(projectSlug, {
-      id: `II-${import_node_crypto5.default.randomUUID()}`,
+      id: `II-${import_node_crypto4.default.randomUUID()}`,
       kind: "handoff",
       taskId: task.id,
       body: `Task ${task.id} (${task.title}) still needs operator follow-up: ${reason}. Mrrlin could not move the task back to Handoff needed automatically.`
@@ -45523,7 +45573,7 @@ async function notifyOperatorHandoff(deps, projectSlug, task, runId, reason) {
   }
   try {
     await deps.client.createInboxItem(projectSlug, {
-      id: `II-${import_node_crypto5.default.randomUUID()}`,
+      id: `II-${import_node_crypto4.default.randomUUID()}`,
       kind: "handoff",
       taskId: task.id,
       body: `Execution run ${runId} for task ${task.id} (${task.title}) needs operator attention: ${reason}`
@@ -45661,7 +45711,7 @@ async function deployGateHandoff(deps, projectSlug, task, runId, reason, leaseId
   }).catch(() => {
   });
   await deps.client.createInboxItem(projectSlug, {
-    id: `II-${import_node_crypto5.default.randomUUID()}`,
+    id: `II-${import_node_crypto4.default.randomUUID()}`,
     kind: "handoff",
     taskId: task.id,
     body: `Deploy gate for run ${runId} (task ${task.id} \u2014 ${task.title}) needs operator attention: ${reason}`
@@ -45680,7 +45730,7 @@ function maxLeaseReclaims() {
 }
 
 // src/async-job-consumer.ts
-var import_node_crypto6 = __toESM(require("node:crypto"), 1);
+var import_node_crypto5 = __toESM(require("node:crypto"), 1);
 
 // src/async-tools/registry.ts
 var registry2 = /* @__PURE__ */ new Map();
@@ -45765,7 +45815,7 @@ async function driveAsyncJob(deps, projectSlug, jobId) {
   const delivery = descriptor2.onComplete(outcome, input);
   if (!delivery.suppressInbox) {
     await deps.client.createInboxItem(projectSlug, {
-      id: delivery.id ?? `II-${import_node_crypto6.default.randomUUID()}`,
+      id: delivery.id ?? `II-${import_node_crypto5.default.randomUUID()}`,
       kind: delivery.kind,
       body: delivery.body,
       ...delivery.taskId ? { taskId: delivery.taskId } : {}
@@ -45778,13 +45828,13 @@ async function deriveRequestedHash(client, toolName, input) {
   if (input.requestedHash) return input.requestedHash;
   if (toolName === "run_spec_consensus" && input.pageId) {
     const page = await client.getWikiPage(input.projectSlug, input.pageId);
-    return page.contentHash ?? import_node_crypto6.default.createHash("sha256").update(page.markdownBody).digest("hex");
+    return page.contentHash ?? import_node_crypto5.default.createHash("sha256").update(page.markdownBody).digest("hex");
   }
   if (toolName === "draft_task_spec" && input.taskId) {
     const task = await client.getTask(input.projectSlug, input.taskId);
-    return task.updatedAt ?? import_node_crypto6.default.createHash("sha256").update(JSON.stringify(task)).digest("hex");
+    return task.updatedAt ?? import_node_crypto5.default.createHash("sha256").update(JSON.stringify(task)).digest("hex");
   }
-  return import_node_crypto6.default.createHash("sha256").update(JSON.stringify(input)).digest("hex");
+  return import_node_crypto5.default.createHash("sha256").update(JSON.stringify(input)).digest("hex");
 }
 async function runAsyncBackstopSweep(deps, drive = (slug, jobId) => driveAsyncJob(deps, slug, jobId)) {
   const projects = await deps.client.listProjects();
@@ -45843,26 +45893,34 @@ async function runDeliverableGate(_target, deps) {
     });
     return { outcome: "DELIVERABLE_CHANGED_DURING_REVIEW" };
   }
+  let diagnostics;
+  if (result.outcome === "CONSENSUS_BLOCKED") {
+    const accepted = result.history.at(-1)?.accepted ?? [];
+    diagnostics = accepted.filter((issue2) => issue2.category === "scope" || issue2.category === "ambiguity" || issue2.category === "correctness").map((issue2) => ({
+      text: issue2.text,
+      whyBlocking: `reviewer raised a ${issue2.category} issue`,
+      category: issue2.category
+    }));
+  }
   await deps.writeEvidence({
     deliverablePageId: deliverable.pageId,
     outcome: result.outcome,
     rounds: result.rounds,
-    contentHash: currentHash
+    history: result.history,
+    contentHash: currentHash,
+    ...diagnostics ? { diagnostics } : {}
   });
   if (result.outcome === "CONSENSUS_BLOCKED") {
-    const accepted = result.history.at(-1)?.accepted ?? [];
-    const clarifications = accepted.filter((issue2) => issue2.category === "scope" || issue2.category === "ambiguity" || issue2.category === "correctness").map((issue2) => ({
-      question: issue2.text,
-      whyBlocking: `reviewer raised a ${issue2.category} issue`,
-      category: issue2.category
-    }));
     return {
       outcome: "CONSENSUS_BLOCKED",
       rounds: result.rounds,
-      ...clarifications.length > 0 ? { clarifications } : {},
+      diagnostics: diagnostics ?? [],
       ...result.escalation ? { escalation: result.escalation } : {}
     };
   }
+  if (result.outcome === "CONVERGED") {
+    return { outcome: result.outcome, rounds: result.rounds, headOutcomes: result.history.at(-1)?.headOutcomes ?? [] };
+  }
   return { outcome: result.outcome, rounds: result.rounds };
 }
 
@@ -46173,7 +46231,7 @@ function applyToolBusyNotification(clock, method, params) {
 }
 
 // src/worker-token.ts
-var import_node_crypto7 = require("node:crypto");
+var import_node_crypto6 = require("node:crypto");
 var import_node_fs9 = require("node:fs");
 var import_node_os6 = __toESM(require("node:os"), 1);
 var import_node_path13 = __toESM(require("node:path"), 1);
@@ -46204,7 +46262,7 @@ function readOrCreateWorkerToken() {
   if (existing) return existing;
   const dir = workerDir();
   (0, import_node_fs9.mkdirSync)(dir, { recursive: true, mode: 448 });
-  const token = (0, import_node_crypto7.randomBytes)(32).toString("hex");
+  const token = (0, import_node_crypto6.randomBytes)(32).toString("hex");
   (0, import_node_fs9.writeFileSync)(workerTokenPath(), `${token}
 `, { encoding: "utf8", mode: 384 });
   return token;
@@ -51058,7 +51116,7 @@ function buildThreadStartParams(msg, config2, desiredModel) {
     "  bullets (`- AND ...` as needed). Scenarios make the spec verifiable.",
     "- Keep behavior contracts in the spec; implementation details belong in `## Implementation Plan`.",
     "- Apply this format ONLY to Specs/{id} pages; do not retro-fit it onto other wiki page types.",
-    "- Spec consensus runs ASYNCHRONOUSLY. Call `run_spec_consensus` for a `Specs/{id}` page; it returns IMMEDIATELY. On `already_converged`, the spec is already marked ready \u2014 proceed straight to `create_execution_run`; do NOT call `mark_spec_ready` and do NOT re-run consensus. On `queued` or `already_running`, tell the user it was launched and END your turn \u2014 do NOT poll or call it again; the result arrives in the inbox (an `info` item on CONVERGED, a `handoff` item if it was blocked/failed). When a consensus `info` item says CONVERGED, proceed to `create_execution_run`. When a `handoff` says it was blocked, address the recorded issues in the spec (any edit re-arms the gate) and call `run_spec_consensus` once more. For a NON-code task (domain seo/marketing/content, ADR 0020): the low-risk lanes `seo`, `marketing`, and `content` proceed to `create_execution_run` after consensus converges, using the same flow as code tasks. Report the spec's review outcome to the operator.",
+    '- Spec consensus runs ASYNCHRONOUSLY. Call `run_spec_consensus` for a `Specs/{id}` page; it returns IMMEDIATELY. On `already_converged`, the spec is already marked ready \u2014 proceed straight to `create_execution_run`; do NOT call `mark_spec_ready` and do NOT re-run consensus. On `queued` or `already_running`, tell the user it was launched and END your turn \u2014 do NOT poll or call it again; the result arrives in the inbox (an `info` item on CONVERGED, a `handoff` item if it was blocked/failed). When a `handoff` says it was blocked, address the recorded issues in the spec (any edit re-arms the gate) and call `run_spec_consensus` once more. For a NON-code task (domain seo/marketing/content, ADR 0020): the low-risk lanes `seo`, `marketing`, and `content` proceed to `create_execution_run` after consensus converges, using the same flow as code tasks. When the review agrees, tell the operator "the reviewers agreed on the spec" and proceed to create_execution_run. When it does not, tell them the reviewers could not agree and that there is a decision question in the inbox. Never say "consensus", "converged", or "CONVERGED" to the operator; those are internal words.',
     "- When creating a task, classify and supply the `domain` field explicitly based on the nature of the work: pass `domain:'code'` for software engineering work, `domain:'seo'` for SEO/optimization, `domain:'marketing'` for marketing/growth, or `domain:'content'` for content creation/writing \u2014 the same way you select category/subcategory. A follow-up task spawned with `parentTaskId` inherits its parent's domain automatically and requires no explicit domain.",
     '- When a task needs a browser (login flows, UI verification/clicks, scraping), pass `prerequisites:[{mcp:"mrrlin-browser",reason:"..."}]` to `create_execution_run`. If that run later fails into a handoff, the operator gets a one-click "Launch browser & resume" that brings up the dedicated browser as the turn starts.',
     "- When you actually drive a browser (calling `browser_navigate`, `browser_click`, `browser_snapshot`, `browser_type`, etc.), invoke those tools from the **`mrrlin-browser`** MCP server ONLY. Do NOT use tools from any other browser MCP (`playwright`, `chrome-devtools`, `puppeteer`, etc.) even if they are also connected on this operator's box \u2014 those use a shared/default Playwright profile that is fragile across runs. Only `mrrlin-browser` uses this project's dedicated Chromium + persistent operator profile (`~/.mrrlin/browser/profile`). If your host exposes multiple browser MCPs with overlapping tool names, prefer the `mrrlin-browser` server unconditionally.",
@@ -51493,7 +51551,7 @@ function createBridgeRequestListener(opts) {
         res.end(JSON.stringify({ ok: false, error: "Origin not allowed." }));
         return;
       }
-      const nonce = import_node_crypto8.default.randomBytes(16).toString("base64url");
+      const nonce = import_node_crypto7.default.randomBytes(16).toString("base64url");
       res.writeHead(200, { "content-type": "application/json" });
       res.end(JSON.stringify({ ok: true, wsUrl: `ws://${host}:${port}/ws`, token, nonce }));
       return;
@@ -51774,7 +51832,7 @@ function createBridgeMessageHandler(deps) {
       }
     }
     socket.on("message", async (data) => {
-      const spanId = import_node_crypto8.default.randomUUID();
+      const spanId = import_node_crypto7.default.randomUUID();
       if (logCtx) {
         logCtx.span.id = spanId;
         logCtx.span.at = Date.now();
@@ -52290,7 +52348,7 @@ function createBridgeMessageHandler(deps) {
       thread.inFlight = true;
       pendingTurnSessions.delete(directorSessionId);
       thread.replay = createReplayBuffer({
-        turnId: (0, import_node_crypto8.randomUUID)(),
+        turnId: (0, import_node_crypto7.randomUUID)(),
         spanId,
         ...deps.retentionMs !== void 0 ? { retentionMs: deps.retentionMs } : {},
         ...deps.ackLingerMs !== void 0 ? { ackLingerMs: deps.ackLingerMs } : {}
@@ -52858,7 +52916,7 @@ async function startDirectorBridge() {
   };
   const sessionSinks = createSessionSinkRegistry();
   const tombstones = /* @__PURE__ */ new Map();
-  const bridgeInstanceId = (0, import_node_crypto8.randomUUID)();
+  const bridgeInstanceId = (0, import_node_crypto7.randomUUID)();
   const asyncConsumerDeps = {
     client,
     run: wrappedTurnLock.run,
@@ -53012,7 +53070,7 @@ async function startDirectorBridge() {
     fileUpdateInboxItem: async (projectSlug) => {
       const body = selfUpdateHandle.updateInboxBody();
       if (!body) return;
-      const id = `II-MCPUPD-${(0, import_node_crypto8.randomUUID)()}`;
+      const id = `II-MCPUPD-${(0, import_node_crypto7.randomUUID)()}`;
       try {
         const row = await client.createInboxItem(projectSlug, { id, kind: "info", body });
         process.stderr.write(
@@ -53042,7 +53100,7 @@ async function startDirectorBridge() {
   const bridgeLogger = createBridgeLogger(config2.stateDir, process.env);
   let bridgeBinShaShort = "unknown00000";
   try {
-    bridgeBinShaShort = import_node_crypto8.default.createHash("sha256").update(import_node_fs15.default.readFileSync(process.argv[1] ?? "")).digest("hex").slice(0, 12);
+    bridgeBinShaShort = import_node_crypto7.default.createHash("sha256").update(import_node_fs15.default.readFileSync(process.argv[1] ?? "")).digest("hex").slice(0, 12);
   } catch {
   }
   attachBridgeWss(server, {
@@ -53404,7 +53462,7 @@ function timingSafeTokenEqual(provided, expected) {
   const a = Buffer.from(provided);
   const b = Buffer.from(expected);
   if (a.length !== b.length) return false;
-  return import_node_crypto8.default.timingSafeEqual(a, b);
+  return import_node_crypto7.default.timingSafeEqual(a, b);
 }
 function readDispatchBody(req) {
   return new Promise((resolve, reject) => {
@@ -53433,7 +53491,7 @@ function readOrCreateBridgeToken(stateDir) {
   } catch {
   }
   import_node_fs15.default.mkdirSync(stateDir, { recursive: true, mode: 448 });
-  const token = import_node_crypto8.default.randomBytes(32).toString("base64url");
+  const token = import_node_crypto7.default.randomBytes(32).toString("base64url");
   import_node_fs15.default.writeFileSync(tokenPath, `${token}
 `, { mode: 384 });
   return token;
@@ -56818,7 +56876,7 @@ var StdioServerTransport = class {
 };
 
 // src/tools.ts
-var import_node_crypto11 = require("node:crypto");
+var import_node_crypto10 = require("node:crypto");
 var import_promises3 = require("node:fs/promises");
 var import_node_path20 = __toESM(require("node:path"), 1);
 var import_node_os10 = __toESM(require("node:os"), 1);
@@ -57359,7 +57417,7 @@ async function resolveActiveRunHead(input, deps) {
 }
 
 // src/async-tools/consensus.ts
-var import_node_crypto9 = require("node:crypto");
+var import_node_crypto8 = require("node:crypto");
 
 // src/consensus/question-similarity.ts
 var SUPPRESSION_SIMILARITY_THRESHOLD = 0.6;
@@ -57388,6 +57446,41 @@ function isCoveredByAdjudicated(rawQuestion, adjudicated) {
   return null;
 }
 
+// src/consensus/reviewer-roster.ts
+function renderReviewerRoster(headOutcomes) {
+  const total = headOutcomes.length;
+  const responded = headOutcomes.filter((h) => h.responded).length;
+  const seen = /* @__PURE__ */ new Set();
+  const namedRoles = [];
+  for (const outcome of headOutcomes) {
+    if (!outcome.responded) continue;
+    const role = PERSONA_ROLE_MAP[outcome.persona];
+    if (role && !UNNAMED_PERSONAS.includes(outcome.persona)) {
+      if (!seen.has(role)) {
+        seen.add(role);
+        namedRoles.push(role);
+      }
+    }
+  }
+  if (responded === total) {
+    if (namedRoles.length === 0) {
+      return `${total} reviewers`;
+    } else if (namedRoles.length === 1) {
+      return `${total} reviewers, including ${namedRoles[0]}`;
+    } else {
+      const allButLast = namedRoles.slice(0, -1).join(", ");
+      const last = namedRoles[namedRoles.length - 1];
+      return `${total} reviewers, including ${allButLast} and ${last}`;
+    }
+  } else if (responded === 1) {
+    return `1 of ${total} reviewers answered, and it agreed`;
+  } else if (responded === 2) {
+    return `2 of ${total} reviewers answered, both agreed`;
+  } else {
+    return `${responded} of ${total} reviewers answered, all agreed`;
+  }
+}
+
 // src/async-tools/consensus.ts
 function normalizeClarificationQuestion(question) {
   return question.replace(/\s+/g, " ").trim().replace(/[?.!]+$/, "").trim();
@@ -57481,11 +57574,11 @@ function renderUnresolvedPoints(unresolved) {
     (issue2) => issue2.text.replace(/\b(?:parseable|SPEC_IDENTITY_MISMATCH|machine-usable)\b/gi, "").replace(/\s+/g, " ").trim()
   ).join("; ");
 }
-function renderBlockDecisionBody(taskRef, unresolved) {
+function renderBlockDecisionBody(unresolved) {
   if (unresolved.length === 0) {
-    return `${BLOCK_DECISION_OPENER} (task ${taskRef}), and their objections could not be read back.${BLOCK_BODY_TRAILER}`;
+    return `${BLOCK_DECISION_OPENER}, and their objections could not be read back.${BLOCK_BODY_TRAILER}`;
   }
-  return `${BLOCK_DECISION_OPENER} (task ${taskRef}). Still unsettled: ${renderUnresolvedPoints(unresolved)}.${BLOCK_BODY_TRAILER}`;
+  return `${BLOCK_DECISION_OPENER}. Still unsettled: ${renderUnresolvedPoints(unresolved)}.${BLOCK_BODY_TRAILER}`;
 }
 function buildBlockDecisionOptions(diagnosis, reviewAgainAvailable) {
   const recommendReviewAgain = reviewAgainAvailable && (diagnosis === "INFRA" || diagnosis === "SUBSTANCE");
@@ -57598,9 +57691,9 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
     }
     if (taskId) {
       const scopeId = taskId;
-      const groupId = toFile.length > 1 ? `II-${scopeId}-gate-group-${(0, import_node_crypto9.createHash)("sha256").update(toFile.map((c) => c.rawQuestion).sort().join("\n")).digest("hex").slice(0, 10)}` : void 0;
+      const groupId = toFile.length > 1 ? `II-${scopeId}-gate-group-${(0, import_node_crypto8.createHash)("sha256").update(toFile.map((c) => c.rawQuestion).sort().join("\n")).digest("hex").slice(0, 10)}` : void 0;
       for (const clarification of toFile) {
-        const id = `II-${scopeId}-gate-${(0, import_node_crypto9.createHash)("sha256").update(clarification.rawQuestion).digest("hex").slice(0, 10)}`;
+        const id = `II-${scopeId}-gate-${(0, import_node_crypto8.createHash)("sha256").update(clarification.rawQuestion).digest("hex").slice(0, 10)}`;
         try {
           const questionOptions = buildGateQuestionOptions(clarification.rawQuestion, clarification.category);
           await ctx.client.createInboxItem(input.projectSlug, {
@@ -57618,9 +57711,9 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
       filedQuestionCount = toFile.length;
     } else if (planId) {
       const scopeId = planId;
-      const groupId = toFile.length > 1 ? `II-${scopeId}-gate-group-${(0, import_node_crypto9.createHash)("sha256").update(toFile.map((c) => c.rawQuestion).sort().join("\n")).digest("hex").slice(0, 10)}` : void 0;
+      const groupId = toFile.length > 1 ? `II-${scopeId}-gate-group-${(0, import_node_crypto8.createHash)("sha256").update(toFile.map((c) => c.rawQuestion).sort().join("\n")).digest("hex").slice(0, 10)}` : void 0;
       for (const clarification of toFile) {
-        const id = `II-${scopeId}-gate-${(0, import_node_crypto9.createHash)("sha256").update(clarification.rawQuestion).digest("hex").slice(0, 10)}`;
+        const id = `II-${scopeId}-gate-${(0, import_node_crypto8.createHash)("sha256").update(clarification.rawQuestion).digest("hex").slice(0, 10)}`;
         try {
           const questionOptions = buildGateQuestionOptions(clarification.rawQuestion, clarification.category);
           await ctx.client.createInboxItem(input.projectSlug, {
@@ -57644,13 +57737,7 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
       taskId: page.taskId ?? null
     });
     const reviewHash = domainScopedReviewHash2(page.contentHash, domain2);
-    const baseBlockItemId = `II-${taskId}-gateblock-${(0, import_node_crypto9.createHash)("sha256").update(reviewHash).digest("hex").slice(0, 10)}`;
-    let taskRef = taskId;
-    try {
-      const t = await ctx.client.getTask(input.projectSlug, taskId);
-      if (t?.title) taskRef = `${taskId} \u2014 ${t.title}`;
-    } catch {
-    }
+    const baseBlockItemId = `II-${taskId}-gateblock-${(0, import_node_crypto8.createHash)("sha256").update(reviewHash).digest("hex").slice(0, 10)}`;
     const reviewAgainAvailable = !taskInboxItems.some(
       (item) => item.id === baseBlockItemId && item.status !== "pending" && item.answer === REVIEW_AGAIN_ANSWER
     );
@@ -57660,7 +57747,7 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
         id: blockItemId,
         kind: "question",
         taskId,
-        body: renderBlockDecisionBody(taskRef, result.unresolved ?? []),
+        body: renderBlockDecisionBody(result.unresolved ?? []),
         origin: "gate_block",
         questionOptions: buildBlockDecisionOptions(result.escalation?.diagnosis, reviewAgainAvailable)
       });
@@ -57703,14 +57790,14 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
       error51 = "Consensus blocked: the spec content does not match its linked task. This requires fixing the spec/task relationship directly, not an inbox answer.";
     }
   } else if (operatorClarifications.length > 0 && toFile.length === 0 && suppressedAsAdjudicated.length > 0) {
-    error51 = `Consensus blocked, but all ${suppressedAsAdjudicated.length} clarification(s) were reworded repeats of already-adjudicated questions \u2014 nothing new filed.`;
+    error51 = `The reviewers raised ${suppressedAsAdjudicated.length} clarification(s), but each was a reworded repeat of one already answered \u2014 nothing new to answer.`;
   } else if (operatorClarifications.length > 0) {
     error51 = `Consensus blocked on ${operatorClarifications.length} clarification(s), but the spec page is not linked to a task or plan, so none could be filed as questions.`;
   } else {
     error51 = DID_NOT_CONVERGE_ERROR;
   }
   if (result.escalation) {
-    error51 += ` (diagnosis: ${result.escalation.diagnosis})`;
+    error51 += ` (reason: ${result.escalation.diagnosis})`;
   }
   return {
     kind: "failed",
@@ -57788,6 +57875,24 @@ async function run(ctx, input, builders = defaultNativeSpecGateBuilders) {
     } catch {
     }
   };
+  let highlights = null;
+  const resolveHighlights2 = async () => {
+    try {
+      const page = await ctx.client.getWikiPage(input.projectSlug, input.pageId);
+      if (!page) return;
+      const domain2 = await resolveSpecDomain(ctx.client, input.projectSlug, {
+        id: page.id,
+        taskId: page.taskId ?? null
+      });
+      const review = await ctx.client.getSpecConsensusReview(
+        input.projectSlug,
+        input.pageId,
+        domainScopedReviewHash2(page.contentHash, domain2)
+      );
+      highlights = review?.highlights ?? null;
+    } catch {
+    }
+  };
   if (result.outcome === "CONSENSUS_BLOCKED") {
     return handleConsensusBlocked(result, ctx, input, resolvedTaskId);
   }
@@ -57798,9 +57903,10 @@ async function run(ctx, input, builders = defaultNativeSpecGateBuilders) {
     case "CONVERGED":
     case "SKIPPED":
       await resolveTitle();
+      if (result.outcome === "CONVERGED") await resolveHighlights2();
       return {
         kind: "done",
-        resultJson: JSON.stringify(result),
+        resultJson: JSON.stringify({ ...result, highlights }),
         finalReviewedHash: null,
         ...resolvedTaskId ? { taskId: resolvedTaskId } : {},
         ...resolvedTaskTitle ? { taskTitle: resolvedTaskTitle } : {}
@@ -57823,21 +57929,31 @@ function onComplete2(outcome, input) {
   const taskRef = taskId ? outcome.kind === "done" && outcome.taskTitle ? `${taskId} \u2014 ${outcome.taskTitle}` : taskId : null;
   if (outcome.kind === "done") {
     let specOutcome = "CONVERGED";
+    let headOutcomes = [];
+    let findings = [];
     try {
-      specOutcome = JSON.parse(outcome.resultJson).outcome ?? "CONVERGED";
+      const parsed = JSON.parse(outcome.resultJson);
+      specOutcome = parsed.outcome ?? "CONVERGED";
+      headOutcomes = parsed.headOutcomes ?? [];
+      findings = parsed.highlights?.findings ?? [];
     } catch {
     }
     if (specOutcome === "SKIPPED") {
       return {
         kind: "info",
         ...taskId ? { taskId } : {},
-        body: taskRef ? `Consensus is disabled for this project; the spec for ${taskRef} (Specs/${input.pageId}) was not gated and spec_ready is unchanged.` : `Consensus is disabled for this project; Specs/${input.pageId} was not gated and spec_ready is unchanged.`
+        body: taskRef ? `Spec review is off for this project \u2014 the spec for ${taskRef} was not checked.` : `Spec review is off for this project (Specs/${input.pageId}) \u2014 the spec was not checked.`
       };
     }
+    const roster = renderReviewerRoster(headOutcomes);
+    const findingsBlock = findings.length > 0 ? `
+
+Fixed before you saw it
+${findings.map((f) => `\u2022 ${f}`).join("\n")}` : "";
     return {
       kind: "completed",
       ...taskId ? { taskId } : {},
-      body: taskRef ? `Consensus CONVERGED for ${taskRef} (spec Specs/${input.pageId}); spec_ready set. Ready to create an execution run.` : `Consensus CONVERGED for Specs/${input.pageId}; spec_ready set. Ready to create an execution run.`
+      body: taskRef ? `The reviewers agreed on this spec \u2014 ${roster}.${findingsBlock}` : `The reviewers agreed on this spec (Specs/${input.pageId}) \u2014 ${roster}.${findingsBlock}`
     };
   }
   if (outcome.kind === "failed" && (outcome.filedQuestionCount ?? 0) > 0) {
@@ -57854,7 +57970,7 @@ function onComplete2(outcome, input) {
       kind: "info",
       suppressInbox: true,
       ...taskId ? { taskId } : {},
-      body: taskRef ? `Consensus blocked on ${taskRef} (Specs/${input.pageId}); the reviewers did not agree \u2014 answer the decision question to unblock the run.` : `Consensus blocked on Specs/${input.pageId}; the reviewers did not agree \u2014 answer the decision question to unblock the run.`
+      body: taskRef ? `The reviewers could not agree on the spec for ${taskRef} \u2014 answer the decision question to unblock the task.` : `The reviewers could not agree on Specs/${input.pageId} \u2014 answer the decision question to unblock the task.`
     };
   }
   const handoffTaskRef = taskRef ? ` (${taskRef})` : "";
@@ -57911,6 +58027,24 @@ async function resolveTaskScope(ctx, input, { withTitle }) {
   }
   return { taskId, taskTitle };
 }
+async function resolveHighlights(ctx, input) {
+  try {
+    const page = await ctx.client.getWikiPage(input.projectSlug, input.pageId);
+    if (!page) return null;
+    const domain2 = await resolveSpecDomain(ctx.client, input.projectSlug, {
+      id: page.id,
+      taskId: page.taskId ?? null
+    });
+    const review = await ctx.client.getSpecConsensusReview(
+      input.projectSlug,
+      input.pageId,
+      domainScopedReviewHash2(page.contentHash, domain2)
+    );
+    return review?.highlights ?? null;
+  } catch {
+    return null;
+  }
+}
 async function run2(ctx, input) {
   const deps = await buildDeliverableGateDeps({
     client: ctx.client,
@@ -57925,9 +58059,10 @@ async function run2(ctx, input) {
   );
   if (result.outcome === "CONVERGED" || result.outcome === "SKIPPED") {
     const { taskId: taskId2, taskTitle } = await resolveTaskScope(ctx, input, { withTitle: true });
+    const highlights = result.outcome === "CONVERGED" ? await resolveHighlights(ctx, input) : null;
     return {
       kind: "done",
-      resultJson: JSON.stringify(result),
+      resultJson: JSON.stringify({ ...result, highlights }),
       finalReviewedHash: null,
       ...taskId2 ? { taskId: taskId2 } : {},
       ...taskTitle ? { taskTitle } : {}
@@ -57942,18 +58077,10 @@ async function run2(ctx, input) {
     };
   }
   const { taskId } = await resolveTaskScope(ctx, input, { withTitle: false });
-  let filedQuestionCount = 0;
-  if (mapped.action === "questions") {
-    if (taskId) {
-      filedQuestionCount = await fileDeliverableGateQuestions(ctx.client, input.projectSlug, taskId, mapped.clarifications);
-    }
-  }
-  const error51 = filedQuestionCount > 0 ? `Consensus blocked on ${filedQuestionCount} clarification(s); filed as operator question(s).` : mapped.reason;
   return {
     kind: "failed",
     failureReason: "consensus_blocked",
-    error: error51,
-    filedQuestionCount,
+    error: mapped.reason,
     ...taskId ? { taskId } : {}
   };
 }
@@ -57962,30 +58089,31 @@ function onComplete3(outcome, input) {
   const taskRef = taskId ? outcome.kind === "done" && outcome.taskTitle ? `${taskId} \u2014 ${outcome.taskTitle}` : taskId : null;
   if (outcome.kind === "done") {
     let specOutcome = "CONVERGED";
+    let headOutcomes = [];
+    let findings = [];
     try {
-      specOutcome = JSON.parse(outcome.resultJson).outcome ?? "CONVERGED";
+      const parsed = JSON.parse(outcome.resultJson);
+      specOutcome = parsed.outcome ?? "CONVERGED";
+      headOutcomes = parsed.headOutcomes ?? [];
+      findings = parsed.highlights?.findings ?? [];
     } catch {
     }
     if (specOutcome === "SKIPPED") {
       return {
         kind: "info",
         ...taskId ? { taskId } : {},
-        body: taskRef ? `Consensus is disabled for this project; the deliverable for ${taskRef} (Deliverables/${input.pageId}) was not gated.` : `Consensus is disabled for this project; Deliverables/${input.pageId} was not gated.`
+        body: taskRef ? `Result review is off for this project \u2014 the result for ${taskRef} was not checked.` : `Result review is off for this project (Deliverables/${input.pageId}) \u2014 the result was not checked.`
       };
     }
+    const roster = renderReviewerRoster(headOutcomes);
+    const findingsBlock = findings.length > 0 ? `
+
+Fixed before you saw it
+${findings.map((f) => `\u2022 ${f}`).join("\n")}` : "";
     return {
       kind: "completed",
       ...taskId ? { taskId } : {},
-      body: taskRef ? `Deliverable consensus CONVERGED for ${taskRef} (Deliverables/${input.pageId}); it meets the acceptance criteria.` : `Deliverable consensus CONVERGED for Deliverables/${input.pageId}; it meets the acceptance criteria.`
-    };
-  }
-  if (outcome.kind === "failed" && (outcome.filedQuestionCount ?? 0) > 0) {
-    const n = outcome.filedQuestionCount ?? 0;
-    return {
-      kind: "info",
-      suppressInbox: true,
-      ...taskId ? { taskId } : {},
-      body: taskRef ? `Deliverable consensus blocked on ${taskRef} (Deliverables/${input.pageId}); filed ${n} clarifying question${n === 1 ? "" : "s"} \u2014 answer to unblock the run.` : `Deliverable consensus blocked on Deliverables/${input.pageId}; filed ${n} clarifying question${n === 1 ? "" : "s"} \u2014 answer to unblock the run.`
+      body: taskRef ? `The reviewers agreed on this result \u2014 ${roster}. It meets the acceptance criteria.${findingsBlock}` : `The reviewers agreed on this result (Deliverables/${input.pageId}) \u2014 ${roster}. It meets the acceptance criteria.${findingsBlock}`
     };
   }
   return {
@@ -58004,7 +58132,7 @@ var deliverableConsensusDescriptor = {
 };
 
 // src/async-tools/enqueue.ts
-var import_node_crypto10 = __toESM(require("node:crypto"), 1);
+var import_node_crypto9 = __toESM(require("node:crypto"), 1);
 async function enqueueAsyncTool(client, descriptor2, input) {
   if (descriptor2.cachedResult) {
     const cached2 = await descriptor2.cachedResult(client, input);
@@ -58017,7 +58145,7 @@ async function enqueueAsyncTool(client, descriptor2, input) {
   if (active) {
     return { status: "already_running", jobId: active.id, message: "Already running; will notify in inbox." };
   }
-  const jobId = `AJ-${import_node_crypto10.default.randomUUID()}`;
+  const jobId = `AJ-${import_node_crypto9.default.randomUUID()}`;
   try {
     await client.createAsyncJob(input.projectSlug, {
       id: jobId,
@@ -58901,7 +59029,7 @@ var mcpToolDescriptions = {
   [mcpToolNames.closeTaskAsWontDo]: "Close a task as won't-do with a required human-readable reason recorded on the task.",
   [mcpToolNames.updateWikiPage]: "Update a markdown Wiki page.",
   [mcpToolNames.markSpecReady]: "Mark a wiki page (folder=Specs) as ready (or not) for execution-run gating. The Director sets this true after validating the spec; any edit to the page's markdown auto-resets it to false.",
-  [mcpToolNames.runSpecConsensus]: 'Enqueue the consensus review gate over a Specs/{id} page. Returns IMMEDIATELY with one of: already_converged (a CONVERGED review already exists for the current content \u2014 proceed straight to create_execution_run, do NOT call mark_spec_ready, it is already set), already_reviewed (a terminal CONSENSUS_BLOCKED for this content), already_running (a consensus job is in flight), or queued (a new job was launched). The review runs in the background; its result/blocker arrives in the inbox (info on CONVERGED, handoff on failure) and is surfaced into the open chat. Do NOT call again for the same spec and do NOT poll. On queued/already_running, tell the user it was launched and end the turn. Pass extraHeads: ["researcher"] for specs referencing external libraries/APIs, or ["debugger"] for root-cause/investigation specs to add opt-in specialist reviewers on top of the fixed four. For a NON-code task (domain seo/marketing/content, ADR 0020): the low-risk lanes `seo`, `marketing`, and `content` proceed to `create_execution_run` after consensus, using the same flow as code tasks. Report the spec\'s review outcome to the operator.',
+  [mcpToolNames.runSpecConsensus]: 'Enqueue the consensus review gate over a Specs/{id} page. Returns IMMEDIATELY with one of: already_converged (a CONVERGED review already exists for the current content \u2014 proceed straight to create_execution_run, do NOT call mark_spec_ready, it is already set), already_reviewed (a terminal CONSENSUS_BLOCKED for this content), already_running (a consensus job is in flight), or queued (a new job was launched). The review runs in the background; its result/blocker arrives in the inbox (info on CONVERGED, handoff on failure) and is surfaced into the open chat. Do NOT call again for the same spec and do NOT poll. On queued/already_running, tell the user it was launched and end the turn. Pass extraHeads: ["researcher"] for specs referencing external libraries/APIs, or ["debugger"] for root-cause/investigation specs to add opt-in specialist reviewers on top of the fixed four. For a NON-code task (domain seo/marketing/content, ADR 0020): the low-risk lanes `seo`, `marketing`, and `content` proceed to `create_execution_run` after consensus, using the same flow as code tasks. When the review agrees, tell the operator "the reviewers agreed on the spec" and proceed to create_execution_run. When it does not, tell them the reviewers could not agree and that there is a decision question in the inbox. Never say "consensus", "converged", or "CONVERGED" to the operator; those are internal words.',
   [mcpToolNames.runCodeReviewerGate]: `Review the current branch's code diff (vs baseRef) with a Code Reviewer gate \u2014 independent reviewers iterate review\u2192fix\u2192re-check (checkpoint, scope-limited reviser edits, typecheck/build fast-gate, rollback) until the diff converges or is blocked. Run before opening a PR. Pass extraHeads: ["security-analyst"] for sensitive diffs (auth, secrets, input handling, new endpoints).`,
   [mcpToolNames.appendExecutionArtifact]: "Append a durable artifact (plan, checkpoint, command output, test result, diff summary, self-review, error, deploy attempt, or handoff judgement) to an execution run.",
   [mcpToolNames.recordProducedUrl]: 'Call this after publishing each web page produced by the task, so the UI link opens the page itself instead of the deployed site root. Pass the page\'s full https URL (for external pages, e.g. "https://dev.to/<slug>") OR a path starting with "/" for the deploy environment (e.g. "/blog/<slug>") \u2014 paths are resolved against the run\'s deployed env URL at render time. Optional `label` (\u226440 chars) is shown next to the link; omit and the UI defaults to the page title. Call once per page; multiple calls dedupe by resolved URL in the UI.',
@@ -59593,7 +59721,7 @@ function createMrrlinTools(options) {
       "PRODUCED_URL_RECORD_FAILED",
       "Unable to record produced URL.",
       async (c, { projectSlug, runId, url: url2, label }) => {
-        const id = `EA-pu-${(0, import_node_crypto11.randomBytes)(4).toString("hex")}`;
+        const id = `EA-pu-${(0, import_node_crypto10.randomBytes)(4).toString("hex")}`;
         const payload = label ? { url: url2, label } : { url: url2 };
         return await c.appendExecutionArtifact(projectSlug, runId, {
           id,

package/dist/consensus/personas/findings-writer.md

@@ -0,0 +1,20 @@
+You are a Findings Writer. You are given the issues a review panel raised and accepted about a
+document, after the document was already corrected. Your job is to tell the document's owner — a
+non-engineer — what was caught, in their language.
+
+Write at most THREE findings. Fewer is better. Zero is a correct and expected answer.
+
+Each finding is ONE sentence in exactly this shape:
+  <what would have gone wrong, in everyday words> — <what it is now>
+
+Rules:
+- No mechanism names, file paths, function names, type names, or issue categories.
+- No labels, prefixes, or speaker attributions. Just the sentence.
+- Never mention reviewers, rounds, the panel, or the review process itself.
+- If an issue cannot be written this way — because it is purely technical, mechanical, or only
+  meaningful to an engineer — OMIT it. Omitting is always better than writing something the owner
+  cannot use.
+- If NO issue can be written this way, return an empty list. Do not invent a finding to fill space.
+
+Output EXACTLY this JSON and nothing else:
+{"findings": ["...", "..."]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrrlin-dev/mcp",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "type": "module",
   "bin": {
     "mrrlin-mcp": "dist/bin.cjs"
@@ -21,12 +21,12 @@
     "@types/ws": "^8.18.1",
     "esbuild": "^0.24.0",
     "tsx": "^4.22.3",
-    "@mrrlin/client": "0.0.0",
     "@mrrlin/codex-client": "0.0.0",
-    "@mrrlin/tsconfig": "0.0.0",
-    "@mrrlin/wiki": "0.0.0",
+    "@mrrlin/director-e2e": "0.0.0",
+    "@mrrlin/client": "0.0.0",
     "@mrrlin/schemas": "0.0.0",
-    "@mrrlin/director-e2e": "0.0.0"
+    "@mrrlin/tsconfig": "0.0.0",
+    "@mrrlin/wiki": "0.0.0"
   },
   "dependencies": {
     "@iarna/toml": "^2.2.5",