pi-subagents 0.23.1 → 0.24.0

package/src/tui/render.ts

@@ -9,6 +9,8 @@ import { Container, Markdown, Spacer, Text, visibleWidth, type Component } from
 import {
 	type AgentProgress,
 	type AsyncJobState,
+	type AsyncJobStep,
+	type AsyncParallelGroupStatus,
 	type Details,
 	MAX_WIDGET_JOBS,
 	WIDGET_KEY,
@@ -16,6 +18,7 @@ import {
 import { formatTokens, formatUsage, formatDuration, formatToolCall, shortenPath } from "../shared/formatters.ts";
 import { getDisplayItems, getLastActivity, getSingleResultOutput } from "../shared/utils.ts";
 import { flatToLogicalStepIndex } from "../runs/background/parallel-groups.ts";
+import { aggregateStepStatus, formatActivityLabel, formatAgentRunningLabel, formatParallelOutcome } from "../shared/status-format.ts";
 
 type Theme = ExtensionContext["ui"]["theme"];
 
@@ -166,23 +169,6 @@ function getToolCallLines(
 	return result.toolCalls?.map((toolCall) => expanded ? toolCall.expandedText : toolCall.text) ?? [];
 }
 
-function formatActivityAge(ms: number): string {
-	if (ms < 1000) return "now";
-	if (ms < 60000) return `${Math.floor(ms / 1000)}s`;
-	return `${Math.floor(ms / 60000)}m`;
-}
-
-function formatActivityLabel(lastActivityAt: number | undefined, activityState?: AgentProgress["activityState"], now = Date.now()): string | undefined {
-	if (lastActivityAt === undefined) {
-		if (activityState === "needs_attention") return "needs attention";
-		if (activityState === "active_long_running") return "active but long-running";
-		return undefined;
-	}
-	const age = formatActivityAge(Math.max(0, now - lastActivityAt));
-	if (activityState === "needs_attention") return `no activity for ${age}`;
-	if (activityState === "active_long_running") return `active but long-running · last activity ${age} ago`;
-	return age === "now" ? "active now" : `active ${age} ago`;
-}
 
 function formatCurrentToolLine(progress: Pick<AgentProgress, "currentTool" | "currentToolArgs" | "currentToolStartedAt">, availableWidth: number, expanded: boolean): string | undefined {
 	if (!progress.currentTool) return undefined;
@@ -310,7 +296,7 @@ function widgetStatusGlyph(job: AsyncJobState, theme: Theme): string {
 	return theme.fg("error", "✗");
 }
 
-function widgetStepGlyph(status: string, theme: Theme): string {
+function widgetStepGlyph(status: AsyncJobStep["status"], theme: Theme): string {
 	if (status === "running") return theme.fg("accent", spinnerFrame());
 	if (status === "complete" || status === "completed") return theme.fg("success", "✓");
 	if (status === "failed") return theme.fg("error", "✗");
@@ -318,7 +304,7 @@ function widgetStepGlyph(status: string, theme: Theme): string {
 	return theme.fg("muted", "◦");
 }
 
-function widgetStepStatus(status: string, theme: Theme): string {
+function widgetStepStatus(status: AsyncJobStep["status"], theme: Theme): string {
 	if (status === "running") return theme.fg("accent", "running");
 	if (status === "complete" || status === "completed") return theme.fg("success", "complete");
 	if (status === "failed") return theme.fg("error", "failed");
@@ -340,48 +326,24 @@ function widgetStepActivity(step: NonNullable<AsyncJobState["steps"]>[number]):
 	return facts.join(" · ");
 }
 
-function widgetAggregateStepStatus(steps: NonNullable<AsyncJobState["steps"]>): string {
-	if (steps.some((step) => step.status === "running")) return "running";
-	if (steps.some((step) => step.status === "failed")) return "failed";
-	if (steps.some((step) => step.status === "paused")) return "paused";
-	if (steps.length > 0 && steps.every((step) => step.status === "complete" || step.status === "completed")) return "complete";
-	return "pending";
-}
-
-function widgetParallelOutcome(steps: NonNullable<AsyncJobState["steps"]>, total: number): string {
-	const running = steps.filter((step) => step.status === "running").length;
-	const done = steps.filter((step) => step.status === "complete" || step.status === "completed").length;
-	const failed = steps.filter((step) => step.status === "failed").length;
-	const paused = steps.filter((step) => step.status === "paused").length;
-	const parts = [`${done}/${total} done`];
-	if (running > 0) parts.unshift(formatAgentRunningLabel(running));
-	if (failed > 0) parts.push(`${failed} failed`);
-	if (paused > 0) parts.push(`${paused} paused`);
-	return parts.join(" · ");
-}
 
 function widgetChainDetails(job: AsyncJobState, theme: Theme, expanded = false, width = getTermWidth()): string[] {
 	if (!job.steps?.length) return [];
 	const total = job.chainStepCount ?? job.steps.length;
-	const groups = job.parallelGroups ?? [];
 	const lines: string[] = [];
-	let flatIndex = 0;
-	for (let stepIndex = 0; stepIndex < total; stepIndex++) {
-		const group = groups.find((candidate) => candidate.stepIndex === stepIndex);
-		if (group) {
-			const steps = job.steps.slice(group.start, group.start + group.count);
-			const status = widgetAggregateStepStatus(steps);
-			lines.push(`  ${widgetStepGlyph(status, theme)} Step ${stepIndex + 1}/${total}: ${themeBold(theme, "parallel group")} ${theme.fg("dim", "·")} ${theme.fg("dim", widgetParallelOutcome(steps, group.count))}`);
-			flatIndex = Math.max(flatIndex, group.start + group.count);
+	for (const span of buildAsyncChainStepSpans(total, job.steps.length, job.parallelGroups)) {
+		const steps = job.steps.slice(span.start, span.start + span.count);
+		if (span.isParallel) {
+			const status = aggregateStepStatus(steps);
+			lines.push(`  ${widgetStepGlyph(status, theme)} Step ${span.stepIndex + 1}/${total}: ${themeBold(theme, "parallel group")} ${theme.fg("dim", "·")} ${theme.fg("dim", formatParallelOutcome(steps, span.count))}`);
 			continue;
 		}
-		const step = job.steps[flatIndex];
+		const step = steps[0];
 		if (!step) {
-			lines.push(`  ${theme.fg("dim", `◦ Step ${stepIndex + 1}/${total}: pending`)}`);
+			lines.push(`  ${theme.fg("dim", `◦ Step ${span.stepIndex + 1}/${total}: pending`)}`);
 			continue;
 		}
-		lines.push(...foregroundStyleWidgetStepLines(job, theme, step, "Step", stepIndex + 1, total, expanded, width));
-		flatIndex++;
+		lines.push(...foregroundStyleWidgetStepLines(job, theme, step, "Step", span.stepIndex + 1, total, expanded, width));
 	}
 	return lines;
 }
@@ -434,8 +396,20 @@ function buildChainStepSpans(chainAgents: string[] | undefined): ChainStepSpan[]
 	return spans;
 }
 
-function formatAgentRunningLabel(count: number): string {
-	return count === 1 ? "1 agent running" : `${count} agents running`;
+function buildAsyncChainStepSpans(total: number, stepCount: number, parallelGroups: AsyncParallelGroupStatus[] = []): ChainStepSpan[] {
+	const spans: ChainStepSpan[] = [];
+	let flatIndex = 0;
+	for (let stepIndex = 0; stepIndex < total; stepIndex++) {
+		const group = parallelGroups.find((candidate) => candidate.stepIndex === stepIndex);
+		if (group) {
+			spans.push({ stepIndex, start: group.start, count: group.count, isParallel: true });
+			flatIndex = Math.max(flatIndex, group.start + group.count);
+			continue;
+		}
+		spans.push({ stepIndex, start: flatIndex, count: flatIndex < stepCount ? 1 : 0, isParallel: false });
+		flatIndex++;
+	}
+	return spans;
 }
 
 function isDoneResult(result: Details["results"][number]): boolean {
@@ -670,7 +644,7 @@ function buildSingleWidgetLines(job: AsyncJobState, theme: Theme, width: number,
 	const mode = widgetJobName(job);
 	const title = `async subagent ${mode}${count && count > 1 ? ` (${count})` : ""}`;
 	return [
-		`${theme.fg("toolTitle", themeBold(theme, title))} ${theme.fg("dim", "· background · /subagents-status")}`,
+		`${theme.fg("toolTitle", themeBold(theme, title))} ${theme.fg("dim", "· background")}`,
 		`${widgetStatusGlyph(job, theme)} ${themeBold(theme, mode)}${stats ? ` ${theme.fg("dim", "·")} ${stats}` : ""}`,
 		...foregroundStyleWidgetDetails(job, theme, expanded, width),
 	].map((line) => truncLine(line, width));
@@ -690,7 +664,7 @@ function compactSingleWidgetLines(job: AsyncJobState, theme: Theme, width: numbe
 		const activitySuffix = activity ? ` ${theme.fg("dim", "·")} ${theme.fg("dim", activity)}` : "";
 		lines.push(`  ${widgetStepGlyph(step.status, theme)} ${itemTitle} ${index + 1}/${total}: ${themeBold(theme, step.agent)} ${theme.fg("dim", "·")} ${status}${activitySuffix}${stepStats ? ` ${theme.fg("dim", "·")} ${stepStats}` : ""}`);
 	}
-	if (job.steps.some((step) => step.status === "running")) lines.push(theme.fg("accent", "  Press Ctrl+O for live detail · /subagents-status for output paths"));
+	if (job.steps.some((step) => step.status === "running")) lines.push(theme.fg("accent", "  Press Ctrl+O for live detail"));
 	return lines.map((line) => truncLine(line, width));
 }
 
@@ -703,8 +677,8 @@ function fitWidgetLineBudget(lines: string[], theme: Theme, width: number, expan
 	const visibleLines = Math.max(1, budget - 1);
 	const hiddenCount = lines.length - visibleLines;
 	const hint = expanded
-		? `… ${hiddenCount} live-detail lines hidden · /subagents-status for full detail`
-		: `… ${hiddenCount} lines hidden · Ctrl+O expands · /subagents-status for full detail`;
+		? `… ${hiddenCount} live-detail lines hidden`
+		: `… ${hiddenCount} lines hidden · Ctrl+O expands`;
 	return [...lines.slice(0, visibleLines), truncLine(theme.fg("dim", hint), width)];
 }
 
@@ -731,7 +705,7 @@ export function buildWidgetLines(jobs: AsyncJobState[], theme: Theme, width = ge
 
 	const lines: string[] = [];
 	const hasActive = running.length > 0 || queued.length > 0;
-	lines.push(truncLine(`${theme.fg(hasActive ? "accent" : "dim", hasActive ? "●" : "○")} ${theme.fg(hasActive ? "accent" : "dim", "Async agents")} ${theme.fg("dim", "· background · /subagents-status")}`, width));
+	lines.push(truncLine(`${theme.fg(hasActive ? "accent" : "dim", hasActive ? "●" : "○")} ${theme.fg(hasActive ? "accent" : "dim", "Async agents")} ${theme.fg("dim", "· background")}`, width));
 
 	const items: string[][] = [];
 	let hiddenRunning = 0;

package/src/agents/agent-templates.ts

@@ -1,60 +0,0 @@
-import type { AgentConfig } from "./agents.ts";
-
-export interface AgentTemplate {
-	name: string;
-	config: Partial<AgentConfig>;
-}
-
-export type TemplateItem =
-	| { type: "agent"; name: string; config: Partial<AgentConfig> }
-	| { type: "chain"; name: string; description: string }
-	| { type: "separator"; label: string };
-
-export const TEMPLATE_ITEMS: TemplateItem[] = [
-	{ type: "separator", label: "Agents" },
-	{
-		type: "agent",
-		name: "Blank",
-		config: { description: "Describe this agent", systemPrompt: "" },
-	},
-	{
-		type: "agent",
-		name: "Scout",
-		config: {
-			description: "Analyzes codebases and reports findings",
-			systemPrompt: "You are a code analysis agent. Given a codebase and a question, thoroughly investigate the relevant files and report your findings. Focus on accuracy — read the actual code rather than guessing.",
-			tools: ["read", "bash"],
-			output: "analysis.md",
-		},
-	},
-	{
-		type: "agent",
-		name: "Code Reviewer",
-		config: {
-			description: "Reviews code for bugs, style, and correctness",
-			systemPrompt: "You are a code review agent. Examine the code changes or files provided and identify bugs, style issues, performance concerns, and correctness problems. Be specific — cite line numbers and explain why each issue matters.",
-			tools: ["read", "bash"],
-		},
-	},
-	{
-		type: "agent",
-		name: "Planner",
-		config: {
-			description: "Creates implementation plans from requirements",
-			systemPrompt: "You are a planning agent. Given a task or requirements, create a detailed implementation plan. Break the work into concrete steps, identify which files need changes, and note any risks or dependencies.",
-			tools: ["read", "bash"],
-			output: "plan.md",
-		},
-	},
-	{
-		type: "agent",
-		name: "Implementer",
-		config: {
-			description: "Implements code changes from a plan",
-			systemPrompt: "You are an implementation agent. Given a plan or task, make the necessary code changes. Write clean, tested code that follows existing patterns. Run tests after making changes.",
-			defaultProgress: true,
-		},
-	},
-	{ type: "separator", label: "Chains" },
-	{ type: "chain", name: "Blank Chain", description: "Empty chain to configure" },
-];

package/src/manager-ui/agent-manager-chain-detail.ts

@@ -1,164 +0,0 @@
-import type { Theme } from "@mariozechner/pi-coding-agent";
-import { matchesKey, truncateToWidth } from "@mariozechner/pi-tui";
-import type { ChainConfig, ChainStepConfig } from "../agents/agents.ts";
-import { row, renderFooter, renderHeader, formatPath, formatScrollInfo } from "../tui/render-helpers.ts";
-import { isParallelStep, type ChainStep } from "../shared/settings.ts";
-
-export interface ChainDetailState {
-	scrollOffset: number;
-}
-
-export type ChainDetailAction =
-	| { type: "back" }
-	| { type: "launch" }
-	| { type: "edit" };
-
-const CHAIN_DETAIL_VIEWPORT_HEIGHT = 12;
-
-type DetailChainStep = ChainStepConfig | ChainStep;
-
-function buildDependencyMap(steps: DetailChainStep[]): Map<number, number[]> {
-	const outputMap = new Map<string, number>();
-	const deps = new Map<number, number[]>();
-	for (let i = 0; i < steps.length; i++) {
-		const step = steps[i]!;
-		if (isParallelStep(step as ChainStep)) {
-			const reads = step.parallel.flatMap((task) => Array.isArray(task.reads) ? task.reads : []);
-			const sources = reads
-				.map((file) => outputMap.get(file))
-				.filter((idx): idx is number => idx !== undefined);
-			if (sources.length > 0) deps.set(i, [...new Set(sources)]);
-			for (const task of step.parallel) {
-				if (typeof task.output === "string" && task.output.length > 0) outputMap.set(task.output, i);
-			}
-			continue;
-		}
-		if (typeof step.output === "string" && step.output.length > 0) outputMap.set(step.output, i);
-		if (Array.isArray(step.reads) && step.reads.length > 0) {
-			const sources = step.reads
-				.map((file) => outputMap.get(file))
-				.filter((idx): idx is number => idx !== undefined);
-			if (sources.length > 0) deps.set(i, sources);
-		}
-	}
-	return deps;
-}
-
-function buildChainDetailLines(chain: ChainConfig, width: number): string[] {
-	const contentWidth = width - 3;
-	const lines: string[] = [];
-	const steps = chain.steps as DetailChainStep[];
-	const dependencyMap = buildDependencyMap(steps);
-	lines.push(truncateToWidth(chain.description, contentWidth));
-	if (chain.packageName) {
-		lines.push(truncateToWidth(`Local name: ${chain.localName ?? chain.name}`, contentWidth));
-		lines.push(truncateToWidth(`Package: ${chain.packageName}`, contentWidth));
-	}
-	lines.push("");
-	lines.push(truncateToWidth(`File: ${formatPath(chain.filePath)}`, contentWidth));
-	lines.push("");
-	lines.push(truncateToWidth("── Flow ──", contentWidth));
-
-	for (let i = 0; i < steps.length; i++) {
-		const step = steps[i]!;
-		const sources = dependencyMap.get(i);
-		const fromText = sources && sources.length > 0 ? ` (from ${sources.map((s) => s + 1).join(", ")})` : "";
-		if (isParallelStep(step as ChainStep)) {
-			lines.push(truncateToWidth(`  ${i + 1}  Parallel: ${step.parallel.map((task) => task.agent).join(" + ")}`, contentWidth));
-			if (step.concurrency !== undefined) lines.push(truncateToWidth(`     concurrency: ${step.concurrency}`, contentWidth));
-			if (step.failFast !== undefined) lines.push(truncateToWidth(`     fail fast: ${step.failFast ? "on" : "off"}`, contentWidth));
-			if (step.worktree !== undefined) lines.push(truncateToWidth(`     worktree: ${step.worktree ? "on" : "off"}`, contentWidth));
-			for (let taskIndex = 0; taskIndex < step.parallel.length; taskIndex++) {
-				const task = step.parallel[taskIndex]!;
-				lines.push(truncateToWidth(`     ${taskIndex + 1}. ${task.agent}`, contentWidth));
-				const taskPreview = (task.task ?? "").split("\n")[0] ?? "";
-				if (taskPreview) lines.push(truncateToWidth(`        task: ${taskPreview}`, contentWidth));
-				if (Array.isArray(task.reads) && task.reads.length > 0) lines.push(truncateToWidth(`        ← reads: ${task.reads.join(", ")}${fromText}`, contentWidth));
-				else if (task.reads === false) lines.push(truncateToWidth("        ← reads: (disabled)", contentWidth));
-				if (typeof task.output === "string" && task.output.length > 0) lines.push(truncateToWidth(`        → output: ${task.output}`, contentWidth));
-				else if (task.output === false) lines.push(truncateToWidth("        → output: (disabled)", contentWidth));
-				if (task.model) lines.push(truncateToWidth(`        model: ${task.model}`, contentWidth));
-				if (task.skill !== undefined) {
-					const skillsText =
-						task.skill === false
-							? "(disabled)"
-							: Array.isArray(task.skill)
-								? (task.skill.length > 0 ? task.skill.join(", ") : "(none)")
-								: task.skill;
-					lines.push(truncateToWidth(`        skills: ${skillsText}`, contentWidth));
-				}
-				if (task.progress !== undefined) lines.push(truncateToWidth(`        progress: ${task.progress ? "on" : "off"}`, contentWidth));
-			}
-			lines.push("");
-			continue;
-		}
-		lines.push(truncateToWidth(`  ${i + 1}  ${step.agent}`, contentWidth));
-		const taskPreview = step.task.split("\n")[0] ?? "";
-		lines.push(truncateToWidth(`     task: ${taskPreview || "(none)"}`, contentWidth));
-		if (Array.isArray(step.reads) && step.reads.length > 0) {
-			lines.push(truncateToWidth(`     ← reads: ${step.reads.join(", ")}${fromText}`, contentWidth));
-		} else if (step.reads === false) {
-			lines.push(truncateToWidth("     ← reads: (disabled)", contentWidth));
-		}
-		if (typeof step.output === "string" && step.output.length > 0) {
-			lines.push(truncateToWidth(`     → output: ${step.output}`, contentWidth));
-		} else if (step.output === false) {
-			lines.push(truncateToWidth("     → output: (disabled)", contentWidth));
-		}
-		if (step.model) lines.push(truncateToWidth(`     model: ${step.model}`, contentWidth));
-		if (step.skills !== undefined) {
-			const skillsText =
-				step.skills === false
-					? "(disabled)"
-					: step.skills.length > 0
-						? step.skills.join(", ")
-						: "(none)";
-			lines.push(truncateToWidth(`     skills: ${skillsText}`, contentWidth));
-		}
-		if (step.progress !== undefined) {
-			lines.push(truncateToWidth(`     progress: ${step.progress ? "on" : "off"}`, contentWidth));
-		}
-		lines.push("");
-	}
-
-	if (chain.steps.length === 0) {
-		lines.push(truncateToWidth("(no steps)", contentWidth));
-	}
-
-	return lines;
-}
-
-export function handleChainDetailInput(state: ChainDetailState, data: string): ChainDetailAction | undefined {
-	if (matchesKey(data, "escape") || matchesKey(data, "ctrl+c")) return { type: "back" };
-	if (data === "l") return { type: "launch" };
-	if (data === "e") return { type: "edit" };
-	if (matchesKey(data, "up")) { state.scrollOffset--; return; }
-	if (matchesKey(data, "down")) { state.scrollOffset++; return; }
-	if (matchesKey(data, "pageup") || matchesKey(data, "shift+up")) { state.scrollOffset -= CHAIN_DETAIL_VIEWPORT_HEIGHT; return; }
-	if (matchesKey(data, "pagedown") || matchesKey(data, "shift+down")) { state.scrollOffset += CHAIN_DETAIL_VIEWPORT_HEIGHT; return; }
-	return;
-}
-
-export function renderChainDetail(
-	state: ChainDetailState,
-	chain: ChainConfig,
-	width: number,
-	theme: Theme,
-): string[] {
-	const lines: string[] = [];
-	const scopeBadge = chain.source === "user" ? "[user]" : "[proj]";
-	lines.push(renderHeader(` ${chain.name} [chain] ${scopeBadge} `, width, theme));
-	lines.push(row("", width, theme));
-
-	const contentLines = buildChainDetailLines(chain, width);
-	const maxOffset = Math.max(0, contentLines.length - CHAIN_DETAIL_VIEWPORT_HEIGHT);
-	state.scrollOffset = Math.max(0, Math.min(state.scrollOffset, maxOffset));
-	const visible = contentLines.slice(state.scrollOffset, state.scrollOffset + CHAIN_DETAIL_VIEWPORT_HEIGHT);
-	for (const line of visible) lines.push(row(` ${line}`, width, theme));
-	for (let i = visible.length; i < CHAIN_DETAIL_VIEWPORT_HEIGHT; i++) lines.push(row("", width, theme));
-
-	const scrollInfo = formatScrollInfo(state.scrollOffset, Math.max(0, contentLines.length - (state.scrollOffset + CHAIN_DETAIL_VIEWPORT_HEIGHT)));
-	lines.push(row(scrollInfo ? ` ${theme.fg("dim", scrollInfo)}` : "", width, theme));
-	lines.push(renderFooter(" [l]aunch  [e]dit  [↑↓] scroll  [esc] back ", width, theme));
-	return lines;
-}

package/src/manager-ui/agent-manager-detail.ts

@@ -1,235 +0,0 @@
-import type { Theme } from "@mariozechner/pi-coding-agent";
-import { matchesKey, truncateToWidth } from "@mariozechner/pi-tui";
-import type { AgentConfig } from "../agents/agents.ts";
-import { formatDuration } from "../shared/formatters.ts";
-import type { RunEntry } from "../runs/shared/run-history.ts";
-import { buildSkillInjection, resolveSkills } from "../agents/skills.ts";
-import { ensureCursorVisible, getCursorDisplayPos, renderEditor, wrapText } from "../tui/text-editor.ts";
-import type { TextEditorState } from "../tui/text-editor.ts";
-import { pad, row, renderHeader, renderFooter, formatPath, formatScrollInfo } from "../tui/render-helpers.ts";
-
-export interface DetailState {
-	resolved: boolean;
-	scrollOffset: number;
-	recentRuns?: RunEntry[];
-}
-
-export type DetailAction =
-	| { type: "back" }
-	| { type: "edit" }
-	| { type: "launch" };
-
-const DETAIL_VIEWPORT_HEIGHT = 12;
-
-function renderFieldLine(
-	label: string,
-	value: string,
-	width: number,
-	theme: Theme,
-): string {
-	const labelWidth = 12;
-	const labelText = theme.fg("dim", pad(label, labelWidth));
-	const available = Math.max(0, width - labelWidth);
-	return `${labelText}${truncateToWidth(value, available)}`;
-}
-
-function formatRelativeTime(ts: number): string {
-	const diff = Math.max(0, Math.floor(Date.now() / 1000) - ts);
-	if (diff < 60) return `${diff}s ago`;
-	if (diff < 3600) return `${Math.floor(diff / 60)}m ago`;
-	if (diff < 86400) return `${Math.floor(diff / 3600)}h ago`;
-	return `${Math.floor(diff / 86400)}d ago`;
-}
-
-function buildDetailLines(
-	agent: AgentConfig,
-	resolved: boolean,
-	recentRuns: RunEntry[] | undefined,
-	cwd: string,
-	width: number,
-	theme: Theme,
-): string[] {
-	const contentWidth = width - 3;
-	const lines: string[] = [];
-
-	const tools = agent.tools && agent.tools.length > 0 ? agent.tools.join(", ") : "(none)";
-	const mcp = agent.mcpDirectTools && agent.mcpDirectTools.length > 0 ? agent.mcpDirectTools.join(", ") : "(none)";
-	const skillsList = agent.skills && agent.skills.length > 0 ? agent.skills.join(", ") : "(none)";
-	const output = agent.output ?? "(none)";
-	const reads = agent.defaultReads && agent.defaultReads.length > 0 ? agent.defaultReads.join(", ") : "(none)";
-	const progress = agent.defaultProgress ? "on" : "off";
-	const defaultContext = agent.defaultContext ?? "auto";
-	const maxSubagentDepth = agent.maxSubagentDepth !== undefined ? String(agent.maxSubagentDepth) : "(default)";
-
-	if (agent.packageName) {
-		lines.push(renderFieldLine("Local name:", agent.localName ?? agent.name, contentWidth, theme));
-		lines.push(renderFieldLine("Package:", agent.packageName, contentWidth, theme));
-	}
-	lines.push(renderFieldLine("Model:", agent.model ?? "default", contentWidth, theme));
-	lines.push(renderFieldLine("Prompt mode:", agent.systemPromptMode, contentWidth, theme));
-	lines.push(renderFieldLine("Project ctx:", agent.inheritProjectContext ? "on" : "off", contentWidth, theme));
-	lines.push(renderFieldLine("Skills ctx:", agent.inheritSkills ? "on" : "off", contentWidth, theme));
-	lines.push(renderFieldLine("Run context:", defaultContext, contentWidth, theme));
-	if (agent.source === "builtin") {
-		lines.push(renderFieldLine("Disabled:", agent.disabled ? "on" : "off", contentWidth, theme));
-	}
-	if (agent.override) {
-		const overrideLabel = `${agent.override.scope} · ${formatPath(agent.override.path)}`;
-		lines.push(renderFieldLine("Override:", overrideLabel, contentWidth, theme));
-	}
-	lines.push(renderFieldLine("Thinking:", agent.thinking ?? "off", contentWidth, theme));
-	lines.push(renderFieldLine("Tools:", tools, contentWidth, theme));
-	lines.push(renderFieldLine("MCP:", mcp, contentWidth, theme));
-	lines.push(renderFieldLine("Skills:", skillsList, contentWidth, theme));
-	const extensionsList = agent.extensions !== undefined
-		? (agent.extensions.length > 0 ? agent.extensions.join(", ") : "(none)")
-		: "(all)";
-	lines.push(renderFieldLine("Extensions:", extensionsList, contentWidth, theme));
-	lines.push(renderFieldLine("Output:", output, contentWidth, theme));
-	lines.push(renderFieldLine("Reads:", reads, contentWidth, theme));
-	lines.push(renderFieldLine("Progress:", progress, contentWidth, theme));
-	lines.push(renderFieldLine("Max depth:", maxSubagentDepth, contentWidth, theme));
-
-	if (agent.extraFields) {
-		for (const [key, value] of Object.entries(agent.extraFields)) {
-			lines.push(truncateToWidth(`${key}: ${value}`, contentWidth));
-		}
-	}
-
-	lines.push("");
-	const sectionTitle = `── System Prompt (${resolved ? "resolved" : "raw"}) ──`;
-	lines.push(truncateToWidth(sectionTitle, contentWidth));
-
-	let prompt = agent.systemPrompt ?? "";
-	if (resolved) {
-		const { resolved: resolvedSkills } = resolveSkills(agent.skills ?? [], cwd);
-		const injection = buildSkillInjection(resolvedSkills);
-		if (injection) prompt = `${prompt}\n\n${injection}`;
-	}
-
-	const wrapped = wrapText(prompt, contentWidth);
-	lines.push(...wrapped.lines);
-	lines.push("");
-	lines.push(truncateToWidth("── Recent Runs ──", contentWidth));
-	if (!recentRuns || recentRuns.length === 0) {
-		lines.push(truncateToWidth("  (none)", contentWidth));
-		return lines;
-	}
-
-	for (const run of recentRuns) {
-		const when = pad(formatRelativeTime(run.ts), 8);
-		const status = run.status;
-		const task = truncateToWidth(`"${run.task}"`, 34);
-		const tail = run.status === "ok" ? formatDuration(run.duration) : `exit ${run.exit ?? 1}`;
-		lines.push(truncateToWidth(`  ${when} ${status} ${task} ${tail}`, contentWidth));
-	}
-
-	return lines;
-}
-
-export function handleDetailInput(state: DetailState, data: string): DetailAction | undefined {
-	if (matchesKey(data, "escape") || matchesKey(data, "ctrl+c")) return { type: "back" };
-	if (data === "e") return { type: "edit" };
-	if (data === "l") return { type: "launch" };
-	if (data === "v") { state.resolved = !state.resolved; state.scrollOffset = 0; return; }
-	if (matchesKey(data, "up")) { state.scrollOffset--; return; }
-	if (matchesKey(data, "down")) { state.scrollOffset++; return; }
-	if (matchesKey(data, "pageup") || matchesKey(data, "shift+up")) { state.scrollOffset -= DETAIL_VIEWPORT_HEIGHT; return; }
-	if (matchesKey(data, "pagedown") || matchesKey(data, "shift+down")) { state.scrollOffset += DETAIL_VIEWPORT_HEIGHT; return; }
-	return;
-}
-
-export function renderDetail(
-	state: DetailState,
-	agent: AgentConfig,
-	cwd: string,
-	width: number,
-	theme: Theme,
-): string[] {
-	const lines: string[] = [];
-	const scopeBadge = agent.source === "builtin"
-		? (agent.disabled
-			? (agent.override ? `[builtin off+${agent.override.scope}]` : "[builtin off]")
-			: (agent.override ? `[builtin+${agent.override.scope}]` : "[builtin]"))
-		: agent.source === "project"
-			? "[proj]"
-			: "[user]";
-	const headerText = ` ${agent.name} ${scopeBadge} ${formatPath(agent.filePath)} `;
-	lines.push(renderHeader(headerText, width, theme));
-	lines.push(row("", width, theme));
-
-	const contentLines = buildDetailLines(agent, state.resolved, state.recentRuns, cwd, width, theme);
-	const maxOffset = Math.max(0, contentLines.length - DETAIL_VIEWPORT_HEIGHT);
-	state.scrollOffset = Math.max(0, Math.min(state.scrollOffset, maxOffset));
-
-	const visible = contentLines.slice(state.scrollOffset, state.scrollOffset + DETAIL_VIEWPORT_HEIGHT);
-	for (const line of visible) {
-		lines.push(row(` ${line}`, width, theme));
-	}
-	for (let i = visible.length; i < DETAIL_VIEWPORT_HEIGHT; i++) {
-		lines.push(row("", width, theme));
-	}
-
-	const scrollInfo = formatScrollInfo(state.scrollOffset, Math.max(0, contentLines.length - (state.scrollOffset + DETAIL_VIEWPORT_HEIGHT)));
-	lines.push(row(scrollInfo ? ` ${theme.fg("dim", scrollInfo)}` : "", width, theme));
-
-	const footer = agent.source === "builtin"
-		? agent.override
-			? (agent.disabled
-				? " [e]dit override  [v] raw/resolved  [↑↓] scroll  [esc] back "
-				: " [l]aunch  [e]dit override  [v] raw/resolved  [↑↓] scroll  [esc] back ")
-			: (agent.disabled
-				? " [e]create override  [v] raw/resolved  [↑↓] scroll  [esc] back "
-				: " [l]aunch  [e]create override  [v] raw/resolved  [↑↓] scroll  [esc] back ")
-		: " [l]aunch  [e]dit  [v] raw/resolved  [↑↓] scroll  [esc] back ";
-	lines.push(renderFooter(footer, width, theme));
-	return lines;
-}
-
-export interface LaunchToggleState {
-	fork: boolean;
-	background: boolean;
-	worktree?: boolean;
-}
-
-export function renderTaskInput(
-	title: string,
-	editor: TextEditorState,
-	skipClarify: boolean,
-	width: number,
-	theme: Theme,
-	launchToggles?: LaunchToggleState,
-): string[] {
-	const lines: string[] = [];
-	lines.push(renderHeader(` ${title} `, width, theme));
-	lines.push(row("", width, theme));
-	lines.push(row(` ${theme.fg("dim", "Task:")}`, width, theme));
-
-	const innerW = width - 2;
-	const boxInnerWidth = Math.max(10, innerW - 4);
-	const top = `┌${"─".repeat(boxInnerWidth)}┐`;
-	const bottom = `└${"─".repeat(boxInnerWidth)}┘`;
-
-	lines.push(row(` ${top}`, width, theme));
-	const editorState = { ...editor };
-	const { starts } = wrapText(editorState.buffer, boxInnerWidth);
-	const cursorPos = getCursorDisplayPos(editorState.cursor, starts);
-	editorState.viewportOffset = ensureCursorVisible(cursorPos.line, 2, editorState.viewportOffset);
-	const editorLines = renderEditor(editorState, boxInnerWidth, 2);
-	for (const line of editorLines) {
-		lines.push(row(` │${pad(line, boxInnerWidth)}│`, width, theme));
-	}
-	lines.push(row(` ${bottom}`, width, theme));
-
-	lines.push(row("", width, theme));
-	const quickLabel = skipClarify ? "on" : "off";
-	const footerParts = ["[enter] run", `[tab] quick:${quickLabel}`];
-	if (launchToggles) {
-		footerParts.push(`[ctrl+f] fork:${launchToggles.fork ? "on" : "off"}`);
-		footerParts.push(`[ctrl+b] bg:${launchToggles.background ? "on" : "off"}`);
-		if (launchToggles.worktree !== undefined) footerParts.push(`[ctrl+w] worktree:${launchToggles.worktree ? "on" : "off"}`);
-	}
-	footerParts.push("[esc]");
-	lines.push(renderFooter(` ${footerParts.join("  ")} `, width, theme));
-	return lines;
-}