@mrkaran/hodor 0.7.5 → 0.7.7

package/dist/{chunk-AFEJ4DRL.js → chunk-PXII2GLB.js}

@@ -111,24 +111,48 @@ function renderSummaryMarkdown(review, options = {}) {
     else if (finding.priority === 2) counts.important++;
     else counts.minor++;
   }
-  lines.push("");
+  const totalOpen = counts.blocking + counts.important + counts.minor;
   lines.push(
-    `**Open findings:** ${counts.blocking} blocking \xB7 ${counts.important} important \xB7 ${counts.minor} minor`
+    "",
+    "| Open findings | Count |",
+    "| --- | ---: |",
+    `| Critical (P0/P1) | ${counts.blocking} |`,
+    `| Important (P2) | ${counts.important} |`,
+    `| Minor (P3) | ${counts.minor} |`,
+    ""
   );
-  if (options.inlineCreated != null || options.inlineDeduplicated != null) {
+  const verdict = totalOpen === 0 ? "No open findings" : counts.blocking > 0 ? "Blocking findings remain" : "Non-blocking findings remain";
+  lines.push(`**Overall verdict:** ${verdict}`);
+  const scope = options.reviewMode ? `${options.reviewMode[0].toUpperCase()}${options.reviewMode.slice(1)} review` : "Latest review";
+  if (review.overall_explanation) {
     lines.push("");
-    lines.push(
-      `**Inline delivery:** ${options.inlineCreated ?? 0} new \xB7 ${options.inlineDeduplicated ?? 0} already open`
-    );
+    lines.push(`**${scope}:** ${review.overall_explanation}`);
+  }
+  const inlineCreated = options.inlineCreated ?? 0;
+  const inlineDeduplicated = options.inlineDeduplicated ?? 0;
+  if (inlineCreated > 0 || inlineDeduplicated > 0) {
+    const delivery = [
+      inlineCreated > 0 ? `${inlineCreated} new` : null,
+      inlineDeduplicated > 0 ? `${inlineDeduplicated} already open` : null
+    ].filter((item) => item !== null);
+    lines.push("");
+    lines.push(`**Inline comments:** ${delivery.join(" \xB7 ")}`);
   }
-  const scope = options.reviewMode ? `${options.reviewMode} review` : "Latest review";
-  lines.push("");
-  lines.push(`**${scope}:** ${review.overall_explanation}`);
   if (fallbackFindings.length > 0) {
-    lines.push("");
-    lines.push(`### ${options.fallbackHeading ?? "Findings"}`);
+    lines.push(
+      "",
+      `### ${options.fallbackHeading ?? "Findings"}`,
+      "",
+      "| Finding | Location | Priority |",
+      "| --- | --- | ---: |"
+    );
     for (const finding of fallbackFindings) {
-      lines.push(formatFinding(finding));
+      const title = escapeTableCell(finding.title);
+      const body = escapeTableCell(finding.body);
+      const location = escapeTableCell(formatLocation(finding.code_location));
+      lines.push(
+        `| **${title}**<br>${body} | \`${location}\` | P${finding.priority} |`
+      );
     }
   }
   return lines.join("\n").trimEnd() + "\n";
@@ -140,6 +164,9 @@ function formatFinding(f) {
   return `${title}
 ${body}`;
 }
+function escapeTableCell(value) {
+  return value.replace(/\r?\n/g, " ").replace(/\|/g, "\\|");
+}
 function formatLocation(loc) {
   let filePath = loc.absolute_file_path;
   const buildsMatch = filePath.match(/\/builds\/[^/]+\/[^/]+\/(.+)/);
@@ -1279,22 +1306,19 @@ function formatDuration(seconds) {
 }
 function formatMetricsMarkdown(metrics) {
   const totalInput = metrics.inputTokens + metrics.cacheReadTokens;
-  const parts = [`in \`${tok(totalInput)}\``, `fresh \`${tok(metrics.inputTokens)}\``];
+  const work = `${metrics.turns} turns \xB7 ${metrics.toolCalls} tool calls \xB7 ${formatDuration(metrics.durationSeconds)}`;
+  const cost = metrics.cost > 0 ? ` \xB7 \`$${metrics.cost.toFixed(4)}\`` : "";
+  const tokens = [`\`${tok(totalInput)}\` input`];
   if (metrics.cacheReadTokens > 0) {
-    parts.push(`cache read \`${tok(metrics.cacheReadTokens)}\``);
+    const hitPct = (metrics.cacheReadTokens / totalInput * 100).toFixed(0);
+    tokens.push(`\`${tok(metrics.cacheReadTokens)}\` cached (${hitPct}%)`);
   }
+  tokens.push(`\`${tok(metrics.outputTokens)}\` output`);
   if (metrics.cacheWriteTokens > 0) {
-    parts.push(`cache write \`${tok(metrics.cacheWriteTokens)}\``);
+    tokens.push(`\`${tok(metrics.cacheWriteTokens)}\` cache write`);
   }
-  parts.push(`out \`${tok(metrics.outputTokens)}\``);
-  const lines = [
-    `**Review Metrics** \u2014 ${metrics.turns} turns, ${metrics.toolCalls} tool calls, ${formatDuration(metrics.durationSeconds)}`,
-    `- Tokens: ${parts.join(" | ")} (total \`${tok(metrics.totalTokens)}\`)`
-  ];
-  if (metrics.cost > 0) {
-    lines.push(`- Cost: \`$${metrics.cost.toFixed(4)}\``);
-  }
-  return lines.join("\n");
+  return `**Review metrics:** ${work}${cost}
+- Tokens: ${tokens.join(" \xB7 ")}`;
 }
 function printMetrics(metrics, stream = process.stderr) {
   const dim = chalk2.dim;
@@ -1944,7 +1968,7 @@ async function postGitlabReviewCommitStatus(parsed, findings, diffRefs) {
 function appendReviewDetails(body, model, metricsFooter) {
   if (!model && !metricsFooter) return body;
   const details = ["<details>", "<summary>Review details</summary>", ""];
-  if (model) details.push(`- Model: \`${model}\``);
+  if (model) details.push(`- Model: \`${displayModel(model)}\``);
   if (metricsFooter) {
     if (model) details.push("");
     details.push(metricsFooter);
@@ -1955,6 +1979,10 @@ function appendReviewDetails(body, model, metricsFooter) {
 ${details.join("\n")}
 `;
 }
+function displayModel(model) {
+  const baseModel = model.slice(model.lastIndexOf("@") + 1);
+  return baseModel.startsWith("arn:") ? baseModel.slice(baseModel.lastIndexOf("/") + 1) : baseModel;
+}
 async function postReviewComment(opts) {
   const { prUrl, reviewText, model, metricsFooter, headSha, cacheMarker } = opts;
   const platform = detectPlatform(prUrl);
@@ -2205,11 +2233,11 @@ ${finding.suggestion}
   }
   let summaryPosted = false;
   if (!skipSummary && (reviewStyle === "summary" || reviewStyle === "hybrid" || review.findings.length === 0 || failedFindings.length > 0)) {
-    const fallbackFindings = reviewStyle === "summary" ? review.findings : failedFindings;
+    const summaryFindings = review.findings;
     let summaryBody = renderSummaryMarkdown(review, {
       openFindings: reviewFindings,
-      fallbackFindings,
-      fallbackHeading: reviewStyle === "summary" ? "Findings" : "Findings not posted inline",
+      fallbackFindings: summaryFindings,
+      fallbackHeading: reviewMode === "incremental" ? "Findings from this review" : "Findings",
       inlineCreated: reviewStyle === "summary" ? void 0 : inlineCreated,
       inlineDeduplicated: reviewStyle === "summary" ? void 0 : inlineDeduplicated,
       reviewMode
@@ -3767,4 +3795,4 @@ export {
   postReviewStructured,
   reviewPr
 };
-//# sourceMappingURL=chunk-AFEJ4DRL.js.map
+//# sourceMappingURL=chunk-PXII2GLB.js.map