glassbox 0.8.1 → 0.8.2-rc.1

package/dist/cli.js

@@ -395,7 +395,7 @@ init_connection();
 init_queries();
 import { mkdirSync as mkdirSync9, realpathSync } from "fs";
 import { tmpdir } from "os";
-import { join as join13, resolve as resolve6 } from "path";
+import { join as join13, resolve as resolve7 } from "path";
 
 // src/debug.ts
 var debugEnabled = false;
@@ -425,10 +425,37 @@ function debugLog(...args) {
   }
 }
 
-// src/ai/config.ts
+// src/global-config.ts
 import { chmodSync, existsSync, mkdirSync as mkdirSync2, readFileSync, writeFileSync } from "fs";
 import { homedir } from "os";
 import { join as join2 } from "path";
+var GLOBAL_CONFIG_DIR = join2(homedir(), ".glassbox");
+var GLOBAL_CONFIG_PATH = join2(GLOBAL_CONFIG_DIR, "config.json");
+function readGlobalConfig() {
+  try {
+    if (existsSync(GLOBAL_CONFIG_PATH)) {
+      const parsed = JSON.parse(readFileSync(GLOBAL_CONFIG_PATH, "utf-8"));
+      if (typeof parsed === "object" && parsed !== null && !Array.isArray(parsed)) {
+        return parsed;
+      }
+    }
+  } catch {
+  }
+  return {};
+}
+function writeGlobalConfig(config) {
+  mkdirSync2(GLOBAL_CONFIG_DIR, { recursive: true });
+  writeFileSync(GLOBAL_CONFIG_PATH, JSON.stringify(config, null, 2), "utf-8");
+  try {
+    chmodSync(GLOBAL_CONFIG_PATH, 384);
+  } catch {
+  }
+}
+function updateGlobalConfig(mutator) {
+  const cfg = readGlobalConfig();
+  const result = mutator(cfg);
+  writeGlobalConfig(result === void 0 ? cfg : result);
+}
 
 // src/ai/api-keys.ts
 import { spawnSync as spawnSync2 } from "child_process";
@@ -589,11 +616,12 @@ function saveAPIKey(platform, key, storage) {
   if (storage === "keychain") {
     saveKeyToKeychain(platform, key);
   } else {
-    const config = readConfigFile();
-    if (config.ai === void 0) config.ai = {};
-    if (config.ai.keys === void 0) config.ai.keys = {};
-    config.ai.keys[platform] = Buffer.from(key).toString("base64");
-    writeConfigFile(config);
+    updateGlobalConfig((raw) => {
+      const cfg = raw;
+      if (cfg.ai === void 0) cfg.ai = {};
+      if (cfg.ai.keys === void 0) cfg.ai.keys = {};
+      cfg.ai.keys[platform] = Buffer.from(key).toString("base64");
+    });
   }
 }
 function deleteAPIKey(platform) {
@@ -610,11 +638,13 @@ function deleteAPIKey(platform) {
     }
   } catch {
   }
-  const config = readConfigFile();
-  if (config.ai?.keys !== void 0) {
-    config.ai.keys[platform] = "";
-    writeConfigFile(config);
-  }
+  if (readConfigFile().ai?.keys === void 0) return;
+  updateGlobalConfig((raw) => {
+    const cfg = raw;
+    if (cfg.ai?.keys !== void 0) {
+      cfg.ai.keys[platform] = "";
+    }
+  });
 }
 function detectAvailablePlatforms() {
   const results = [];
@@ -628,24 +658,8 @@ function detectAvailablePlatforms() {
 }
 
 // src/ai/config.ts
-var CONFIG_DIR = join2(homedir(), ".glassbox");
-var CONFIG_PATH = join2(CONFIG_DIR, "config.json");
 function readConfigFile() {
-  try {
-    if (existsSync(CONFIG_PATH)) {
-      return JSON.parse(readFileSync(CONFIG_PATH, "utf-8"));
-    }
-  } catch {
-  }
-  return {};
-}
-function writeConfigFile(config) {
-  mkdirSync2(CONFIG_DIR, { recursive: true });
-  writeFileSync(CONFIG_PATH, JSON.stringify(config, null, 2), "utf-8");
-  try {
-    chmodSync(CONFIG_PATH, 384);
-  } catch {
-  }
+  return readGlobalConfig();
 }
 function loadAIConfig() {
   const config = readConfigFile();
@@ -655,11 +669,12 @@ function loadAIConfig() {
   return { platform, model, apiKey: key, keySource: source };
 }
 function saveAIConfigPreferences(platform, model) {
-  const config = readConfigFile();
-  if (config.ai === void 0) config.ai = {};
-  config.ai.platform = platform;
-  config.ai.model = model;
-  writeConfigFile(config);
+  updateGlobalConfig((config) => {
+    const cfg = config;
+    if (cfg.ai === void 0) cfg.ai = {};
+    cfg.ai.platform = platform;
+    cfg.ai.model = model;
+  });
 }
 function loadGuidedReviewConfig() {
   const config = readConfigFile();
@@ -669,9 +684,9 @@ function loadGuidedReviewConfig() {
   };
 }
 function saveGuidedReviewConfig(settings) {
-  const config = readConfigFile();
-  config.guidedReview = { enabled: settings.enabled, topics: settings.topics };
-  writeConfigFile(config);
+  updateGlobalConfig((config) => {
+    config.guidedReview = { enabled: settings.enabled, topics: settings.topics };
+  });
 }
 
 // src/db/ai-queries.ts
@@ -1308,6 +1323,18 @@ function getHeadCommit(cwd) {
   return spawnSync3("git", ["rev-parse", "HEAD"], { cwd, encoding: "utf-8" }).stdout.trim();
 }
 
+// src/git/parseDiffData.ts
+function parseDiffData(raw) {
+  if (raw === null || raw === void 0 || raw === "") return null;
+  try {
+    const parsed = JSON.parse(raw);
+    if (typeof parsed !== "object" || parsed === null) return null;
+    return parsed;
+  } catch {
+    return null;
+  }
+}
+
 // src/git/diff.ts
 function git2(args, cwd) {
   const result = spawnSync4("git", args, { cwd, encoding: "utf-8", maxBuffer: 50 * 1024 * 1024 });
@@ -1689,7 +1716,7 @@ async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
   for (const [path, existingFile] of existingByPath) {
     const newDiff = newDiffsByPath.get(path);
     if (newDiff) {
-      const oldDiff = JSON.parse(existingFile.diff_data ?? "{}");
+      const oldDiff = parseDiffData(existingFile.diff_data) ?? {};
       const annotations = await getAnnotationsForFile(existingFile.id);
       if (annotations.length > 0) {
         stale += await migrateAnnotations(annotations, oldDiff, newDiff);
@@ -1701,7 +1728,7 @@ async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
       if (annotations.length === 0) {
         await deleteReviewFile(existingFile.id);
       } else {
-        const oldDiff = JSON.parse(existingFile.diff_data ?? "{}");
+        const oldDiff = parseDiffData(existingFile.diff_data) ?? {};
         for (const a of annotations) {
           if (!a.is_stale) {
             const content = findLineContent(oldDiff, a.line_number, a.side);
@@ -1725,9 +1752,8 @@ async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
 // src/server.ts
 import { serve } from "@hono/node-server";
 import { exec } from "child_process";
-import { existsSync as existsSync9, readFileSync as readFileSync12 } from "fs";
-import { Hono as Hono8 } from "hono";
-import { homedir as homedir6 } from "os";
+import { existsSync as existsSync8, readFileSync as readFileSync12 } from "fs";
+import { Hono as Hono16 } from "hono";
 import { dirname as dirname2, join as join10 } from "path";
 import { fileURLToPath as fileURLToPath2 } from "url";
 
@@ -1985,7 +2011,7 @@ function buildFileContexts(files, charBudget) {
   const contexts = [];
   const perFileBudget = Math.floor(charBudget / Math.max(files.length, 1));
   for (const file of files) {
-    const diff = JSON.parse(file.diff_data !== null && file.diff_data !== "" ? file.diff_data : "{}");
+    const diff = parseDiffData(file.diff_data) ?? {};
     let added = 0;
     let removed = 0;
     const hunks = diff.hunks;
@@ -2048,6 +2074,45 @@ function extractJSON(text) {
   }
   throw new Error(`Could not extract JSON from AI response: ${text.slice(0, 300)}`);
 }
+async function runAnalysisBatch(files, config, repoRoot, options) {
+  const contextWindow = getModelContextWindow(config.platform, config.model);
+  const charBudget = Math.floor(contextWindow * 0.7 * 3);
+  const contexts = buildFileContexts(files, charBudget);
+  const validPaths = new Set(files.map((f) => f.file_path));
+  const initialPrompt = [
+    options.initialPromptHeader(files.length),
+    "",
+    formatContextsForPrompt(contexts)
+  ].join("\n");
+  const messages = [{ role: "user", content: initialPrompt }];
+  for (let round = 0; round < 3; round++) {
+    const response = await sendAIRequest(config, options.systemPrompt, messages);
+    const parsed = extractJSON(response.content);
+    if (isNeedContext(parsed)) {
+      const safePaths = parsed.needContext.filter((p) => validPaths.has(p));
+      if (safePaths.length === 0) {
+        throw new Error("AI requested context for files not in the review");
+      }
+      const fileContents = safePaths.map((path) => ({
+        path,
+        content: getFileContent(path, "working", repoRoot)
+      }));
+      messages.push({ role: "assistant", content: response.content });
+      messages.push({
+        role: "user",
+        content: `Here is the full content of the requested files:
+
+${formatAdditionalContext(fileContents)}`
+      });
+      continue;
+    }
+    if (!Array.isArray(parsed)) {
+      throw new Error(`Expected an array of ${options.resultLabel} from AI`);
+    }
+    return parsed;
+  }
+  throw new Error(`${options.analysisName} did not converge after 3 context rounds`);
+}
 
 // src/ai/analyze-guided.ts
 var TOPIC_DISPLAY = {
@@ -2132,45 +2197,13 @@ If you need full file content to explain accurately, output ONLY: {"needContext"
 
 CRITICAL: Your entire response must be parseable by JSON.parse(). No prose, no markdown, no explanation.`;
 }
-async function runGuidedAnalysisBatch(files, config, repoRoot, guidedReview) {
-  const contextWindow = getModelContextWindow(config.platform, config.model);
-  const charBudget = Math.floor(contextWindow * 0.7 * 3);
-  const systemPrompt = buildSystemPrompt(guidedReview);
-  const contexts = buildFileContexts(files, charBudget);
-  const validPaths = new Set(files.map((f) => f.file_path));
-  const initialPrompt = [
-    `Provide educational walkthrough notes for these ${String(files.length)} changed files:`,
-    "",
-    formatContextsForPrompt(contexts)
-  ].join("\n");
-  const messages = [{ role: "user", content: initialPrompt }];
-  for (let round = 0; round < 3; round++) {
-    const response = await sendAIRequest(config, systemPrompt, messages);
-    const parsed = extractJSON(response.content);
-    if (isNeedContext(parsed)) {
-      const safePaths = parsed.needContext.filter((p) => validPaths.has(p));
-      if (safePaths.length === 0) {
-        throw new Error("AI requested context for files not in the review");
-      }
-      const fileContents = safePaths.map((path) => ({
-        path,
-        content: getFileContent(path, "working", repoRoot)
-      }));
-      messages.push({ role: "assistant", content: response.content });
-      messages.push({
-        role: "user",
-        content: `Here is the full content of the requested files:
-
-${formatAdditionalContext(fileContents)}`
-      });
-      continue;
-    }
-    if (!Array.isArray(parsed)) {
-      throw new Error("Expected an array of guided review notes from AI");
-    }
-    return parsed;
-  }
-  throw new Error("Guided analysis did not converge after 3 context rounds");
+function runGuidedAnalysisBatch(files, config, repoRoot, guidedReview) {
+  return runAnalysisBatch(files, config, repoRoot, {
+    systemPrompt: buildSystemPrompt(guidedReview),
+    initialPromptHeader: (n) => `Provide educational walkthrough notes for these ${String(n)} changed files:`,
+    resultLabel: "guided review notes",
+    analysisName: "Guided analysis"
+  });
 }
 
 // src/ai/guided-review.ts
@@ -2260,45 +2293,14 @@ Every file in the diff must appear exactly once.
 If you need full file content to determine dependencies, output ONLY: {"needContext":["path/to/file.ts"]}
 
 CRITICAL: Your entire response must be parseable by JSON.parse(). No prose, no markdown, no explanation.`;
-async function runNarrativeAnalysisBatch(files, config, repoRoot, guidedReview) {
-  const contextWindow = getModelContextWindow(config.platform, config.model);
-  const charBudget = Math.floor(contextWindow * 0.7 * 3);
+function runNarrativeAnalysisBatch(files, config, repoRoot, guidedReview) {
   const systemPrompt = SYSTEM_PROMPT + (guidedReview !== void 0 ? buildGuidedReviewSuffix(guidedReview, "narrative") : "");
-  const contexts = buildFileContexts(files, charBudget);
-  const validPaths = new Set(files.map((f) => f.file_path));
-  const initialPrompt = [
-    `Determine the best reading order for reviewing these ${String(files.length)} changed files:`,
-    "",
-    formatContextsForPrompt(contexts)
-  ].join("\n");
-  const messages = [{ role: "user", content: initialPrompt }];
-  for (let round = 0; round < 3; round++) {
-    const response = await sendAIRequest(config, systemPrompt, messages);
-    const parsed = extractJSON(response.content);
-    if (isNeedContext(parsed)) {
-      const safePaths = parsed.needContext.filter((p) => validPaths.has(p));
-      if (safePaths.length === 0) {
-        throw new Error("AI requested context for files not in the review");
-      }
-      const fileContents = safePaths.map((path) => ({
-        path,
-        content: getFileContent(path, "working", repoRoot)
-      }));
-      messages.push({ role: "assistant", content: response.content });
-      messages.push({
-        role: "user",
-        content: `Here is the full content of the requested files:
-
-${formatAdditionalContext(fileContents)}`
-      });
-      continue;
-    }
-    if (!Array.isArray(parsed)) {
-      throw new Error("Expected an array of narrative ordering from AI");
-    }
-    return parsed;
-  }
-  throw new Error("Narrative analysis did not converge after 3 context rounds");
+  return runAnalysisBatch(files, config, repoRoot, {
+    systemPrompt,
+    initialPromptHeader: (n) => `Determine the best reading order for reviewing these ${String(n)} changed files:`,
+    resultLabel: "narrative ordering",
+    analysisName: "Narrative analysis"
+  });
 }
 function mergeNarrativeOrders(batchResults, batchCount) {
   if (batchCount <= 1) {
@@ -2359,45 +2361,14 @@ The aggregate should be the MAX of all individual dimension scores (if a file ha
 If you need full file content to assess accurately, output ONLY: {"needContext":["path/to/file.ts"]}
 
 CRITICAL: Your entire response must be parseable by JSON.parse(). No prose, no markdown, no explanation.`;
-async function runRiskAnalysisBatch(files, config, repoRoot, guidedReview) {
-  const contextWindow = getModelContextWindow(config.platform, config.model);
-  const charBudget = Math.floor(contextWindow * 0.7 * 3);
+function runRiskAnalysisBatch(files, config, repoRoot, guidedReview) {
   const systemPrompt = SYSTEM_PROMPT2 + (guidedReview !== void 0 ? buildGuidedReviewSuffix(guidedReview, "risk") : "");
-  const contexts = buildFileContexts(files, charBudget);
-  const validPaths = new Set(files.map((f) => f.file_path));
-  const initialPrompt = [
-    `Analyze the following ${String(files.length)} file diffs for risk:`,
-    "",
-    formatContextsForPrompt(contexts)
-  ].join("\n");
-  const messages = [{ role: "user", content: initialPrompt }];
-  for (let round = 0; round < 3; round++) {
-    const response = await sendAIRequest(config, systemPrompt, messages);
-    const parsed = extractJSON(response.content);
-    if (isNeedContext(parsed)) {
-      const safePaths = parsed.needContext.filter((p) => validPaths.has(p));
-      if (safePaths.length === 0) {
-        throw new Error("AI requested context for files not in the review");
-      }
-      const fileContents = safePaths.map((path) => ({
-        path,
-        content: getFileContent(path, "working", repoRoot)
-      }));
-      messages.push({ role: "assistant", content: response.content });
-      messages.push({
-        role: "user",
-        content: `Here is the full content of the requested files:
-
-${formatAdditionalContext(fileContents)}`
-      });
-      continue;
-    }
-    if (!Array.isArray(parsed)) {
-      throw new Error("Expected an array of risk assessments from AI");
-    }
-    return parsed;
-  }
-  throw new Error("Risk analysis did not converge after 3 context rounds");
+  return runAnalysisBatch(files, config, repoRoot, {
+    systemPrompt,
+    initialPromptHeader: (n) => `Analyze the following ${String(n)} file diffs for risk:`,
+    resultLabel: "risk assessments",
+    analysisName: "Risk analysis"
+  });
 }
 
 // src/ai/batch-planner.ts
@@ -2538,8 +2509,8 @@ function isRetriable(err) {
   return msg.includes("429") || msg.includes("500") || msg.includes("502") || msg.includes("503") || msg.includes("504") || msg.includes("rate_limit");
 }
 function sleep(ms) {
-  return new Promise((resolve7) => {
-    setTimeout(resolve7, ms);
+  return new Promise((resolve8) => {
+    setTimeout(resolve8, ms);
   });
 }
 
@@ -2583,8 +2554,8 @@ function randomLines(count) {
   return lines.sort((a, b) => a.line - b.line);
 }
 function sleep2(ms) {
-  return new Promise((resolve7) => {
-    setTimeout(resolve7, ms);
+  return new Promise((resolve8) => {
+    setTimeout(resolve8, ms);
   });
 }
 async function mockRiskAnalysisBatch(files) {
@@ -2653,6 +2624,24 @@ async function mockGuidedAnalysisBatch(files) {
 
 // src/routes/ai-analysis.ts
 init_queries();
+
+// src/utils/resolveReviewId.ts
+function resolveReviewId(c) {
+  return c.req.query("reviewId") ?? c.get("reviewId");
+}
+
+// src/utils/validate.ts
+function checkEnum(value, name, allowed) {
+  if (typeof value !== "string" || !allowed.includes(value)) {
+    return { error: `${name} must be one of: ${allowed.join(", ")}` };
+  }
+  return { ok: value };
+}
+function isNonEmptyString(value) {
+  return typeof value === "string" && value.trim() !== "";
+}
+
+// src/routes/ai-analysis.ts
 var aiAnalysisRoutes = new Hono();
 var VALID_SORT_MODES = ["folder", "risk", "narrative", "guided"];
 var VALID_RISK_DIMENSIONS = ["aggregate", "security", "correctness", "error-handling", "maintainability", "architecture", "performance"];
@@ -2661,15 +2650,14 @@ var VALID_IMAGE_MODES = ["metadata", "side-by-side", "difference", "slice"];
 var VALID_ANALYSIS_TYPES = ["risk", "narrative", "guided"];
 var cancelledAnalyses = /* @__PURE__ */ new Set();
 aiAnalysisRoutes.post("/analyze", async (c) => {
-  const reviewId = c.req.query("reviewId") ?? "";
+  const reviewId = resolveReviewId(c);
   const repoRoot = c.get("repoRoot");
   const body = await c.req.json();
   const analysisType = body.type;
   const invalidateCache = body.invalidateCache === true;
   debugLog(`POST /analyze: type=${analysisType}, reviewId=${reviewId}`);
-  if (!VALID_ANALYSIS_TYPES.includes(analysisType)) {
-    return c.json({ error: `type must be one of: ${VALID_ANALYSIS_TYPES.join(", ")}` }, 400);
-  }
+  const typeCheck = checkEnum(analysisType, "type", VALID_ANALYSIS_TYPES);
+  if ("error" in typeCheck) return c.json({ error: typeCheck.error }, 400);
   const testMode = isAIServiceTest();
   const config = loadAIConfig();
   if (config.apiKey === null && !testMode) {
@@ -2719,113 +2707,115 @@ aiAnalysisRoutes.post("/analyze", async (c) => {
   const analysis = await createAnalysis(reviewId, analysisType);
   debugLog(`POST /analyze: created new analysis id=${analysis.id}`);
   const guidedReview = loadGuidedReviewConfig();
-  void (async () => {
-    try {
-      debugLog("Background analysis starting...");
-      const contextWindow = getModelContextWindow(config.platform, config.model);
-      debugLog(`Context window: ${String(contextWindow)} tokens`);
-      const { batches, binaryFiles } = planBatches(files, contextWindow);
-      const fileIdMap = new Map(files.map((f) => [f.file_path, f.id]));
-      const totalAnalyzable = batches.reduce((sum, b) => sum + b.files.length, 0);
-      debugLog(`Analysis plan: ${String(totalAnalyzable)} analyzable + ${String(binaryFiles.length)} binary = ${String(totalAnalyzable + binaryFiles.length)} total files in ${String(batches.length)} batch(es)`);
-      const prevScores = invalidateCache ? [] : await getPreviousScores(reviewId, analysisType, analysis.id);
-      const binaryPathSet = new Set(binaryFiles.map((f) => f.file_path));
-      const unchangedPaths = /* @__PURE__ */ new Set();
-      const cachedScores = prevScores.filter((s) => {
-        if (fileIdMap.has(s.file_path) && !binaryPathSet.has(s.file_path)) {
-          unchangedPaths.add(s.file_path);
-          return true;
-        }
-        return false;
-      });
-      debugLog(`Cache: ${String(cachedScores.length)} scores from previous analysis, ${String(totalAnalyzable - cachedScores.length)} files need processing`);
-      if (cachedScores.length > 0) {
-        const cachedForInsert = cachedScores.map((s) => ({
-          reviewFileId: fileIdMap.get(s.file_path) ?? s.review_file_id,
-          filePath: s.file_path,
-          sortOrder: s.sort_order,
-          aggregateScore: s.aggregate_score,
-          rationale: s.rationale,
-          dimensionScores: s.dimension_scores !== null ? JSON.parse(s.dimension_scores) : null,
-          notes: s.notes !== null ? JSON.parse(s.notes) : null
-        }));
-        await appendFileScores(analysis.id, cachedForInsert);
-      }
-      const filteredBatches = batches.map((batch) => {
-        const remaining = batch.files.filter((f) => !unchangedPaths.has(f.file_path));
-        return { files: remaining, estimatedTokens: batch.estimatedTokens };
-      }).filter((batch) => batch.files.length > 0);
-      const filteredAnalyzable = filteredBatches.reduce((sum, b) => sum + b.files.length, 0);
-      const totalForProgress = filteredAnalyzable + binaryFiles.length + cachedScores.length;
-      debugLog(`After cache: ${String(filteredAnalyzable)} files to analyze in ${String(filteredBatches.length)} batch(es)`);
-      await updateAnalysisProgress(analysis.id, cachedScores.length, totalForProgress);
-      if (binaryFiles.length > 0) {
-        debugLog(`Saving ${String(binaryFiles.length)} binary files with score 0`);
-        const binaryScoreEntries = binaryFiles.map((f, idx) => ({
-          reviewFileId: fileIdMap.get(f.file_path) ?? "",
-          filePath: f.file_path,
-          sortOrder: 99999 + idx,
-          // Will be re-sorted later
-          aggregateScore: analysisType === "risk" ? 0 : null,
-          rationale: "Binary file \u2014 not analyzed",
-          dimensionScores: analysisType === "risk" ? { security: 0, correctness: 0, "error-handling": 0, maintainability: 0, architecture: 0, performance: 0 } : null,
-          notes: null
-        }));
-        await appendFileScores(analysis.id, binaryScoreEntries);
-        await updateAnalysisProgress(analysis.id, cachedScores.length + binaryFiles.length, totalForProgress);
-      }
-      if (filteredBatches.length === 0) {
-        debugLog("No batches to process (all files cached or binary), marking completed");
-        await updateAnalysisStatus(analysis.id, "completed");
-        return;
-      }
-      const shouldCancel = () => cancelledAnalyses.has(analysis.id);
-      const progressOffset = cachedScores.length + binaryFiles.length;
-      if (analysisType === "risk") {
-        await runBatchedRiskAnalysis(analysis.id, filteredBatches, files, config, repoRoot, fileIdMap, totalForProgress, progressOffset, shouldCancel, guidedReview);
-      } else if (analysisType === "narrative") {
-        await runBatchedNarrativeAnalysis(analysis.id, filteredBatches, files, config, repoRoot, fileIdMap, totalForProgress, progressOffset, shouldCancel, guidedReview);
-      } else {
-        await runBatchedGuidedAnalysis(analysis.id, filteredBatches, files, config, repoRoot, fileIdMap, totalForProgress, progressOffset, shouldCancel, guidedReview);
-      }
-      if (cancelledAnalyses.has(analysis.id)) {
-        cancelledAnalyses.delete(analysis.id);
-        debugLog(`Analysis ${analysis.id} was cancelled (user switched modes)`);
-        await updateAnalysisStatus(analysis.id, "failed", "Cancelled");
-        return;
-      }
-      cancelledAnalyses.delete(analysis.id);
-      debugLog(`Analysis ${analysis.id} completed successfully`);
-      await updateAnalysisStatus(analysis.id, "completed");
-    } catch (err) {
-      const message = err instanceof Error ? err.message : "Unknown error";
-      console.error(`Analysis failed: ${message}`);
-      debugLog(`Analysis ${analysis.id} failed: ${message}`);
-      await updateAnalysisStatus(analysis.id, "failed", message);
-    }
-  })();
+  void executeAnalysis({
+    analysisId: analysis.id,
+    analysisType: typeCheck.ok,
+    reviewId,
+    files,
+    config,
+    repoRoot,
+    guidedReview,
+    invalidateCache
+  });
   return c.json({ analysisId: analysis.id, status: "running" });
 });
-async function runBatchedRiskAnalysis(analysisId, batches, allFiles, config, repoRoot, fileIdMap, progressTotal, progressOffset, shouldCancel, guidedReview) {
+async function executeAnalysis(input) {
+  const { analysisId, analysisType, reviewId, files, config, repoRoot, guidedReview, invalidateCache } = input;
+  try {
+    debugLog("Background analysis starting...");
+    const contextWindow = getModelContextWindow(config.platform, config.model);
+    debugLog(`Context window: ${String(contextWindow)} tokens`);
+    const { batches, binaryFiles } = planBatches(files, contextWindow);
+    const fileIdMap = new Map(files.map((f) => [f.file_path, f.id]));
+    const totalAnalyzable = batches.reduce((sum, b) => sum + b.files.length, 0);
+    debugLog(`Analysis plan: ${String(totalAnalyzable)} analyzable + ${String(binaryFiles.length)} binary = ${String(totalAnalyzable + binaryFiles.length)} total files in ${String(batches.length)} batch(es)`);
+    const prevScores = invalidateCache ? [] : await getPreviousScores(reviewId, analysisType, analysisId);
+    const binaryPathSet = new Set(binaryFiles.map((f) => f.file_path));
+    const unchangedPaths = /* @__PURE__ */ new Set();
+    const cachedScores = prevScores.filter((s) => {
+      if (fileIdMap.has(s.file_path) && !binaryPathSet.has(s.file_path)) {
+        unchangedPaths.add(s.file_path);
+        return true;
+      }
+      return false;
+    });
+    debugLog(`Cache: ${String(cachedScores.length)} scores from previous analysis, ${String(totalAnalyzable - cachedScores.length)} files need processing`);
+    if (cachedScores.length > 0) {
+      const cachedForInsert = cachedScores.map((s) => ({
+        reviewFileId: fileIdMap.get(s.file_path) ?? s.review_file_id,
+        filePath: s.file_path,
+        sortOrder: s.sort_order,
+        aggregateScore: s.aggregate_score,
+        rationale: s.rationale,
+        dimensionScores: s.dimension_scores !== null ? JSON.parse(s.dimension_scores) : null,
+        notes: s.notes !== null ? JSON.parse(s.notes) : null
+      }));
+      await appendFileScores(analysisId, cachedForInsert);
+    }
+    const filteredBatches = batches.map((batch) => {
+      const remaining = batch.files.filter((f) => !unchangedPaths.has(f.file_path));
+      return { files: remaining, estimatedTokens: batch.estimatedTokens };
+    }).filter((batch) => batch.files.length > 0);
+    const filteredAnalyzable = filteredBatches.reduce((sum, b) => sum + b.files.length, 0);
+    const totalForProgress = filteredAnalyzable + binaryFiles.length + cachedScores.length;
+    debugLog(`After cache: ${String(filteredAnalyzable)} files to analyze in ${String(filteredBatches.length)} batch(es)`);
+    await updateAnalysisProgress(analysisId, cachedScores.length, totalForProgress);
+    if (binaryFiles.length > 0) {
+      debugLog(`Saving ${String(binaryFiles.length)} binary files with score 0`);
+      const binaryScoreEntries = binaryFiles.map((f, idx) => ({
+        reviewFileId: fileIdMap.get(f.file_path) ?? "",
+        filePath: f.file_path,
+        sortOrder: 99999 + idx,
+        // Will be re-sorted later
+        aggregateScore: analysisType === "risk" ? 0 : null,
+        rationale: "Binary file \u2014 not analyzed",
+        dimensionScores: analysisType === "risk" ? { security: 0, correctness: 0, "error-handling": 0, maintainability: 0, architecture: 0, performance: 0 } : null,
+        notes: null
+      }));
+      await appendFileScores(analysisId, binaryScoreEntries);
+      await updateAnalysisProgress(analysisId, cachedScores.length + binaryFiles.length, totalForProgress);
+    }
+    if (filteredBatches.length === 0) {
+      debugLog("No batches to process (all files cached or binary), marking completed");
+      await updateAnalysisStatus(analysisId, "completed");
+      return;
+    }
+    const shouldCancel = () => cancelledAnalyses.has(analysisId);
+    const progressOffset = cachedScores.length + binaryFiles.length;
+    const runArgs = [analysisId, filteredBatches, files.length, totalForProgress, progressOffset];
+    if (analysisType === "risk") {
+      await runBatchedAnalysis(...runArgs, riskAnalysisConfig(config, repoRoot, fileIdMap, guidedReview, analysisId), shouldCancel);
+    } else if (analysisType === "narrative") {
+      await runBatchedAnalysis(...runArgs, narrativeAnalysisConfig(config, repoRoot, fileIdMap, guidedReview, analysisId), shouldCancel);
+    } else {
+      await runBatchedAnalysis(...runArgs, guidedAnalysisConfig(config, repoRoot, fileIdMap, guidedReview), shouldCancel);
+    }
+    if (cancelledAnalyses.has(analysisId)) {
+      cancelledAnalyses.delete(analysisId);
+      debugLog(`Analysis ${analysisId} was cancelled (user switched modes)`);
+      await updateAnalysisStatus(analysisId, "failed", "Cancelled");
+      return;
+    }
+    cancelledAnalyses.delete(analysisId);
+    debugLog(`Analysis ${analysisId} completed successfully`);
+    await updateAnalysisStatus(analysisId, "completed");
+  } catch (err) {
+    const message = err instanceof Error ? err.message : "Unknown error";
+    console.error(`Analysis failed: ${message}`);
+    debugLog(`Analysis ${analysisId} failed: ${message}`);
+    await updateAnalysisStatus(analysisId, "failed", message);
+  }
+}
+async function runBatchedAnalysis(analysisId, batches, totalFiles, progressTotal, progressOffset, cfg, shouldCancel) {
   const allResults = await runBatches(
     batches,
-    allFiles.length,
-    async (batch) => isAIServiceTest() ? mockRiskAnalysisBatch(batch.files) : runRiskAnalysisBatch(batch.files, config, repoRoot, guidedReview),
+    totalFiles,
+    async (batch) => cfg.runBatch(batch.files),
     async (_batchIndex, results) => {
-      for (const r of results) {
-        const maxDimension = Math.max(...Object.values(r.scores));
-        r.aggregate = Math.max(r.aggregate, maxDimension);
+      if (cfg.postProcessResult) {
+        for (const r of results) cfg.postProcessResult(r);
       }
-      const scores = results.map((r) => ({
-        reviewFileId: fileIdMap.get(r.filePath) ?? "",
-        filePath: r.filePath,
-        sortOrder: 0,
-        // Placeholder — final sort happens after all batches
-        aggregateScore: r.aggregate,
-        rationale: r.rationale,
-        dimensionScores: r.scores,
-        notes: r.notes ?? null
-      }));
+      const scores = results.map((r, idx) => cfg.mapResult(r, idx));
       await appendFileScores(analysisId, scores);
     },
     async (progress) => {
@@ -2833,91 +2823,98 @@ async function runBatchedRiskAnalysis(analysisId, batches, allFiles, config, rep
     },
     1,
     shouldCancel,
-    "risk"
+    cfg.analysisType
   );
-  const sorted = allResults.slice().sort((a, b) => b.aggregate - a.aggregate);
-  const sortMap = new Map(sorted.map((r, idx) => [r.filePath, idx]));
+  if (cfg.finalize) await cfg.finalize(allResults, batches.length);
+}
+async function updateSortOrders(analysisId, entries) {
   const { getDb: getDb2 } = await Promise.resolve().then(() => (init_connection(), connection_exports));
   const db2 = await getDb2();
-  for (const [filePath, sortOrder] of sortMap) {
+  for (const [filePath, sortOrder] of entries) {
     await db2.query(
       "UPDATE ai_file_scores SET sort_order = $1 WHERE analysis_id = $2 AND file_path = $3",
       [sortOrder, analysisId, filePath]
     );
   }
 }
-async function runBatchedNarrativeAnalysis(analysisId, batches, allFiles, config, repoRoot, fileIdMap, progressTotal, progressOffset, shouldCancel, guidedReview) {
-  const allResults = await runBatches(
-    batches,
-    allFiles.length,
-    async (batch) => isAIServiceTest() ? mockNarrativeAnalysisBatch(batch.files) : runNarrativeAnalysisBatch(batch.files, config, repoRoot, guidedReview),
-    async (_batchIndex, results) => {
-      const scores = results.map((r) => ({
-        reviewFileId: fileIdMap.get(r.filePath) ?? "",
-        filePath: r.filePath,
-        sortOrder: r.position,
-        // Batch-local position — will be re-sorted after merge
-        aggregateScore: null,
-        rationale: r.rationale,
-        dimensionScores: null,
-        notes: r.notes ?? null
-      }));
-      await appendFileScores(analysisId, scores);
-    },
-    async (progress) => {
-      await updateAnalysisProgress(analysisId, progressOffset + progress.completedFiles, progressTotal);
+function pickRunner(real, mock) {
+  return isAIServiceTest() ? mock() : real();
+}
+function riskAnalysisConfig(config, repoRoot, fileIdMap, guidedReview, analysisId) {
+  return {
+    analysisType: "risk",
+    runBatch: (files) => pickRunner(
+      () => runRiskAnalysisBatch(files, config, repoRoot, guidedReview),
+      () => mockRiskAnalysisBatch(files)
+    ),
+    postProcessResult: (r) => {
+      const maxDimension = Math.max(...Object.values(r.scores));
+      r.aggregate = Math.max(r.aggregate, maxDimension);
     },
-    1,
-    shouldCancel,
-    "narrative"
-  );
-  if (allResults.length > 0) {
-    const mergedPositions = mergeNarrativeOrders(allResults, batches.length);
-    const { getDb: getDb2 } = await Promise.resolve().then(() => (init_connection(), connection_exports));
-    const db2 = await getDb2();
-    for (const [filePath, position] of mergedPositions) {
-      await db2.query(
-        "UPDATE ai_file_scores SET sort_order = $1 WHERE analysis_id = $2 AND file_path = $3",
-        [position, analysisId, filePath]
-      );
+    mapResult: (r) => ({
+      reviewFileId: fileIdMap.get(r.filePath) ?? "",
+      filePath: r.filePath,
+      sortOrder: 0,
+      // Placeholder — final sort happens after all batches
+      aggregateScore: r.aggregate,
+      rationale: r.rationale,
+      dimensionScores: r.scores,
+      notes: r.notes ?? null
+    }),
+    finalize: async (allResults) => {
+      const sorted = allResults.slice().sort((a, b) => b.aggregate - a.aggregate);
+      await updateSortOrders(analysisId, sorted.map((r, idx) => [r.filePath, idx]));
     }
-  }
+  };
 }
-async function runBatchedGuidedAnalysis(analysisId, batches, allFiles, config, repoRoot, fileIdMap, progressTotal, progressOffset, shouldCancel, guidedReview) {
-  await runBatches(
-    batches,
-    allFiles.length,
-    async (batch) => {
-      if (isAIServiceTest()) return mockGuidedAnalysisBatch(batch.files);
+function narrativeAnalysisConfig(config, repoRoot, fileIdMap, guidedReview, analysisId) {
+  return {
+    analysisType: "narrative",
+    runBatch: (files) => pickRunner(
+      () => runNarrativeAnalysisBatch(files, config, repoRoot, guidedReview),
+      () => mockNarrativeAnalysisBatch(files)
+    ),
+    mapResult: (r) => ({
+      reviewFileId: fileIdMap.get(r.filePath) ?? "",
+      filePath: r.filePath,
+      sortOrder: r.position,
+      // Batch-local position — will be re-sorted after merge
+      aggregateScore: null,
+      rationale: r.rationale,
+      dimensionScores: null,
+      notes: r.notes ?? null
+    }),
+    finalize: async (allResults, batchCount) => {
+      if (allResults.length === 0) return;
+      const merged = mergeNarrativeOrders(allResults, batchCount);
+      await updateSortOrders(analysisId, merged);
+    }
+  };
+}
+function guidedAnalysisConfig(config, repoRoot, fileIdMap, guidedReview) {
+  return {
+    analysisType: "guided",
+    runBatch: (files) => {
+      if (isAIServiceTest()) return mockGuidedAnalysisBatch(files);
       if (guidedReview === void 0) throw new Error("Guided review config required");
-      return runGuidedAnalysisBatch(batch.files, config, repoRoot, guidedReview);
-    },
-    async (_batchIndex, results) => {
-      const scores = results.map((r, idx) => ({
-        reviewFileId: fileIdMap.get(r.filePath) ?? "",
-        filePath: r.filePath,
-        sortOrder: idx,
-        aggregateScore: null,
-        rationale: null,
-        dimensionScores: null,
-        notes: r.notes
-      }));
-      await appendFileScores(analysisId, scores);
-    },
-    async (progress) => {
-      await updateAnalysisProgress(analysisId, progressOffset + progress.completedFiles, progressTotal);
+      return runGuidedAnalysisBatch(files, config, repoRoot, guidedReview);
     },
-    1,
-    shouldCancel,
-    "guided"
-  );
-}
+    mapResult: (r, idx) => ({
+      reviewFileId: fileIdMap.get(r.filePath) ?? "",
+      filePath: r.filePath,
+      sortOrder: idx,
+      aggregateScore: null,
+      rationale: null,
+      dimensionScores: null,
+      notes: r.notes
+    })
+  };
+}
 aiAnalysisRoutes.get("/analysis/:type", async (c) => {
-  const reviewId = c.req.query("reviewId") ?? "";
+  const reviewId = resolveReviewId(c);
   const analysisType = c.req.param("type");
-  if (!VALID_ANALYSIS_TYPES.includes(analysisType)) {
-    return c.json({ error: `type must be one of: ${VALID_ANALYSIS_TYPES.join(", ")}` }, 400);
-  }
+  const typeCheck = checkEnum(analysisType, "type", VALID_ANALYSIS_TYPES);
+  if ("error" in typeCheck) return c.json({ error: typeCheck.error }, 400);
   const analysis = await getLatestAnalysis(reviewId, analysisType);
   if (analysis === void 0) {
     debugLog(`GET /analysis/${analysisType}: no analysis found`);
@@ -2948,11 +2945,10 @@ aiAnalysisRoutes.get("/analysis/:type", async (c) => {
   });
 });
 aiAnalysisRoutes.get("/analysis/:type/status", async (c) => {
-  const reviewId = c.req.query("reviewId") ?? "";
+  const reviewId = resolveReviewId(c);
   const analysisType = c.req.param("type");
-  if (!VALID_ANALYSIS_TYPES.includes(analysisType)) {
-    return c.json({ error: `type must be one of: ${VALID_ANALYSIS_TYPES.join(", ")}` }, 400);
-  }
+  const typeCheck = checkEnum(analysisType, "type", VALID_ANALYSIS_TYPES);
+  if ("error" in typeCheck) return c.json({ error: typeCheck.error }, 400);
   const analysis = await getLatestAnalysis(reviewId, analysisType);
   if (analysis === void 0) {
     debugLog(`GET /analysis/${analysisType}/status: no analysis found`);
@@ -2992,11 +2988,13 @@ aiAnalysisRoutes.get("/preferences", async (c) => {
 });
 aiAnalysisRoutes.post("/preferences", async (c) => {
   const body = await c.req.json();
-  if (body.sort_mode !== void 0 && !VALID_SORT_MODES.includes(body.sort_mode)) {
-    return c.json({ error: `sort_mode must be one of: ${VALID_SORT_MODES.join(", ")}` }, 400);
+  if (body.sort_mode !== void 0) {
+    const v = checkEnum(body.sort_mode, "sort_mode", VALID_SORT_MODES);
+    if ("error" in v) return c.json({ error: v.error }, 400);
   }
-  if (body.risk_sort_dimension !== void 0 && !VALID_RISK_DIMENSIONS.includes(body.risk_sort_dimension)) {
-    return c.json({ error: `risk_sort_dimension must be one of: ${VALID_RISK_DIMENSIONS.join(", ")}` }, 400);
+  if (body.risk_sort_dimension !== void 0) {
+    const v = checkEnum(body.risk_sort_dimension, "risk_sort_dimension", VALID_RISK_DIMENSIONS);
+    if ("error" in v) return c.json({ error: v.error }, 400);
   }
   if (body.show_risk_scores !== void 0 && typeof body.show_risk_scores !== "boolean") {
     return c.json({ error: "show_risk_scores must be a boolean" }, 400);
@@ -3004,11 +3002,13 @@ aiAnalysisRoutes.post("/preferences", async (c) => {
   if (body.ignore_whitespace !== void 0 && typeof body.ignore_whitespace !== "boolean") {
     return c.json({ error: "ignore_whitespace must be a boolean" }, 400);
   }
-  if (body.svg_view_mode !== void 0 && !VALID_SVG_VIEW_MODES.includes(body.svg_view_mode)) {
-    return c.json({ error: `svg_view_mode must be one of: ${VALID_SVG_VIEW_MODES.join(", ")}` }, 400);
+  if (body.svg_view_mode !== void 0) {
+    const v = checkEnum(body.svg_view_mode, "svg_view_mode", VALID_SVG_VIEW_MODES);
+    if ("error" in v) return c.json({ error: v.error }, 400);
   }
-  if (body.last_image_mode !== void 0 && !VALID_IMAGE_MODES.includes(body.last_image_mode)) {
-    return c.json({ error: `last_image_mode must be one of: ${VALID_IMAGE_MODES.join(", ")}` }, 400);
+  if (body.last_image_mode !== void 0) {
+    const v = checkEnum(body.last_image_mode, "last_image_mode", VALID_IMAGE_MODES);
+    if ("error" in v) return c.json({ error: v.error }, 400);
   }
   await saveUserPreferences(body);
   return c.json({ ok: true });
@@ -3031,10 +3031,9 @@ aiConfigRoutes.get("/config", (c) => {
 });
 aiConfigRoutes.post("/config", async (c) => {
   const body = await c.req.json();
-  if (!VALID_PLATFORMS.includes(body.platform)) {
-    return c.json({ error: `platform must be one of: ${VALID_PLATFORMS.join(", ")}` }, 400);
-  }
-  if (typeof body.model !== "string" || body.model.trim() === "") {
+  const platformCheck = checkEnum(body.platform, "platform", VALID_PLATFORMS);
+  if ("error" in platformCheck) return c.json({ error: platformCheck.error }, 400);
+  if (!isNonEmptyString(body.model)) {
     return c.json({ error: "model must be a non-empty string" }, 400);
   }
   if (body.guidedReview !== void 0) {
@@ -3050,7 +3049,7 @@ aiConfigRoutes.post("/config", async (c) => {
       return c.json({ error: "guidedReview.topics must be an array of strings" }, 400);
     }
   }
-  saveAIConfigPreferences(body.platform, body.model);
+  saveAIConfigPreferences(platformCheck.ok, body.model);
   if (body.guidedReview !== void 0) {
     saveGuidedReviewConfig(body.guidedReview);
   }
@@ -3078,28 +3077,25 @@ aiConfigRoutes.get("/key-status", (c) => {
 });
 aiConfigRoutes.post("/key", async (c) => {
   const body = await c.req.json();
-  if (!VALID_PLATFORMS.includes(body.platform)) {
-    return c.json({ error: `platform must be one of: ${VALID_PLATFORMS.join(", ")}` }, 400);
-  }
-  if (typeof body.key !== "string" || body.key.trim() === "") {
+  const platformCheck = checkEnum(body.platform, "platform", VALID_PLATFORMS);
+  if ("error" in platformCheck) return c.json({ error: platformCheck.error }, 400);
+  if (!isNonEmptyString(body.key)) {
     return c.json({ error: "key must be a non-empty string" }, 400);
   }
-  if (!VALID_KEY_STORAGES.includes(body.storage)) {
-    return c.json({ error: `storage must be one of: ${VALID_KEY_STORAGES.join(", ")}` }, 400);
-  }
+  const storageCheck = checkEnum(body.storage, "storage", VALID_KEY_STORAGES);
+  if ("error" in storageCheck) return c.json({ error: storageCheck.error }, 400);
   saveAPIKey(
-    body.platform,
+    platformCheck.ok,
     body.key,
-    body.storage
+    storageCheck.ok
   );
   return c.json({ ok: true });
 });
 aiConfigRoutes.delete("/key", (c) => {
   const platform = c.req.query("platform") ?? "anthropic";
-  if (!VALID_PLATFORMS.includes(platform)) {
-    return c.json({ error: `platform must be one of: ${VALID_PLATFORMS.join(", ")}` }, 400);
-  }
-  deleteAPIKey(platform);
+  const platformCheck = checkEnum(platform, "platform", VALID_PLATFORMS);
+  if ("error" in platformCheck) return c.json({ error: platformCheck.error }, 400);
+  deleteAPIKey(platformCheck.ok);
   return c.json({ ok: true });
 });
 
@@ -3109,12 +3105,11 @@ aiApiRoutes.route("/", aiConfigRoutes);
 aiApiRoutes.route("/", aiAnalysisRoutes);
 
 // src/routes/api.ts
+import { Hono as Hono12 } from "hono";
+
+// src/routes/api/annotations.ts
 init_queries();
-import { execFileSync, spawnSync as spawnSync7 } from "child_process";
-import { existsSync as existsSync6, mkdirSync as mkdirSync4, readFileSync as readFileSync8, writeFileSync as writeFileSync5 } from "fs";
 import { Hono as Hono4 } from "hono";
-import { homedir as homedir3 } from "os";
-import { join as join7, resolve as resolve4 } from "path";
 
 // src/export/generate.ts
 init_queries();
@@ -3265,10 +3260,166 @@ function scheduleAutoExport(reviewId, repoRoot) {
   }, DEBOUNCE_MS);
 }
 
+// src/routes/api/annotations.ts
+var annotationsRoutes = new Hono4();
+var VALID_CATEGORIES = ["bug", "fix", "style", "pattern-follow", "pattern-avoid", "note", "remember"];
+var VALID_SIDES = ["old", "new"];
+function autoExport(c) {
+  scheduleAutoExport(c.get("reviewId"), c.get("repoRoot"));
+}
+annotationsRoutes.post("/annotations", async (c) => {
+  const body = await c.req.json();
+  if (!isNonEmptyString(body.reviewFileId)) {
+    return c.json({ error: "reviewFileId must be a non-empty string" }, 400);
+  }
+  if (typeof body.lineNumber !== "number" || !Number.isInteger(body.lineNumber) || body.lineNumber < 1) {
+    return c.json({ error: "lineNumber must be a positive integer" }, 400);
+  }
+  const sideCheck = checkEnum(body.side, "side", VALID_SIDES);
+  if ("error" in sideCheck) return c.json({ error: sideCheck.error }, 400);
+  const categoryCheck = checkEnum(body.category, "category", VALID_CATEGORIES);
+  if ("error" in categoryCheck) return c.json({ error: categoryCheck.error }, 400);
+  if (!isNonEmptyString(body.content)) {
+    return c.json({ error: "content must be a non-empty string" }, 400);
+  }
+  const annotation = await addAnnotation(
+    body.reviewFileId,
+    body.lineNumber,
+    sideCheck.ok,
+    categoryCheck.ok,
+    body.content
+  );
+  autoExport(c);
+  return c.json(annotation, 201);
+});
+annotationsRoutes.patch("/annotations/:id", async (c) => {
+  const { content, category } = await c.req.json();
+  if (!isNonEmptyString(content)) {
+    return c.json({ error: "content must be a non-empty string" }, 400);
+  }
+  const categoryCheck = checkEnum(category, "category", VALID_CATEGORIES);
+  if ("error" in categoryCheck) return c.json({ error: categoryCheck.error }, 400);
+  await updateAnnotation(c.req.param("id"), content, categoryCheck.ok);
+  autoExport(c);
+  return c.json({ ok: true });
+});
+annotationsRoutes.delete("/annotations/:id", async (c) => {
+  await deleteAnnotation(c.req.param("id"));
+  autoExport(c);
+  return c.json({ ok: true });
+});
+annotationsRoutes.patch("/annotations/:id/move", async (c) => {
+  const { lineNumber, side } = await c.req.json();
+  if (typeof lineNumber !== "number" || !Number.isInteger(lineNumber) || lineNumber < 1) {
+    return c.json({ error: "lineNumber must be a positive integer" }, 400);
+  }
+  const sideCheck = checkEnum(side, "side", VALID_SIDES);
+  if ("error" in sideCheck) return c.json({ error: sideCheck.error }, 400);
+  await moveAnnotation(c.req.param("id"), lineNumber, sideCheck.ok);
+  autoExport(c);
+  return c.json({ ok: true });
+});
+annotationsRoutes.post("/annotations/:id/keep", async (c) => {
+  await markAnnotationCurrent(c.req.param("id"));
+  autoExport(c);
+  return c.json({ ok: true });
+});
+annotationsRoutes.post("/annotations/stale/delete-all", async (c) => {
+  const reviewId = resolveReviewId(c);
+  await deleteStaleAnnotations(reviewId);
+  autoExport(c);
+  return c.json({ ok: true });
+});
+annotationsRoutes.post("/annotations/stale/keep-all", async (c) => {
+  const reviewId = resolveReviewId(c);
+  await keepAllStaleAnnotations(reviewId);
+  autoExport(c);
+  return c.json({ ok: true });
+});
+annotationsRoutes.get("/annotations/all", async (c) => {
+  const reviewId = resolveReviewId(c);
+  const annotations = await getAnnotationsForReview(reviewId);
+  return c.json(annotations);
+});
+
+// src/routes/api/context.ts
+init_queries();
+import { Hono as Hono5 } from "hono";
+var contextRoutes = new Hono5();
+contextRoutes.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 content = getFileContent(file.file_path, "working", repoRoot);
+  const allLines = content.split("\n");
+  const clampedStart = Math.max(1, startLine);
+  const clampedEnd = Math.min(allLines.length, endLine);
+  const lines = [];
+  for (let i = clampedStart; i <= clampedEnd; i++) {
+    lines.push({ num: i, content: allLines[i - 1] || "" });
+  }
+  return c.json({ lines });
+});
+
+// src/routes/api/files.ts
+init_queries();
+import { execFileSync } from "child_process";
+import { Hono as Hono6 } from "hono";
+import { resolve as resolve3 } from "path";
+var filesRoutes = new Hono6();
+var VALID_FILE_STATUSES = ["pending", "reviewed"];
+filesRoutes.get("/files", async (c) => {
+  const reviewId = resolveReviewId(c);
+  const files = await getReviewFiles(reviewId);
+  const annotationCounts = {};
+  for (const file of files) {
+    const annotations = await getAnnotationsForFile(file.id);
+    annotationCounts[file.id] = annotations.length;
+  }
+  const staleCounts = await getStaleCountsForReview(reviewId);
+  return c.json({ files, annotationCounts, staleCounts });
+});
+filesRoutes.get("/files/:fileId", async (c) => {
+  const file = await getReviewFile(c.req.param("fileId"));
+  if (!file) return c.json({ error: "Not found" }, 404);
+  const annotations = await getAnnotationsForFile(file.id);
+  return c.json({ file, annotations });
+});
+filesRoutes.patch("/files/:fileId/status", async (c) => {
+  const { status } = await c.req.json();
+  const v = checkEnum(status, "status", VALID_FILE_STATUSES);
+  if ("error" in v) return c.json({ error: v.error }, 400);
+  await updateFileStatus(c.req.param("fileId"), v.ok);
+  return c.json({ ok: true });
+});
+filesRoutes.post("/files/:fileId/reveal", async (c) => {
+  const file = await getReviewFile(c.req.param("fileId"));
+  if (!file) return c.json({ error: "Not found" }, 404);
+  const repoRoot = c.get("repoRoot");
+  const fullPath = resolve3(repoRoot, file.file_path);
+  try {
+    if (process.platform === "darwin") {
+      execFileSync("open", ["-R", fullPath]);
+    } else if (process.platform === "win32") {
+      execFileSync("explorer", ["/select," + fullPath]);
+    } else {
+      execFileSync("xdg-open", [resolve3(fullPath, "..")]);
+    }
+  } catch {
+  }
+  return c.json({ ok: true });
+});
+
+// src/routes/api/image.ts
+init_queries();
+import { Hono as Hono7 } from "hono";
+
 // src/git/image.ts
 import { spawnSync as spawnSync6 } from "child_process";
 import { readFileSync as readFileSync6 } from "fs";
-import { resolve as resolve3 } from "path";
+import { resolve as resolve4 } from "path";
 
 // src/git/image-metadata.ts
 var IMAGE_EXTENSIONS = /* @__PURE__ */ new Set([".png", ".jpg", ".jpeg", ".gif", ".webp", ".svg"]);
@@ -3538,7 +3689,7 @@ function gitShowFile(ref, filePath, repoRoot) {
 }
 function readWorkingFile(filePath, repoRoot) {
   try {
-    return readFileSync6(resolve3(repoRoot, filePath));
+    return readFileSync6(resolve4(repoRoot, filePath));
   } catch {
     return null;
   }
@@ -3705,6 +3856,65 @@ async function rasterizeSvg(svgData) {
   return png;
 }
 
+// src/routes/api/image.ts
+var imageRoutes = new Hono7();
+imageRoutes.get("/image/:fileId/metadata", async (c) => {
+  const fileId = c.req.param("fileId");
+  const file = await getReviewFile(fileId);
+  if (!file) return c.json({ error: "Not found" }, 404);
+  const repoRoot = c.get("repoRoot");
+  const review = await getReview(file.review_id);
+  if (!review) return c.json({ error: "Review not found" }, 404);
+  const mode = parseModeString(review.mode);
+  const diff = parseDiffData(file.diff_data);
+  const oldPath = diff?.oldPath ?? null;
+  const status = diff?.status ?? "modified";
+  const oldImage = status !== "added" ? getOldImage(mode, file.file_path, oldPath, repoRoot) : null;
+  const newImage = status !== "deleted" ? getNewImage(mode, file.file_path, repoRoot) : null;
+  const oldMeta = oldImage !== null ? extractMetadata(oldImage.data, oldPath ?? file.file_path) : null;
+  const newMeta = newImage !== null ? extractMetadata(newImage.data, file.file_path) : null;
+  return c.json({
+    old: oldMeta ? formatMetadataLines(oldMeta) : null,
+    new: newMeta ? formatMetadataLines(newMeta) : null
+  });
+});
+imageRoutes.get("/image/:fileId/:side", async (c) => {
+  const fileId = c.req.param("fileId");
+  const side = c.req.param("side");
+  if (side !== "old" && side !== "new") return c.text("Invalid side", 400);
+  const file = await getReviewFile(fileId);
+  if (!file) return c.text("Not found", 404);
+  const repoRoot = c.get("repoRoot");
+  const review = await getReview(file.review_id);
+  if (!review) return c.text("Review not found", 404);
+  const mode = parseModeString(review.mode);
+  const diff = parseDiffData(file.diff_data);
+  const oldPath = diff?.oldPath ?? null;
+  const image = side === "old" ? getOldImage(mode, file.file_path, oldPath, repoRoot) : getNewImage(mode, file.file_path, repoRoot);
+  if (!image) return c.text("Image not available", 404);
+  if (isSvgFile(file.file_path)) {
+    try {
+      const png = await rasterizeSvg(image.data);
+      return new Response(new Uint8Array(png), {
+        headers: { "Content-Type": "image/png", "Cache-Control": "no-cache" }
+      });
+    } catch {
+      return c.text("SVG rasterization failed", 500);
+    }
+  }
+  const contentType = getContentType(file.file_path);
+  return new Response(new Uint8Array(image.data), {
+    headers: { "Content-Type": contentType, "Cache-Control": "no-cache" }
+  });
+});
+
+// src/routes/api/outline.ts
+init_queries();
+import { spawnSync as spawnSync7 } from "child_process";
+import { readFileSync as readFileSync8 } from "fs";
+import { Hono as Hono8 } from "hono";
+import { resolve as resolve5 } from "path";
+
 // src/outline/parser.ts
 var BRACE_LANGS = /* @__PURE__ */ new Set([
   "javascript",
@@ -4023,262 +4233,45 @@ function pushIndentSymbol(root, stack, sym, indent, lines, lineIdx) {
   stack.push({ symbol: sym, indent });
 }
 
-// src/routes/api.ts
-var apiRoutes = new Hono4();
-var VALID_CATEGORIES = ["bug", "fix", "style", "pattern-follow", "pattern-avoid", "note", "remember"];
-var VALID_SIDES = ["old", "new"];
-var VALID_FILE_STATUSES = ["pending", "reviewed"];
-function resolveReviewId(c) {
-  return c.req.query("reviewId") ?? c.get("reviewId");
-}
-apiRoutes.get("/reviews", async (c) => {
-  const repoRoot = c.get("repoRoot");
-  const reviews = await listReviews(repoRoot);
-  return c.json(reviews);
-});
-apiRoutes.get("/review", async (c) => {
-  const reviewId = resolveReviewId(c);
-  const review = await getReview(reviewId);
-  return c.json(review);
-});
-apiRoutes.post("/review/complete", async (c) => {
-  const reviewId = resolveReviewId(c);
-  const currentReviewId = c.get("currentReviewId");
-  const repoRoot = c.get("repoRoot");
-  await updateReviewStatus(reviewId, "completed");
-  const isCurrent = reviewId === currentReviewId;
-  const exportPath = await generateReviewExport(reviewId, repoRoot, isCurrent);
-  const gitignorePrompt = shouldPromptGitignore(repoRoot);
-  return c.json({ status: "completed", exportPath, isCurrent, reviewId, gitignorePrompt });
-});
-apiRoutes.post("/gitignore/add", (c) => {
+// src/routes/api/outline.ts
+var outlineRoutes = new Hono8();
+outlineRoutes.get("/outline/:fileId", async (c) => {
   const repoRoot = c.get("repoRoot");
-  addGlassboxToGitignore(repoRoot);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/gitignore/dismiss", (c) => {
-  const repoRoot = c.get("repoRoot");
-  dismissGitignorePrompt(repoRoot);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/review/reopen", async (c) => {
-  const reviewId = resolveReviewId(c);
-  await updateReviewStatus(reviewId, "in_progress");
-  return c.json({ status: "in_progress" });
+  const file = await getReviewFile(c.req.param("fileId"));
+  if (!file) return c.json({ error: "Not found" }, 404);
+  const diff = parseDiffData(file.diff_data);
+  const isDeleted = diff?.status === "deleted";
+  let content = "";
+  try {
+    content = isDeleted ? getFileContent(file.file_path, "HEAD", repoRoot) : getFileContent(file.file_path, "working", repoRoot);
+  } catch {
+  }
+  if (!content) return c.json({ symbols: [] });
+  const symbols = parseOutline(content, file.file_path);
+  return c.json({ symbols });
 });
-apiRoutes.post("/review/refresh", async (c) => {
+outlineRoutes.get("/symbol-definition", async (c) => {
+  const name = c.req.query("name");
+  const currentFileId = c.req.query("currentFileId");
+  if (name === void 0 || name === "") return c.json({ definitions: [] });
   const reviewId = resolveReviewId(c);
   const repoRoot = c.get("repoRoot");
-  const review = await getReview(reviewId);
-  if (!review) return c.json({ error: "Review not found" }, 404);
-  const mode = parseModeString(review.mode);
-  const headCommit = getHeadCommit(repoRoot);
-  const diffs = getFileDiffs(mode, repoRoot);
-  const result = await updateReviewDiffs(reviewId, diffs, headCommit);
-  return c.json({
-    updated: result.updated,
-    added: result.added,
-    stale: result.stale,
-    fileCount: diffs.length
-  });
-});
-apiRoutes.delete("/review/:id", async (c) => {
-  const reviewId = c.req.param("id");
-  const currentReviewId = c.get("currentReviewId");
-  if (reviewId === currentReviewId) {
-    return c.json({ error: "Cannot delete the current review" }, 400);
-  }
-  const repoRoot = c.get("repoRoot");
-  deleteReviewExport(reviewId, repoRoot);
-  await deleteReview(reviewId);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/reviews/delete-completed", async (c) => {
-  const currentReviewId = c.get("currentReviewId");
-  const repoRoot = c.get("repoRoot");
-  const reviews = await listReviews(repoRoot);
-  const toDelete = reviews.filter((r) => r.status === "completed" && r.id !== currentReviewId);
-  for (const r of toDelete) {
-    deleteReviewExport(r.id, repoRoot);
-    await deleteReview(r.id);
-  }
-  return c.json({ deleted: toDelete.length });
-});
-apiRoutes.post("/reviews/delete-all", async (c) => {
-  const currentReviewId = c.get("currentReviewId");
-  const repoRoot = c.get("repoRoot");
-  const reviews = await listReviews(repoRoot);
-  const toDelete = reviews.filter((r) => r.id !== currentReviewId);
-  for (const r of toDelete) {
-    deleteReviewExport(r.id, repoRoot);
-    await deleteReview(r.id);
-  }
-  return c.json({ deleted: toDelete.length });
-});
-apiRoutes.get("/files", async (c) => {
-  const reviewId = resolveReviewId(c);
-  const files = await getReviewFiles(reviewId);
-  const annotationCounts = {};
-  for (const file of files) {
-    const annotations = await getAnnotationsForFile(file.id);
-    annotationCounts[file.id] = annotations.length;
-  }
-  const staleCounts = await getStaleCountsForReview(reviewId);
-  return c.json({ files, annotationCounts, staleCounts });
-});
-apiRoutes.get("/files/:fileId", async (c) => {
-  const file = await getReviewFile(c.req.param("fileId"));
-  if (!file) return c.json({ error: "Not found" }, 404);
-  const annotations = await getAnnotationsForFile(file.id);
-  return c.json({ file, annotations });
-});
-apiRoutes.patch("/files/:fileId/status", async (c) => {
-  const { status } = await c.req.json();
-  if (!VALID_FILE_STATUSES.includes(status)) {
-    return c.json({ error: `status must be one of: ${VALID_FILE_STATUSES.join(", ")}` }, 400);
-  }
-  await updateFileStatus(c.req.param("fileId"), status);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/files/:fileId/reveal", async (c) => {
-  const file = await getReviewFile(c.req.param("fileId"));
-  if (!file) return c.json({ error: "Not found" }, 404);
-  const repoRoot = c.get("repoRoot");
-  const fullPath = resolve4(repoRoot, file.file_path);
-  try {
-    if (process.platform === "darwin") {
-      execFileSync("open", ["-R", fullPath]);
-    } else if (process.platform === "win32") {
-      execFileSync("explorer", ["/select," + fullPath]);
-    } else {
-      execFileSync("xdg-open", [resolve4(fullPath, "..")]);
-    }
-  } catch {
-  }
-  return c.json({ ok: true });
-});
-function autoExport(c) {
-  scheduleAutoExport(c.get("reviewId"), c.get("repoRoot"));
-}
-apiRoutes.post("/annotations", async (c) => {
-  const body = await c.req.json();
-  if (typeof body.reviewFileId !== "string" || body.reviewFileId === "") {
-    return c.json({ error: "reviewFileId must be a non-empty string" }, 400);
-  }
-  if (typeof body.lineNumber !== "number" || !Number.isInteger(body.lineNumber) || body.lineNumber < 1) {
-    return c.json({ error: "lineNumber must be a positive integer" }, 400);
-  }
-  if (!VALID_SIDES.includes(body.side)) {
-    return c.json({ error: `side must be one of: ${VALID_SIDES.join(", ")}` }, 400);
-  }
-  if (!VALID_CATEGORIES.includes(body.category)) {
-    return c.json({ error: `category must be one of: ${VALID_CATEGORIES.join(", ")}` }, 400);
-  }
-  if (typeof body.content !== "string" || body.content.trim() === "") {
-    return c.json({ error: "content must be a non-empty string" }, 400);
-  }
-  const annotation = await addAnnotation(
-    body.reviewFileId,
-    body.lineNumber,
-    body.side,
-    body.category,
-    body.content
-  );
-  autoExport(c);
-  return c.json(annotation, 201);
-});
-apiRoutes.patch("/annotations/:id", async (c) => {
-  const { content, category } = await c.req.json();
-  if (typeof content !== "string" || content.trim() === "") {
-    return c.json({ error: "content must be a non-empty string" }, 400);
-  }
-  if (!VALID_CATEGORIES.includes(category)) {
-    return c.json({ error: `category must be one of: ${VALID_CATEGORIES.join(", ")}` }, 400);
-  }
-  await updateAnnotation(c.req.param("id"), content, category);
-  autoExport(c);
-  return c.json({ ok: true });
-});
-apiRoutes.delete("/annotations/:id", async (c) => {
-  await deleteAnnotation(c.req.param("id"));
-  autoExport(c);
-  return c.json({ ok: true });
-});
-apiRoutes.patch("/annotations/:id/move", async (c) => {
-  const { lineNumber, side } = await c.req.json();
-  if (typeof lineNumber !== "number" || !Number.isInteger(lineNumber) || lineNumber < 1) {
-    return c.json({ error: "lineNumber must be a positive integer" }, 400);
-  }
-  if (!VALID_SIDES.includes(side)) {
-    return c.json({ error: `side must be one of: ${VALID_SIDES.join(", ")}` }, 400);
-  }
-  await moveAnnotation(c.req.param("id"), lineNumber, side);
-  autoExport(c);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/annotations/:id/keep", async (c) => {
-  await markAnnotationCurrent(c.req.param("id"));
-  autoExport(c);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/annotations/stale/delete-all", async (c) => {
-  const reviewId = resolveReviewId(c);
-  await deleteStaleAnnotations(reviewId);
-  autoExport(c);
-  return c.json({ ok: true });
-});
-apiRoutes.post("/annotations/stale/keep-all", async (c) => {
-  const reviewId = resolveReviewId(c);
-  await keepAllStaleAnnotations(reviewId);
-  autoExport(c);
-  return c.json({ ok: true });
-});
-apiRoutes.get("/annotations/all", async (c) => {
-  const reviewId = resolveReviewId(c);
-  const annotations = await getAnnotationsForReview(reviewId);
-  return c.json(annotations);
-});
-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 isDeleted = diff.status === "deleted";
-  let content = "";
-  try {
-    if (isDeleted) {
-      content = getFileContent(file.file_path, "HEAD", repoRoot);
-    } else {
-      content = getFileContent(file.file_path, "working", repoRoot);
-    }
-  } catch {
-  }
-  if (!content) return c.json({ symbols: [] });
-  const symbols = parseOutline(content, file.file_path);
-  return c.json({ symbols });
-});
-apiRoutes.get("/symbol-definition", async (c) => {
-  const name = c.req.query("name");
-  const currentFileId = c.req.query("currentFileId");
-  if (name === void 0 || name === "") return c.json({ definitions: [] });
-  const reviewId = resolveReviewId(c);
-  const repoRoot = c.get("repoRoot");
-  const definitions = [];
-  const searchedPaths = /* @__PURE__ */ new Set();
-  const reviewFiles = await getReviewFiles(reviewId);
-  for (const file of reviewFiles) {
-    searchedPaths.add(file.file_path);
-    const diff = JSON.parse(file.diff_data ?? "{}");
-    const isDeleted = diff.status === "deleted";
-    let content = "";
-    try {
-      content = isDeleted ? getFileContent(file.file_path, "HEAD", repoRoot) : getFileContent(file.file_path, "working", repoRoot);
-    } catch {
-      continue;
-    }
-    if (!content) continue;
-    const symbols = parseOutline(content, file.file_path);
-    collectDefinitions(symbols, name, file.id, file.file_path, definitions);
+  const definitions = [];
+  const searchedPaths = /* @__PURE__ */ new Set();
+  const reviewFiles = await getReviewFiles(reviewId);
+  for (const file of reviewFiles) {
+    searchedPaths.add(file.file_path);
+    const diff = parseDiffData(file.diff_data);
+    const isDeleted = diff?.status === "deleted";
+    let content = "";
+    try {
+      content = isDeleted ? getFileContent(file.file_path, "HEAD", repoRoot) : getFileContent(file.file_path, "working", repoRoot);
+    } catch {
+      continue;
+    }
+    if (!content) continue;
+    const symbols = parseOutline(content, file.file_path);
+    collectDefinitions(symbols, name, file.id, file.file_path, definitions);
   }
   if (definitions.length === 0) {
     try {
@@ -4289,7 +4282,7 @@ apiRoutes.get("/symbol-definition", async (c) => {
         if (!/\.(js|mjs|cjs|jsx|ts|tsx|mts|cts|java|go|rs|c|h|cpp|cc|cxx|hpp|cs|swift|php|kt|kts|scala|dart|groovy|py|rb)$/i.test(ext)) continue;
         let content = "";
         try {
-          content = readFileSync8(resolve4(repoRoot, filePath), "utf-8");
+          content = readFileSync8(resolve5(repoRoot, filePath), "utf-8");
         } catch {
           continue;
         }
@@ -4322,27 +4315,17 @@ function collectDefinitions(symbols, targetName, fileId, filePath, out) {
     }
   }
 }
-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 content = getFileContent(file.file_path, "working", repoRoot);
-  const allLines = content.split("\n");
-  const clampedStart = Math.max(1, startLine);
-  const clampedEnd = Math.min(allLines.length, endLine);
-  const lines = [];
-  for (let i = clampedStart; i <= clampedEnd; i++) {
-    lines.push({ num: i, content: allLines[i - 1] || "" });
-  }
-  return c.json({ lines });
-});
+
+// src/routes/api/project-settings.ts
+import { existsSync as existsSync6, mkdirSync as mkdirSync4, readFileSync as readFileSync9, writeFileSync as writeFileSync5 } from "fs";
+import { Hono as Hono9 } from "hono";
+import { join as join7 } from "path";
+var projectSettingsRoutes = new Hono9();
 function readProjectSettings(repoRoot) {
   const settingsPath = join7(repoRoot, ".glassbox", "settings.json");
   try {
     if (existsSync6(settingsPath)) {
-      return JSON.parse(readFileSync8(settingsPath, "utf-8"));
+      return JSON.parse(readFileSync9(settingsPath, "utf-8"));
     }
   } catch {
   }
@@ -4353,11 +4336,11 @@ function writeProjectSettings(repoRoot, settings) {
   mkdirSync4(dir, { recursive: true });
   writeFileSync5(join7(dir, "settings.json"), JSON.stringify(settings, null, 2), "utf-8");
 }
-apiRoutes.get("/project-settings", (c) => {
+projectSettingsRoutes.get("/project-settings", (c) => {
   const repoRoot = c.get("repoRoot");
   return c.json(readProjectSettings(repoRoot));
 });
-apiRoutes.patch("/project-settings", async (c) => {
+projectSettingsRoutes.patch("/project-settings", async (c) => {
   const repoRoot = c.get("repoRoot");
   const body = await c.req.json();
   if (body.appName !== void 0 && typeof body.appName !== "string") {
@@ -4368,120 +4351,146 @@ apiRoutes.patch("/project-settings", async (c) => {
   writeProjectSettings(repoRoot, current);
   return c.json(current);
 });
-apiRoutes.get("/image/:fileId/metadata", async (c) => {
-  const fileId = c.req.param("fileId");
-  const file = await getReviewFile(fileId);
-  if (!file) return c.json({ error: "Not found" }, 404);
+
+// src/routes/api/reviews.ts
+init_queries();
+import { Hono as Hono10 } from "hono";
+var reviewsRoutes = new Hono10();
+reviewsRoutes.get("/reviews", async (c) => {
   const repoRoot = c.get("repoRoot");
-  const review = await getReview(file.review_id);
+  const reviews = await listReviews(repoRoot);
+  return c.json(reviews);
+});
+reviewsRoutes.get("/review", async (c) => {
+  const reviewId = resolveReviewId(c);
+  const review = await getReview(reviewId);
+  return c.json(review);
+});
+reviewsRoutes.post("/review/complete", async (c) => {
+  const reviewId = resolveReviewId(c);
+  const currentReviewId = c.get("currentReviewId");
+  const repoRoot = c.get("repoRoot");
+  await updateReviewStatus(reviewId, "completed");
+  const isCurrent = reviewId === currentReviewId;
+  const exportPath = await generateReviewExport(reviewId, repoRoot, isCurrent);
+  const gitignorePrompt = shouldPromptGitignore(repoRoot);
+  return c.json({ status: "completed", exportPath, isCurrent, reviewId, gitignorePrompt });
+});
+reviewsRoutes.post("/gitignore/add", (c) => {
+  const repoRoot = c.get("repoRoot");
+  addGlassboxToGitignore(repoRoot);
+  return c.json({ ok: true });
+});
+reviewsRoutes.post("/gitignore/dismiss", (c) => {
+  const repoRoot = c.get("repoRoot");
+  dismissGitignorePrompt(repoRoot);
+  return c.json({ ok: true });
+});
+reviewsRoutes.post("/review/reopen", async (c) => {
+  const reviewId = resolveReviewId(c);
+  await updateReviewStatus(reviewId, "in_progress");
+  return c.json({ status: "in_progress" });
+});
+reviewsRoutes.post("/review/refresh", async (c) => {
+  const reviewId = resolveReviewId(c);
+  const repoRoot = c.get("repoRoot");
+  const review = await getReview(reviewId);
   if (!review) return c.json({ error: "Review not found" }, 404);
   const mode = parseModeString(review.mode);
-  const diff = JSON.parse(file.diff_data ?? "{}");
-  const oldPath = diff.oldPath ?? null;
-  const status = diff.status ?? "modified";
-  const oldImage = status !== "added" ? getOldImage(mode, file.file_path, oldPath, repoRoot) : null;
-  const newImage = status !== "deleted" ? getNewImage(mode, file.file_path, repoRoot) : null;
-  const oldMeta = oldImage !== null ? extractMetadata(oldImage.data, oldPath ?? file.file_path) : null;
-  const newMeta = newImage !== null ? extractMetadata(newImage.data, file.file_path) : null;
+  const headCommit = getHeadCommit(repoRoot);
+  const diffs = getFileDiffs(mode, repoRoot);
+  const result = await updateReviewDiffs(reviewId, diffs, headCommit);
   return c.json({
-    old: oldMeta ? formatMetadataLines(oldMeta) : null,
-    new: newMeta ? formatMetadataLines(newMeta) : null
+    updated: result.updated,
+    added: result.added,
+    stale: result.stale,
+    fileCount: diffs.length
   });
 });
-apiRoutes.get("/image/:fileId/:side", async (c) => {
-  const fileId = c.req.param("fileId");
-  const side = c.req.param("side");
-  if (side !== "old" && side !== "new") return c.text("Invalid side", 400);
-  const file = await getReviewFile(fileId);
-  if (!file) return c.text("Not found", 404);
+reviewsRoutes.delete("/review/:id", async (c) => {
+  const reviewId = c.req.param("id");
+  const currentReviewId = c.get("currentReviewId");
+  if (reviewId === currentReviewId) {
+    return c.json({ error: "Cannot delete the current review" }, 400);
+  }
   const repoRoot = c.get("repoRoot");
-  const review = await getReview(file.review_id);
-  if (!review) return c.text("Review not found", 404);
-  const mode = parseModeString(review.mode);
-  const diff = JSON.parse(file.diff_data ?? "{}");
-  const oldPath = diff.oldPath ?? null;
-  const image = side === "old" ? getOldImage(mode, file.file_path, oldPath, repoRoot) : getNewImage(mode, file.file_path, repoRoot);
-  if (!image) return c.text("Image not available", 404);
-  if (isSvgFile(file.file_path)) {
-    try {
-      const png = await rasterizeSvg(image.data);
-      return new Response(new Uint8Array(png), {
-        headers: { "Content-Type": "image/png", "Cache-Control": "no-cache" }
-      });
-    } catch {
-      return c.text("SVG rasterization failed", 500);
-    }
+  deleteReviewExport(reviewId, repoRoot);
+  await deleteReview(reviewId);
+  return c.json({ ok: true });
+});
+reviewsRoutes.post("/reviews/delete-completed", async (c) => {
+  const currentReviewId = c.get("currentReviewId");
+  const repoRoot = c.get("repoRoot");
+  const reviews = await listReviews(repoRoot);
+  const toDelete = reviews.filter((r) => r.status === "completed" && r.id !== currentReviewId);
+  for (const r of toDelete) {
+    deleteReviewExport(r.id, repoRoot);
+    await deleteReview(r.id);
   }
-  const contentType = getContentType(file.file_path);
-  return new Response(new Uint8Array(image.data), {
-    headers: { "Content-Type": contentType, "Cache-Control": "no-cache" }
-  });
+  return c.json({ deleted: toDelete.length });
 });
-function readGlobalConfig() {
-  const configPath = join7(homedir3(), ".glassbox", "config.json");
-  try {
-    if (existsSync6(configPath)) {
-      return JSON.parse(readFileSync8(configPath, "utf-8"));
-    }
-  } catch {
+reviewsRoutes.post("/reviews/delete-all", async (c) => {
+  const currentReviewId = c.get("currentReviewId");
+  const repoRoot = c.get("repoRoot");
+  const reviews = await listReviews(repoRoot);
+  const toDelete = reviews.filter((r) => r.id !== currentReviewId);
+  for (const r of toDelete) {
+    deleteReviewExport(r.id, repoRoot);
+    await deleteReview(r.id);
   }
-  return {};
-}
-function writeGlobalConfig(config) {
-  const configDir = join7(homedir3(), ".glassbox");
-  mkdirSync4(configDir, { recursive: true });
-  writeFileSync5(join7(configDir, "config.json"), JSON.stringify(config, null, 2), "utf-8");
-}
-apiRoutes.get("/share-prompt/state", (c) => {
+  return c.json({ deleted: toDelete.length });
+});
+
+// src/routes/api/share-prompt.ts
+import { Hono as Hono11 } from "hono";
+var sharePromptRoutes = new Hono11();
+sharePromptRoutes.get("/share-prompt/state", (c) => {
   const config = readGlobalConfig();
   const sp = config.sharePrompt;
   const dismissedAt = sp !== void 0 && typeof sp.dismissedAt === "number" ? sp.dismissedAt : null;
   const totalOpenMs = sp !== void 0 && typeof sp.totalOpenMs === "number" ? sp.totalOpenMs : 0;
   return c.json({ dismissedAt, totalOpenMs });
 });
-apiRoutes.post("/share-prompt/dismiss", (c) => {
-  const config = readGlobalConfig();
-  if (config.sharePrompt === void 0) config.sharePrompt = {};
-  const sp = config.sharePrompt;
-  sp.dismissedAt = Date.now();
-  writeGlobalConfig(config);
+sharePromptRoutes.post("/share-prompt/dismiss", (c) => {
+  updateGlobalConfig((config) => {
+    if (config.sharePrompt === void 0) config.sharePrompt = {};
+    config.sharePrompt.dismissedAt = Date.now();
+  });
   return c.json({ ok: true });
 });
-apiRoutes.post("/share-prompt/tick", async (c) => {
+sharePromptRoutes.post("/share-prompt/tick", async (c) => {
   const body = await c.req.json();
-  const config = readGlobalConfig();
-  if (config.sharePrompt === void 0) config.sharePrompt = {};
-  const sp = config.sharePrompt;
-  const current = typeof sp.totalOpenMs === "number" ? sp.totalOpenMs : 0;
-  sp.totalOpenMs = current + (body.sessionMs > 0 ? body.sessionMs : 0);
-  writeGlobalConfig(config);
-  return c.json({ totalOpenMs: sp.totalOpenMs });
+  let totalOpenMs = 0;
+  updateGlobalConfig((config) => {
+    if (config.sharePrompt === void 0) config.sharePrompt = {};
+    const sp = config.sharePrompt;
+    const current = typeof sp.totalOpenMs === "number" ? sp.totalOpenMs : 0;
+    const next = current + (body.sessionMs > 0 ? body.sessionMs : 0);
+    sp.totalOpenMs = next;
+    totalOpenMs = next;
+  });
+  return c.json({ totalOpenMs });
 });
 
+// src/routes/api.ts
+var apiRoutes = new Hono12();
+apiRoutes.route("/", reviewsRoutes);
+apiRoutes.route("/", filesRoutes);
+apiRoutes.route("/", annotationsRoutes);
+apiRoutes.route("/", outlineRoutes);
+apiRoutes.route("/", contextRoutes);
+apiRoutes.route("/", projectSettingsRoutes);
+apiRoutes.route("/", imageRoutes);
+apiRoutes.route("/", sharePromptRoutes);
+
 // src/routes/channel-api.ts
 import { spawnSync as spawnSync8 } from "child_process";
-import { existsSync as existsSync7, mkdirSync as mkdirSync5, readFileSync as readFileSync9, writeFileSync as writeFileSync6 } from "fs";
-import { Hono as Hono5 } from "hono";
-import { homedir as homedir4 } from "os";
+import { mkdirSync as mkdirSync5 } from "fs";
+import { Hono as Hono13 } from "hono";
 import { join as join8 } from "path";
-var channelApiRoutes = new Hono5();
-var CONFIG_DIR2 = join8(homedir4(), ".glassbox");
-var CONFIG_PATH2 = join8(CONFIG_DIR2, "config.json");
-function readGlobalConfig2() {
-  try {
-    if (existsSync7(CONFIG_PATH2)) {
-      return JSON.parse(readFileSync9(CONFIG_PATH2, "utf-8"));
-    }
-  } catch {
-  }
-  return {};
-}
-function writeGlobalConfig2(config) {
-  mkdirSync5(CONFIG_DIR2, { recursive: true });
-  writeFileSync6(CONFIG_PATH2, JSON.stringify(config, null, 2), "utf-8");
-}
+var channelApiRoutes = new Hono13();
 channelApiRoutes.get("/status", async (c) => {
-  const config = readGlobalConfig2();
+  const config = readGlobalConfig();
   const enabled = config.channelEnabled === true;
   const repoRoot = c.get("repoRoot");
   const dataDir = join8(repoRoot, ".glassbox");
@@ -4489,9 +4498,9 @@ channelApiRoutes.get("/status", async (c) => {
   return c.json({ enabled, connected });
 });
 channelApiRoutes.post("/enable", (c) => {
-  const config = readGlobalConfig2();
-  config.channelEnabled = true;
-  writeGlobalConfig2(config);
+  updateGlobalConfig((config) => {
+    config.channelEnabled = true;
+  });
   const repoRoot = c.get("repoRoot");
   const dataDir = join8(repoRoot, ".glassbox");
   mkdirSync5(dataDir, { recursive: true });
@@ -4499,9 +4508,9 @@ channelApiRoutes.post("/enable", (c) => {
   return c.json({ ok: true });
 });
 channelApiRoutes.post("/disable", (c) => {
-  const config = readGlobalConfig2();
-  config.channelEnabled = false;
-  writeGlobalConfig2(config);
+  updateGlobalConfig((config) => {
+    config.channelEnabled = false;
+  });
   const repoRoot = c.get("repoRoot");
   const dataDir = join8(repoRoot, ".glassbox");
   unregisterChannel(dataDir);
@@ -4509,6 +4518,9 @@ channelApiRoutes.post("/disable", (c) => {
 });
 channelApiRoutes.post("/trigger", async (c) => {
   const body = await c.req.json();
+  if (!isNonEmptyString(body.message)) {
+    return c.json({ error: "message must be a non-empty string" }, 400);
+  }
   const repoRoot = c.get("repoRoot");
   const dataDir = join8(repoRoot, ".glassbox");
   const sent = await triggerChannel(dataDir, body.message);
@@ -4539,8 +4551,8 @@ channelApiRoutes.get("/claude-check", (c) => {
 
 // src/routes/pages.tsx
 import { readFileSync as readFileSync11 } from "fs";
-import { Hono as Hono6 } from "hono";
-import { resolve as resolve5 } from "path";
+import { Hono as Hono14 } from "hono";
+import { resolve as resolve6 } from "path";
 
 // src/utils/escapeHtml.ts
 function escapeHtml(str) {
@@ -5173,147 +5185,64 @@ function UnifiedDiff({ hunks, annotationsByLine }) {
             "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}`] ?? [];
-          const segments = charDiffs.get(line);
-          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", "data-line-number": line.oldNum ?? "" }),
-                  /* @__PURE__ */ jsx("span", { className: "gutter-new", "data-line-number": line.newNum ?? "" }),
-                  /* @__PURE__ */ jsx("span", { className: "code", children: segments ? renderSegments(segments) : 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: /* @__PURE__ */ jsx(IconEdit, {}) }),
-          /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-icon btn-danger", "data-action": "delete", title: "Delete", children: /* @__PURE__ */ jsx(IconTrash, {}) })
-        ] })
-      ]
-    }
-  )) });
-}
-
-// 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 }) })
+              "@@ -",
+              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}`] ?? [];
+          const segments = charDiffs.get(line);
+          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", "data-line-number": line.oldNum ?? "" }),
+                  /* @__PURE__ */ jsx("span", { className: "gutter-new", "data-line-number": line.newNum ?? "" }),
+                  /* @__PURE__ */ jsx("span", { className: "code", children: segments ? renderSegments(segments) : line.content })
+                ]
+              }
+            ),
+            anns.length > 0 ? /* @__PURE__ */ jsx(AnnotationRows, { annotations: anns }) : null
+          ] });
+        })
       ] });
     }),
-    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
-      ] });
-    })
+    /* @__PURE__ */ jsx("div", { className: "hunk-separator hunk-expander-tail", "data-start": tailStart, children: "\u2195 Show remaining lines" })
   ] });
 }
-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 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: /* @__PURE__ */ jsx(IconEdit, {}) }),
+          /* @__PURE__ */ jsx("button", { className: "btn btn-xs btn-icon btn-danger", "data-action": "delete", title: "Delete", children: /* @__PURE__ */ jsx(IconTrash, {}) })
+        ] })
+      ]
+    }
+  )) });
 }
 
 // src/themes/built-in.ts
@@ -5786,39 +5715,23 @@ function themeToInlineStyle(colors) {
 }
 
 // src/themes/config.ts
-import { existsSync as existsSync8, mkdirSync as mkdirSync6, readdirSync, readFileSync as readFileSync10, unlinkSync as unlinkSync2, writeFileSync as writeFileSync7 } from "fs";
-import { homedir as homedir5 } from "os";
+import { existsSync as existsSync7, mkdirSync as mkdirSync6, readdirSync, readFileSync as readFileSync10, unlinkSync as unlinkSync2, writeFileSync as writeFileSync6 } from "fs";
 import { join as join9 } from "path";
-var CONFIG_DIR3 = join9(homedir5(), ".glassbox");
-var CONFIG_PATH3 = join9(CONFIG_DIR3, "config.json");
-var THEMES_DIR = join9(CONFIG_DIR3, "themes");
-function readConfigFile2() {
-  try {
-    if (existsSync8(CONFIG_PATH3)) {
-      return JSON.parse(readFileSync10(CONFIG_PATH3, "utf-8"));
-    }
-  } catch {
-  }
-  return {};
-}
-function writeConfigFile2(config) {
-  mkdirSync6(CONFIG_DIR3, { recursive: true });
-  writeFileSync7(CONFIG_PATH3, JSON.stringify(config, null, 2), "utf-8");
-}
+var THEMES_DIR = join9(GLOBAL_CONFIG_DIR, "themes");
 function getActiveThemeId() {
-  const config = readConfigFile2();
+  const config = readGlobalConfig();
   const theme = config.theme;
   const active = theme?.active;
   return active ?? DEFAULT_THEME_ID;
 }
 function setActiveThemeId(id) {
-  const config = readConfigFile2();
-  if (config.theme === void 0) config.theme = {};
-  config.theme.active = id;
-  writeConfigFile2(config);
+  updateGlobalConfig((config) => {
+    if (config.theme === void 0) config.theme = {};
+    config.theme.active = id;
+  });
 }
 function loadCustomThemes() {
-  if (!existsSync8(THEMES_DIR)) return [];
+  if (!existsSync7(THEMES_DIR)) return [];
   const themes = [];
   try {
     const files = readdirSync(THEMES_DIR).filter((f) => f.endsWith(".json"));
@@ -5838,17 +5751,17 @@ function loadCustomThemes() {
 function saveCustomTheme(theme) {
   mkdirSync6(THEMES_DIR, { recursive: true });
   const filePath = join9(THEMES_DIR, `${theme.id}.json`);
-  writeFileSync7(filePath, JSON.stringify(theme, null, 2), "utf-8");
+  writeFileSync6(filePath, JSON.stringify(theme, null, 2), "utf-8");
 }
 function deleteCustomTheme(id) {
   const filePath = join9(THEMES_DIR, `${id}.json`);
-  if (existsSync8(filePath)) {
+  if (existsSync7(filePath)) {
     unlinkSync2(filePath);
   }
 }
 function getCustomTheme(id) {
   const filePath = join9(THEMES_DIR, `${id}.json`);
-  if (!existsSync8(filePath)) return void 0;
+  if (!existsSync7(filePath)) return void 0;
   try {
     const data = JSON.parse(readFileSync10(filePath, "utf-8"));
     return { ...data, builtIn: false };
@@ -5958,20 +5871,92 @@ function ReviewHistory({ reviews, currentReviewId }) {
   ] });
 }
 
-// src/routes/pages.tsx
-init_queries();
-var pageRoutes = new Hono6();
-pageRoutes.get("/", async (c) => {
-  const reviewId = c.get("reviewId");
-  const review = await getReview(reviewId);
-  if (!review) return c.text("Review not found", 404);
-  const files = await getReviewFiles(reviewId);
-  const annotationCounts = {};
+// src/components/fileList.tsx
+function buildFileTree(files) {
+  const root = { name: "", children: [], files: [] };
   for (const f of files) {
-    const anns = await getAnnotationsForFile(f.id);
-    annotationCounts[f.id] = anns.length;
+    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;
   }
-  const html = /* @__PURE__ */ jsx(Layout, { title: `Glassbox - ${review.repo_name}`, reviewId, children: /* @__PURE__ */ jsx("div", { className: "review-app", "data-review-id": reviewId, children: [
+  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 = parseDiffData(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/reviewShell.tsx
+function ReviewShell({ reviewId, review, files, annotationCounts, staleCounts, footer }) {
+  return /* @__PURE__ */ jsx("div", { className: "review-app", "data-review-id": reviewId, children: [
     /* @__PURE__ */ jsx("div", { id: "update-banner", className: "update-banner", style: "display:none", children: [
       /* @__PURE__ */ jsx("span", { id: "update-banner-label", children: "Update available" }),
       /* @__PURE__ */ jsx("div", { className: "update-banner-actions", children: [
@@ -5989,12 +5974,9 @@ pageRoutes.get("/", async (c) => {
           ] })
         ] }),
         /* @__PURE__ */ jsx("div", { className: "file-filter", children: /* @__PURE__ */ jsx("input", { type: "text", className: "file-filter-input", id: "file-filter", placeholder: "Filter files..." }) }),
-        /* @__PURE__ */ jsx(FileList, { files, annotationCounts, staleCounts: {} }),
+        /* @__PURE__ */ jsx(FileList, { files, annotationCounts, staleCounts }),
         /* @__PURE__ */ jsx("div", { className: "sidebar-share", id: "sidebar-share" }),
-        /* @__PURE__ */ jsx("div", { className: "sidebar-footer", children: [
-          /* @__PURE__ */ jsx("button", { className: "btn btn-primary btn-complete", id: "complete-review", children: "Complete Review" }),
-          /* @__PURE__ */ jsx("a", { href: "/history", className: "btn btn-sm btn-link", children: "Review History" })
-        ] })
+        /* @__PURE__ */ jsx("div", { className: "sidebar-footer", children: footer })
       ] }),
       /* @__PURE__ */ jsx("div", { className: "sidebar-resize", id: "sidebar-resize" }),
       /* @__PURE__ */ jsx("main", { className: "main-content", children: [
@@ -6045,7 +6027,27 @@ pageRoutes.get("/", async (c) => {
         ] })
       ] })
     ] })
-  ] }) });
+  ] });
+}
+
+// src/routes/pages.tsx
+init_queries();
+var pageRoutes = new Hono14();
+pageRoutes.get("/", async (c) => {
+  const reviewId = c.get("reviewId");
+  const review = await getReview(reviewId);
+  if (!review) return c.text("Review not found", 404);
+  const files = await getReviewFiles(reviewId);
+  const annotationCounts = {};
+  for (const f of files) {
+    const anns = await getAnnotationsForFile(f.id);
+    annotationCounts[f.id] = anns.length;
+  }
+  const footer = /* @__PURE__ */ jsx(Fragment, { children: [
+    /* @__PURE__ */ jsx("button", { className: "btn btn-primary btn-complete", id: "complete-review", children: "Complete Review" }),
+    /* @__PURE__ */ jsx("a", { href: "/history", className: "btn btn-sm btn-link", children: "Review History" })
+  ] });
+  const html = /* @__PURE__ */ jsx(Layout, { title: `Glassbox - ${review.repo_name}`, reviewId, children: /* @__PURE__ */ jsx(ReviewShell, { reviewId, review, files, annotationCounts, staleCounts: {}, footer }) });
   return c.html(html.toString());
 });
 pageRoutes.get("/file/:fileId", async (c) => {
@@ -6055,7 +6057,7 @@ pageRoutes.get("/file/:fileId", async (c) => {
   const view = c.req.query("view");
   const file = await getReviewFile(fileId);
   if (!file) return c.text("File not found", 404);
-  const diff = JSON.parse(file.diff_data ?? "{}");
+  const diff = parseDiffData(file.diff_data) ?? {};
   if (view === "rendered" && isSvgFile(file.file_path)) {
     const repoRoot = c.get("repoRoot");
     const review = await getReview(file.review_id);
@@ -6119,7 +6121,7 @@ pageRoutes.get("/file-raw", (c) => {
   const repoRoot = c.get("repoRoot");
   let content;
   try {
-    content = readFileSync11(resolve5(repoRoot, filePath), "utf-8");
+    content = readFileSync11(resolve6(repoRoot, filePath), "utf-8");
   } catch {
     return c.text("File not found", 404);
   }
@@ -6160,82 +6162,12 @@ pageRoutes.get("/review/:reviewId", async (c) => {
     const anns = await getAnnotationsForFile(f.id);
     annotationCounts[f.id] = anns.length;
   }
-  const html = /* @__PURE__ */ jsx(Layout, { title: `Glassbox - ${review.repo_name}`, reviewId, children: /* @__PURE__ */ jsx("div", { className: "review-app", "data-review-id": reviewId, children: [
-    /* @__PURE__ */ jsx("div", { id: "update-banner", className: "update-banner", style: "display:none", children: [
-      /* @__PURE__ */ jsx("span", { id: "update-banner-label", children: "Update available" }),
-      /* @__PURE__ */ jsx("div", { className: "update-banner-actions", children: [
-        /* @__PURE__ */ jsx("button", { id: "update-install-btn", className: "btn btn-sm btn-accent", children: "Install Update" }),
-        /* @__PURE__ */ jsx("button", { id: "update-banner-dismiss", className: "btn btn-sm", children: "Later" })
-      ] })
-    ] }),
-    /* @__PURE__ */ jsx("div", { className: "review-body", children: [
-      /* @__PURE__ */ jsx("aside", { className: "sidebar", children: [
-        /* @__PURE__ */ jsx("div", { className: "sidebar-header", children: [
-          /* @__PURE__ */ jsx("h2", { children: review.repo_name }),
-          /* @__PURE__ */ jsx("span", { className: "review-mode", children: [
-            review.mode,
-            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..." }) }),
-        /* @__PURE__ */ jsx(FileList, { files, annotationCounts, staleCounts: {} }),
-        /* @__PURE__ */ jsx("div", { className: "sidebar-share", id: "sidebar-share" }),
-        /* @__PURE__ */ jsx("div", { className: "sidebar-footer", children: [
-          review.status === "completed" ? /* @__PURE__ */ jsx("button", { className: "btn btn-primary", id: "reopen-review", children: "Reopen Review" }) : /* @__PURE__ */ jsx("button", { className: "btn btn-primary btn-complete", id: "complete-review", children: "Complete Review" }),
-          /* @__PURE__ */ jsx("a", { href: "/history", className: "btn btn-sm btn-link", children: "Review History" }),
-          /* @__PURE__ */ jsx("a", { href: "/", className: "btn btn-sm btn-link", children: "Back to current review" })
-        ] })
-      ] }),
-      /* @__PURE__ */ jsx("div", { className: "sidebar-resize", id: "sidebar-resize" }),
-      /* @__PURE__ */ jsx("main", { className: "main-content", children: [
-        /* @__PURE__ */ jsx("div", { className: "welcome-message", children: [
-          /* @__PURE__ */ jsx("h3", { children: "Select a file to begin reviewing" }),
-          /* @__PURE__ */ jsx("p", { children: [
-            files.length,
-            " file(s) to review"
-          ] }),
-          /* @__PURE__ */ jsx("p", { className: "progress-summary", id: "progress-summary" })
-        ] }),
-        /* @__PURE__ */ jsx("div", { className: "diff-nav-bar", id: "diff-nav-bar", style: "display:none", children: [
-          /* @__PURE__ */ jsx("button", { className: "nav-btn disabled", id: "nav-back-btn", disabled: true, title: "Back", children: /* @__PURE__ */ jsx("svg", { xmlns: "http://www.w3.org/2000/svg", width: "14", height: "14", viewBox: "0 0 24 24", fill: "none", stroke: "currentColor", strokeWidth: "2", strokeLinecap: "round", strokeLinejoin: "round", children: /* @__PURE__ */ jsx("path", { d: "m15 18-6-6 6-6" }) }) }),
-          /* @__PURE__ */ jsx("button", { className: "nav-btn disabled", id: "nav-forward-btn", disabled: true, title: "Forward", children: /* @__PURE__ */ jsx("svg", { xmlns: "http://www.w3.org/2000/svg", width: "14", height: "14", viewBox: "0 0 24 24", fill: "none", stroke: "currentColor", strokeWidth: "2", strokeLinecap: "round", strokeLinejoin: "round", children: /* @__PURE__ */ jsx("path", { d: "m9 18 6-6-6-6" }) }) }),
-          /* @__PURE__ */ jsx("span", { className: "nav-file-path", id: "nav-file-path" })
-        ] }),
-        /* @__PURE__ */ jsx("div", { className: "diff-container", id: "diff-container", style: "display:none" }),
-        /* @__PURE__ */ jsx("div", { className: "diff-toolbar", id: "diff-toolbar", style: "display:none", children: [
-          /* @__PURE__ */ jsx("div", { className: "diff-toolbar-svg-toggle", style: "display:none", children: /* @__PURE__ */ jsx("div", { className: "segmented-control", children: [
-            /* @__PURE__ */ jsx("button", { className: "segment active", "data-svg-mode": "code", children: "Code" }),
-            /* @__PURE__ */ jsx("button", { className: "segment", "data-svg-mode": "rendered", children: "Rendered" })
-          ] }) }),
-          /* @__PURE__ */ jsx("div", { className: "diff-toolbar-text", children: [
-            /* @__PURE__ */ jsx("div", { className: "diff-toolbar-left", children: [
-              /* @__PURE__ */ jsx("div", { className: "segmented-control", children: [
-                /* @__PURE__ */ jsx("button", { className: "segment active", "data-diff-mode": "split", children: "Split" }),
-                /* @__PURE__ */ jsx("button", { className: "segment", "data-diff-mode": "unified", children: "Unified" })
-              ] }),
-              /* @__PURE__ */ jsx("button", { className: "toolbar-btn", id: "wrap-toggle", children: "Wrap" }),
-              /* @__PURE__ */ jsx("button", { className: "toolbar-btn", id: "whitespace-toggle", children: "Ignore Whitespace" })
-            ] }),
-            /* @__PURE__ */ jsx("div", { className: "diff-toolbar-right", children: /* @__PURE__ */ jsx("button", { className: "toolbar-btn", id: "language-btn", children: "Plain Text" }) })
-          ] }),
-          /* @__PURE__ */ jsx("div", { className: "diff-toolbar-image", style: "display:none", children: [
-            /* @__PURE__ */ jsx("div", { className: "diff-toolbar-left", children: /* @__PURE__ */ jsx("div", { className: "segmented-control", children: [
-              /* @__PURE__ */ jsx("button", { className: "segment active", "data-image-mode": "metadata", children: "Metadata" }),
-              /* @__PURE__ */ jsx("button", { className: "segment", "data-image-mode": "difference", children: "Difference" }),
-              /* @__PURE__ */ jsx("button", { className: "segment", "data-image-mode": "slice", children: "Slice" }),
-              /* @__PURE__ */ jsx("button", { className: "segment", "data-image-mode": "image", style: "display:none", children: "Image" })
-            ] }) }),
-            /* @__PURE__ */ jsx("div", { className: "diff-toolbar-right", children: [
-              /* @__PURE__ */ jsx("button", { className: "image-zoom-btn", "data-zoom-action": "out", title: "Zoom out", children: /* @__PURE__ */ jsx(IconZoomOut, {}) }),
-              /* @__PURE__ */ jsx("button", { className: "image-zoom-btn", "data-zoom-action": "fit", title: "Fit to view", children: /* @__PURE__ */ jsx(IconFit, {}) }),
-              /* @__PURE__ */ jsx("button", { className: "image-zoom-btn", "data-zoom-action": "actual", title: "Actual size (1:1)", children: /* @__PURE__ */ jsx(IconActualSize, {}) }),
-              /* @__PURE__ */ jsx("button", { className: "image-zoom-btn", "data-zoom-action": "in", title: "Zoom in", children: /* @__PURE__ */ jsx(IconZoomIn, {}) })
-            ] })
-          ] })
-        ] })
-      ] })
-    ] })
-  ] }) });
+  const footer = /* @__PURE__ */ jsx(Fragment, { children: [
+    review.status === "completed" ? /* @__PURE__ */ jsx("button", { className: "btn btn-primary", id: "reopen-review", children: "Reopen Review" }) : /* @__PURE__ */ jsx("button", { className: "btn btn-primary btn-complete", id: "complete-review", children: "Complete Review" }),
+    /* @__PURE__ */ jsx("a", { href: "/history", className: "btn btn-sm btn-link", children: "Review History" }),
+    /* @__PURE__ */ jsx("a", { href: "/", className: "btn btn-sm btn-link", children: "Back to current review" })
+  ] });
+  const html = /* @__PURE__ */ jsx(Layout, { title: `Glassbox - ${review.repo_name}`, reviewId, children: /* @__PURE__ */ jsx(ReviewShell, { reviewId, review, files, annotationCounts, staleCounts: {}, footer }) });
   return c.html(html.toString());
 });
 pageRoutes.get("/history", async (c) => {
@@ -6247,8 +6179,8 @@ pageRoutes.get("/history", async (c) => {
 });
 
 // src/routes/theme-api.ts
-import { Hono as Hono7 } from "hono";
-var themeApiRoutes = new Hono7();
+import { Hono as Hono15 } from "hono";
+var themeApiRoutes = new Hono15();
 function validateColors(colors) {
   if (typeof colors !== "object" || colors === null || Array.isArray(colors)) {
     return "colors must be an object";
@@ -6284,7 +6216,7 @@ themeApiRoutes.get("/active", (c) => {
 });
 themeApiRoutes.post("/active", async (c) => {
   const body = await c.req.json();
-  if (typeof body.id !== "string" || body.id === "") return c.json({ error: "id must be a non-empty string" }, 400);
+  if (!isNonEmptyString(body.id)) return c.json({ error: "id must be a non-empty string" }, 400);
   const theme = resolveTheme(body.id);
   if (!theme) return c.json({ error: "Theme not found" }, 404);
   setActiveThemeId(body.id);
@@ -6292,8 +6224,8 @@ themeApiRoutes.post("/active", async (c) => {
 });
 themeApiRoutes.post("/", async (c) => {
   const body = await c.req.json();
-  if (typeof body.sourceId !== "string" || body.sourceId === "") return c.json({ error: "sourceId must be a non-empty string" }, 400);
-  if (body.name !== void 0 && (typeof body.name !== "string" || body.name.trim() === "")) {
+  if (!isNonEmptyString(body.sourceId)) return c.json({ error: "sourceId must be a non-empty string" }, 400);
+  if (body.name !== void 0 && !isNonEmptyString(body.name)) {
     return c.json({ error: "name must be a non-empty string when provided" }, 400);
   }
   const source = resolveTheme(body.sourceId);
@@ -6313,7 +6245,7 @@ themeApiRoutes.post("/", async (c) => {
 themeApiRoutes.post("/:id/edit", async (c) => {
   const id = c.req.param("id");
   const body = await c.req.json();
-  if (body.name !== void 0 && (typeof body.name !== "string" || body.name.trim() === "")) {
+  if (body.name !== void 0 && !isNonEmptyString(body.name)) {
     return c.json({ error: "name must be a non-empty string when provided" }, 400);
   }
   if (body.colors !== void 0) {
@@ -6353,7 +6285,7 @@ themeApiRoutes.patch("/:id", async (c) => {
     return c.json({ error: "Theme not found" }, 404);
   }
   const body = await c.req.json();
-  if (body.name !== void 0 && (typeof body.name !== "string" || body.name.trim() === "")) {
+  if (body.name !== void 0 && !isNonEmptyString(body.name)) {
     return c.json({ error: "name must be a non-empty string when provided" }, 400);
   }
   if (body.colors !== void 0) {
@@ -6382,10 +6314,10 @@ themeApiRoutes.delete("/:id", (c) => {
 
 // src/server.ts
 function tryServe(appFetch, port) {
-  return new Promise((resolve7, reject) => {
+  return new Promise((resolve8, reject) => {
     const server = serve({ fetch: appFetch, port, hostname: "127.0.0.1" });
     server.on("listening", () => {
-      resolve7(port);
+      resolve8(port);
     });
     server.on("error", (err) => {
       if (err.code === "EADDRINUSE") {
@@ -6397,7 +6329,7 @@ function tryServe(appFetch, port) {
   });
 }
 async function startServer(port, reviewId, repoRoot, options) {
-  const app = new Hono8();
+  const app = new Hono16();
   app.use("*", async (c, next) => {
     c.set("reviewId", reviewId);
     c.set("currentReviewId", reviewId);
@@ -6405,7 +6337,7 @@ async function startServer(port, reviewId, repoRoot, options) {
     await next();
   });
   const selfDir = dirname2(fileURLToPath2(import.meta.url));
-  const distDir = existsSync9(join10(selfDir, "client", "styles.css")) ? join10(selfDir, "client") : join10(selfDir, "..", "dist", "client");
+  const distDir = existsSync8(join10(selfDir, "client", "styles.css")) ? join10(selfDir, "client") : join10(selfDir, "..", "dist", "client");
   app.get("/static/styles.css", (c) => {
     const css = readFileSync12(join10(distDir, "styles.css"), "utf-8");
     return c.text(css, 200, { "Content-Type": "text/css", "Cache-Control": "no-cache" });
@@ -6447,13 +6379,10 @@ async function startServer(port, reviewId, repoRoot, options) {
   Glassbox running at ${url}
 `);
   try {
-    const homePath = join10(homedir6(), ".glassbox", "config.json");
-    if (existsSync9(homePath)) {
-      const globalConfig = JSON.parse(readFileSync12(homePath, "utf-8"));
-      if (globalConfig.channelEnabled === true) {
-        const dataDir = join10(repoRoot, ".glassbox");
-        registerChannel(dataDir);
-      }
+    const globalConfig = readGlobalConfig();
+    if (globalConfig.channelEnabled === true) {
+      const dataDir = join10(repoRoot, ".glassbox");
+      registerChannel(dataDir);
     }
   } catch {
   }
@@ -6464,7 +6393,7 @@ async function startServer(port, reviewId, repoRoot, options) {
 }
 
 // src/skills.ts
-import { existsSync as existsSync10, mkdirSync as mkdirSync7, readFileSync as readFileSync13, writeFileSync as writeFileSync8 } from "fs";
+import { existsSync as existsSync9, mkdirSync as mkdirSync7, readFileSync as readFileSync13, writeFileSync as writeFileSync7 } from "fs";
 import { join as join11 } from "path";
 var SKILL_VERSION = 1;
 function versionHeader() {
@@ -6476,14 +6405,14 @@ function parseVersionHeader(content) {
   return parseInt(match[1], 10);
 }
 function updateFile(path, content) {
-  if (existsSync10(path)) {
+  if (existsSync9(path)) {
     const existing = readFileSync13(path, "utf-8");
     const version = parseVersionHeader(existing);
     if (version !== null && version >= SKILL_VERSION) {
       return false;
     }
   }
-  writeFileSync8(path, content, "utf-8");
+  writeFileSync7(path, content, "utf-8");
   return true;
 }
 function skillBody() {
@@ -6565,28 +6494,28 @@ function ensureWindsurfRules(cwd) {
 function ensureSkills() {
   const cwd = process.cwd();
   const platforms = [];
-  if (existsSync10(join11(cwd, ".claude"))) {
+  if (existsSync9(join11(cwd, ".claude"))) {
     if (ensureClaudeSkills(cwd)) platforms.push("Claude Code");
   }
-  if (existsSync10(join11(cwd, ".cursor"))) {
+  if (existsSync9(join11(cwd, ".cursor"))) {
     if (ensureCursorRules(cwd)) platforms.push("Cursor");
   }
-  if (existsSync10(join11(cwd, ".github", "prompts")) || existsSync10(join11(cwd, ".github", "copilot-instructions.md"))) {
+  if (existsSync9(join11(cwd, ".github", "prompts")) || existsSync9(join11(cwd, ".github", "copilot-instructions.md"))) {
     if (ensureCopilotPrompts(cwd)) platforms.push("GitHub Copilot");
   }
-  if (existsSync10(join11(cwd, ".windsurf"))) {
+  if (existsSync9(join11(cwd, ".windsurf"))) {
     if (ensureWindsurfRules(cwd)) platforms.push("Windsurf");
   }
   return platforms;
 }
 
 // src/update-check.ts
-import { existsSync as existsSync11, mkdirSync as mkdirSync8, readFileSync as readFileSync14, writeFileSync as writeFileSync9 } from "fs";
+import { existsSync as existsSync10, mkdirSync as mkdirSync8, readFileSync as readFileSync14, writeFileSync as writeFileSync8 } from "fs";
 import { get } from "https";
-import { homedir as homedir7 } from "os";
+import { homedir as homedir3 } from "os";
 import { dirname as dirname3, join as join12 } from "path";
 import { fileURLToPath as fileURLToPath3 } from "url";
-var DATA_DIR = join12(homedir7(), ".glassbox");
+var DATA_DIR = join12(homedir3(), ".glassbox");
 var CHECK_FILE = join12(DATA_DIR, "last-update-check");
 var PACKAGE_NAME = "glassbox";
 function getCurrentVersion() {
@@ -6600,7 +6529,7 @@ function getCurrentVersion() {
 }
 function getLastCheckDate() {
   try {
-    if (existsSync11(CHECK_FILE)) {
+    if (existsSync10(CHECK_FILE)) {
       return readFileSync14(CHECK_FILE, "utf-8").trim();
     }
   } catch {
@@ -6609,7 +6538,7 @@ function getLastCheckDate() {
 }
 function saveCheckDate() {
   mkdirSync8(DATA_DIR, { recursive: true });
-  writeFileSync9(CHECK_FILE, (/* @__PURE__ */ new Date()).toISOString().slice(0, 10), "utf-8");
+  writeFileSync8(CHECK_FILE, (/* @__PURE__ */ new Date()).toISOString().slice(0, 10), "utf-8");
 }
 function isFirstUseToday() {
   const last = getLastCheckDate();
@@ -6618,10 +6547,10 @@ function isFirstUseToday() {
   return last !== today;
 }
 function fetchLatestVersion() {
-  return new Promise((resolve7) => {
+  return new Promise((resolve8) => {
     const req = get(`https://registry.npmjs.org/${PACKAGE_NAME}/latest`, { timeout: 5e3 }, (res) => {
       if (res.statusCode !== 200) {
-        resolve7(null);
+        resolve8(null);
         return;
       }
       let data = "";
@@ -6630,18 +6559,18 @@ function fetchLatestVersion() {
       });
       res.on("end", () => {
         try {
-          resolve7(JSON.parse(data).version);
+          resolve8(JSON.parse(data).version);
         } catch {
-          resolve7(null);
+          resolve8(null);
         }
       });
     });
     req.on("error", () => {
-      resolve7(null);
+      resolve8(null);
     });
     req.on("timeout", () => {
       req.destroy();
-      resolve7(null);
+      resolve8(null);
     });
   });
 }
@@ -6779,7 +6708,7 @@ function parseArgs(argv) {
         port = parseInt(args[++i], 10);
         break;
       case "--data-dir":
-        dataDir = resolve6(args[++i]);
+        dataDir = resolve7(args[++i]);
         break;
       case "--resume":
         resume = true;
@@ -6835,7 +6764,7 @@ async function main() {
     console.log("AI service test mode enabled \u2014 using mock AI responses");
   }
   if (debug) {
-    console.log(`[debug] Build timestamp: ${"2026-04-08T04:46:44.646Z"}`);
+    console.log(`[debug] Build timestamp: ${"2026-05-02T23:58:45.229Z"}`);
   }
   if (projectDir !== null) {
     process.chdir(projectDir);