glassbox 0.1.2 → 0.1.4

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);
@@ -1218,21 +1325,196 @@ function jsx(tag, props) {
   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" })
+// src/components/diffView.tsx
+function DiffView({ file, diff, annotations, mode }) {
+  const annotationsByLine = {};
+  for (const a of annotations) {
+    const key = `${a.line_number}:${a.side}`;
+    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: [
+    /* @__PURE__ */ jsx("div", { className: "diff-header", children: [
+      /* @__PURE__ */ jsx("span", { className: "file-path", children: diff.filePath }),
+      /* @__PURE__ */ jsx("div", { className: "diff-header-actions", children: /* @__PURE__ */ jsx("span", { className: `file-status ${diff.status}`, children: diff.status }) })
     ] }),
-    /* @__PURE__ */ jsx("body", { "data-review-id": reviewId, children: [
-      children,
-      /* @__PURE__ */ jsx("script", { src: "/static/app.js" })
-    ] })
+    diff.isBinary ? /* @__PURE__ */ jsx("div", { className: "hunk-separator", children: "Binary file" }) : diff.status === "added" || diff.status === "deleted" || mode === "unified" ? /* @__PURE__ */ jsx(UnifiedDiff, { hunks: diff.hunks, annotationsByLine }) : /* @__PURE__ */ jsx(SplitDiff, { hunks: diff.hunks, annotationsByLine })
+  ] });
+}
+function SplitDiff({ hunks, annotationsByLine }) {
+  const lastHunk = hunks[hunks.length - 1];
+  const tailStart = lastHunk ? lastHunk.newStart + lastHunk.newCount : 1;
+  return /* @__PURE__ */ jsx("div", { className: "diff-table-split", children: [
+    hunks.map((hunk, hunkIdx) => {
+      const pairs = pairLines(hunk.lines);
+      return /* @__PURE__ */ jsx("div", { className: "hunk-block", children: [
+        /* @__PURE__ */ jsx(
+          "div",
+          {
+            className: "hunk-separator",
+            "data-hunk-idx": hunkIdx,
+            "data-old-start": hunk.oldStart,
+            "data-old-count": hunk.oldCount,
+            "data-new-start": hunk.newStart,
+            "data-new-count": hunk.newCount,
+            children: [
+              "@@ -",
+              hunk.oldStart,
+              ",",
+              hunk.oldCount,
+              " +",
+              hunk.newStart,
+              ",",
+              hunk.newCount,
+              " @@"
+            ]
+          }
+        ),
+        pairs.map((pair) => {
+          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: [
+              /* @__PURE__ */ jsx(
+                "div",
+                {
+                  className: `diff-line split-left ${pair.left?.type || "empty"}`,
+                  "data-line": pair.left?.oldNum ?? "",
+                  "data-side": "old",
+                  "data-new-line": pair.left?.newNum ?? pair.right?.newNum ?? "",
+                  children: [
+                    /* @__PURE__ */ jsx("span", { className: "gutter", children: pair.left?.oldNum ?? "" }),
+                    /* @__PURE__ */ jsx("span", { className: "code", children: pair.left ? raw(escapeHtml(pair.left.content)) : "" })
+                  ]
+                }
+              ),
+              /* @__PURE__ */ jsx(
+                "div",
+                {
+                  className: `diff-line split-right ${pair.right?.type || "empty"}`,
+                  "data-line": pair.right?.newNum ?? "",
+                  "data-side": "new",
+                  children: [
+                    /* @__PURE__ */ jsx("span", { className: "gutter", children: pair.right?.newNum ?? "" }),
+                    /* @__PURE__ */ jsx("span", { className: "code", children: pair.right ? raw(escapeHtml(pair.right.content)) : "" })
+                  ]
+                }
+              )
+            ] }),
+            allAnns.length > 0 ? /* @__PURE__ */ jsx(AnnotationRows, { annotations: allAnns }) : null
+          ] });
+        })
+      ] });
+    }),
+    /* @__PURE__ */ jsx("div", { className: "hunk-separator hunk-expander-tail", "data-start": tailStart, children: "\u2195 Show remaining lines" })
+  ] });
+}
+function pairLines(lines) {
+  const pairs = [];
+  let i = 0;
+  while (i < lines.length) {
+    const line = lines[i];
+    if (line.type === "context") {
+      pairs.push({ left: line, right: line });
+      i++;
+    } else if (line.type === "remove") {
+      const removes = [];
+      while (i < lines.length && lines[i].type === "remove") {
+        removes.push(lines[i]);
+        i++;
+      }
+      const adds = [];
+      while (i < lines.length && lines[i].type === "add") {
+        adds.push(lines[i]);
+        i++;
+      }
+      const max = Math.max(removes.length, adds.length);
+      for (let j = 0; j < max; j++) {
+        pairs.push({
+          left: j < removes.length ? removes[j] : null,
+          right: j < adds.length ? adds[j] : null
+        });
+      }
+    } else {
+      pairs.push({ left: null, right: line });
+      i++;
+    }
+  }
+  return pairs;
+}
+function UnifiedDiff({ hunks, annotationsByLine }) {
+  const lastHunk = hunks[hunks.length - 1];
+  const tailStart = lastHunk ? lastHunk.newStart + lastHunk.newCount : 1;
+  return /* @__PURE__ */ jsx("div", { className: "diff-table-unified", children: [
+    hunks.map((hunk, hunkIdx) => /* @__PURE__ */ jsx("div", { className: "hunk-block", children: [
+      /* @__PURE__ */ jsx(
+        "div",
+        {
+          className: "hunk-separator",
+          "data-hunk-idx": hunkIdx,
+          "data-old-start": hunk.oldStart,
+          "data-old-count": hunk.oldCount,
+          "data-new-start": hunk.newStart,
+          "data-new-count": hunk.newCount,
+          children: [
+            "@@ -",
+            hunk.oldStart,
+            ",",
+            hunk.oldCount,
+            " +",
+            hunk.newStart,
+            ",",
+            hunk.newCount,
+            " @@"
+          ]
+        }
+      ),
+      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}`] ?? [];
+        return /* @__PURE__ */ jsx("div", { children: [
+          /* @__PURE__ */ jsx(
+            "div",
+            {
+              className: `diff-line ${line.type}${anns.length ? " has-annotation" : ""}`,
+              "data-line": lineNum,
+              "data-side": side,
+              children: [
+                /* @__PURE__ */ jsx("span", { className: "gutter-old", children: line.oldNum ?? "" }),
+                /* @__PURE__ */ jsx("span", { className: "gutter-new", children: line.newNum ?? "" }),
+                /* @__PURE__ */ jsx("span", { className: "code", children: raw(escapeHtml(line.content)) })
+              ]
+            }
+          ),
+          anns.length > 0 ? /* @__PURE__ */ jsx(AnnotationRows, { annotations: anns }) : null
+        ] });
+      })
+    ] })),
+    /* @__PURE__ */ jsx("div", { className: "hunk-separator hunk-expander-tail", "data-start": tailStart, children: "\u2195 Show remaining lines" })
   ] });
 }
+function AnnotationRows({ annotations }) {
+  return /* @__PURE__ */ jsx("div", { className: "annotation-row", children: annotations.map((a) => /* @__PURE__ */ jsx(
+    "div",
+    {
+      className: `annotation-item${a.is_stale ? " annotation-stale" : ""}`,
+      "data-annotation-id": a.id,
+      "data-is-stale": a.is_stale ? "true" : void 0,
+      children: [
+        /* @__PURE__ */ jsx("span", { className: "annotation-drag-handle", draggable: "true", title: "Drag to move", children: "\u283F" }),
+        /* @__PURE__ */ jsx("span", { className: `annotation-category category-${a.category}`, "data-action": "reclassify", children: a.category }),
+        /* @__PURE__ */ jsx("span", { className: "annotation-text", children: a.content }),
+        /* @__PURE__ */ jsx("div", { className: "annotation-actions", children: [
+          a.is_stale ? /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-keep", "data-action": "keep", children: "Keep" }) : null,
+          /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-icon", "data-action": "edit", title: "Edit", children: raw('<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M17 3a2.85 2.85 0 1 1 4 4L7.5 20.5 2 22l1.5-5.5Z"/><path d="m15 5 4 4"/></svg>') }),
+          /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-icon btn-danger", "data-action": "delete", title: "Delete", children: raw('<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" 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>') })
+        ] })
+      ]
+    }
+  )) });
+}
 
 // src/components/fileList.tsx
 function buildFileTree(files) {
@@ -1298,14 +1580,14 @@ function TreeView({ node, depth, annotationCounts, staleCounts }) {
       ] });
     }),
     node.files.map((f) => {
-      const diff = JSON.parse(f.diff_data || "{}");
+      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();
+      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 || "" }),
+        /* @__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
       ] });
@@ -1317,187 +1599,21 @@ function FileList({ files, annotationCounts, staleCounts }) {
   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/diffView.tsx
-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] = [];
-    annotationsByLine[key].push(a);
-  }
-  return /* @__PURE__ */ jsx("div", { className: "diff-view", "data-file-id": file.id, "data-file-path": file.file_path, children: [
-    /* @__PURE__ */ jsx("div", { className: "diff-header", children: [
-      /* @__PURE__ */ jsx("span", { className: "file-path", children: diff.filePath }),
-      /* @__PURE__ */ jsx("div", { className: "diff-header-actions", children: /* @__PURE__ */ jsx("span", { className: `file-status ${diff.status}`, children: diff.status }) })
+// 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" })
     ] }),
-    diff.isBinary ? /* @__PURE__ */ jsx("div", { className: "hunk-separator", children: "Binary file" }) : diff.status === "added" || diff.status === "deleted" || mode === "unified" ? /* @__PURE__ */ jsx(UnifiedDiff, { hunks: diff.hunks, annotationsByLine }) : /* @__PURE__ */ jsx(SplitDiff, { hunks: diff.hunks, annotationsByLine })
+    /* @__PURE__ */ jsx("body", { "data-review-id": reviewId, children: [
+      children,
+      /* @__PURE__ */ jsx("script", { src: "/static/app.js" })
+    ] })
   ] });
 }
-function SplitDiff({ hunks, annotationsByLine }) {
-  return /* @__PURE__ */ jsx("div", { className: "diff-table-split", children: hunks.map((hunk, hunkIdx) => {
-    const pairs = pairLines(hunk.lines);
-    return /* @__PURE__ */ jsx("div", { className: "hunk-block", children: [
-      /* @__PURE__ */ jsx(
-        "div",
-        {
-          className: "hunk-separator",
-          "data-hunk-idx": hunkIdx,
-          "data-old-start": hunk.oldStart,
-          "data-old-count": hunk.oldCount,
-          "data-new-start": hunk.newStart,
-          "data-new-count": hunk.newCount,
-          children: [
-            "@@ -",
-            hunk.oldStart,
-            ",",
-            hunk.oldCount,
-            " +",
-            hunk.newStart,
-            ",",
-            hunk.newCount,
-            " @@"
-          ]
-        }
-      ),
-      pairs.map((pair) => {
-        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: [
-            /* @__PURE__ */ jsx(
-              "div",
-              {
-                className: `diff-line split-left ${pair.left?.type || "empty"}`,
-                "data-line": pair.left?.oldNum ?? "",
-                "data-side": "old",
-                children: [
-                  /* @__PURE__ */ jsx("span", { className: "gutter", children: pair.left?.oldNum ?? "" }),
-                  /* @__PURE__ */ jsx("span", { className: "code", children: pair.left ? raw(escapeHtml(pair.left.content)) : "" })
-                ]
-              }
-            ),
-            /* @__PURE__ */ jsx(
-              "div",
-              {
-                className: `diff-line split-right ${pair.right?.type || "empty"}`,
-                "data-line": pair.right?.newNum ?? "",
-                "data-side": "new",
-                children: [
-                  /* @__PURE__ */ jsx("span", { className: "gutter", children: pair.right?.newNum ?? "" }),
-                  /* @__PURE__ */ jsx("span", { className: "code", children: pair.right ? raw(escapeHtml(pair.right.content)) : "" })
-                ]
-              }
-            )
-          ] }),
-          allAnns.length > 0 ? /* @__PURE__ */ jsx(AnnotationRows, { annotations: allAnns }) : null
-        ] });
-      })
-    ] });
-  }) });
-}
-function pairLines(lines) {
-  const pairs = [];
-  let i = 0;
-  while (i < lines.length) {
-    const line = lines[i];
-    if (line.type === "context") {
-      pairs.push({ left: line, right: line });
-      i++;
-    } else if (line.type === "remove") {
-      const removes = [];
-      while (i < lines.length && lines[i].type === "remove") {
-        removes.push(lines[i]);
-        i++;
-      }
-      const adds = [];
-      while (i < lines.length && lines[i].type === "add") {
-        adds.push(lines[i]);
-        i++;
-      }
-      const max = Math.max(removes.length, adds.length);
-      for (let j = 0; j < max; j++) {
-        pairs.push({
-          left: j < removes.length ? removes[j] : null,
-          right: j < adds.length ? adds[j] : null
-        });
-      }
-    } else if (line.type === "add") {
-      pairs.push({ left: null, right: line });
-      i++;
-    } else {
-      i++;
-    }
-  }
-  return pairs;
-}
-function UnifiedDiff({ hunks, annotationsByLine }) {
-  return /* @__PURE__ */ jsx("div", { className: "diff-table-unified", children: hunks.map((hunk, hunkIdx) => /* @__PURE__ */ jsx("div", { className: "hunk-block", children: [
-    /* @__PURE__ */ jsx(
-      "div",
-      {
-        className: "hunk-separator",
-        "data-hunk-idx": hunkIdx,
-        "data-old-start": hunk.oldStart,
-        "data-old-count": hunk.oldCount,
-        "data-new-start": hunk.newStart,
-        "data-new-count": hunk.newCount,
-        children: [
-          "@@ -",
-          hunk.oldStart,
-          ",",
-          hunk.oldCount,
-          " +",
-          hunk.newStart,
-          ",",
-          hunk.newCount,
-          " @@"
-        ]
-      }
-    ),
-    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}`] || [];
-      return /* @__PURE__ */ jsx("div", { children: [
-        /* @__PURE__ */ jsx(
-          "div",
-          {
-            className: `diff-line ${line.type}${anns.length ? " has-annotation" : ""}`,
-            "data-line": lineNum,
-            "data-side": side,
-            children: [
-              /* @__PURE__ */ jsx("span", { className: "gutter-old", children: line.oldNum ?? "" }),
-              /* @__PURE__ */ jsx("span", { className: "gutter-new", children: line.newNum ?? "" }),
-              /* @__PURE__ */ jsx("span", { className: "code", children: raw(escapeHtml(line.content)) })
-            ]
-          }
-        ),
-        anns.length > 0 ? /* @__PURE__ */ jsx(AnnotationRows, { annotations: anns }) : null
-      ] });
-    })
-  ] })) });
-}
-function AnnotationRows({ annotations }) {
-  return /* @__PURE__ */ jsx("div", { className: "annotation-row", children: annotations.map((a) => /* @__PURE__ */ jsx(
-    "div",
-    {
-      className: `annotation-item${a.is_stale ? " annotation-stale" : ""}`,
-      "data-annotation-id": a.id,
-      "data-is-stale": a.is_stale ? "true" : void 0,
-      children: [
-        /* @__PURE__ */ jsx("span", { className: "annotation-drag-handle", draggable: "true", title: "Drag to move", children: "\u283F" }),
-        /* @__PURE__ */ jsx("span", { className: `annotation-category category-${a.category}`, "data-action": "reclassify", children: a.category }),
-        /* @__PURE__ */ jsx("span", { className: "annotation-text", children: a.content }),
-        /* @__PURE__ */ jsx("div", { className: "annotation-actions", children: [
-          a.is_stale ? /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-keep", "data-action": "keep", children: "Keep" }) : null,
-          /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-icon", "data-action": "edit", title: "Edit", children: raw('<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M17 3a2.85 2.85 0 1 1 4 4L7.5 20.5 2 22l1.5-5.5Z"/><path d="m15 5 4 4"/></svg>') }),
-          /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-icon btn-danger", "data-action": "delete", title: "Delete", children: raw('<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" 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>') })
-        ] })
-      ]
-    }
-  )) });
-}
 
 // src/components/reviewHistory.tsx
 function titleCase(s) {
@@ -1506,15 +1622,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 +1641,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 +1776,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 +1817,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 +1841,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 +1889,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 +1927,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 +1945,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 +1977,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 +1990,7 @@ function fetchLatestVersion() {
       }
       let data = "";
       res.on("data", (chunk) => {
-        data += chunk;
+        data += chunk.toString();
       });
       res.on("end", () => {
         try {
@@ -1882,7 +2000,9 @@ function fetchLatestVersion() {
         }
       });
     });
-    req.on("error", () => resolve2(null));
+    req.on("error", () => {
+      resolve2(null);
+    });
     req.on("timeout", () => {
       req.destroy();
       resolve2(null);
@@ -1916,7 +2036,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 +2055,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 +2101,7 @@ function parseArgs(argv) {
       case "-h":
         printUsage();
         process.exit(0);
+      // falls through
       case "--uncommitted":
         mode = { type: "uncommitted" };
         break;
@@ -2141,7 +2159,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-06T22:43:31.204Z"}`);
   }
   await checkForUpdates(forceUpdateCheck);
   const cwd = process.cwd();