tanuki-telemetry 1.3.8 → 1.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tanuki-telemetry",
-  "version": "1.3.8",
+  "version": "1.4.0",
   "description": "Workflow monitor and telemetry dashboard for Claude Code autonomous agents",
   "type": "module",
   "bin": {

package/src/dashboard.ts

@@ -19,6 +19,8 @@ import type { Session, Event, Iteration, Screenshot, Artifact, Insight, PlanStep
 import { listCoordinatorSessions, getCoordinatorState, getCoordinatorHistory } from "./coordinator.js";
 import { fileURLToPath } from "url";
 
+const DATA_DIR = process.env.DATA_DIR || "/data";
+
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 const app = express();
@@ -39,7 +41,7 @@ const SqliteStore = BetterSqlite3SessionStore(session);
 // (the store hardcodes table name "sessions" which conflicts with our telemetry sessions table)
 let sessionStoreInstance: InstanceType<typeof SqliteStore> | undefined;
 if (AUTH_ENABLED) {
-  const dbPath = process.env.DB_PATH || "/data/telemetry.db";
+  const dbPath = process.env.DB_PATH || path.join(DATA_DIR, "telemetry.db");
   const sessionDbPath = dbPath.replace(/\.db$/, "-sessions.db");
   const sessionDb = new Database(sessionDbPath);
   sessionDb.pragma("journal_mode = WAL");
@@ -97,13 +99,26 @@ app.get("/health", (_req, res) => {
   res.json({ ok: true, version: TANUKI_VERSION });
 });
 
+/** True when `a` is strictly newer than `b` using major.minor.patch comparison. */
+function isNewerVersion(a: string, b: string): boolean {
+  const pa = a.split(".").map(Number);
+  const pb = b.split(".").map(Number);
+  for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
+    const va = pa[i] ?? 0;
+    const vb = pb[i] ?? 0;
+    if (va > vb) return true;
+    if (va < vb) return false;
+  }
+  return false;
+}
+
 // Version endpoint
 app.get("/api/version", async (_req, res) => {
   const latest = await getLatestNpmVersion();
   res.json({
     current: TANUKI_VERSION,
     latest: latest ?? TANUKI_VERSION,
-    updateAvailable: latest ? latest !== TANUKI_VERSION : false,
+    updateAvailable: latest ? isNewerVersion(latest, TANUKI_VERSION) : false,
   });
 });
 
@@ -122,7 +137,7 @@ const upload = multer({
       const d = getDb();
       const session = d.prepare("SELECT worktree_name FROM sessions WHERE id = ?").get(sessionId) as { worktree_name: string } | undefined;
       const dirName = session?.worktree_name || sessionId;
-      const dir = path.join("/data", dirName, "screenshots");
+      const dir = path.join(DATA_DIR, dirName, "screenshots");
       fs.mkdirSync(dir, { recursive: true });
       cb(null, dir);
     },
@@ -488,8 +503,10 @@ app.get("/api/artifacts/by-id/:id", (req, res) => {
   const candidates = [
     artifact.stored_path,
     artifact.file_path,
-    artifact.file_path?.replace(/^.*?outputs\//, "/data/"),
-    artifact.file_path?.replace(/^.*?\.tanuki\/data\//, "/data/"),
+    artifact.file_path?.replace(/^.*?outputs\//, DATA_DIR + "/"),
+    artifact.file_path?.replace(/^.*?\.tanuki\/data\//, DATA_DIR + "/"),
+    artifact.file_path?.replace(/^.*?outputs\//, "/outputs/"),
+    artifact.file_path?.replace(/^.*?outputs/, "/outputs"),
   ].filter(Boolean) as string[];
 
   for (const candidate of candidates) {
@@ -527,8 +544,8 @@ app.get("/api/screenshots/by-id/:id", (req, res) => {
   const candidates = [
     screenshot.stored_path,
     screenshot.file_path,
-    screenshot.file_path?.replace(/^.*?outputs\//, "/data/"),
-    screenshot.file_path?.replace(/^.*?\.tanuki\/data\//, "/data/"),
+    screenshot.file_path?.replace(/^.*?outputs\//, DATA_DIR + "/"),
+    screenshot.file_path?.replace(/^.*?\.tanuki\/data\//, DATA_DIR + "/"),
   ].filter(Boolean) as string[];
 
   for (const candidate of candidates) {
@@ -604,11 +621,11 @@ app.get("/api/screenshots/*", (req, res) => {
   if (requestedPath.startsWith("/")) {
     filePath = requestedPath;
   } else {
-    filePath = path.join("/data", requestedPath);
+    filePath = path.join(DATA_DIR, requestedPath);
   }
 
   const resolved = path.resolve(filePath);
-  if (!resolved.startsWith("/data")) {
+  if (!resolved.startsWith(DATA_DIR)) {
     res.status(403).json({ error: "Access denied" });
     return;
   }

package/src/db.ts

@@ -14,7 +14,8 @@ import type {
   WalkthroughScreenshot,
 } from "./types.js";
 
-const DB_PATH = process.env.DB_PATH || "/data/telemetry.db";
+const DATA_DIR = process.env.DATA_DIR || "/data";
+const DB_PATH = process.env.DB_PATH || path.join(DATA_DIR, "telemetry.db");
 
 let db: Database.Database;
 
@@ -256,15 +257,15 @@ function initTables(): void {
   `);
 
   // Ensure storage directories exist
-  const screenshotsDir = "/data/screenshots";
+  const screenshotsDir = path.join(DATA_DIR, "screenshots");
   if (!fs.existsSync(screenshotsDir)) {
     fs.mkdirSync(screenshotsDir, { recursive: true });
   }
-  const artifactsDir = "/data/artifacts";
+  const artifactsDir = path.join(DATA_DIR, "artifacts");
   if (!fs.existsSync(artifactsDir)) {
     fs.mkdirSync(artifactsDir, { recursive: true });
   }
-  const walkthroughScreenshotsDir = "/data/walkthrough-screenshots";
+  const walkthroughScreenshotsDir = path.join(DATA_DIR, "walkthrough-screenshots");
   if (!fs.existsSync(walkthroughScreenshotsDir)) {
     fs.mkdirSync(walkthroughScreenshotsDir, { recursive: true });
   }
@@ -404,15 +405,15 @@ export function insertScreenshot(
   const ext = path.extname(file_path) || ".png";
   const storedName = `${session_id}_${screenshotId}${ext}`;
   const thumbName = `${session_id}_${screenshotId}_thumb${ext}`;
-  const storedPath = path.join("/data", "screenshots", storedName);
-  const thumbPath = path.join("/data", "screenshots", thumbName);
+  const storedPath = path.join(DATA_DIR, "screenshots", storedName);
+  const thumbPath = path.join(DATA_DIR, "screenshots", thumbName);
 
   try {
     // The file_path might be a host path — try multiple locations
     const candidates = [
       file_path,
-      // Map host path into /data mount: /Users/.../outputs/foo → /data/foo
-      file_path.replace(/^.*?outputs\//, "/data/"),
+      // Map host path into data dir: /Users/.../outputs/foo → <DATA_DIR>/foo
+      file_path.replace(/^.*?outputs\//, DATA_DIR + "/"),
     ];
 
     for (const candidate of candidates) {
@@ -491,12 +492,12 @@ export function insertArtifact(
 
   // Try to copy the file into /data/artifacts/ for self-contained serving
   const storedName = `${session_id}_${artifactId}${ext}`;
-  const storedPath = path.join("/data", "artifacts", storedName);
+  const storedPath = path.join(DATA_DIR, "artifacts", storedName);
 
   try {
     const candidates = [
       file_path,
-      file_path.replace(/^.*?outputs\//, "/data/"),
+      file_path.replace(/^.*?outputs\//, DATA_DIR + "/"),
     ];
 
     for (const candidate of candidates) {
@@ -959,7 +960,7 @@ export function insertWalkthroughScreenshot(
 
   const safeName = name.replace(/[^a-zA-Z0-9_-]/g, "_");
   const filename = `wt_${walkthrough_id}_${Date.now()}_${safeName}.png`;
-  const storedPath = path.join("/data", "walkthrough-screenshots", filename);
+  const storedPath = path.join(DATA_DIR, "walkthrough-screenshots", filename);
 
   if (file_path_input && fs.existsSync(file_path_input)) {
     // Copy from file path