@pi-spice/minimal-subagents 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -0,0 +1,7 @@
+# @pi-spice/minimal-subagents
+
+## 0.1.1
+
+### Patch Changes
+
+- 245d41f: Details panel: alt+a now toggles (also closes an open panel); panel data is seeded the moment spawn_agents starts, so alt+a works during the child-startup window and after an interrupt instead of reporting "no data"; alt+a with no run at all shows pi's notify; panel content gets inner padding and footer hints that fit narrow panels.

package/README.md

@@ -31,7 +31,8 @@ Quick test from this repo: `pi -e ./extensions/minimal-subagents`
 
 - One tab per sub-agent (`←`/`→` or `1`-`8`; the tab bar compacts automatically on narrow panels), labeled with name and live status (⏳/✓/✗).
 - Each tab is the agent's full timeline: task, tool calls, tool-result previews (first 10 lines), assistant output rendered as markdown, usage. Thinking is not shown.
-- Terminal-style scrolling: `↑/↓`, `PgUp/PgDn`, `Home`/`g`, `End`/`G`, mouse wheel — pinned to the bottom while following new output, scrolling up pauses, `End` resumes. `Esc` closes.
+- Terminal-style scrolling: `↑/↓`, `PgUp/PgDn`, `Home`/`g`, `End`/`G`, mouse wheel — pinned to the bottom while following new output, scrolling up pauses, `End` resumes. `alt+a` toggles (same key opens and closes); `Esc` also closes.
+- Pressing `alt+a` before any `spawn_agents` run shows pi's notify message above the input instead of opening an empty panel.
 - Shows the latest call only. Two platform limits: it is an overlay (the transcript is covered, not reflowed), and mouse wheel works only under `--tui-mode fullscreen` — the only mode where pi enables terminal mouse reporting.
 
 ## No nesting

package/index.ts

@@ -28,7 +28,7 @@ import {
 	type SubagentDetails,
 } from "./spawn.ts";
 import { renderSpawnCall, renderSpawnResult } from "./render.ts";
-import { openAgentPanel, setPanelDetails } from "./panel.ts";
+import { hasPanelDetails, openAgentPanel, setPanelDetails } from "./panel.ts";
 
 const MAX_AGENTS = 8;
 const MAX_CONCURRENCY = 4;
@@ -138,6 +138,13 @@ export default function (pi: ExtensionAPI) {
 				}
 			};
 
+			// Seed the panel the moment the tool starts: alt+a can then open it and
+			// show "running" placeholders. Without this, the panel would stay
+			// "no data" until the first child event arrives — child boot plus the
+			// child's first model turn can take 5-20s, and interrupting (or just
+			// peeking) inside that window would claim there is nothing to show.
+			emitParallelUpdate();
+
 			const results = await mapWithConcurrencyLimit(params.agents, MAX_CONCURRENCY, async (spec, index) => {
 				// On abort, return what exists instead of throwing finished work away.
 				const abortPlaceholder = (): SingleResult => {
@@ -202,7 +209,15 @@ export default function (pi: ExtensionAPI) {
 	});
 
 	pi.registerShortcut("alt+a", {
-		description: "Open the sub-agent details panel (tabs per agent, full timeline)",
-		handler: (ctx) => openAgentPanel(ctx),
+		description: "Toggle the sub-agent details panel (tabs per agent, full timeline); alt+a or Esc closes it",
+		handler: (ctx) => {
+			// No run yet → pi's notify message, not an empty overlay stuck in the
+			// corner.
+			if (!hasPanelDetails()) {
+				ctx.ui.notify("No sub-agent data yet — run spawn_agents first", "info");
+				return;
+			}
+			openAgentPanel(ctx);
+		},
 	});
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-spice/minimal-subagents",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Create sub-agents dynamically and run them in parallel; single blocking tool, no orchestration, nesting prevented",
   "keywords": ["pi-package"],
   "license": "MIT",

package/panel.ts

@@ -1,10 +1,15 @@
 /**
  * panel.ts — sub-agent details panel (overlay) for minimal-subagents
  *
- * Opened with alt+a (registered in index.ts). Shows the latest spawn_agents
- * call: one tab per sub-agent, a full scrollable timeline per tab (task,
- * tool calls, tool-result previews, assistant output rendered as markdown,
- * usage), live-updating while agents run.
+ * Opened with alt+a (registered in index.ts), closed with alt+a or Esc —
+ * while the panel is focused the host routes all input here, so the open
+ * shortcut never fires; the panel must recognize alt+a itself to toggle.
+ * With no spawn_agents data yet, the shortcut shows a notify message
+ * instead of opening an empty overlay (guarded in index.ts via
+ * hasPanelDetails).
+ * Shows the latest spawn_agents call: one tab per sub-agent, a full scrollable
+ * timeline per tab (task, tool calls, tool-result previews, assistant output
+ * rendered as markdown, usage), live-updating while agents run.
  *
  * Rendering is line-based: the timeline is flattened into styled lines and
  * windowed by a hand-rolled viewport (offset math) — the overlay contract is
@@ -13,7 +18,7 @@
  * while the overlay is focused.
  */
 
-import { Markdown, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
+import { Markdown, matchesKey, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
 import { getMarkdownTheme } from "@earendil-works/pi-coding-agent";
 import { type Message } from "@earendil-works/pi-ai";
 import { formatToolCall, formatUsageStats, type RenderTheme } from "./render.ts";
@@ -32,6 +37,11 @@ export function setPanelDetails(details: SubagentDetails): void {
 	for (const listener of listeners) listener();
 }
 
+/** True once at least one spawn_agents result (even still running) exists. */
+export function hasPanelDetails(): boolean {
+	return currentDetails !== null && currentDetails.results.length > 0;
+}
+
 // ---------------------------------------------------------------------------
 // Panel opening
 // ---------------------------------------------------------------------------
@@ -39,7 +49,7 @@ export function setPanelDetails(details: SubagentDetails): void {
 let opening: Promise<unknown> | null = null;
 
 export function openAgentPanel(ctx: { ui: any; hasUI?: boolean }): void {
-	if (opening) return; // already open; Esc closes
+	if (opening) return; // already open; alt+a or Esc closes
 	if (ctx.hasUI === false) return;
 	opening = ctx.ui
 		.custom(
@@ -187,6 +197,12 @@ class AgentPanel {
 		// A newer spawn_agents call may have fewer agents — clamp the tab.
 		this.activeTab = Math.min(this.activeTab, details.results.length - 1);
 
+		// Inner padding: 1 column each side and 1 blank line above/below the
+		// body, so content breathes away from the rules and panel edges. The
+		// rules span the full width; content wraps at width - 2.
+		const innerWidth = Math.max(10, width - 2);
+		const pad = (line: string) => (line.length === 0 ? "" : ` ${line}`);
+
 		// --- header: tab bar ------------------------------------------------
 		// Full labels (index + name + status) when they fit the panel width;
 		// otherwise degrade to compact slots (index + status) which always fit
@@ -200,35 +216,47 @@ class AgentPanel {
 			return `${i + 1} ${name} ${statusIcon(r, theme)}`;
 		});
 		const fullFits =
-			7 + fullLabels.reduce((sum, l) => sum + visibleWidth(l) + 2, 0) + (fullLabels.length - 1) <= width;
+			7 + fullLabels.reduce((sum, l) => sum + visibleWidth(l) + 2, 0) + (fullLabels.length - 1) <= innerWidth;
 		const tabs = fullFits
 			? renderSlots(fullLabels)
 			: renderSlots(details.results.map((r, i) => `${i + 1}${statusIcon(r, theme)}`));
-		const header = [theme.fg("toolTitle", theme.bold("agents ")) + tabs, theme.fg("muted", "─".repeat(width))];
+		const header = [pad(theme.fg("toolTitle", theme.bold("agents ")) + tabs), theme.fg("muted", "─".repeat(width))];
 
 		// --- body: windowed timeline ----------------------------------------
 		const rows = this.tui?.terminal?.rows ?? process.stdout.rows ?? 24;
-		const headerH = header.length; // tab bar + rule
-		const footerH = 2; // rule + status line
+		const headerH = header.length + 1; // blank pad + tab bar + rule
+		const footerH = 2 + 1; // rule + status line + blank pad above the rule
 		this.bodyHeight = Math.max(4, rows - headerH - footerH);
 
-		const lines = buildTimeline(details.results[this.activeTab], theme, width);
+		const lines = buildTimeline(details.results[this.activeTab], theme, innerWidth);
 		this.lineCount = lines.length;
 
 		const maxOffset = Math.max(0, this.lineCount - this.bodyHeight);
 		if (this.follow) this.offset = maxOffset;
 		this.offset = Math.min(Math.max(0, this.offset), maxOffset);
-		const body = lines.slice(this.offset, this.offset + this.bodyHeight);
+		const body = lines.slice(this.offset, this.offset + this.bodyHeight).map(pad);
 		while (body.length < this.bodyHeight) body.push(""); // stable panel height
 
 		// --- footer: scroll position + hints ---------------------------------
 		const pos = this.lineCount > 0 ? `${this.offset + 1}-${Math.min(this.offset + this.bodyHeight, this.lineCount)}/${this.lineCount}` : "0";
 		const mode = this.follow ? "following" : "paused";
-		const footer =
-			theme.fg("dim", `${pos} ${mode}`) +
-			theme.fg("muted", " · ←/→ tab · ↑/↓ wheel scroll · End follow · Esc close");
+		const statusText = theme.fg("dim", `${pos} ${mode}`);
+		// Hint set degrades as the panel narrows; truncate is only a backstop.
+		const hintVariants = [
+			" · ←/→ tab · ↑/↓ scroll · End follow · alt+a/Esc close",
+			" · ←/→ tab · End follow · alt+a/Esc close",
+			" · alt+a/Esc close",
+		];
+		let hints = "";
+		for (const hint of hintVariants) {
+			if (visibleWidth(statusText) + hint.length <= innerWidth) {
+				hints = theme.fg("muted", hint);
+				break;
+			}
+		}
+		const footer = truncateToWidth(pad(statusText + hints), width);
 
-		return [...header, ...body, theme.fg("muted", "─".repeat(width)), footer];
+		return ["", ...header, ...body, "", theme.fg("muted", "─".repeat(width)), footer];
 	}
 
 	handleInput(data: string): void {
@@ -241,6 +269,14 @@ class AgentPanel {
 	}
 
 	private handleInputInner(data: string): void {
+		// alt+a toggles: while we hold focus the host shortcut cannot fire, so
+		// the panel closes on the same key that opened it (matchesKey covers
+		// legacy ESC+a and kitty/CSI-u encodings alike).
+		if (matchesKey(data, "alt+a")) {
+			this.close();
+			return;
+		}
+
 		const details = currentDetails;
 		if (!details || details.results.length === 0) {
 			if (data === "\x1b") this.close();