glassbox 0.4.4 → 0.5.1

package/dist/cli.js

@@ -395,7 +395,7 @@ init_queries();
 init_connection();
 import { mkdirSync as mkdirSync7 } from "fs";
 import { tmpdir } from "os";
-import { join as join10, resolve as resolve4 } from "path";
+import { join as join10, resolve as resolve5 } from "path";
 
 // src/debug.ts
 var debugEnabled = false;
@@ -1723,7 +1723,7 @@ async function updateReviewDiffs(reviewId, newDiffs, headCommit) {
 // src/server.ts
 import { serve } from "@hono/node-server";
 import { exec } from "child_process";
-import { existsSync as existsSync6, readFileSync as readFileSync8 } from "fs";
+import { existsSync as existsSync6, readFileSync as readFileSync9 } from "fs";
 import { Hono as Hono4 } from "hono";
 import { dirname, join as join7 } from "path";
 import { fileURLToPath } from "url";
@@ -2445,8 +2445,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((resolve5) => {
-    setTimeout(resolve5, ms);
+  return new Promise((resolve6) => {
+    setTimeout(resolve6, ms);
   });
 }
 
@@ -2490,8 +2490,8 @@ function randomLines(count) {
   return lines.sort((a, b) => a.line - b.line);
 }
 function sleep2(ms) {
-  return new Promise((resolve5) => {
-    setTimeout(resolve5, ms);
+  return new Promise((resolve6) => {
+    setTimeout(resolve6, ms);
   });
 }
 async function mockRiskAnalysisBatch(files) {
@@ -4059,6 +4059,71 @@ apiRoutes.get("/outline/:fileId", async (c) => {
   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) 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);
+  }
+  if (definitions.length === 0) {
+    try {
+      const allFiles = execSync5("git ls-files", { cwd: repoRoot, encoding: "utf-8" }).trim().split("\n").filter(Boolean);
+      for (const filePath of allFiles) {
+        if (searchedPaths.has(filePath)) continue;
+        const ext = filePath.slice(filePath.lastIndexOf("."));
+        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 = readFileSync7(resolve3(repoRoot, filePath), "utf-8");
+        } catch {
+          continue;
+        }
+        if (!content) continue;
+        const symbols = parseOutline(content, filePath);
+        collectDefinitions(symbols, name, null, filePath, definitions);
+        if (definitions.length > 0) break;
+      }
+    } catch {
+    }
+  }
+  definitions.sort((a, b) => {
+    if (a.fileId === currentFileId && b.fileId !== currentFileId) return -1;
+    if (b.fileId === currentFileId && a.fileId !== currentFileId) return 1;
+    if (a.fileId !== null && b.fileId === null) return -1;
+    if (b.fileId !== null && a.fileId === null) return 1;
+    if (a.kind === "class" && b.kind !== "class") return -1;
+    if (b.kind === "class" && a.kind !== "class") return 1;
+    return 0;
+  });
+  return c.json({ definitions });
+});
+function collectDefinitions(symbols, targetName, fileId, filePath, out) {
+  for (const sym of symbols) {
+    if (sym.name === targetName) {
+      out.push({ fileId, filePath, name: sym.name, kind: sym.kind, line: sym.line });
+    }
+    if (sym.children?.length > 0) {
+      collectDefinitions(sym.children, targetName, fileId, filePath, out);
+    }
+  }
+}
 apiRoutes.get("/context/:fileId", async (c) => {
   const repoRoot = c.get("repoRoot");
   const file = await getReviewFile(c.req.param("fileId"));
@@ -4155,7 +4220,9 @@ apiRoutes.get("/image/:fileId/:side", async (c) => {
 });
 
 // src/routes/pages.tsx
+import { readFileSync as readFileSync8 } from "fs";
 import { Hono as Hono3 } from "hono";
+import { resolve as resolve4 } from "path";
 
 // src/utils/escapeHtml.ts
 function escapeHtml(str) {
@@ -5059,6 +5126,11 @@ pageRoutes.get("/", async (c) => {
           ] }),
           /* @__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: [
@@ -5161,6 +5233,39 @@ pageRoutes.get("/file/:fileId", async (c) => {
   const html = /* @__PURE__ */ jsx(DiffView, { file, diff: finalDiff, annotations, mode });
   return c.html(html.toString());
 });
+pageRoutes.get("/file-raw", (c) => {
+  const filePath = c.req.query("path");
+  if (!filePath) return c.text("Missing path", 400);
+  const repoRoot = c.get("repoRoot");
+  let content;
+  try {
+    content = readFileSync8(resolve4(repoRoot, filePath), "utf-8");
+  } catch {
+    return c.text("File not found", 404);
+  }
+  const lines = content.split("\n");
+  const diff = {
+    filePath,
+    oldPath: null,
+    status: "added",
+    isBinary: false,
+    hunks: [{
+      oldStart: 0,
+      oldCount: 0,
+      newStart: 1,
+      newCount: lines.length,
+      lines: lines.map((line, i) => ({
+        type: "context",
+        oldNum: i + 1,
+        newNum: i + 1,
+        content: line
+      }))
+    }]
+  };
+  const fakeFile = { id: "", review_id: "", file_path: filePath, status: "reviewed", diff_data: null };
+  const html = /* @__PURE__ */ jsx(DiffView, { file: fakeFile, diff, annotations: [], mode: "unified" });
+  return c.html(html.toString());
+});
 pageRoutes.get("/review/:reviewId", async (c) => {
   const reviewId = c.req.param("reviewId");
   const currentReviewId = c.get("reviewId");
@@ -5210,6 +5315,11 @@ pageRoutes.get("/review/:reviewId", async (c) => {
           ] }),
           /* @__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: [
@@ -5257,10 +5367,10 @@ pageRoutes.get("/history", async (c) => {
 
 // src/server.ts
 function tryServe(fetch2, port) {
-  return new Promise((resolve5, reject) => {
+  return new Promise((resolve6, reject) => {
     const server = serve({ fetch: fetch2, port });
     server.on("listening", () => {
-      resolve5(port);
+      resolve6(port);
     });
     server.on("error", (err) => {
       if (err.code === "EADDRINUSE") {
@@ -5282,15 +5392,15 @@ async function startServer(port, reviewId, repoRoot, options) {
   const selfDir = dirname(fileURLToPath(import.meta.url));
   const distDir = existsSync6(join7(selfDir, "client", "styles.css")) ? join7(selfDir, "client") : join7(selfDir, "..", "dist", "client");
   app.get("/static/styles.css", (c) => {
-    const css = readFileSync8(join7(distDir, "styles.css"), "utf-8");
+    const css = readFileSync9(join7(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 = readFileSync8(join7(distDir, "app.global.js"), "utf-8");
+    const js = readFileSync9(join7(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 = readFileSync8(join7(distDir, "history.global.js"), "utf-8");
+    const js = readFileSync9(join7(distDir, "history.global.js"), "utf-8");
     return c.text(js, 200, { "Content-Type": "application/javascript", "Cache-Control": "no-cache" });
   });
   app.route("/api", apiRoutes);
@@ -5326,7 +5436,7 @@ async function startServer(port, reviewId, repoRoot, options) {
 }
 
 // src/skills.ts
-import { existsSync as existsSync7, mkdirSync as mkdirSync5, readFileSync as readFileSync9, writeFileSync as writeFileSync5 } from "fs";
+import { existsSync as existsSync7, mkdirSync as mkdirSync5, readFileSync as readFileSync10, writeFileSync as writeFileSync5 } from "fs";
 import { join as join8 } from "path";
 var SKILL_VERSION = 1;
 function versionHeader() {
@@ -5339,7 +5449,7 @@ function parseVersionHeader(content) {
 }
 function updateFile(path, content) {
   if (existsSync7(path)) {
-    const existing = readFileSync9(path, "utf-8");
+    const existing = readFileSync10(path, "utf-8");
     const version = parseVersionHeader(existing);
     if (version !== null && version >= SKILL_VERSION) {
       return false;
@@ -5443,7 +5553,7 @@ function ensureSkills() {
 }
 
 // src/update-check.ts
-import { existsSync as existsSync8, mkdirSync as mkdirSync6, readFileSync as readFileSync10, writeFileSync as writeFileSync6 } from "fs";
+import { existsSync as existsSync8, mkdirSync as mkdirSync6, readFileSync as readFileSync11, writeFileSync as writeFileSync6 } from "fs";
 import { get } from "https";
 import { homedir as homedir3 } from "os";
 import { dirname as dirname2, join as join9 } from "path";
@@ -5454,7 +5564,7 @@ var PACKAGE_NAME = "glassbox";
 function getCurrentVersion() {
   try {
     const dir = dirname2(fileURLToPath2(import.meta.url));
-    const pkg = JSON.parse(readFileSync10(join9(dir, "..", "package.json"), "utf-8"));
+    const pkg = JSON.parse(readFileSync11(join9(dir, "..", "package.json"), "utf-8"));
     return pkg.version;
   } catch {
     return "0.0.0";
@@ -5463,7 +5573,7 @@ function getCurrentVersion() {
 function getLastCheckDate() {
   try {
     if (existsSync8(CHECK_FILE)) {
-      return readFileSync10(CHECK_FILE, "utf-8").trim();
+      return readFileSync11(CHECK_FILE, "utf-8").trim();
     }
   } catch {
   }
@@ -5480,10 +5590,10 @@ function isFirstUseToday() {
   return last !== today;
 }
 function fetchLatestVersion() {
-  return new Promise((resolve5) => {
+  return new Promise((resolve6) => {
     const req = get(`https://registry.npmjs.org/${PACKAGE_NAME}/latest`, { timeout: 5e3 }, (res) => {
       if (res.statusCode !== 200) {
-        resolve5(null);
+        resolve6(null);
         return;
       }
       let data = "";
@@ -5492,18 +5602,18 @@ function fetchLatestVersion() {
       });
       res.on("end", () => {
         try {
-          resolve5(JSON.parse(data).version);
+          resolve6(JSON.parse(data).version);
         } catch {
-          resolve5(null);
+          resolve6(null);
         }
       });
     });
     req.on("error", () => {
-      resolve5(null);
+      resolve6(null);
     });
     req.on("timeout", () => {
       req.destroy();
-      resolve5(null);
+      resolve6(null);
     });
   });
 }
@@ -5641,7 +5751,7 @@ function parseArgs(argv) {
         port = parseInt(args[++i], 10);
         break;
       case "--data-dir":
-        dataDir = resolve4(args[++i]);
+        dataDir = resolve5(args[++i]);
         break;
       case "--resume":
         resume = true;
@@ -5697,7 +5807,7 @@ async function main() {
     console.log("AI service test mode enabled \u2014 using mock AI responses");
   }
   if (debug) {
-    console.log(`[debug] Build timestamp: ${"2026-03-21T23:17:12.455Z"}`);
+    console.log(`[debug] Build timestamp: ${"2026-03-26T23:51:09.647Z"}`);
   }
   if (projectDir) {
     process.chdir(projectDir);