@bastani/atomic 0.9.16-alpha.2 → 0.9.16-alpha.4

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

@@ -1,450 +0,0 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { getProjectAgentSettingsPath, getUserAgentSettingsPath } from "./agent-paths.js";
-import {
-	type AgentConfig,
-	type BuiltinAgentOverrideBase,
-	type BuiltinAgentOverrideConfig,
-	EMPTY_SUBAGENT_SETTINGS,
-	type SubagentSettings,
-} from "./agent-types.js";
-
-export function splitToolList(rawTools: string[] | undefined): { tools?: string[]; mcpDirectTools?: string[] } {
-	const mcpDirectTools: string[] = [];
-	const tools: string[] = [];
-	for (const tool of rawTools ?? []) {
-		if (tool.startsWith("mcp:")) {
-			mcpDirectTools.push(tool.slice(4));
-		} else {
-			tools.push(tool);
-		}
-	}
-	return {
-		...(tools.length > 0 ? { tools } : {}),
-		...(mcpDirectTools.length > 0 ? { mcpDirectTools } : {}),
-	};
-}
-
-function joinToolList(config: Pick<AgentConfig, "tools" | "mcpDirectTools">): string[] | undefined {
-	const joined = [...(config.tools ?? []), ...(config.mcpDirectTools ?? []).map((tool) => `mcp:${tool}`)];
-	return joined.length > 0 ? joined : undefined;
-}
-
-function arraysEqual(a: string[] | undefined, b: string[] | undefined): boolean {
-	if (!a && !b) return true;
-	if (!a || !b) return false;
-	if (a.length !== b.length) return false;
-	for (let i = 0; i < a.length; i++) {
-		if (a[i] !== b[i]) return false;
-	}
-	return true;
-}
-
-function cloneOverrideBase(agent: AgentConfig): BuiltinAgentOverrideBase {
-	return {
-		model: agent.model,
-		fallbackModels: agent.fallbackModels ? [...agent.fallbackModels] : undefined,
-		fallbackThinkingLevels: agent.fallbackThinkingLevels ? [...agent.fallbackThinkingLevels] : undefined,
-		thinking: agent.thinking,
-		systemPromptMode: agent.systemPromptMode,
-		inheritProjectContext: agent.inheritProjectContext,
-		inheritSkills: agent.inheritSkills,
-		defaultContext: agent.defaultContext,
-		disabled: agent.disabled,
-		systemPrompt: agent.systemPrompt,
-		skills: agent.skills ? [...agent.skills] : undefined,
-		tools: agent.tools ? [...agent.tools] : undefined,
-		mcpDirectTools: agent.mcpDirectTools ? [...agent.mcpDirectTools] : undefined,
-	};
-}
-
-function cloneOverrideValue(override: BuiltinAgentOverrideConfig): BuiltinAgentOverrideConfig {
-	return {
-		...(override.model !== undefined ? { model: override.model } : {}),
-		...(override.fallbackModels !== undefined
-			? { fallbackModels: override.fallbackModels === false ? false : [...override.fallbackModels] }
-			: {}),
-		...(override.fallbackThinkingLevels !== undefined
-			? {
-					fallbackThinkingLevels:
-						override.fallbackThinkingLevels === false ? false : [...override.fallbackThinkingLevels],
-				}
-			: {}),
-		...(override.thinking !== undefined ? { thinking: override.thinking } : {}),
-		...(override.systemPromptMode !== undefined ? { systemPromptMode: override.systemPromptMode } : {}),
-		...(override.inheritProjectContext !== undefined
-			? { inheritProjectContext: override.inheritProjectContext }
-			: {}),
-		...(override.inheritSkills !== undefined ? { inheritSkills: override.inheritSkills } : {}),
-		...(override.defaultContext !== undefined ? { defaultContext: override.defaultContext } : {}),
-		...(override.disabled !== undefined ? { disabled: override.disabled } : {}),
-		...(override.systemPrompt !== undefined ? { systemPrompt: override.systemPrompt } : {}),
-		...(override.skills !== undefined ? { skills: override.skills === false ? false : [...override.skills] } : {}),
-		...(override.tools !== undefined ? { tools: override.tools === false ? false : [...override.tools] } : {}),
-	};
-}
-
-function readSettingsFileStrict(filePath: string): Record<string, unknown> {
-	if (!fs.existsSync(filePath)) return {};
-	let raw: string;
-	try {
-		raw = fs.readFileSync(filePath, "utf-8");
-	} catch (error) {
-		const message = error instanceof Error ? error.message : String(error);
-		throw new Error(`Failed to read settings file '${filePath}': ${message}`, { cause: error });
-	}
-
-	let parsed: unknown;
-	try {
-		parsed = JSON.parse(raw);
-	} catch (error) {
-		const message = error instanceof Error ? error.message : String(error);
-		throw new Error(`Failed to parse settings file '${filePath}': ${message}`, { cause: error });
-	}
-	if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
-		throw new Error(`Settings file '${filePath}' must contain a JSON object.`);
-	}
-	return parsed as Record<string, unknown>;
-}
-
-function writeSettingsFile(filePath: string, settings: Record<string, unknown>): void {
-	fs.mkdirSync(path.dirname(filePath), { recursive: true });
-	fs.writeFileSync(filePath, `${JSON.stringify(settings, null, 2)}\n`, "utf-8");
-}
-
-function parseOverrideStringArrayOrFalse(
-	value: unknown,
-	meta: { filePath: string; name: string; field: string },
-): string[] | false | undefined {
-	if (value === undefined) return undefined;
-	if (value === false) return false;
-	if (!Array.isArray(value)) {
-		throw new Error(
-			`Builtin override '${meta.name}' in '${meta.filePath}' has invalid '${meta.field}'; expected an array of strings or false.`,
-		);
-	}
-
-	const items: string[] = [];
-	for (const item of value) {
-		if (typeof item !== "string") {
-			throw new Error(
-				`Builtin override '${meta.name}' in '${meta.filePath}' has invalid '${meta.field}'; expected an array of strings or false.`,
-			);
-		}
-		const trimmed = item.trim();
-		if (trimmed) items.push(trimmed);
-	}
-	return items;
-}
-
-function parseBuiltinOverrideEntry(
-	name: string,
-	value: unknown,
-	filePath: string,
-): BuiltinAgentOverrideConfig | undefined {
-	if (!value || typeof value !== "object" || Array.isArray(value)) {
-		throw new Error(`Builtin override '${name}' in '${filePath}' must be an object.`);
-	}
-
-	const input = value as Record<string, unknown>;
-	const override: BuiltinAgentOverrideConfig = {};
-
-	if ("model" in input) {
-		if (typeof input.model === "string" || input.model === false) override.model = input.model;
-		else
-			throw new Error(
-				`Builtin override '${name}' in '${filePath}' has invalid 'model'; expected a string or false.`,
-			);
-	}
-
-	if ("thinking" in input) {
-		if (typeof input.thinking === "string" || input.thinking === false) override.thinking = input.thinking;
-		else
-			throw new Error(
-				`Builtin override '${name}' in '${filePath}' has invalid 'thinking'; expected a string or false.`,
-			);
-	}
-
-	if ("systemPromptMode" in input) {
-		if (input.systemPromptMode === "append" || input.systemPromptMode === "replace") {
-			override.systemPromptMode = input.systemPromptMode;
-		} else {
-			throw new Error(
-				`Builtin override '${name}' in '${filePath}' has invalid 'systemPromptMode'; expected 'append' or 'replace'.`,
-			);
-		}
-	}
-
-	if ("inheritProjectContext" in input) {
-		if (typeof input.inheritProjectContext === "boolean") {
-			override.inheritProjectContext = input.inheritProjectContext;
-		} else {
-			throw new Error(
-				`Builtin override '${name}' in '${filePath}' has invalid 'inheritProjectContext'; expected a boolean.`,
-			);
-		}
-	}
-
-	if ("inheritSkills" in input) {
-		if (typeof input.inheritSkills === "boolean") {
-			override.inheritSkills = input.inheritSkills;
-		} else {
-			throw new Error(
-				`Builtin override '${name}' in '${filePath}' has invalid 'inheritSkills'; expected a boolean.`,
-			);
-		}
-	}
-
-	if ("defaultContext" in input) {
-		if (input.defaultContext === "fresh" || input.defaultContext === "fork" || input.defaultContext === false) {
-			override.defaultContext = input.defaultContext;
-		} else {
-			throw new Error(
-				`Builtin override '${name}' in '${filePath}' has invalid 'defaultContext'; expected 'fresh', 'fork', or false.`,
-			);
-		}
-	}
-
-	if ("disabled" in input) {
-		if (typeof input.disabled === "boolean") {
-			override.disabled = input.disabled;
-		} else {
-			throw new Error(`Builtin override '${name}' in '${filePath}' has invalid 'disabled'; expected a boolean.`);
-		}
-	}
-
-	if ("systemPrompt" in input) {
-		if (typeof input.systemPrompt === "string") override.systemPrompt = input.systemPrompt;
-		else
-			throw new Error(`Builtin override '${name}' in '${filePath}' has invalid 'systemPrompt'; expected a string.`);
-	}
-
-	const fallbackModels = parseOverrideStringArrayOrFalse(input.fallbackModels, {
-		filePath,
-		name,
-		field: "fallbackModels",
-	});
-	if (fallbackModels !== undefined) override.fallbackModels = fallbackModels;
-
-	const fallbackThinkingLevels = parseOverrideStringArrayOrFalse(input.fallbackThinkingLevels, {
-		filePath,
-		name,
-		field: "fallbackThinkingLevels",
-	});
-	if (fallbackThinkingLevels !== undefined) override.fallbackThinkingLevels = fallbackThinkingLevels;
-
-	const skills = parseOverrideStringArrayOrFalse(input.skills, { filePath, name, field: "skills" });
-	if (skills !== undefined) override.skills = skills;
-
-	const tools = parseOverrideStringArrayOrFalse(input.tools, { filePath, name, field: "tools" });
-	if (tools !== undefined) override.tools = tools;
-
-	return Object.keys(override).length > 0 ? override : undefined;
-}
-
-function readSubagentSettings(filePath: string | null): SubagentSettings {
-	if (!filePath) return EMPTY_SUBAGENT_SETTINGS;
-	const settings = readSettingsFileStrict(filePath);
-	const subagents = settings.subagents;
-	if (!subagents || typeof subagents !== "object" || Array.isArray(subagents)) return EMPTY_SUBAGENT_SETTINGS;
-
-	const subagentsObject = subagents as Record<string, unknown>;
-	let disableBuiltins: boolean | undefined;
-	if ("disableBuiltins" in subagentsObject) {
-		if (typeof subagentsObject.disableBuiltins === "boolean") {
-			disableBuiltins = subagentsObject.disableBuiltins;
-		} else {
-			throw new Error(`Subagent settings in '${filePath}' have invalid 'disableBuiltins'; expected a boolean.`);
-		}
-	}
-
-	const parsed: Record<string, BuiltinAgentOverrideConfig> = {};
-	const agentOverrides = subagentsObject.agentOverrides;
-	if (!agentOverrides || typeof agentOverrides !== "object" || Array.isArray(agentOverrides)) {
-		return { overrides: parsed, disableBuiltins };
-	}
-	for (const [name, value] of Object.entries(agentOverrides)) {
-		const override = parseBuiltinOverrideEntry(name, value, filePath);
-		if (override) parsed[name] = override;
-	}
-	return { overrides: parsed, disableBuiltins };
-}
-
-export function readMergedSubagentSettings(filePaths: string[]): { settings: SubagentSettings; path: string | null } {
-	let settings = EMPTY_SUBAGENT_SETTINGS;
-	let path: string | null = null;
-	for (let i = filePaths.length - 1; i >= 0; i--) {
-		const filePath = filePaths[i]!;
-		if (!fs.existsSync(filePath)) continue;
-		const next = readSubagentSettings(filePath);
-		settings = {
-			disableBuiltins: next.disableBuiltins ?? settings.disableBuiltins,
-			overrides: { ...settings.overrides, ...next.overrides },
-		};
-		path = filePath;
-	}
-	return { settings, path };
-}
-
-function applyBuiltinOverride(
-	agent: AgentConfig,
-	override: BuiltinAgentOverrideConfig,
-	meta: { scope: "user" | "project"; path: string },
-): AgentConfig {
-	const next: AgentConfig = {
-		...agent,
-		override: { ...meta, base: cloneOverrideBase(agent) },
-	};
-
-	if (override.model !== undefined) next.model = override.model === false ? undefined : override.model;
-	if (override.fallbackModels !== undefined) {
-		next.fallbackModels = override.fallbackModels === false ? undefined : [...override.fallbackModels];
-	}
-	if (override.fallbackThinkingLevels !== undefined) {
-		next.fallbackThinkingLevels =
-			override.fallbackThinkingLevels === false ? undefined : [...override.fallbackThinkingLevels];
-	}
-	if (override.thinking !== undefined) next.thinking = override.thinking === false ? undefined : override.thinking;
-	if (override.systemPromptMode !== undefined) next.systemPromptMode = override.systemPromptMode;
-	if (override.inheritProjectContext !== undefined) next.inheritProjectContext = override.inheritProjectContext;
-	if (override.inheritSkills !== undefined) next.inheritSkills = override.inheritSkills;
-	if (override.defaultContext !== undefined)
-		next.defaultContext = override.defaultContext === false ? undefined : override.defaultContext;
-	if (override.disabled !== undefined) next.disabled = override.disabled;
-	if (override.systemPrompt !== undefined) next.systemPrompt = override.systemPrompt;
-	if (override.skills !== undefined) next.skills = override.skills === false ? undefined : [...override.skills];
-	if (override.tools !== undefined) {
-		const { tools, mcpDirectTools } = splitToolList(override.tools === false ? [] : override.tools);
-		next.tools = tools;
-		next.mcpDirectTools = mcpDirectTools;
-	}
-
-	return next;
-}
-
-export function applyBuiltinOverrides(
-	builtinAgents: AgentConfig[],
-	userSettings: SubagentSettings,
-	projectSettings: SubagentSettings,
-	userSettingsPath: string,
-	projectSettingsPath: string | null,
-): AgentConfig[] {
-	const projectBulkDisabled = projectSettings.disableBuiltins === true && projectSettingsPath !== null;
-	const userBulkDisabled = projectSettings.disableBuiltins === undefined && userSettings.disableBuiltins === true;
-
-	return builtinAgents.map((agent) => {
-		const projectOverride = projectSettings.overrides[agent.name];
-		if (projectOverride && projectSettingsPath) {
-			return applyBuiltinOverride(agent, projectOverride, { scope: "project", path: projectSettingsPath });
-		}
-
-		if (projectBulkDisabled && projectSettingsPath) {
-			return applyBuiltinOverride(agent, { disabled: true }, { scope: "project", path: projectSettingsPath });
-		}
-
-		const userOverride = userSettings.overrides[agent.name];
-		if (userOverride) {
-			return applyBuiltinOverride(agent, userOverride, { scope: "user", path: userSettingsPath });
-		}
-
-		if (userBulkDisabled) {
-			return applyBuiltinOverride(agent, { disabled: true }, { scope: "user", path: userSettingsPath });
-		}
-
-		return agent;
-	});
-}
-
-export function buildBuiltinOverrideConfig(
-	base: BuiltinAgentOverrideBase,
-	draft: Pick<
-		AgentConfig,
-		| "model"
-		| "fallbackModels"
-		| "fallbackThinkingLevels"
-		| "thinking"
-		| "systemPromptMode"
-		| "inheritProjectContext"
-		| "inheritSkills"
-		| "defaultContext"
-		| "disabled"
-		| "systemPrompt"
-		| "skills"
-		| "tools"
-		| "mcpDirectTools"
-	>,
-): BuiltinAgentOverrideConfig | undefined {
-	const override: BuiltinAgentOverrideConfig = {};
-
-	if (draft.model !== base.model) override.model = draft.model ?? false;
-	if (!arraysEqual(draft.fallbackModels, base.fallbackModels))
-		override.fallbackModels = draft.fallbackModels ? [...draft.fallbackModels] : false;
-	if (!arraysEqual(draft.fallbackThinkingLevels, base.fallbackThinkingLevels))
-		override.fallbackThinkingLevels = draft.fallbackThinkingLevels ? [...draft.fallbackThinkingLevels] : false;
-	if (draft.thinking !== base.thinking) override.thinking = draft.thinking ?? false;
-	if (draft.systemPromptMode !== base.systemPromptMode) override.systemPromptMode = draft.systemPromptMode;
-	if (draft.inheritProjectContext !== base.inheritProjectContext)
-		override.inheritProjectContext = draft.inheritProjectContext;
-	if (draft.inheritSkills !== base.inheritSkills) override.inheritSkills = draft.inheritSkills;
-	if (draft.defaultContext !== base.defaultContext) override.defaultContext = draft.defaultContext ?? false;
-	if (draft.disabled !== base.disabled) override.disabled = draft.disabled ?? false;
-	if (draft.systemPrompt !== base.systemPrompt) override.systemPrompt = draft.systemPrompt;
-	if (!arraysEqual(draft.skills, base.skills)) override.skills = draft.skills ? [...draft.skills] : false;
-
-	const baseTools = joinToolList(base);
-	const draftTools = joinToolList(draft);
-	if (!arraysEqual(draftTools, baseTools)) override.tools = draftTools ? [...draftTools] : false;
-	return Object.keys(override).length > 0 ? override : undefined;
-}
-
-export function saveBuiltinAgentOverride(
-	cwd: string,
-	name: string,
-	scope: "user" | "project",
-	override: BuiltinAgentOverrideConfig,
-): string {
-	const filePath = scope === "project" ? getProjectAgentSettingsPath(cwd) : getUserAgentSettingsPath();
-	if (!filePath) throw new Error("Project override is not available here. No project config root was found.");
-
-	const settings = readSettingsFileStrict(filePath);
-	const subagents =
-		settings.subagents && typeof settings.subagents === "object" && !Array.isArray(settings.subagents)
-			? { ...(settings.subagents as Record<string, unknown>) }
-			: {};
-	const agentOverrides =
-		subagents.agentOverrides &&
-		typeof subagents.agentOverrides === "object" &&
-		!Array.isArray(subagents.agentOverrides)
-			? { ...(subagents.agentOverrides as Record<string, unknown>) }
-			: {};
-
-	agentOverrides[name] = cloneOverrideValue(override);
-	subagents.agentOverrides = agentOverrides;
-	settings.subagents = subagents;
-	writeSettingsFile(filePath, settings);
-	return filePath;
-}
-
-export function removeBuiltinAgentOverride(cwd: string, name: string, scope: "user" | "project"): string {
-	const filePath = scope === "project" ? getProjectAgentSettingsPath(cwd) : getUserAgentSettingsPath();
-	if (!filePath) throw new Error("Project override is not available here. No project config root was found.");
-	if (!fs.existsSync(filePath)) return filePath;
-
-	const settings = readSettingsFileStrict(filePath);
-	const subagents = settings.subagents;
-	if (!subagents || typeof subagents !== "object" || Array.isArray(subagents)) return filePath;
-	const nextSubagents = { ...(subagents as Record<string, unknown>) };
-	const agentOverrides = nextSubagents.agentOverrides;
-	if (!agentOverrides || typeof agentOverrides !== "object" || Array.isArray(agentOverrides)) return filePath;
-
-	const nextOverrides = { ...(agentOverrides as Record<string, unknown>) };
-	delete nextOverrides[name];
-	if (Object.keys(nextOverrides).length > 0) nextSubagents.agentOverrides = nextOverrides;
-	else delete nextSubagents.agentOverrides;
-
-	if (Object.keys(nextSubagents).length > 0) settings.subagents = nextSubagents;
-	else delete settings.subagents;
-
-	writeSettingsFile(filePath, settings);
-	return filePath;
-}

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

@@ -1,69 +0,0 @@
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-import { fileURLToPath } from "node:url";
-import { CONFIG_DIR_NAME, getAgentConfigPaths, getProjectConfigDirs } from "@bastani/atomic";
-
-export function getUserAgentDirs(): string[] {
-	return getAgentConfigPaths("agents");
-}
-
-export function isDirectory(p: string): boolean {
-	try {
-		return fs.statSync(p).isDirectory();
-	} catch {
-		return false;
-	}
-}
-
-export function findNearestProjectRoot(cwd: string): string | null {
-	let currentDir = cwd;
-	while (true) {
-		if (getProjectConfigDirs(currentDir).some(isDirectory) || isDirectory(path.join(currentDir, ".agents"))) {
-			return currentDir;
-		}
-
-		const parentDir = path.dirname(currentDir);
-		if (parentDir === currentDir) return null;
-		currentDir = parentDir;
-	}
-}
-
-export function getUserAgentSettingsPath(): string {
-	return getAgentConfigPaths("settings.json")[0] ?? path.join(os.homedir(), CONFIG_DIR_NAME, "agent", "settings.json");
-}
-
-export function getUserAgentSettingsPaths(): string[] {
-	return getAgentConfigPaths("settings.json");
-}
-
-export function getProjectAgentSettingsPath(cwd: string): string | null {
-	const projectRoot = findNearestProjectRoot(cwd);
-	return projectRoot ? path.join(getProjectConfigDirs(projectRoot)[0]!, "settings.json") : null;
-}
-
-export function getProjectAgentSettingsPaths(cwd: string): string[] {
-	const projectRoot = findNearestProjectRoot(cwd);
-	return projectRoot ? getProjectConfigDirs(projectRoot).map((dir) => path.join(dir, "settings.json")) : [];
-}
-
-export function resolveNearestProjectAgentDirs(cwd: string): { readDirs: string[]; preferredDir: string | null } {
-	const projectRoot = findNearestProjectRoot(cwd);
-	if (!projectRoot) return { readDirs: [], preferredDir: null };
-
-	const legacyDir = path.join(projectRoot, ".agents");
-	const preferredDir = path.join(getProjectConfigDirs(projectRoot)[0]!, "agents");
-	const readDirs: string[] = [];
-	if (isDirectory(legacyDir)) readDirs.push(legacyDir);
-	for (const configDir of getProjectConfigDirs(projectRoot).reverse()) {
-		const agentsDir = path.join(configDir, "agents");
-		if (isDirectory(agentsDir)) readDirs.push(agentsDir);
-	}
-
-	return {
-		readDirs,
-		preferredDir,
-	};
-}
-
-export const BUILTIN_AGENTS_DIR = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..", "..", "agents");

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

@@ -1,6 +0,0 @@
-import type { AgentScope } from "./agents.js";
-
-export function resolveExecutionAgentScope(scope: unknown): AgentScope {
-	if (scope === "user" || scope === "project" || scope === "both") return scope;
-	return "both";
-}

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

@@ -1,23 +0,0 @@
-import type { AgentConfig, AgentScope } from "./agents.js";
-
-export function mergeAgentsForScope(
-	scope: AgentScope,
-	userAgents: AgentConfig[],
-	projectAgents: AgentConfig[],
-	builtinAgents: AgentConfig[] = [],
-): AgentConfig[] {
-	const agentMap = new Map<string, AgentConfig>();
-
-	for (const agent of builtinAgents) agentMap.set(agent.name, agent);
-
-	if (scope === "both") {
-		for (const agent of userAgents) agentMap.set(agent.name, agent);
-		for (const agent of projectAgents) agentMap.set(agent.name, agent);
-	} else if (scope === "user") {
-		for (const agent of userAgents) agentMap.set(agent.name, agent);
-	} else {
-		for (const agent of projectAgents) agentMap.set(agent.name, agent);
-	}
-
-	return Array.from(agentMap.values());
-}

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

@@ -1,113 +0,0 @@
-import { stringify } from "yaml";
-import type { AgentConfig } from "./agents.js";
-import type { FrontmatterValue } from "./frontmatter.js";
-import { frontmatterNameForConfig } from "./identity.js";
-
-export const KNOWN_FIELDS = new Set([
-	"name",
-	"package",
-	"description",
-	"tools",
-	"model",
-	"fallbackModels",
-	"fallbackThinkingLevels",
-	"thinking",
-	"systemPromptMode",
-	"inheritProjectContext",
-	"inheritSkills",
-	"defaultContext",
-	"skill",
-	"skills",
-	"extensions",
-	"output",
-	"defaultReads",
-	"defaultProgress",
-	"interactive",
-]);
-
-/**
- * Frontmatter keys whose feature is gone. They are neither preserved on load nor
- * re-emitted on serialize, so an agent-management rewrite strips a stale key from
- * the file. Each is spelled with a splice so the removed identifier never appears
- * as contiguous source text.
- */
-const REMOVED_AGENT_FRONTMATTER_FIELDS = new Set<string>([`completion${"Guard"}`, `maxSubagent${"Depth"}`]);
-
-export function shouldPreserveAgentExtraField(key: string): boolean {
-	return !KNOWN_FIELDS.has(key) && !REMOVED_AGENT_FRONTMATTER_FIELDS.has(key);
-}
-
-function joinComma(values: string[] | undefined): string | undefined {
-	if (!values || values.length === 0) return undefined;
-	return values.join(", ");
-}
-
-/**
- * Render one extra frontmatter field through the real YAML emitter so an
- * unknown custom field survives an agent update without changing meaning.
- * The yaml package quotes scalars a hand-joined spelling corrupts: `a #b`
- * would open a comment mid-flow-sequence and leave the file unparseable,
- * `a, b` would split into two items, `a: b` would start a nested mapping,
- * and an unquoted `""` or `"true"` would change type on re-parse. Keys are
- * emitted by the same call, so a custom key that needs quoting gets it.
- * Sequences render as block collections and may span several lines; nested
- * maps are dropped at parse (they have no agent-frontmatter spelling) and
- * so never reach this path. Every emitted shape re-parses to itself, so
- * load → serialize → load is stable.
- */
-function serializeExtraFieldEntry(key: string, value: FrontmatterValue): string[] {
-	return stringify({ [key]: value }, { lineWidth: 0 })
-		.trimEnd()
-		.split("\n");
-}
-
-export function serializeAgent(config: AgentConfig): string {
-	const lines: string[] = [];
-	lines.push("---");
-	lines.push(`name: ${frontmatterNameForConfig(config)}`);
-	if (config.packageName) lines.push(`package: ${config.packageName}`);
-	lines.push(`description: ${config.description}`);
-
-	const tools = [...(config.tools ?? []), ...(config.mcpDirectTools ?? []).map((tool) => `mcp:${tool}`)];
-	const toolsValue = joinComma(tools);
-	if (toolsValue) lines.push(`tools: ${toolsValue}`);
-
-	if (config.model) lines.push(`model: ${config.model}`);
-	const fallbackModelsValue = joinComma(config.fallbackModels);
-	if (fallbackModelsValue) lines.push(`fallbackModels: ${fallbackModelsValue}`);
-	const fallbackThinkingLevelsValue = joinComma(config.fallbackThinkingLevels);
-	if (fallbackThinkingLevelsValue) lines.push(`fallbackThinkingLevels: ${fallbackThinkingLevelsValue}`);
-	if (config.thinking && config.thinking !== "off") lines.push(`thinking: ${config.thinking}`);
-	lines.push(`systemPromptMode: ${config.systemPromptMode}`);
-	lines.push(`inheritProjectContext: ${config.inheritProjectContext ? "true" : "false"}`);
-	lines.push(`inheritSkills: ${config.inheritSkills ? "true" : "false"}`);
-	if (config.defaultContext) lines.push(`defaultContext: ${config.defaultContext}`);
-
-	const skillsValue = joinComma(config.skills);
-	if (skillsValue) lines.push(`skills: ${skillsValue}`);
-
-	if (config.extensions !== undefined) {
-		const extensionsValue = joinComma(config.extensions);
-		lines.push(`extensions: ${extensionsValue ?? ""}`);
-	}
-
-	if (config.output) lines.push(`output: ${config.output}`);
-
-	const readsValue = joinComma(config.defaultReads);
-	if (readsValue) lines.push(`defaultReads: ${readsValue}`);
-
-	if (config.defaultProgress) lines.push("defaultProgress: true");
-	if (config.interactive) lines.push("interactive: true");
-
-	if (config.extraFields) {
-		for (const [key, value] of Object.entries(config.extraFields)) {
-			if (!shouldPreserveAgentExtraField(key)) continue;
-			lines.push(...serializeExtraFieldEntry(key, value));
-		}
-	}
-
-	lines.push("---");
-
-	const body = config.systemPrompt ?? "";
-	return `${lines.join("\n")}\n\n${body}\n`;
-}