pi-inspect 0.5.0 → 0.6.0

package/public/sw.js

@@ -1,8 +1,7 @@
-// pi-inspect service worker — network-first for dynamic data, cache-first for static shell.
-const VERSION = 'pi-inspect-v6';
-// Resolve relative to the SW scope so this works under any subpath (e.g. GitHub Pages).
+// Network first, so a new server version shows on the next load; the cache only serves the shell offline (GitHub Pages share links).
+const VERSION = 'pi-inspect-v8';
 const BASE = new URL('./', self.registration?.scope || self.location.href).pathname;
-const SHELL = ['', 'index.html', 'style.css', 'app.js', 'share.js', 'manifest.webmanifest', 'icon.svg'].map((p) => BASE + p);
+const SHELL = ['', 'index.html', 'style.css', 'app.js', 'share.js', 'fonts/fonts.css', 'manifest.webmanifest', 'icon.svg'].map((p) => BASE + p);
 
 self.addEventListener('install', (event) => {
   event.waitUntil(caches.open(VERSION).then((c) => c.addAll(SHELL)).catch(() => {}));
@@ -20,24 +19,17 @@ self.addEventListener('fetch', (event) => {
   const req = event.request;
   if (req.method !== 'GET') return;
   const url = new URL(req.url);
-  if (url.origin !== self.location.origin) return;
+  if (url.origin !== self.location.origin || url.pathname.startsWith('/api/')) return;
 
-  // Never cache API or SSE — always go to network.
-  if (url.pathname.startsWith('/api/')) return;
-
-  // Static shell: cache-first, fall back to network, then update cache.
   event.respondWith(
-    caches.match(req).then((cached) => {
-      const fetchPromise = fetch(req)
-        .then((res) => {
-          if (res && res.ok) {
-            const copy = res.clone();
-            caches.open(VERSION).then((c) => c.put(req, copy)).catch(() => {});
-          }
-          return res;
-        })
-        .catch(() => cached);
-      return cached || fetchPromise;
-    }),
+    fetch(req)
+      .then((res) => {
+        if (res.ok) {
+          const copy = res.clone();
+          caches.open(VERSION).then((c) => c.put(req, copy)).catch(() => {});
+        }
+        return res;
+      })
+      .catch(() => caches.match(req, { ignoreSearch: true }).then((hit) => hit || Response.error())),
   );
 });

package/server.js

@@ -1,206 +1,35 @@
 #!/usr/bin/env node
-const express = require('express');
+'use strict';
 const path = require('node:path');
-const fs = require('node:fs');
-const fsp = require('node:fs/promises');
-const os = require('node:os');
-const chokidar = require('chokidar');
-const open = require('open').default || require('open');
-const snapshots = require('./lib/snapshot');
-const githubSource = require('./lib/github-source');
-const pkg = require('./package.json');
-
-const PORT = Number(process.env.PORT) || 5462;
-const args = process.argv.slice(2);
-const shouldOpen = args.includes('--open');
-const openSession = (() => {
-  const i = args.indexOf('--session');
-  return i >= 0 ? args[i + 1] : null;
-})();
-
-const BUILTIN_THEME_DIR = path.join(__dirname, 'themes');
-const USER_THEME_DIR = process.env.INSPECT_THEME_DIR
-  || path.join(os.homedir(), '.pi', 'agent', 'inspect', 'themes');
-
-const app = express();
-app.disable('x-powered-by');
-app.use(express.json({ limit: '2mb' }));
-
-app.get('/api/version', (_req, res) => {
-  res.json({ name: pkg.name, version: pkg.version });
-});
-
-app.get('/api/sessions', async (_req, res) => {
-  try {
-    const sessions = await snapshots.readIndex();
-    const sorted = [...sessions].sort((a, b) => (b.capturedAt || 0) - (a.capturedAt || 0));
-    res.json({ sessions: sorted });
-  } catch (e) {
-    res.status(500).json({ error: e.message });
-  }
-});
-
-app.post('/api/sessions/cleanup', async (req, res) => {
-  try {
-    const keep = (req.body && req.body.keep) || req.query.keep || null;
-    const result = keep
-      ? await snapshots.keepOnly(String(keep))
-      : await snapshots.cleanupIndex();
-    res.json({ ok: true, ...result });
-  } catch (e) {
-    res.status(500).json({ ok: false, error: e.message });
-  }
-});
-
-const githubMemo = new Map(); // sessionId -> Record<root, {url, source}>
-
-function collectSourceRoots(snap) {
-  const roots = new Set();
-  const add = (x) => {
-    const baseDir = x?.sourceInfo?.baseDir;
-    if (typeof baseDir === 'string' && baseDir) roots.add(baseDir);
-  };
-  for (const t of snap.tools ?? []) add(t);
-  for (const c of snap.commands ?? []) add(c);
-  return [...roots];
+const { parseOptions } = require('./lib/args');
+const { inspectDir } = require('./lib/agent-dir');
+const { createApp, SESSION_PATH } = require('./lib/app');
+const { createSnapshotStore } = require('./lib/snapshot');
+const { watchSnapshots } = require('./lib/watch');
+
+let options;
+try {
+  options = parseOptions(process.argv.slice(2));
+} catch (e) {
+  console.error(`pi-inspect: ${e.message}`);
+  process.exit(1);
 }
 
-app.get('/api/introspect', async (req, res) => {
-  try {
-    const sid = req.query.session ? String(req.query.session) : null;
-    const snap = sid ? await snapshots.readSnapshot(sid) : await snapshots.readLatestSnapshot();
-    if (!snap) return res.status(404).json({ error: 'no snapshot found', sessionId: sid });
-    const memoKey = snap.sessionId;
-    let githubSources = memoKey ? githubMemo.get(memoKey) : null;
-    if (!githubSources) {
-      githubSources = await githubSource.resolveMany(collectSourceRoots(snap));
-      if (memoKey) githubMemo.set(memoKey, githubSources);
-    }
-    res.json({ ...snap, githubSources });
-  } catch (e) {
-    res.status(500).json({ error: e.message });
-  }
-});
+const INSPECT_DIR = inspectDir();
+const store = createSnapshotStore(path.join(INSPECT_DIR, 'snapshots'));
+const { app, broadcast } = createApp({ store, inspectDir: INSPECT_DIR, host: options.host });
 
-const themes = new Map();
-async function loadThemes() {
-  for (const dir of [BUILTIN_THEME_DIR, USER_THEME_DIR]) {
-    let entries;
-    try { entries = await fsp.readdir(dir); }
-    catch (e) {
-      if (e.code !== 'ENOENT') console.warn(`themes: cannot read ${dir}: ${e.message}`);
-      continue;
-    }
-    for (const f of entries.filter((n) => n.toLowerCase().endsWith('.json'))) {
-      try {
-        const raw = await fsp.readFile(path.join(dir, f), 'utf8');
-        const obj = JSON.parse(raw);
-        const id = f.replace(/\.json$/i, '');
-        themes.set(id, obj);
-      } catch (e) {
-        console.warn(`themes: skip ${f}: ${e.message}`);
-      }
-    }
-  }
-}
+watchSnapshots(store.dir, (files) => broadcast('snapshot', { files }));
 
-app.get('/api/themes', (_req, res) => {
-  res.json({ themes: [...themes.values()] });
-});
-
-app.post('/api/open', (req, res) => {
-  const target = (req.body && req.body.path) || '';
-  if (!target || typeof target !== 'string') return res.status(400).json({ ok: false, error: 'missing path' });
-  const normalized = path.normalize(target);
-  if (!fs.existsSync(normalized)) return res.status(404).json({ ok: false, error: `path not found: ${normalized}` });
-  const editor = process.env.EDITOR || process.env.VISUAL || (process.platform === 'win32' ? 'code' : 'vi');
-  try {
-    const { spawn } = require('node:child_process');
-    const isWin = process.platform === 'win32';
-    const parts = editor.split(/\s+/);
-    const cmd = parts.shift();
-    const args = [...parts, normalized];
-    const quotedArgs = isWin ? args.map((a) => `"${String(a).replace(/"/g, '\\"')}"`) : args;
-    const child = spawn(cmd, quotedArgs, {
-      detached: true,
-      stdio: 'ignore',
-      shell: isWin,
-      windowsVerbatimArguments: isWin,
-    });
-    child.on('error', (err) => console.warn(`open failed: ${err.message}`));
-    child.unref();
-    res.json({ ok: true, editor, path: normalized });
-  } catch (e) {
-    res.status(500).json({ ok: false, error: e.message });
-  }
-});
-
-const REQ_DIR = path.join(os.homedir(), '.pi', 'agent', 'inspect', 'requests');
-
-app.post('/api/toggle', async (req, res) => {
-  try {
-    const { action, resourceKind, path: target, scope } = req.body || {};
-    if (action !== 'enable' && action !== 'disable') return res.status(400).json({ ok: false, error: 'action must be enable|disable' });
-    if (!['skills', 'prompts', 'extensions', 'themes'].includes(resourceKind)) return res.status(400).json({ ok: false, error: 'invalid resourceKind' });
-    if (!target || typeof target !== 'string') return res.status(400).json({ ok: false, error: 'missing path' });
-    const useScope = scope === 'project' ? 'project' : 'user';
-    await fsp.mkdir(REQ_DIR, { recursive: true });
-    const id = `${Date.now()}-${Math.random().toString(36).slice(2, 10)}`;
-    const file = path.join(REQ_DIR, `${id}.json`);
-    const tmp = `${file}.tmp`;
-    const body = JSON.stringify({ id, ts: Date.now(), action, resourceKind, path: target, scope: useScope });
-    await fsp.writeFile(tmp, body, 'utf8');
-    await fsp.rename(tmp, file);
-    res.json({ ok: true, id });
-  } catch (e) {
-    res.status(500).json({ ok: false, error: e.message });
+const server = app.listen(options.port, options.host, async (err) => {
+  if (err) {
+    console.error(`pi-inspect: cannot listen on port ${options.port}: ${err.message}`);
+    process.exit(1);
   }
+  const base = `http://localhost:${server.address().port}`;
+  console.log(`pi-inspect listening on ${base}`);
+  if (!options.open) return;
+  const url = options.session ? base + SESSION_PATH.replace('{id}', encodeURIComponent(options.session)) : base;
+  const { default: open } = await import('open');
+  open(url).catch(() => {});
 });
-
-app.post('/api/focus', (req, res) => {
-  const sid = (req.body && req.body.session) || req.query.session || null;
-  const count = sseClients.size;
-  if (count > 0) broadcast('navigate', { session: sid });
-  res.json({ delivered: count });
-});
-
-// SSE
-const sseClients = new Set();
-app.get('/api/events', (req, res) => {
-  res.set({
-    'Content-Type': 'text/event-stream',
-    'Cache-Control': 'no-cache',
-    Connection: 'keep-alive',
-  });
-  res.flushHeaders();
-  res.write(': hello\n\n');
-  sseClients.add(res);
-  req.on('close', () => sseClients.delete(res));
-});
-
-function broadcast(event, data) {
-  const payload = `event: ${event}\ndata: ${JSON.stringify(data)}\n\n`;
-  for (const c of sseClients) {
-    try { c.write(payload); } catch {}
-  }
-}
-
-const SNAP_DIR = snapshots.snapshotDir();
-fs.mkdirSync(SNAP_DIR, { recursive: true });
-chokidar
-  .watch(SNAP_DIR, { ignoreInitial: true, depth: 1 })
-  .on('all', (kind, file) => {
-    if (!file.endsWith('.json')) return;
-    broadcast('snapshot', { kind, file: path.basename(file) });
-  });
-
-app.use(express.static(path.join(__dirname, 'public')));
-
-(async () => {
-  await loadThemes();
-  app.listen(PORT, () => {
-    const url = `http://localhost:${PORT}${openSession ? `/?session=${encodeURIComponent(openSession)}` : ''}`;
-    console.log(`pi-inspect listening on http://localhost:${PORT}`);
-    if (shouldOpen) open(url).catch(() => {});
-  });
-})();

package/themes/pi-dark.json

@@ -1,21 +0,0 @@
-{
-  "name": "pi-dark",
-  "displayName": "pi.dev Dark",
-  "mode": "dark",
-  "colors": {
-    "bgDeep": "#1a1a1a",
-    "bgSurface": "#212121",
-    "bgElevated": "#2a2a2a",
-    "bgHover": "#333333",
-    "border": "#3a3a3a",
-    "textPrimary": "#f0ebe6",
-    "textSecondary": "#c9c2bb",
-    "textTertiary": "#8b857e",
-    "textMuted": "#5f5a55",
-    "accent": "#e8927c",
-    "accentText": "#f5b8a8",
-    "tool": "#6ea8d8",
-    "command": "#a78bfa",
-    "injected": "#f0b14d"
-  }
-}

package/themes/pi-light.json

@@ -1,21 +0,0 @@
-{
-  "name": "pi-light",
-  "displayName": "pi.dev Light",
-  "mode": "light",
-  "colors": {
-    "bgDeep": "#ebe7e4",
-    "bgSurface": "#f4f1ee",
-    "bgElevated": "#e0dcd8",
-    "bgHover": "#d5d1cd",
-    "border": "#b5b0aa",
-    "textPrimary": "#0d1116",
-    "textSecondary": "#212730",
-    "textTertiary": "#495059",
-    "textMuted": "#757d89",
-    "accent": "#2f5f8a",
-    "accentText": "#1d4870",
-    "tool": "#2f5f8a",
-    "command": "#7c3aed",
-    "injected": "#b45309"
-  }
-}