@mrkaran/hodor 0.6.2 → 0.6.3-rc.1

package/dist/{chunk-LQDQ7CHL.js → chunk-AIXGPWDG.js}

@@ -13,8 +13,9 @@ import {
   renderMarkdown,
   renderSummaryMarkdown,
   resolveGitlabDiscussions,
-  summarizeGitlabNotes
-} from "./chunk-DVJVQTVW.js";
+  summarizeGitlabNotes,
+  summarizeHodorNotes
+} from "./chunk-LO2QJD45.js";
 import {
   relativizeWorkspacePath
 } from "./chunk-AMUK6GDX.js";
@@ -215,15 +216,22 @@ function buildMrSections(mrMetadata) {
     contextSection += "\n";
   }
   let notesSection = "";
-  const notesSummary = summarizeGitlabNotes(mrMetadata.Notes);
-  if (notesSummary) {
-    notesSection = `## Existing MR Notes
-${notesSummary}
+  const humanNotesSummary = summarizeGitlabNotes(mrMetadata.Notes);
+  const hodorNotesSummary = summarizeHodorNotes(mrMetadata.Notes);
+  if (humanNotesSummary) {
+    notesSection += `## Existing Human MR Notes
+${humanNotesSummary}
+`;
+  }
+  if (hodorNotesSummary) {
+    notesSection += `## Prior Hodor Reviews (deduplication only)
+${hodorNotesSummary}
+Use this history only to avoid repeating the same finding. Re-check the current diff independently.
 `;
   }
   let reminderSection = "";
-  if (notesSummary) {
-    reminderSection = "## Review Note Deduplication\n\nThe discussions above may already cover some issues. Before reporting a finding:\n1. Check if it's already mentioned in existing notes\n2. Only report if your finding is materially different or more specific\n3. If an existing note is incorrect/outdated, explain why in your finding\n\nFocus on discovering NEW issues not yet discussed.\n";
+  if (humanNotesSummary || hodorNotesSummary) {
+    reminderSection = "## Review Note Deduplication\n\nThe human notes and prior Hodor reviews above may already cover some issues. Before reporting a finding:\n1. Check if it's already mentioned in existing notes\n2. Only report if your finding is materially different or more specific\n3. If an existing note is incorrect/outdated, explain why in your finding\n\nFocus on discovering NEW issues not yet discussed.\n";
   }
   return { contextSection, notesSection, reminderSection };
 }
@@ -545,100 +553,87 @@ function getPriorityFromTitle(title) {
   return REVIEW_PRIORITY_TAGS.get(`[${match[1]}]`) ?? null;
 }
 
-// src/agent.ts
-import { existsSync } from "fs";
-import { join as join2 } from "path";
-import { Value } from "@sinclair/typebox/value";
-
-// src/github.ts
-var GitHubAPIError = class extends Error {
-  constructor(message) {
-    super(message);
-    this.name = "GitHubAPIError";
+// src/platform.ts
+function detectPlatform(prUrl) {
+  const url = new URL(prUrl);
+  const hostname = url.hostname;
+  if (prUrl.includes("/-/merge_requests/") || hostname.includes("gitlab")) {
+    return "gitlab";
   }
-};
-async function fetchGithubPrInfo(owner, repo, prNumber) {
-  const fields = [
-    "number",
-    "title",
-    "body",
-    "author",
-    "baseRefName",
-    "headRefName",
-    "baseRefOid",
-    "headRefOid",
-    "changedFiles",
-    "labels",
-    "comments",
-    "state",
-    "isDraft",
-    "createdAt",
-    "updatedAt",
-    "mergeable",
-    "url"
-  ];
-  const repoFullPath = `${owner}/${repo}`;
-  try {
-    return await execJson("gh", [
-      "pr",
-      "view",
-      String(prNumber),
-      "-R",
-      repoFullPath,
-      "--json",
-      fields.join(",")
-    ]);
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    throw new GitHubAPIError(msg);
+  if (prUrl.includes("/pulls/") || hostname.includes("gitea") || hostname.includes("forgejo") || hostname.includes("codeberg")) {
+    return "gitea";
   }
+  if (prUrl.includes("/pull/") || hostname.includes("github")) {
+    return "github";
+  }
+  throw new Error(
+    `Cannot detect platform for URL: ${prUrl}. Expected a GitHub (/pull/), GitLab (/-/merge_requests/), or Gitea/Forgejo (/pulls/) URL.`
+  );
 }
-function normalizeGithubMetadata(raw) {
-  const author = raw.author ?? {};
-  const labels = raw.labels ?? [];
-  const comments = raw.comments;
-  return {
-    title: raw.title,
-    description: raw.body ?? "",
-    source_branch: raw.headRefName,
-    target_branch: raw.baseRefName,
-    changes_count: raw.changedFiles,
-    labels: labels.map((lbl) => ({ name: lbl.name ?? lbl.id })),
-    author: {
-      username: author.login ?? author.name,
-      name: author.name
-    },
-    Notes: githubCommentsToNotes(comments)
-  };
-}
-function githubCommentsToNotes(comments) {
-  if (!comments) return [];
-  let nodes;
-  if (Array.isArray(comments)) {
-    nodes = comments;
-  } else if (typeof comments === "object") {
-    nodes = comments.nodes ?? comments.edges ?? [];
-    if (nodes.length > 0 && typeof nodes[0] === "object" && "node" in nodes[0]) {
-      nodes = nodes.map(
-        (edge) => edge.node ?? {}
+function parsePrUrl(prUrl) {
+  const url = new URL(prUrl);
+  const pathParts = url.pathname.split("/").filter(Boolean);
+  const host = url.host;
+  if (pathParts.length >= 4 && pathParts[2] === "pull") {
+    return {
+      owner: pathParts[0],
+      repo: pathParts[1],
+      prNumber: parsePositiveNumber(pathParts[3], "PR", prUrl, "/pull/"),
+      host
+    };
+  }
+  if (pathParts.length >= 4 && pathParts[2] === "pulls") {
+    return {
+      owner: pathParts[0],
+      repo: pathParts[1],
+      prNumber: parsePositiveNumber(pathParts[3], "PR", prUrl, "/pulls/"),
+      host
+    };
+  }
+  const mrIndex = pathParts.indexOf("merge_requests");
+  if (mrIndex >= 0) {
+    if (mrIndex < 2 || mrIndex + 1 >= pathParts.length) {
+      throw new Error(
+        `Invalid GitLab MR URL format: ${prUrl}. Expected .../-/merge_requests/<number>`
       );
     }
-  } else {
-    nodes = [];
-  }
-  return nodes.map((node) => {
-    const author = node.author ?? {};
+    if (pathParts[mrIndex - 1] !== "-") {
+      throw new Error(
+        `Invalid GitLab MR URL format: ${prUrl}. Missing '/-/' segment before merge_requests.`
+      );
+    }
+    const repo = pathParts[mrIndex - 2];
+    const ownerParts = pathParts.slice(0, mrIndex - 2);
+    const owner = ownerParts.length > 0 ? ownerParts.join("/") : pathParts[0];
     return {
-      body: node.body ?? "",
-      author: {
-        username: author.login ?? author.name,
-        name: author.name
-      },
-      created_at: node.createdAt
+      owner,
+      repo,
+      prNumber: parsePositiveNumber(
+        pathParts[mrIndex + 1],
+        "MR",
+        prUrl,
+        "/merge_requests/"
+      ),
+      host
     };
-  });
+  }
+  throw new Error(
+    `Invalid PR/MR URL format: ${prUrl}. Expected GitHub (/pull/), GitLab (/-/merge_requests/), or Gitea/Forgejo (/pulls/) URL.`
+  );
+}
+function parsePositiveNumber(raw, kind, prUrl, segment) {
+  const number = Number(raw);
+  if (!Number.isSafeInteger(number) || number <= 0) {
+    throw new Error(
+      `Invalid ${kind} number in URL: ${prUrl}. Expected a positive integer after ${segment}.`
+    );
+  }
+  return number;
 }
 
+// src/publisher.ts
+import { createHash } from "crypto";
+
 // src/gitea.ts
 var GiteaAPIError = class extends Error {
   constructor(message) {
@@ -801,72 +796,459 @@ async function postGiteaPrComment(owner, repo, prNumber, body, host) {
   }
 }
 
-// src/workspace.ts
-import { mkdtemp, rm } from "fs/promises";
-import { tmpdir } from "os";
-import { join } from "path";
-var WorkspaceError = class extends Error {
-  constructor(message) {
-    super(message);
-    this.name = "WorkspaceError";
-  }
-};
-function envOrNull(name) {
-  const value = process.env[name]?.trim();
-  return value ? value : null;
+// src/publisher.ts
+var FINDING_MARKER_RE = /<!--\s*hodor:finding:([a-f0-9]{64})\s*-->/i;
+function getFindingFingerprint(finding, workspacePath) {
+  const path = relativizeWorkspacePath(
+    finding.code_location.absolute_file_path,
+    workspacePath ?? void 0
+  );
+  const title = finding.title.replace(/^\[P[0-3]\]\s*/, "").trim().toLowerCase();
+  return createHash("sha256").update(`${path}
+${title}`).digest("hex");
 }
-async function detectCiWorkspace(owner, repo) {
-  const expected = `${owner}/${repo}`;
-  if (process.env.GITLAB_CI === "true") {
-    const projectDir = envOrNull("CI_PROJECT_DIR");
-    const projectPath = envOrNull("CI_PROJECT_PATH");
-    const targetBranch = envOrNull("CI_MERGE_REQUEST_TARGET_BRANCH_NAME");
-    const diffBaseSha = envOrNull("CI_MERGE_REQUEST_DIFF_BASE_SHA");
-    if (projectDir && projectPath && (projectPath === expected || projectPath.endsWith(`/${expected}`))) {
-      if (await isSameRepo(projectDir, owner, repo)) {
-        logger.info(`Detected GitLab CI environment (target: ${targetBranch ?? "unknown"})`);
-        return { path: projectDir, targetBranch, diffBaseSha };
-      }
-      logger.warn(
-        `Detected GitLab CI for ${projectPath}, but ${projectDir} is not a git checkout of ${expected}; falling back to clone`
-      );
-    }
-  }
-  if (process.env.GITEA_ACTIONS === "true" || process.env.FORGEJO_ACTIONS === "true") {
-    const workspaceDir = envOrNull("GITHUB_WORKSPACE");
-    const repository = envOrNull("GITHUB_REPOSITORY");
-    const baseRef = envOrNull("GITHUB_BASE_REF");
-    if (workspaceDir && repository === expected) {
-      const ciType = process.env.FORGEJO_ACTIONS ? "Forgejo" : "Gitea";
-      if (await isSameRepo(workspaceDir, owner, repo)) {
-        logger.info(`Detected ${ciType} Actions environment (base: ${baseRef ?? "unknown"})`);
-        return { path: workspaceDir, targetBranch: baseRef, diffBaseSha: null };
-      }
-      logger.warn(
-        `Detected ${ciType} Actions for ${repository}, but ${workspaceDir} is not a git checkout of ${expected}; falling back to clone`
-      );
+function getDiscussionFingerprint(body) {
+  return body.match(FINDING_MARKER_RE)?.[1]?.toLowerCase() ?? null;
+}
+async function postGitlabReviewCommitStatus(parsed, review, diffRefs) {
+  const blocking = review.findings.filter((finding) => finding.priority <= 1).length;
+  const state = blocking > 0 ? "failed" : "success";
+  const description = blocking > 0 ? `${blocking} blocking issue(s) found` : review.findings.length > 0 ? `${review.findings.length} non-blocking issue(s)` : "No issues found";
+  await postGitlabCommitStatus(
+    parsed.owner,
+    parsed.repo,
+    diffRefs.head_sha,
+    state,
+    parsed.host,
+    { description }
+  );
+}
+async function postReviewComment(opts) {
+  const { prUrl, reviewText, model, metricsFooter, headSha } = opts;
+  const platform = detectPlatform(prUrl);
+  const parsed = parsePrUrl(prUrl);
+  let body = reviewText;
+  if (headSha) body = `<!-- hodor:sha:${headSha} -->
+${body}`;
+  if (model) body += `
+---
+
+Review generated by Hodor (model: \`${model}\`)`;
+  if (metricsFooter) body += `
+
+${metricsFooter}`;
+  try {
+    if (platform === "github") {
+      await exec("gh", [
+        "pr",
+        "review",
+        String(parsed.prNumber),
+        "--repo",
+        `${parsed.owner}/${parsed.repo}`,
+        "--comment",
+        "--body",
+        body
+      ]);
+      return { success: true, platform, prNumber: parsed.prNumber };
     }
-  }
-  if (process.env.GITHUB_ACTIONS === "true") {
-    const workspaceDir = envOrNull("GITHUB_WORKSPACE");
-    const repository = envOrNull("GITHUB_REPOSITORY");
-    const baseRef = envOrNull("GITHUB_BASE_REF");
-    if (workspaceDir && repository === expected) {
-      if (await isSameRepo(workspaceDir, owner, repo)) {
-        logger.info(`Detected GitHub Actions environment (base: ${baseRef ?? "unknown"})`);
-        return { path: workspaceDir, targetBranch: baseRef, diffBaseSha: null };
-      }
-      logger.warn(
-        `Detected GitHub Actions for ${repository}, but ${workspaceDir} is not a git checkout of ${expected}; falling back to clone`
+    if (platform === "gitea") {
+      await postGiteaPrComment(
+        parsed.owner,
+        parsed.repo,
+        parsed.prNumber,
+        body,
+        parsed.host
       );
+      return { success: true, platform, prNumber: parsed.prNumber };
     }
+    await postGitlabMrComment(
+      parsed.owner,
+      parsed.repo,
+      parsed.prNumber,
+      body,
+      parsed.host
+    );
+    return {
+      success: true,
+      platform,
+      mrNumber: parsed.prNumber,
+      summaryPosted: true
+    };
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
+    logger.error(`Failed to post comment: ${message}`);
+    return { success: false, platform, error: message };
   }
-  return { path: null, targetBranch: null, diffBaseSha: null };
 }
-function normalizeGitRemotePath(remoteUrl) {
-  const trimmed = remoteUrl.trim().replace(/\.git$/, "");
-  try {
-    const url = new URL(trimmed);
+async function postReviewStructured(opts) {
+  const {
+    prUrl,
+    review,
+    model,
+    metricsFooter,
+    reviewStyle = "hybrid",
+    commitStatus = false,
+    headSha,
+    workspacePath,
+    reconcileDiscussions = false
+  } = opts;
+  const platform = detectPlatform(prUrl);
+  if (platform !== "gitlab" || reviewStyle === "summary") {
+    return postReviewComment({
+      prUrl,
+      reviewText: renderMarkdown(review),
+      model,
+      metricsFooter,
+      headSha
+    });
+  }
+  const parsed = parsePrUrl(prUrl);
+  const errors = [];
+  let diffRefs;
+  try {
+    diffRefs = await getGitlabMrDiffRefs(
+      parsed.owner,
+      parsed.repo,
+      parsed.prNumber,
+      parsed.host
+    );
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
+    logger.warn(`Failed to get diff_refs, falling back to summary mode: ${message}`);
+    return postReviewComment({
+      prUrl,
+      reviewText: renderMarkdown(review),
+      model,
+      metricsFooter,
+      headSha
+    });
+  }
+  const existingByFingerprint = /* @__PURE__ */ new Map();
+  try {
+    const discussions = await listHodorDiscussions(
+      parsed.owner,
+      parsed.repo,
+      parsed.prNumber,
+      parsed.host
+    );
+    for (const discussion of discussions) {
+      if (discussion.resolved) continue;
+      const fingerprint = getDiscussionFingerprint(discussion.body);
+      if (!fingerprint) continue;
+      const ids = existingByFingerprint.get(fingerprint) ?? /* @__PURE__ */ new Set();
+      ids.add(discussion.discussionId);
+      existingByFingerprint.set(fingerprint, ids);
+    }
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
+    if (reconcileDiscussions) {
+      errors.push(`discussion listing: ${message}`);
+    }
+    logger.warn(`Failed to list open Hodor discussions for deduplication: ${message}`);
+  }
+  let inlineCreated = 0;
+  let inlineFailed = 0;
+  let inlineDeduplicated = 0;
+  for (const finding of review.findings) {
+    const fingerprint = getFindingFingerprint(finding, workspacePath);
+    if (existingByFingerprint.has(fingerprint)) {
+      inlineDeduplicated++;
+      continue;
+    }
+    const relPath = relativizeWorkspacePath(
+      finding.code_location.absolute_file_path,
+      workspacePath ?? void 0
+    );
+    const title = /^\[P[0-3]\]/.test(finding.title) ? finding.title : `[P${finding.priority}] ${finding.title}`;
+    let body = `${HODOR_REVIEW_MARKER}
+<!-- hodor:finding:${fingerprint} -->
+**${title}**
+
+${finding.body}`;
+    if (finding.suggestion) {
+      const { start, end } = finding.code_location.line_range;
+      const span = Math.max(0, end - start);
+      body += `
+
+\`\`\`suggestion:-0+${span}
+${finding.suggestion}
+\`\`\``;
+    }
+    try {
+      await createGitlabDraftNote(
+        parsed.owner,
+        parsed.repo,
+        parsed.prNumber,
+        body,
+        parsed.host,
+        {
+          filePath: relPath,
+          line: finding.code_location.line_range.start,
+          diffRefs
+        }
+      );
+      inlineCreated++;
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      errors.push(`inline note for ${finding.title}: ${message}`);
+      logger.warn(`Failed to create inline note for "${finding.title}": ${message}`);
+      inlineFailed++;
+    }
+  }
+  logger.info(
+    `Created ${inlineCreated} inline draft note(s)${inlineDeduplicated > 0 ? ` (${inlineDeduplicated} already open)` : ""}${inlineFailed > 0 ? ` (${inlineFailed} failed)` : ""}`
+  );
+  let draftsPublished = false;
+  if (inlineCreated > 0) {
+    try {
+      await bulkPublishGitlabDraftNotes(
+        parsed.owner,
+        parsed.repo,
+        parsed.prNumber,
+        parsed.host
+      );
+      draftsPublished = true;
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      errors.push(`draft publish: ${message}`);
+      logger.warn(`Failed to bulk publish draft notes: ${message}`);
+    }
+  }
+  let summaryPosted = false;
+  if (reviewStyle === "hybrid" || review.findings.length === 0) {
+    let summaryBody = renderSummaryMarkdown(review);
+    if (headSha) summaryBody = `<!-- hodor:sha:${headSha} -->
+${summaryBody}`;
+    if (model) summaryBody += `
+---
+
+Review generated by Hodor (model: \`${model}\`)`;
+    if (metricsFooter) summaryBody += `
+
+${metricsFooter}`;
+    try {
+      await postGitlabMrComment(
+        parsed.owner,
+        parsed.repo,
+        parsed.prNumber,
+        summaryBody,
+        parsed.host
+      );
+      summaryPosted = true;
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      errors.push(`summary comment: ${message}`);
+      logger.warn(`Failed to post summary comment: ${message}`);
+    }
+  }
+  let commitStatusPosted = false;
+  if (commitStatus) {
+    try {
+      await postGitlabReviewCommitStatus(parsed, review, diffRefs);
+      commitStatusPosted = true;
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      errors.push(`commit status: ${message}`);
+      logger.warn(`Failed to post commit status: ${message}`);
+    }
+  }
+  let reconciledDiscussions = 0;
+  const baseDeliveryComplete = reviewStyle === "hybrid" ? summaryPosted && 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))
+    );
+    const staleDiscussionIds = [...existingByFingerprint.entries()].filter(([fingerprint]) => !currentFingerprints.has(fingerprint)).flatMap(([, ids]) => [...ids]);
+    if (staleDiscussionIds.length > 0) {
+      reconciledDiscussions = await resolveGitlabDiscussions(
+        parsed.owner,
+        parsed.repo,
+        parsed.prNumber,
+        staleDiscussionIds,
+        parsed.host
+      );
+      if (reconciledDiscussions !== staleDiscussionIds.length) {
+        errors.push(
+          `discussion reconciliation: resolved ${reconciledDiscussions}/${staleDiscussionIds.length}`
+        );
+      }
+    }
+  }
+  const success = baseDeliveryComplete && (!commitStatus || commitStatusPosted) && (!reconcileDiscussions || !errors.some((error) => error.startsWith("discussion ")));
+  return {
+    success,
+    platform: "gitlab",
+    mrNumber: parsed.prNumber,
+    error: success ? void 0 : errors[0] ?? "Review delivery was incomplete",
+    errors,
+    summaryPosted,
+    inlineCreated,
+    inlineFailed,
+    draftsPublished,
+    commitStatusPosted,
+    reconciledDiscussions
+  };
+}
+
+// src/agent.ts
+import { existsSync } from "fs";
+import { join as join2 } from "path";
+
+// src/github.ts
+var GitHubAPIError = class extends Error {
+  constructor(message) {
+    super(message);
+    this.name = "GitHubAPIError";
+  }
+};
+async function fetchGithubPrInfo(owner, repo, prNumber) {
+  const fields = [
+    "number",
+    "title",
+    "body",
+    "author",
+    "baseRefName",
+    "headRefName",
+    "baseRefOid",
+    "headRefOid",
+    "changedFiles",
+    "labels",
+    "comments",
+    "state",
+    "isDraft",
+    "createdAt",
+    "updatedAt",
+    "mergeable",
+    "url"
+  ];
+  const repoFullPath = `${owner}/${repo}`;
+  try {
+    return await execJson("gh", [
+      "pr",
+      "view",
+      String(prNumber),
+      "-R",
+      repoFullPath,
+      "--json",
+      fields.join(",")
+    ]);
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    throw new GitHubAPIError(msg);
+  }
+}
+function normalizeGithubMetadata(raw) {
+  const author = raw.author ?? {};
+  const labels = raw.labels ?? [];
+  const comments = raw.comments;
+  return {
+    title: raw.title,
+    description: raw.body ?? "",
+    source_branch: raw.headRefName,
+    target_branch: raw.baseRefName,
+    changes_count: raw.changedFiles,
+    labels: labels.map((lbl) => ({ name: lbl.name ?? lbl.id })),
+    author: {
+      username: author.login ?? author.name,
+      name: author.name
+    },
+    Notes: githubCommentsToNotes(comments)
+  };
+}
+function githubCommentsToNotes(comments) {
+  if (!comments) return [];
+  let nodes;
+  if (Array.isArray(comments)) {
+    nodes = comments;
+  } else if (typeof comments === "object") {
+    nodes = comments.nodes ?? comments.edges ?? [];
+    if (nodes.length > 0 && typeof nodes[0] === "object" && "node" in nodes[0]) {
+      nodes = nodes.map(
+        (edge) => edge.node ?? {}
+      );
+    }
+  } else {
+    nodes = [];
+  }
+  return nodes.map((node) => {
+    const author = node.author ?? {};
+    return {
+      body: node.body ?? "",
+      author: {
+        username: author.login ?? author.name,
+        name: author.name
+      },
+      created_at: node.createdAt
+    };
+  });
+}
+
+// src/workspace.ts
+import { mkdtemp, rm } from "fs/promises";
+import { tmpdir } from "os";
+import { join } from "path";
+var WorkspaceError = class extends Error {
+  constructor(message) {
+    super(message);
+    this.name = "WorkspaceError";
+  }
+};
+function envOrNull(name) {
+  const value = process.env[name]?.trim();
+  return value ? value : null;
+}
+async function detectCiWorkspace(owner, repo) {
+  const expected = `${owner}/${repo}`;
+  if (process.env.GITLAB_CI === "true") {
+    const projectDir = envOrNull("CI_PROJECT_DIR");
+    const projectPath = envOrNull("CI_PROJECT_PATH");
+    const targetBranch = envOrNull("CI_MERGE_REQUEST_TARGET_BRANCH_NAME");
+    const diffBaseSha = envOrNull("CI_MERGE_REQUEST_DIFF_BASE_SHA");
+    if (projectDir && projectPath && (projectPath === expected || projectPath.endsWith(`/${expected}`))) {
+      if (await isSameRepo(projectDir, owner, repo)) {
+        logger.info(`Detected GitLab CI environment (target: ${targetBranch ?? "unknown"})`);
+        return { path: projectDir, targetBranch, diffBaseSha };
+      }
+      logger.warn(
+        `Detected GitLab CI for ${projectPath}, but ${projectDir} is not a git checkout of ${expected}; falling back to clone`
+      );
+    }
+  }
+  if (process.env.GITEA_ACTIONS === "true" || process.env.FORGEJO_ACTIONS === "true") {
+    const workspaceDir = envOrNull("GITHUB_WORKSPACE");
+    const repository = envOrNull("GITHUB_REPOSITORY");
+    const baseRef = envOrNull("GITHUB_BASE_REF");
+    if (workspaceDir && repository === expected) {
+      const ciType = process.env.FORGEJO_ACTIONS ? "Forgejo" : "Gitea";
+      if (await isSameRepo(workspaceDir, owner, repo)) {
+        logger.info(`Detected ${ciType} Actions environment (base: ${baseRef ?? "unknown"})`);
+        return { path: workspaceDir, targetBranch: baseRef, diffBaseSha: null };
+      }
+      logger.warn(
+        `Detected ${ciType} Actions for ${repository}, but ${workspaceDir} is not a git checkout of ${expected}; falling back to clone`
+      );
+    }
+  }
+  if (process.env.GITHUB_ACTIONS === "true") {
+    const workspaceDir = envOrNull("GITHUB_WORKSPACE");
+    const repository = envOrNull("GITHUB_REPOSITORY");
+    const baseRef = envOrNull("GITHUB_BASE_REF");
+    if (workspaceDir && repository === expected) {
+      if (await isSameRepo(workspaceDir, owner, repo)) {
+        logger.info(`Detected GitHub Actions environment (base: ${baseRef ?? "unknown"})`);
+        return { path: workspaceDir, targetBranch: baseRef, diffBaseSha: null };
+      }
+      logger.warn(
+        `Detected GitHub Actions for ${repository}, but ${workspaceDir} is not a git checkout of ${expected}; falling back to clone`
+      );
+    }
+  }
+  return { path: null, targetBranch: null, diffBaseSha: null };
+}
+function normalizeGitRemotePath(remoteUrl) {
+  const trimmed = remoteUrl.trim().replace(/\.git$/, "");
+  try {
+    const url = new URL(trimmed);
     return url.pathname.replace(/^\/+/, "").replace(/\.git$/, "");
   } catch {
   }
@@ -1271,247 +1653,88 @@ function resolveReviewLocations(review, opts) {
       const buf = readFileSync2(path);
       if (buf.byteLength <= MAX_RESOLVE_BYTES) content = buf.toString("utf-8");
     } catch {
-      content = null;
-    }
-    fileCache.set(path, content);
-    return content;
-  };
-  const findings = review.findings.map((finding) => {
-    stats.total++;
-    const { existing_code: existingCode, code_location: loc } = finding;
-    if (!existingCode) {
-      stats.noSnippet++;
-      return finding;
-    }
-    if (workspaceRoot && !isWithinWorkspace(workspaceRoot, loc.absolute_file_path)) {
-      stats.unmatched++;
-      logger.warn(
-        `Location resolution: ${loc.absolute_file_path} is outside the workspace for "${finding.title}"; keeping model range`
-      );
-      return finding;
-    }
-    const fileContent = readFile(loc.absolute_file_path);
-    if (fileContent === null) {
-      stats.unmatched++;
-      logger.warn(`Location resolution: could not read ${loc.absolute_file_path} for "${finding.title}"`);
-      return finding;
-    }
-    let changedLines;
-    for (const [relPath, lines] of changedByFile) {
-      if (loc.absolute_file_path.endsWith(`/${relPath}`) || loc.absolute_file_path === relPath) {
-        changedLines = lines;
-        break;
-      }
-    }
-    const result = resolveLineRange({
-      existingCode,
-      fileContent,
-      modelRange: loc.line_range,
-      changedLines
-    });
-    if (result.status === "confirmed") stats.confirmed++;
-    else if (result.status === "corrected") stats.corrected++;
-    else stats.unmatched++;
-    if (result.status === "corrected") {
-      logger.info(
-        `Location resolution: corrected "${finding.title}" ${loc.line_range.start}-${loc.line_range.end} -> ${result.start}-${result.end}`
-      );
-      return {
-        ...finding,
-        code_location: { ...loc, line_range: { start: result.start, end: result.end } }
-      };
-    }
-    if (result.status === "unmatched") {
-      logger.warn(
-        `Location resolution: snippet not found for "${finding.title}" in ${loc.absolute_file_path}; keeping model range ${loc.line_range.start}-${loc.line_range.end}`
-      );
-    }
-    return finding;
-  });
-  return { review: { ...review, findings }, stats };
-}
-
-// src/system-prompt.ts
-var REVIEW_SYSTEM_PROMPT = `You are a code review agent. You analyze pull request diffs to find production bugs.
-
-<ROLE>
-* You are in READ-ONLY mode. Do NOT modify any files, create files, commit, or install dependencies.
-* Your only job is to analyze the diff, identify bugs, and produce a review.
-* Submit the final review via the \`submit_review\` tool. Do NOT output the final review as normal assistant text.
-* Be proportional: scale your analysis depth to the diff size. A small, single-file diff needs only a few iterations; a large multi-file refactor warrants deeper investigation.
-* Do NOT write to PLAN.md or AGENTS.md.
-* Do NOT run package managers (npm install, go mod download, pip install, etc.).
-* Follow the instructions in the user prompt exactly as given.
-</ROLE>
-
-<EFFICIENCY>
-* Combine multiple bash commands where possible (e.g. \`cmd1 && cmd2\`).
-* 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.
-* Keep reasoning proportional to the task. A small diff does not need extensive deliberation.
-</EFFICIENCY>`;
-
-// src/agent.ts
-function detectPlatform(prUrl) {
-  const url = new URL(prUrl);
-  const hostname = url.hostname;
-  if (prUrl.includes("/-/merge_requests/") || hostname.includes("gitlab")) {
-    return "gitlab";
-  }
-  if (prUrl.includes("/pulls/") || hostname.includes("gitea") || hostname.includes("forgejo") || hostname.includes("codeberg")) {
-    return "gitea";
-  }
-  if (prUrl.includes("/pull/") || hostname.includes("github")) {
-    return "github";
-  }
-  throw new Error(
-    `Cannot detect platform for URL: ${prUrl}. Expected a GitHub (/pull/), GitLab (/-/merge_requests/), or Gitea/Forgejo (/pulls/) URL.`
-  );
-}
-function parsePrUrl(prUrl) {
-  const url = new URL(prUrl);
-  const pathParts = url.pathname.split("/").filter(Boolean);
-  const host = url.host;
-  if (pathParts.length >= 4 && pathParts[2] === "pull") {
-    const prNumber = parseInt(pathParts[3], 10);
-    if (!Number.isSafeInteger(prNumber) || prNumber <= 0) {
-      throw new Error(`Invalid PR number in URL: ${prUrl}. Expected a positive integer after /pull/.`);
-    }
-    return {
-      owner: pathParts[0],
-      repo: pathParts[1],
-      prNumber,
-      host
-    };
-  }
-  if (pathParts.length >= 4 && pathParts[2] === "pulls") {
-    const prNumber = parseInt(pathParts[3], 10);
-    if (!Number.isSafeInteger(prNumber) || prNumber <= 0) {
-      throw new Error(`Invalid PR number in URL: ${prUrl}. Expected a positive integer after /pulls/.`);
-    }
-    return {
-      owner: pathParts[0],
-      repo: pathParts[1],
-      prNumber,
-      host
-    };
-  }
-  const mrIndex = pathParts.indexOf("merge_requests");
-  if (mrIndex >= 0) {
-    if (mrIndex < 2 || mrIndex + 1 >= pathParts.length) {
-      throw new Error(
-        `Invalid GitLab MR URL format: ${prUrl}. Expected .../-/merge_requests/<number>`
-      );
-    }
-    if (pathParts[mrIndex - 1] !== "-") {
-      throw new Error(
-        `Invalid GitLab MR URL format: ${prUrl}. Missing '/-/' segment before merge_requests.`
-      );
-    }
-    const repo = pathParts[mrIndex - 2];
-    const ownerParts = pathParts.slice(0, mrIndex - 2);
-    const owner = ownerParts.length > 0 ? ownerParts.join("/") : pathParts[0];
-    const prNumber = parseInt(pathParts[mrIndex + 1], 10);
-    if (!Number.isSafeInteger(prNumber) || prNumber <= 0) {
-      throw new Error(`Invalid MR number in URL: ${prUrl}. Expected a positive integer after /merge_requests/.`);
-    }
-    return { owner, repo, prNumber, host };
-  }
-  throw new Error(
-    `Invalid PR/MR URL format: ${prUrl}. Expected GitHub (/pull/), GitLab (/-/merge_requests/), or Gitea/Forgejo (/pulls/) URL.`
-  );
-}
-function formatLocationRelative(loc, workspacePath) {
-  return relativizeWorkspacePath(loc.absolute_file_path, workspacePath ?? void 0);
-}
-async function postGitlabReviewCommitStatus(parsed, review, diffRefs) {
-  const blocking = review.findings.filter((f) => f.priority <= 1).length;
-  const state = blocking > 0 ? "failed" : "success";
-  const description = blocking > 0 ? `${blocking} blocking issue(s) found` : review.findings.length > 0 ? `${review.findings.length} non-blocking issue(s)` : "No issues found";
-  await postGitlabCommitStatus(
-    parsed.owner,
-    parsed.repo,
-    diffRefs.head_sha,
-    state,
-    parsed.host,
-    { description }
-  );
-}
-async function postReviewComment(opts) {
-  const { prUrl, reviewText, model, metricsFooter, headSha } = opts;
-  const platform = detectPlatform(prUrl);
-  logger.info(`Posting comment to ${platform} PR/MR: ${prUrl}`);
-  let parsed;
-  try {
-    parsed = parsePrUrl(prUrl);
-  } catch (err) {
-    return { success: false, error: String(err) };
-  }
-  let body = reviewText;
-  if (headSha) {
-    body = `<!-- hodor:sha:${headSha} -->
-${body}`;
-  }
-  if (model) {
-    body = `${body}
-
----
-
-Review generated by Hodor (model: \`${model}\`)`;
-  }
-  if (metricsFooter) {
-    body = `${body}
-
-${metricsFooter}`;
-  }
-  try {
-    if (platform === "github") {
-      await exec("gh", [
-        "pr",
-        "review",
-        String(parsed.prNumber),
-        "--repo",
-        `${parsed.owner}/${parsed.repo}`,
-        "--comment",
-        "--body",
-        body
-      ]);
-      logger.info(`Successfully posted review to GitHub PR #${parsed.prNumber}`);
-      return { success: true, platform: "github", prNumber: parsed.prNumber };
-    } else if (platform === "gitea") {
-      await postGiteaPrComment(
-        parsed.owner,
-        parsed.repo,
-        parsed.prNumber,
-        body,
-        parsed.host
-      );
-      logger.info(`Successfully posted review to Gitea PR #${parsed.prNumber}`);
-      return { success: true, platform: "gitea", prNumber: parsed.prNumber };
-    } else {
-      await postGitlabMrComment(
-        parsed.owner,
-        parsed.repo,
-        parsed.prNumber,
-        body,
-        parsed.host
+      content = null;
+    }
+    fileCache.set(path, content);
+    return content;
+  };
+  const findings = review.findings.map((finding) => {
+    stats.total++;
+    const { existing_code: existingCode, code_location: loc } = finding;
+    if (!existingCode) {
+      stats.noSnippet++;
+      return finding;
+    }
+    if (workspaceRoot && !isWithinWorkspace(workspaceRoot, loc.absolute_file_path)) {
+      stats.unmatched++;
+      logger.warn(
+        `Location resolution: ${loc.absolute_file_path} is outside the workspace for "${finding.title}"; keeping model range`
       );
+      return finding;
+    }
+    const fileContent = readFile(loc.absolute_file_path);
+    if (fileContent === null) {
+      stats.unmatched++;
+      logger.warn(`Location resolution: could not read ${loc.absolute_file_path} for "${finding.title}"`);
+      return finding;
+    }
+    let changedLines;
+    for (const [relPath, lines] of changedByFile) {
+      if (loc.absolute_file_path.endsWith(`/${relPath}`) || loc.absolute_file_path === relPath) {
+        changedLines = lines;
+        break;
+      }
+    }
+    const result = resolveLineRange({
+      existingCode,
+      fileContent,
+      modelRange: loc.line_range,
+      changedLines
+    });
+    if (result.status === "confirmed") stats.confirmed++;
+    else if (result.status === "corrected") stats.corrected++;
+    else stats.unmatched++;
+    if (result.status === "corrected") {
       logger.info(
-        `Successfully posted review to GitLab MR !${parsed.prNumber}`
+        `Location resolution: corrected "${finding.title}" ${loc.line_range.start}-${loc.line_range.end} -> ${result.start}-${result.end}`
       );
       return {
-        success: true,
-        platform: "gitlab",
-        mrNumber: parsed.prNumber
+        ...finding,
+        code_location: { ...loc, line_range: { start: result.start, end: result.end } }
       };
     }
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    logger.error(`Failed to post comment: ${msg}`);
-    return { success: false, error: msg };
-  }
+    if (result.status === "unmatched") {
+      logger.warn(
+        `Location resolution: snippet not found for "${finding.title}" in ${loc.absolute_file_path}; keeping model range ${loc.line_range.start}-${loc.line_range.end}`
+      );
+    }
+    return finding;
+  });
+  return { review: { ...review, findings }, stats };
 }
+
+// src/system-prompt.ts
+var REVIEW_SYSTEM_PROMPT = `You are a code review agent. You analyze pull request diffs to find production bugs.
+
+<ROLE>
+* You are in READ-ONLY mode. Do NOT modify any files, create files, commit, or install dependencies.
+* Your only job is to analyze the diff, identify bugs, and produce a review.
+* Submit the final review via the \`submit_review\` tool. Do NOT output the final review as normal assistant text.
+* Be proportional: scale your analysis depth to the diff size. A small, single-file diff needs only a few iterations; a large multi-file refactor warrants deeper investigation.
+* Do NOT write to PLAN.md or AGENTS.md.
+* Do NOT run package managers (npm install, go mod download, pip install, etc.).
+* Follow the instructions in the user prompt exactly as given.
+</ROLE>
+
+<EFFICIENCY>
+* Combine multiple bash commands where possible (e.g. \`cmd1 && cmd2\`).
+* 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.
+* Keep reasoning proportional to the task. A small diff does not need extensive deliberation.
+</EFFICIENCY>`;
+
+// src/review-diff.ts
 var HODOR_REVIEW_SHA_RE = /^\s*<!--\s*hodor:sha:([a-f0-9]{40})\s*-->/i;
 function getHodorReviewShaCandidates(notes) {
   if (!notes || notes.length === 0) return [];
@@ -1534,14 +1757,7 @@ function getHodorReviewShaCandidates(notes) {
     if (a.createdAtMs == null && b.createdAtMs != null) return 1;
     return a.index - b.index;
   });
-  const seen = /* @__PURE__ */ new Set();
-  const shas = [];
-  for (const { sha } of candidates) {
-    if (seen.has(sha)) continue;
-    seen.add(sha);
-    shas.push(sha);
-  }
-  return shas;
+  return [...new Set(candidates.map(({ sha }) => sha))];
 }
 async function findLatestValidReviewSha(notes, workspacePath) {
   const candidates = getHodorReviewShaCandidates(notes);
@@ -1549,222 +1765,26 @@ async function findLatestValidReviewSha(notes, workspacePath) {
   logger.info(`Found ${candidates.length} previous Hodor review marker(s)`);
   for (const sha of candidates) {
     try {
-      const { stdout: objType } = await exec("git", ["cat-file", "-t", sha], { cwd: workspacePath });
-      if (objType.trim() !== "commit") throw new Error("not a commit");
-      await exec("git", ["merge-base", "--is-ancestor", sha, "HEAD"], { cwd: workspacePath });
+      const { 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;
     } catch {
-      logger.info(`Skipping previous review SHA ${sha.slice(0, 8)}; not a valid ancestor of HEAD`);
-    }
-  }
-  return null;
-}
-async function postReviewStructured(opts) {
-  const {
-    prUrl,
-    review,
-    model,
-    metricsFooter,
-    reviewStyle,
-    commitStatus,
-    codeQualityPath,
-    headSha,
-    workspacePath
-  } = opts;
-  const platform = detectPlatform(prUrl);
-  if (platform === "github") {
-    return postReviewComment({
-      prUrl,
-      reviewText: renderMarkdown(review),
-      model,
-      metricsFooter,
-      headSha
-    });
-  }
-  if (reviewStyle === "summary") {
-    return postReviewComment({
-      prUrl,
-      reviewText: renderMarkdown(review),
-      model,
-      metricsFooter,
-      headSha
-    });
-  }
-  const parsed = parsePrUrl(prUrl);
-  try {
-    const discussions = await listHodorDiscussions(
-      parsed.owner,
-      parsed.repo,
-      parsed.prNumber,
-      parsed.host
-    );
-    const unresolvedIds = [...new Set(
-      discussions.filter((d) => !d.resolved).map((d) => d.discussionId)
-    )];
-    if (unresolvedIds.length > 0) {
-      const resolved = await resolveGitlabDiscussions(
-        parsed.owner,
-        parsed.repo,
-        parsed.prNumber,
-        unresolvedIds,
-        parsed.host
-      );
-      if (resolved > 0) logger.info(`Resolved ${resolved} old Hodor discussion(s)`);
-    }
-  } catch (err) {
-    logger.warn(`Failed to resolve old discussions: ${err instanceof Error ? err.message : err}`);
-  }
-  let diffRefs = null;
-  try {
-    diffRefs = await getGitlabMrDiffRefs(
-      parsed.owner,
-      parsed.repo,
-      parsed.prNumber,
-      parsed.host
-    );
-  } catch (err) {
-    logger.warn(`Failed to get diff_refs, falling back to summary mode: ${err instanceof Error ? err.message : err}`);
-  }
-  if (!diffRefs) {
-    return postReviewComment({
-      prUrl,
-      reviewText: renderMarkdown(review),
-      model,
-      metricsFooter,
-      headSha
-    });
-  }
-  let inlineCount = 0;
-  let failedCount = 0;
-  let summaryPosted = false;
-  let draftsPublished = false;
-  let statusPosted = false;
-  const postingErrors = [];
-  for (const finding of review.findings) {
-    const relPath = formatLocationRelative(finding.code_location, workspacePath);
-    const priorityTag = `[P${finding.priority}]`;
-    const title = /^\[P[0-3]\]/.test(finding.title) ? finding.title : `${priorityTag} ${finding.title}`;
-    let body = `${HODOR_REVIEW_MARKER}
-**${title}**
-
-${finding.body}`;
-    if (finding.suggestion) {
-      const { start, end } = finding.code_location.line_range;
-      const span = Math.max(0, end - start);
-      body += `
-
-\`\`\`suggestion:-0+${span}
-${finding.suggestion}
-\`\`\``;
-    }
-    try {
-      await createGitlabDraftNote(
-        parsed.owner,
-        parsed.repo,
-        parsed.prNumber,
-        body,
-        parsed.host,
-        {
-          filePath: relPath,
-          line: finding.code_location.line_range.start,
-          diffRefs
-        }
-      );
-      inlineCount++;
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      logger.warn(`Failed to create inline note for "${finding.title}": ${msg}`);
-      postingErrors.push(`inline note: ${msg}`);
-      failedCount++;
-    }
-  }
-  logger.info(`Created ${inlineCount} inline draft note(s)${failedCount > 0 ? ` (${failedCount} failed)` : ""}`);
-  if (reviewStyle === "hybrid" || reviewStyle === void 0) {
-    let summaryBody = renderSummaryMarkdown(review);
-    if (headSha) summaryBody = `<!-- hodor:sha:${headSha} -->
-${summaryBody}`;
-    if (model) summaryBody += `
----
-
-Review generated by Hodor (model: \`${model}\`)`;
-    if (metricsFooter) summaryBody += `
-
-${metricsFooter}`;
-    try {
-      await postGitlabMrComment(
-        parsed.owner,
-        parsed.repo,
-        parsed.prNumber,
-        summaryBody,
-        parsed.host
-      );
-      summaryPosted = true;
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      logger.warn(`Failed to post summary comment: ${msg}`);
-      postingErrors.push(`summary comment: ${msg}`);
-    }
-  }
-  if (inlineCount > 0) {
-    try {
-      await bulkPublishGitlabDraftNotes(
-        parsed.owner,
-        parsed.repo,
-        parsed.prNumber,
-        parsed.host
+      logger.info(
+        `Skipping previous review SHA ${sha.slice(0, 8)}; not a valid ancestor of HEAD`
       );
-      logger.info("Published all draft notes");
-      draftsPublished = true;
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      logger.warn(`Failed to bulk publish draft notes: ${msg}`);
-      postingErrors.push(`draft publish: ${msg}`);
-    }
-  }
-  if (commitStatus && diffRefs) {
-    try {
-      await postGitlabReviewCommitStatus(parsed, review, diffRefs);
-      logger.info("Posted commit status");
-      statusPosted = true;
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      logger.warn(`Failed to post commit status: ${msg}`);
-      postingErrors.push(`commit status: ${msg}`);
-    }
-  }
-  if (codeQualityPath) {
-    try {
-      const { formatCodeQualityReport } = await import("./codequality-DTJK2LGF.js");
-      const report = formatCodeQualityReport(review, workspacePath ?? void 0);
-      const { writeFileSync } = await import("fs");
-      writeFileSync(codeQualityPath, report, "utf-8");
-      logger.info(`Wrote code quality report to ${codeQualityPath}`);
-    } catch (err) {
-      logger.warn(`Failed to write code quality report: ${err instanceof Error ? err.message : err}`);
     }
   }
-  const visibleResult = summaryPosted || inlineCount > 0 && draftsPublished || statusPosted;
-  const expectedInlineComments = reviewStyle === "inline" && review.findings.length > 0;
-  if (postingErrors.length > 0 && !visibleResult || expectedInlineComments && inlineCount === 0) {
-    return {
-      success: false,
-      platform: "gitlab",
-      mrNumber: parsed.prNumber,
-      error: postingErrors[0] ?? "No GitLab inline comments were created"
-    };
-  }
-  return {
-    success: true,
-    platform: "gitlab",
-    mrNumber: parsed.prNumber
-  };
+  return null;
 }
 var DIFF_SKIP_PATTERNS = [
   /(?:^|\/)testdata\//,
-  // test fixture directories
   /(?:^|\/)(?:package-lock\.json|yarn\.lock|pnpm-lock\.yaml|go\.sum|Cargo\.lock|poetry\.lock|Gemfile\.lock|composer\.lock)$/,
   /\.mdx?$/
-  // markdown docs
 ];
 function filterEmbeddedDiff(rawDiff) {
   const skippedFiles = [];
@@ -1777,7 +1797,7 @@ function filterEmbeddedDiff(rawDiff) {
       continue;
     }
     const filePath = match[1];
-    if (DIFF_SKIP_PATTERNS.some((re) => re.test(filePath))) {
+    if (DIFF_SKIP_PATTERNS.some((pattern) => pattern.test(filePath))) {
       skippedFiles.push(filePath);
     } else {
       kept.push(section);
@@ -1785,6 +1805,9 @@ function filterEmbeddedDiff(rawDiff) {
   }
   return { filtered: kept.join(""), skippedFiles };
 }
+
+// src/review-recovery.ts
+import { Value } from "@sinclair/typebox/value";
 var SUBMIT_REVIEW_RECOVERY_ATTEMPTS = 2;
 function buildSubmitReviewRecoveryPrompt(attempt, maxAttempts) {
   const finalAttempt = attempt >= maxAttempts ? "\nThis is the final automatic recovery attempt; do not end the turn without calling `submit_review`." : "";
@@ -1798,64 +1821,56 @@ function buildSubmitReviewRecoveryPrompt(attempt, maxAttempts) {
   ].filter(Boolean).join("\n");
 }
 function parseReviewFromAssistantText(text) {
-  const candidates = getJsonCandidates(text);
-  for (const candidate of candidates) {
+  for (const candidate of getJsonCandidates(text)) {
     try {
       const parsed = JSON.parse(candidate);
-      if (!Value.Check(SUBMIT_REVIEW_SCHEMA, parsed)) {
-        continue;
-      }
+      if (!Value.Check(SUBMIT_REVIEW_SCHEMA, parsed)) continue;
       return validateReviewOutput(parsed);
     } catch {
     }
   }
   return null;
 }
+function summarizeLastAssistantMessage(session) {
+  const messages = session.messages;
+  const lastAssistant = [...messages].reverse().find((message) => message.role === "assistant");
+  if (!lastAssistant) return "no assistant message";
+  const stopReason = typeof lastAssistant.stopReason === "string" ? lastAssistant.stopReason : "unknown";
+  const errorMessage = typeof lastAssistant.errorMessage === "string" ? `, error=${JSON.stringify(truncateForLog(lastAssistant.errorMessage, 300))}` : "";
+  const content = Array.isArray(lastAssistant.content) ? lastAssistant.content.map((item) => {
+    const block = item;
+    const type = typeof block.type === "string" ? block.type : "unknown";
+    return type === "toolCall" && typeof block.name === "string" ? `toolCall:${block.name}` : type;
+  }).join(",") : "unknown";
+  const rawText = session.getLastAssistantText()?.trim();
+  const textSummary = rawText ? `, text=${JSON.stringify(truncateForLog(rawText.replace(/\s+/g, " "), 500))}` : "";
+  return `stopReason=${stopReason}, content=[${content || "none"}]${errorMessage}${textSummary}`;
+}
 function getJsonCandidates(text) {
   const candidates = [];
   const seen = /* @__PURE__ */ new Set();
-  const addCandidate = (value) => {
+  const add = (value) => {
     const trimmed = value.trim();
     if (!trimmed || seen.has(trimmed)) return;
     seen.add(trimmed);
     candidates.push(trimmed);
   };
-  addCandidate(text);
-  const fencedJson = /```(?:json)?\s*([\s\S]*?)```/gi;
-  for (const match of text.matchAll(fencedJson)) {
-    addCandidate(match[1] ?? "");
+  add(text);
+  for (const match of text.matchAll(/```(?:json)?\s*([\s\S]*?)```/gi)) {
+    add(match[1] ?? "");
   }
   const firstBrace = text.indexOf("{");
   const lastBrace = text.lastIndexOf("}");
   if (firstBrace >= 0 && lastBrace > firstBrace) {
-    addCandidate(text.slice(firstBrace, lastBrace + 1));
+    add(text.slice(firstBrace, lastBrace + 1));
   }
   return candidates;
 }
-function summarizeLastAssistantMessage(session) {
-  const messages = session.messages;
-  const lastAssistant = [...messages].reverse().find((msg) => msg.role === "assistant");
-  if (!lastAssistant) {
-    return "no assistant message";
-  }
-  const stopReason = typeof lastAssistant.stopReason === "string" ? lastAssistant.stopReason : "unknown";
-  const errorMessage = typeof lastAssistant.errorMessage === "string" ? `, error=${JSON.stringify(truncateForLog(lastAssistant.errorMessage, 300))}` : "";
-  const content = Array.isArray(lastAssistant.content) ? lastAssistant.content.map((item) => {
-    const block = item;
-    const type = typeof block.type === "string" ? block.type : "unknown";
-    if (type === "toolCall" && typeof block.name === "string") {
-      return `toolCall:${block.name}`;
-    }
-    return type;
-  }).join(",") : "unknown";
-  const rawText = session.getLastAssistantText()?.trim();
-  const textSummary = rawText ? `, text=${JSON.stringify(truncateForLog(rawText.replace(/\s+/g, " "), 500))}` : "";
-  return `stopReason=${stopReason}, content=[${content || "none"}]${errorMessage}${textSummary}`;
-}
 function truncateForLog(text, maxLength) {
-  if (text.length <= maxLength) return text;
-  return `${text.slice(0, maxLength - 1)}\u2026`;
+  return text.length <= maxLength ? text : `${text.slice(0, maxLength - 1)}\u2026`;
 }
+
+// src/agent.ts
 async function reviewPr(opts) {
   const {
     prUrl,
@@ -2367,4 +2382,4 @@ export {
   postReviewStructured,
   reviewPr
 };
-//# sourceMappingURL=chunk-LQDQ7CHL.js.map
+//# sourceMappingURL=chunk-AIXGPWDG.js.map