glassbox 0.16.0 → 0.17.0-beta.3

package/dist/cli.js

@@ -46,6 +46,8 @@ var init_schema = __esm({
   CREATE TABLE IF NOT EXISTS annotations (
     id TEXT PRIMARY KEY,
     review_file_id TEXT NOT NULL REFERENCES review_files(id) ON DELETE CASCADE,
+    -- 0 marks an image-level annotation (doc 23); line annotations use the
+    -- real 1-based line number.
     line_number INTEGER NOT NULL,
     side TEXT NOT NULL DEFAULT 'new',
     category TEXT NOT NULL DEFAULT 'note',
@@ -53,6 +55,9 @@ var init_schema = __esm({
     is_stale BOOLEAN NOT NULL DEFAULT FALSE,
     original_content TEXT,
     reply_to_note_id TEXT,
+    -- JSON {x,y,w,h} normalized fractions for an image-region annotation
+    -- (doc 23), or NULL for line annotations and general image comments.
+    region_data TEXT,
     created_at TIMESTAMP NOT NULL DEFAULT NOW(),
     updated_at TIMESTAMP NOT NULL DEFAULT NOW()
   );
@@ -148,6 +153,7 @@ async function initSchema(db2) {
   await addColumnIfMissing(db2, "annotations", "is_stale", "BOOLEAN NOT NULL DEFAULT FALSE");
   await addColumnIfMissing(db2, "annotations", "original_content", "TEXT");
   await addColumnIfMissing(db2, "annotations", "reply_to_note_id", "TEXT");
+  await addColumnIfMissing(db2, "annotations", "region_data", "TEXT");
   await addColumnIfMissing(db2, "ai_file_scores", "notes", "TEXT");
   await addColumnIfMissing(db2, "ai_analyses", "progress_completed", "INTEGER NOT NULL DEFAULT 0");
   await addColumnIfMissing(db2, "ai_analyses", "progress_total", "INTEGER NOT NULL DEFAULT 0");
@@ -15382,7 +15388,7 @@ function parseJsonColumn(schema, raw2) {
   const result = schema.safeParse(parsed);
   return result.success ? result.data : null;
 }
-var TimestampSchema, ReviewSchema, ReviewFileSchema, AnnotationSchema, AnnotationWithFilePathSchema, AIAnalysisSchema, AIFileScoreSchema, UserPreferencesSchema, DimensionScoresSchema, FileScoreNotesSchema;
+var TimestampSchema, ReviewSchema, ReviewFileSchema, ImageRegionSchema, AnnotationSchema, AnnotationWithFilePathSchema, AIAnalysisSchema, AIFileScoreSchema, UserPreferencesSchema, DimensionScoresSchema, FileScoreNotesSchema;
 var init_schemas3 = __esm({
   "src/db/schemas.ts"() {
     "use strict";
@@ -15414,10 +15420,24 @@ var init_schemas3 = __esm({
       diff_data: external_exports.string().nullable(),
       created_at: TimestampSchema
     });
+    ImageRegionSchema = external_exports.object({
+      x: external_exports.number().min(0).max(1),
+      y: external_exports.number().min(0).max(1),
+      w: external_exports.number().min(0).max(1),
+      h: external_exports.number().min(0).max(1),
+      // Optional per-side scope (doc 23 §23.6 / §23.10): `old` = applies to the A
+      // (old) image only, `new` = applies to the B (new) image only. Absent means
+      // the region applies to both sides (the default), which is how every region
+      // created before this field existed is interpreted.
+      side: external_exports.enum(["old", "new"]).optional()
+    });
     AnnotationSchema = external_exports.object({
       id: external_exports.string(),
-      review_file_id: external_exports.string(),
+      // 0 marks an image-level annotation (doc 23): a general image comment, or —
+      // when `region_data` is set — a comment anchored to a rectangle on the image.
+      // Line-anchored text-diff annotations use the real 1-based line number.
       line_number: external_exports.number(),
+      review_file_id: external_exports.string(),
       side: external_exports.string(),
       category: external_exports.string(),
       content: external_exports.string(),
@@ -15427,6 +15447,10 @@ var init_schemas3 = __esm({
       // threading), or null for a normal annotation. `.default(null)` tolerates
       // rows written before the column existed.
       reply_to_note_id: external_exports.string().nullable().default(null),
+      // JSON-encoded {@link ImageRegion} for image-region annotations (doc 23), or
+      // null for line annotations and general image comments. `.default(null)`
+      // tolerates rows written before the column existed.
+      region_data: external_exports.string().nullable().default(null),
       created_at: TimestampSchema,
       updated_at: TimestampSchema
     });
@@ -15501,6 +15525,7 @@ __export(queries_exports, {
   markAnnotationStale: () => markAnnotationStale,
   moveAnnotation: () => moveAnnotation,
   updateAnnotation: () => updateAnnotation,
+  updateAnnotationRegion: () => updateAnnotationRegion,
   updateFileDiff: () => updateFileDiff,
   updateFileStatus: () => updateFileStatus,
   updateReviewHead: () => updateReviewHead,
@@ -15598,13 +15623,22 @@ async function deleteReviewFile(id) {
   await db2.query("DELETE FROM annotations WHERE review_file_id = $1", [id]);
   await db2.query("DELETE FROM review_files WHERE id = $1", [id]);
 }
-async function addAnnotation(reviewFileId, lineNumber, side, category, content, replyToNoteId) {
+async function addAnnotation(reviewFileId, lineNumber, side, category, content, replyToNoteId, region) {
   const db2 = await getDb();
   const id = generateId();
   const result = await db2.query(
-    `INSERT INTO annotations (id, review_file_id, line_number, side, category, content, reply_to_note_id)
-     VALUES ($1, $2, $3, $4, $5, $6, $7) RETURNING *`,
-    [id, reviewFileId, lineNumber, side, category, content, replyToNoteId ?? null]
+    `INSERT INTO annotations (id, review_file_id, line_number, side, category, content, reply_to_note_id, region_data)
+     VALUES ($1, $2, $3, $4, $5, $6, $7, $8) RETURNING *`,
+    [
+      id,
+      reviewFileId,
+      lineNumber,
+      side,
+      category,
+      content,
+      replyToNoteId ?? null,
+      region !== void 0 ? JSON.stringify(region) : null
+    ]
   );
   const annotation = parseRow(AnnotationSchema, result.rows[0]);
   if (annotation === void 0) throw new Error("addAnnotation: INSERT did not return a row");
@@ -15640,6 +15674,13 @@ async function deleteAnnotation(id) {
   const db2 = await getDb();
   await db2.query("DELETE FROM annotations WHERE id = $1", [id]);
 }
+async function updateAnnotationRegion(id, region) {
+  const db2 = await getDb();
+  await db2.query(
+    "UPDATE annotations SET region_data = $1, updated_at = NOW() WHERE id = $2",
+    [JSON.stringify(region), id]
+  );
+}
 async function moveAnnotation(id, lineNumber, side) {
   const db2 = await getDb();
   await db2.query(
@@ -15725,6 +15766,48 @@ var init_queries = __esm({
   }
 });
 
+// src/git/image-blobs.ts
+var image_blobs_exports = {};
+__export(image_blobs_exports, {
+  clearImageBlobs: () => clearImageBlobs,
+  readImageBlob: () => readImageBlob,
+  writeImageBlob: () => writeImageBlob
+});
+import { existsSync as existsSync2, mkdirSync as mkdirSync3, readFileSync as readFileSync2, rmSync as rmSync2, writeFileSync as writeFileSync2 } from "fs";
+import { join as join3 } from "path";
+function blobDir(dataDir) {
+  return join3(dataDir, "image-blobs");
+}
+function blobName(fileId, side) {
+  return `${fileId.replace(/[^a-z0-9]/gi, "")}-${side}`;
+}
+function writeImageBlob(dataDir, fileId, side, bytes) {
+  if (bytes.length === 0) return;
+  const dir = blobDir(dataDir);
+  mkdirSync3(dir, { recursive: true });
+  writeFileSync2(join3(dir, blobName(fileId, side)), bytes);
+}
+function readImageBlob(dataDir, fileId, side) {
+  const path = join3(blobDir(dataDir), blobName(fileId, side));
+  if (!existsSync2(path)) return null;
+  try {
+    return readFileSync2(path);
+  } catch {
+    return null;
+  }
+}
+function clearImageBlobs(dataDir) {
+  try {
+    rmSync2(blobDir(dataDir), { recursive: true, force: true });
+  } catch {
+  }
+}
+var init_image_blobs = __esm({
+  "src/git/image-blobs.ts"() {
+    "use strict";
+  }
+});
+
 // src/review-notes/types.ts
 function isNoteKind(value) {
   return NOTE_KINDS.includes(value);
@@ -15848,18 +15931,18 @@ var init_view = __esm({
 // src/review-notes/store.ts
 import { spawnSync as spawnSync5 } from "child_process";
 import { createHash } from "crypto";
-import { existsSync as existsSync5, mkdirSync as mkdirSync4, readdirSync, readFileSync as readFileSync5, statSync as statSync2, unlinkSync, writeFileSync as writeFileSync5 } from "fs";
-import { dirname as dirname3, join as join6 } from "path";
+import { existsSync as existsSync6, mkdirSync as mkdirSync5, readdirSync, readFileSync as readFileSync6, statSync as statSync2, unlinkSync, writeFileSync as writeFileSync6 } from "fs";
+import { dirname as dirname3, join as join7 } from "path";
 function sanitizeRel(file2) {
   const rel = file2.replace(/\\/g, "/").replace(/^\/+/, "").replace(/(^|\/)\.\.(?=\/|$)/g, "$1_");
   return rel === "" ? "file" : rel;
 }
 function shardPath(repoRoot, safeRel, index) {
-  return join6(repoRoot, NOTES_SUBDIR, `${safeRel}.${String(index).padStart(6, "0")}.sarif`);
+  return join7(repoRoot, NOTES_SUBDIR, `${safeRel}.${String(index).padStart(6, "0")}.sarif`);
 }
 function listShardIndices(repoRoot, safeRel) {
-  const dir = join6(repoRoot, NOTES_SUBDIR, dirname3(safeRel));
-  if (!existsSync5(dir)) return [];
+  const dir = join7(repoRoot, NOTES_SUBDIR, dirname3(safeRel));
+  if (!existsSync6(dir)) return [];
   const base = safeRel.split("/").pop() ?? safeRel;
   const indices = [];
   for (const entry of readdirSync(dir)) {
@@ -15873,7 +15956,7 @@ function totalResults(log) {
   return log.runs.reduce((sum, run) => sum + run.results.length, 0);
 }
 function readLog(path) {
-  const raw2 = JSON.parse(readFileSync5(path, "utf-8"));
+  const raw2 = JSON.parse(readFileSync6(path, "utf-8"));
   const parsed = SarifLogShapeSchema.safeParse(raw2);
   if (!parsed.success) {
     throw new Error(`existing notes shard is not a SARIF log we recognize, refusing to overwrite: ${path}`);
@@ -15899,7 +15982,7 @@ function gitValue(repoRoot, args) {
 }
 function anchorSnippet(repoRoot, safeRel, startLine, endLine) {
   try {
-    const lines = readFileSync5(join6(repoRoot, safeRel), "utf-8").split("\n");
+    const lines = readFileSync6(join7(repoRoot, safeRel), "utf-8").split("\n");
     const slice = lines.slice(Math.max(0, startLine - 1), Math.max(startLine, endLine));
     if (slice.length === 0) return {};
     const snippet = slice.join("\n");
@@ -15937,7 +16020,7 @@ function writeReviewNote(repoRoot, input, opts = {}) {
   let index = indices.length > 0 ? indices[indices.length - 1] : 0;
   let path = shardPath(repoRoot, safeRel, index);
   let log;
-  if (existsSync5(path)) {
+  if (existsSync6(path)) {
     log = readLog(path);
     if (totalResults(log) >= cap) {
       index += 1;
@@ -15948,12 +16031,12 @@ function writeReviewNote(repoRoot, input, opts = {}) {
     log = emptyLog(producer, { producerVersion: input.producerVersion, ...vcs });
   }
   findOrAddRun(log, producer, input.producerVersion, vcs).results.push(result);
-  mkdirSync4(dirname3(path), { recursive: true });
-  writeFileSync5(path, JSON.stringify(log, null, 2) + "\n", "utf-8");
+  mkdirSync5(dirname3(path), { recursive: true });
+  writeFileSync6(path, JSON.stringify(log, null, 2) + "\n", "utf-8");
   return { path, guid: guid3 };
 }
 function writeLog(path, log) {
-  writeFileSync5(path, JSON.stringify(log, null, 2) + "\n", "utf-8");
+  writeFileSync6(path, JSON.stringify(log, null, 2) + "\n", "utf-8");
 }
 function hashArtifacts(repoRoot, artifacts) {
   const out = {};
@@ -15961,22 +16044,22 @@ function hashArtifacts(repoRoot, artifacts) {
     const safe = uri.replace(/\\/g, "/").replace(/^\/+/, "");
     if (/(^|\/)\.\.(\/|$)/.test(safe)) continue;
     try {
-      const abs = join6(repoRoot, safe);
+      const abs = join7(repoRoot, safe);
       const stat = statSync2(abs);
       if (!stat.isFile() || stat.size > ARTIFACT_HASH_MAX_BYTES) continue;
-      out[uri] = createHash("sha256").update(readFileSync5(abs)).digest("hex");
+      out[uri] = createHash("sha256").update(readFileSync6(abs)).digest("hex");
     } catch {
     }
   }
   return out;
 }
 function ensureArtifactLfsFilter(repoRoot) {
-  const path = join6(repoRoot, ".gitattributes");
+  const path = join7(repoRoot, ".gitattributes");
   try {
-    const existing = existsSync5(path) ? readFileSync5(path, "utf-8") : "";
+    const existing = existsSync6(path) ? readFileSync6(path, "utf-8") : "";
     if (existing.includes(".pr-notes/artifacts/**")) return;
     const prefix = existing === "" || existing.endsWith("\n") ? "" : "\n";
-    writeFileSync5(path, `${existing}${prefix}${LFS_FILTER_LINE}
+    writeFileSync6(path, `${existing}${prefix}${LFS_FILTER_LINE}
 `, "utf-8");
   } catch {
   }
@@ -15985,15 +16068,15 @@ function listShardPaths(repoRoot, safeRel) {
   return listShardIndices(repoRoot, safeRel).map((i) => shardPath(repoRoot, safeRel, i));
 }
 function allShardPaths(repoRoot) {
-  const root = join6(repoRoot, NOTES_SUBDIR);
-  if (!existsSync5(root)) return [];
+  const root = join7(repoRoot, NOTES_SUBDIR);
+  if (!existsSync6(root)) return [];
   const entries = readdirSync(root, { recursive: true });
-  return entries.filter((e) => SHARD_RE.test(e)).map((e) => join6(root, e));
+  return entries.filter((e) => SHARD_RE.test(e)).map((e) => join7(root, e));
 }
 function persistOrDelete(path, log) {
   log.runs = log.runs.filter((run) => run.results.length > 0);
   if (log.runs.length === 0) {
-    if (existsSync5(path)) unlinkSync(path);
+    if (existsSync6(path)) unlinkSync(path);
   } else {
     writeLog(path, log);
   }
@@ -16080,8 +16163,8 @@ function coalesceFile(repoRoot, file2) {
   return toRemove.length;
 }
 function coalesceAll(repoRoot) {
-  const root = join6(repoRoot, NOTES_SUBDIR);
-  if (!existsSync5(root)) return 0;
+  const root = join7(repoRoot, NOTES_SUBDIR);
+  if (!existsSync6(root)) return 0;
   const sources = /* @__PURE__ */ new Set();
   for (const entry of readdirSync(root, { recursive: true })) {
     const m = SHARD_RE.exec(entry);
@@ -16095,10 +16178,10 @@ function readArtifactText(repoRoot, uri) {
   const safe = uri.replace(/\\/g, "/").replace(/^\/+/, "");
   if (/(^|\/)\.\.(\/|$)/.test(safe)) return void 0;
   try {
-    const abs = join6(repoRoot, safe);
+    const abs = join7(repoRoot, safe);
     const stat = statSync2(abs);
     if (!stat.isFile() || stat.size > ARTIFACT_MAX_BYTES) return void 0;
-    const buf = readFileSync5(abs);
+    const buf = readFileSync6(abs);
     if (buf.includes(0)) return void 0;
     return buf.toString("utf-8");
   } catch {
@@ -16161,7 +16244,7 @@ var init_store = __esm({
     init_sarif();
     init_types();
     init_view();
-    NOTES_SUBDIR = join6(".pr-notes", "notes");
+    NOTES_SUBDIR = join7(".pr-notes", "notes");
     SHARD_RE = /\.(\d{6})\.sarif$/;
     ARTIFACT_HASH_MAX_BYTES = 5e7;
     LFS_FILTER_LINE = ".pr-notes/artifacts/** filter=lfs diff=lfs merge=lfs -text";
@@ -16169,48 +16252,6 @@ var init_store = __esm({
   }
 });
 
-// src/difftool/blob-store.ts
-var blob_store_exports = {};
-__export(blob_store_exports, {
-  clearDifftoolBlobs: () => clearDifftoolBlobs,
-  readDifftoolBlob: () => readDifftoolBlob,
-  writeDifftoolBlob: () => writeDifftoolBlob
-});
-import { existsSync as existsSync7, mkdirSync as mkdirSync6, readFileSync as readFileSync7, rmSync as rmSync4, writeFileSync as writeFileSync7 } from "fs";
-import { join as join8 } from "path";
-function blobDir(dataDir) {
-  return join8(dataDir, "difftool-blobs");
-}
-function blobName(fileId, side) {
-  return `${fileId.replace(/[^a-z0-9]/gi, "")}-${side}`;
-}
-function writeDifftoolBlob(dataDir, fileId, side, bytes) {
-  if (bytes.length === 0) return;
-  const dir = blobDir(dataDir);
-  mkdirSync6(dir, { recursive: true });
-  writeFileSync7(join8(dir, blobName(fileId, side)), bytes);
-}
-function readDifftoolBlob(dataDir, fileId, side) {
-  const path = join8(blobDir(dataDir), blobName(fileId, side));
-  if (!existsSync7(path)) return null;
-  try {
-    return readFileSync7(path);
-  } catch {
-    return null;
-  }
-}
-function clearDifftoolBlobs(dataDir) {
-  try {
-    rmSync4(blobDir(dataDir), { recursive: true, force: true });
-  } catch {
-  }
-}
-var init_blob_store = __esm({
-  "src/difftool/blob-store.ts"() {
-    "use strict";
-  }
-});
-
 // src/difftool/session.ts
 var session_exports = {};
 __export(session_exports, {
@@ -16715,17 +16756,17 @@ __export(difftool_discovery_exports, {
   tryAcquireStartingLock: () => tryAcquireStartingLock,
   writeDiscovery: () => writeDiscovery
 });
-import { existsSync as existsSync14, mkdirSync as mkdirSync12, readFileSync as readFileSync18, rmSync as rmSync5, statSync as statSync4, writeFileSync as writeFileSync12 } from "fs";
+import { existsSync as existsSync13, mkdirSync as mkdirSync12, readFileSync as readFileSync17, rmSync as rmSync5, statSync as statSync4, writeFileSync as writeFileSync12 } from "fs";
 import { homedir as homedir4 } from "os";
-import { join as join17 } from "path";
+import { join as join16 } from "path";
 function difftoolHome() {
-  return join17(homedir4(), ".glassbox");
+  return join16(homedir4(), ".glassbox");
 }
 function discoveryPath(home = difftoolHome()) {
-  return join17(home, "difftool.lock");
+  return join16(home, "difftool.lock");
 }
 function startingLockPath(home = difftoolHome()) {
-  return join17(home, "difftool-starting.lock");
+  return join16(home, "difftool-starting.lock");
 }
 function parseDiscovery(raw2) {
   let parsed;
@@ -16739,9 +16780,9 @@ function parseDiscovery(raw2) {
 }
 function readDiscovery(home = difftoolHome()) {
   const path = discoveryPath(home);
-  if (!existsSync14(path)) return null;
+  if (!existsSync13(path)) return null;
   try {
-    return parseDiscovery(readFileSync18(path, "utf-8"));
+    return parseDiscovery(readFileSync17(path, "utf-8"));
   } catch {
     return null;
   }
@@ -16797,9 +16838,9 @@ var init_difftool_discovery = __esm({
 // src/cli.ts
 init_connection();
 init_queries();
-import { existsSync as existsSync15, mkdirSync as mkdirSync13, realpathSync, statSync as statSync5 } from "fs";
+import { existsSync as existsSync14, mkdirSync as mkdirSync13, realpathSync, statSync as statSync5 } from "fs";
 import { tmpdir as tmpdir2 } from "os";
-import { basename as basename2, join as join18, resolve as resolve10 } from "path";
+import { basename as basename2, join as join17, resolve as resolve10 } from "path";
 
 // src/debug.ts
 var debugEnabled = false;
@@ -17383,7 +17424,18 @@ async function saveUserPreferences(prefs) {
 }
 
 // src/demo.ts
+init_connection();
 init_queries();
+init_image_blobs();
+function seedSvgBlobs(fileId, diff) {
+  if (!diff.filePath.toLowerCase().endsWith(".svg")) return;
+  const dataDir = getDataDir();
+  if (dataDir === null) return;
+  const lines = diff.hunks.flatMap((h) => h.lines);
+  const sideBytes = (keep) => Buffer.from(lines.filter((l) => keep(l.type)).map((l) => l.content).join("\n"), "utf8");
+  writeImageBlob(dataDir, fileId, "old", sideBytes((t) => t !== "add"));
+  writeImageBlob(dataDir, fileId, "new", sideBytes((t) => t !== "remove"));
+}
 function demoReviewNotes(filePath) {
   if (filePath !== "src/auth/session.ts") return [];
   return [
@@ -17655,6 +17707,26 @@ var DEMO_FILES = [
     status: "modified",
     isBinary: true,
     hunks: []
+  },
+  {
+    // A real SVG change so the demo (and the e2e suite) exercises the SVG
+    // *rendered* view and its vector zoom (GB-941). Modeled as a rename so both
+    // sides resolve to real committed icons (old = red, new = blue, both 20x20
+    // with explicit width/height so the <img> reports a natural size). Default
+    // view is the code (text) diff; the Rendered toggle switches to ImageDiff.
+    path: "tests/fixtures/diff/new/icon.svg",
+    oldPath: "tests/fixtures/diff/old/icon.svg",
+    status: "modified",
+    hunks: [{
+      oldStart: 1,
+      oldCount: 1,
+      newStart: 1,
+      newCount: 1,
+      lines: [
+        { type: "remove", oldNum: 1, newNum: null, content: '<svg xmlns="http://www.w3.org/2000/svg" width="20" height="20"><rect width="20" height="20" fill="red"/></svg>' },
+        { type: "add", oldNum: null, newNum: 1, content: '<svg xmlns="http://www.w3.org/2000/svg" width="20" height="20"><rect width="20" height="20" fill="blue"/></svg>' }
+      ]
+    }]
   }
 ];
 var GUIDED_NOTES = {
@@ -17823,6 +17895,7 @@ async function setupDemoReview(scenario) {
     };
     const rf = await addReviewFile(review.id, file2.path, JSON.stringify(diff));
     fileIdMap.set(file2.path, rf.id);
+    seedSvgBlobs(rf.id, diff);
   }
   const { updateFileStatus: updateFileStatus2 } = await Promise.resolve().then(() => (init_queries(), queries_exports));
   const reviewedPaths = ["src/utils/password.ts", "src/db/redis.ts", "package.json"];
@@ -17925,9 +17998,9 @@ async function setupReviewNoteReply(fileIdMap) {
 }
 
 // src/git/diff.ts
-import { existsSync as existsSync2, mkdirSync as mkdirSync3, mkdtempSync, readFileSync as readFileSync2, rmSync as rmSync2, statSync, writeFileSync as writeFileSync2 } from "fs";
+import { existsSync as existsSync3, mkdirSync as mkdirSync4, mkdtempSync, readFileSync as readFileSync3, rmSync as rmSync3, statSync, writeFileSync as writeFileSync3 } from "fs";
 import { tmpdir } from "os";
-import { basename, dirname, join as join3, resolve } from "path";
+import { basename, dirname, join as join4, resolve } from "path";
 
 // src/git/repo.ts
 import { spawnSync as spawnSync4 } from "child_process";
@@ -18082,16 +18155,16 @@ function getDirectComparisonFiles(mode, cwd) {
 function diffRawContent(displayPath, oldContent, newContent) {
   const rel = displayPath.replace(/\\/g, "/").replace(/^\/+/, "").replace(/(^|\/)\.\.(?=\/|$)/g, "$1_");
   const safeRel = rel === "" ? "file" : rel;
-  const work = mkdtempSync(join3(tmpdir(), "glassbox-difftool-append-"));
-  const rootA = join3(work, "a");
-  const rootB = join3(work, "b");
-  const oldAbs = join3(rootA, safeRel);
-  const newAbs = join3(rootB, safeRel);
+  const work = mkdtempSync(join4(tmpdir(), "glassbox-difftool-append-"));
+  const rootA = join4(work, "a");
+  const rootB = join4(work, "b");
+  const oldAbs = join4(rootA, safeRel);
+  const newAbs = join4(rootB, safeRel);
   try {
-    mkdirSync3(dirname(oldAbs), { recursive: true });
-    mkdirSync3(dirname(newAbs), { recursive: true });
-    writeFileSync2(oldAbs, oldContent);
-    writeFileSync2(newAbs, newContent);
+    mkdirSync4(dirname(oldAbs), { recursive: true });
+    mkdirSync4(dirname(newAbs), { recursive: true });
+    writeFileSync3(oldAbs, oldContent);
+    writeFileSync3(newAbs, newContent);
     const rawDiff = gitOrEmpty(["diff", "--no-index", "-U3", toGitArg(oldAbs), toGitArg(newAbs)], work);
     const diffs = normalizeDiffPaths(parseDiff(rawDiff), rootA, rootB);
     if (diffs.length === 0) {
@@ -18104,7 +18177,7 @@ function diffRawContent(displayPath, oldContent, newContent) {
     return { ...diff, filePath: safeRel, oldPath: null, status };
   } finally {
     try {
-      rmSync2(work, { recursive: true, force: true });
+      rmSync3(work, { recursive: true, force: true });
     } catch {
     }
   }
@@ -18139,7 +18212,7 @@ function getAllFiles(repoRoot) {
 function createNewFileDiff(filePath, repoRoot) {
   let content;
   try {
-    const buf = readFileSync2(resolve(repoRoot, filePath));
+    const buf = readFileSync3(resolve(repoRoot, filePath));
     const checkLen = Math.min(buf.length, 8192);
     for (let i = 0; i < checkLen; i++) {
       if (buf[i] === 0) {
@@ -18259,7 +18332,7 @@ function getFileContent(filePath, ref, cwd) {
   const repoRoot = getRepoRoot(cwd);
   try {
     if (ref === "working") {
-      return readFileSync2(resolve(repoRoot, filePath), "utf-8");
+      return readFileSync3(resolve(repoRoot, filePath), "utf-8");
     }
     return git(["show", `${ref}:${filePath}`], repoRoot);
   } catch {
@@ -18269,9 +18342,9 @@ function getFileContent(filePath, ref, cwd) {
 function getModeFileContent(mode, filePath, side, cwd) {
   if (mode.type === "diff") {
     const { rootA, rootB } = directComparisonRoots(mode);
-    const abs = join3(side === "old" ? rootA : rootB, filePath);
+    const abs = join4(side === "old" ? rootA : rootB, filePath);
     try {
-      return readFileSync2(abs, "utf-8");
+      return readFileSync3(abs, "utf-8");
     } catch {
       return "";
     }
@@ -18307,9 +18380,9 @@ function getSingleFileDiff(mode, filePath, repoRoot, extraFlags = "") {
   }
   if (mode.type === "diff") {
     const { rootA, rootB } = directComparisonRoots(mode);
-    const oldAbs = join3(rootA, filePath);
-    const newAbs = join3(rootB, filePath);
-    if (!existsSync2(oldAbs) || !existsSync2(newAbs)) return null;
+    const oldAbs = join4(rootA, filePath);
+    const newAbs = join4(rootB, filePath);
+    if (!existsSync3(oldAbs) || !existsSync3(newAbs)) return null;
     const args2 = ["diff", "--no-index", "-U3"];
     if (extraFlags) args2.push(...extraFlags.split(" ").filter(Boolean));
     args2.push(toGitArg(oldAbs), toGitArg(newAbs));
@@ -18371,15 +18444,15 @@ function getModeArgs(mode) {
 
 // src/lock.ts
 init_zod();
-import { existsSync as existsSync3, readFileSync as readFileSync3, rmSync as rmSync3, writeFileSync as writeFileSync3 } from "fs";
-import { join as join4 } from "path";
+import { existsSync as existsSync4, readFileSync as readFileSync4, rmSync as rmSync4, writeFileSync as writeFileSync4 } from "fs";
+import { join as join5 } from "path";
 var lockPath = null;
 var LockFileSchema = external_exports.object({ pid: external_exports.number().int() });
 function acquireLock(dataDir) {
-  lockPath = join4(dataDir, "glassbox.lock");
-  if (existsSync3(lockPath)) {
+  lockPath = join5(dataDir, "glassbox.lock");
+  if (existsSync4(lockPath)) {
     try {
-      const raw2 = JSON.parse(readFileSync3(lockPath, "utf-8"));
+      const raw2 = JSON.parse(readFileSync4(lockPath, "utf-8"));
       const contents = LockFileSchema.parse(raw2);
       const pid = contents.pid;
       try {
@@ -18392,13 +18465,13 @@ function acquireLock(dataDir) {
         process.exit(1);
       } catch {
         console.log(`  Removing stale lock from PID ${pid}`);
-        rmSync3(lockPath, { force: true });
+        rmSync4(lockPath, { force: true });
       }
     } catch {
-      rmSync3(lockPath, { force: true });
+      rmSync4(lockPath, { force: true });
     }
   }
-  writeFileSync3(lockPath, JSON.stringify({ pid: process.pid, startedAt: (/* @__PURE__ */ new Date()).toISOString() }));
+  writeFileSync4(lockPath, JSON.stringify({ pid: process.pid, startedAt: (/* @__PURE__ */ new Date()).toISOString() }));
   const cleanup = () => {
     releaseLock();
   };
@@ -18415,7 +18488,7 @@ function acquireLock(dataDir) {
 function releaseLock() {
   if (lockPath !== null) {
     try {
-      rmSync3(lockPath, { force: true });
+      rmSync4(lockPath, { force: true });
     } catch {
     }
     lockPath = null;
@@ -18528,15 +18601,15 @@ async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
 
 // src/server.ts
 import { serve } from "@hono/node-server";
-import { existsSync as existsSync11, readFileSync as readFileSync15 } from "fs";
+import { existsSync as existsSync10, readFileSync as readFileSync14 } from "fs";
 import { Hono as Hono19 } from "hono";
-import { dirname as dirname4, join as join14 } from "path";
+import { dirname as dirname4, join as join13 } from "path";
 import { fileURLToPath as fileURLToPath2 } from "url";
 
 // src/channel-config.ts
 init_zod();
-import { existsSync as existsSync4, readFileSync as readFileSync4, writeFileSync as writeFileSync4 } from "fs";
-import { dirname as dirname2, join as join5, resolve as resolve2 } from "path";
+import { existsSync as existsSync5, readFileSync as readFileSync5, writeFileSync as writeFileSync5 } from "fs";
+import { dirname as dirname2, join as join6, resolve as resolve2 } from "path";
 import { fileURLToPath } from "url";
 var MCP_SERVER_KEY = "glassbox-channel";
 var McpConfigSchema = external_exports.object({
@@ -18546,11 +18619,11 @@ var HealthResponseSchema = external_exports.object({ ok: external_exports.boolea
 function getChannelServerPath() {
   const thisDir = dirname2(fileURLToPath(import.meta.url));
   const distPath = resolve2(thisDir, "channel.js");
-  if (existsSync4(distPath)) {
+  if (existsSync5(distPath)) {
     return { command: process.execPath, args: [distPath] };
   }
   const srcPath = resolve2(thisDir, "channel.ts");
-  if (existsSync4(srcPath)) {
+  if (existsSync5(srcPath)) {
     return { command: "npx", args: ["tsx", srcPath] };
   }
   return { command: process.execPath, args: [distPath] };
@@ -18560,12 +18633,12 @@ function projectRoot(dataDir) {
 }
 function registerChannel(dataDir) {
   const root = projectRoot(dataDir);
-  const mcpPath = join5(root, ".mcp.json");
+  const mcpPath = join6(root, ".mcp.json");
   const { command, args } = getChannelServerPath();
   let config2 = {};
-  if (existsSync4(mcpPath)) {
+  if (existsSync5(mcpPath)) {
     try {
-      const raw2 = JSON.parse(readFileSync4(mcpPath, "utf-8"));
+      const raw2 = JSON.parse(readFileSync5(mcpPath, "utf-8"));
       const parsed = McpConfigSchema.safeParse(raw2);
       if (parsed.success) config2 = parsed.data;
     } catch {
@@ -18576,14 +18649,14 @@ function registerChannel(dataDir) {
     command,
     args: [...args, "--data-dir", dataDir]
   };
-  writeFileSync4(mcpPath, JSON.stringify(config2, null, 2) + "\n", "utf-8");
+  writeFileSync5(mcpPath, JSON.stringify(config2, null, 2) + "\n", "utf-8");
 }
 function unregisterChannel(dataDir) {
   const root = projectRoot(dataDir);
-  const mcpPath = join5(root, ".mcp.json");
-  if (!existsSync4(mcpPath)) return;
+  const mcpPath = join6(root, ".mcp.json");
+  if (!existsSync5(mcpPath)) return;
   try {
-    const raw2 = JSON.parse(readFileSync4(mcpPath, "utf-8"));
+    const raw2 = JSON.parse(readFileSync5(mcpPath, "utf-8"));
     const parsed = McpConfigSchema.safeParse(raw2);
     if (!parsed.success) return;
     const config2 = parsed.data;
@@ -18591,14 +18664,14 @@ function unregisterChannel(dataDir) {
       const servers = { ...config2.mcpServers };
       delete servers[MCP_SERVER_KEY];
       config2.mcpServers = servers;
-      writeFileSync4(mcpPath, JSON.stringify(config2, null, 2) + "\n", "utf-8");
+      writeFileSync5(mcpPath, JSON.stringify(config2, null, 2) + "\n", "utf-8");
     }
   } catch {
   }
 }
 function getChannelPort(dataDir) {
   try {
-    const portStr = readFileSync4(join5(dataDir, "channel-port"), "utf-8").trim();
+    const portStr = readFileSync5(join6(dataDir, "channel-port"), "utf-8").trim();
     const port = parseInt(portStr, 10);
     return isNaN(port) ? null : port;
   } catch {
@@ -19887,6 +19960,7 @@ __export(annotations_exports, {
   DeleteAnnotationReqSchema: () => DeleteAnnotationReqSchema,
   DeleteAnnotationRespSchema: () => DeleteAnnotationRespSchema,
   DeleteStaleAnnotationsRespSchema: () => DeleteStaleAnnotationsRespSchema,
+  ImageRegionSchema: () => ImageRegionSchema,
   KeepAllStaleAnnotationsRespSchema: () => KeepAllStaleAnnotationsRespSchema,
   KeepAnnotationReqSchema: () => KeepAnnotationReqSchema,
   KeepAnnotationRespSchema: () => KeepAnnotationRespSchema,
@@ -19897,6 +19971,9 @@ __export(annotations_exports, {
   UpdateAnnotationBodySchema: () => UpdateAnnotationBodySchema,
   UpdateAnnotationReqSchema: () => UpdateAnnotationReqSchema,
   UpdateAnnotationRespSchema: () => UpdateAnnotationRespSchema,
+  UpdateRegionBodySchema: () => UpdateRegionBodySchema,
+  UpdateRegionReqSchema: () => UpdateRegionReqSchema,
+  UpdateRegionRespSchema: () => UpdateRegionRespSchema,
   createAnnotation: () => createAnnotation,
   deleteAnnotation: () => deleteAnnotation2,
   deleteStaleAnnotations: () => deleteStaleAnnotations2,
@@ -19904,7 +19981,8 @@ __export(annotations_exports, {
   keepAnnotation: () => keepAnnotation,
   listAllAnnotations: () => listAllAnnotations,
   moveAnnotation: () => moveAnnotation2,
-  updateAnnotation: () => updateAnnotation2
+  updateAnnotation: () => updateAnnotation2,
+  updateAnnotationRegion: () => updateAnnotationRegion2
 });
 init_zod();
 init_schemas3();
@@ -19920,12 +19998,17 @@ var AnnotationCategorySchema = external_exports.enum([
 var AnnotationSideSchema = external_exports.enum(["old", "new"]);
 var CreateAnnotationReqSchema = external_exports.object({
   reviewFileId: external_exports.string().min(1),
-  lineNumber: external_exports.number().int().min(1),
+  // `0` creates an image-level annotation (doc 23): a general image comment, or
+  // — when `region` is set — a comment anchored to a rectangle on the image.
+  // Line-anchored annotations pass the real 1-based line number.
+  lineNumber: external_exports.number().int().min(0),
   side: AnnotationSideSchema,
   category: AnnotationCategorySchema,
   content: external_exports.string().min(1),
   /** SARIF guid of the AI review note this annotation replies to (doc 20 threading). */
-  replyToNoteId: external_exports.string().optional()
+  replyToNoteId: external_exports.string().optional(),
+  /** Normalized rectangle for an image-region annotation (doc 23). */
+  region: ImageRegionSchema.optional()
 });
 var CreateAnnotationRespSchema = AnnotationSchema;
 var UpdateAnnotationReqSchema = external_exports.object({
@@ -19944,6 +20027,12 @@ var MoveAnnotationReqSchema = external_exports.object({
 });
 var MoveAnnotationBodySchema = MoveAnnotationReqSchema.omit({ id: true });
 var MoveAnnotationRespSchema = OkResponseSchema;
+var UpdateRegionReqSchema = external_exports.object({
+  id: external_exports.string(),
+  region: ImageRegionSchema
+});
+var UpdateRegionBodySchema = UpdateRegionReqSchema.omit({ id: true });
+var UpdateRegionRespSchema = OkResponseSchema;
 var KeepAnnotationReqSchema = external_exports.object({ id: external_exports.string() });
 var KeepAnnotationRespSchema = OkResponseSchema;
 var DeleteStaleAnnotationsRespSchema = OkResponseSchema;
@@ -19965,6 +20054,10 @@ async function moveAnnotation2(req) {
   const { id, ...body } = req;
   return apiCall(MoveAnnotationRespSchema, `/annotations/${id}/move`, { method: "PATCH", body });
 }
+async function updateAnnotationRegion2(req) {
+  const { id, ...body } = req;
+  return apiCall(UpdateRegionRespSchema, `/annotations/${id}/region`, { method: "PATCH", body });
+}
 async function keepAnnotation(req) {
   return apiCall(KeepAnnotationRespSchema, `/annotations/${req.id}/keep`, { method: "POST" });
 }
@@ -21666,25 +21759,26 @@ init_queries();
 // src/export/generate.ts
 init_zod();
 init_queries();
+init_schemas3();
 import { spawnSync as spawnSync6 } from "child_process";
-import { appendFileSync, existsSync as existsSync6, mkdirSync as mkdirSync5, readFileSync as readFileSync6, unlinkSync as unlinkSync2, writeFileSync as writeFileSync6 } from "fs";
+import { appendFileSync, existsSync as existsSync7, mkdirSync as mkdirSync6, readFileSync as readFileSync7, unlinkSync as unlinkSync2, writeFileSync as writeFileSync7 } from "fs";
 import { homedir as homedir2 } from "os";
-import { join as join7 } from "path";
-var DISMISS_FILE = join7(homedir2(), ".glassbox", "gitignore-dismissed.json");
+import { join as join8 } from "path";
+var DISMISS_FILE = join8(homedir2(), ".glassbox", "gitignore-dismissed.json");
 var DISMISS_DAYS = 30;
 var DismissalsSchema = external_exports.record(external_exports.string(), external_exports.number());
 function loadDismissals() {
   try {
-    const parsed = DismissalsSchema.safeParse(JSON.parse(readFileSync6(DISMISS_FILE, "utf-8")));
+    const parsed = DismissalsSchema.safeParse(JSON.parse(readFileSync7(DISMISS_FILE, "utf-8")));
     return parsed.success ? parsed.data : {};
   } catch {
     return {};
   }
 }
 function saveDismissals(data) {
-  const dir = join7(homedir2(), ".glassbox");
-  mkdirSync5(dir, { recursive: true });
-  writeFileSync6(DISMISS_FILE, JSON.stringify(data), "utf-8");
+  const dir = join8(homedir2(), ".glassbox");
+  mkdirSync6(dir, { recursive: true });
+  writeFileSync7(DISMISS_FILE, JSON.stringify(data), "utf-8");
 }
 function isGlassboxGitignored(repoRoot) {
   const result = spawnSync6("git", ["check-ignore", "-q", ".glassbox"], { cwd: repoRoot, stdio: "pipe" });
@@ -21702,16 +21796,16 @@ function shouldPromptGitignore(repoRoot) {
   return true;
 }
 function addGlassboxToGitignore(repoRoot) {
-  const gitignorePath = join7(repoRoot, ".gitignore");
-  if (existsSync6(gitignorePath)) {
-    const content = readFileSync6(gitignorePath, "utf-8");
+  const gitignorePath = join8(repoRoot, ".gitignore");
+  if (existsSync7(gitignorePath)) {
+    const content = readFileSync7(gitignorePath, "utf-8");
     if (!content.endsWith("\n")) {
       appendFileSync(gitignorePath, "\n.glassbox/\n", "utf-8");
     } else {
       appendFileSync(gitignorePath, ".glassbox/\n", "utf-8");
     }
   } else {
-    writeFileSync6(gitignorePath, ".glassbox/\n", "utf-8");
+    writeFileSync7(gitignorePath, ".glassbox/\n", "utf-8");
   }
 }
 function dismissGitignorePrompt2(repoRoot) {
@@ -21720,17 +21814,30 @@ function dismissGitignorePrompt2(repoRoot) {
   saveDismissals(dismissals);
 }
 function deleteReviewExport(reviewId, repoRoot) {
-  const exportDir = join7(repoRoot, ".glassbox");
-  const archivePath = join7(exportDir, `review-${reviewId}.md`);
-  if (existsSync6(archivePath)) unlinkSync2(archivePath);
+  const exportDir = join8(repoRoot, ".glassbox");
+  const archivePath = join8(exportDir, `review-${reviewId}.md`);
+  if (existsSync7(archivePath)) unlinkSync2(archivePath);
+}
+function annotationAnchorLabel(a) {
+  if (a.line_number !== 0) return `**Line ${a.line_number}**`;
+  if (a.region_data !== null) {
+    const parsed = ImageRegionSchema.safeParse(JSON.parse(a.region_data));
+    if (parsed.success) {
+      const { x, y, w, h, side } = parsed.data;
+      const pct = (n) => `${Math.round(n * 100)}%`;
+      const scope = side === "old" ? ", A image only" : side === "new" ? ", B image only" : "";
+      return `**Image region (${pct(x)}, ${pct(y)}, ${pct(w)}\xD7${pct(h)}${scope})**`;
+    }
+  }
+  return "**Image comment**";
 }
 async function generateReviewExport(reviewId, repoRoot, isCurrent) {
   const review = await getReview(reviewId);
   if (!review) throw new Error("Review not found");
   const files = await getReviewFiles(reviewId);
   const annotations = await getAnnotationsForReview(reviewId);
-  const exportDir = join7(repoRoot, ".glassbox");
-  mkdirSync5(exportDir, { recursive: true });
+  const exportDir = join8(repoRoot, ".glassbox");
+  mkdirSync6(exportDir, { recursive: true });
   const byFile = {};
   for (const a of annotations) {
     if (!(a.file_path in byFile)) byFile[a.file_path] = [];
@@ -21766,7 +21873,8 @@ async function generateReviewExport(reviewId, repoRoot, isCurrent) {
     lines.push("> project configuration (CLAUDE.md, .cursorrules, etc.) with these preferences/rules.");
     lines.push("");
     for (const item of rememberItems) {
-      lines.push(`- **${item.file_path}:${item.line_number}** - ${item.content}`);
+      const anchor = item.line_number === 0 ? `${item.file_path} (image)` : `${item.file_path}:${item.line_number}`;
+      lines.push(`- **${anchor}** - ${item.content}`);
     }
     lines.push("");
   }
@@ -21777,7 +21885,7 @@ async function generateReviewExport(reviewId, repoRoot, isCurrent) {
     lines.push(`### ${filePath}`);
     lines.push("");
     for (const a of fileAnns) {
-      lines.push(`- **Line ${a.line_number}** [${a.category}]: ${a.content}`);
+      lines.push(`- ${annotationAnchorLabel(a)} [${a.category}]: ${a.content}`);
     }
     lines.push("");
   }
@@ -21797,11 +21905,11 @@ async function generateReviewExport(reviewId, repoRoot, isCurrent) {
   lines.push("6. **note** annotations are informational context. Consider them but they may not require code changes.");
   lines.push("");
   const content = lines.join("\n");
-  const archivePath = join7(exportDir, `review-${review.id}.md`);
-  writeFileSync6(archivePath, content, "utf-8");
+  const archivePath = join8(exportDir, `review-${review.id}.md`);
+  writeFileSync7(archivePath, content, "utf-8");
   if (isCurrent) {
-    const latestPath = join7(exportDir, "latest-review.md");
-    writeFileSync6(latestPath, content, "utf-8");
+    const latestPath = join8(exportDir, "latest-review.md");
+    writeFileSync7(latestPath, content, "utf-8");
     return latestPath;
   }
   return archivePath;
@@ -21833,7 +21941,8 @@ annotationsRoutes.post("/annotations", async (c) => {
     body.side,
     body.category,
     body.content,
-    body.replyToNoteId
+    body.replyToNoteId,
+    body.region
   );
   autoExport(c);
   return c.json(annotation, 201);
@@ -21854,6 +21963,15 @@ annotationsRoutes.delete("/annotations/:id", async (c) => {
   autoExport(c);
   return c.json({ ok: true });
 });
+annotationsRoutes.patch("/annotations/:id/region", async (c) => {
+  const id = requirePathParam(c, "id");
+  if (!id.ok) return id.response;
+  const parsed = await parseBody(c, UpdateRegionBodySchema);
+  if (!parsed.ok) return parsed.response;
+  await updateAnnotationRegion(id.data, parsed.data.region);
+  autoExport(c);
+  return c.json({ ok: true });
+});
 annotationsRoutes.patch("/annotations/:id/move", async (c) => {
   const id = requirePathParam(c, "id");
   if (!id.ok) return id.response;
@@ -22026,7 +22144,6 @@ filesRoutes.post("/files/:fileId/open", async (c) => {
 // src/routes/api/image.ts
 init_connection();
 init_queries();
-init_blob_store();
 import { Hono as Hono7 } from "hono";
 
 // src/git/image.ts
@@ -22355,255 +22472,22 @@ function getNewImage(mode, filePath, repoRoot) {
   return { data, size: data.length };
 }
 
-// src/git/svg-rasterize.ts
-import { Worker } from "worker_threads";
-
-// src/git/svg-rasterize-render.ts
-import { existsSync as existsSync8, readFileSync as readFileSync9 } from "fs";
-import { createRequire } from "module";
-import { join as join10 } from "path";
-var initialized = false;
-var ResvgClass;
-var fontBuffers = [];
-async function ensureRenderInit() {
-  if (initialized) return;
-  const require2 = createRequire(import.meta.url);
-  const resvgPath = require2.resolve("@resvg/resvg-wasm");
-  const wasmPath = resvgPath.replace(/index\.(js|mjs)$/, "index_bg.wasm");
-  const wasmBuffer = readFileSync9(wasmPath);
-  const mod = await import("@resvg/resvg-wasm");
-  await mod.initWasm(wasmBuffer);
-  ResvgClass = mod.Resvg;
-  fontBuffers = loadSystemFonts();
-  initialized = true;
-}
-function loadSystemFonts() {
-  const buffers = [];
-  const candidates = getFontCandidates();
-  for (const path of candidates) {
-    if (!existsSync8(path)) continue;
-    try {
-      buffers.push(readFileSync9(path));
-    } catch {
-    }
-  }
-  return buffers;
-}
-function getFontCandidates() {
-  const os = process.platform;
-  if (os === "darwin") {
-    const sys = "/System/Library/Fonts";
-    const sup = "/System/Library/Fonts/Supplemental";
-    return [
-      // Core system fonts (serif, sans-serif, monospace)
-      join10(sys, "Helvetica.ttc"),
-      join10(sys, "Times.ttc"),
-      join10(sys, "Courier.ttc"),
-      join10(sys, "Menlo.ttc"),
-      join10(sys, "SFPro.ttf"),
-      join10(sys, "SFNS.ttf"),
-      join10(sys, "SFNSMono.ttf"),
-      // Supplemental (common named fonts in SVGs)
-      join10(sup, "Arial.ttf"),
-      join10(sup, "Arial Bold.ttf"),
-      join10(sup, "Georgia.ttf"),
-      join10(sup, "Verdana.ttf"),
-      join10(sup, "Tahoma.ttf"),
-      join10(sup, "Trebuchet MS.ttf"),
-      join10(sup, "Impact.ttf"),
-      join10(sup, "Comic Sans MS.ttf"),
-      join10(sup, "Courier New.ttf"),
-      join10(sup, "Times New Roman.ttf")
-    ];
-  }
-  if (os === "linux") {
-    return [
-      // DejaVu (most common Linux fallback)
-      "/usr/share/fonts/truetype/dejavu/DejaVuSans.ttf",
-      "/usr/share/fonts/truetype/dejavu/DejaVuSans-Bold.ttf",
-      "/usr/share/fonts/truetype/dejavu/DejaVuSerif.ttf",
-      "/usr/share/fonts/truetype/dejavu/DejaVuSansMono.ttf",
-      // Liberation (metric-compatible with Arial/Times/Courier)
-      "/usr/share/fonts/truetype/liberation/LiberationSans-Regular.ttf",
-      "/usr/share/fonts/truetype/liberation/LiberationSerif-Regular.ttf",
-      "/usr/share/fonts/truetype/liberation/LiberationMono-Regular.ttf",
-      // Noto (common on modern distros)
-      "/usr/share/fonts/truetype/noto/NotoSans-Regular.ttf"
-    ];
-  }
-  if (os === "win32") {
-    const winFonts = join10(process.env.WINDIR ?? "C:\\Windows", "Fonts");
-    return [
-      join10(winFonts, "arial.ttf"),
-      join10(winFonts, "arialbd.ttf"),
-      join10(winFonts, "times.ttf"),
-      join10(winFonts, "cour.ttf"),
-      join10(winFonts, "verdana.ttf"),
-      join10(winFonts, "tahoma.ttf"),
-      join10(winFonts, "georgia.ttf"),
-      join10(winFonts, "consola.ttf"),
-      join10(winFonts, "segoeui.ttf")
-    ];
-  }
-  return [];
-}
-function parseSvgDimensions(svg) {
-  const widthMatch = svg.match(/\bwidth\s*=\s*["']([^"']+)["']/);
-  const heightMatch = svg.match(/\bheight\s*=\s*["']([^"']+)["']/);
-  const viewBoxMatch = svg.match(/\bviewBox\s*=\s*["']([^"']+)["']/);
-  let width = widthMatch ? parseFloat(widthMatch[1]) : NaN;
-  let height = heightMatch ? parseFloat(heightMatch[1]) : NaN;
-  if ((isNaN(width) || isNaN(height)) && viewBoxMatch) {
-    const parts = viewBoxMatch[1].split(/[\s,]+/);
-    if (parts.length >= 4) {
-      if (isNaN(width)) width = parseFloat(parts[2]);
-      if (isNaN(height)) height = parseFloat(parts[3]);
-    }
-  }
-  if (isNaN(width)) width = 300;
-  if (isNaN(height)) height = 150;
-  return { width, height };
-}
-function svgUsesExternalFonts(svgData) {
-  const svg = svgData.toString("utf-8");
-  return /<text[\s>]/i.test(svg) || /font-family/i.test(svg) || /@font-face/i.test(svg);
-}
-var MAX_RENDER_DIM = 4e3;
-async function renderSvgToPng(svgString) {
-  await ensureRenderInit();
-  const { width, height } = parseSvgDimensions(svgString);
-  const maxDim = Math.max(width, height);
-  const scale = Math.min(10, MAX_RENDER_DIM / maxDim);
-  const targetWidth = Math.round(width * scale);
-  const resvg = new ResvgClass(svgString, {
-    fitTo: { mode: "width", value: targetWidth },
-    font: {
-      loadSystemFonts: false,
-      fontBuffers,
-      defaultFontFamily: "Helvetica"
-    }
-  });
-  const rendered = resvg.render();
-  const png = Buffer.from(rendered.asPng());
-  rendered.free();
-  resvg.free();
-  return png;
-}
-
-// src/git/svg-rasterize.ts
-var RENDER_TIMEOUT_MS = 15e3;
-var worker = null;
-var workerReady = false;
-var workerDisabled = false;
-var nextJobId = 0;
-var pending = /* @__PURE__ */ new Map();
-function resolveWorkerUrl() {
-  return import.meta.url.endsWith(".ts") ? new URL("./svg-rasterize-worker-boot.mjs", import.meta.url) : new URL("./svg-rasterize-worker.js", import.meta.url);
-}
-function clearJobTimer(job) {
-  if (job.timer !== void 0) clearTimeout(job.timer);
-}
-function fallbackAllPending() {
-  for (const [id, job] of pending) {
-    pending.delete(id);
-    clearJobTimer(job);
-    renderSvgToPng(job.svg).then(job.resolve, job.reject);
-  }
-}
-function rejectAllPending(err) {
-  for (const [id, job] of pending) {
-    pending.delete(id);
-    clearJobTimer(job);
-    job.reject(err);
-  }
-}
-function disposeWorker() {
-  if (worker) {
-    worker.removeAllListeners();
-    void worker.terminate();
-  }
-  worker = null;
-  workerReady = false;
-}
-function onJobTimeout(id) {
-  const job = pending.get(id);
-  if (!job) return;
-  pending.delete(id);
-  clearJobTimer(job);
-  const requeue = [...pending.values()];
-  pending.clear();
-  for (const r of requeue) clearJobTimer(r);
-  disposeWorker();
-  job.reject(new Error(`SVG rasterization timed out after ${RENDER_TIMEOUT_MS} ms`));
-  for (const r of requeue) submit(r);
-}
-function getWorker() {
-  if (workerDisabled) return null;
-  if (worker) return worker;
-  let spawned;
-  try {
-    spawned = new Worker(resolveWorkerUrl());
-  } catch {
-    workerDisabled = true;
-    return null;
-  }
-  spawned.on("message", (msg) => {
-    if (msg.type === "ready") {
-      workerReady = true;
-      return;
-    }
-    const job = pending.get(msg.id);
-    if (!job) return;
-    pending.delete(msg.id);
-    clearJobTimer(job);
-    if (msg.type === "result") job.resolve(Buffer.from(msg.png));
-    else job.reject(new Error(msg.message));
-  });
-  const handleFailure = (err) => {
-    const startupFailure = !workerReady;
-    disposeWorker();
-    if (startupFailure) {
-      workerDisabled = true;
-      fallbackAllPending();
-    } else {
-      rejectAllPending(err);
-    }
-  };
-  spawned.on("error", handleFailure);
-  spawned.on("exit", (code) => {
-    if (code !== 0) handleFailure(new Error(`SVG rasterization worker exited with code ${code}`));
-  });
-  worker = spawned;
-  return spawned;
-}
-function submit(job) {
-  const activeWorker = getWorker();
-  if (!activeWorker) {
-    renderSvgToPng(job.svg).then(job.resolve, job.reject);
-    return;
-  }
-  const id = nextJobId++;
-  job.timer = setTimeout(() => {
-    onJobTimeout(id);
-  }, RENDER_TIMEOUT_MS);
-  pending.set(id, job);
-  activeWorker.postMessage({ id, svg: job.svg });
-}
-async function rasterizeSvg(svgData) {
-  const svg = svgData.toString("utf-8");
-  return new Promise((resolve11, reject) => {
-    submit({ svg, resolve: resolve11, reject });
-  });
-}
-
 // src/routes/api/image.ts
+init_image_blobs();
 var imageRoutes = new Hono7();
-function difftoolImageSide(fileId, side) {
+function blobImageSide(fileId, side) {
   const dataDir = getDataDir();
   if (dataDir === null) return null;
-  const bytes = readDifftoolBlob(dataDir, fileId, side);
+  const bytes = readImageBlob(dataDir, fileId, side);
   return bytes !== null ? { data: bytes, size: bytes.length } : null;
 }
+function resolveImageSide(fileId, side, status, mode, filePath, oldPath, repoRoot, isDifftool) {
+  if (side === "old" && status === "added") return null;
+  if (side === "new" && status === "deleted") return null;
+  if (isDifftool) return blobImageSide(fileId, side);
+  const fromGit = side === "old" ? getOldImage(mode, filePath, oldPath, repoRoot) : getNewImage(mode, filePath, repoRoot);
+  return fromGit ?? blobImageSide(fileId, side);
+}
 imageRoutes.get("/image/:fileId/metadata", async (c) => {
   const fileIdParam = requirePathParam(c, "fileId");
   if (!fileIdParam.ok) return fileIdParam.response;
@@ -22617,8 +22501,8 @@ imageRoutes.get("/image/:fileId/metadata", async (c) => {
   const oldPath = diff?.oldPath ?? null;
   const status = diff?.status ?? "modified";
   const isDifftool = review.mode === "difftool";
-  const oldImage = status === "added" ? null : isDifftool ? difftoolImageSide(fileIdParam.data, "old") : getOldImage(mode, file2.file_path, oldPath, repoRoot);
-  const newImage = status === "deleted" ? null : isDifftool ? difftoolImageSide(fileIdParam.data, "new") : getNewImage(mode, file2.file_path, repoRoot);
+  const oldImage = resolveImageSide(fileIdParam.data, "old", status, mode, file2.file_path, oldPath, repoRoot, isDifftool);
+  const newImage = resolveImageSide(fileIdParam.data, "new", status, mode, file2.file_path, oldPath, repoRoot, isDifftool);
   const oldMeta = oldImage !== null ? extractMetadata(oldImage.data, oldPath ?? file2.file_path) : null;
   const newMeta = newImage !== null ? extractMetadata(newImage.data, file2.file_path) : null;
   return c.json({
@@ -22639,19 +22523,19 @@ imageRoutes.get("/image/:fileId/:side", async (c) => {
   const mode = parseModeString(review.mode);
   const diff = parseDiffData(file2.diff_data);
   const oldPath = diff?.oldPath ?? null;
-  const image = review.mode === "difftool" ? difftoolImageSide(fileIdParam.data, side) : side === "old" ? getOldImage(mode, file2.file_path, oldPath, repoRoot) : getNewImage(mode, file2.file_path, repoRoot);
+  const status = diff?.status ?? "modified";
+  const image = resolveImageSide(
+    fileIdParam.data,
+    side,
+    status,
+    mode,
+    file2.file_path,
+    oldPath,
+    repoRoot,
+    review.mode === "difftool"
+  );
   if (!image) return c.text("Image not available", 404);
   const sidePath = side === "old" ? oldPath ?? file2.file_path : file2.file_path;
-  if (isSvgFile(sidePath)) {
-    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(sidePath);
   return new Response(new Uint8Array(image.data), {
     headers: { "Content-Type": contentType, "Cache-Control": "no-cache" }
@@ -22661,7 +22545,7 @@ imageRoutes.get("/image/:fileId/:side", async (c) => {
 // src/routes/api/outline.ts
 init_queries();
 import { spawnSync as spawnSync8 } from "child_process";
-import { readFileSync as readFileSync10 } from "fs";
+import { readFileSync as readFileSync9 } from "fs";
 import { Hono as Hono8 } from "hono";
 import { resolve as resolve6 } from "path";
 
@@ -23039,7 +22923,7 @@ outlineRoutes.get("/symbol-definition", async (c) => {
         }
         let content = "";
         try {
-          content = readFileSync10(resolve6(repoRoot, filePath), "utf-8");
+          content = readFileSync9(resolve6(repoRoot, filePath), "utf-8");
         } catch {
           continue;
         }
@@ -23075,15 +22959,15 @@ function collectDefinitions(symbols, targetName, fileId, filePath, out) {
 }
 
 // src/routes/api/project-settings.ts
-import { existsSync as existsSync9, mkdirSync as mkdirSync7, readFileSync as readFileSync11, writeFileSync as writeFileSync8 } from "fs";
+import { existsSync as existsSync8, mkdirSync as mkdirSync7, readFileSync as readFileSync10, writeFileSync as writeFileSync8 } from "fs";
 import { Hono as Hono9 } from "hono";
-import { join as join11 } from "path";
+import { join as join10 } from "path";
 var projectSettingsRoutes = new Hono9();
 function readProjectSettings(repoRoot) {
-  const settingsPath = join11(repoRoot, ".glassbox", "settings.json");
+  const settingsPath = join10(repoRoot, ".glassbox", "settings.json");
   try {
-    if (existsSync9(settingsPath)) {
-      const raw2 = JSON.parse(readFileSync11(settingsPath, "utf-8"));
+    if (existsSync8(settingsPath)) {
+      const raw2 = JSON.parse(readFileSync10(settingsPath, "utf-8"));
       const parsed = ProjectSettingsSchema.safeParse(raw2);
       if (parsed.success) return parsed.data;
     }
@@ -23092,9 +22976,9 @@ function readProjectSettings(repoRoot) {
   return {};
 }
 function writeProjectSettings(repoRoot, settings) {
-  const dir = join11(repoRoot, ".glassbox");
+  const dir = join10(repoRoot, ".glassbox");
   mkdirSync7(dir, { recursive: true });
-  writeFileSync8(join11(dir, "settings.json"), JSON.stringify(settings, null, 2), "utf-8");
+  writeFileSync8(join10(dir, "settings.json"), JSON.stringify(settings, null, 2), "utf-8");
 }
 projectSettingsRoutes.get("/project-settings", (c) => {
   const repoRoot = c.get("repoRoot");
@@ -23112,7 +22996,7 @@ projectSettingsRoutes.patch("/project-settings", async (c) => {
 
 // src/routes/api/review-notes.ts
 init_store();
-import { readFileSync as readFileSync12, statSync as statSync3 } from "fs";
+import { readFileSync as readFileSync11, statSync as statSync3 } from "fs";
 import { Hono as Hono10 } from "hono";
 import { extname, relative, resolve as resolve7 } from "path";
 var reviewNotesRoutes = new Hono10();
@@ -23139,7 +23023,7 @@ reviewNotesRoutes.get("/review-notes/artifact", (c) => {
   try {
     const stat = statSync3(abs);
     if (!stat.isFile() || stat.size > ARTIFACT_SERVE_MAX_BYTES) return c.text("Not found", 404);
-    const body = readFileSync12(abs);
+    const body = readFileSync11(abs);
     return c.body(body, 200, { "Content-Type": contentType });
   } catch {
     return c.text("Not found", 404);
@@ -23316,13 +23200,13 @@ apiRoutes.route("/", systemRoutes);
 import { spawnSync as spawnSync9 } from "child_process";
 import { mkdirSync as mkdirSync8 } from "fs";
 import { Hono as Hono15 } from "hono";
-import { join as join12 } from "path";
+import { join as join11 } from "path";
 var channelApiRoutes = new Hono15();
 channelApiRoutes.get("/status", async (c) => {
   const config2 = readGlobalConfig();
   const enabled = config2.channelEnabled === true;
   const repoRoot = c.get("repoRoot");
-  const dataDir = join12(repoRoot, ".glassbox");
+  const dataDir = join11(repoRoot, ".glassbox");
   const connected = enabled ? await isChannelAlive(dataDir) : false;
   return c.json({ enabled, connected });
 });
@@ -23331,7 +23215,7 @@ channelApiRoutes.post("/enable", (c) => {
     config2.channelEnabled = true;
   });
   const repoRoot = c.get("repoRoot");
-  const dataDir = join12(repoRoot, ".glassbox");
+  const dataDir = join11(repoRoot, ".glassbox");
   mkdirSync8(dataDir, { recursive: true });
   registerChannel(dataDir);
   return c.json({ ok: true });
@@ -23341,7 +23225,7 @@ channelApiRoutes.post("/disable", (c) => {
     config2.channelEnabled = false;
   });
   const repoRoot = c.get("repoRoot");
-  const dataDir = join12(repoRoot, ".glassbox");
+  const dataDir = join11(repoRoot, ".glassbox");
   unregisterChannel(dataDir);
   return c.json({ ok: true });
 });
@@ -23349,7 +23233,7 @@ channelApiRoutes.post("/trigger", async (c) => {
   const parsed = await parseBody(c, TriggerChannelReqSchema);
   if (!parsed.ok) return parsed.response;
   const repoRoot = c.get("repoRoot");
-  const dataDir = join12(repoRoot, ".glassbox");
+  const dataDir = join11(repoRoot, ".glassbox");
   const sent = await triggerChannel(dataDir, parsed.data.message);
   if (!sent) {
     return c.json({ error: "Channel not connected" }, 503);
@@ -23380,9 +23264,9 @@ channelApiRoutes.get("/claude-check", (c) => {
 import { Hono as Hono16 } from "hono";
 init_connection();
 init_queries();
-init_blob_store();
 init_session();
 init_difftool();
+init_image_blobs();
 var difftoolApiRoutes = new Hono16();
 difftoolApiRoutes.get("/status", (c) => {
   return c.json(getDifftoolStatus("global"));
@@ -23423,8 +23307,8 @@ difftoolApiRoutes.post("/append", async (c) => {
   if (diff.isBinary || isSvgFile(diff.filePath)) {
     const dataDir = getDataDir();
     if (dataDir !== null) {
-      writeDifftoolBlob(dataDir, fileId, "old", oldContent);
-      writeDifftoolBlob(dataDir, fileId, "new", newContent);
+      writeImageBlob(dataDir, fileId, "old", oldContent);
+      writeImageBlob(dataDir, fileId, "new", newContent);
     }
   }
   return c.json({ ok: true, fileId });
@@ -23460,7 +23344,7 @@ difftoolApiRoutes.post("/end", (c) => {
 });
 
 // src/routes/pages.tsx
-import { readFileSync as readFileSync14 } from "fs";
+import { readFileSync as readFileSync13 } from "fs";
 import { Hono as Hono17 } from "hono";
 import { resolve as resolve8 } from "path";
 
@@ -23645,25 +23529,31 @@ function ImageDiff({ file: file2, diff, fontWarning, baseWidth, baseHeight }) {
         /* @__PURE__ */ jsx2("div", { className: "image-diff-panel image-diff-metadata", "data-panel": "metadata", children: /* @__PURE__ */ jsx2("div", { className: "image-metadata-loading", children: "Loading metadata..." }) }),
         hasComparison && /* @__PURE__ */ jsx2("div", { className: "image-diff-panel image-diff-visual", "data-panel": "difference", children: /* @__PURE__ */ jsx2("div", { className: "image-visual-canvas", "data-zoomable": "true", children: /* @__PURE__ */ jsxs2("div", { className: "image-zoom-wrap", children: [
           /* @__PURE__ */ jsx2("img", { className: "image-layer image-layer-old", src: `/api/image/${fileId}/old`, alt: "Old version" }),
-          /* @__PURE__ */ jsx2("img", { className: "image-layer image-layer-new image-blend", src: `/api/image/${fileId}/new`, alt: "New version" })
+          /* @__PURE__ */ jsx2("img", { className: "image-layer image-layer-new image-blend", src: `/api/image/${fileId}/new`, alt: "New version" }),
+          /* @__PURE__ */ jsx2("div", { className: "region-overlay", "data-region-overlay": true })
         ] }) }) }),
         hasComparison && /* @__PURE__ */ jsx2("div", { className: "image-diff-panel image-diff-visual", "data-panel": "slice", children: /* @__PURE__ */ jsxs2("div", { className: "image-visual-canvas", "data-zoomable": "true", children: [
           /* @__PURE__ */ jsxs2("div", { className: "image-zoom-wrap", children: [
             /* @__PURE__ */ jsx2("img", { className: "image-layer image-layer-old", src: `/api/image/${fileId}/old`, alt: "Old version" }),
-            /* @__PURE__ */ jsx2("img", { className: "image-layer image-layer-new image-slice-clipped", src: `/api/image/${fileId}/new`, alt: "New version" })
+            /* @__PURE__ */ jsx2("img", { className: "image-layer image-layer-new image-slice-clipped", src: `/api/image/${fileId}/new`, alt: "New version" }),
+            /* @__PURE__ */ jsx2("div", { className: "region-overlay", "data-region-overlay": true })
           ] }),
           /* @__PURE__ */ jsx2("div", { className: "slice-line" }),
           /* @__PURE__ */ jsx2("div", { className: "slice-handle slice-handle-a" }),
           /* @__PURE__ */ jsx2("div", { className: "slice-handle slice-handle-b" })
         ] }) }),
-        !hasComparison && /* @__PURE__ */ jsx2("div", { className: "image-diff-panel image-diff-visual", "data-panel": "image", children: /* @__PURE__ */ jsx2("div", { className: "image-visual-canvas", "data-zoomable": "true", children: /* @__PURE__ */ jsx2("div", { className: "image-zoom-wrap", children: /* @__PURE__ */ jsx2(
-          "img",
-          {
-            className: "image-layer image-layer-old",
-            src: `/api/image/${fileId}/${isAdded ? "new" : "old"}`,
-            alt: isAdded ? "New image" : "Deleted image"
-          }
-        ) }) }) })
+        !hasComparison && /* @__PURE__ */ jsx2("div", { className: "image-diff-panel image-diff-visual", "data-panel": "image", children: /* @__PURE__ */ jsx2("div", { className: "image-visual-canvas", "data-zoomable": "true", children: /* @__PURE__ */ jsxs2("div", { className: "image-zoom-wrap", children: [
+          /* @__PURE__ */ jsx2(
+            "img",
+            {
+              className: "image-layer image-layer-old",
+              src: `/api/image/${fileId}/${isAdded ? "new" : "old"}`,
+              alt: isAdded ? "New image" : "Deleted image"
+            }
+          ),
+          /* @__PURE__ */ jsx2("div", { className: "region-overlay", "data-region-overlay": true })
+        ] }) }) }),
+        /* @__PURE__ */ jsx2("div", { className: "image-feedback", "data-image-feedback": true })
       ]
     }
   );
@@ -24087,9 +23977,9 @@ function AnnotationRows({ annotations }) {
 }
 
 // src/themes/config.ts
-import { existsSync as existsSync10, mkdirSync as mkdirSync9, readdirSync as readdirSync2, readFileSync as readFileSync13, unlinkSync as unlinkSync3, writeFileSync as writeFileSync9 } from "fs";
-import { join as join13 } from "path";
-var THEMES_DIR = join13(GLOBAL_CONFIG_DIR, "themes");
+import { existsSync as existsSync9, mkdirSync as mkdirSync9, readdirSync as readdirSync2, readFileSync as readFileSync12, unlinkSync as unlinkSync3, writeFileSync as writeFileSync9 } from "fs";
+import { join as join12 } from "path";
+var THEMES_DIR = join12(GLOBAL_CONFIG_DIR, "themes");
 function getActiveThemeId() {
   const config2 = readGlobalConfig();
   const theme = config2.theme;
@@ -24103,13 +23993,13 @@ function setActiveThemeId(id) {
   });
 }
 function loadCustomThemes() {
-  if (!existsSync10(THEMES_DIR)) return [];
+  if (!existsSync9(THEMES_DIR)) return [];
   const themes = [];
   try {
     const files = readdirSync2(THEMES_DIR).filter((f) => f.endsWith(".json"));
     for (const file2 of files) {
       try {
-        const parsed = StoredCustomThemeSchema.safeParse(JSON.parse(readFileSync13(join13(THEMES_DIR, file2), "utf-8")));
+        const parsed = StoredCustomThemeSchema.safeParse(JSON.parse(readFileSync12(join12(THEMES_DIR, file2), "utf-8")));
         if (!parsed.success) continue;
         const d = parsed.data;
         themes.push({ id: d.id, name: d.name, colors: d.colors, builtIn: false, baseTheme: d.baseTheme ?? "" });
@@ -24122,20 +24012,20 @@ function loadCustomThemes() {
 }
 function saveCustomTheme(theme) {
   mkdirSync9(THEMES_DIR, { recursive: true });
-  const filePath = join13(THEMES_DIR, `${theme.id}.json`);
+  const filePath = join12(THEMES_DIR, `${theme.id}.json`);
   writeFileSync9(filePath, JSON.stringify(theme, null, 2), "utf-8");
 }
 function deleteCustomTheme(id) {
-  const filePath = join13(THEMES_DIR, `${id}.json`);
-  if (existsSync10(filePath)) {
+  const filePath = join12(THEMES_DIR, `${id}.json`);
+  if (existsSync9(filePath)) {
     unlinkSync3(filePath);
   }
 }
 function getCustomTheme(id) {
-  const filePath = join13(THEMES_DIR, `${id}.json`);
-  if (!existsSync10(filePath)) return void 0;
+  const filePath = join12(THEMES_DIR, `${id}.json`);
+  if (!existsSync9(filePath)) return void 0;
   try {
-    const parsed = StoredCustomThemeSchema.safeParse(JSON.parse(readFileSync13(filePath, "utf-8")));
+    const parsed = StoredCustomThemeSchema.safeParse(JSON.parse(readFileSync12(filePath, "utf-8")));
     if (!parsed.success) return void 0;
     const d = parsed.data;
     return { id: d.id, name: d.name, colors: d.colors, builtIn: false, baseTheme: d.baseTheme ?? "" };
@@ -24435,6 +24325,29 @@ function ReviewShell({ reviewId, review, files, annotationCounts, staleCounts, f
 // src/routes/pages.tsx
 init_queries();
 
+// src/git/svg-meta.ts
+function parseSvgDimensions(svg) {
+  const widthMatch = svg.match(/\bwidth\s*=\s*["']([^"']+)["']/);
+  const heightMatch = svg.match(/\bheight\s*=\s*["']([^"']+)["']/);
+  const viewBoxMatch = svg.match(/\bviewBox\s*=\s*["']([^"']+)["']/);
+  let width = widthMatch ? parseFloat(widthMatch[1]) : NaN;
+  let height = heightMatch ? parseFloat(heightMatch[1]) : NaN;
+  if ((isNaN(width) || isNaN(height)) && viewBoxMatch) {
+    const parts = viewBoxMatch[1].split(/[\s,]+/);
+    if (parts.length >= 4) {
+      if (isNaN(width)) width = parseFloat(parts[2]);
+      if (isNaN(height)) height = parseFloat(parts[3]);
+    }
+  }
+  if (isNaN(width)) width = 300;
+  if (isNaN(height)) height = 150;
+  return { width, height };
+}
+function svgUsesExternalFonts(svgData) {
+  const svg = svgData.toString("utf-8");
+  return /<text[\s>]/i.test(svg) || /font-family/i.test(svg) || /@font-face/i.test(svg);
+}
+
 // src/review-notes/reanchor.ts
 var MATCH_RADIUS = 50;
 function reanchorReviewNotes(notes, diff) {
@@ -24558,7 +24471,7 @@ pageRoutes.get("/file-raw", (c) => {
   const repoRoot = c.get("repoRoot");
   let content;
   try {
-    content = readFileSync14(resolve8(repoRoot, filePath), "utf-8");
+    content = readFileSync13(resolve8(repoRoot, filePath), "utf-8");
   } catch {
     return c.text("File not found", 404);
   }
@@ -24748,17 +24661,17 @@ async function startServer(port, reviewId, repoRoot, options) {
     await next();
   });
   const selfDir = dirname4(fileURLToPath2(import.meta.url));
-  const distDir = existsSync11(join14(selfDir, "client", "styles.css")) ? join14(selfDir, "client") : join14(selfDir, "..", "dist", "client");
+  const distDir = existsSync10(join13(selfDir, "client", "styles.css")) ? join13(selfDir, "client") : join13(selfDir, "..", "dist", "client");
   app.get("/static/styles.css", (c) => {
-    const css = readFileSync15(join14(distDir, "styles.css"), "utf-8");
+    const css = readFileSync14(join13(distDir, "styles.css"), "utf-8");
     return c.text(css, 200, { "Content-Type": "text/css", "Cache-Control": "no-cache" });
   });
   app.get("/static/app.js", (c) => {
-    const js = readFileSync15(join14(distDir, "app.global.js"), "utf-8");
+    const js = readFileSync14(join13(distDir, "app.global.js"), "utf-8");
     return c.text(js, 200, { "Content-Type": "application/javascript", "Cache-Control": "no-cache" });
   });
   app.get("/static/history.js", (c) => {
-    const js = readFileSync15(join14(distDir, "history.global.js"), "utf-8");
+    const js = readFileSync14(join13(distDir, "history.global.js"), "utf-8");
     return c.text(js, 200, { "Content-Type": "application/javascript", "Cache-Control": "no-cache" });
   });
   app.get("/favicon.ico", (c) => c.body(null, 204));
@@ -24796,7 +24709,7 @@ async function startServer(port, reviewId, repoRoot, options) {
   try {
     const globalConfig2 = readGlobalConfig();
     if (globalConfig2.channelEnabled === true) {
-      const dataDir = join14(repoRoot, ".glassbox");
+      const dataDir = join13(repoRoot, ".glassbox");
       registerChannel(dataDir);
     }
   } catch {
@@ -24811,8 +24724,8 @@ async function startServer(port, reviewId, repoRoot, options) {
 }
 
 // src/skills.ts
-import { existsSync as existsSync12, mkdirSync as mkdirSync10, readFileSync as readFileSync16, writeFileSync as writeFileSync10 } from "fs";
-import { join as join15 } from "path";
+import { existsSync as existsSync11, mkdirSync as mkdirSync10, readFileSync as readFileSync15, writeFileSync as writeFileSync10 } from "fs";
+import { join as join14 } from "path";
 var SKILL_VERSION = 1;
 function versionHeader() {
   return `<!-- glassbox-skill-version: ${SKILL_VERSION} -->`;
@@ -24823,8 +24736,8 @@ function parseVersionHeader(content) {
   return parseInt(match[1], 10);
 }
 function updateFile(path, content) {
-  if (existsSync12(path)) {
-    const existing = readFileSync16(path, "utf-8");
+  if (existsSync11(path)) {
+    const existing = readFileSync15(path, "utf-8");
     const version2 = parseVersionHeader(existing);
     if (version2 !== null && version2 >= SKILL_VERSION) {
       return false;
@@ -24850,7 +24763,7 @@ function skillBody() {
   ].join("\n");
 }
 function ensureClaudeSkills(cwd) {
-  const dir = join15(cwd, ".claude", "skills", "glassbox");
+  const dir = join14(cwd, ".claude", "skills", "glassbox");
   mkdirSync10(dir, { recursive: true });
   const content = [
     "---",
@@ -24863,10 +24776,10 @@ function ensureClaudeSkills(cwd) {
     skillBody(),
     ""
   ].join("\n");
-  return updateFile(join15(dir, "SKILL.md"), content);
+  return updateFile(join14(dir, "SKILL.md"), content);
 }
 function ensureCursorRules(cwd) {
-  const rulesDir = join15(cwd, ".cursor", "rules");
+  const rulesDir = join14(cwd, ".cursor", "rules");
   mkdirSync10(rulesDir, { recursive: true });
   const content = [
     "---",
@@ -24878,10 +24791,10 @@ function ensureCursorRules(cwd) {
     skillBody(),
     ""
   ].join("\n");
-  return updateFile(join15(rulesDir, "glassbox.mdc"), content);
+  return updateFile(join14(rulesDir, "glassbox.mdc"), content);
 }
 function ensureCopilotPrompts(cwd) {
-  const promptsDir = join15(cwd, ".github", "prompts");
+  const promptsDir = join14(cwd, ".github", "prompts");
   mkdirSync10(promptsDir, { recursive: true });
   const content = [
     "---",
@@ -24892,10 +24805,10 @@ function ensureCopilotPrompts(cwd) {
     skillBody(),
     ""
   ].join("\n");
-  return updateFile(join15(promptsDir, "glassbox.prompt.md"), content);
+  return updateFile(join14(promptsDir, "glassbox.prompt.md"), content);
 }
 function ensureWindsurfRules(cwd) {
-  const rulesDir = join15(cwd, ".windsurf", "rules");
+  const rulesDir = join14(cwd, ".windsurf", "rules");
   mkdirSync10(rulesDir, { recursive: true });
   const content = [
     "---",
@@ -24907,21 +24820,21 @@ function ensureWindsurfRules(cwd) {
     skillBody(),
     ""
   ].join("\n");
-  return updateFile(join15(rulesDir, "glassbox.md"), content);
+  return updateFile(join14(rulesDir, "glassbox.md"), content);
 }
 function ensureSkills() {
   const cwd = process.cwd();
   const platforms = [];
-  if (existsSync12(join15(cwd, ".claude"))) {
+  if (existsSync11(join14(cwd, ".claude"))) {
     if (ensureClaudeSkills(cwd)) platforms.push("Claude Code");
   }
-  if (existsSync12(join15(cwd, ".cursor"))) {
+  if (existsSync11(join14(cwd, ".cursor"))) {
     if (ensureCursorRules(cwd)) platforms.push("Cursor");
   }
-  if (existsSync12(join15(cwd, ".github", "prompts")) || existsSync12(join15(cwd, ".github", "copilot-instructions.md"))) {
+  if (existsSync11(join14(cwd, ".github", "prompts")) || existsSync11(join14(cwd, ".github", "copilot-instructions.md"))) {
     if (ensureCopilotPrompts(cwd)) platforms.push("GitHub Copilot");
   }
-  if (existsSync12(join15(cwd, ".windsurf"))) {
+  if (existsSync11(join14(cwd, ".windsurf"))) {
     if (ensureWindsurfRules(cwd)) platforms.push("Windsurf");
   }
   return platforms;
@@ -24929,19 +24842,19 @@ function ensureSkills() {
 
 // src/update-check.ts
 init_zod();
-import { existsSync as existsSync13, mkdirSync as mkdirSync11, readFileSync as readFileSync17, writeFileSync as writeFileSync11 } from "fs";
+import { existsSync as existsSync12, mkdirSync as mkdirSync11, readFileSync as readFileSync16, writeFileSync as writeFileSync11 } from "fs";
 import { get } from "https";
 import { homedir as homedir3 } from "os";
-import { dirname as dirname5, join as join16 } from "path";
+import { dirname as dirname5, join as join15 } from "path";
 import { fileURLToPath as fileURLToPath3 } from "url";
-var DATA_DIR = join16(homedir3(), ".glassbox");
-var CHECK_FILE = join16(DATA_DIR, "last-update-check");
+var DATA_DIR = join15(homedir3(), ".glassbox");
+var CHECK_FILE = join15(DATA_DIR, "last-update-check");
 var PACKAGE_NAME = "glassbox";
 var VersionPayloadSchema = external_exports.object({ version: external_exports.string() });
 function getCurrentVersion() {
   try {
     const dir = dirname5(fileURLToPath3(import.meta.url));
-    const raw2 = JSON.parse(readFileSync17(join16(dir, "..", "package.json"), "utf-8"));
+    const raw2 = JSON.parse(readFileSync16(join15(dir, "..", "package.json"), "utf-8"));
     return VersionPayloadSchema.parse(raw2).version;
   } catch {
     return "0.0.0";
@@ -24949,8 +24862,8 @@ function getCurrentVersion() {
 }
 function getLastCheckDate() {
   try {
-    if (existsSync13(CHECK_FILE)) {
-      return readFileSync17(CHECK_FILE, "utf-8").trim();
+    if (existsSync12(CHECK_FILE)) {
+      return readFileSync16(CHECK_FILE, "utf-8").trim();
     }
   } catch {
   }
@@ -25258,22 +25171,22 @@ async function main() {
     console.log("AI service test mode enabled \u2014 using mock AI responses");
   }
   if (debug) {
-    console.log(`[debug] Build timestamp: ${"2026-06-20T04:34:27.118Z"}`);
+    console.log(`[debug] Build timestamp: ${"2026-06-22T06:43:57.425Z"}`);
   }
   if (projectDir !== null) {
     process.chdir(projectDir);
   }
   if (dataDir === null) {
-    dataDir = join18(process.cwd(), ".glassbox");
+    dataDir = join17(process.cwd(), ".glassbox");
   }
   if (difftoolServe) {
     const { initDifftoolSession: initDifftoolSession2 } = await Promise.resolve().then(() => (init_session(), session_exports));
     const { writeDiscovery: writeDiscovery2, clearDiscovery: clearDiscovery2, releaseStartingLock: releaseStartingLock2 } = await Promise.resolve().then(() => (init_difftool_discovery(), difftool_discovery_exports));
-    const { clearDifftoolBlobs: clearDifftoolBlobs2 } = await Promise.resolve().then(() => (init_blob_store(), blob_store_exports));
+    const { clearImageBlobs: clearImageBlobs2 } = await Promise.resolve().then(() => (init_image_blobs(), image_blobs_exports));
     mkdirSync13(dataDir, { recursive: true });
     setDataDir(dataDir);
     const sessionDataDir = dataDir;
-    clearDifftoolBlobs2(sessionDataDir);
+    clearImageBlobs2(sessionDataDir);
     const repoRoot2 = process.cwd();
     const review2 = await createReview(repoRoot2, "git difftool", "difftool");
     const { port: actualPort, server } = await startServer(port, review2.id, repoRoot2, { noOpen, strictPort });
@@ -25285,7 +25198,7 @@ async function main() {
           server.close();
         } catch {
         }
-        clearDifftoolBlobs2(sessionDataDir);
+        clearImageBlobs2(sessionDataDir);
         clearDiscovery2();
         releaseStartingLock2();
         process.exit(0);
@@ -25305,7 +25218,7 @@ async function main() {
       }
       process.exit(1);
     }
-    dataDir = join18(tmpdir2(), `glassbox-demo-${demo}-${Date.now()}`);
+    dataDir = join17(tmpdir2(), `glassbox-demo-${demo}-${Date.now()}`);
     setDemoMode(demo);
     console.log(`
   DEMO MODE: ${scenario.label}
@@ -25333,7 +25246,7 @@ async function main() {
   if (mode.type === "diff") {
     const { pathA, pathB } = mode;
     for (const p of [pathA, pathB]) {
-      if (!existsSync15(p)) {
+      if (!existsSync14(p)) {
         console.error(`Error: path does not exist: ${p}`);
         process.exit(1);
       }