@bastani/atomic 0.9.11-alpha.7 → 0.9.11-alpha.9

package/dist/builtin/subagents/src/extension/doctor.ts

@@ -1,16 +1,16 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { discoverAgentsAll, type AgentSource } from "../agents/agents.ts";
-import { isAsyncAvailable } from "../runs/background/async-execution.ts";
-import { diagnoseIntercomBridge, type IntercomBridgeDiagnostic } from "../intercom/intercom-bridge.ts";
+import { type AgentSource, discoverAgentsAll } from "../agents/agents.ts";
 import { discoverAvailableSkills, type SkillSource } from "../agents/skills.ts";
+import { diagnoseIntercomBridge, type IntercomBridgeDiagnostic } from "../intercom/intercom-bridge.ts";
+import { isAsyncAvailable } from "../runs/background/async-execution.ts";
 import {
 	ASYNC_DIR,
 	CHAIN_RUNS_DIR,
-	RESULTS_DIR,
-	TEMP_ROOT_DIR,
 	type ExtensionConfig,
+	RESULTS_DIR,
 	type SubagentState,
+	TEMP_ROOT_DIR,
 } from "../shared/types.ts";
 
 interface DoctorPaths {
@@ -98,9 +98,7 @@ function formatSkillSourceCounts(skills: Array<{ source: SkillSource }>): string
 		"builtin",
 		"unknown",
 	];
-	const parts = ordered
-		.map((source) => `${source} ${counts.get(source) ?? 0}`)
-		.filter((part) => !part.endsWith(" 0"));
+	const parts = ordered.map((source) => `${source} ${counts.get(source) ?? 0}`).filter((part) => !part.endsWith(" 0"));
 	return parts.length > 0 ? parts.join(", ") : "none";
 }
 
@@ -135,10 +133,13 @@ function formatDiscovery(input: DoctorReportInput, deps: DoctorDeps): string[] {
 				user: discovered.user.length,
 				project: discovered.project.length,
 			};
-			const chainCounts = discovered.chains.reduce<Record<AgentSource, number>>((counts, chain) => {
-				counts[chain.source] += 1;
-				return counts;
-			}, { builtin: 0, user: 0, project: 0 });
+			const chainCounts = discovered.chains.reduce<Record<AgentSource, number>>(
+				(counts, chain) => {
+					counts[chain.source] += 1;
+					return counts;
+				},
+				{ builtin: 0, user: 0, project: 0 },
+			);
 			return [
 				`- agents: total ${agentCounts.builtin + agentCounts.user + agentCounts.project} (${formatSourceCounts(agentCounts)})`,
 				`- chains: total ${discovered.chains.length} (${formatSourceCounts(chainCounts)})`,
@@ -151,7 +152,10 @@ function formatDiscovery(input: DoctorReportInput, deps: DoctorDeps): string[] {
 	];
 }
 
-function formatIntercomDiagnostic(diagnostic: IntercomBridgeDiagnostic, context: "fresh" | "fork" | undefined): string[] {
+function formatIntercomDiagnostic(
+	diagnostic: IntercomBridgeDiagnostic,
+	context: "fresh" | "fork" | undefined,
+): string[] {
 	const lines = [
 		`- bridge: ${diagnostic.active ? "active" : "inactive"}${diagnostic.reason ? ` (${diagnostic.reason})` : ""}`,
 		`- mode: ${diagnostic.mode}; context: ${context ?? "unspecified"}`,
@@ -159,7 +163,9 @@ function formatIntercomDiagnostic(diagnostic: IntercomBridgeDiagnostic, context:
 		`- pi-intercom: ${diagnostic.piIntercomAvailable ? "available" : "unavailable"} at ${diagnostic.extensionDir}`,
 	];
 	if (diagnostic.configPath && diagnostic.intercomConfigEnabled !== undefined) {
-		lines.push(`- intercom config: ${diagnostic.intercomConfigEnabled === false ? "disabled" : "enabled or absent"} (${diagnostic.configPath})`);
+		lines.push(
+			`- intercom config: ${diagnostic.intercomConfigEnabled === false ? "disabled" : "enabled or absent"} (${diagnostic.configPath})`,
+		);
 	}
 	if (diagnostic.intercomConfigError) {
 		lines.push(`- intercom config warning: ${diagnostic.intercomConfigError}; runtime assumes enabled`);
@@ -188,12 +194,17 @@ export function buildDoctorReport(input: DoctorReportInput): string {
 		...formatDiscovery(input, deps),
 		"",
 		"Intercom bridge",
-		...lineFromCheck("intercom bridge", () => formatIntercomDiagnostic(deps.diagnoseIntercomBridge({
-			config: input.config.intercomBridge,
-			context: input.context,
-			orchestratorTarget: input.orchestratorTarget,
-			cwd: input.cwd,
-		}), input.context).join("\n")).split("\n"),
+		...lineFromCheck("intercom bridge", () =>
+			formatIntercomDiagnostic(
+				deps.diagnoseIntercomBridge({
+					config: input.config.intercomBridge,
+					context: input.context,
+					orchestratorTarget: input.orchestratorTarget,
+					cwd: input.cwd,
+				}),
+				input.context,
+			).join("\n"),
+		).split("\n"),
 	];
 	return lines.join("\n");
 }

package/dist/builtin/subagents/src/extension/fanout-child.ts

@@ -1,21 +1,25 @@
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
-
-import { APP_NAME, getEnvValue } from "@bastani/atomic";
 import type { ExtensionAPI, ToolDefinition } from "@bastani/atomic";
+import { APP_NAME, getEnvValue } from "@bastani/atomic";
 
 import { discoverAgents } from "../agents/agents.ts";
 import { resolveSubagentIntercomTarget } from "../intercom/intercom-bridge.ts";
 import { deliverSubagentIntercomMessageEvent } from "../intercom/result-intercom.ts";
 import { createSubagentExecutor, type SubagentParamsLike } from "../runs/foreground/subagent-executor.ts";
-import { readNestedControlRequests, resolveNestedRouteFromEnv, writeNestedControlResult, type NestedRoute } from "../runs/shared/nested-events.ts";
+import {
+	type NestedRoute,
+	readNestedControlRequests,
+	resolveNestedRouteFromEnv,
+	writeNestedControlResult,
+} from "../runs/shared/nested-events.ts";
 import { SUBAGENT_CHILD_ENV, SUBAGENT_FANOUT_CHILD_ENV } from "../runs/shared/pi-args.ts";
 import { getArtifactsDir } from "../shared/artifacts.ts";
-import { type Details, type SubagentState } from "../shared/types.ts";
+import type { Details, SubagentState } from "../shared/types.ts";
+import { beginApiLifecycle } from "./api-lifecycle.ts";
 import { loadConfig } from "./config.ts";
 import { SubagentParams } from "./schemas.ts";
-import { beginApiLifecycle } from "./api-lifecycle.ts";
 
 function getSubagentSessionRoot(parentSessionFile: string | null): string {
 	if (parentSessionFile) {
@@ -86,9 +90,11 @@ function dropControlRequestFile(filePath: string, requestId: string): void {
 }
 
 function shouldDropPendingResult(pending: PendingControlResult, pendingSize: number, now: number): boolean {
-	return pending.attempts >= MAX_PENDING_RESULT_ATTEMPTS
-		|| now - pending.firstFailureAt >= MAX_PENDING_RESULT_AGE_MS
-		|| pendingSize > MAX_PENDING_RESULTS;
+	return (
+		pending.attempts >= MAX_PENDING_RESULT_ATTEMPTS ||
+		now - pending.firstFailureAt >= MAX_PENDING_RESULT_AGE_MS ||
+		pendingSize > MAX_PENDING_RESULTS
+	);
 }
 
 function buildNestedControlResult(
@@ -152,7 +158,7 @@ export function startNestedControlInboxListener(pi: ExtensionAPI, state: Subagen
 				void (async () => {
 					try {
 						const pending = pendingResults.get(request.requestId);
-						const result = pending?.result ?? await buildNestedControlResult(pi, state, request);
+						const result = pending?.result ?? (await buildNestedControlResult(pi, state, request));
 						try {
 							writeNestedControlResult(route, result);
 						} catch (error) {
@@ -167,9 +173,15 @@ export function startNestedControlInboxListener(pi: ExtensionAPI, state: Subagen
 								pendingResults.delete(request.requestId);
 								rememberSeenRequest(seen, request.requestId);
 								dropControlRequestFile(request.filePath, request.requestId);
-								console.error(`Dropping nested control result for request '${request.requestId}' targeting '${request.targetRunId}' after ${nextPending.attempts} failed write attempts via inbox '${route.controlInbox}':`, error);
+								console.error(
+									`Dropping nested control result for request '${request.requestId}' targeting '${request.targetRunId}' after ${nextPending.attempts} failed write attempts via inbox '${route.controlInbox}':`,
+									error,
+								);
 							} else if (nextPending.attempts === 1) {
-								console.error(`Failed to write nested control result for request '${request.requestId}' targeting '${request.targetRunId}' via inbox '${route.controlInbox}'; keeping request for retry:`, error);
+								console.error(
+									`Failed to write nested control result for request '${request.requestId}' targeting '${request.targetRunId}' via inbox '${route.controlInbox}'; keeping request for retry:`,
+									error,
+								);
 							}
 							return;
 						}
@@ -182,7 +194,10 @@ export function startNestedControlInboxListener(pi: ExtensionAPI, state: Subagen
 				})();
 			}
 		} catch (error) {
-			console.error(`Failed to poll nested control inbox '${route.controlInbox}' for root '${route.rootRunId}':`, error);
+			console.error(
+				`Failed to poll nested control inbox '${route.controlInbox}' for root '${route.rootRunId}':`,
+				error,
+			);
 		}
 	}, 200);
 	timer.unref?.();

package/dist/builtin/subagents/src/extension/index.ts

@@ -2,35 +2,83 @@
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
+import {
+	APP_NAME,
+	type ExtensionAPI,
+	type ExtensionContext,
+	getEnvValue,
+	keyHintIfBound,
+	type ToolDefinition,
+} from "@bastani/atomic";
 import type { AgentToolResult } from "@earendil-works/pi-agent-core";
-import { APP_NAME, getEnvValue, keyHintIfBound } from "@bastani/atomic";
-import { type ExtensionAPI, type ExtensionContext, type ToolDefinition } from "@bastani/atomic";
-import { Box, Container, Spacer, Text, truncateToWidth, visibleWidth, wrapTextWithAnsi, type Component } from "@earendil-works/pi-tui";
+import {
+	Box,
+	type Component,
+	Container,
+	Spacer,
+	Text,
+	truncateToWidth,
+	visibleWidth,
+	wrapTextWithAnsi,
+} from "@earendil-works/pi-tui";
 import { discoverAgents } from "../agents/agents.ts";
-import { getArtifactsDir } from "../shared/artifacts.ts";
-import { resolveCurrentSessionId } from "../shared/session-identity.ts";
-import { advanceResultPulseFrame, renderLiveSubagentResult, renderSubagentResult, stopResultAnimations, stopWidgetAnimation, type SubagentResultRenderState } from "../tui/render.ts";
-import { SubagentParams } from "./schemas.ts";
-import { createSubagentExecutor, type SubagentParamsLike } from "../runs/foreground/subagent-executor.ts";
 import { createAsyncJobTracker } from "../runs/background/async-job-tracker.ts";
-import { registerSlashCommands } from "../slash/slash-commands.ts";
-import { registerPromptTemplateDelegationBridge } from "../slash/prompt-template-bridge.ts";
-import { registerSlashSubagentBridge } from "../slash/slash-bridge.ts";
-import { clearSlashSnapshots, getSlashRenderableSnapshot, resolveSlashMessageDetails, restoreSlashFinalSnapshots, type SlashMessageDetails } from "../slash/slash-live-state.ts";
 import registerSubagentNotify, { type SubagentNotifyDetails } from "../runs/background/notify.ts";
+import { createSubagentExecutor, type SubagentParamsLike } from "../runs/foreground/subagent-executor.ts";
 import { SUBAGENT_CHILD_ENV, SUBAGENT_FANOUT_CHILD_ENV } from "../runs/shared/pi-args.ts";
-import registerFanoutChildSubagentExtension from "./fanout-child.ts";
+import { getArtifactsDir } from "../shared/artifacts.ts";
 import { formatDuration, shortenPath } from "../shared/formatters.ts";
+import { resolveCurrentSessionId } from "../shared/session-identity.ts";
+import { registerPromptTemplateDelegationBridge } from "../slash/prompt-template-bridge.ts";
+import { registerSlashSubagentBridge } from "../slash/slash-bridge.ts";
+import { registerSlashCommands } from "../slash/slash-commands.ts";
+import {
+	clearSlashSnapshots,
+	getSlashRenderableSnapshot,
+	resolveSlashMessageDetails,
+	restoreSlashFinalSnapshots,
+	type SlashMessageDetails,
+} from "../slash/slash-live-state.ts";
+import {
+	advanceResultPulseFrame,
+	renderLiveSubagentResult,
+	renderSubagentResult,
+	type SubagentResultRenderState,
+	stopResultAnimations,
+	stopWidgetAnimation,
+} from "../tui/render.ts";
 import { loadConfig } from "./config.ts";
-import { DEFAULT_PROMPT_GUIDANCE } from "./prompt-guidance.ts";
+import registerFanoutChildSubagentExtension from "./fanout-child.ts";
 import { parseSubagentNotifyContent } from "./notification-content.ts";
+import { DEFAULT_PROMPT_GUIDANCE } from "./prompt-guidance.ts";
+import { SubagentParams } from "./schemas.ts";
 import { SUBAGENT_TOOL_DESCRIPTION } from "./tool-description.ts";
+
 export { SUBAGENT_TOOL_DESCRIPTION } from "./tool-description.ts";
-import { type Details, type SubagentState, ASYNC_DIR, DEFAULT_ARTIFACT_CONFIG, RESULTS_DIR, SLASH_RESULT_TYPE, SUBAGENT_ASYNC_COMPLETE_EVENT, SUBAGENT_ASYNC_STARTED_EVENT, SUBAGENT_CONTROL_EVENT } from "../shared/types.ts";
-import { clearPendingForegroundControlNotices, formatSubagentControlNotice, handleSubagentControlNotice, SUBAGENT_CONTROL_MESSAGE_TYPE, type SubagentControlMessageDetails } from "./control-notices.ts";
-import { createSubagentStartupMaintenance } from "./startup-maintenance.ts";
+
+import {
+	ASYNC_DIR,
+	DEFAULT_ARTIFACT_CONFIG,
+	type Details,
+	RESULTS_DIR,
+	SLASH_RESULT_TYPE,
+	SUBAGENT_ASYNC_COMPLETE_EVENT,
+	SUBAGENT_ASYNC_STARTED_EVENT,
+	SUBAGENT_CONTROL_EVENT,
+	type SubagentState,
+} from "../shared/types.ts";
 import { beginApiLifecycle, getApiScopedSet } from "./api-lifecycle.ts";
+import {
+	clearPendingForegroundControlNotices,
+	formatSubagentControlNotice,
+	handleSubagentControlNotice,
+	SUBAGENT_CONTROL_MESSAGE_TYPE,
+	type SubagentControlMessageDetails,
+} from "./control-notices.ts";
+import { createSubagentStartupMaintenance } from "./startup-maintenance.ts";
+
 export { loadConfig } from "./config.ts";
+
 function getSubagentSessionRoot(parentSessionFile: string | null): string {
 	if (parentSessionFile) {
 		const baseName = path.basename(parentSessionFile, ".jsonl");
@@ -49,19 +97,22 @@ function ensureAccessibleDir(dirPath: string): void {
 	} catch {
 		try {
 			fs.rmSync(dirPath, { recursive: true, force: true });
-		} catch {
-		}
+		} catch {}
 		fs.mkdirSync(dirPath, { recursive: true });
 		fs.accessSync(dirPath, fs.constants.R_OK | fs.constants.W_OK);
 	}
 }
 function isSlashResultRunning(result: { details?: Details }): boolean {
-	return result.details?.progress?.some((entry) => entry.status === "running")
-		|| result.details?.results.some((entry) => entry.progress?.status === "running")
-		|| false;
+	return (
+		result.details?.progress?.some((entry) => entry.status === "running") ||
+		result.details?.results.some((entry) => entry.progress?.status === "running") ||
+		false
+	);
 }
 function isSlashResultError(result: { details?: Details }): boolean {
-	return result.details?.results.some((entry) => entry.exitCode !== 0 && entry.progress?.status !== "running") || false;
+	return (
+		result.details?.results.some((entry) => entry.exitCode !== 0 && entry.progress?.status !== "running") || false
+	);
 }
 type SubagentToolRenderState = SubagentResultRenderState;
 function rebuildSlashResultContainer(
@@ -72,7 +123,11 @@ function rebuildSlashResultContainer(
 ): void {
 	container.clear();
 	container.addChild(new Spacer(1));
-	const boxTheme = isSlashResultRunning(result) ? "toolPendingBg" : isSlashResultError(result) ? "toolErrorBg" : "toolSuccessBg";
+	const boxTheme = isSlashResultRunning(result)
+		? "toolPendingBg"
+		: isSlashResultError(result)
+			? "toolErrorBg"
+			: "toolSuccessBg";
 	const box = new Box(1, 1, (text: string) => theme.bg(boxTheme, text));
 	box.addChild(renderSubagentResult(result, options, theme));
 	container.addChild(box);
@@ -93,7 +148,12 @@ function createSlashResultComponent(
 			lastVersion = snapshot.version;
 			lastSnapshotNow = Date.now();
 			pulseFrame = advanceResultPulseFrame(pulseFrame);
-			rebuildSlashResultContainer(container, snapshot.result, { ...options, now: lastSnapshotNow, pulseFrame }, theme);
+			rebuildSlashResultContainer(
+				container,
+				snapshot.result,
+				{ ...options, now: lastSnapshotNow, pulseFrame },
+				theme,
+			);
 		}
 		return Container.prototype.render.call(container, width);
 	};
@@ -107,16 +167,18 @@ export function renderSubagentNotification(
 	const content = typeof message.content === "string" ? message.content : "";
 	const details = (message.details as SubagentNotifyDetails | undefined) ?? parseSubagentNotifyContent(content);
 	if (!details) return new Text(content, 0, 0);
-	const icon = details.status === "completed"
-		? theme.fg("success", "✓")
-		: details.status === "paused"
-			? theme.fg("warning", "■")
-			: theme.fg("error", "✗");
+	const icon =
+		details.status === "completed"
+			? theme.fg("success", "✓")
+			: details.status === "paused"
+				? theme.fg("warning", "■")
+				: theme.fg("error", "✗");
 	const parts: string[] = [];
 	if (details.taskInfo) parts.push(details.taskInfo);
 	if (details.durationMs !== undefined) parts.push(formatDuration(details.durationMs));
 	let text = `${icon} ${theme.bold(details.agent)} ${theme.fg("dim", details.status)}`;
-	if (parts.length > 0) text += ` ${theme.fg("dim", "·")} ${parts.map((part) => theme.fg("dim", part)).join(` ${theme.fg("dim", "·")} `)}`;
+	if (parts.length > 0)
+		text += ` ${theme.fg("dim", "·")} ${parts.map((part) => theme.fg("dim", part)).join(` ${theme.fg("dim", "·")} `)}`;
 	const trimmedPreview = details.resultPreview.trim();
 	const previewLines = options.expanded
 		? trimmedPreview.split("\n").filter((line) => line.trim())
@@ -201,7 +263,8 @@ export default function registerSubagentExtension(pi: ExtensionAPI): void {
 		registrationFailureCleanups.push(() => maintenance.stop());
 		maintenance.startResultWatcherDeferred();
 		maintenance.primeExistingResultsDeferred();
-		const { ensurePoller, handleStarted, handleComplete, resetJobs, hydrateActiveJobsDeferred } = createAsyncJobTracker(pi, state, ASYNC_DIR);
+		const { ensurePoller, handleStarted, handleComplete, resetJobs, hydrateActiveJobsDeferred } =
+			createAsyncJobTracker(pi, state, ASYNC_DIR);
 		const executor = createSubagentExecutor({
 			pi,
 			state,
@@ -218,13 +281,25 @@ export default function registerSubagentExtension(pi: ExtensionAPI): void {
 			return createSlashResultComponent(details, options, theme, pi);
 		});
 		pi.registerMessageRenderer<SubagentNotifyDetails>("subagent-notify", renderSubagentNotification);
-		pi.registerMessageRenderer<SubagentControlMessageDetails>(SUBAGENT_CONTROL_MESSAGE_TYPE, (message, _options, theme) => {
-			const details = message.details as SubagentControlMessageDetails | undefined;
-			if (!details?.event) return undefined;
-			const content = typeof message.content === "string" ? message.content : undefined;
-			return new SubagentControlNoticeComponent({ ...details, noticeText: formatSubagentControlNotice(details, content) }, theme);
-		});
-		const executeSubagentCollapsed = (id: string, params: SubagentParamsLike, signal: AbortSignal, onUpdate: ((result: AgentToolResult<Details>) => void) | undefined, ctx: ExtensionContext) => {
+		pi.registerMessageRenderer<SubagentControlMessageDetails>(
+			SUBAGENT_CONTROL_MESSAGE_TYPE,
+			(message, _options, theme) => {
+				const details = message.details as SubagentControlMessageDetails | undefined;
+				if (!details?.event) return undefined;
+				const content = typeof message.content === "string" ? message.content : undefined;
+				return new SubagentControlNoticeComponent(
+					{ ...details, noticeText: formatSubagentControlNotice(details, content) },
+					theme,
+				);
+			},
+		);
+		const executeSubagentCollapsed = (
+			id: string,
+			params: SubagentParamsLike,
+			signal: AbortSignal,
+			onUpdate: ((result: AgentToolResult<Details>) => void) | undefined,
+			ctx: ExtensionContext,
+		) => {
 			if (ctx.hasUI) {
 				state.lastUiContext = ctx;
 				ctx.ui.setToolsExpanded(false);
@@ -234,8 +309,7 @@ export default function registerSubagentExtension(pi: ExtensionAPI): void {
 		const slashBridge = registerSlashSubagentBridge({
 			events: pi.events,
 			getContext: () => state.lastUiContext,
-			execute: (id, params, signal, onUpdate, ctx) =>
-				executeSubagentCollapsed(id, params, signal, onUpdate, ctx),
+			execute: (id, params, signal, onUpdate, ctx) => executeSubagentCollapsed(id, params, signal, onUpdate, ctx),
 		});
 		registrationFailureCleanups.push(() => {
 			slashBridge.cancelAll();
@@ -283,7 +357,8 @@ export default function registerSubagentExtension(pi: ExtensionAPI): void {
 		function effectiveParallelTaskCount(tasks: Array<{ count?: unknown }> | undefined): number {
 			if (!tasks || tasks.length === 0) return 0;
 			return tasks.reduce((total, task) => {
-				const count = typeof task.count === "number" && Number.isInteger(task.count) && task.count >= 1 ? task.count : 1;
+				const count =
+					typeof task.count === "number" && Number.isInteger(task.count) && task.count >= 1 ? task.count : 1;
 				return total + count;
 			}, 0);
 		}
@@ -302,7 +377,8 @@ export default function registerSubagentExtension(pi: ExtensionAPI): void {
 					const target = args.agent || args.chainName || "";
 					return new Text(
 						`${theme.fg("toolTitle", theme.bold("subagent "))}${args.action}${target ? ` ${theme.fg("accent", target)}` : ""}`,
-						0, 0,
+						0,
+						0,
 					);
 				}
 				const isParallel = (args.tasks?.length ?? 0) > 0;

package/dist/builtin/subagents/src/extension/prompt-guidance.ts

@@ -9,4 +9,3 @@ export const DEFAULT_PROMPT_GUIDANCE: string[] = [
   - If an agent has no declared model or fallback policy, consult packages/coding-agent/docs/models/model-selection.md for the role's Pareto-efficient recommendations, then use workflow({ action: "models" }) when available to inspect the configured catalog. Pin only a returned fullId and only append a thinking level listed for that entry. If the catalog tool is unavailable, the catalog is empty, or no recommended model is present, leave the child unpinned and state the limitation rather than inventing a model. Do not inspect credentials or infer why a model is absent.
   - Workflow stages automatically receive their invocation-scoped Intercom group, and delegated children inherit the launching session's group across single, parallel, chain, async, and follow-up work. Do not create or propagate group identifiers for ordinary workflow-stage delegation; use an explicit group only for an intentional topology override. Keep contact_supervisor available for cross-group escalation.`,
 ];
-