glassbox 0.1.2 → 0.1.3

package/dist/cli.js

@@ -1,21 +1,10 @@
 #!/usr/bin/env node
 
-// src/server.ts
-import { Hono as Hono3 } from "hono";
-import { serve } from "@hono/node-server";
-import { exec } from "child_process";
-import { readFileSync as readFileSync3, existsSync as existsSync2 } from "fs";
-import { join as join3, dirname } from "path";
-import { fileURLToPath } from "url";
-
-// src/routes/api.ts
-import { Hono } from "hono";
-
 // src/db/connection.ts
 import { PGlite } from "@electric-sql/pglite";
-import { join } from "path";
-import { homedir } from "os";
 import { mkdirSync } from "fs";
+import { homedir } from "os";
+import { join } from "path";
 var dataDir = join(homedir(), ".glassbox", "data");
 mkdirSync(dataDir, { recursive: true });
 var db = null;
@@ -101,7 +90,7 @@ async function getReview(id) {
 }
 async function listReviews(repoPath) {
   const db2 = await getDb();
-  if (repoPath) {
+  if (repoPath !== void 0 && repoPath !== "") {
     const result2 = await db2.query(
       "SELECT * FROM reviews WHERE repo_path = $1 ORDER BY created_at DESC",
       [repoPath]
@@ -266,157 +255,16 @@ async function getStaleCountsForReview(reviewId) {
   return counts;
 }
 
-// src/export/generate.ts
-import { mkdirSync as mkdirSync2, writeFileSync, readFileSync, unlinkSync, existsSync, appendFileSync } from "fs";
-import { join as join2 } from "path";
-import { homedir as homedir2 } from "os";
-import { execSync } from "child_process";
-var DISMISS_FILE = join2(homedir2(), ".glassbox", "gitignore-dismissed.json");
-var DISMISS_DAYS = 30;
-function loadDismissals() {
-  try {
-    return JSON.parse(readFileSync(DISMISS_FILE, "utf-8"));
-  } catch {
-    return {};
-  }
-}
-function saveDismissals(data) {
-  const dir = join2(homedir2(), ".glassbox");
-  mkdirSync2(dir, { recursive: true });
-  writeFileSync(DISMISS_FILE, JSON.stringify(data), "utf-8");
-}
-function isGlassboxGitignored(repoRoot) {
-  try {
-    execSync("git check-ignore -q .glassbox", { cwd: repoRoot, stdio: "pipe" });
-    return true;
-  } catch {
-    return false;
-  }
-}
-function shouldPromptGitignore(repoRoot) {
-  if (isGlassboxGitignored(repoRoot)) return false;
-  const dismissals = loadDismissals();
-  const dismissed = dismissals[repoRoot];
-  if (dismissed) {
-    const daysSince = (Date.now() - dismissed) / (1e3 * 60 * 60 * 24);
-    if (daysSince < DISMISS_DAYS) return false;
-  }
-  return true;
-}
-function addGlassboxToGitignore(repoRoot) {
-  const gitignorePath = join2(repoRoot, ".gitignore");
-  if (existsSync(gitignorePath)) {
-    const content = readFileSync(gitignorePath, "utf-8");
-    if (!content.endsWith("\n")) {
-      appendFileSync(gitignorePath, "\n.glassbox/\n", "utf-8");
-    } else {
-      appendFileSync(gitignorePath, ".glassbox/\n", "utf-8");
-    }
-  } else {
-    writeFileSync(gitignorePath, ".glassbox/\n", "utf-8");
-  }
-}
-function dismissGitignorePrompt(repoRoot) {
-  const dismissals = loadDismissals();
-  dismissals[repoRoot] = Date.now();
-  saveDismissals(dismissals);
-}
-function deleteReviewExport(reviewId, repoRoot) {
-  const exportDir = join2(repoRoot, ".glassbox");
-  const archivePath = join2(exportDir, `review-${reviewId}.md`);
-  if (existsSync(archivePath)) unlinkSync(archivePath);
-}
-async function generateReviewExport(reviewId, repoRoot, isCurrent) {
-  const review = await getReview(reviewId);
-  if (!review) throw new Error("Review not found");
-  const files = await getReviewFiles(reviewId);
-  const annotations = await getAnnotationsForReview(reviewId);
-  const exportDir = join2(repoRoot, ".glassbox");
-  mkdirSync2(exportDir, { recursive: true });
-  const byFile = {};
-  for (const a of annotations) {
-    if (!byFile[a.file_path]) byFile[a.file_path] = [];
-    byFile[a.file_path].push(a);
-  }
-  const lines = [];
-  lines.push("# Code Review");
-  lines.push("");
-  lines.push(`- **Repository**: ${review.repo_name}`);
-  lines.push(`- **Review mode**: ${review.mode}${review.mode_args ? ` (${review.mode_args})` : ""}`);
-  lines.push(`- **Review ID**: ${review.id}`);
-  lines.push(`- **Date**: ${(/* @__PURE__ */ new Date()).toISOString()}`);
-  lines.push(`- **Files reviewed**: ${files.filter((f) => f.status === "reviewed").length}/${files.length}`);
-  lines.push(`- **Total annotations**: ${annotations.length}`);
-  lines.push("");
-  const categoryCounts = {};
-  for (const a of annotations) {
-    categoryCounts[a.category] = (categoryCounts[a.category] || 0) + 1;
-  }
-  if (Object.keys(categoryCounts).length > 0) {
-    lines.push("## Annotation Summary");
-    lines.push("");
-    for (const [cat, count] of Object.entries(categoryCounts).sort((a, b) => b[1] - a[1])) {
-      lines.push(`- **${cat}**: ${count}`);
-    }
-    lines.push("");
-  }
-  const rememberItems = annotations.filter((a) => a.category === "remember");
-  if (rememberItems.length > 0) {
-    lines.push("## Items to Remember");
-    lines.push("");
-    lines.push("> These annotations are flagged for long-term retention. AI tools should consider updating");
-    lines.push("> project configuration (CLAUDE.md, .cursorrules, etc.) with these preferences/rules.");
-    lines.push("");
-    for (const item of rememberItems) {
-      lines.push(`- **${item.file_path}:${item.line_number}** - ${item.content}`);
-    }
-    lines.push("");
-  }
-  lines.push("## File Annotations");
-  lines.push("");
-  for (const filePath of Object.keys(byFile).sort()) {
-    const fileAnns = byFile[filePath];
-    lines.push(`### ${filePath}`);
-    lines.push("");
-    for (const a of fileAnns) {
-      lines.push(`- **Line ${a.line_number}** [${a.category}]: ${a.content}`);
-    }
-    lines.push("");
-  }
-  lines.push("---");
-  lines.push("");
-  lines.push("## Instructions for AI Tools");
-  lines.push("");
-  lines.push("When processing this code review:");
-  lines.push("");
-  lines.push("1. **bug** and **fix** annotations indicate code that needs to be changed. Apply the suggested fixes.");
-  lines.push("2. **style** annotations indicate stylistic preferences. Apply them to the indicated lines and similar patterns nearby.");
-  lines.push("3. **pattern-follow** annotations highlight good patterns. Continue using these patterns in new code.");
-  lines.push("4. **pattern-avoid** annotations highlight anti-patterns. Refactor the indicated code and avoid the pattern elsewhere.");
-  lines.push("5. **remember** annotations are rules/preferences to persist. Update the project's AI configuration file (e.g., CLAUDE.md) with these.");
-  lines.push("6. **note** annotations are informational context. Consider them but they may not require code changes.");
-  lines.push("");
-  const content = lines.join("\n");
-  const archivePath = join2(exportDir, `review-${review.id}.md`);
-  writeFileSync(archivePath, content, "utf-8");
-  if (isCurrent) {
-    const latestPath = join2(exportDir, "latest-review.md");
-    writeFileSync(latestPath, content, "utf-8");
-    return latestPath;
-  }
-  return archivePath;
-}
-
 // src/git/diff.ts
-import { execSync as execSync2 } from "child_process";
-import { readFileSync as readFileSync2 } from "fs";
+import { execSync } from "child_process";
+import { readFileSync } from "fs";
 import { resolve } from "path";
 function git(args, cwd) {
   try {
-    return execSync2(`git ${args}`, { cwd, encoding: "utf-8", maxBuffer: 50 * 1024 * 1024 });
+    return execSync(`git ${args}`, { cwd, encoding: "utf-8", maxBuffer: 50 * 1024 * 1024 });
   } catch (e) {
     const err = e;
-    if (err.stdout) return err.stdout;
+    if (err.stdout !== void 0 && err.stdout !== "") return err.stdout;
     throw e;
   }
 }
@@ -425,7 +273,7 @@ function getRepoRoot(cwd) {
 }
 function getRepoName(cwd) {
   const root = getRepoRoot(cwd);
-  return root.split("/").pop() || "unknown";
+  return root.split("/").pop() ?? "unknown";
 }
 function isGitRepo(cwd) {
   try {
@@ -488,7 +336,7 @@ function getAllFiles(repoRoot) {
 function createNewFileDiff(filePath, repoRoot) {
   let content;
   try {
-    const buf = readFileSync2(resolve(repoRoot, filePath));
+    const buf = readFileSync(resolve(repoRoot, filePath));
     const checkLen = Math.min(buf.length, 8192);
     for (let i = 0; i < checkLen; i++) {
       if (buf[i] === 0) {
@@ -547,7 +395,7 @@ function parseDiff(raw2) {
     let status = "modified";
     if (header.includes("new file mode")) status = "added";
     else if (header.includes("deleted file mode")) status = "deleted";
-    else if (oldPath) status = "renamed";
+    else if (oldPath !== null) status = "renamed";
     const isBinary = header.includes("Binary file");
     if (isBinary) {
       files.push({ filePath, oldPath, status, hunks: [], isBinary: true });
@@ -564,12 +412,13 @@ function parseHunks(raw2) {
   let match;
   const hunkStarts = [];
   while ((match = hunkRegex.exec(raw2)) !== null) {
+    const groups = match;
     hunkStarts.push({
       index: match.index + match[0].length,
       oldStart: parseInt(match[1], 10),
-      oldCount: match[2] != null ? parseInt(match[2], 10) : 1,
+      oldCount: groups[2] !== void 0 ? parseInt(groups[2], 10) : 1,
       newStart: parseInt(match[3], 10),
-      newCount: match[4] != null ? parseInt(match[4], 10) : 1
+      newCount: groups[4] !== void 0 ? parseInt(groups[4], 10) : 1
     });
   }
   for (let i = 0; i < hunkStarts.length; i++) {
@@ -608,7 +457,7 @@ function getFileContent(filePath, ref, cwd) {
   const repoRoot = getRepoRoot(cwd);
   try {
     if (ref === "working") {
-      return execSync2(`cat "${resolve(repoRoot, filePath)}"`, { encoding: "utf-8", maxBuffer: 10 * 1024 * 1024 });
+      return execSync(`cat "${resolve(repoRoot, filePath)}"`, { encoding: "utf-8", maxBuffer: 10 * 1024 * 1024 });
     }
     return git(`show ${ref}:${filePath}`, repoRoot);
   } catch {
@@ -616,7 +465,7 @@ function getFileContent(filePath, ref, cwd) {
   }
 }
 function getHeadCommit(cwd) {
-  return execSync2("git rev-parse HEAD", { cwd, encoding: "utf-8" }).trim();
+  return execSync("git rev-parse HEAD", { cwd, encoding: "utf-8" }).trim();
 }
 function getModeString(mode) {
   switch (mode.type) {
@@ -648,11 +497,269 @@ function getModeArgs(mode) {
       return mode.name;
     case "files":
       return mode.patterns.join(",");
-    default:
+    case "uncommitted":
+    case "staged":
+    case "unstaged":
+    case "all":
       return void 0;
   }
 }
 
+// src/review-update.ts
+function findLineContent(diff, lineNumber, side) {
+  for (const hunk of diff.hunks) {
+    for (const line of hunk.lines) {
+      if (side === "old" && line.oldNum === lineNumber) return line.content;
+      if (side === "new" && line.newNum === lineNumber) return line.content;
+    }
+  }
+  return null;
+}
+function findMatchingLine(diff, content, origLineNum, side, radius = 10) {
+  let bestMatch = null;
+  let bestDistance = Infinity;
+  for (const hunk of diff.hunks) {
+    for (const line of hunk.lines) {
+      if (line.content !== content) continue;
+      const lineNum = side === "old" ? line.oldNum : line.newNum;
+      if (lineNum == null) continue;
+      const distance = Math.abs(lineNum - origLineNum);
+      if (distance <= radius && distance < bestDistance) {
+        bestDistance = distance;
+        bestMatch = { lineNumber: lineNum, side };
+      }
+    }
+  }
+  return bestMatch;
+}
+async function migrateAnnotations(annotations, oldDiff, newDiff) {
+  let staleCount = 0;
+  for (const annotation of annotations) {
+    const oldContent = findLineContent(oldDiff, annotation.line_number, annotation.side);
+    if (oldContent === null) {
+      if (!annotation.is_stale) {
+        await markAnnotationStale(annotation.id, null);
+        staleCount++;
+      }
+      continue;
+    }
+    const match = findMatchingLine(newDiff, oldContent, annotation.line_number, annotation.side);
+    if (match) {
+      if (match.lineNumber !== annotation.line_number || match.side !== annotation.side) {
+        await moveAnnotation(annotation.id, match.lineNumber, match.side);
+      } else if (annotation.is_stale) {
+        await markAnnotationCurrent(annotation.id);
+      }
+    } else {
+      if (!annotation.is_stale) {
+        await markAnnotationStale(annotation.id, oldContent);
+        staleCount++;
+      }
+    }
+  }
+  return staleCount;
+}
+async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
+  const existingFiles = await getReviewFiles(reviewId);
+  const existingByPath = /* @__PURE__ */ new Map();
+  for (const f of existingFiles) {
+    existingByPath.set(f.file_path, f);
+  }
+  const newDiffsByPath = /* @__PURE__ */ new Map();
+  for (const d of newDiffs) {
+    newDiffsByPath.set(d.filePath, d);
+  }
+  let updated = 0;
+  let added = 0;
+  let stale = 0;
+  for (const [path, existingFile] of existingByPath) {
+    const newDiff = newDiffsByPath.get(path);
+    if (newDiff) {
+      const oldDiff = JSON.parse(existingFile.diff_data ?? "{}");
+      const annotations = await getAnnotationsForFile(existingFile.id);
+      if (annotations.length > 0) {
+        stale += await migrateAnnotations(annotations, oldDiff, newDiff);
+      }
+      await updateFileDiff(existingFile.id, JSON.stringify(newDiff));
+      updated++;
+    } else {
+      const annotations = await getAnnotationsForFile(existingFile.id);
+      if (annotations.length === 0) {
+        await deleteReviewFile(existingFile.id);
+      } else {
+        const oldDiff = JSON.parse(existingFile.diff_data ?? "{}");
+        for (const a of annotations) {
+          if (!a.is_stale) {
+            const content = findLineContent(oldDiff, a.line_number, a.side);
+            await markAnnotationStale(a.id, content);
+            stale++;
+          }
+        }
+      }
+    }
+  }
+  for (const [path, diff] of newDiffsByPath) {
+    if (!existingByPath.has(path)) {
+      await addReviewFile(reviewId, path, JSON.stringify(diff));
+      added++;
+    }
+  }
+  await updateReviewHead(reviewId, headCommit);
+  return { updated, added, stale };
+}
+
+// src/server.ts
+import { serve } from "@hono/node-server";
+import { exec } from "child_process";
+import { existsSync as existsSync2, readFileSync as readFileSync3 } from "fs";
+import { Hono as Hono3 } from "hono";
+import { dirname, join as join3 } from "path";
+import { fileURLToPath } from "url";
+
+// src/routes/api.ts
+import { Hono } from "hono";
+
+// src/export/generate.ts
+import { execSync as execSync2 } from "child_process";
+import { appendFileSync, existsSync, mkdirSync as mkdirSync2, readFileSync as readFileSync2, unlinkSync, writeFileSync } from "fs";
+import { homedir as homedir2 } from "os";
+import { join as join2 } from "path";
+var DISMISS_FILE = join2(homedir2(), ".glassbox", "gitignore-dismissed.json");
+var DISMISS_DAYS = 30;
+function loadDismissals() {
+  try {
+    return JSON.parse(readFileSync2(DISMISS_FILE, "utf-8"));
+  } catch {
+    return {};
+  }
+}
+function saveDismissals(data) {
+  const dir = join2(homedir2(), ".glassbox");
+  mkdirSync2(dir, { recursive: true });
+  writeFileSync(DISMISS_FILE, JSON.stringify(data), "utf-8");
+}
+function isGlassboxGitignored(repoRoot) {
+  try {
+    execSync2("git check-ignore -q .glassbox", { cwd: repoRoot, stdio: "pipe" });
+    return true;
+  } catch {
+    return false;
+  }
+}
+function shouldPromptGitignore(repoRoot) {
+  if (isGlassboxGitignored(repoRoot)) return false;
+  const dismissals = loadDismissals();
+  const dismissed = dismissals[repoRoot];
+  if (dismissed) {
+    const daysSince = (Date.now() - dismissed) / (1e3 * 60 * 60 * 24);
+    if (daysSince < DISMISS_DAYS) return false;
+  }
+  return true;
+}
+function addGlassboxToGitignore(repoRoot) {
+  const gitignorePath = join2(repoRoot, ".gitignore");
+  if (existsSync(gitignorePath)) {
+    const content = readFileSync2(gitignorePath, "utf-8");
+    if (!content.endsWith("\n")) {
+      appendFileSync(gitignorePath, "\n.glassbox/\n", "utf-8");
+    } else {
+      appendFileSync(gitignorePath, ".glassbox/\n", "utf-8");
+    }
+  } else {
+    writeFileSync(gitignorePath, ".glassbox/\n", "utf-8");
+  }
+}
+function dismissGitignorePrompt(repoRoot) {
+  const dismissals = loadDismissals();
+  dismissals[repoRoot] = Date.now();
+  saveDismissals(dismissals);
+}
+function deleteReviewExport(reviewId, repoRoot) {
+  const exportDir = join2(repoRoot, ".glassbox");
+  const archivePath = join2(exportDir, `review-${reviewId}.md`);
+  if (existsSync(archivePath)) unlinkSync(archivePath);
+}
+async function generateReviewExport(reviewId, repoRoot, isCurrent) {
+  const review = await getReview(reviewId);
+  if (!review) throw new Error("Review not found");
+  const files = await getReviewFiles(reviewId);
+  const annotations = await getAnnotationsForReview(reviewId);
+  const exportDir = join2(repoRoot, ".glassbox");
+  mkdirSync2(exportDir, { recursive: true });
+  const byFile = {};
+  for (const a of annotations) {
+    if (!(a.file_path in byFile)) byFile[a.file_path] = [];
+    byFile[a.file_path].push(a);
+  }
+  const lines = [];
+  lines.push("# Code Review");
+  lines.push("");
+  lines.push(`- **Repository**: ${review.repo_name}`);
+  lines.push(`- **Review mode**: ${review.mode}${review.mode_args !== null && review.mode_args !== "" ? ` (${review.mode_args})` : ""}`);
+  lines.push(`- **Review ID**: ${review.id}`);
+  lines.push(`- **Date**: ${(/* @__PURE__ */ new Date()).toISOString()}`);
+  lines.push(`- **Files reviewed**: ${files.filter((f) => f.status === "reviewed").length}/${files.length}`);
+  lines.push(`- **Total annotations**: ${annotations.length}`);
+  lines.push("");
+  const categoryCounts = {};
+  for (const a of annotations) {
+    categoryCounts[a.category] = (categoryCounts[a.category] || 0) + 1;
+  }
+  if (Object.keys(categoryCounts).length > 0) {
+    lines.push("## Annotation Summary");
+    lines.push("");
+    for (const [cat, count] of Object.entries(categoryCounts).sort((a, b) => b[1] - a[1])) {
+      lines.push(`- **${cat}**: ${count}`);
+    }
+    lines.push("");
+  }
+  const rememberItems = annotations.filter((a) => a.category === "remember");
+  if (rememberItems.length > 0) {
+    lines.push("## Items to Remember");
+    lines.push("");
+    lines.push("> These annotations are flagged for long-term retention. AI tools should consider updating");
+    lines.push("> project configuration (CLAUDE.md, .cursorrules, etc.) with these preferences/rules.");
+    lines.push("");
+    for (const item of rememberItems) {
+      lines.push(`- **${item.file_path}:${item.line_number}** - ${item.content}`);
+    }
+    lines.push("");
+  }
+  lines.push("## File Annotations");
+  lines.push("");
+  for (const filePath of Object.keys(byFile).sort()) {
+    const fileAnns = byFile[filePath];
+    lines.push(`### ${filePath}`);
+    lines.push("");
+    for (const a of fileAnns) {
+      lines.push(`- **Line ${a.line_number}** [${a.category}]: ${a.content}`);
+    }
+    lines.push("");
+  }
+  lines.push("---");
+  lines.push("");
+  lines.push("## Instructions for AI Tools");
+  lines.push("");
+  lines.push("When processing this code review:");
+  lines.push("");
+  lines.push("1. **bug** and **fix** annotations indicate code that needs to be changed. Apply the suggested fixes.");
+  lines.push("2. **style** annotations indicate stylistic preferences. Apply them to the indicated lines and similar patterns nearby.");
+  lines.push("3. **pattern-follow** annotations highlight good patterns. Continue using these patterns in new code.");
+  lines.push("4. **pattern-avoid** annotations highlight anti-patterns. Refactor the indicated code and avoid the pattern elsewhere.");
+  lines.push("5. **remember** annotations are rules/preferences to persist. Update the project's AI configuration file (e.g., CLAUDE.md) with these.");
+  lines.push("6. **note** annotations are informational context. Consider them but they may not require code changes.");
+  lines.push("");
+  const content = lines.join("\n");
+  const archivePath = join2(exportDir, `review-${review.id}.md`);
+  writeFileSync(archivePath, content, "utf-8");
+  if (isCurrent) {
+    const latestPath = join2(exportDir, "latest-review.md");
+    writeFileSync(latestPath, content, "utf-8");
+    return latestPath;
+  }
+  return archivePath;
+}
+
 // src/outline/parser.ts
 var BRACE_LANGS = /* @__PURE__ */ new Set([
   "javascript",
@@ -715,7 +822,7 @@ function langFromPath(filePath) {
 }
 function parseOutline(content, filePath) {
   const lang = langFromPath(filePath);
-  if (!lang) return [];
+  if (lang === null) return [];
   if (BRACE_LANGS.has(lang)) return parseBraces(content, lang);
   if (INDENT_LANGS.has(lang)) return parseIndent(content, lang);
   return [];
@@ -809,7 +916,6 @@ function parseBraces(content, lang) {
   let braceDepth = 0;
   let inString = false;
   let stringChar = "";
-  let inLineComment = false;
   let inBlockComment = false;
   let inTemplateLiteral = false;
   const { top: topFuncPatterns, method: methodFuncPatterns } = getFuncPatterns(lang);
@@ -848,7 +954,6 @@ function parseBraces(content, lang) {
     for (let j = 0; j < line.length; j++) {
       const ch = line[j];
       const next = line[j + 1];
-      if (inLineComment) break;
       if (inBlockComment) {
         if (ch === "*" && next === "/") {
           inBlockComment = false;
@@ -877,7 +982,6 @@ function parseBraces(content, lang) {
         continue;
       }
       if (ch === "/" && next === "/") {
-        inLineComment = true;
         break;
       }
       if (ch === "/" && next === "*") {
@@ -899,11 +1003,12 @@ function parseBraces(content, lang) {
         braceDepth--;
         while (stack.length > 0 && stack[stack.length - 1].depth >= braceDepth) {
           const closed = stack.pop();
-          closed.symbol.endLine = lineNum;
+          if (closed !== void 0) {
+            closed.symbol.endLine = lineNum;
+          }
         }
       }
     }
-    inLineComment = false;
   }
   const lastLine = lines.length;
   for (const item of stack) {
@@ -950,7 +1055,9 @@ function parseIndent(content, lang) {
 function pushIndentSymbol(root, stack, sym, indent, lines, lineIdx) {
   while (stack.length > 0 && stack[stack.length - 1].indent >= indent) {
     const closed = stack.pop();
-    closed.symbol.endLine = lineIdx;
+    if (closed !== void 0) {
+      closed.symbol.endLine = lineIdx;
+    }
   }
   let endLine = lines.length;
   for (let j = lineIdx + 1; j < lines.length; j++) {
@@ -974,7 +1081,7 @@ function pushIndentSymbol(root, stack, sym, indent, lines, lineIdx) {
 // src/routes/api.ts
 var apiRoutes = new Hono();
 function resolveReviewId(c) {
-  return c.req.query("reviewId") || c.get("reviewId");
+  return c.req.query("reviewId") ?? c.get("reviewId");
 }
 apiRoutes.get("/reviews", async (c) => {
   const repoRoot = c.get("repoRoot");
@@ -996,12 +1103,12 @@ apiRoutes.post("/review/complete", async (c) => {
   const gitignorePrompt = shouldPromptGitignore(repoRoot);
   return c.json({ status: "completed", exportPath, isCurrent, reviewId, gitignorePrompt });
 });
-apiRoutes.post("/gitignore/add", async (c) => {
+apiRoutes.post("/gitignore/add", (c) => {
   const repoRoot = c.get("repoRoot");
   addGlassboxToGitignore(repoRoot);
   return c.json({ ok: true });
 });
-apiRoutes.post("/gitignore/dismiss", async (c) => {
+apiRoutes.post("/gitignore/dismiss", (c) => {
   const repoRoot = c.get("repoRoot");
   dismissGitignorePrompt(repoRoot);
   return c.json({ ok: true });
@@ -1114,7 +1221,7 @@ apiRoutes.get("/outline/:fileId", async (c) => {
   const repoRoot = c.get("repoRoot");
   const file = await getReviewFile(c.req.param("fileId"));
   if (!file) return c.json({ error: "Not found" }, 404);
-  const diff = JSON.parse(file.diff_data || "{}");
+  const diff = JSON.parse(file.diff_data ?? "{}");
   const isDeleted = diff.status === "deleted";
   let content = "";
   try {
@@ -1133,8 +1240,8 @@ apiRoutes.get("/context/:fileId", async (c) => {
   const repoRoot = c.get("repoRoot");
   const file = await getReviewFile(c.req.param("fileId"));
   if (!file) return c.json({ error: "Not found" }, 404);
-  const startLine = parseInt(c.req.query("start") || "1", 10);
-  const endLine = parseInt(c.req.query("end") || "20", 10);
+  const startLine = parseInt(c.req.query("start") ?? "1", 10);
+  const endLine = parseInt(c.req.query("end") ?? "20", 10);
   const content = getFileContent(file.file_path, "working", repoRoot);
   const allLines = content.split("\n");
   const clampedStart = Math.max(1, startLine);
@@ -1195,126 +1302,27 @@ function renderChildren(children) {
 }
 function renderAttr(key, value) {
   if (value == null || value === false) return "";
-  if (value === true) return ` ${key}`;
-  const name = key === "className" ? "class" : key === "htmlFor" ? "for" : key;
-  let strValue;
-  if (value instanceof SafeHtml) {
-    strValue = value.__html;
-  } else if (typeof value === "number") {
-    strValue = String(value);
-  } else if (typeof value === "string") {
-    strValue = escapeAttr(value);
-  } else {
-    strValue = "";
-  }
-  return ` ${name}="${strValue}"`;
-}
-function jsx(tag, props) {
-  if (typeof tag === "function") return tag(props);
-  const { children, ...attrs } = props;
-  const attrStr = Object.entries(attrs).map(([k, v]) => renderAttr(k, v)).join("");
-  if (VOID_TAGS.has(tag)) return new SafeHtml(`<${tag}${attrStr}>`);
-  const childStr = children != null ? renderChildren(children) : "";
-  return new SafeHtml(`<${tag}${attrStr}>${childStr}</${tag}>`);
-}
-
-// src/components/layout.tsx
-function Layout({ title, reviewId, children }) {
-  return /* @__PURE__ */ jsx("html", { lang: "en", children: [
-    /* @__PURE__ */ jsx("head", { children: [
-      /* @__PURE__ */ jsx("meta", { charset: "utf-8" }),
-      /* @__PURE__ */ jsx("meta", { name: "viewport", content: "width=device-width, initial-scale=1" }),
-      /* @__PURE__ */ jsx("title", { children: title }),
-      /* @__PURE__ */ jsx("link", { rel: "stylesheet", href: "/static/styles.css" })
-    ] }),
-    /* @__PURE__ */ jsx("body", { "data-review-id": reviewId, children: [
-      children,
-      /* @__PURE__ */ jsx("script", { src: "/static/app.js" })
-    ] })
-  ] });
-}
-
-// src/components/fileList.tsx
-function buildFileTree(files) {
-  const root = { name: "", children: [], files: [] };
-  for (const f of files) {
-    const parts = f.file_path.split("/");
-    let node = root;
-    for (let i = 0; i < parts.length - 1; i++) {
-      let child = node.children.find((c) => c.name === parts[i]);
-      if (!child) {
-        child = { name: parts[i], children: [], files: [] };
-        node.children.push(child);
-      }
-      node = child;
-    }
-    node.files.push(f);
-  }
-  compressTree(root);
-  return root;
-}
-function compressTree(node) {
-  for (let i = 0; i < node.children.length; i++) {
-    let child = node.children[i];
-    while (child.children.length === 1 && child.files.length === 0) {
-      const grandchild = child.children[0];
-      child = { name: child.name + "/" + grandchild.name, children: grandchild.children, files: grandchild.files };
-      node.children[i] = child;
-    }
-    compressTree(child);
-  }
-}
-function countFiles(node) {
-  let count = node.files.length;
-  for (const child of node.children) count += countFiles(child);
-  return count;
-}
-function hasStale(node, staleCounts) {
-  for (const f of node.files) {
-    if (staleCounts[f.id]) return true;
-  }
-  for (const child of node.children) {
-    if (hasStale(child, staleCounts)) return true;
+  if (value === true) return ` ${key}`;
+  const name = key === "className" ? "class" : key === "htmlFor" ? "for" : key;
+  let strValue;
+  if (value instanceof SafeHtml) {
+    strValue = value.__html;
+  } else if (typeof value === "number") {
+    strValue = String(value);
+  } else if (typeof value === "string") {
+    strValue = escapeAttr(value);
+  } else {
+    strValue = "";
   }
-  return false;
-}
-function TreeView({ node, depth, annotationCounts, staleCounts }) {
-  const sortedChildren = [...node.children].sort((a, b) => a.name.localeCompare(b.name));
-  return /* @__PURE__ */ jsx("div", { children: [
-    sortedChildren.map((child) => {
-      const total = countFiles(child);
-      const isCollapsible = total > 1;
-      const stale = hasStale(child, staleCounts);
-      return /* @__PURE__ */ jsx("div", { className: "folder-group", children: [
-        /* @__PURE__ */ jsx("div", { className: `folder-header${isCollapsible ? " collapsible" : ""}`, style: `padding-left:${16 + depth * 12}px`, children: [
-          isCollapsible ? /* @__PURE__ */ jsx("span", { className: "folder-arrow", children: "\u25BE" }) : /* @__PURE__ */ jsx("span", { className: "folder-arrow-spacer" }),
-          /* @__PURE__ */ jsx("span", { className: "folder-name", children: [
-            child.name,
-            "/"
-          ] }),
-          stale ? /* @__PURE__ */ jsx("span", { className: "stale-dot" }) : null
-        ] }),
-        /* @__PURE__ */ jsx("div", { className: "folder-content", children: /* @__PURE__ */ jsx(TreeView, { node: child, depth: depth + 1, annotationCounts, staleCounts }) })
-      ] });
-    }),
-    node.files.map((f) => {
-      const diff = JSON.parse(f.diff_data || "{}");
-      const count = annotationCounts[f.id] || 0;
-      const stale = staleCounts[f.id] || 0;
-      const fileName = f.file_path.split("/").pop();
-      return /* @__PURE__ */ jsx("div", { className: "file-item", "data-file-id": f.id, style: `padding-left:${16 + depth * 12}px`, children: [
-        /* @__PURE__ */ jsx("span", { className: `status-dot ${f.status}` }),
-        /* @__PURE__ */ jsx("span", { className: "file-name", title: f.file_path, children: fileName }),
-        /* @__PURE__ */ jsx("span", { className: `file-status ${diff.status || ""}`, children: diff.status || "" }),
-        stale > 0 ? /* @__PURE__ */ jsx("span", { className: "stale-dot" }) : null,
-        count > 0 ? /* @__PURE__ */ jsx("span", { className: "annotation-count", children: count }) : null
-      ] });
-    })
-  ] });
+  return ` ${name}="${strValue}"`;
 }
-function FileList({ files, annotationCounts, staleCounts }) {
-  const tree = buildFileTree(files);
-  return /* @__PURE__ */ jsx("div", { className: "file-list", children: /* @__PURE__ */ jsx("div", { className: "file-list-items", children: /* @__PURE__ */ jsx(TreeView, { node: tree, depth: 0, annotationCounts, staleCounts }) }) });
+function jsx(tag, props) {
+  if (typeof tag === "function") return tag(props);
+  const { children, ...attrs } = props;
+  const attrStr = Object.entries(attrs).map(([k, v]) => renderAttr(k, v)).join("");
+  if (VOID_TAGS.has(tag)) return new SafeHtml(`<${tag}${attrStr}>`);
+  const childStr = children != null ? renderChildren(children) : "";
+  return new SafeHtml(`<${tag}${attrStr}>${childStr}</${tag}>`);
 }
 
 // src/components/diffView.tsx
@@ -1322,7 +1330,7 @@ function DiffView({ file, diff, annotations, mode }) {
   const annotationsByLine = {};
   for (const a of annotations) {
     const key = `${a.line_number}:${a.side}`;
-    if (!annotationsByLine[key]) annotationsByLine[key] = [];
+    if (!(key in annotationsByLine)) annotationsByLine[key] = [];
     annotationsByLine[key].push(a);
   }
   return /* @__PURE__ */ jsx("div", { className: "diff-view", "data-file-id": file.id, "data-file-path": file.file_path, children: [
@@ -1360,8 +1368,8 @@ function SplitDiff({ hunks, annotationsByLine }) {
         }
       ),
       pairs.map((pair) => {
-        const leftAnns = pair.left ? annotationsByLine[`${pair.left.oldNum}:old`] || [] : [];
-        const rightAnns = pair.right ? annotationsByLine[`${pair.right.newNum}:new`] || [] : [];
+        const leftAnns = pair.left ? annotationsByLine[`${pair.left.oldNum}:old`] ?? [] : [];
+        const rightAnns = pair.right ? annotationsByLine[`${pair.right.newNum}:new`] ?? [] : [];
         const allAnns = [...leftAnns, ...rightAnns];
         return /* @__PURE__ */ jsx("div", { children: [
           /* @__PURE__ */ jsx("div", { className: "split-row", children: [
@@ -1422,10 +1430,8 @@ function pairLines(lines) {
           right: j < adds.length ? adds[j] : null
         });
       }
-    } else if (line.type === "add") {
-      pairs.push({ left: null, right: line });
-      i++;
     } else {
+      pairs.push({ left: null, right: line });
       i++;
     }
   }
@@ -1458,7 +1464,7 @@ function UnifiedDiff({ hunks, annotationsByLine }) {
     hunk.lines.map((line) => {
       const lineNum = line.type === "remove" ? line.oldNum : line.newNum;
       const side = line.type === "remove" ? "old" : "new";
-      const anns = annotationsByLine[`${lineNum}:${side}`] || [];
+      const anns = annotationsByLine[`${lineNum}:${side}`] ?? [];
       return /* @__PURE__ */ jsx("div", { children: [
         /* @__PURE__ */ jsx(
           "div",
@@ -1499,6 +1505,105 @@ function AnnotationRows({ annotations }) {
   )) });
 }
 
+// src/components/fileList.tsx
+function buildFileTree(files) {
+  const root = { name: "", children: [], files: [] };
+  for (const f of files) {
+    const parts = f.file_path.split("/");
+    let node = root;
+    for (let i = 0; i < parts.length - 1; i++) {
+      let child = node.children.find((c) => c.name === parts[i]);
+      if (!child) {
+        child = { name: parts[i], children: [], files: [] };
+        node.children.push(child);
+      }
+      node = child;
+    }
+    node.files.push(f);
+  }
+  compressTree(root);
+  return root;
+}
+function compressTree(node) {
+  for (let i = 0; i < node.children.length; i++) {
+    let child = node.children[i];
+    while (child.children.length === 1 && child.files.length === 0) {
+      const grandchild = child.children[0];
+      child = { name: child.name + "/" + grandchild.name, children: grandchild.children, files: grandchild.files };
+      node.children[i] = child;
+    }
+    compressTree(child);
+  }
+}
+function countFiles(node) {
+  let count = node.files.length;
+  for (const child of node.children) count += countFiles(child);
+  return count;
+}
+function hasStale(node, staleCounts) {
+  for (const f of node.files) {
+    if (staleCounts[f.id]) return true;
+  }
+  for (const child of node.children) {
+    if (hasStale(child, staleCounts)) return true;
+  }
+  return false;
+}
+function TreeView({ node, depth, annotationCounts, staleCounts }) {
+  const sortedChildren = [...node.children].sort((a, b) => a.name.localeCompare(b.name));
+  return /* @__PURE__ */ jsx("div", { children: [
+    sortedChildren.map((child) => {
+      const total = countFiles(child);
+      const isCollapsible = total > 1;
+      const stale = hasStale(child, staleCounts);
+      return /* @__PURE__ */ jsx("div", { className: "folder-group", children: [
+        /* @__PURE__ */ jsx("div", { className: `folder-header${isCollapsible ? " collapsible" : ""}`, style: `padding-left:${16 + depth * 12}px`, children: [
+          isCollapsible ? /* @__PURE__ */ jsx("span", { className: "folder-arrow", children: "\u25BE" }) : /* @__PURE__ */ jsx("span", { className: "folder-arrow-spacer" }),
+          /* @__PURE__ */ jsx("span", { className: "folder-name", children: [
+            child.name,
+            "/"
+          ] }),
+          stale ? /* @__PURE__ */ jsx("span", { className: "stale-dot" }) : null
+        ] }),
+        /* @__PURE__ */ jsx("div", { className: "folder-content", children: /* @__PURE__ */ jsx(TreeView, { node: child, depth: depth + 1, annotationCounts, staleCounts }) })
+      ] });
+    }),
+    node.files.map((f) => {
+      const diff = JSON.parse(f.diff_data ?? "{}");
+      const count = annotationCounts[f.id] || 0;
+      const stale = staleCounts[f.id] || 0;
+      const fileName = f.file_path.split("/").pop() ?? "";
+      return /* @__PURE__ */ jsx("div", { className: "file-item", "data-file-id": f.id, style: `padding-left:${16 + depth * 12}px`, children: [
+        /* @__PURE__ */ jsx("span", { className: `status-dot ${f.status}` }),
+        /* @__PURE__ */ jsx("span", { className: "file-name", title: f.file_path, children: fileName }),
+        /* @__PURE__ */ jsx("span", { className: `file-status ${diff.status ?? ""}`, children: diff.status ?? "" }),
+        stale > 0 ? /* @__PURE__ */ jsx("span", { className: "stale-dot" }) : null,
+        count > 0 ? /* @__PURE__ */ jsx("span", { className: "annotation-count", children: count }) : null
+      ] });
+    })
+  ] });
+}
+function FileList({ files, annotationCounts, staleCounts }) {
+  const tree = buildFileTree(files);
+  return /* @__PURE__ */ jsx("div", { className: "file-list", children: /* @__PURE__ */ jsx("div", { className: "file-list-items", children: /* @__PURE__ */ jsx(TreeView, { node: tree, depth: 0, annotationCounts, staleCounts }) }) });
+}
+
+// src/components/layout.tsx
+function Layout({ title, reviewId, children }) {
+  return /* @__PURE__ */ jsx("html", { lang: "en", children: [
+    /* @__PURE__ */ jsx("head", { children: [
+      /* @__PURE__ */ jsx("meta", { charset: "utf-8" }),
+      /* @__PURE__ */ jsx("meta", { name: "viewport", content: "width=device-width, initial-scale=1" }),
+      /* @__PURE__ */ jsx("title", { children: title }),
+      /* @__PURE__ */ jsx("link", { rel: "stylesheet", href: "/static/styles.css" })
+    ] }),
+    /* @__PURE__ */ jsx("body", { "data-review-id": reviewId, children: [
+      children,
+      /* @__PURE__ */ jsx("script", { src: "/static/app.js" })
+    ] })
+  ] });
+}
+
 // src/components/reviewHistory.tsx
 function titleCase(s) {
   return s.replace(/[_-]/g, " ").replace(/\b\w/g, (c) => c.toUpperCase());
@@ -1506,15 +1611,15 @@ function titleCase(s) {
 var trashIcon = '<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M3 6h18"/><path d="M19 6v14c0 1-1 2-2 2H7c-1 0-2-1-2-2V6"/><path d="M8 6V4c0-1 1-2 2-2h4c1 0 2 1 2 2v2"/><line x1="10" y1="11" x2="10" y2="17"/><line x1="14" y1="11" x2="14" y2="17"/></svg>';
 function shortenArgs(args) {
   const shaPattern = /\b([0-9a-f]{7,40})\b/gi;
-  let hasLong = false;
+  const result = { hasLong: false };
   const short = args.replace(shaPattern, (match) => {
     if (match.length > 8) {
-      hasLong = true;
+      result.hasLong = true;
       return match.slice(0, 7);
     }
     return match;
   });
-  return { short, full: hasLong ? args : "" };
+  return { short, full: result.hasLong ? args : "" };
 }
 function ReviewHistory({ reviews, currentReviewId }) {
   const hasOtherReviews = reviews.some((r) => r.id !== currentReviewId);
@@ -1525,9 +1630,9 @@ function ReviewHistory({ reviews, currentReviewId }) {
       const isCurrent = r.id === currentReviewId;
       const href = isCurrent ? "/" : `/review/${r.id}`;
       let argsDisplay = null;
-      if (r.mode_args) {
+      if (r.mode_args !== null && r.mode_args !== "") {
         const { short, full } = shortenArgs(r.mode_args);
-        argsDisplay = full ? /* @__PURE__ */ jsx("span", { title: full, children: [
+        argsDisplay = full !== "" ? /* @__PURE__ */ jsx("span", { title: full, children: [
           ": ",
           short
         ] }) : /* @__PURE__ */ jsx("span", { children: [
@@ -1660,7 +1765,7 @@ pageRoutes.get("/", async (c) => {
         /* @__PURE__ */ jsx("h2", { children: review.repo_name }),
         /* @__PURE__ */ jsx("span", { className: "review-mode", children: [
           review.mode,
-          review.mode_args ? `: ${review.mode_args}` : ""
+          review.mode_args !== null && review.mode_args !== "" ? `: ${review.mode_args}` : ""
         ] })
       ] }),
       /* @__PURE__ */ jsx("div", { className: "file-filter", children: /* @__PURE__ */ jsx("input", { type: "text", className: "file-filter-input", id: "file-filter", placeholder: "Filter files..." }) }),
@@ -1701,7 +1806,7 @@ pageRoutes.get("/file/:fileId", async (c) => {
   const file = await getReviewFile(fileId);
   if (!file) return c.text("File not found", 404);
   const annotations = await getAnnotationsForFile(fileId);
-  const diff = JSON.parse(file.diff_data || "{}");
+  const diff = JSON.parse(file.diff_data ?? "{}");
   const html = /* @__PURE__ */ jsx(DiffView, { file, diff, annotations, mode });
   return c.html(html.toString());
 });
@@ -1725,7 +1830,7 @@ pageRoutes.get("/review/:reviewId", async (c) => {
         /* @__PURE__ */ jsx("h2", { children: review.repo_name }),
         /* @__PURE__ */ jsx("span", { className: "review-mode", children: [
           review.mode,
-          review.mode_args ? `: ${review.mode_args}` : ""
+          review.mode_args !== null && review.mode_args !== "" ? `: ${review.mode_args}` : ""
         ] })
       ] }),
       /* @__PURE__ */ jsx("div", { className: "file-filter", children: /* @__PURE__ */ jsx("input", { type: "text", className: "file-filter-input", id: "file-filter", placeholder: "Filter files..." }) }),
@@ -1773,7 +1878,9 @@ pageRoutes.get("/history", async (c) => {
 function tryServe(fetch, port) {
   return new Promise((resolve2, reject) => {
     const server = serve({ fetch, port });
-    server.on("listening", () => resolve2(port));
+    server.on("listening", () => {
+      resolve2(port);
+    });
     server.on("error", (err) => {
       if (err.code === "EADDRINUSE") {
         reject(err);
@@ -1809,7 +1916,7 @@ async function startServer(port, reviewId, repoRoot) {
       actualPort = await tryServe(app.fetch, port + attempt);
       break;
     } catch (err) {
-      if (err.code === "EADDRINUSE" && attempt < 19) {
+      if (err instanceof Error && err.code === "EADDRINUSE" && attempt < 19) {
         continue;
       }
       throw err;
@@ -1827,10 +1934,10 @@ async function startServer(port, reviewId, repoRoot) {
 }
 
 // src/update-check.ts
-import { readFileSync as readFileSync4, writeFileSync as writeFileSync2, mkdirSync as mkdirSync3, existsSync as existsSync3 } from "fs";
-import { join as join4, dirname as dirname2 } from "path";
-import { homedir as homedir3 } from "os";
+import { existsSync as existsSync3, mkdirSync as mkdirSync3, readFileSync as readFileSync4, writeFileSync as writeFileSync2 } from "fs";
 import { get } from "https";
+import { homedir as homedir3 } from "os";
+import { dirname as dirname2, join as join4 } from "path";
 import { fileURLToPath as fileURLToPath2 } from "url";
 var DATA_DIR = join4(homedir3(), ".glassbox");
 var CHECK_FILE = join4(DATA_DIR, "last-update-check");
@@ -1859,7 +1966,7 @@ function saveCheckDate() {
 }
 function isFirstUseToday() {
   const last = getLastCheckDate();
-  if (!last) return true;
+  if (last === null) return true;
   const today = (/* @__PURE__ */ new Date()).toISOString().slice(0, 10);
   return last !== today;
 }
@@ -1872,7 +1979,7 @@ function fetchLatestVersion() {
       }
       let data = "";
       res.on("data", (chunk) => {
-        data += chunk;
+        data += chunk.toString();
       });
       res.on("end", () => {
         try {
@@ -1882,7 +1989,9 @@ function fetchLatestVersion() {
         }
       });
     });
-    req.on("error", () => resolve2(null));
+    req.on("error", () => {
+      resolve2(null);
+    });
     req.on("timeout", () => {
       req.destroy();
       resolve2(null);
@@ -1916,7 +2025,7 @@ async function checkForUpdates(force) {
   const current = getCurrentVersion();
   const latest = await fetchLatestVersion();
   saveCheckDate();
-  if (!latest || compareVersions(current, latest) >= 0) return;
+  if (latest === null || compareVersions(current, latest) >= 0) return;
   const cmd = detectUpgradeCommand();
   const updateLine = `Update available: ${current} \u2192 ${latest}`;
   const cmdLine = `Run: ${cmd}`;
@@ -1935,109 +2044,6 @@ async function checkForUpdates(force) {
   console.log("");
 }
 
-// src/review-update.ts
-function findLineContent(diff, lineNumber, side) {
-  for (const hunk of diff.hunks || []) {
-    for (const line of hunk.lines) {
-      if (side === "old" && line.oldNum === lineNumber) return line.content;
-      if (side === "new" && line.newNum === lineNumber) return line.content;
-    }
-  }
-  return null;
-}
-function findMatchingLine(diff, content, origLineNum, side, radius = 10) {
-  let bestMatch = null;
-  let bestDistance = Infinity;
-  for (const hunk of diff.hunks || []) {
-    for (const line of hunk.lines) {
-      if (line.content !== content) continue;
-      const lineNum = side === "old" ? line.oldNum : line.newNum;
-      if (lineNum == null) continue;
-      const distance = Math.abs(lineNum - origLineNum);
-      if (distance <= radius && distance < bestDistance) {
-        bestDistance = distance;
-        bestMatch = { lineNumber: lineNum, side };
-      }
-    }
-  }
-  return bestMatch;
-}
-async function migrateAnnotations(annotations, oldDiff, newDiff) {
-  let staleCount = 0;
-  for (const annotation of annotations) {
-    const oldContent = findLineContent(oldDiff, annotation.line_number, annotation.side);
-    if (!oldContent) {
-      if (!annotation.is_stale) {
-        await markAnnotationStale(annotation.id, null);
-        staleCount++;
-      }
-      continue;
-    }
-    const match = findMatchingLine(newDiff, oldContent, annotation.line_number, annotation.side);
-    if (match) {
-      if (match.lineNumber !== annotation.line_number || match.side !== annotation.side) {
-        await moveAnnotation(annotation.id, match.lineNumber, match.side);
-      } else if (annotation.is_stale) {
-        await markAnnotationCurrent(annotation.id);
-      }
-    } else {
-      if (!annotation.is_stale) {
-        await markAnnotationStale(annotation.id, oldContent);
-        staleCount++;
-      }
-    }
-  }
-  return staleCount;
-}
-async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
-  const existingFiles = await getReviewFiles(reviewId);
-  const existingByPath = /* @__PURE__ */ new Map();
-  for (const f of existingFiles) {
-    existingByPath.set(f.file_path, f);
-  }
-  const newDiffsByPath = /* @__PURE__ */ new Map();
-  for (const d of newDiffs) {
-    newDiffsByPath.set(d.filePath, d);
-  }
-  let updated = 0;
-  let added = 0;
-  let stale = 0;
-  for (const [path, existingFile] of existingByPath) {
-    const newDiff = newDiffsByPath.get(path);
-    if (newDiff) {
-      const oldDiff = JSON.parse(existingFile.diff_data || "{}");
-      const annotations = await getAnnotationsForFile(existingFile.id);
-      if (annotations.length > 0) {
-        stale += await migrateAnnotations(annotations, oldDiff, newDiff);
-      }
-      await updateFileDiff(existingFile.id, JSON.stringify(newDiff));
-      updated++;
-    } else {
-      const annotations = await getAnnotationsForFile(existingFile.id);
-      if (annotations.length === 0) {
-        await deleteReviewFile(existingFile.id);
-      } else {
-        const oldDiff = JSON.parse(existingFile.diff_data || "{}");
-        for (const a of annotations) {
-          if (!a.is_stale) {
-            const content = findLineContent(oldDiff, a.line_number, a.side);
-            await markAnnotationStale(a.id, content);
-            stale++;
-          }
-        }
-      }
-    }
-  }
-  for (const [path, diff] of newDiffsByPath) {
-    if (!existingByPath.has(path)) {
-      await addReviewFile(reviewId, path, JSON.stringify(diff));
-      added++;
-    }
-  }
-  await updateReviewHead(reviewId, headCommit);
-  return { updated, added, stale };
-}
-
 // src/cli.ts
 function printUsage() {
   console.log(`
@@ -2084,6 +2090,7 @@ function parseArgs(argv) {
       case "-h":
         printUsage();
         process.exit(0);
+      // falls through
       case "--uncommitted":
         mode = { type: "uncommitted" };
         break;
@@ -2141,7 +2148,7 @@ async function main() {
   }
   const { mode, port, resume, forceUpdateCheck, debug } = parsed;
   if (debug) {
-    console.log(`Build timestamp: ${"2026-03-06T01:38:17.060Z"}`);
+    console.log(`Build timestamp: ${"2026-03-06T05:23:12.429Z"}`);
   }
   await checkForUpdates(forceUpdateCheck);
   const cwd = process.cwd();