pi-inspect 0.4.0 → 0.6.0

package/lib/snapshot.js

@@ -1,102 +1,80 @@
-const fs = require('node:fs');
+'use strict';
 const fsp = require('node:fs/promises');
 const path = require('node:path');
-const os = require('node:os');
 
-const SNAPSHOT_DIR = process.env.INSPECT_SNAPSHOT_DIR
-  || path.join(os.homedir(), '.pi', 'agent', 'inspect', 'snapshots');
-const INDEX_PATH = path.join(SNAPSHOT_DIR, 'index.json');
-
-function snapshotDir() {
-  return SNAPSHOT_DIR;
-}
-
-function snapshotPath(sessionId) {
-  return path.join(SNAPSHOT_DIR, `${sanitize(sessionId)}.json`);
-}
+const INDEX_FILE = 'index.json';
 
+// Same rule as the extension's file names, so an id maps to the file it wrote.
 function sanitize(id) {
   return String(id).replace(/[^a-zA-Z0-9._-]/g, '_');
 }
 
-async function readIndex() {
-  try {
-    const raw = await fsp.readFile(INDEX_PATH, 'utf8');
-    const parsed = JSON.parse(raw);
-    return Array.isArray(parsed?.sessions) ? parsed.sessions : [];
-  } catch (e) {
-    if (e.code !== 'ENOENT') console.warn(`pi-inspect: read index: ${e.message}`);
-    return [];
-  }
+async function writeJsonAtomic(file, value) {
+  await fsp.mkdir(path.dirname(file), { recursive: true });
+  const tmp = `${file}.${process.pid}.tmp`;
+  await fsp.writeFile(tmp, JSON.stringify(value, null, 2), 'utf8');
+  await fsp.rename(tmp, file);
 }
 
-async function readSnapshot(sessionId) {
-  try {
-    const raw = await fsp.readFile(snapshotPath(sessionId), 'utf8');
-    return JSON.parse(raw);
-  } catch (e) {
-    if (e.code === 'ENOENT') return null;
-    throw e;
-  }
-}
+const byCapturedDesc = (a, b) => (b.capturedAt || 0) - (a.capturedAt || 0);
 
-async function writeIndex(sessions) {
-  await fsp.mkdir(SNAPSHOT_DIR, { recursive: true });
-  await fsp.writeFile(INDEX_PATH, JSON.stringify({ sessions }, null, 2), 'utf8');
-}
+function createSnapshotStore(dir) {
+  const indexPath = path.join(dir, INDEX_FILE);
+  const snapshotPath = (id) => path.join(dir, `${sanitize(id)}.json`);
 
-async function cleanupIndex() {
-  const sessions = await readIndex();
-  const kept = [];
-  const removed = [];
-  for (const s of sessions) {
+  async function readIndex() {
     try {
-      await fsp.access(snapshotPath(s.id));
-      kept.push(s);
-    } catch {
-      removed.push(s.id);
+      const parsed = JSON.parse(await fsp.readFile(indexPath, 'utf8'));
+      return Array.isArray(parsed?.sessions) ? parsed.sessions : [];
+    } catch (e) {
+      if (e.code !== 'ENOENT') console.warn(`pi-inspect: read index: ${e.message}`);
+      return [];
     }
   }
-  if (removed.length) await writeIndex(kept);
-  return { kept: kept.length, removed };
-}
 
-async function keepOnly(sessionId) {
-  if (!sessionId) throw new Error('sessionId required');
-  const sessions = await readIndex();
-  const kept = sessions.filter((s) => s.id === sessionId);
-  const removed = [];
-  let files;
-  try { files = await fsp.readdir(SNAPSHOT_DIR); }
-  catch { files = []; }
-  const keepPath = path.basename(snapshotPath(sessionId));
-  for (const f of files) {
-    if (!f.endsWith('.json') || f === 'index.json' || f === keepPath) continue;
+  function writeIndex(sessions) {
+    return writeJsonAtomic(indexPath, { sessions });
+  }
+
+  async function listSessions() {
+    return (await readIndex()).sort(byCapturedDesc);
+  }
+
+  async function readSnapshot(id) {
     try {
-      await fsp.unlink(path.join(SNAPSHOT_DIR, f));
-      removed.push(f.replace(/\.json$/, ''));
+      return JSON.parse(await fsp.readFile(snapshotPath(id), 'utf8'));
     } catch (e) {
-      console.warn(`pi-inspect: unlink ${f}: ${e.message}`);
+      if (e.code === 'ENOENT') return null;
+      throw e;
     }
   }
-  await writeIndex(kept);
-  return { kept: kept.length, removed };
-}
 
-async function readLatestSnapshot() {
-  const sessions = await readIndex();
-  if (!sessions.length) return null;
-  const sorted = [...sessions].sort((a, b) => (b.capturedAt || 0) - (a.capturedAt || 0));
-  return readSnapshot(sorted[0].id);
+  async function readLatestSnapshot() {
+    const [latest] = await listSessions();
+    return latest ? readSnapshot(latest.id) : null;
+  }
+
+  async function keepOnly(id) {
+    if (!id) throw new Error('session id required');
+    const keepFile = path.basename(snapshotPath(id));
+    let files;
+    try { files = await fsp.readdir(dir); } catch { files = []; }
+    const removed = [];
+    for (const f of files) {
+      if (!f.endsWith('.json') || f === INDEX_FILE || f === keepFile) continue;
+      try {
+        await fsp.unlink(path.join(dir, f));
+        removed.push(f.slice(0, -'.json'.length));
+      } catch (e) {
+        console.warn(`pi-inspect: unlink ${f}: ${e.message}`);
+      }
+    }
+    const kept = (await readIndex()).filter((s) => s.id === id);
+    await writeIndex(kept);
+    return { kept: kept.length, removed };
+  }
+
+  return { dir, snapshotPath, writeIndex, listSessions, readSnapshot, readLatestSnapshot, keepOnly };
 }
 
-module.exports = {
-  snapshotDir,
-  snapshotPath,
-  readIndex,
-  writeIndex,
-  cleanupIndex,
-  keepOnly,
-  readSnapshot,
-  readLatestSnapshot,
-};
+module.exports = { createSnapshotStore, sanitize, writeJsonAtomic };

package/lib/watch.js

@@ -0,0 +1,29 @@
+'use strict';
+const fs = require('node:fs');
+
+// fs.watch fires several events per write, so changes are batched into one call per quiet period.
+function watchSnapshots(dir, onChange, { delayMs = 150 } = {}) {
+  fs.mkdirSync(dir, { recursive: true });
+  let timer = null;
+  const changed = new Set();
+  // libuv aborts when it watches a Windows 8.3 short path (C:\Users\NAME~1\...), so watch the long form.
+  const watcher = fs.watch(fs.realpathSync.native(dir), (_event, filename) => {
+    const name = filename && String(filename);
+    if (!name || !name.endsWith('.json')) return;
+    changed.add(name);
+    clearTimeout(timer);
+    timer = setTimeout(() => {
+      const files = [...changed];
+      changed.clear();
+      onChange(files);
+    }, delayMs);
+  });
+  return {
+    close() {
+      clearTimeout(timer);
+      watcher.close();
+    },
+  };
+}
+
+module.exports = { watchSnapshots };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-inspect",
-  "version": "0.4.0",
+  "version": "0.6.0",
   "description": "Introspection dashboard for the pi coding agent — tools, slash commands, skills, and the system prompt injected on init.",
   "keywords": [
     "pi-package",
@@ -29,22 +29,23 @@
     "lib/",
     "public/",
     "extensions/",
-    "themes/",
-    "README.md"
+    "README.md",
+    "CHANGELOG.md"
   ],
   "scripts": {
     "start": "node server.js",
     "dev": "node server.js --open",
+    "test": "node --test test/*.test.js",
     "typecheck:extensions": "tsc -p extensions/tsconfig.json --noEmit"
   },
   "dependencies": {
-    "chokidar": "^3.5.3",
-    "express": "^4.19.2",
-    "open": "^10.1.0"
+    "express": "^5.3.0",
+    "open": "^11.0.4"
   },
   "devDependencies": {
-    "@types/node": "^25.6.2",
-    "typescript": "^5.8.2"
+    "@earendil-works/pi-coding-agent": "^1.4.0",
+    "@types/node": "^25.9.9",
+    "typescript": "^5.9.3"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*"