@narumitw/pi-subagents 2.1.3 → 3.0.0

package/src/inspect-registration.ts

@@ -1,64 +0,0 @@
-import type { ExtensionAPI, ToolDefinition } from "@earendil-works/pi-coding-agent";
-import { cachedModuleLoader, throwIfAborted } from "./cached-module-loader.js";
-import type { SubagentInspectRuntime } from "./inspect.js";
-import { renderInspectCall, renderInspectResult } from "./inspect-render.js";
-import { SubagentInspectParams } from "./inspect-tool.js";
-
-interface InspectExecutionModule {
-	executeSubagentInspect: typeof import("./inspect.js").executeSubagentInspect;
-}
-
-export interface InspectRegistrationDependencies {
-	loadExecution?: () => Promise<InspectExecutionModule>;
-}
-
-export function registerSubagentInspect(
-	pi: ExtensionAPI,
-	runtime: SubagentInspectRuntime,
-	dependencies: InspectRegistrationDependencies = {},
-): void {
-	const loadExecution = cachedModuleLoader(
-		dependencies.loadExecution ?? (() => import("./inspect.js")),
-	);
-	let lifecycleGeneration = 0;
-	pi.on("session_start", () => {
-		lifecycleGeneration += 1;
-	});
-	pi.on("session_shutdown", () => {
-		lifecycleGeneration += 1;
-	});
-	const definition: ToolDefinition<typeof SubagentInspectParams, Record<string, unknown>> = {
-		name: "subagent_inspect",
-		label: "Inspect Subagents",
-		description:
-			"Inspect available subagent definitions, models, retained runs, persisted blocking workflows, runtime status, and diagnostics without changing subagent or workspace state. This tool never starts a child, sends or acknowledges messages, interrupts or closes runs, changes settings, or modifies files.",
-		promptSnippet: "Inspect subagent metadata and runtime state without changing it",
-		parameters: SubagentInspectParams,
-		async execute(_toolCallId, params, signal, _onUpdate, ctx) {
-			const ownerGeneration = lifecycleGeneration;
-			throwIfAborted(signal, "Subagent inspection loading was cancelled");
-			let executionModule: InspectExecutionModule;
-			try {
-				executionModule = await loadExecution();
-			} catch (error) {
-				throwIfAborted(signal, "Subagent inspection loading was cancelled");
-				if (ownerGeneration !== lifecycleGeneration) {
-					throw new DOMException("Subagent inspection session was replaced", "AbortError");
-				}
-				throw error;
-			}
-			throwIfAborted(signal, "Subagent inspection loading was cancelled");
-			if (ownerGeneration !== lifecycleGeneration) {
-				throw new DOMException("Subagent inspection session was replaced", "AbortError");
-			}
-			return executionModule.executeSubagentInspect(params, ctx, runtime);
-		},
-		renderCall(args, theme) {
-			return renderInspectCall(args, theme);
-		},
-		renderResult(result, options, theme, context) {
-			return renderInspectResult(result, options, theme, context);
-		},
-	};
-	pi.registerTool(definition);
-}

package/src/inspect-render.ts

@@ -1,334 +0,0 @@
-import type { AgentToolResult } from "@earendil-works/pi-agent-core";
-import type { Theme, ToolRenderResultOptions } from "@earendil-works/pi-coding-agent";
-import { Text } from "@earendil-works/pi-tui";
-import type { SubagentInspectParams } from "./inspect.js";
-import {
-	booleanValue,
-	COLLAPSED_LIST_LIMIT,
-	expansionHint,
-	numberValue,
-	recordList,
-	recordValue,
-	renderFallbackResult,
-	safeBlock,
-	safeLine,
-	statusBadge,
-	stringValue,
-	type ToolRendererContext,
-	toolHeader,
-} from "./render-common.js";
-
-export function renderInspectCall(args: Partial<SubagentInspectParams>, theme: Theme) {
-	const action = safeLine(args.action, "...", 128);
-	const metadata: string[] = [];
-	if (args.agentScope) metadata.push(`[${args.agentScope}]`);
-	if (args.agent) metadata.push(`agent:${safeLine(args.agent, "", 256)}`);
-	if (args.agentId) metadata.push(`id:${safeLine(args.agentId, "", 256)}`);
-	if (args.workflowId) metadata.push(`workflow:${safeLine(args.workflowId, "", 256)}`);
-	if (args.includeClosed) metadata.push("include closed");
-	return new Text(toolHeader(theme, "subagent_inspect", action, metadata), 0, 0);
-}
-
-export function renderInspectResult(
-	result: AgentToolResult<Record<string, unknown>>,
-	options: ToolRenderResultOptions,
-	theme: Theme,
-	context: ToolRendererContext<SubagentInspectParams>,
-) {
-	const details = recordValue(result.details);
-	const action = stringValue(details?.action);
-	if (!details || !action) return renderFallbackResult(result, options, theme, context.isError);
-
-	const rendered = renderAction(action, details, options.expanded, theme);
-	if (!rendered) return renderFallbackResult(result, options, theme, context.isError);
-	return new Text(rendered, 0, 0);
-}
-
-function renderAction(
-	action: string,
-	details: Record<string, unknown>,
-	expanded: boolean,
-	theme: Theme,
-): string | undefined {
-	switch (action) {
-		case "list_agents":
-			return renderList("agent", recordList(details.agents), details, expanded, theme, formatAgent);
-		case "get_agent": {
-			const agent = recordValue(details.agent);
-			if (!agent) return undefined;
-			const lines = [
-				`${statusBadge(theme, "completed")} · ${theme.fg("toolTitle", theme.bold(safeLine(agent.name, "agent", 256)))}${theme.fg("muted", ` (${safeLine(agent.source, "unknown", 128)})`)}`,
-				formatAgent(agent, theme, true),
-			];
-			if (!expanded) lines.push(expansionHint());
-			return lines.join("\n");
-		}
-		case "list_runs":
-			return renderList("run", recordList(details.runs), details, expanded, theme, formatRun);
-		case "get_run": {
-			const run = recordValue(details.run);
-			if (!run) return undefined;
-			const lines = [
-				`${statusBadge(theme, "completed")} · run ${theme.fg("accent", safeLine(run.id, "run", 256))}`,
-				formatRun(run, theme, true),
-			];
-			if (!expanded) lines.push(expansionHint());
-			return lines.join("\n");
-		}
-		case "list_workflows":
-			return renderList(
-				"workflow",
-				recordList(details.workflows),
-				details,
-				expanded,
-				theme,
-				formatWorkflow,
-			);
-		case "get_workflow": {
-			const workflow = recordValue(details.workflow);
-			if (!workflow) return undefined;
-			return [
-				`${statusBadge(theme, "completed")} · workflow ${theme.fg("accent", safeLine(workflow.workflowId, "workflow", 256))}`,
-				formatWorkflow(workflow, theme, expanded),
-			].join("\n");
-		}
-		case "list_models":
-			return renderList(
-				"model",
-				recordList(details.models),
-				details,
-				expanded,
-				theme,
-				formatModel,
-				stringValue(details.source),
-			);
-		case "preview_context": {
-			const preview = recordValue(details.preview);
-			if (!preview) return undefined;
-			return [
-				`${statusBadge(theme, "completed")} · context ${theme.fg("accent", safeLine(preview.mode, "none", 128))}`,
-				theme.fg(
-					"dim",
-					`${numberValue(preview.turns)} turns · ${numberValue(preview.sourceCount)} sources · ${numberValue(preview.bytes)} bytes${preview.truncated === true ? " · truncated" : ""}`,
-				),
-			].join("\n");
-		}
-		case "status":
-			return renderStatus(details, expanded, theme);
-		case "diagnose":
-			return renderDiagnose(details, expanded, theme);
-		default:
-			return undefined;
-	}
-}
-
-function renderList(
-	label: string,
-	items: Record<string, unknown>[],
-	details: Record<string, unknown>,
-	expanded: boolean,
-	theme: Theme,
-	format: (item: Record<string, unknown>, theme: Theme, expanded: boolean) => string,
-	source = "",
-): string {
-	const returned = numberValue(details.returned, items.length);
-	const omitted = numberValue(details.omitted);
-	const noun = `${label}${returned === 1 ? "" : "s"}`;
-	const lines = [
-		`${statusBadge(theme, "completed")} · ${returned} ${noun}${source ? theme.fg("muted", ` · ${safeLine(source, "", 256)}`) : ""}`,
-	];
-	const selected = expanded ? items : items.slice(0, COLLAPSED_LIST_LIMIT);
-	for (const item of selected) lines.push(format(item, theme, expanded));
-	const hidden = Math.max(0, items.length - selected.length) + omitted;
-	if (hidden > 0) lines.push(theme.fg("muted", `… ${hidden} omitted`));
-	if (items.length === 0) lines.push(theme.fg("muted", `(no ${label}s)`));
-	if (!expanded && (items.length > 0 || omitted > 0)) lines.push(expansionHint());
-	return lines.join("\n");
-}
-
-function formatAgent(agent: Record<string, unknown>, theme: Theme, expanded: boolean): string {
-	const name = safeLine(agent.name, "agent", 256);
-	const source = safeLine(agent.source, "unknown", 128);
-	const model = stringValue(agent.model);
-	const toolCount = typeof agent.toolCount === "number" ? agent.toolCount : undefined;
-	const description = safeBlock(agent.description, "", 512).trim();
-	const lines = [
-		`${theme.fg("muted", "• ")}${theme.fg("accent", name)} ${theme.fg("muted", source)}${model ? theme.fg("dim", ` · ${safeLine(model, "", 256)}`) : ""}${toolCount !== undefined ? theme.fg("dim", ` · ${toolCount} tools`) : ""}`,
-	];
-	if (description) lines.push(`  ${theme.fg("toolOutput", description)}`);
-	if (expanded) {
-		const path = stringValue(agent.path);
-		const consultTools = stringList(agent.consultTools);
-		if (path) lines.push(`  ${theme.fg("dim", `path: ${safeLine(path, "", 2 * 1024)}`)}`);
-		lines.push(`  ${theme.fg("dim", `consult tools: ${consultTools.join(", ") || "none"}`)}`);
-	}
-	return lines.join("\n");
-}
-
-function formatRun(run: Record<string, unknown>, theme: Theme, expanded: boolean): string {
-	const id = safeLine(run.id, "run", 256);
-	const agent = safeLine(run.agent, "agent", 256);
-	const state = safeLine(run.state, "unknown", 128);
-	const unread = numberValue(run.unreadMessages);
-	const lines = [
-		`${theme.fg("muted", "• ")}${theme.fg("accent", id)} ${theme.fg("toolOutput", agent)} ${theme.fg("muted", state)}${unread > 0 ? theme.fg("warning", ` · unread:${unread}`) : ""}`,
-	];
-	if (expanded) {
-		const thinking = stringValue(run.thinkingLevel);
-		const timeout = numberValue(run.currentTimeoutMs) || numberValue(run.timeoutMs);
-		const idleTimeout = numberValue(run.currentIdleTimeoutMs) || numberValue(run.idleTimeoutMs);
-		const maxTurns = numberValue(run.currentMaxTurns) || numberValue(run.maxTurns);
-		const maxToolCalls = numberValue(run.currentMaxToolCalls) || numberValue(run.maxToolCalls);
-		const task = safeBlock(run.currentTask, "", 2 * 1024).trim();
-		const error = safeBlock(run.error, "", 2 * 1024).trim();
-		lines.push(
-			`  ${theme.fg("dim", `${numberValue(run.historyCount)} history · ${thinking ? `thinking:${safeLine(thinking, "", 128)} · ` : ""}${timeout ? `timeout:${timeout}ms · ` : ""}${idleTimeout ? `idle:${idleTimeout}ms · ` : ""}${maxTurns ? `turns:${maxTurns} · ` : ""}${maxToolCalls ? `tools:${maxToolCalls} · ` : ""}${numberValue(run.children)} children`)}`,
-		);
-		const context = recordValue(run.context);
-		const telemetry = recordValue(run.telemetry);
-		if (context) {
-			lines.push(
-				`  ${theme.fg("dim", `context: ${numberValue(context.turns)} turns · ${numberValue(context.sources)} sources · ${numberValue(context.bytes)} bytes${context.truncated === true ? " · truncated" : ""}`)}`,
-			);
-		}
-		if (telemetry) {
-			lines.push(
-				`  ${theme.fg("dim", `transport: ${safeLine(telemetry.transport, "unknown", 128)} · phase:${safeLine(telemetry.phase, "unknown", 128)}${stringValue(telemetry.protocol) ? ` · ${safeLine(telemetry.protocol, "", 128)}` : ""}`)}`,
-			);
-		}
-		if (task) lines.push(`  ${theme.fg("dim", `task: ${task}`)}`);
-		if (error) lines.push(`  ${theme.fg("error", `error: ${error}`)}`);
-	}
-	return lines.join("\n");
-}
-
-function formatWorkflow(
-	workflow: Record<string, unknown>,
-	theme: Theme,
-	expanded: boolean,
-): string {
-	const id = safeLine(workflow.workflowId, "workflow", 256);
-	const states = recordValue(workflow.states);
-	const stateText = states
-		? Object.entries(states)
-				.map(([state, count]) => `${safeLine(state, "unknown", 128)}:${numberValue(count)}`)
-				.join(" · ")
-		: "";
-	const lines = [
-		`${theme.fg("muted", "• ")}${theme.fg("accent", id)} ${theme.fg("muted", `${numberValue(workflow.itemCount)} items · generation:${numberValue(workflow.generation)}`)}`,
-	];
-	if (stateText) lines.push(`  ${theme.fg("dim", stateText)}`);
-	if (expanded) {
-		for (const item of recordList(workflow.items).slice(0, COLLAPSED_LIST_LIMIT)) {
-			lines.push(
-				`  ${theme.fg("muted", "- ")}${theme.fg("toolOutput", safeLine(item.id, "task", 256))} ${theme.fg("dim", safeLine(item.state, "unknown", 128))}`,
-			);
-		}
-		const omitted = numberValue(workflow.omittedItems);
-		if (omitted > 0) lines.push(theme.fg("muted", `  … ${omitted} tasks omitted`));
-	}
-	return lines.join("\n");
-}
-
-function formatModel(model: Record<string, unknown>, theme: Theme, expanded: boolean): string {
-	const identity = `${safeLine(model.provider, "provider", 256)}/${safeLine(model.id, "model", 256)}`;
-	const current = booleanValue(model.current) ? theme.fg("success", " · current") : "";
-	const reasoning = booleanValue(model.reasoning) ? theme.fg("dim", " · reasoning") : "";
-	const lines = [`${theme.fg("muted", "• ")}${theme.fg("accent", identity)}${current}${reasoning}`];
-	if (expanded) {
-		const name = stringValue(model.name);
-		const thinking = stringValue(model.thinkingLevel);
-		if (name) lines.push(`  ${theme.fg("toolOutput", safeLine(name, "", 512))}`);
-		lines.push(
-			`  ${theme.fg("dim", `context:${numberValue(model.contextWindow)} · max:${numberValue(model.maxTokens)}${thinking ? ` · thinking:${safeLine(thinking, "", 128)}` : ""}`)}`,
-		);
-	}
-	return lines.join("\n");
-}
-
-function renderStatus(
-	details: Record<string, unknown>,
-	expanded: boolean,
-	theme: Theme,
-): string | undefined {
-	const status = recordValue(details.status);
-	const stateful = recordValue(status?.stateful);
-	if (!status || !stateful) return undefined;
-	const currentLimits = recordValue(status.statefulLimits) ?? recordValue(stateful.limits);
-	const lines = [
-		`${statusBadge(theme, "completed")} · runtime status`,
-		`${theme.fg("muted", "workflow: ")}${theme.fg("accent", safeLine(status.workflow, "unknown", 128))} · ${numberValue(stateful.activeAgents)} active · ${numberValue(stateful.retainedAgents)} retained`,
-		`${theme.fg("muted", "stateful: ")}${stateful.initialized === true ? "initialized" : "not initialized"} · resources: ${safeLine(status.consultResources, "unknown", 128)}`,
-	];
-	if (currentLimits) {
-		lines.push(`${theme.fg("muted", "limits: ")}${formatDetachedLimits(currentLimits)}`);
-	}
-	if (expanded) {
-		const delivery = stringValue(stateful.completionDelivery);
-		const transport = stringValue(stateful.transport);
-		if (transport || delivery) {
-			lines.push(
-				theme.fg(
-					"dim",
-					[
-						transport && `transport:${safeLine(transport)}`,
-						delivery && `delivery:${safeLine(delivery)}`,
-					]
-						.filter(Boolean)
-						.join(" · "),
-				),
-			);
-		}
-		const configured = recordValue(status.configuredStatefulLimits);
-		const sources = recordValue(status.configuredStatefulLimitSources);
-		if (configured) {
-			lines.push(theme.fg("dim", `configured: ${formatDetachedLimits(configured, sources)}`));
-		}
-	} else lines.push(expansionHint());
-	return lines.join("\n");
-}
-
-function formatDetachedLimits(
-	limits: Record<string, unknown>,
-	sources?: Record<string, unknown>,
-): string {
-	return [
-		["agents", "maxAgents"],
-		["active", "maxActiveTurns"],
-		["children", "maxChildrenPerAgent"],
-		["depth", "maxDepth"],
-		["stored", "maxStoredAgents"],
-	]
-		.map(([label, field]) => {
-			const source = stringValue(sources?.[field]);
-			return `${label}:${numberValue(limits[field])}${source ? ` (${safeLine(source, "", 64)})` : ""}`;
-		})
-		.join(" · ");
-}
-
-function renderDiagnose(details: Record<string, unknown>, expanded: boolean, theme: Theme): string {
-	const checks = recordList(details.checks);
-	const hasFail = checks.some((check) => check.status === "fail");
-	const hasWarning = checks.some((check) => check.status === "warning");
-	const overall = hasFail ? "failed" : hasWarning ? "warnings" : "passed";
-	const status = hasFail ? "failed" : hasWarning ? "warning" : "completed";
-	const lines = [`${statusBadge(theme, status)} · Diagnostics ${overall}`];
-	const selected = expanded ? checks : checks.slice(0, COLLAPSED_LIST_LIMIT);
-	for (const check of selected) {
-		const checkStatus = safeLine(check.status, "unknown", 64);
-		const color =
-			checkStatus === "fail" ? "error" : checkStatus === "warning" ? "warning" : "success";
-		lines.push(
-			`${theme.fg(color, checkStatus.toUpperCase())} ${theme.fg("accent", safeLine(check.name, "check", 256))} · ${theme.fg("toolOutput", safeBlock(check.message, "", 2 * 1024))}`,
-		);
-	}
-	if (checks.length === 0) lines.push(theme.fg("muted", "(no diagnostic checks)"));
-	if (!expanded && checks.length > 0) lines.push(expansionHint());
-	return lines.join("\n");
-}
-
-function stringList(value: unknown): string[] {
-	return Array.isArray(value)
-		? value.flatMap((item) => (typeof item === "string" ? [safeLine(item, "", 256)] : []))
-		: [];
-}

package/src/inspect-tool.ts

@@ -1,45 +0,0 @@
-import { StringEnum } from "@earendil-works/pi-ai";
-import { type Static, Type } from "typebox";
-
-const INSPECT_ACTIONS = [
-	"list_agents",
-	"get_agent",
-	"list_runs",
-	"get_run",
-	"list_workflows",
-	"get_workflow",
-	"list_models",
-	"preview_context",
-	"status",
-	"diagnose",
-] as const;
-
-const AgentScopeSchema = StringEnum(["user", "project", "both"] as const, {
-	default: "user",
-	description: "Agent definition scope. Project scopes require a trusted project.",
-});
-const LimitSchema = Type.Number({ minimum: 1, maximum: 100, multipleOf: 1 });
-const ContextModeSchema = Type.Union([
-	StringEnum(["none", "all", "summary"] as const),
-	Type.Number({ minimum: 1, multipleOf: 1 }),
-]);
-
-export const SubagentInspectParams = Type.Object(
-	{
-		action: StringEnum(INSPECT_ACTIONS),
-		agent: Type.Optional(Type.String({ minLength: 1 })),
-		agentId: Type.Optional(
-			Type.String({ minLength: 1, description: "Retained agent ID or canonical task path." }),
-		),
-		workflowId: Type.Optional(Type.String({ minLength: 1, maxLength: 256 })),
-		agentScope: Type.Optional(AgentScopeSchema),
-		limit: Type.Optional(LimitSchema),
-		includeClosed: Type.Optional(Type.Boolean({ default: false })),
-		context: Type.Optional(ContextModeSchema),
-		contextEntryIds: Type.Optional(Type.Array(Type.String({ minLength: 1 }))),
-	},
-	{ additionalProperties: false },
-);
-
-export type SubagentInspectParams = Static<typeof SubagentInspectParams>;
-export { INSPECT_ACTIONS };