@mrkaran/hodor 0.6.3-rc.2 → 0.6.3-rc.3

package/README.md

@@ -233,9 +233,14 @@ See [AUTOMATED_REVIEWS.md](./docs/AUTOMATED_REVIEWS.md) for advanced workflows.
 Hodor automatically optimizes token usage:
 
 - **Diff embedding**: For PRs under 200KB, the diff is embedded directly in the prompt, cutting agent turns from ~60 to ~5.
-- **Incremental reviews**: On re-runs, only reviews changes since the last hodor comment (detected via SHA markers in posted comments). Pass `--full` to override this and re-review the entire source-vs-target diff from scratch.
+- **Incremental reviews**: On re-runs, only reviews changes since the last hodor comment. After a force-push or rebase, Hodor compares the last reviewed snapshot directly with the current HEAD instead of reviewing the whole MR again.
+- **Identical-HEAD reuse**: Successful summaries include a versioned, compressed review payload. Pipeline retries with the same HEAD, model, reasoning request, and prompt configuration reuse that result while still regenerating artifacts and retrying delivery.
+- **Adaptive reasoning**: Models that default to `xhigh` use `high` for routine small and incremental diffs, while risky, large, explicitly configured, and `--full` reviews retain the requested depth.
+- **Focused exploration**: Embedded diffs include a changed-file manifest and direct the agent toward bounded context reads without limiting how far it may investigate.
 - **Compaction**: SDK auto-summarizes older conversation turns when context grows too large.
 
+Pass `--full` to bypass incremental mode and identical-HEAD reuse. Pass `--reasoning-effort` to override adaptive reasoning.
+
 ## Skills
 
 Hodor discovers repository-specific review guidelines from `.agents/skills/`, the cross-client Agent Skills convention:

package/dist/{chunk-ZUYM6HY3.js → chunk-N7TSXVUE.js}

@@ -15,7 +15,7 @@ import {
   resolveGitlabDiscussions,
   summarizeGitlabNotes,
   summarizeHodorNotes
-} from "./chunk-LO2QJD45.js";
+} from "./chunk-W7ZUJIFT.js";
 import {
   relativizeWorkspacePath
 } from "./chunk-AMUK6GDX.js";
@@ -39,6 +39,8 @@ function buildPrReviewPrompt(opts) {
     customPromptFile,
     embeddedDiff,
     previousReviewSha,
+    reviewDiffMode,
+    changedFiles = [],
     localMode = false
   } = opts;
   let templateFile;
@@ -68,9 +70,10 @@ function buildPrReviewPrompt(opts) {
   let prDiffCmd;
   let gitDiffCmd;
   if (previousReviewSha) {
-    prDiffCmd = `git --no-pager diff ${previousReviewSha}...HEAD --name-only`;
-    gitDiffCmd = `git --no-pager diff ${previousReviewSha}...HEAD`;
-    logger.info(`Incremental review: diffing from ${previousReviewSha.slice(0, 8)} to HEAD`);
+    const separator = reviewDiffMode === "snapshot" ? " " : "...";
+    prDiffCmd = `git --no-pager diff ${previousReviewSha}${separator}HEAD --name-only`;
+    gitDiffCmd = `git --no-pager diff ${previousReviewSha}${separator}HEAD`;
+    logger.info(`${reviewDiffMode === "snapshot" ? "Snapshot" : "Incremental"} review: diffing from ${previousReviewSha.slice(0, 8)} to HEAD`);
   } else if (localMode) {
     prDiffCmd = `git --no-pager diff ${targetBranch} --name-only`;
     gitDiffCmd = `git --no-pager diff ${targetBranch}`;
@@ -89,7 +92,7 @@ function buildPrReviewPrompt(opts) {
   }
   let diffExplanation;
   if (previousReviewSha) {
-    diffExplanation = `**Incremental mode**: Showing only changes since the last hodor review (commit \`${previousReviewSha.slice(0, 8)}\`).`;
+    diffExplanation = reviewDiffMode === "snapshot" ? `**Snapshot delta mode**: The MR history was rewritten. This directly compares the last reviewed snapshot (commit \`${previousReviewSha.slice(0, 8)}\`) with the current HEAD; it does not imply ancestry.` : `**Incremental mode**: Showing only changes since the last hodor review (commit \`${previousReviewSha.slice(0, 8)}\`).`;
   } else if (diffBaseSha) {
     diffExplanation = `**GitLab CI Advantage**: This uses GitLab's pre-calculated merge base SHA (\`CI_MERGE_REQUEST_DIFF_BASE_SHA\`), which matches exactly what the GitLab UI shows. This is more reliable than three-dot syntax because it handles force pushes, rebases, and messy histories correctly.`;
   } else {
@@ -98,25 +101,20 @@ function buildPrReviewPrompt(opts) {
   const { contextSection, notesSection, reminderSection } = buildMrSections(mrMetadata);
   let incrementalSection = "";
   if (previousReviewSha) {
-    incrementalSection = `## Incremental Review Mode
+    incrementalSection = `## ${reviewDiffMode === "snapshot" ? "Snapshot Delta" : "Incremental Review"} Mode
 
-This is a follow-up review. A previous hodor review was done at commit \`${previousReviewSha.slice(0, 8)}\`. The diff below shows ONLY changes since that review. Your job is to review that delta, not the whole MR again.
-
-Rules for incremental reviews:
-1. Only report bugs introduced or still affected by the new delta.
-2. Do not re-report issues that are already mentioned in existing notes unless the new delta changes the same code and the issue remains newly relevant.
-3. If the delta is small and self-contained, decide from the embedded diff and submit the review without broad repository exploration.
-4. For mechanical changes like route/path/string renames, verify the direct call sites or tests only when the diff itself leaves a concrete compatibility question.
-5. If the delta does not introduce a production bug, submit no findings.
-
-`;
+This is a follow-up review. A previous hodor review was done at commit \`${previousReviewSha.slice(0, 8)}\`. ` + (reviewDiffMode === "snapshot" ? "The branch history was rewritten, so the diff below compares that reviewed snapshot directly with the current HEAD. " : "The diff below shows ONLY changes since that review. ") + "Your job is to review that delta, not the whole MR again.\n\nRules for incremental reviews:\n1. Only report bugs introduced or still affected by the new delta.\n2. Do not re-report issues that are already mentioned in existing notes unless the new delta changes the same code and the issue remains newly relevant.\n3. If the delta is small and self-contained, decide from the embedded diff and submit the review without broad repository exploration.\n4. For mechanical changes like route/path/string renames, verify the direct call sites or tests only when the diff itself leaves a concrete compatibility question.\n5. If the delta does not introduce a production bug, submit no findings.\n\n";
   }
   let embeddedDiffSection;
   let diffFetchInstructions;
   let reviewProcessSection;
   let startInstruction;
   if (embeddedDiff) {
-    embeddedDiffSection = "## Full Diff (Pre-fetched)\n\nThe complete diff for this PR is provided below. Analyze it directly. Use `read` or `grep` only if you need additional file context beyond what the diff shows.\n\n````diff\n" + embeddedDiff + "\n````\n";
+    const changedFileManifest = changedFiles.length > 0 ? `
+Changed files (${changedFiles.length}):
+${changedFiles.map((file) => `- \`${file}\``).join("\n")}
+` : "";
+    embeddedDiffSection = "## Full Diff (Pre-fetched)\n\nThe complete diff for this PR is provided below. Analyze it directly. Do not run another command to list changed files. Use `read` or `grep` only if you need additional file context beyond what the diff shows.\n" + changedFileManifest + "\n````diff\n" + embeddedDiff + "\n````\n";
     diffFetchInstructions = `## Review the Diff Above
 
 ### Critical Rules
@@ -127,7 +125,7 @@ Rules for incremental reviews:
 - NEVER flag "dependency version downgrade" (branch not rebased)
 - NEVER compare entire codebase to ${targetBranch} - DIFF ONLY
 `;
-    reviewProcessSection = "## Review Process\n\n1. Analyze the embedded diff above thoroughly\n2. Use `grep` to search for patterns when needed\n3. Use `read` only when surrounding context is essential\n4. Submit your review using `submit_review`\n";
+    reviewProcessSection = "## Review Process\n\n1. Analyze the embedded diff above thoroughly\n2. Use `grep` to search for patterns when needed\n3. Use bounded line-range reads when surrounding context is essential; avoid reading entire large files\n4. Do not repeat a diff, grep, or read operation whose result is already in context\n5. Submit your review using `submit_review`\n";
     startInstruction = previousReviewSha ? "Analyze only the incremental diff provided above. If it is self-contained, submit your review without extra tool calls." : "Analyze the diff provided above, then submit your review using `submit_review`.";
   } else {
     embeddedDiffSection = "";
@@ -329,8 +327,25 @@ function normalizeModelMatchValue(value) {
 }
 function getDefaultReasoningEffortForModel(model) {
   const values = [model.id, model.name].filter((value) => Boolean(value));
-  const isOpus47 = values.map(normalizeModelMatchValue).some((value) => value.includes("opus-4-7"));
-  return isOpus47 ? "xhigh" : void 0;
+  const isAdaptiveOpus = values.map(normalizeModelMatchValue).some((value) => value.includes("opus-4-7") || value.includes("opus-4-8"));
+  return isAdaptiveOpus ? "xhigh" : void 0;
+}
+var HIGH_RISK_PATH_RE = /(?:^|\/)(?:migrations?|schema|auth|security|permissions?|crypto|iam)(?:\/|\.|$)|\.tf$/im;
+var HIGH_RISK_CHANGE_RE = /^\+.*\b(?:authorization|authentication|permission|transaction|mutex|semaphore|encrypt|decrypt|credential|secret)\b/im;
+function selectReasoningEffort(opts) {
+  const requested = mapReasoningEffort(opts.requested);
+  if (requested) return requested;
+  const modelDefault = opts.modelDefault;
+  if (modelDefault !== "xhigh") return modelDefault;
+  if (opts.forcedFull) return modelDefault;
+  const risky = Boolean(
+    opts.diff && (HIGH_RISK_PATH_RE.test(opts.diff) || HIGH_RISK_CHANGE_RE.test(opts.diff))
+  );
+  if (risky) return modelDefault;
+  if (opts.mode === "incremental" || opts.mode === "snapshot") return "high";
+  const changedLines = (opts.stats?.additions ?? 0) + (opts.stats?.deletions ?? 0);
+  if (opts.stats && opts.stats.files <= 10 && changedLines <= 500) return "high";
+  return modelDefault;
 }
 function getApiKey(model) {
   const llmKey = process.env.LLM_API_KEY;
@@ -363,9 +378,12 @@ function formatDuration(seconds) {
 }
 function formatMetricsMarkdown(metrics) {
   const totalInput = metrics.inputTokens + metrics.cacheReadTokens;
-  const parts = [`in \`${tok(totalInput)}\``];
+  const parts = [`in \`${tok(totalInput)}\``, `fresh \`${tok(metrics.inputTokens)}\``];
   if (metrics.cacheReadTokens > 0) {
-    parts.push(`cached \`${tok(metrics.cacheReadTokens)}\``);
+    parts.push(`cache read \`${tok(metrics.cacheReadTokens)}\``);
+  }
+  if (metrics.cacheWriteTokens > 0) {
+    parts.push(`cache write \`${tok(metrics.cacheWriteTokens)}\``);
   }
   parts.push(`out \`${tok(metrics.outputTokens)}\``);
   const lines = [
@@ -390,6 +408,9 @@ function printMetrics(metrics, stream = process.stderr) {
     const hitPct = (metrics.cacheReadTokens / totalInput * 100).toFixed(0);
     tokenLine += dim(` (${tok(metrics.cacheReadTokens)} cached ${hitPct}% \xB7 ${tok(metrics.inputTokens)} fresh)`);
   }
+  if (metrics.cacheWriteTokens > 0) {
+    tokenLine += dim(` \xB7 ${tok(metrics.cacheWriteTokens)} cache write`);
+  }
   tokenLine += `  ${bold(tok(metrics.outputTokens))} out`;
   tokenLine += dim(`  (${tok(metrics.totalTokens)} total)`);
   write(tokenLine);
@@ -447,6 +468,21 @@ async function pushMetrics(opts) {
     `# HELP hodor_review_duration_seconds Review duration in seconds`,
     `# TYPE hodor_review_duration_seconds gauge`,
     `hodor_review_duration_seconds${labelSuffix} ${metrics.durationSeconds}`,
+    `# HELP hodor_review_diff_files Number of files included in the reviewed diff`,
+    `# TYPE hodor_review_diff_files gauge`,
+    `hodor_review_diff_files${labelSuffix} ${metrics.diffFiles ?? 0}`,
+    `# HELP hodor_review_diff_additions Added lines included in the reviewed diff`,
+    `# TYPE hodor_review_diff_additions gauge`,
+    `hodor_review_diff_additions${labelSuffix} ${metrics.diffAdditions ?? 0}`,
+    `# HELP hodor_review_diff_deletions Deleted lines included in the reviewed diff`,
+    `# TYPE hodor_review_diff_deletions gauge`,
+    `hodor_review_diff_deletions${labelSuffix} ${metrics.diffDeletions ?? 0}`,
+    `# HELP hodor_review_diff_bytes Size of the reviewed diff in bytes`,
+    `# TYPE hodor_review_diff_bytes gauge`,
+    `hodor_review_diff_bytes${labelSuffix} ${metrics.diffBytes ?? 0}`,
+    `# HELP hodor_review_reused Whether an existing identical-HEAD review was reused`,
+    `# TYPE hodor_review_reused gauge`,
+    `hodor_review_reused${labelSuffix} ${metrics.reused ? 1 : 0}`,
     ""
   ];
   const body = lines.join("\n");
@@ -827,12 +863,15 @@ async function postGitlabReviewCommitStatus(parsed, review, diffRefs) {
   );
 }
 async function postReviewComment(opts) {
-  const { prUrl, reviewText, model, metricsFooter, headSha } = opts;
+  const { prUrl, reviewText, model, metricsFooter, headSha, cacheMarker } = opts;
   const platform = detectPlatform(prUrl);
   const parsed = parsePrUrl(prUrl);
   let body = reviewText;
   if (headSha) body = `<!-- hodor:sha:${headSha} -->
 ${body}`;
+  if (cacheMarker) body = body.replace("\n", `
+${cacheMarker}
+`);
   if (model) body += `
 ---
 
@@ -893,7 +932,9 @@ async function postReviewStructured(opts) {
     commitStatus = false,
     headSha,
     workspacePath,
-    reconcileDiscussions = false
+    reconcileDiscussions = false,
+    cacheMarker,
+    skipSummary = false
   } = opts;
   const platform = detectPlatform(prUrl);
   if (platform !== "gitlab" || reviewStyle === "summary") {
@@ -902,7 +943,8 @@ async function postReviewStructured(opts) {
       reviewText: renderMarkdown(review),
       model,
       metricsFooter,
-      headSha
+      headSha,
+      cacheMarker
     });
   }
   const parsed = parsePrUrl(prUrl);
@@ -923,7 +965,8 @@ async function postReviewStructured(opts) {
       reviewText: renderMarkdown(review),
       model,
       metricsFooter,
-      headSha
+      headSha,
+      cacheMarker
     });
   }
   const existingByFingerprint = /* @__PURE__ */ new Map();
@@ -1018,10 +1061,13 @@ ${finding.suggestion}
     }
   }
   let summaryPosted = false;
-  if (reviewStyle === "hybrid" || review.findings.length === 0) {
+  if (!skipSummary && (reviewStyle === "hybrid" || review.findings.length === 0)) {
     let summaryBody = renderSummaryMarkdown(review);
     if (headSha) summaryBody = `<!-- hodor:sha:${headSha} -->
 ${summaryBody}`;
+    if (cacheMarker) summaryBody = summaryBody.replace("\n", `
+${cacheMarker}
+`);
     if (model) summaryBody += `
 ---
 
@@ -1056,7 +1102,7 @@ ${metricsFooter}`;
     }
   }
   let reconciledDiscussions = 0;
-  const baseDeliveryComplete = reviewStyle === "hybrid" ? summaryPosted && inlineFailed === 0 && (inlineCreated === 0 || draftsPublished) : inlineFailed === 0 && (review.findings.length === 0 ? summaryPosted : inlineCreated === 0 || draftsPublished);
+  const baseDeliveryComplete = reviewStyle === "hybrid" ? (summaryPosted || skipSummary) && inlineFailed === 0 && (inlineCreated === 0 || draftsPublished) : inlineFailed === 0 && (review.findings.length === 0 ? summaryPosted : inlineCreated === 0 || draftsPublished);
   if (reconcileDiscussions && baseDeliveryComplete) {
     const currentFingerprints = new Set(
       review.findings.map((finding) => getFindingFingerprint(finding, workspacePath))
@@ -1117,6 +1163,7 @@ async function fetchGithubPrInfo(owner, repo, prNumber) {
     "changedFiles",
     "labels",
     "comments",
+    "reviews",
     "state",
     "isDraft",
     "createdAt",
@@ -1144,6 +1191,7 @@ function normalizeGithubMetadata(raw) {
   const author = raw.author ?? {};
   const labels = raw.labels ?? [];
   const comments = raw.comments;
+  const reviews = raw.reviews;
   return {
     title: raw.title,
     description: raw.body ?? "",
@@ -1155,7 +1203,10 @@ function normalizeGithubMetadata(raw) {
       username: author.login ?? author.name,
       name: author.name
     },
-    Notes: githubCommentsToNotes(comments)
+    Notes: [
+      ...githubCommentsToNotes(comments),
+      ...githubCommentsToNotes(reviews)
+    ]
   };
 }
 function githubCommentsToNotes(comments) {
@@ -1734,6 +1785,8 @@ var REVIEW_SYSTEM_PROMPT = `You are a code review agent. You analyze pull reques
 * Use the grep and find tools for code search \u2014 do not shell out to grep/find.
 * Prefer \`git diff\` to see changes for specific files. Only use read when you need surrounding context that the diff alone cannot provide.
 * Do not use cat/head/tail to read files.
+* Prefer bounded line-range reads. Do not read an entire large file when a changed hunk, symbol, or caller range is sufficient.
+* Do not repeat diff, grep, or read operations whose results are already in context.
 * Keep reasoning proportional to the task. A small diff does not need extensive deliberation.
 </EFFICIENCY>`;
 
@@ -1762,28 +1815,68 @@ function getHodorReviewShaCandidates(notes) {
   });
   return [...new Set(candidates.map(({ sha }) => sha))];
 }
-async function findLatestValidReviewSha(notes, workspacePath) {
+async function findLatestReviewBase(notes, workspacePath) {
   const candidates = getHodorReviewShaCandidates(notes);
   if (candidates.length === 0) return null;
   logger.info(`Found ${candidates.length} previous Hodor review marker(s)`);
   for (const sha of candidates) {
     try {
-      const { stdout: objectType } = await exec("git", ["cat-file", "-t", sha], {
-        cwd: workspacePath
-      });
+      let objectType;
+      try {
+        ({ stdout: objectType } = await exec("git", ["cat-file", "-t", sha], {
+          cwd: workspacePath
+        }));
+      } catch {
+        await exec("git", ["fetch", "--quiet", "origin", sha], {
+          cwd: workspacePath
+        });
+        ({ stdout: objectType } = await exec("git", ["cat-file", "-t", sha], {
+          cwd: workspacePath
+        }));
+      }
       if (objectType.trim() !== "commit") throw new Error("not a commit");
-      await exec("git", ["merge-base", "--is-ancestor", sha, "HEAD"], {
-        cwd: workspacePath
-      });
-      return sha;
+      try {
+        await exec("git", ["merge-base", "--is-ancestor", sha, "HEAD"], {
+          cwd: workspacePath
+        });
+        return { sha, mode: "incremental" };
+      } catch {
+        logger.info(
+          `Previous review SHA ${sha.slice(0, 8)} is not an ancestor; using snapshot delta`
+        );
+        return { sha, mode: "snapshot" };
+      }
     } catch {
       logger.info(
-        `Skipping previous review SHA ${sha.slice(0, 8)}; not a valid ancestor of HEAD`
+        `Skipping previous review SHA ${sha.slice(0, 8)}; commit is unavailable`
       );
     }
   }
   return null;
 }
+function getDiffStats(diff) {
+  let files = 0;
+  let additions = 0;
+  let deletions = 0;
+  for (const line of diff.split("\n")) {
+    if (line.startsWith("diff --git ")) files++;
+    else if (line.startsWith("+") && !line.startsWith("+++")) additions++;
+    else if (line.startsWith("-") && !line.startsWith("---")) deletions++;
+  }
+  return {
+    files,
+    additions,
+    deletions,
+    bytes: Buffer.byteLength(diff, "utf-8")
+  };
+}
+function getChangedFiles(diff) {
+  const files = [];
+  for (const match of diff.matchAll(/^diff --git a\/(.*?) b\/(.*?)$/gm)) {
+    files.push(match[2]);
+  }
+  return [...new Set(files)];
+}
 var DIFF_SKIP_PATTERNS = [
   /(?:^|\/)testdata\//,
   /(?:^|\/)(?:package-lock\.json|yarn\.lock|pnpm-lock\.yaml|go\.sum|Cargo\.lock|poetry\.lock|Gemfile\.lock|composer\.lock)$/,
@@ -1809,6 +1902,66 @@ function filterEmbeddedDiff(rawDiff) {
   return { filtered: kept.join(""), skippedFiles };
 }
 
+// src/review-cache.ts
+import { createHash as createHash2 } from "crypto";
+import { gzipSync, gunzipSync } from "zlib";
+import { readFileSync as readFileSync3 } from "fs";
+var REVIEW_PROMPT_VERSION = "2026-07-16.1";
+var CACHE_MARKER_RE = /<!--\s*hodor:cache:v1:([A-Za-z0-9_-]+)\s*-->/;
+function getReviewCacheKey(opts) {
+  let promptFileContents = "";
+  if (opts.promptFile) {
+    promptFileContents = readFileSync3(opts.promptFile, "utf-8");
+  }
+  return createHash2("sha256").update(JSON.stringify({
+    version: REVIEW_PROMPT_VERSION,
+    headSha: opts.headSha,
+    model: opts.model,
+    // "auto" deliberately stays stable when an identical HEAD changes from
+    // a full review to an empty incremental diff on a pipeline retry.
+    reasoning: opts.requestedReasoningEffort?.toLowerCase() ?? "auto",
+    customPrompt: opts.customPrompt ?? "",
+    promptFileContents
+  })).digest("hex");
+}
+function buildReviewCacheMarker(key, review, workspacePath) {
+  const portableReview = {
+    ...review,
+    findings: review.findings.map((finding) => ({
+      ...finding,
+      code_location: {
+        ...finding.code_location,
+        absolute_file_path: `/workspace/${relativizeWorkspacePath(
+          finding.code_location.absolute_file_path,
+          workspacePath ?? void 0
+        )}`
+      }
+    }))
+  };
+  const payload = { key, review: portableReview };
+  const encoded = gzipSync(JSON.stringify(payload)).toString("base64url");
+  return `<!-- hodor:cache:v1:${encoded} -->`;
+}
+function findCachedReview(notes, key) {
+  if (!notes) return null;
+  const newestFirst = [...notes].sort(
+    (a, b) => Date.parse(b.created_at ?? "") - Date.parse(a.created_at ?? "")
+  );
+  for (const note of newestFirst) {
+    const encoded = note.body?.match(CACHE_MARKER_RE)?.[1];
+    if (!encoded || encoded.length > 5e5) continue;
+    try {
+      const payload = JSON.parse(
+        gunzipSync(Buffer.from(encoded, "base64url"), { maxOutputLength: 1e6 }).toString("utf-8")
+      );
+      if (payload.key !== key || !payload.review) continue;
+      return validateReviewOutput(payload.review);
+    } catch {
+    }
+  }
+  return null;
+}
+
 // src/review-recovery.ts
 import { Value } from "@sinclair/typebox/value";
 var SUBMIT_REVIEW_RECOVERY_ATTEMPTS = 2;
@@ -1966,10 +2119,7 @@ async function reviewPr(opts) {
       );
     }
   }
-  const thinkingLevel = mapReasoningEffort(reasoningEffort) ?? getDefaultReasoningEffortForModel(piModel);
-  if (!reasoningEffort && thinkingLevel) {
-    logger.info(`Default reasoning effort for ${piModel.name}: ${thinkingLevel}`);
-  }
+  const modelDefaultThinkingLevel = getDefaultReasoningEffortForModel(piModel);
   if (parsed.provider !== "amazon-bedrock") {
     const resolvedKey = await modelRegistry.getApiKeyForProvider(parsed.provider);
     if (!resolvedKey) {
@@ -2075,26 +2225,84 @@ async function reviewPr(opts) {
         logger.warn(`Failed to fetch Gitea metadata: ${err}`);
       }
     }
-    const previousReviewSha = full ? null : await findLatestValidReviewSha(mrMetadata?.Notes, workspacePath);
-    if (full) {
-      logger.info("Full review mode: ignoring previous hodor reviews, diffing entire source-vs-target range");
-    } else if (previousReviewSha) {
-      logger.info(`Incremental mode: previous review at ${previousReviewSha.slice(0, 8)}`);
-    }
     let headSha = null;
     if (!localMode) {
       const { stdout: headShaRaw } = await exec("git", ["rev-parse", "HEAD"], { cwd: workspacePath });
       headSha = headShaRaw.trim();
     }
+    let reviewCacheKey = null;
+    if (!localMode && !full && headSha) {
+      reviewCacheKey = getReviewCacheKey({
+        headSha,
+        model,
+        requestedReasoningEffort: reasoningEffort,
+        customPrompt,
+        promptFile
+      });
+      const cachedReview = findCachedReview(mrMetadata?.Notes, reviewCacheKey);
+      if (cachedReview) {
+        logger.info(`Reusing cached Hodor review for HEAD ${headSha.slice(0, 8)}`);
+        const metrics2 = {
+          inputTokens: 0,
+          outputTokens: 0,
+          cacheReadTokens: 0,
+          cacheWriteTokens: 0,
+          totalTokens: 0,
+          cost: 0,
+          turns: 0,
+          toolCalls: 0,
+          durationSeconds: 0,
+          reviewMode: "reused",
+          reasoningEffort: reasoningEffort ?? "auto",
+          diffFiles: 0,
+          diffAdditions: 0,
+          diffDeletions: 0,
+          diffBytes: 0,
+          reused: true
+        };
+        logger.info(`Review telemetry: ${JSON.stringify({
+          reviewMode: metrics2.reviewMode,
+          reasoningEffort: metrics2.reasoningEffort,
+          reused: true,
+          headSha: headSha.slice(0, 12),
+          findings: cachedReview.findings.length
+        })}`);
+        printMetrics(metrics2);
+        return {
+          review: cachedReview,
+          metricsFooter: includeMetricsFooter ? formatMetricsMarkdown(metrics2) : null,
+          headSha,
+          metrics: metrics2,
+          workspacePath,
+          cacheMarker: null,
+          reusedReview: true
+        };
+      }
+    }
+    const previousReviewBase = full || localMode ? null : await findLatestReviewBase(mrMetadata?.Notes, workspacePath);
+    const previousReviewSha = previousReviewBase?.sha ?? null;
+    let reviewMode = localMode ? "local" : previousReviewBase?.mode ?? "full";
+    if (full) {
+      reviewMode = "full";
+      logger.info("Full review mode: ignoring previous hodor reviews, diffing entire source-vs-target range");
+    } else if (previousReviewBase) {
+      logger.info(`${previousReviewBase.mode === "snapshot" ? "Snapshot delta" : "Incremental"} mode: previous review at ${previousReviewSha?.slice(0, 8)}`);
+    }
     const MAX_EMBED_BYTES = 200 * 1024;
     let embeddedDiff = null;
+    let reviewDiff = null;
+    let diffStats = null;
+    let changedFiles = [];
     try {
-      const diffArgs = previousReviewSha ? ["--no-pager", "diff", `${previousReviewSha}...HEAD`] : diffBaseSha ? ["--no-pager", "diff", diffBaseSha, "HEAD"] : localMode ? ["--no-pager", "diff", targetBranch] : ["--no-pager", "diff", `origin/${targetBranch}...HEAD`];
+      const diffArgs = previousReviewSha ? previousReviewBase?.mode === "snapshot" ? ["--no-pager", "diff", previousReviewSha, "HEAD"] : ["--no-pager", "diff", `${previousReviewSha}...HEAD`] : diffBaseSha ? ["--no-pager", "diff", diffBaseSha, "HEAD"] : localMode ? ["--no-pager", "diff", targetBranch] : ["--no-pager", "diff", `origin/${targetBranch}...HEAD`];
       const { stdout: rawDiff } = await exec("git", diffArgs, { cwd: workspacePath });
       const { filtered: filteredDiff, skippedFiles } = filterEmbeddedDiff(rawDiff);
       if (skippedFiles.length > 0) {
         logger.info(`Filtered ${skippedFiles.length} file(s) from embedded diff: ${skippedFiles.join(", ")}`);
       }
+      reviewDiff = filteredDiff;
+      diffStats = getDiffStats(filteredDiff);
+      changedFiles = getChangedFiles(filteredDiff);
       if (Buffer.byteLength(filteredDiff, "utf-8") <= MAX_EMBED_BYTES) {
         embeddedDiff = filteredDiff;
         logger.info(`Embedding diff in prompt (${Buffer.byteLength(filteredDiff, "utf-8")} bytes, raw: ${Buffer.byteLength(rawDiff, "utf-8")} bytes)`);
@@ -2104,6 +2312,17 @@ async function reviewPr(opts) {
     } catch (err) {
       logger.warn(`Failed to pre-fetch diff, falling back to command mode: ${err}`);
     }
+    const thinkingLevel = selectReasoningEffort({
+      requested: reasoningEffort,
+      modelDefault: modelDefaultThinkingLevel,
+      mode: reviewMode,
+      forcedFull: full,
+      diff: reviewDiff,
+      stats: diffStats
+    });
+    if (thinkingLevel) {
+      logger.info(`Reasoning effort for ${piModel.name}: ${thinkingLevel}${reasoningEffort ? " (explicit)" : " (adaptive)"}`);
+    }
     const prompt = buildPrReviewPrompt({
       prUrl: prUrl ?? `local diff (against ${targetBranch})`,
       platform,
@@ -2114,6 +2333,8 @@ async function reviewPr(opts) {
       customPromptFile: promptFile,
       embeddedDiff,
       previousReviewSha,
+      reviewDiffMode: reviewMode,
+      changedFiles,
       localMode
     });
     const startTime = Date.now();
@@ -2345,14 +2566,47 @@ async function reviewPr(opts) {
       cost,
       turns: turnCount,
       toolCalls: toolCallCount,
-      durationSeconds: Math.round(durationSeconds)
+      durationSeconds: Math.round(durationSeconds),
+      reviewMode,
+      reasoningEffort: thinkingLevel ?? "none",
+      diffFiles: diffStats?.files ?? 0,
+      diffAdditions: diffStats?.additions ?? 0,
+      diffDeletions: diffStats?.deletions ?? 0,
+      diffBytes: diffStats?.bytes ?? 0,
+      reused: false
     };
+    logger.info(`Review telemetry: ${JSON.stringify({
+      reviewMode: metrics.reviewMode,
+      reasoningEffort: metrics.reasoningEffort,
+      reused: false,
+      diffFiles: metrics.diffFiles,
+      diffAdditions: metrics.diffAdditions,
+      diffDeletions: metrics.diffDeletions,
+      diffBytes: metrics.diffBytes,
+      turns: metrics.turns,
+      toolCalls: metrics.toolCalls,
+      inputTokens: metrics.inputTokens,
+      cacheReadTokens: metrics.cacheReadTokens,
+      cacheWriteTokens: metrics.cacheWriteTokens,
+      outputTokens: metrics.outputTokens,
+      cost: metrics.cost,
+      findings: review.findings.length
+    })}`);
     printMetrics(metrics);
     let metricsFooter = null;
     if (includeMetricsFooter) {
       metricsFooter = formatMetricsMarkdown(metrics);
     }
-    return { review, metricsFooter, headSha, metrics, workspacePath };
+    const cacheMarker = reviewCacheKey ? buildReviewCacheMarker(reviewCacheKey, review, workspacePath) : null;
+    return {
+      review,
+      metricsFooter,
+      headSha,
+      metrics,
+      workspacePath,
+      cacheMarker,
+      reusedReview: false
+    };
   } finally {
     activeSession?.dispose();
     for (const [key, val] of Object.entries(envSnapshot)) {
@@ -2385,4 +2639,4 @@ export {
   postReviewStructured,
   reviewPr
 };
-//# sourceMappingURL=chunk-ZUYM6HY3.js.map
+//# sourceMappingURL=chunk-N7TSXVUE.js.map