pi-kanban 0.3.1 → 1.0.0

package/README.md

@@ -24,7 +24,7 @@ When you're ready to run it on your own pi sessions, install below.
 pi install npm:pi-kanban
 ```
 
-Then use `/kanban start | stop | restart | status | open | web | app | pin | sticky-pin | unpin | preview | link` from inside pi.
+Then use `/kanban start | stop | restart | status | open web|app` for global controls, and `/kanban session open | pin | sticky-pin | unpin | view-doc | link-doc` for session-scoped actions.
 
 Or run standalone:
 

package/extensions/kanban.ts

@@ -6,9 +6,10 @@ import { homedir } from "node:os";
 import { isAbsolute, join as joinPath, resolve as resolvePath } from "node:path";
 import { fileURLToPath } from "node:url";
 import { Type } from "typebox";
-import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+
+const extDir = fileURLToPath(new URL('.', import.meta.url));
 
-const extDir = fileURLToPath(new URL(".", import.meta.url));
 const taskStore = createRequire(import.meta.url)(
 	resolvePath(extDir, "..", "lib", "task-store.js"),
 ) as {
@@ -61,15 +62,19 @@ const SUBCOMMANDS = [
 	"restart",
 	"status",
 	"open",
-	"web",
-	"app",
+	"session",
+] as const;
+type Sub = (typeof SUBCOMMANDS)[number];
+
+const SESSION_SUBCOMMANDS = [
+	"open",
 	"pin",
 	"sticky-pin",
 	"unpin",
-	"preview",
-	"link",
+	"view-doc",
+	"link-doc",
 ] as const;
-type Sub = (typeof SUBCOMMANDS)[number];
+type SessionSub = (typeof SESSION_SUBCOMMANDS)[number];
 
 function probePort(p: number, timeoutMs = 250): Promise<boolean> {
 	return new Promise((resolve) => {
@@ -231,7 +236,7 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 			const cwd = ctx?.sessionManager?.getCwd?.() ?? process.cwd();
 			const inputPath = String(params.path ?? "").trim();
 			if (!inputPath) {
-				return { content: [{ type: "text", text: "Error: path is required." }], isError: true };
+				return { content: [{ type: "text", text: "Error: path is required." }], isError: true, details: { path: inputPath, title: null, sessionId } };
 			}
 			const absPath = isAbsolute(inputPath) ? inputPath : resolvePath(cwd, inputPath);
 
@@ -239,7 +244,7 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 			try {
 				const stat = statSync(absPath);
 				if (!stat.isFile()) {
-					return { content: [{ type: "text", text: `Error: ${absPath} is not a file.` }], isError: true };
+					return { content: [{ type: "text", text: `Error: ${absPath} is not a file.` }], isError: true, details: { path: absPath, title: null, sessionId } };
 				}
 				const buf = readFileSync(absPath, { encoding: "utf8" });
 				const head = buf.slice(0, PLAN_MAX_BYTES);
@@ -248,6 +253,7 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 				return {
 					content: [{ type: "text", text: `Error reading ${absPath}: ${e?.message ?? e}` }],
 					isError: true,
+					details: { path: absPath, title: null, sessionId },
 				};
 			}
 
@@ -255,6 +261,7 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 				return {
 					content: [{ type: "text", text: "Error: no active session id; cannot bind plan." }],
 					isError: true,
+					details: { path: absPath, title: title ?? null, sessionId },
 				};
 			}
 			try {
@@ -262,12 +269,12 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 				if (!res.ok) {
 					const msg = `pi-kanban server rejected bind_plan: HTTP ${res.status}. Is /kanban running?`;
 					console.warn(msg);
-					return { content: [{ type: "text", text: msg }], isError: true };
+					return { content: [{ type: "text", text: msg }], isError: true, details: { path: absPath, title, sessionId } };
 				}
 			} catch (e: any) {
 				const msg = `pi-kanban: bind_plan POST failed (server may be stopped): ${e?.message ?? e}`;
 				console.warn(msg);
-				return { content: [{ type: "text", text: msg }], isError: true };
+				return { content: [{ type: "text", text: msg }], isError: true, details: { path: absPath, title: title ?? null, sessionId } };
 			}
 
 			const summary = title ? `Bound plan "${title}" → ${absPath}` : `Bound plan → ${absPath}`;
@@ -282,7 +289,7 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 
 	async function startServer(notify: Notify, opts: { silentSuccess?: boolean } = {}): Promise<boolean> {
 		if (await probePort(port)) {
-			notify(`pi-kanban already listening on ${url} — run /kanban open or /kanban web`);
+			notify(`pi-kanban already listening on ${url} — run /kanban open web to launch it`);
 			return true;
 		}
 		lastStderr = "";
@@ -312,7 +319,7 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 		child.stderr?.resume();
 		child.unref();
 		if (!opts.silentSuccess) {
-			notify(`pi-kanban started → ${url} — run /kanban open or /kanban web to launch it`);
+			notify(`pi-kanban started → ${url} — run /kanban open web to launch it`);
 		}
 		return true;
 	}
@@ -344,15 +351,46 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 
 	pi.registerCommand("kanban", {
 		description:
-			"pi-kanban dashboard: start | stop | restart | status | open | web | app | pin | sticky-pin | unpin | preview | link",
-		getArgumentCompletions: (prefix) =>
-			SUBCOMMANDS.filter((s) => s.startsWith(prefix)).map((s) => ({ value: s, label: s })),
+			"pi-kanban dashboard: start | stop | restart | status | open web|app | session <verb>",
+		getArgumentCompletions: (prefix) => {
+			const tokens = splitArgs(prefix);
+			if (tokens.length >= 1 && tokens[0] === "session") {
+				const verbPrefix = tokens[1] ?? "";
+				return SESSION_SUBCOMMANDS
+					.filter((s) => s.startsWith(verbPrefix))
+					.map((s) => ({ value: `session ${s}`, label: s }));
+			}
+			return SUBCOMMANDS.filter((s) => s.startsWith(prefix)).map((s) => ({ value: s, label: s }));
+		},
 		handler: async (args, ctx) => {
 			const tokens = splitArgs(args);
 			const sub = (tokens[0] || "start") as Sub;
 			const rest = tokens.slice(1);
 			const notify = (m: string, l: "info" | "error" = "info") => ctx.ui.notify(m, l);
 
+			function showHelp() {
+				const lines = [
+					"Usage: /kanban <command>",
+					"",
+					"Global commands:",
+					"  start            Start the dashboard server",
+					"  stop             Stop the dashboard server",
+					"  restart          Restart the dashboard server",
+					"  status           Show server status",
+					"  open web          Open dashboard in browser",
+					"  open app          Open dashboard as PWA",
+					"",
+					"Session commands (default to current session):",
+					"  session open [<id>]",
+					"  session pin [<id>]",
+					"  session sticky-pin [<id>]",
+					"  session unpin [<id>]",
+					"  session view-doc <file> [<id>]",
+					"  session link-doc <file> [<id>]",
+				];
+				notify(lines.join("\n"));
+			}
+
 			if (sub === "start") {
 				await startServer(notify);
 				return;
@@ -378,88 +416,104 @@ export default function kanbanExtension(pi: ExtensionAPI) {
 			}
 
 			if (sub === "open") {
-				if (!(await ensureRunning(notify))) return;
-				const id = rest[0] ?? ctx.sessionManager.getSessionId() ?? null;
-				if (!id) {
-					notify("Usage: /kanban open <session-id> (no current session to default to)", "error");
+				const verb = rest[0];
+				if (verb === "web") {
+					const { default: open } = await import("open");
+					await open(url);
 					return;
 				}
-				const res = await postSessionOpen(id);
-				if (!res.ok) {
-					notify(`open failed (${res.status}): ${await res.text()}`, "error");
+				if (verb === "app") {
+					const { default: open, apps } = await import("open");
+					for (const name of [apps.chrome, apps.edge, apps.browser]) {
+						try {
+							await open(url, { app: { name, arguments: [`--app=${url}`] } });
+							return;
+						} catch {}
+					}
+					notify("Could not find Chrome/Edge for PWA window mode", "error");
 					return;
 				}
-				const usedCurrent = !rest[0];
-				notify(`opened: ${id}${usedCurrent ? " (current)" : ""}`);
+				notify(`Usage: /kanban open web|app${verb ? ` (unknown: ${verb})` : ""}`, "error");
 				return;
 			}
 
-			if (sub === "web") {
-				const { default: open } = await import("open");
-				await open(url);
-				return;
-			}
-
-			if (sub === "app") {
-				const { default: open, apps } = await import("open");
-				for (const name of [apps.chrome, apps.edge, apps.browser]) {
-					try {
-						await open(url, { app: { name, arguments: [`--app=${url}`] } });
-						return;
-					} catch {}
-				}
-				notify("Could not find Chrome/Edge for PWA window mode", "error");
-				return;
-			}
+			if (sub === "session") {
+				const verb = rest[0] as SessionSub | undefined;
+				const verbRest = rest.slice(1);
 
-			if (sub === "pin" || sub === "sticky-pin" || sub === "unpin") {
-				if (!(await ensureRunning(notify))) return;
-				const id = rest[0] ?? ctx.sessionManager.getSessionId() ?? null;
-				if (!id) {
-					notify(`Usage: /kanban ${sub} <session-id> (no current session to default to)`, "error");
+				if (!verb) {
+					showHelp();
 					return;
 				}
-				const PIN_STATE = { "sticky-pin": "sticky", "unpin": "none", "pin": "pinned" } as const;
-				const state = PIN_STATE[sub];
-				const res = await postPin(id, state);
-				if (!res.ok) {
-					notify(`${sub} failed (${res.status}): ${await res.text()}`, "error");
+
+				if (!(SESSION_SUBCOMMANDS as readonly string[]).includes(verb)) {
+					notify(`Unknown session verb: ${verb}. Try: ${SESSION_SUBCOMMANDS.join(", ")}`, "error");
 					return;
 				}
-				const usedCurrent = !rest[0];
-				const verb = sub === "unpin" ? "unpinned" : state;
-				notify(`${verb}: ${id}${usedCurrent ? " (current)" : ""}`);
-				return;
-			}
 
-			if (sub === "preview" || sub === "link") {
 				if (!(await ensureRunning(notify))) return;
-				const file = rest[0];
-				if (!file) {
-					notify(`Usage: /kanban ${sub} <file.md> [session-id]`, "error");
+
+				if (verb === "open") {
+					const id = verbRest[0] ?? ctx.sessionManager.getSessionId() ?? null;
+					if (!id) {
+						notify("Usage: /kanban session open <session-id> (no current session to default to)", "error");
+						return;
+					}
+					const res = await postSessionOpen(id);
+					if (!res.ok) {
+						notify(`open failed (${res.status}): ${await res.text()}`, "error");
+						return;
+					}
+					notify(`opened: ${id}${!verbRest[0] ? " (current)" : ""}`);
 					return;
 				}
-				const sessionId = rest[1] ?? ctx.sessionManager.getSessionId() ?? null;
-				if (sub === "link" && !sessionId) {
-					notify("Usage: /kanban link <file.md> <session-id> (no current session to default to)", "error");
+
+				if (verb === "pin" || verb === "sticky-pin" || verb === "unpin") {
+					const id = verbRest[0] ?? ctx.sessionManager.getSessionId() ?? null;
+					if (!id) {
+						notify(`Usage: /kanban session ${verb} <session-id> (no current session to default to)`, "error");
+						return;
+					}
+					const PIN_STATE = { "sticky-pin": "sticky", "unpin": "none", "pin": "pinned" } as const;
+					const state = PIN_STATE[verb];
+					const res = await postPin(id, state);
+					if (!res.ok) {
+						notify(`${verb} failed (${res.status}): ${await res.text()}`, "error");
+						return;
+					}
+					const pinVerb = verb === "unpin" ? "unpinned" : state;
+					notify(`${pinVerb}: ${id}${!verbRest[0] ? " (current)" : ""}`);
 					return;
 				}
-				const expanded =
-					file === "~" || file.startsWith("~/") || file.startsWith("~\\")
-						? joinPath(homedir(), file.slice(1))
-						: file;
-				const abs = isAbsolute(expanded) ? expanded : resolvePath(process.cwd(), expanded);
-				const res = await postPreview(abs, sessionId, sub === "link");
-				if (!res.ok) {
-					notify(`${sub} failed (${res.status}): ${await res.text()}`, "error");
+
+				if (verb === "view-doc" || verb === "link-doc") {
+					const file = verbRest[0];
+					if (!file) {
+						notify(`Usage: /kanban session ${verb} <file.md> [session-id]`, "error");
+						return;
+					}
+					const sessionId = verbRest[1] ?? ctx.sessionManager.getSessionId() ?? null;
+					if (verb === "link-doc" && !sessionId) {
+						notify("Usage: /kanban session link-doc <file.md> <session-id> (no current session to default to)", "error");
+						return;
+					}
+					const expanded =
+						file === "~" || file.startsWith("~/") || file.startsWith("~\\")
+							? joinPath(homedir(), file.slice(1))
+							: file;
+					const abs = isAbsolute(expanded) ? expanded : resolvePath(process.cwd(), expanded);
+					const res = await postPreview(abs, sessionId, verb === "link-doc");
+					if (!res.ok) {
+						notify(`${verb} failed (${res.status}): ${await res.text()}`, "error");
+						return;
+					}
+					const usedCurrent = !verbRest[1] && sessionId;
+					notify(`${verb}: ${abs}${sessionId ? ` → session ${sessionId}${usedCurrent ? " (current)" : ""}` : ""}`);
 					return;
 				}
-				const usedCurrent = !rest[1] && sessionId;
-				notify(`${sub}: ${abs}${sessionId ? ` → session ${sessionId}${usedCurrent ? " (current)" : ""}` : ""}`);
-				return;
 			}
 
-			notify(`Unknown subcommand: ${sub}. Try: ${SUBCOMMANDS.join(", ")}`, "error");
+			showHelp();
 		},
 	});
 }

package/extensions/package.json

@@ -0,0 +1 @@
+{ "type": "module" }

package/extensions/tsconfig.json

@@ -0,0 +1,16 @@
+{
+  "compilerOptions": {
+    "target": "ES2024",
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "esModuleInterop": true,
+    "skipLibCheck": true,
+    "strict": true,
+    "forceConsistentCasingInFileNames": true,
+    "noEmit": true,
+    "resolveJsonModule": true,
+    "lib": ["es2024"],
+    "types": ["node"]
+  },
+  "include": ["./**/*.ts"]
+}

package/lib/pi-parsers.js

@@ -9,6 +9,13 @@ const taskStore = require('./task-store');
 let onBranchResolved = null;
 function setOnBranchResolved(fn) { onBranchResolved = typeof fn === 'function' ? fn : null; }
 
+// Detect local absolute image paths in user message text — Windows and Unix.
+// Lookbehinds reject URL contexts: `(?<![a-zA-Z])` blocks scheme suffix (`s` in `https:`);
+// `(?!\/)` blocks `://`; `(?<![:\w\\/])` blocks `/path` after letter/colon/slash.
+const LOCAL_IMAGE_PATH_RE = /((?<![a-zA-Z])[A-Za-z]:[\\/](?!\/)[^\s"'<>]+?\.(png|jpe?g|gif|webp|bmp)|(?<![:\w\\/])\/[^\s"'<>]+?\.(png|jpe?g|gif|webp|bmp))/i;
+const MEDIA_TYPES_BY_EXT = { '.png': 'image/png', '.jpg': 'image/jpeg', '.jpeg': 'image/jpeg', '.gif': 'image/gif', '.webp': 'image/webp', '.bmp': 'image/bmp' };
+function guessMediaTypeFromPath(p) { return MEDIA_TYPES_BY_EXT[path.extname(p).toLowerCase()] || 'application/octet-stream'; }
+
 const PI_DIR = process.env.PI_DIR || path.join(os.homedir(), '.pi');
 const SESSIONS_DIR = path.join(PI_DIR, 'agent', 'sessions');
 
@@ -74,6 +81,40 @@ function parseLine(line) {
   try { return JSON.parse(line); } catch { return null; }
 }
 
+// Stream the first `maxLines` lines and extract session-header fields used by
+// subagent matching. Avoids reading whole .jsonl files when only the header is needed.
+async function readSessionHeader(file, maxLines = 10) {
+  return new Promise((resolve) => {
+    const out = { firstTs: null, modelId: null, firstUserText: null, parentSession: null };
+    let count = 0;
+    let done = false;
+    const stream = fs.createReadStream(file, { encoding: 'utf8' });
+    const rl = readline.createInterface({ input: stream, crlfDelay: Infinity });
+    const finish = () => { if (done) return; done = true; rl.close(); stream.destroy(); resolve(out); };
+    rl.on('line', (line) => {
+      if (done) return;
+      if (line.trim()) {
+        const ev = parseLine(line);
+        if (ev) {
+          if (ev.type === 'session') {
+            if (!out.firstTs) out.firstTs = ev.timestamp || null;
+            if (ev.parentSession) out.parentSession = ev.parentSession;
+          }
+          if (!out.modelId && ev.type === 'model_change') out.modelId = ev.modelId || null;
+          if (!out.firstUserText && ev.type === 'message' && ev.message?.role === 'user') {
+            const c = ev.message.content;
+            const txt = Array.isArray(c) ? c.find((x) => x && x.type === 'text')?.text : null;
+            if (txt) out.firstUserText = txt;
+          }
+        }
+      }
+      if (++count >= maxLines) finish();
+    });
+    rl.on('close', finish);
+    rl.on('error', finish);
+  });
+}
+
 async function readSessionEntriesUncached(file) {
   const lines = await readJsonlLines(file);
   const entries = [];
@@ -305,17 +346,33 @@ function summarize(entries) {
             target.progressSummary = r.progressSummary || null;
             target.sessionFile = r.sessionFile || null;
             target.artifactPaths = r.artifactPaths || null;
+            if (r.model) target.model = r.model;
             target.stoppedAt = e.timestamp || null;
             if (r.exitCode != null && r.exitCode !== 0 && !r.error && (r.finalOutput == null || r.finalOutput === '')) {
               target._cancelled = true;
             }
           });
           if (results && results.length === 0 && pendingArr && pendingArr.length) {
-            const text = Array.isArray(m.content) ? (m.content.find((c) => c && c.type === 'text')?.text || '') : '';
-            const cancelled = /cancel/i.test(text);
-            for (const rec of pendingArr) {
-              rec.stoppedAt = e.timestamp || null;
-              if (cancelled) rec._error = rec._error || 'Chain cancelled';
+            if (m.details && m.details.asyncId) {
+              // Async launch: empty results are expected — completion arrives via subagent-notify.
+              // Mark recs so the orphan check and enrichment handle them correctly.
+              for (const rec of pendingArr) {
+                rec._async = true;
+                rec._asyncDir = m.details.asyncDir || null;
+              }
+            } else {
+              // Sync empty results: failure or chain cancellation.
+              const text = Array.isArray(m.content) ? (m.content.find((c) => c && c.type === 'text')?.text || '') : '';
+              const cancelled = /cancel/i.test(text);
+              for (const rec of pendingArr) {
+                rec.stoppedAt = e.timestamp || null;
+                if (cancelled) rec._error = rec._error || 'Chain cancelled';
+                else {
+                  // Spawn failed (e.g. unknown agent): record error and mark so UI can hide it.
+                  rec._error = rec._error || (text ? text.split('\n')[0].slice(0, 200) : 'Spawn failed');
+                  rec._spawnFailed = true;
+                }
+              }
             }
           }
         }
@@ -493,6 +550,63 @@ async function enrichPendingSubagents(meta, subagents) {
   pending = subagents.filter((r) => !r.runId && !r.stoppedAt);
   if (pending.length === 0) return;
   await matchLiveSubagentRuns(meta, pending, subagents);
+
+  // For active subagents still missing model info, scan sibling session files that
+  // were spawned from this parent session (identified by parentSession containing the
+  // parent's UUID). The child session's model_change event appears early (line ~2) and
+  // is available while the subagent is still running.
+  await findModelFromSiblingSessions(meta, subagents);
+}
+
+// Reads sibling .jsonl files in the same sessions directory, finds ones whose
+// `session` entry's `parentSession` field contains the parent session UUID, then
+// assigns the modelId from their early model_change event to matching active subagents.
+async function findModelFromSiblingSessions(meta, subagents) {
+  const needsModel = subagents.filter((r) => !r.model && !r.stoppedAt);
+  if (needsModel.length === 0) return;
+
+  const sessionsDir = path.dirname(meta.file);
+  const parentSessionId = slugFromFile(meta.file);
+  if (!parentSessionId) return;
+
+  let files;
+  try { files = await fs.promises.readdir(sessionsDir); } catch { return; }
+
+  const candidates = [];
+  for (const f of files) {
+    if (!f.endsWith('.jsonl')) continue;
+    if (f === path.basename(meta.file)) continue;
+    const fpath = path.join(sessionsDir, f);
+    try {
+      const h = await readSessionHeader(fpath);
+      // parentSession is path-separator-agnostic; UUID substring match works on Windows/Linux.
+      if (h.parentSession && h.parentSession.includes(parentSessionId)) {
+        candidates.push({ fpath, modelId: h.modelId, firstTs: h.firstTs, firstUserText: h.firstUserText });
+      }
+    } catch {}
+  }
+
+  if (candidates.length === 0) return;
+
+  const used = new Set();
+  for (const rec of needsModel) {
+    const startMs = rec.startedAt ? new Date(rec.startedAt).getTime() : 0;
+    const idx = candidates.findIndex((c, i) => {
+      if (used.has(i)) return false;
+      const cMs = c.firstTs ? new Date(c.firstTs).getTime() : 0;
+      // Child session must start within a reasonable window of the subagent spawn time
+      if (startMs && cMs && (cMs < startMs - 2000 || cMs > startMs + 60000)) return false;
+      // For fork-mode subagents the first user message in the child session is the
+      // inherited parent prompt, not the task text — so don't reject on mismatch.
+      // Timestamp proximity + parentSession UUID match is sufficient.
+      return true;
+    });
+    if (idx < 0) continue;
+    used.add(idx);
+    const c = candidates[idx];
+    if (c.modelId) rec.model = c.modelId;
+    if (!rec.sessionFile) rec.sessionFile = c.fpath;
+  }
 }
 
 async function matchLiveSubagentRuns(meta, pending, allSubagents) {
@@ -514,21 +628,8 @@ async function matchLiveSubagentRuns(meta, pending, allSubagents) {
       const jsonl = path.join(runDirPath, r, 'session.jsonl');
       try {
         const stat = await fs.promises.stat(jsonl);
-        const raw = await fs.promises.readFile(jsonl, 'utf8');
-        const lines = raw.split('\n').filter(Boolean).slice(0, 10);
-        let firstTs = null;
-        let firstUserText = null;
-        for (const line of lines) {
-          try {
-            const ev = JSON.parse(line);
-            if (ev.type === 'session' && !firstTs) firstTs = ev.timestamp;
-            if (!firstUserText && ev.type === 'message' && ev.message && ev.message.role === 'user' && Array.isArray(ev.message.content)) {
-              const txt = ev.message.content.find((c) => c && c.type === 'text');
-              if (txt && typeof txt.text === 'string') firstUserText = txt.text;
-            }
-          } catch {}
-        }
-        liveRuns.push({ runId: d.name, runIndex: parseInt(m[1], 10), jsonl, firstTs, firstUserText, mtime: stat.mtimeMs });
+        const h = await readSessionHeader(jsonl);
+        liveRuns.push({ runId: d.name, runIndex: parseInt(m[1], 10), jsonl, firstTs: h.firstTs, firstUserText: h.firstUserText, modelId: h.modelId, mtime: stat.mtimeMs });
       } catch {}
     }
   }
@@ -552,6 +653,17 @@ async function matchLiveSubagentRuns(meta, pending, allSubagents) {
     rec.runId = lr.runId;
     rec.runIndex = lr.runIndex;
     rec.sessionFile = lr.jsonl;
+    if (lr.modelId && !rec.model) rec.model = lr.modelId;
+    // For async subagents, check status.json to detect completion and set stoppedAt.
+    if (rec._async && rec._asyncDir && !rec.stoppedAt) {
+      try {
+        const statusRaw = await fs.promises.readFile(path.join(rec._asyncDir, 'status.json'), 'utf8');
+        const statusData = JSON.parse(statusRaw);
+        if (statusData.state === 'complete' && statusData.lastActivityAt) {
+          rec.stoppedAt = new Date(statusData.lastActivityAt).toISOString();
+        }
+      } catch {}
+    }
   }
 
   // Structural staleness: while a subagent is running, the parent assistant is blocked
@@ -562,6 +674,7 @@ async function matchLiveSubagentRuns(meta, pending, allSubagents) {
   const parentEntries = (await getCachedSession(meta.file)).entries;
   for (const rec of pending) {
     if (rec.stoppedAt) continue;
+    if (rec._async) continue; // async subagents: parent continues executing, not orphaned
     if (!rec.startedAt) continue;
     const startMs = new Date(rec.startedAt).getTime();
     if (!startMs) continue;
@@ -587,6 +700,7 @@ async function listAgentsForSession(meta, summary = null) {
   await enrichPendingSubagents(meta, recs);
   const out = [];
   for (const rec of recs) {
+    if (rec._spawnFailed) continue; // never started — hide from agent log
     let lm = await readSubagentOutput(rec);
     if ((!lm || !lm.trim()) && rec._error) lm = `**Error:**\n\n\`\`\`\n${rec._error}\n\`\`\``;
     out.push(subagentToApi(rec, lm));
@@ -906,7 +1020,20 @@ async function buildMessages(file, limit = 50, before = null) {
 
     if (m.role === 'user') {
       const text = flattenContentToText(m.content);
-      messages.push({ type: 'user', role: 'user', text, timestamp: ts });
+      const images = [];
+      if (Array.isArray(m.content)) {
+        m.content.forEach((block, idx) => {
+          if (block && block.type === 'text' && typeof block.text === 'string') {
+            const match = block.text.match(LOCAL_IMAGE_PATH_RE);
+            if (match) images.push({ blockIndex: idx, mediaType: guessMediaTypeFromPath(match[1]), filePath: match[1] });
+          } else if (block && block.type === 'image' && block.source?.type === 'base64') {
+            images.push({ blockIndex: idx, mediaType: block.source.media_type || 'image/png', filePath: null });
+          }
+        });
+      }
+      const entry = { type: 'user', role: 'user', text, timestamp: ts };
+      if (images.length) { entry.images = images; entry.id = e.id; }
+      messages.push(entry);
     } else if (m.role === 'assistant') {
       const text = flattenContentToText(m.content);
       if (text && text.trim()) {
@@ -980,6 +1107,33 @@ async function buildMessages(file, limit = 50, before = null) {
   return { messages: sliced, hasMore: sliced.length < total, sessionId: null };
 }
 
+async function readUserImage(file, msgId, blockIndex) {
+  if (!file || !msgId) return null;
+  const idx = Number(blockIndex);
+  if (!Number.isInteger(idx) || idx < 0) return null;
+  const { entries } = await getCachedSession(file);
+  for (const e of entries) {
+    if (e.type !== 'message' || e.id !== msgId) continue;
+    const m = e.message;
+    if (!Array.isArray(m?.content)) return null;
+    const block = m.content[idx];
+    if (!block) return null;
+    if (block.type === 'image' && block.source?.type === 'base64')
+      return { mediaType: block.source.media_type || 'image/png', data: block.source.data };
+    if (block.type === 'text' && typeof block.text === 'string') {
+      const match = block.text.match(LOCAL_IMAGE_PATH_RE);
+      if (match) {
+        try {
+          const buf = await fs.promises.readFile(match[1]);
+          return { mediaType: guessMediaTypeFromPath(match[1]), data: buf.toString('base64') };
+        } catch (_) {}
+      }
+    }
+    return null;
+  }
+  return null;
+}
+
 module.exports = {
   getPiDir,
   getSessionsDir,
@@ -997,4 +1151,5 @@ module.exports = {
   setOnBranchResolved,
   flattenContentToText,
   invalidateSessionCache,
+  readUserImage,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-kanban",
-  "version": "0.3.1",
+  "version": "1.0.0",
   "description": "Workspace for the pi coding agent — sessions, todos, subagents, and more.",
   "main": "server.js",
   "bin": {
@@ -14,9 +14,12 @@
     "docs:dev": "vitepress dev docs",
     "docs:build": "vitepress build docs",
     "docs:preview": "vitepress preview docs",
-    "release:local": "npm version \"$(git describe --tags --exact-match)\" --no-git-tag-version --allow-same-version && npm publish --access public && git checkout -- package.json package-lock.json"
+    "release:local": "npm version \"$(git describe --tags --exact-match)\" --no-git-tag-version --allow-same-version && npm publish --access public && git checkout -- package.json package-lock.json",
+    "typecheck:extensions": "tsc -p extensions/tsconfig.json --noEmit"
   },
   "devDependencies": {
+    "@types/node": "^25.6.2",
+    "typescript": "^5.8.2",
     "vitepress": "^1.6.4"
   },
   "keywords": [
@@ -48,10 +51,11 @@
   "dependencies": {
     "chokidar": "^3.5.3",
     "express": "^4.18.2",
-    "open": "^11.0.0"
+    "open": "^11.0.0",
+    "typebox": "^1.1.38"
   },
   "peerDependencies": {
-    "@mariozechner/pi-coding-agent": "*"
+    "@earendil-works/pi-coding-agent": "*"
   },
   "pi": {
     "extensions": [

package/public/app.js

@@ -4,6 +4,7 @@ let currentSessionId = null;
 let currentTasks = [];
 let viewMode = 'session';
 let sessionFilter = 'active';
+let showPins = false; // include pinned sessions when Active filter is active
 let sessionLimit = '20';
 let filterProject = '__recent__'; // null = all, '__recent__' = last 24h, or project path
 let recentProjects = new Set();
@@ -62,6 +63,9 @@ function getUrlState() {
       ? params.get('messages') === '1'
       : localStorage.getItem('message-panel-open') === 'true',
     projectView: params.get('projectView'),
+    pins: params.has('pins')
+      ? params.get('pins') === '1'
+      : localStorage.getItem('show-pins') === '1',
   };
 }
 
@@ -71,6 +75,7 @@ function updateUrl() {
   if (viewMode === 'project' && currentProjectPath) params.set('projectView', btoa(currentProjectPath));
   if (currentSessionId) params.set('session', currentSessionId);
   if (sessionFilter !== 'active') params.set('filter', sessionFilter);
+  if (showPins) params.set('pins', '1');
   if (sessionLimit !== '20') params.set('limit', sessionLimit);
   if (filterProject && filterProject !== '__recent__') params.set('project', filterProject);
   if (searchQuery) params.set('search', searchQuery);
@@ -107,6 +112,8 @@ function resetState() {
     localStorage.removeItem(LAST_VIEW_KEY);
   } catch (_) {}
   sessionFilter = 'active';
+  showPins = false;
+  localStorage.removeItem('show-pins');
   sessionLimit = '20';
   filterProject = '__recent__';
   searchQuery = '';
@@ -981,11 +988,19 @@ function renderMessageList(messages) {
           </div>`);
       } else {
         const cmd = parseCommandMessage(m.text);
+        const imgCount = m.images?.length || 0;
         const displayText = cmd ? cmd : escapeHtml(cleanMessageText(m.text));
         const isCmd = !!cmd;
+        const chips = [];
+        if (imgCount) chips.push(`<span class="user-attach-chip">${imgCount} image${imgCount > 1 ? 's' : ''}</span>`);
+        const chipsHtml = chips.length ? `<div class="user-attach-chips">${chips.join('')}</div>` : '';
+        let textHtml;
+        if (displayText) textHtml = isCmd ? `<code>${escapeHtml(displayText)}</code>` : displayText;
+        else if (chips.length) textHtml = '<em class="msg-text-muted">(attachment)</em>';
+        else textHtml = '';
         parts.push(`<div class="msg-item msg-user${isCmd ? ' msg-cmd' : ''}" ${clickable}>
             ${MSG_ICON_USER}
-            <div class="msg-body"><div class="msg-text">${isCmd ? `<code>${escapeHtml(displayText)}</code>` : displayText}</div><div class="msg-time">${formatDate(m.timestamp)}</div></div>${pinBtn}
+            <div class="msg-body"><div class="msg-text">${textHtml}</div>${chipsHtml}<div class="msg-time">${formatDate(m.timestamp)}</div></div>${pinBtn}
           </div>`);
       }
     } else if (m.type === 'assistant') {
@@ -1373,6 +1388,17 @@ const linkSvg = (size) =>
 //#endregion
 
 //#region MODALS
+function renderUserAttachments(m) {
+  if (!m.images?.length || !m.id || !currentSessionId) return '';
+  const imgs = m.images
+    .map((img) => {
+      const url = `/api/sessions/${encodeURIComponent(currentSessionId)}/user-image/${encodeURIComponent(m.id)}/${img.blockIndex}`;
+      return `<img src="${url}" loading="lazy" alt="attached image" class="user-attach-image" onerror="this.style.display='none'" />`;
+    })
+    .join('');
+  return `<div class="user-attach-section"><div class="user-attach-label">Attached images</div><div class="user-attach-images">${imgs}</div></div>`;
+}
+
 function showMsgDetail(idx) {
   currentMsgDetailIdx = idx;
   const m = currentMessages[idx];
@@ -1439,13 +1465,14 @@ function showMsgDetail(idx) {
     }
     body.innerHTML = mainHtml + toolParamsHtml + taskResultHtml + (hasAgentTabs || isParallelAgent ? '' : toolResultHtml) + agentExtraHtml;
   } else {
-    const rawText = stripAnsi(m.fullText || m.text);
+    let rawText = stripAnsi(m.fullText || m.text);
+    const userExtras = renderUserAttachments(m);
     const cmd = m.type === 'user' ? parseCommandMessage(rawText) : null;
     document.getElementById('msg-detail-title').textContent =
       m.type === 'assistant' ? 'Pi' : m.systemLabel ? 'System' : 'User';
     document.getElementById('msg-detail-agent-btn').style.display = 'none';
     if (m.compactSummary) {
-      body.innerHTML = renderMarkdown(m.compactSummary);
+      body.innerHTML = renderMarkdown(m.compactSummary) + userExtras;
     } else if (cmd) {
       const argsMatch = rawText.match(/<command-args>([^<]*)<\/command-args>/);
       const args = argsMatch?.[1].trim() ? argsMatch[1].trim() : null;
@@ -1455,9 +1482,9 @@ function showMsgDetail(idx) {
         .trim();
       let cmdHtml = `<code>${escapeHtml(cmd)}${args ? ` ${escapeHtml(args)}` : ''}</code>`;
       if (cleanBody) cmdHtml += `<div style="margin-top:10px">${renderMarkdown(cleanBody)}</div>`;
-      body.innerHTML = cmdHtml;
+      body.innerHTML = cmdHtml + userExtras;
     } else {
-      body.innerHTML = renderMarkdown(rawText);
+      body.innerHTML = renderMarkdown(rawText) + userExtras;
     }
   }
   const modal = document.getElementById('msg-detail-modal').querySelector('.modal');
@@ -2184,7 +2211,7 @@ function showAgentModal(agentId) {
   const modalNameLabel = agent.agentName ? ` · ${escapeHtml(agent.agentName)}` : '';
   title.innerHTML = `${statusDot} ${escapeHtml(agent.type || 'unknown')}${modalNameLabel}`;
 
-  const shortModel = agent.model ? agent.model.replace(/^claude-/, '').replace(/-\d{8}$/, '') : null;
+  const shortModel = agent.model ?? null;
   const shortId = agent.agentId ? agent.agentId.slice(0, 8) : '';
   const chip = (label, value, opts = {}) => {
     const cls = opts.cls ? ` ${opts.cls}` : '';
@@ -2330,7 +2357,7 @@ function _renderSessions() {
   const LIVE_INDICATOR_MS = 10 * 1000;
   let filteredSessions = sessions;
   if (sessionFilter === 'active') {
-    const ACTIVE_PLAN_MS = 15 * 60 * 1000;
+    const ACTIVE_PLAN_MS = 10 * 60 * 1000;
     const now = Date.now();
     const activeSessionIds = new Set();
     filteredSessions = filteredSessions.filter((s) => {
@@ -2384,15 +2411,15 @@ function _renderSessions() {
     filteredSessions = filteredSessions.filter(matchesSearch);
 
     // Re-add pinned/sticky sessions that match the query but were excluded by active filter
-    if (pinnedSessionIds.size > 0 || stickySessionIds.size > 0) {
+    if (showPins && (pinnedSessionIds.size > 0 || stickySessionIds.size > 0)) {
       const filteredIds = new Set(filteredSessions.map((s) => s.id));
       const missingPinned = sessions.filter((s) => isAnyPinned(s.id) && !filteredIds.has(s.id) && matchesSearch(s));
       if (missingPinned.length) filteredSessions = [...missingPinned, ...filteredSessions];
     }
   }
 
-  // Include pinned/sticky sessions even if they don't match active/recent filter
-  if (!searchQuery && (pinnedSessionIds.size > 0 || stickySessionIds.size > 0)) {
+  // Include pinned/sticky sessions even if they don't match active filter
+  if (sessionFilter === 'active' && showPins && !searchQuery && (pinnedSessionIds.size > 0 || stickySessionIds.size > 0)) {
     const filteredIds = new Set(filteredSessions.map((s) => s.id));
     const missingPinned = sessions.filter((s) => isAnyPinned(s.id) && !filteredIds.has(s.id));
     if (missingPinned.length) filteredSessions = [...missingPinned, ...filteredSessions];
@@ -2605,31 +2632,7 @@ function _renderSessions() {
 
     sessionsList.innerHTML = html;
   } else {
-    const sticky = filteredSessions.filter((s) => isPlacedSticky(s.id));
-    const idlePinned = filteredSessions.filter((s) => isPlacedPinned(s.id) && !isSessionActive(s));
-    const rest = filteredSessions.filter(
-      (s) => (!isPlacedPinned(s.id) && !isPlacedSticky(s.id)) || (isPlacedPinned(s.id) && isSessionActive(s)),
-    );
-    let html = '';
-    if (sticky.length > 0) {
-      html += sticky.map(renderSessionCard).join('');
-    }
-    const isCollapsed = collapsedProjectGroups.has('__pinned__');
-    const hasPinned = pinnedSessionIds.size > 0 && filteredSessions.some((s) => pinnedSessionIds.has(s.id));
-    if (idlePinned.length > 0 || (hasPinned && isCollapsed)) {
-      html += `
-            <div class="project-group-header${isCollapsed ? ' collapsed' : ''}" data-group-path="__pinned__">
-              <svg class="group-chevron" width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><polyline points="6 9 12 15 18 9"/></svg>
-              <span class="group-name">Pinned</span>
-              <span class="group-count">${idlePinned.length}</span>
-            </div>
-            <div class="project-group-sessions${isCollapsed ? ' collapsed' : ''}">
-              ${idlePinned.map(renderSessionCard).join('')}
-            </div>
-          `;
-    }
-    html += rest.map(renderSessionCard).join('');
-    sessionsList.innerHTML = html;
+    sessionsList.innerHTML = filteredSessions.map(renderSessionCard).join('');
   }
 
   const navItems = getNavigableItems();
@@ -4301,15 +4304,6 @@ function handleSessionOpenEvent(data) {
     showToast(`Session not found: ${id.slice(0, 8)}`);
     return;
   }
-  if (sessionFilter !== 'active') {
-    sessionFilter = 'active';
-    const sel = document.getElementById('session-filter');
-    if (sel) sel.value = 'active';
-    updateUrl();
-  }
-  if (!isSessionActive(target)) {
-    stickySessionIds.add(id);
-  }
   fetchTasks(id);
 }
 
@@ -4329,7 +4323,7 @@ async function handlePreviewOpenEvent(data) {
     if (_infoModalSessionId === sessionId) refreshInfoModalLinkedDocs();
     renderSessions();
   }
-  openPreviewModal(filePath, content);
+  if (!link) openPreviewModal(filePath, content);
   if (link) updatePreviewLinkBtn();
 }
 
@@ -4930,8 +4924,16 @@ function filterBySessions(value) {
   sessionFilter = value;
   updateUrl();
   renderSessions();
+  syncFilterUI();
 }
 
+// biome-ignore lint/correctness/noUnusedVariables: used in HTML
+function toggleShowPins(checked) {
+  showPins = checked;
+  localStorage.setItem('show-pins', showPins ? '1' : '0');
+  updateUrl();
+  renderSessions();
+}
 // biome-ignore lint/correctness/noUnusedVariables: used in HTML
 function changeSessionLimit(value) {
   sessionLimit = value;
@@ -5217,7 +5219,8 @@ function initSidebarResize() {
   });
 
   function onMove(e) {
-    const w = Math.min(600, Math.max(200, startWidth + e.clientX - startX));
+    const maxW = Math.floor(window.innerWidth * 0.9);
+    const w = Math.min(maxW, Math.max(200, startWidth + e.clientX - startX));
     sidebar.style.setProperty('--sidebar-width', `${w}px`);
     sidebar.style.width = `${w}px`;
   }
@@ -5249,7 +5252,8 @@ function initPanelResize(panelId, handleId, cssVar, storageKey) {
   });
 
   function onMove(e) {
-    const w = Math.min(900, Math.max(320, startWidth - (e.clientX - startX)));
+    const maxW = Math.floor(window.innerWidth * 0.9);
+    const w = Math.min(maxW, Math.max(320, startWidth - (e.clientX - startX)));
     panel.style.setProperty(cssVar, `${w}px`);
   }
 
@@ -5276,9 +5280,16 @@ function loadPanelWidths() {
 //#endregion
 
 //#region PREFERENCES
+function syncFilterUI() {
+  const pinsLabel = document.getElementById('show-pins-label');
+  if (pinsLabel) pinsLabel.style.display = sessionFilter === 'active' ? '' : 'none';
+  const pinsChk = document.getElementById('show-pins');
+  if (pinsChk) pinsChk.checked = showPins;
+}
 function loadPreferences() {
   document.getElementById('session-filter').value = sessionFilter;
   document.getElementById('session-limit').value = sessionLimit;
+  syncFilterUI();
 }
 
 //#endregion
@@ -5789,6 +5800,7 @@ fetch('/api/version')
 
 const urlState = getUrlState();
 sessionFilter = urlState.filter || 'active';
+showPins = urlState.pins;
 sessionLimit = urlState.limit || '20';
 filterProject = urlState.project || '__recent__';
 searchQuery = urlState.search || '';
@@ -5841,6 +5853,7 @@ fetchSessions()
 window.addEventListener('popstate', () => {
   const s = getUrlState();
   sessionFilter = s.filter || 'active';
+  showPins = s.pins;
   sessionLimit = s.limit || '20';
   filterProject = s.project || '__recent__';
   searchQuery = s.search || '';

package/public/index.html

@@ -53,15 +53,15 @@
           </div>
           <span class="logo-text">Dashboard</span>
         </div>
-        <div id="connection-status" class="connection">
-          <span class="connection-dot"></span>
-          <span>Connecting</span>
-        </div>
         <button id="sidebar-toggle" class="sidebar-toggle-btn" onclick="toggleSidebar()" title="Toggle sidebar" aria-label="Toggle sidebar">
           <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2">
             <path d="M15 18l-6-6 6-6"/>
           </svg>
         </button>
+        <div id="connection-status" class="connection">
+          <span class="connection-dot"></span>
+          <span>Connecting</span>
+        </div>
       </header>
 
       <!-- Tasks -->
@@ -93,8 +93,15 @@
             </select>
             <select id="session-filter" class="filter-dropdown" onchange="filterBySessions(this.value)" aria-label="Filter by session status">
               <option value="all">All Sessions</option>
-              <option value="active">Active Only</option>
+              <option value="active">Active</option>
             </select>
+            <label class="filter-checkbox-label" id="show-pins-label" title="Include pinned sessions in Active filter">
+              <input type="checkbox" id="show-pins" onchange="toggleShowPins(this.checked)" />
+              <span class="filter-pins-btn">
+                <svg width="11" height="11" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><line x1="12" y1="17" x2="12" y2="3"/><path d="M5 10l7-7 7 7"/><line x1="4" y1="21" x2="20" y2="21"/></svg>
+                Pins
+              </span>
+            </label>
           </div>
           <div class="filter-row">
             <select id="session-limit" class="filter-dropdown" onchange="changeSessionLimit(this.value)" aria-label="Number of sessions to show">

package/public/style.css

@@ -325,6 +325,7 @@ body::before {
   display: flex;
   gap: 6px;
   padding: 0 16px 10px;
+  align-items: center;
 }
 
 .reset-btn {
@@ -361,6 +362,49 @@ body::before {
   color: var(--text-primary);
 }
 
+.filter-checkbox-label {
+  display: flex;
+  align-items: center;
+  gap: 0;
+  flex-shrink: 0;
+  user-select: none;
+  cursor: pointer;
+}
+
+.filter-checkbox-label input[type='checkbox'] {
+  position: absolute;
+  opacity: 0;
+  width: 0;
+  height: 0;
+  pointer-events: none;
+}
+
+.filter-pins-btn {
+  display: flex;
+  align-items: center;
+  gap: 5px;
+  padding: 7px 10px;
+  border-radius: 6px;
+  border: 1px solid transparent;
+  background: var(--bg-deep);
+  font-family: var(--mono);
+  font-size: 12px;
+  color: var(--text-secondary);
+  cursor: pointer;
+  white-space: nowrap;
+  transition: all 0.15s ease;
+  line-height: 1;
+}
+
+.filter-pins-btn:hover {
+  border-color: var(--border);
+}
+
+.filter-checkbox-label input[type='checkbox']:checked + .filter-pins-btn {
+  border-color: var(--border);
+  color: var(--text-primary);
+}
+
 .filter-dropdown:focus {
   outline: none;
   border-color: var(--accent);
@@ -1728,7 +1772,7 @@ body::before {
   display: flex;
 }
 .message-panel.msg-expanded-wide {
-  width: 60vw;
+  width: 90vw;
 }
 .message-panel-header {
   padding: 16px 20px;
@@ -3758,3 +3802,47 @@ pre.mermaid svg {
 }
 /* #endregion */
 
+/* #region USER_ATTACHMENTS */
+.user-attach-chips {
+  display: flex;
+  gap: 4px;
+  flex-wrap: wrap;
+  margin-top: 4px;
+}
+.user-attach-chip {
+  font-size: 10px;
+  padding: 1px 6px;
+  border-radius: 10px;
+  background: var(--bg-hover);
+  color: var(--text-secondary);
+}
+.msg-text-muted {
+  color: var(--text-secondary);
+  font-size: 12px;
+}
+.user-attach-section {
+  margin-top: 12px;
+  padding-top: 10px;
+  border-top: 1px solid var(--border);
+}
+.user-attach-label {
+  font-size: 11px;
+  color: var(--text-secondary);
+  margin-bottom: 6px;
+  text-transform: uppercase;
+  letter-spacing: 0.5px;
+}
+.user-attach-images {
+  display: flex;
+  flex-wrap: wrap;
+  gap: 8px;
+}
+.user-attach-image {
+  max-width: 100%;
+  max-height: 480px;
+  border-radius: 4px;
+  border: 1px solid var(--border);
+  background: var(--bg-hover);
+}
+/* #endregion */
+

package/server.js

@@ -213,6 +213,21 @@ app.get('/api/sessions/:id/tool-result/:toolUseId', async (req, res) => {
 });
 
 
+app.get('/api/sessions/:id/user-image/:msgId/:blockIndex', async (req, res) => {
+  const meta = await parsers.findSessionFileById(req.params.id);
+  if (!meta?.file) return res.status(404).end();
+  try {
+    const img = await parsers.readUserImage(meta.file, req.params.msgId, Number(req.params.blockIndex));
+    if (!img) return res.status(404).end();
+    res.setHeader('Content-Type', img.mediaType);
+    res.setHeader('Cache-Control', 'no-store');
+    res.end(Buffer.from(img.data, 'base64'));
+  } catch (err) {
+    res.status(500).end();
+  }
+});
+
+
 app.get('/api/sessions/:id/agents', async (req, res) => {
   const meta = await parsers.findSessionFileById(req.params.id);
   if (!meta) return res.status(404).json({ agents: [], error: 'not found' });
@@ -371,7 +386,7 @@ app.post('/api/preview', async (req, res) => {
     const { path: filePath, sessionId, base, link } = req.body || {};
     const abs = resolvePreviewPath(filePath, base);
     if (!abs) return res.status(400).json({ error: 'path is required' });
-    const content = await readMarkdownFile(abs);
+    const content = link ? null : await readMarkdownFile(abs);
     sseSend({ type: 'preview:open', path: abs, content, sessionId: sessionId || null, link: !!link });
     res.json({ success: true });
   } catch (err) {
@@ -535,11 +550,38 @@ parsers.setOnBranchResolved((cwd, branch) => {
 
 const queueSessionUpdate = debounceByKey(75, (sid) => sseSend({ type: 'update', sessionId: sid }));
 
+// When a subagent session file changes, notify the parent session so the
+// frontend refreshes agent chips with model info immediately.
+async function notifyParentSession(f) {
+  const base = path.basename(f);
+  if (base === 'session.jsonl') {
+    // Path: .../sessions/<enc>/<sessionName>/<runId>/run-N/session.jsonl
+    // Parent session dir is three levels up.
+    const parentId = parsers.slugFromFile(path.dirname(path.dirname(path.dirname(f))));
+    if (parentId && parentId !== 'session') queueSessionUpdate(parentId);
+  } else if (base.endsWith('.jsonl')) {
+    // Possible fork-mode sibling session — read first line to check parentSession.
+    try {
+      const fd = await fsp.open(f, 'r');
+      const buf = Buffer.alloc(512);
+      const { bytesRead } = await fd.read(buf, 0, 512, 0);
+      await fd.close();
+      const firstLine = buf.slice(0, bytesRead).toString('utf8').split('\n')[0];
+      const ev = JSON.parse(firstLine);
+      if (ev.parentSession) {
+        const parentId = parsers.slugFromFile(ev.parentSession);
+        if (parentId) queueSessionUpdate(parentId);
+      }
+    } catch {}
+  }
+}
+
 for (const ev of ['add', 'change', 'unlink']) {
   watcher.on(ev, (f) => {
     parsers.invalidateSessionCache(f);
     agentLogStreams.notify(f);
     queueSessionUpdate(parsers.slugFromFile(f));
+    notifyParentSession(f).catch(() => {});
   });
 }