@bastani/atomic 0.9.10 → 0.9.11-alpha.10

package/dist/builtin/subagents/src/agents/agent-management-helpers.ts

@@ -1,15 +1,22 @@
+import { MAX_SUBAGENT_NESTING_DEPTH, type SubagentToolResult } from "../shared/types.ts";
+import type { ManagementContext, ManagementScope } from "./agent-management.ts";
 import type { AgentConfig, AgentScope, ChainConfig, ChainStepConfig } from "./agents.ts";
 import { discoverAgentsAll, parsePackageName } from "./agents.ts";
 import { discoverAvailableSkills } from "./skills.ts";
-import { MAX_SUBAGENT_NESTING_DEPTH, type SubagentToolResult } from "../shared/types.ts";
-import type { ManagementContext, ManagementScope } from "./agent-management.ts";
 
 export function result(text: string, isError = false): SubagentToolResult {
 	return { content: [{ type: "text", text }], isError, details: { mode: "management", results: [] } };
 }
 
 export function parseCsv(value: string): string[] {
-	return [...new Set(value.split(",").map((v) => v.trim()).filter(Boolean))];
+	return [
+		...new Set(
+			value
+				.split(",")
+				.map((v) => v.trim())
+				.filter(Boolean),
+		),
+	];
 }
 
 export function configObject(config: unknown): { value?: Record<string, unknown>; error?: string } {
@@ -27,7 +34,7 @@ export function configObject(config: unknown): { value?: Record<string, unknown>
 }
 
 export function hasKey(obj: Record<string, unknown>, key: string): boolean {
-	return Object.prototype.hasOwnProperty.call(obj, key);
+	return Object.hasOwn(obj, key);
 }
 
 export function asDisambiguationScope(scope: unknown): ManagementScope | undefined {
@@ -42,7 +49,13 @@ export function normalizeListScope(scope: unknown): AgentScope | undefined {
 }
 
 export function sanitizeName(name: string): string {
-	return name.toLowerCase().trim().replace(/\s+/g, "-").replace(/[^a-z0-9-]/g, "").replace(/-+/g, "-").replace(/^-+|-+$/g, "");
+	return name
+		.toLowerCase()
+		.trim()
+		.replace(/\s+/g, "-")
+		.replace(/[^a-z0-9-]/g, "")
+		.replace(/-+/g, "-")
+		.replace(/^-+|-+$/g, "");
 }
 
 export function parsePackageConfig(value: unknown): { packageName?: string; error?: string } {
@@ -71,8 +84,8 @@ export function findAgents(name: string, cwd: string, scope: AgentScope = "both"
 export function findChains(name: string, cwd: string, scope: AgentScope = "both"): ChainConfig[] {
 	const raw = name.trim();
 	const sanitized = sanitizeName(raw);
-	return discoverAgentsAll(cwd).chains
-		.filter((c) => (scope === "both" || c.source === scope) && (c.name === raw || c.name === sanitized))
+	return discoverAgentsAll(cwd)
+		.chains.filter((c) => (scope === "both" || c.source === scope) && (c.name === raw || c.name === sanitized))
 		.sort((a, b) => a.source.localeCompare(b.source));
 }
 
@@ -108,9 +121,7 @@ export function chainStepAgentNames(step: ChainStepConfig): string[] {
 export function unknownChainAgents(cwd: string, steps: ChainStepConfig[]): string[] {
 	const d = discoverAgentsAll(cwd);
 	const known = new Set(allAgents(d).map((a) => a.name));
-	const unknown = steps
-		.flatMap((step) => chainStepAgentNames(step))
-		.filter((agent) => !known.has(agent));
+	const unknown = steps.flatMap((step) => chainStepAgentNames(step)).filter((agent) => !known.has(agent));
 	return [...new Set(unknown)].sort((a, b) => a.localeCompare(b));
 }
 
@@ -120,12 +131,18 @@ export function chainStepWarnings(ctx: ManagementContext, steps: ChainStepConfig
 	for (let i = 0; i < steps.length; i++) {
 		const s = steps[i]!;
 		if (s.model) {
-			const found = ctx.modelRegistry.getAvailable().some((m) => `${m.provider}/${m.id}` === s.model || m.id === s.model);
-			if (!found) warnings.push(`Warning: step ${i + 1} (${s.agent}): model '${s.model}' is not in the current model registry.`);
+			const found = ctx.modelRegistry
+				.getAvailable()
+				.some((m) => `${m.provider}/${m.id}` === s.model || m.id === s.model);
+			if (!found)
+				warnings.push(
+					`Warning: step ${i + 1} (${s.agent}): model '${s.model}' is not in the current model registry.`,
+				);
 		}
 		if (Array.isArray(s.skills) && s.skills.length > 0) {
 			const missing = s.skills.filter((sk) => !available.has(sk));
-			if (missing.length) warnings.push(`Warning: step ${i + 1} (${s.agent}): skills not found: ${missing.join(", ")}.`);
+			if (missing.length)
+				warnings.push(`Warning: step ${i + 1} (${s.agent}): skills not found: ${missing.join(", ")}.`);
 		}
 	}
 	return warnings;
@@ -137,11 +154,16 @@ export function modelWarning(ctx: ManagementContext, model: string | undefined):
 	return found ? undefined : `Warning: model '${model}' is not in the current model registry.`;
 }
 
-export function fallbackModelsWarning(ctx: ManagementContext, fallbackModels: string[] | undefined): string | undefined {
+export function fallbackModelsWarning(
+	ctx: ManagementContext,
+	fallbackModels: string[] | undefined,
+): string | undefined {
 	if (!fallbackModels || fallbackModels.length === 0) return undefined;
 	const available = new Set(ctx.modelRegistry.getAvailable().flatMap((m) => [`${m.provider}/${m.id}`, m.id]));
 	const missing = fallbackModels.filter((model) => !available.has(model));
-	return missing.length ? `Warning: fallback models not in the current model registry: ${missing.join(", ")}.` : undefined;
+	return missing.length
+		? `Warning: fallback models not in the current model registry: ${missing.join(", ")}.`
+		: undefined;
 }
 
 export function skillsWarning(cwd: string, skills: string[] | undefined): string | undefined {
@@ -157,9 +179,11 @@ export function parseStepList(raw: unknown): { steps?: ChainStepConfig[]; error?
 	const steps: ChainStepConfig[] = [];
 	for (let i = 0; i < raw.length; i++) {
 		const item = raw[i];
-		if (!item || typeof item !== "object" || Array.isArray(item)) return { error: `config.steps[${i}] must be an object.` };
+		if (!item || typeof item !== "object" || Array.isArray(item))
+			return { error: `config.steps[${i}] must be an object.` };
 		const s = item as Record<string, unknown>;
-		if (typeof s.agent !== "string" || !s.agent.trim()) return { error: `config.steps[${i}].agent must be a non-empty string.` };
+		if (typeof s.agent !== "string" || !s.agent.trim())
+			return { error: `config.steps[${i}].agent must be a non-empty string.` };
 		const step: ChainStepConfig = { agent: s.agent.trim(), task: typeof s.task === "string" ? s.task : "" };
 		if (hasKey(s, "phase")) {
 			if (typeof s.phase === "string") step.phase = s.phase;
@@ -188,7 +212,11 @@ export function parseStepList(raw: unknown): { steps?: ChainStepConfig[]; error?
 		}
 		if (hasKey(s, "reads")) {
 			if (s.reads === false) step.reads = false;
-			else if (Array.isArray(s.reads)) step.reads = s.reads.filter((v): v is string => typeof v === "string").map((v) => v.trim()).filter(Boolean);
+			else if (Array.isArray(s.reads))
+				step.reads = s.reads
+					.filter((v): v is string => typeof v === "string")
+					.map((v) => v.trim())
+					.filter(Boolean);
 			else return { error: `config.steps[${i}].reads must be an array or false.` };
 		}
 		if (hasKey(s, "model")) {
@@ -197,7 +225,11 @@ export function parseStepList(raw: unknown): { steps?: ChainStepConfig[]; error?
 		}
 		if (hasKey(s, "skills")) {
 			if (s.skills === false) step.skills = false;
-			else if (Array.isArray(s.skills)) step.skills = s.skills.filter((v): v is string => typeof v === "string").map((v) => v.trim()).filter(Boolean);
+			else if (Array.isArray(s.skills))
+				step.skills = s.skills
+					.filter((v): v is string => typeof v === "string")
+					.map((v) => v.trim())
+					.filter(Boolean);
 			else return { error: `config.steps[${i}].skills must be an array or false.` };
 		}
 		if (hasKey(s, "progress")) {
@@ -218,7 +250,10 @@ export function parseTools(raw: string): { tools?: string[]; mcpDirectTools?: st
 			if (direct) mcpDirectTools.push(direct);
 		} else tools.push(item);
 	}
-	return { tools: tools.length ? tools : undefined, mcpDirectTools: mcpDirectTools.length ? mcpDirectTools : undefined };
+	return {
+		tools: tools.length ? tools : undefined,
+		mcpDirectTools: mcpDirectTools.length ? mcpDirectTools : undefined,
+	};
 }
 
 export function applyAgentConfig(target: AgentConfig, cfg: Record<string, unknown>): string | undefined {
@@ -246,14 +281,21 @@ export function applyAgentConfig(target: AgentConfig, cfg: Record<string, unknow
 		} else return "config.fallbackModels must be a comma-separated string, string array, or false when provided.";
 	}
 	if (hasKey(cfg, "tools")) {
-		if (cfg.tools === false || cfg.tools === "") { target.tools = undefined; target.mcpDirectTools = undefined; }
-		else if (typeof cfg.tools === "string") { const parsed = parseTools(cfg.tools); target.tools = parsed.tools; target.mcpDirectTools = parsed.mcpDirectTools; }
-		else return "config.tools must be a comma-separated string or false when provided.";
+		if (cfg.tools === false || cfg.tools === "") {
+			target.tools = undefined;
+			target.mcpDirectTools = undefined;
+		} else if (typeof cfg.tools === "string") {
+			const parsed = parseTools(cfg.tools);
+			target.tools = parsed.tools;
+			target.mcpDirectTools = parsed.mcpDirectTools;
+		} else return "config.tools must be a comma-separated string or false when provided.";
 	}
 	if (hasKey(cfg, "skills")) {
 		if (cfg.skills === false || cfg.skills === "") target.skills = undefined;
-		else if (typeof cfg.skills === "string") { const skills = parseCsv(cfg.skills); target.skills = skills.length ? skills : undefined; }
-		else return "config.skills must be a comma-separated string or false when provided.";
+		else if (typeof cfg.skills === "string") {
+			const skills = parseCsv(cfg.skills);
+			target.skills = skills.length ? skills : undefined;
+		} else return "config.skills must be a comma-separated string or false when provided.";
 	}
 	if (hasKey(cfg, "extensions")) {
 		if (cfg.extensions === false) target.extensions = undefined;
@@ -267,11 +309,13 @@ export function applyAgentConfig(target: AgentConfig, cfg: Record<string, unknow
 		else return "config.thinking must be a string or false when provided.";
 	}
 	if (hasKey(cfg, "systemPromptMode")) {
-		if (cfg.systemPromptMode === "append" || cfg.systemPromptMode === "replace") target.systemPromptMode = cfg.systemPromptMode;
+		if (cfg.systemPromptMode === "append" || cfg.systemPromptMode === "replace")
+			target.systemPromptMode = cfg.systemPromptMode;
 		else return "config.systemPromptMode must be 'append' or 'replace' when provided.";
 	}
 	if (hasKey(cfg, "inheritProjectContext")) {
-		if (typeof cfg.inheritProjectContext !== "boolean") return "config.inheritProjectContext must be a boolean when provided.";
+		if (typeof cfg.inheritProjectContext !== "boolean")
+			return "config.inheritProjectContext must be a boolean when provided.";
 		target.inheritProjectContext = cfg.inheritProjectContext;
 	}
 	if (hasKey(cfg, "inheritSkills")) {
@@ -280,7 +324,8 @@ export function applyAgentConfig(target: AgentConfig, cfg: Record<string, unknow
 	}
 	if (hasKey(cfg, "defaultContext")) {
 		if (cfg.defaultContext === false || cfg.defaultContext === "") target.defaultContext = undefined;
-		else if (cfg.defaultContext === "fresh" || cfg.defaultContext === "fork") target.defaultContext = cfg.defaultContext;
+		else if (cfg.defaultContext === "fresh" || cfg.defaultContext === "fork")
+			target.defaultContext = cfg.defaultContext;
 		else return "config.defaultContext must be 'fresh', 'fork', or false when provided.";
 	}
 	if (hasKey(cfg, "output")) {
@@ -301,10 +346,14 @@ export function applyAgentConfig(target: AgentConfig, cfg: Record<string, unknow
 	}
 	if (hasKey(cfg, "maxSubagentDepth")) {
 		if (cfg.maxSubagentDepth === false || cfg.maxSubagentDepth === "") target.maxSubagentDepth = undefined;
-		else if (typeof cfg.maxSubagentDepth === "number" && Number.isInteger(cfg.maxSubagentDepth) && cfg.maxSubagentDepth >= 0) {
+		else if (
+			typeof cfg.maxSubagentDepth === "number" &&
+			Number.isInteger(cfg.maxSubagentDepth) &&
+			cfg.maxSubagentDepth >= 0
+		) {
 			target.maxSubagentDepth = Math.min(cfg.maxSubagentDepth, MAX_SUBAGENT_NESTING_DEPTH);
-		} else return `config.maxSubagentDepth must be an integer >= 0 or false when provided; values above ${MAX_SUBAGENT_NESTING_DEPTH} are clamped.`;
+		} else
+			return `config.maxSubagentDepth must be an integer >= 0 or false when provided; values above ${MAX_SUBAGENT_NESTING_DEPTH} are clamped.`;
 	}
 	return undefined;
 }
-

package/dist/builtin/subagents/src/agents/agent-management.ts

@@ -1,23 +1,43 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { CONFIG_DIR_NAME } from "@bastani/atomic";
 import type { ExtensionContext } from "@bastani/atomic";
+import { CONFIG_DIR_NAME } from "@bastani/atomic";
+import type { SubagentToolResult } from "../shared/types.ts";
+import {
+	allAgents,
+	applyAgentConfig,
+	asDisambiguationScope,
+	availableNames,
+	chainStepWarnings,
+	configObject,
+	fallbackModelsWarning,
+	findAgents,
+	findChains,
+	hasKey,
+	modelWarning,
+	nameExistsInScope,
+	normalizeListScope,
+	parsePackageConfig,
+	parseStepList,
+	result,
+	sanitizeName,
+	skillsWarning,
+	unknownChainAgents,
+} from "./agent-management-helpers.ts";
+import { serializeAgent } from "./agent-serializer.ts";
 import {
 	type AgentConfig,
 	type AgentSource,
+	buildRuntimeName,
 	type ChainConfig,
 	type ChainStepConfig,
 	defaultInheritProjectContext,
 	defaultInheritSkills,
 	defaultSystemPromptMode,
 	discoverAgentsAll,
-	buildRuntimeName,
 	frontmatterNameForConfig,
 } from "./agents.ts";
-import { serializeAgent } from "./agent-serializer.ts";
 import { serializeChain, serializeJsonChain } from "./chain-serializer.ts";
-import { allAgents, applyAgentConfig, asDisambiguationScope, availableNames, chainStepWarnings, configObject, fallbackModelsWarning, findAgents, findChains, hasKey, modelWarning, nameExistsInScope, normalizeListScope, parsePackageConfig, parseStepList, result, sanitizeName, skillsWarning, unknownChainAgents } from "./agent-management-helpers.ts";
-import type { SubagentToolResult } from "../shared/types.ts";
 
 type ManagementAction = "list" | "get" | "create" | "update" | "delete";
 export type ManagementScope = "user" | "project";
@@ -47,20 +67,34 @@ function resolveTarget<T extends { source: AgentSource; filePath: string }>(
 	const mutable = matches.filter(isMutableDefinition);
 	if (mutable.length === 0) {
 		if (matches.length > 0) {
-			return result(`${kind === "agent" ? "Agent" : "Chain"} '${name}' is builtin and cannot be modified. Create a same-named ${kind} in user or project scope to override it.`, true);
+			return result(
+				`${kind === "agent" ? "Agent" : "Chain"} '${name}' is builtin and cannot be modified. Create a same-named ${kind} in user or project scope to override it.`,
+				true,
+			);
 		}
 		const available = availableNames(cwd, kind);
-		return result(`${kind === "agent" ? "Agent" : "Chain"} '${name}' not found. Available: ${available.join(", ") || "none"}.`, true);
+		return result(
+			`${kind === "agent" ? "Agent" : "Chain"} '${name}' not found. Available: ${available.join(", ") || "none"}.`,
+			true,
+		);
 	}
 	if (mutable.length === 1) return mutable[0]!;
 	const scope = asDisambiguationScope(scopeHint);
 	if (!scope) {
 		const paths = mutable.map((m) => `${m.source}: ${m.filePath}`).join("\n");
-		return result(`${kind === "agent" ? "Agent" : "Chain"} '${name}' exists in both scopes. Specify agentScope: 'user' or 'project'.\n${paths}`, true);
+		return result(
+			`${kind === "agent" ? "Agent" : "Chain"} '${name}' exists in both scopes. Specify agentScope: 'user' or 'project'.\n${paths}`,
+			true,
+		);
 	}
 	const scoped = mutable.filter((m) => m.source === scope);
-	if (scoped.length === 0) return result(`${kind === "agent" ? "Agent" : "Chain"} '${name}' not found in scope '${scope}'.`, true);
-	if (scoped.length > 1) return result(`Multiple ${kind}s named '${name}' found in scope '${scope}': ${scoped.map((m) => m.filePath).join(", ")}`, true);
+	if (scoped.length === 0)
+		return result(`${kind === "agent" ? "Agent" : "Chain"} '${name}' not found in scope '${scope}'.`, true);
+	if (scoped.length > 1)
+		return result(
+			`Multiple ${kind}s named '${name}' found in scope '${scope}': ${scoped.map((m) => m.filePath).join(", ")}`,
+			true,
+		);
 	return scoped[0]!;
 }
 
@@ -71,11 +105,14 @@ function renamePath(
 	scope: ManagementScope,
 	cwd: string,
 ): { filePath?: string; error?: string } {
-	if (nameExistsInScope(cwd, scope, newName, currentPath)) return { error: `Name '${newName}' already exists in ${scope} scope.` };
+	if (nameExistsInScope(cwd, scope, newName, currentPath))
+		return { error: `Name '${newName}' already exists in ${scope} scope.` };
 	const ext = kind === "agent" ? ".md" : currentPath.endsWith(".chain.json") ? ".chain.json" : ".chain.md";
 	const filePath = path.join(path.dirname(currentPath), `${newName}${ext}`);
 	if (fs.existsSync(filePath) && filePath !== currentPath) {
-		return { error: `File already exists at ${filePath} but is not a valid ${kind} definition. Remove or rename it first.` };
+		return {
+			error: `File already exists at ${filePath} but is not a valid ${kind} definition. Remove or rename it first.`,
+		};
 	}
 	fs.renameSync(currentPath, filePath);
 	return { filePath };
@@ -83,7 +120,11 @@ function renamePath(
 
 function formatAgentDetail(agent: AgentConfig): string {
 	const tools = [...(agent.tools ?? []), ...(agent.mcpDirectTools ?? []).map((t) => `mcp:${t}`)];
-	const lines: string[] = [`Agent: ${agent.name} (${agent.source})`, `Path: ${agent.filePath}`, `Description: ${agent.description}`];
+	const lines: string[] = [
+		`Agent: ${agent.name} (${agent.source})`,
+		`Path: ${agent.filePath}`,
+		`Description: ${agent.description}`,
+	];
 	if (agent.packageName) {
 		lines.push(`Local name: ${frontmatterNameForConfig(agent)}`);
 		lines.push(`Package: ${agent.packageName}`);
@@ -97,7 +138,8 @@ function formatAgentDetail(agent: AgentConfig): string {
 	lines.push(`Inherit skills: ${agent.inheritSkills ? "true" : "false"}`);
 	if (agent.defaultContext) lines.push(`Default context: ${agent.defaultContext}`);
 	if (agent.source === "builtin") lines.push(`Disabled: ${agent.disabled ? "true" : "false"}`);
-	if (agent.extensions !== undefined) lines.push(`Extensions: ${agent.extensions.length ? agent.extensions.join(", ") : "(none)"}`);
+	if (agent.extensions !== undefined)
+		lines.push(`Extensions: ${agent.extensions.length ? agent.extensions.join(", ") : "(none)"}`);
 	if (agent.thinking) lines.push(`Thinking: ${agent.thinking}`);
 	if (agent.output) lines.push(`Output: ${agent.output}`);
 	if (agent.defaultReads?.length) lines.push(`Reads: ${agent.defaultReads.join(", ")}`);
@@ -110,9 +152,24 @@ function formatAgentDetail(agent: AgentConfig): string {
 function formatChainStepDetail(step: ChainStepConfig, index: number): string[] {
 	const lines: string[] = [];
 	if (step.expand || step.collect) {
-		const parallel = step.parallel && !Array.isArray(step.parallel) && typeof step.parallel === "object" ? step.parallel as { agent?: unknown; task?: unknown; label?: unknown; outputSchema?: unknown } : undefined;
-		const expand = step.expand && typeof step.expand === "object" ? step.expand as { from?: { output?: unknown; path?: unknown }; item?: unknown; key?: unknown; maxItems?: unknown; onEmpty?: unknown } : undefined;
-		const collect = step.collect && typeof step.collect === "object" ? step.collect as { as?: unknown; outputSchema?: unknown } : undefined;
+		const parallel =
+			step.parallel && !Array.isArray(step.parallel) && typeof step.parallel === "object"
+				? (step.parallel as { agent?: unknown; task?: unknown; label?: unknown; outputSchema?: unknown })
+				: undefined;
+		const expand =
+			step.expand && typeof step.expand === "object"
+				? (step.expand as {
+						from?: { output?: unknown; path?: unknown };
+						item?: unknown;
+						key?: unknown;
+						maxItems?: unknown;
+						onEmpty?: unknown;
+					})
+				: undefined;
+		const collect =
+			step.collect && typeof step.collect === "object"
+				? (step.collect as { as?: unknown; outputSchema?: unknown })
+				: undefined;
 		lines.push(`${index + 1}. Dynamic fanout${typeof collect?.as === "string" ? ` -> ${collect.as}` : ""}`);
 		if (expand?.from) lines.push(`   Expand: ${String(expand.from.output ?? "?")}${String(expand.from.path ?? "")}`);
 		if (typeof expand?.item === "string") lines.push(`   Item variable: ${expand.item}`);
@@ -143,7 +200,11 @@ function formatChainStepDetail(step: ChainStepConfig, index: number): string[] {
 }
 
 function formatChainDetail(chain: ChainConfig): string {
-	const lines: string[] = [`Chain: ${chain.name} (${chain.source})`, `Path: ${chain.filePath}`, `Description: ${chain.description}`];
+	const lines: string[] = [
+		`Chain: ${chain.name} (${chain.source})`,
+		`Path: ${chain.filePath}`,
+		`Description: ${chain.description}`,
+	];
 	if (chain.packageName) {
 		lines.push(`Local name: ${frontmatterNameForConfig(chain)}`);
 		lines.push(`Package: ${chain.packageName}`);
@@ -158,19 +219,28 @@ function formatChainDetail(chain: ChainConfig): string {
 export function handleList(params: ManagementParams, ctx: ManagementContext): SubagentToolResult {
 	const scope = normalizeListScope(params.agentScope) ?? "both";
 	const d = discoverAgentsAll(ctx.cwd);
-	const scopedAgents = allAgents(d).filter((a) => scope === "both" || a.source === "builtin" || a.source === scope).sort((a, b) => a.name.localeCompare(b.name));
+	const scopedAgents = allAgents(d)
+		.filter((a) => scope === "both" || a.source === "builtin" || a.source === scope)
+		.sort((a, b) => a.name.localeCompare(b.name));
 	const agents = scopedAgents.filter((a) => !a.disabled);
-	const chains = d.chains.filter((c) => scope === "both" || c.source === scope).sort((a, b) => a.name.localeCompare(b.name));
+	const chains = d.chains
+		.filter((c) => scope === "both" || c.source === scope)
+		.sort((a, b) => a.name.localeCompare(b.name));
 	const diagnostics = d.chainDiagnostics.filter((entry) => scope === "both" || entry.source === scope);
 	const lines = [
 		"Executable agents:",
 		...(agents.length
-			? agents.map((a) => `- ${a.name} (${a.source}${a.defaultContext ? `, context: ${a.defaultContext}` : ""}): ${a.description}`)
+			? agents.map(
+					(a) =>
+						`- ${a.name} (${a.source}${a.defaultContext ? `, context: ${a.defaultContext}` : ""}): ${a.description}`,
+				)
 			: ["- (none)"]),
 		"",
 		"Chains:",
 		...(chains.length ? chains.map((c) => `- ${c.name} (${c.source}): ${c.description}`) : ["- (none)"]),
-		...(diagnostics.length ? ["", "Chain diagnostics:", ...diagnostics.map((entry) => `- ${entry.filePath}: ${entry.error}`)] : []),
+		...(diagnostics.length
+			? ["", "Chain diagnostics:", ...diagnostics.map((entry) => `- ${entry.filePath}: ${entry.error}`)]
+			: []),
 	];
 	return result(lines.join("\n"));
 }
@@ -210,10 +280,13 @@ export function handleCreate(params: ManagementParams, ctx: ManagementContext):
 	if (parsedConfig.error) return result(parsedConfig.error, true);
 	const cfg = parsedConfig.value;
 	if (!cfg) return result("config required for create.", true);
-	if (typeof cfg.name !== "string" || !cfg.name.trim()) return result("config.name is required and must be a non-empty string.", true);
-	if (typeof cfg.description !== "string" || !cfg.description.trim()) return result("config.description is required and must be a non-empty string.", true);
+	if (typeof cfg.name !== "string" || !cfg.name.trim())
+		return result("config.name is required and must be a non-empty string.", true);
+	if (typeof cfg.description !== "string" || !cfg.description.trim())
+		return result("config.description is required and must be a non-empty string.", true);
 	const name = sanitizeName(cfg.name);
-	if (!name) return result("config.name is invalid after sanitization. Use letters, numbers, spaces, or hyphens.", true);
+	if (!name)
+		return result("config.name is invalid after sanitization. Use letters, numbers, spaces, or hyphens.", true);
 	const parsedPackage = parsePackageConfig(cfg.package);
 	if (parsedPackage.error) return result(parsedPackage.error, true);
 	const runtimeName = buildRuntimeName(name, parsedPackage.packageName);
@@ -223,18 +296,36 @@ export function handleCreate(params: ManagementParams, ctx: ManagementContext):
 	const isChain = hasKey(cfg, "steps");
 	const d = discoverAgentsAll(ctx.cwd);
 	const targetDir = isChain
-		? scope === "user" ? d.userChainDir : d.projectChainDir ?? path.join(ctx.cwd, CONFIG_DIR_NAME, "chains")
-		: scope === "user" ? d.userDir : d.projectDir ?? path.join(ctx.cwd, CONFIG_DIR_NAME, "agents");
+		? scope === "user"
+			? d.userChainDir
+			: (d.projectChainDir ?? path.join(ctx.cwd, CONFIG_DIR_NAME, "chains"))
+		: scope === "user"
+			? d.userDir
+			: (d.projectDir ?? path.join(ctx.cwd, CONFIG_DIR_NAME, "agents"));
 	fs.mkdirSync(targetDir, { recursive: true });
-	if (nameExistsInScope(ctx.cwd, scope, runtimeName)) return result(`Name '${runtimeName}' already exists in ${scope} scope. Use update instead.`, true);
+	if (nameExistsInScope(ctx.cwd, scope, runtimeName))
+		return result(`Name '${runtimeName}' already exists in ${scope} scope. Use update instead.`, true);
 	const targetPath = path.join(targetDir, isChain ? `${runtimeName}.chain.md` : `${runtimeName}.md`);
-	if (fs.existsSync(targetPath)) return result(`File already exists at ${targetPath} but is not a valid ${isChain ? "chain" : "agent"} definition. Remove or rename it first.`, true);
+	if (fs.existsSync(targetPath))
+		return result(
+			`File already exists at ${targetPath} but is not a valid ${isChain ? "chain" : "agent"} definition. Remove or rename it first.`,
+			true,
+		);
 	const warnings: string[] = [];
-	if (!isChain && d.builtin.some((a) => a.name === runtimeName)) warnings.push(`Note: this shadows the builtin agent '${runtimeName}'.`);
+	if (!isChain && d.builtin.some((a) => a.name === runtimeName))
+		warnings.push(`Note: this shadows the builtin agent '${runtimeName}'.`);
 	if (isChain) {
 		const parsed = parseStepList(cfg.steps);
 		if (parsed.error) return result(parsed.error, true);
-		const chain: ChainConfig = { name: runtimeName, localName: name, packageName: parsedPackage.packageName, description: cfg.description.trim(), source: scope, filePath: targetPath, steps: parsed.steps! };
+		const chain: ChainConfig = {
+			name: runtimeName,
+			localName: name,
+			packageName: parsedPackage.packageName,
+			description: cfg.description.trim(),
+			source: scope,
+			filePath: targetPath,
+			steps: parsed.steps!,
+		};
 		fs.writeFileSync(targetPath, serializeChain(chain), "utf-8");
 		const missing = unknownChainAgents(ctx.cwd, chain.steps);
 		if (missing.length) warnings.push(`Warning: chain steps reference unknown agents: ${missing.join(", ")}.`);
@@ -275,13 +366,21 @@ export function handleUpdate(params: ManagementParams, ctx: ManagementContext):
 	const warnings: string[] = [];
 	if (params.agent) {
 		const scopeHint = asDisambiguationScope(params.agentScope);
-		const targetOrError = resolveTarget("agent", params.agent, findAgents(params.agent, ctx.cwd, scopeHint ?? "both"), ctx.cwd, params.agentScope);
+		const targetOrError = resolveTarget(
+			"agent",
+			params.agent,
+			findAgents(params.agent, ctx.cwd, scopeHint ?? "both"),
+			ctx.cwd,
+			params.agentScope,
+		);
 		if ("content" in targetOrError) return targetOrError;
 		const target = targetOrError;
 		const updated: AgentConfig = { ...target };
 		const oldName = target.name;
-		if (hasKey(cfg, "name") && (typeof cfg.name !== "string" || !cfg.name.trim())) return result("config.name must be a non-empty string when provided.", true);
-		if (hasKey(cfg, "description") && (typeof cfg.description !== "string" || !cfg.description.trim())) return result("config.description must be a non-empty string when provided.", true);
+		if (hasKey(cfg, "name") && (typeof cfg.name !== "string" || !cfg.name.trim()))
+			return result("config.name must be a non-empty string when provided.", true);
+		if (hasKey(cfg, "description") && (typeof cfg.description !== "string" || !cfg.description.trim()))
+			return result("config.description must be a non-empty string when provided.", true);
 		let newLocalName = target.localName ?? frontmatterNameForConfig(target);
 		if (hasKey(cfg, "name")) {
 			newLocalName = sanitizeName(cfg.name as string);
@@ -318,22 +417,33 @@ export function handleUpdate(params: ManagementParams, ctx: ManagementContext):
 		}
 		fs.writeFileSync(updated.filePath, serializeAgent(updated), "utf-8");
 		if (updated.name !== oldName) {
-			const refs = discoverAgentsAll(ctx.cwd).chains.filter((c) => c.steps.some((s) => s.agent === oldName)).map((c) => `${c.name} (${c.source})`);
+			const refs = discoverAgentsAll(ctx.cwd)
+				.chains.filter((c) => c.steps.some((s) => s.agent === oldName))
+				.map((c) => `${c.name} (${c.source})`);
 			if (refs.length) warnings.push(`Warning: chains still reference '${oldName}': ${refs.join(", ")}.`);
 		}
-		const headline = updated.name === oldName
-			? `Updated agent '${updated.name}' at ${updated.filePath}.`
-			: `Updated agent '${oldName}' to '${updated.name}' at ${updated.filePath}.`;
+		const headline =
+			updated.name === oldName
+				? `Updated agent '${updated.name}' at ${updated.filePath}.`
+				: `Updated agent '${oldName}' to '${updated.name}' at ${updated.filePath}.`;
 		return result([headline, ...warnings].join("\n"));
 	}
 	const scopeHint = asDisambiguationScope(params.agentScope);
-	const targetOrError = resolveTarget("chain", params.chainName!, findChains(params.chainName!, ctx.cwd, scopeHint ?? "both"), ctx.cwd, params.agentScope);
+	const targetOrError = resolveTarget(
+		"chain",
+		params.chainName!,
+		findChains(params.chainName!, ctx.cwd, scopeHint ?? "both"),
+		ctx.cwd,
+		params.agentScope,
+	);
 	if ("content" in targetOrError) return targetOrError;
 	const target = targetOrError;
 	const updated: ChainConfig = { ...target, steps: [...target.steps] };
 	const oldName = target.name;
-	if (hasKey(cfg, "name") && (typeof cfg.name !== "string" || !cfg.name.trim())) return result("config.name must be a non-empty string when provided.", true);
-	if (hasKey(cfg, "description") && (typeof cfg.description !== "string" || !cfg.description.trim())) return result("config.description must be a non-empty string when provided.", true);
+	if (hasKey(cfg, "name") && (typeof cfg.name !== "string" || !cfg.name.trim()))
+		return result("config.name must be a non-empty string when provided.", true);
+	if (hasKey(cfg, "description") && (typeof cfg.description !== "string" || !cfg.description.trim()))
+		return result("config.description must be a non-empty string when provided.", true);
 	let newLocalName = target.localName ?? frontmatterNameForConfig(target);
 	if (hasKey(cfg, "name")) {
 		newLocalName = sanitizeName(cfg.name as string);
@@ -366,10 +476,15 @@ export function handleUpdate(params: ManagementParams, ctx: ManagementContext):
 		if (renamed.error) return result(renamed.error, true);
 		updated.filePath = renamed.filePath!;
 	}
-	fs.writeFileSync(updated.filePath, updated.filePath.endsWith(".chain.json") ? serializeJsonChain(updated) : serializeChain(updated), "utf-8");
-	const headline = updated.name === oldName
-		? `Updated chain '${updated.name}' at ${updated.filePath}.`
-		: `Updated chain '${oldName}' to '${updated.name}' at ${updated.filePath}.`;
+	fs.writeFileSync(
+		updated.filePath,
+		updated.filePath.endsWith(".chain.json") ? serializeJsonChain(updated) : serializeChain(updated),
+		"utf-8",
+	);
+	const headline =
+		updated.name === oldName
+			? `Updated chain '${updated.name}' at ${updated.filePath}.`
+			: `Updated chain '${oldName}' to '${updated.name}' at ${updated.filePath}.`;
 	return result([headline, ...warnings].join("\n"));
 }
 
@@ -378,29 +493,53 @@ function handleDelete(params: ManagementParams, ctx: ManagementContext): Subagen
 	if (params.agent && params.chainName) return result("Specify either 'agent' or 'chainName', not both.", true);
 	const scopeHint = asDisambiguationScope(params.agentScope);
 	if (params.agent) {
-		const targetOrError = resolveTarget("agent", params.agent, findAgents(params.agent, ctx.cwd, scopeHint ?? "both"), ctx.cwd, params.agentScope);
+		const targetOrError = resolveTarget(
+			"agent",
+			params.agent,
+			findAgents(params.agent, ctx.cwd, scopeHint ?? "both"),
+			ctx.cwd,
+			params.agentScope,
+		);
 		if ("content" in targetOrError) return targetOrError;
 		const target = targetOrError;
 		fs.unlinkSync(target.filePath);
-		const refs = discoverAgentsAll(ctx.cwd).chains.filter((c) => c.steps.some((s) => s.agent === target.name)).map((c) => `${c.name} (${c.source})`);
+		const refs = discoverAgentsAll(ctx.cwd)
+			.chains.filter((c) => c.steps.some((s) => s.agent === target.name))
+			.map((c) => `${c.name} (${c.source})`);
 		const lines = [`Deleted agent '${target.name}' at ${target.filePath}.`];
 		if (refs.length) lines.push(`Warning: chains reference deleted agent '${target.name}': ${refs.join(", ")}.`);
 		return result(lines.join("\n"));
 	}
-	const targetOrError = resolveTarget("chain", params.chainName!, findChains(params.chainName!, ctx.cwd, scopeHint ?? "both"), ctx.cwd, params.agentScope);
+	const targetOrError = resolveTarget(
+		"chain",
+		params.chainName!,
+		findChains(params.chainName!, ctx.cwd, scopeHint ?? "both"),
+		ctx.cwd,
+		params.agentScope,
+	);
 	if ("content" in targetOrError) return targetOrError;
 	const target = targetOrError;
 	fs.unlinkSync(target.filePath);
 	return result(`Deleted chain '${target.name}' at ${target.filePath}.`);
 }
 
-export function handleManagementAction(action: string, params: ManagementParams, ctx: ManagementContext): SubagentToolResult {
+export function handleManagementAction(
+	action: string,
+	params: ManagementParams,
+	ctx: ManagementContext,
+): SubagentToolResult {
 	switch (action as ManagementAction) {
-		case "list": return handleList(params, ctx);
-		case "get": return handleGet(params, ctx);
-		case "create": return handleCreate(params, ctx);
-		case "update": return handleUpdate(params, ctx);
-		case "delete": return handleDelete(params, ctx);
-		default: return result(`Unknown action: ${action}`, true);
+		case "list":
+			return handleList(params, ctx);
+		case "get":
+			return handleGet(params, ctx);
+		case "create":
+			return handleCreate(params, ctx);
+		case "update":
+			return handleUpdate(params, ctx);
+		case "delete":
+			return handleDelete(params, ctx);
+		default:
+			return result(`Unknown action: ${action}`, true);
 	}
 }