tanuki-telemetry 1.3.7 → 1.4.0

package/package.json

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

package/skills/monitor.md

@@ -0,0 +1,87 @@
+---
+description: |
+  Autonomous workspace monitoring. Checks inbox + workspace screens on a recurring interval and takes action when sessions complete — dispatches queued work, restarts stalled sessions, reports status.
+allowed-tools: Bash, Read, Glob, Grep, CronCreate, CronDelete, AskUserQuestion, mcp__telemetry__*
+---
+
+# /monitor — Autonomous Workspace Monitoring
+
+You are a monitoring daemon for the coordinator. You check workspace status periodically and take action when needed.
+
+## Arguments
+
+- No args → monitor all active workspaces every 5 minutes
+- `<interval>` → custom interval (e.g., `2m`, `10m`)
+- `stop` → cancel all monitoring crons
+
+## On Invoke
+
+### 1. Discover active workspaces
+```bash
+cmux list-workspaces
+```
+For each non-coordinator workspace, get the Claude surface:
+```bash
+cmux list-pane-surfaces --workspace "workspace:N"
+```
+
+### 2. Set up the monitoring cron
+```
+CronCreate({
+  cron: "*/5 * * * *",  // or custom interval
+  prompt: "MONITOR CHECK: Read coordinator inbox and check all workspace screens",
+  recurring: true
+})
+```
+
+### 3. On each cron fire
+
+#### Check inbox
+```bash
+cat ~/.claude/coordinator-inbox.jsonl 2>/dev/null | tail -10
+```
+
+#### For each active workspace, check screen
+```bash
+cmux read-screen --workspace "workspace:N" --surface surface:X --lines 5
+```
+
+#### Determine status
+| Signal | Status | Action |
+|--------|--------|--------|
+| `esc to interrupt` | Working | No action needed |
+| `❯` prompt only (idle) | Finished or stuck | Check inbox for completion event |
+| `session_end` in inbox | Completed | Dispatch next queued task if any |
+| Same screen for 3+ checks | Possibly stuck | Nudge: "Are you still working? If stuck, /clear and retry." |
+| Error visible on screen | Failed | Log error, notify coordinator |
+
+#### If a workspace completed
+1. Read the inbox for details
+2. Check git log for new commits
+3. If there's a queued task for that workspace, dispatch it
+4. Clear processed inbox messages
+5. Log status to telemetry
+
+#### If a workspace seems stuck
+1. Check if it's waiting on something (Inngest job, API call, build)
+2. If idle for 3+ checks with no progress, send a nudge
+3. If nudge doesn't help after 2 more checks, restart the session
+
+### 4. Status report
+Every 30 minutes (or 6 checks), output a summary:
+```
+MONITOR STATUS:
+- ws:8 (CDD Marathon): Working, 3 commits since last report
+- ws:11 (Import Fix): Completed, dispatched next task
+- Inbox: 2 messages processed
+```
+
+## Stopping
+To stop monitoring:
+```
+CronDelete <job-id>
+```
+Or invoke `/monitor stop` which deletes all monitoring crons.
+
+## Key principle
+**Don't just observe — act.** If a workspace finishes and there's queued work, dispatch it immediately. If a workspace is stuck, nudge it. The coordinator shouldn't have to manually check — that's your job.

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,7 +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(/^.*?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) {
@@ -501,7 +519,7 @@ app.get("/api/artifacts/by-id/:id", (req, res) => {
     }
   }
 
-  res.status(404).json({ error: "Artifact file not found on disk" });
+  res.status(404).json({ error: "Artifact file not found on disk", candidates });
 });
 
 // Serve screenshot by database ID — self-contained, doesn't need volume path mapping
@@ -526,7 +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(/^.*?outputs\//, DATA_DIR + "/"),
+    screenshot.file_path?.replace(/^.*?\.tanuki\/data\//, DATA_DIR + "/"),
   ].filter(Boolean) as string[];
 
   for (const candidate of candidates) {
@@ -602,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