@yusukeshib/pi-babysit 0.6.22 → 0.6.24

package/README.md

@@ -50,10 +50,13 @@ at every depth.
 Because sessions are real PTYs, the agent can also **drive interactive
 programs** (installers, wizards, REPLs): type with `babysit_send`
 (text or named keys) and read the rendered screen with
-`babysit_check { screen: true }` — a capability neither retired extension had.
+`babysit_check { description, screen: true }` — a capability neither retired extension had.
 
 ## Tools (LLM-callable)
 
+All five tools require a short, non-blank `description` explaining the purpose
+of that call.
+
 | Tool | What it does |
 | ---- | ------------ |
 | `babysit_run` | Run any command (`description` + `command`, optional `name`/`pty`/`timeout`/`idleTimeout`/`retryOnWorkerDeath`/`notificationGroup`/`lifecycle`). Set `foreground: true` for one process or subagent whose result is needed in the same tool call; use `returnPattern`/`returnLines`/`maxBytes` to keep noisy process output bounded. Foreground and non-interactive process waits default to lifecycle `attached`, which terminates the process tree if that owning tool call is interrupted; background starts default to `detached`. Or start a named background subagent (`description`, `profile: "subagent"`, `task`, optional `name`/`agent`/`model`/`tools`/`maxDepth` and budget fields), then always collect it with `babysit_wait`. `maxDepth` defaults to 1. Subagent `continueAfterStart: true` is accepted as a compatibility alias for this default background behavior. Quick commands return inline; longer process runs notify in the background |
@@ -71,26 +74,35 @@ calls if another extension or preset re-enables `bash`, including shell
 backgrounding (`… &`, `nohup`, `setsid`, `disown`). Set
 `PI_BABYSIT_ALLOW_BASH=1` to retain direct `bash` explicitly.
 
-## Describe every run
+## Describe every operation
 
-Every `babysit_run` requires a non-blank `description`, for both commands and
-subagents. Explain the purpose briefly in language the user can understand;
-`name` remains a stable session identifier, while `command` / `task` contains
-the actual execution instructions. Descriptions are trimmed before saving and
-are display metadata, not shell code or instructions sent to the subagent.
+Every babysit tool call requires a non-blank `description`. Explain the purpose
+briefly in language the user can understand. For `babysit_run`, `name` remains
+a stable session identifier, while `command` / `task` contains the execution
+instructions. Its description is trimmed and saved with the session. For
+`babysit_check`, `babysit_send`, `babysit_wait`, and `babysit_kill`, the description
+explains only that operation and never overwrites the session description.
+Descriptions are display metadata, not shell code, stdin, or instructions sent
+to the subagent.
 
 ```text
 babysit_run { description: "Run regression tests", command: "bun test", foreground: true }
 babysit_run { description: "Review the authentication changes", profile: "subagent", task: "Review the authentication diff for security regressions", maxTurns: 5 }
+babysit_check { description: "Inspect build errors", id: "build", pattern: "ERROR" }
+babysit_wait { description: "Wait for the build to finish", id: "build" }
+babysit_send { description: "Ask the reviewer to check authorization too", id: "review", text: "Also check authorization." }
+babysit_kill { description: "Stop the unused development server", id: "dev-server" }
 ```
 
 In Pi's main transcript the description is the prominent yellow heading, with
-the execution kind in muted gray. The full command or task is always visible
+the tool name and execution kind in muted gray. The full command or task (or
+all other operation arguments, including target IDs and input) is always visible
 in gray beneath it, even when tool results are collapsed. Session lists, the
-widget, and completion notifications also show the description. Older saved sessions and tool calls
-without one remain readable, falling back to their command or task. Callers
-must supply a description for new runs; blank or missing values are rejected
-before launch.
+widget, and completion notifications show the session's original description.
+Older saved sessions and tool calls without descriptions remain readable,
+falling back to their command, task, or other arguments. Callers must supply a
+description for every new call; blank or missing values are rejected before
+execution.
 
 ## Remote processes over SSH
 
@@ -102,9 +114,9 @@ pi-babysit does not install remote software or copy credentials.
 ```text
 babysit_run { description: "Run tests on Linux", host: "yusuke-t4", cwd: "/home/yusuke/project", command: "bun test", foreground: true, returnLines: 30 }
 babysit_run { description: "Open a remote Python REPL", host: "user@server", command: "python3 -i", name: "remote-repl", continueAfterStart: true }
-babysit_check { id: "remote-repl", screen: true }
-babysit_send { id: "remote-repl", text: "print(42)" }
-babysit_kill { id: "remote-repl" }
+babysit_check { description: "Read the remote REPL screen", id: "remote-repl", screen: true }
+babysit_send { description: "Evaluate a value in the remote REPL", id: "remote-repl", text: "print(42)" }
+babysit_kill { description: "Close the remote REPL", id: "remote-repl" }
 ```
 
 `host` omitted (or `"local"`) preserves local behavior. `cwd` is process-only:
@@ -155,9 +167,11 @@ sessions, and click again to return to running-only. Both modes show newest firs
 ten per page; `< 1/4 >` shows the current/total page when needed. Click the
 arrows to page. Rows occupy one line, previewing the description (or the command
 for older processes and task for older agents), with `…` when clipped. Click an
-AGENT or PROCESS row to replace the list with a tall **log viewer in the same widget area**, not an
-overlay or separate screen. `Esc` or the right-hand **Close** button restores
-the list, preserving its mode and page. The viewer uses Pi's `ScrollView` for
+AGENT or PROCESS row to replace the list with a **log viewer in the same widget area**, not an
+overlay or separate screen. The viewer is capped at ten lines including its header
+and footer; all saved history remains available by scrolling. Click anywhere on
+the viewer's header row to restore the list, preserving its mode and page.
+There is no separate Close button or Esc binding. The viewer uses Pi's `ScrollView` for
 scroll position and follow behavior, with a widget-local scrollbar styled using
 Pi's scrollbar theme. Wheel, scrollbar drag/track clicks, and ↑/↓ navigate the
 log without requiring any Pi core changes. Home/End and PageUp/PageDown remain
@@ -204,8 +218,8 @@ pending for the next poll. Larger output stays out of model context. Inspect it
 through the session id without creating another shell session:
 
 ```text
-babysit_check { id: "cargo-test", lines: 50 }
-babysit_check { id: "cargo-test", pattern: "FAIL|ERROR", lines: 50 }
+babysit_check { description: "Inspect the recent test output", id: "cargo-test", lines: 50 }
+babysit_check { description: "Find test failures", id: "cargo-test", pattern: "FAIL|ERROR", lines: 50 }
 ```
 
 Tail and search results are capped at 200 lines and default to a 4 KB total

package/index.ts

@@ -51,6 +51,18 @@ import { Type, type TSchema } from "typebox";
 import { StringEnum } from "@earendil-works/pi-ai";
 import { type AgentConfig, type AgentScope, discoverAgents } from "./agents";
 
+const descriptionSchema = Type.String({
+	minLength: 1,
+	pattern: "\\S",
+	description: "Required short human-readable purpose for this operation. Explain what it does and why; shown prominently to the user.",
+});
+
+function requireDescription(value: unknown): void {
+	if (typeof value !== "string" || !value.trim()) {
+		throw new Error("`description` is required and must be a non-blank string for every babysit tool call.");
+	}
+}
+
 // Dedicated babysit state root so pi-managed sessions never collide with the
 // user's own manual `babysit` sessions. The base is namespaced per pi session
 // on session_start (BABYSIT_DIR=<base>/<session-id>), so each pi session only
@@ -1373,6 +1385,17 @@ function escapeCommandForDisplay(command: string): string {
 		);
 }
 
+function renderOperationCall(name: string, args: Record<string, unknown> | undefined, theme: Theme) {
+	const { description, ...operationArgs } = args ?? {};
+	const purpose = typeof description === "string" ? description.trim() : "";
+	const title = theme.fg("muted", theme.bold(name));
+	const heading = purpose
+		? `${title}  ${theme.fg("warning", theme.bold(escapeCommandForDisplay(purpose)))}`
+		: title;
+	const raw = Object.keys(operationArgs).length ? JSON.stringify(operationArgs) : "";
+	return new Text(heading + (raw ? `\n${theme.fg("muted", escapeCommandForDisplay(raw))}` : ""), 0, 0);
+}
+
 export function readableLaunchCommand(command: string): string {
 	// Older sessions quoted every argv element, including safe paths and flags.
 	if (!command.startsWith("'") || !command.endsWith("'") || !command.includes("' '")) return command;
@@ -2577,6 +2600,7 @@ function renderWidgetSessionHeader(
 // How many trailing output lines to show per session in the widget.
 const WIDGET_COLLAPSED_TAIL_LINES = 1;
 const WIDGET_SESSION_PAGE_SIZE = 10;
+const WIDGET_VIEWER_MAX_LINES = 10;
 
 export interface WidgetSessionListState {
 	mode: "running" | "all";
@@ -2732,7 +2756,11 @@ export function createSessionWidget(options: {
 		viewer?.dispose();
 		source = undefined;
 		selectedId = id;
-		viewer = createLogViewer({ ...options, onClose: close });
+		viewer = createLogViewer({
+			...options,
+			getHeight: () => Math.min(WIDGET_VIEWER_MAX_LINES, options.getHeight()),
+			onClose: close,
+		});
 		refreshViewer();
 		options.requestRender();
 	}
@@ -3286,7 +3314,7 @@ async function waitForTask(
 				ok: completed.ok,
 				text:
 					`Subagent ${id} finished its task (${stats}).\n` +
-					`${subagentReuseHint(readMeta(id)?.reapAfter)} Follow-up: babysit_send { id: "${id}" }, ` +
+					`${subagentReuseHint(readMeta(id)?.reapAfter)} Follow-up: babysit_send { description: "Assign a follow-up task", id: "${id}" }, ` +
 					`or babysit_kill when done.\n\n${completed.body}`,
 				status: st,
 				progress: prog,
@@ -3714,7 +3742,14 @@ export default function (pi: ExtensionAPI) {
 		const execute = tool.execute;
 		pi.registerTool({
 			...tool,
+			description: tool.description + " Always provide a short human-readable `description` explaining the purpose of this operation.",
+			promptGuidelines: [
+				"Every babysit tool call requires a short description explaining its purpose for the user. Operation descriptions for check/send/wait/kill do not change the session's original description.",
+				...(tool.promptGuidelines ?? []),
+			],
+			renderCall: tool.renderCall ?? ((args, theme) => renderOperationCall(tool.name, args as Record<string, unknown>, theme)),
 			async execute(toolCallId, params, signal, onUpdate, ctx) {
+				requireDescription((params as { description?: unknown }).description);
 				const result = await execute(toolCallId, params, signal, onUpdate, ctx);
 				if ((result as typeof result & { isError?: boolean }).isError === true) {
 					declaredToolErrors.add(toolCallId);
@@ -4156,7 +4191,7 @@ export default function (pi: ExtensionAPI) {
 					: d.count && d.count > 1
 						? theme.fg("muted", `×${d.count}`)
 						: "";
-			const header = theme.fg("muted", theme.bold("babysit_run COMMAND")) + (payload ? `  ${payload}` : "");
+			const header = theme.fg("muted", theme.bold("babysit_run PROCESS")) + (payload ? `  ${payload}` : "");
 			const box = new Box(1, 1, (t) => theme.bg("toolSuccessBg", t));
 			box.addChild(new Text(header, 0, 0));
 			box.addChild(new Text(renderStatus(status, theme), 0, 0));
@@ -4319,7 +4354,7 @@ export default function (pi: ExtensionAPI) {
 			block: true,
 			reason:
 				"Use babysit_run for shell commands so output is supervised and captured outside model context. " +
-				"Inspect an existing session log with babysit_check { id, lines, pattern? }. " +
+				"Inspect an existing session log with babysit_check { description, id, lines, pattern? }. " +
 				`Retry with a short purpose description as babysit_run({ description: "Explain the purpose of this command", command: ${JSON.stringify(command)} }).`,
 		};
 	});
@@ -4330,13 +4365,11 @@ export default function (pi: ExtensionAPI) {
 		label: "Babysit: run",
 		description:
 			"Run a supervised shell command, or start a reusable pi subagent with `profile: \"subagent\"`. " +
-			"Always provide a short human-readable `description` of what this run does and why. " +
 			"Use `foreground` for results needed now; otherwise long commands notify on exit. Full logs stay on disk. " +
 			"`returnPattern`/`returnLines` bound foreground output. Sessions support check, wait, send, and kill.",
 		promptSnippet: "Run supervised commands or bounded pi subagents with context-safe logs",
 		promptGuidelines: [
 			"Use babysit_run for shell commands and give meaningful sessions a stable name; bundle tiny related read-only observations into one command.",
-			"Every babysit_run requires a short description explaining its purpose for the user, for both commands and subagents. Describe the actual work, not a generic 'run command' or 'start agent'.",
 			"Use babysit_run foreground mode for one process or subagent whose result is needed now; never issue sibling foreground runs in parallel. For parallel process checks, start background runs with continueAfterStart and collect them with one multi-session babysit_wait.",
 			"Use returnPattern/returnLines for noisy commands. During edit/fix loops run targeted checks first and one full validation suite at the end instead of repeating every full gate.",
 			"After a background process starts, stop the turn for its automatic notification; never poll or sleep. Use continueAfterStart only for specific non-polling process work.",
@@ -4350,11 +4383,7 @@ export default function (pi: ExtensionAPI) {
 			return prepareBabysitRunArguments(args) as never;
 		},
 		parameters: Type.Object({
-			description: Type.String({
-				minLength: 1,
-				pattern: "\\S",
-				description: "Required short human-readable purpose for this command or subagent. Explain what it does and why; shown prominently to the user.",
-			}),
+			description: descriptionSchema,
 			host: Type.Optional(Type.String({ description: "Process SSH destination/config alias (default local). Remote subagents are unsupported." })),
 			cwd: Type.Optional(Type.String({ description: "Process working directory. Remote default is login home; never inferred from local cwd." })),
 			command: Type.Optional(
@@ -4485,9 +4514,6 @@ export default function (pi: ExtensionAPI) {
 
 		}),
 		async execute(_id, params, _signal, _onUpdate, ctx) {
-			if (typeof params.description !== "string" || !params.description.trim()) {
-				throw new Error("`description` is required and must be a non-blank string for both commands and subagents.");
-			}
 			const description = params.description.trim();
 			await requireBabysit();
 			const isSubagent = params.profile === "subagent";
@@ -4702,8 +4728,8 @@ export default function (pi: ExtensionAPI) {
 							text:
 								`${retried ? "Retried once after external worker death.\n" : ""}` +
 								`Process started (id: ${res.id}). ${NOTIFY_MARKER}\nDescription: ${escapeCommandForDisplay(description)}\nLog: ${logPath(res.id)}\n${nextStep}\n` +
-								`Inspect: babysit_check { id: "${res.id}" } (screen: true for TUIs) · ` +
-								`Wait: babysit_wait { id: "${res.id}" } · Kill: babysit_kill { id: "${res.id}" }\n` +
+								`Inspect: babysit_check { description: "Inspect session output", id: "${res.id}" } (screen: true for TUIs) · ` +
+								`Wait: babysit_wait { description: "Wait for completion", id: "${res.id}" } · Kill: babysit_kill { description: "Stop this session", id: "${res.id}" }\n` +
 								`Human can watch/take over: /babysit`,
 						},
 					],
@@ -4810,8 +4836,8 @@ export default function (pi: ExtensionAPI) {
 						text:
 							`Subagent started (id: ${res.id})${agent ? ` [agent: ${agent.name}]` : ""}${res.model ? ` [model: ${res.model}]` : ""} [depth: ${subagentNesting.childDepth}/${subagentNesting.maxDepth}].\n` +
 							`Description: ${escapeCommandForDisplay(description)}\nTask accepted — running in the background. You MUST collect it with babysit_wait before finishing the parent task; use foreground: true next time when no independent work is available.\n` +
-							`Progress: babysit_check { id: "${res.id}" } (only when inspection is needed)\n` +
-							`Collect:  babysit_wait  { id: "${res.id}" }\n` +
+							`Progress: babysit_check { description: "Inspect agent progress", id: "${res.id}" } (only when inspection is needed)\n` +
+							`Collect:  babysit_wait  { description: "Collect the agent result", id: "${res.id}" }\n` +
 							`Human can watch/steer: /babysit (pick ${res.id})`,
 					},
 				],
@@ -4833,7 +4859,7 @@ export default function (pi: ExtensionAPI) {
 		// Legacy calls without descriptions still render their original payload safely.
 		renderCall(args, theme) {
 			const isSubagent = args.profile === "subagent";
-			const kind = isSubagent ? "AGENT" : "COMMAND";
+			const kind = isSubagent ? "AGENT" : "PROCESS";
 			const payload = isSubagent ? args.task : args.command;
 			const agent = isSubagent && args.agent ? ` [${escapeCommandForDisplay(args.agent)}]` : "";
 			const title = theme.fg("muted", `${theme.bold(`babysit_run ${kind}`)}${agent}`);
@@ -4895,6 +4921,7 @@ export default function (pi: ExtensionAPI) {
 			"Do NOT poll this while merely waiting for a process to end — the exit notification is automatic.",
 		promptSnippet: "Check status/progress of babysit sessions (processes and subagents)",
 		parameters: Type.Object({
+			description: descriptionSchema,
 			id: Type.Optional(Type.String({ description: "Session id. Omit to list sessions." })),
 			state: Type.Optional(
 				StringEnum(["running", "terminal", "all"] as const, {
@@ -5135,11 +5162,12 @@ export default function (pi: ExtensionAPI) {
 		description:
 			"Send input to a babysit session. Process: `text` types a line into its stdin (PTY), " +
 			"`keys` presses named keys (Enter, Tab, Esc, Up/Down/Left/Right, C-c, F1…) — use with " +
-			"babysit_check { screen: true } to drive interactive programs. Subagent: `text` is " +
+			"babysit_check { description, screen: true } to drive interactive programs. Subagent: `text` is " +
 			"STEERING while it works, or a NEW TASK after the current task settles (mode: auto/steer/task) — this " +
 			"is how you resume a finished subagent with full context.",
 		promptSnippet: "Send text/keys to a process, or steering/follow-up tasks to a subagent",
 		parameters: Type.Object({
+			description: descriptionSchema,
 			id: Type.String({ description: "Session id." }),
 			text: Type.Optional(
 				Type.String({ description: "Text to send (a line for processes; a message for subagents)." }),
@@ -5211,7 +5239,7 @@ export default function (pi: ExtensionAPI) {
 					content: [
 						{
 							type: "text",
-							text: `Sent to ${params.id}. Read the reaction with babysit_check { id: "${params.id}"${params.keys?.length ? ", screen: true" : ""} } — don't expect the echo of your own input.`,
+							text: `Sent to ${params.id}. Read the reaction with babysit_check { description: "Inspect the response to input", id: "${params.id}"${params.keys?.length ? ", screen: true" : ""} } — don't expect the echo of your own input.`,
 						},
 					],
 					details: { kind: "process" },
@@ -5328,6 +5356,11 @@ export default function (pi: ExtensionAPI) {
 				// New task → new bookkeeping window, so check/wait track THIS task.
 				writeMeta(params.id, {
 					kind: "subagent",
+					name: meta?.name,
+					description: meta?.description,
+					launchCommand: meta?.launchCommand,
+					reapAfter: meta?.reapAfter,
+					startedAt: meta?.startedAt,
 					task: params.text,
 					// Start after the prompt-acceptance response. This excludes a
 					// preceding run that settled between `send` and RPC acceptance.
@@ -5389,6 +5422,7 @@ export default function (pi: ExtensionAPI) {
 			"the exit notification will resume you.",
 		promptSnippet: "Block until session(s) finish — process exit / output pattern / subagent task done",
 		parameters: Type.Object({
+			description: descriptionSchema,
 			id: Type.Optional(Type.String({ description: "Session id (single wait)." })),
 			ids: Type.Optional(
 				Type.Array(Type.String(), {
@@ -5535,7 +5569,10 @@ export default function (pi: ExtensionAPI) {
 		label: "Babysit: kill",
 		description: "Terminate a babysit session (process or subagent).",
 		promptSnippet: "Terminate a babysit session",
-		parameters: Type.Object({ id: Type.String({ description: "Session id." }) }),
+		parameters: Type.Object({
+			description: descriptionSchema,
+			id: Type.String({ description: "Session id." }),
+		}),
 		async execute(_id, params, _signal, _onUpdate, ctx) {
 			await requireBabysit();
 			// Prevent the exit poller racing a requested kill, but restore delivery

package/log-viewer.ts

@@ -244,21 +244,18 @@ export function createLogViewer(options: LogViewerOptions) {
 	const heading = {
 		invalidate() {},
 		render(width: number) {
-			const close = truncateToWidth("[Close]", width);
-			const room = Math.max(0, width - visibleWidth(close) - 1);
-			const title = truncateToWidth(sanitize(header).replace(/\n/g, " "), room);
+			const title = truncateToWidth(sanitize(header).replace(/\n/g, " "), width);
 			return [
 				options.getTheme().fg("accent", title) +
 					" ".repeat(
-						Math.max(0, width - visibleWidth(title) - visibleWidth(close)),
-					) +
-					options.getTheme().fg("muted", close),
+						Math.max(0, width - visibleWidth(title)),
+					),
 			];
 		},
 		handleMouse(e: TuiMouseEvent) {
 			if (
 				e.button !== "left" ||
-				e.x < Math.max(0, e.width - 7) ||
+				e.x < 0 ||
 				e.x >= e.width
 			)
 				return;
@@ -272,7 +269,7 @@ export function createLogViewer(options: LogViewerOptions) {
 	const footer: Component = {
 		invalidate() {},
 		render: (width) => [
-			options.getTheme().fg("dim", truncateToWidth(source ? `↑↓ · bytes ${pages[0]?.start ?? 0}–${pages.at(-1)?.end ?? 0}/${source.size} (window) · Esc close` : "↑↓ · Esc close", width)),
+			options.getTheme().fg("dim", truncateToWidth(source ? `↑↓ · bytes ${pages[0]?.start ?? 0}–${pages.at(-1)?.end ?? 0}/${source.size} (window)` : "↑↓", width)),
 		],
 	};
 	let geometry = {
@@ -489,10 +486,6 @@ export function createLogViewer(options: LogViewerOptions) {
 			options.requestRender();
 		},
 		handleKey(data: string): boolean {
-			if (matchesKey(data, Key.escape)) {
-				options.onClose();
-				return true;
-			}
 			if (matchesKey(data, Key.up)) scrollView.scrollBy(-1);
 			else if (matchesKey(data, Key.down)) scrollView.scrollBy(1);
 			else return false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yusukeshib/pi-babysit",
-  "version": "0.6.22",
+  "version": "0.6.24",
   "description": "Run any shell command and pi subagents under babysit, with context-safe captured output.",
   "keywords": [
     "pi-package",