@emiliosp/pi-maestro 0.17.0 → 0.18.0

package/node_modules/pi-subagents/src/agents/agent-management.js

@@ -1,1522 +0,0 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { defaultInheritProjectContext, defaultInheritSkills, defaultSystemPromptMode, discoverAgentsAll, buildRuntimeName, findBlockingAgentDiagnostic, frontmatterNameForConfig, parsePackageName, mergeBuiltinAgentOverride, removeBuiltinAgentOverride, removeBuiltinAgentOverrideFields, resolveAgentName, } from "./agents.js";
-import { serializeAgent } from "./agent-serializer.js";
-import { mergeAgentsForScope } from "./agent-selection.js";
-import { discoverAvailableSkills, resolveSkills } from "./skills.js";
-import { buildProactiveSkillSubagentRecommendationLines, } from "./proactive-skills.js";
-import { parseFrontmatter, parseFrontmatterList } from "./frontmatter.js";
-import { resolveEffectiveThinking, toModelInfo } from "../shared/model-info.js";
-import { resolveSubagentModelOverride } from "../runs/shared/model-resolution.js";
-import { validateToolBudgetConfig } from "../runs/shared/tool-budget.js";
-import { formatReviewGateLabel, validateAcceptanceInput } from "../runs/shared/acceptance.js";
-import { CODE_OWNED_EXTERNAL_CLI_ADAPTER_LABEL, isCodeOwnedExternalCliAdapterId, resolveExternalCliRunnerStatus, validateCodeOwnedProfileRunner } from "../runs/shared/external-cli-contract.js";
-import { resolveExternalCliBinaryAvailability } from "../runs/shared/external-cli-preflight.js";
-import { getProjectConfigDir } from "../shared/utils.js";
-import { previewDisplayText } from "../shared/display-text.js";
-import { capabilityCeilingAgentRestrictionSources, isAgentAllowedByCapabilityCeiling, resolveCurrentSubagentCapabilityCeiling } from "../runs/shared/capability-ceiling.js";
-import { mergeRuntimeAgents } from "./runtime-agent-registry.js";
-import { listExternalJobProviders } from "../api/external-job-provider.js";
-function discoverCatalog(ctx, cwd = ctx.cwd, provider) {
-    return (ctx.discoverAgentsAll ?? discoverAgentsAll)(cwd, provider);
-}
-function result(text, isError = false, details) {
-    return { content: [{ type: "text", text }], isError, details: { mode: "management", results: [], ...details } };
-}
-function jsonDetails(value) {
-    return JSON.parse(JSON.stringify(value));
-}
-function presentDetails(value) {
-    const cleaned = jsonDetails(value);
-    return Object.keys(cleaned).length > 0 ? cleaned : undefined;
-}
-function parseCsv(value) {
-    return [...new Set(value.split(",").map((v) => v.trim()).filter(Boolean))];
-}
-function configObject(config) {
-    let val = config;
-    if (typeof val === "string") {
-        try {
-            val = JSON.parse(val);
-        }
-        catch (error) {
-            const message = error instanceof Error ? error.message : String(error);
-            return { status: "error", message: `config must be valid JSON: ${message}` };
-        }
-    }
-    if (!val || typeof val !== "object" || Array.isArray(val))
-        return { status: "missing" };
-    return { status: "ok", value: val };
-}
-function hasKey(obj, key) {
-    return Object.prototype.hasOwnProperty.call(obj, key);
-}
-function isJsonSerializable(value) {
-    if (value === null || typeof value === "string" || typeof value === "boolean")
-        return true;
-    if (typeof value === "number")
-        return Number.isFinite(value);
-    if (Array.isArray(value))
-        return value.every(isJsonSerializable);
-    if (value && typeof value === "object")
-        return Object.values(value).every(isJsonSerializable);
-    return false;
-}
-function asDisambiguationScope(scope) {
-    if (scope === "user" || scope === "project")
-        return scope;
-    return undefined;
-}
-function actionScope(scope, action) {
-    if (scope === undefined)
-        return { scope: "user" };
-    const parsed = asDisambiguationScope(scope);
-    return parsed ? { scope: parsed } : { error: result(`agentScope must be 'user' or 'project' for ${action}.`, true) };
-}
-function normalizeListScope(scope) {
-    if (scope === undefined)
-        return "both";
-    if (scope === "user" || scope === "project" || scope === "both")
-        return scope;
-    return undefined;
-}
-function sanitizeName(name) {
-    return name.toLowerCase().trim().replace(/\s+/g, "-").replace(/[^a-z0-9-]/g, "").replace(/-+/g, "-").replace(/^-+|-+$/g, "");
-}
-function parsePackageConfig(value) {
-    return parsePackageName(value, "config.package");
-}
-function allAgents(d) {
-    return [...d.builtin, ...d.package, ...d.user, ...d.project];
-}
-function effectiveAgentsForScope(scope, d, runtimeAgentOwner, preferredModelProvider) {
-    let agents = mergeAgentsForScope(scope, d.user, d.project, d.builtin, d.package);
-    if (runtimeAgentOwner) {
-        agents = mergeRuntimeAgents(runtimeAgentOwner, { agents }, allAgents(d), { cwd: d.cwd, scope, preferredModelProvider }).agents;
-    }
-    return agents;
-}
-function availableAgentNamesFromDiscovery(d, runtimeAgentOwner, preferredModelProvider) {
-    const agents = runtimeAgentOwner ? effectiveAgentsForScope("both", d, runtimeAgentOwner, preferredModelProvider) : allAgents(d);
-    return [...new Set(agents.map((agent) => agent.name))].sort((a, b) => a.localeCompare(b));
-}
-function availableAgentNames(ctx, cwd = ctx.cwd) {
-    return availableAgentNamesFromDiscovery(discoverCatalog(ctx, cwd));
-}
-function findAgentsInDiscovery(name, d, scope = "both", runtimeAgentOwner, preferredModelProvider) {
-    const raw = name.trim();
-    const sanitized = sanitizeName(raw);
-    const scoped = effectiveAgentsForScope(scope, d, runtimeAgentOwner, preferredModelProvider);
-    let resolved = resolveAgentName(raw, scoped);
-    if (!resolved.agent && !resolved.error && sanitized !== raw)
-        resolved = resolveAgentName(sanitized, scoped);
-    if (resolved.agent)
-        return scoped.filter((agent) => agent.name === resolved.agent.name).sort((a, b) => a.source.localeCompare(b.source));
-    return scoped
-        .filter((agent) => Boolean(resolveAgentName(raw, [agent]).agent)
-        || (sanitized !== raw && Boolean(resolveAgentName(sanitized, [agent]).agent)))
-        .sort((a, b) => a.source.localeCompare(b.source));
-}
-function findAgents(name, ctx, scope = "both") {
-    return findAgentsInDiscovery(name, discoverCatalog(ctx), scope);
-}
-function diagnosticsForScope(diagnostics, scope) {
-    if (scope === "both")
-        return diagnostics;
-    const excludedSource = scope === "user" ? "project" : "user";
-    return diagnostics?.filter((diagnostic) => diagnostic.source !== excludedSource);
-}
-const AGENT_SOURCE_PRECEDENCE = { builtin: 0, package: 1, user: 2, project: 3, runtime: 4 };
-// Returns the highest-precedence definition for a resolved canonical name (project > user > package > builtin),
-// matching mergeAgentsForScope for "both", including disabled agents so disable/enable can locate hidden targets.
-function resolveEffectiveAgent(d, name) {
-    const raw = name.trim();
-    const candidates = allAgents(d);
-    let resolved = resolveAgentName(raw, candidates);
-    if (!resolved.agent && !resolved.error) {
-        const sanitized = sanitizeName(raw);
-        if (sanitized !== raw)
-            resolved = resolveAgentName(sanitized, candidates);
-    }
-    if (resolved.error)
-        return { error: resolved.error };
-    if (!resolved.agent)
-        return {};
-    const matches = candidates.filter((agent) => agent.name === resolved.agent.name);
-    return { agent: matches.reduce((best, agent) => (AGENT_SOURCE_PRECEDENCE[agent.source] > AGENT_SOURCE_PRECEDENCE[best.source] ? agent : best)) };
-}
-function nameExistsInScope(ctx, scope, name, excludePath) {
-    const d = discoverCatalog(ctx);
-    for (const a of scope === "user" ? d.user : d.project) {
-        if (a.name === name && a.filePath !== excludePath)
-            return true;
-    }
-    return false;
-}
-function isMutableSource(source) {
-    return source === "user" || source === "project";
-}
-function modelWarning(ctx, model) {
-    if (!model)
-        return undefined;
-    const found = ctx.modelRegistry.getAvailable().some((m) => `${m.provider}/${m.id}` === model || m.id === model);
-    if (found)
-        return undefined;
-    return `Warning: model '${model}' is not in the current model registry. Run subagent({ action: "models" }) to list valid provider/id selectors, then use the exact provider/id form (bare ids resolve only when unique).`;
-}
-function skillsWarning(cwd, agent) {
-    if (!agent.skills?.length)
-        return undefined;
-    const { missing } = resolveSkills(agent.skills, cwd, agent.skillPath, agent.filePath ? path.dirname(agent.filePath) : cwd);
-    return missing.length ? `Warning: skills not found: ${missing.join(", ")}.` : undefined;
-}
-function withDeclaredExtensionPaths(config, filePath) {
-    const { frontmatter } = parseFrontmatter(fs.readFileSync(filePath, "utf-8"));
-    const { extensions: _extensions, subagentOnlyExtensions: _subagentOnlyExtensions, ...withoutResolvedExtensions } = config;
-    return {
-        ...withoutResolvedExtensions,
-        ...(frontmatter.extensions !== undefined ? { extensions: parseFrontmatterList(frontmatter.extensions) } : {}),
-        ...(frontmatter.subagentOnlyExtensions !== undefined
-            ? { subagentOnlyExtensions: parseFrontmatterList(frontmatter.subagentOnlyExtensions) }
-            : {}),
-    };
-}
-export function editableAgentConfig(agent) {
-    const { extensions: _extensions, ...withoutExtensions } = agent;
-    const base = agent.override?.base;
-    const description = base?.description ?? agent.description;
-    const frontmatterFields = agent.source === "builtin" || agent.source === "runtime" ? undefined : readAgentFrontmatterFields(agent.filePath);
-    const hasDeclaredField = (...fields) => frontmatterFields === undefined || fields.some((field) => frontmatterFields.has(field));
-    const withoutSettingsDefaults = (config) => {
-        if (!frontmatterFields)
-            return config;
-        const next = { ...config };
-        if (!hasDeclaredField("model"))
-            delete next.model;
-        if (!hasDeclaredField("thinking"))
-            delete next.thinking;
-        return next;
-    };
-    const { override: _override, description: _description, output: _output, outputMode: _outputMode, defaultReads: _defaultReads, model: _model, fast: _fast, thinking: _thinking, systemPromptMode: _systemPromptMode, inheritProjectContext: _inheritProjectContext, inheritGlobalContext: _inheritGlobalContext, inheritSkills: _inheritSkills, defaultContext: _defaultContext, acceptanceRole: _acceptanceRole, disabled: _disabled, systemPrompt: _systemPrompt, skills: _skills, skillPath: _skillPath, tools: _tools, excludeTools: _excludeTools, mcpDirectTools: _mcpDirectTools, allowNestedSubagents: _allowNestedSubagents, allowedAgents: _allowedAgents, subagentOnlyExtensions: _subagentOnlyExtensions, mutationTools: _mutationTools, toolBudget: _toolBudget, ...editable } = withoutExtensions;
-    if (!base) {
-        return withDeclaredExtensionPaths(withoutSettingsDefaults({
-            ...withoutExtensions,
-            ...(agent.extensionsFromDefault ? {} : agent.extensions !== undefined ? { extensions: [...agent.extensions] } : {}),
-        }), agent.filePath);
-    }
-    return withDeclaredExtensionPaths({
-        ...editable,
-        description,
-        ...(base.output !== undefined ? { output: base.output } : {}),
-        ...(base.outputMode !== undefined ? { outputMode: base.outputMode } : {}),
-        ...(base.defaultReads !== undefined ? { defaultReads: [...base.defaultReads] } : {}),
-        ...(base.model !== undefined && hasDeclaredField("model") ? { model: base.model } : {}),
-        ...(base.fast !== undefined ? { fast: base.fast } : {}),
-        ...(base.thinking !== undefined && hasDeclaredField("thinking") ? { thinking: base.thinking } : {}),
-        systemPromptMode: base.systemPromptMode,
-        inheritProjectContext: base.inheritProjectContext,
-        inheritGlobalContext: base.inheritGlobalContext,
-        inheritSkills: base.inheritSkills,
-        ...(base.defaultContext !== undefined ? { defaultContext: base.defaultContext } : {}),
-        ...(base.acceptanceRole !== undefined ? { acceptanceRole: base.acceptanceRole } : {}),
-        ...(base.disabled !== undefined ? { disabled: base.disabled } : {}),
-        systemPrompt: base.systemPrompt,
-        ...(base.skills !== undefined ? { skills: [...base.skills] } : {}),
-        ...(base.skillPath !== undefined ? { skillPath: [...base.skillPath] } : {}),
-        ...(base.tools !== undefined ? { tools: [...base.tools] } : {}),
-        ...(base.excludeTools !== undefined ? { excludeTools: [...base.excludeTools] } : {}),
-        ...(base.mcpDirectTools !== undefined ? { mcpDirectTools: [...base.mcpDirectTools] } : {}),
-        ...(base.allowNestedSubagents !== undefined ? { allowNestedSubagents: base.allowNestedSubagents } : {}),
-        ...(base.allowedAgents !== undefined ? { allowedAgents: [...base.allowedAgents] } : {}),
-        ...(base.extensions !== undefined ? { extensions: [...base.extensions] } : {}),
-        ...(base.subagentOnlyExtensions !== undefined ? { subagentOnlyExtensions: [...base.subagentOnlyExtensions] } : {}),
-        ...(base.mutationTools !== undefined ? { mutationTools: [...base.mutationTools] } : {}),
-        ...(base.toolBudget !== undefined ? { toolBudget: base.toolBudget } : {}),
-    }, agent.filePath);
-}
-function readAgentFrontmatterFields(filePath) {
-    try {
-        const { frontmatter } = parseFrontmatter(fs.readFileSync(filePath, "utf-8"));
-        return new Set(Object.keys(frontmatter));
-    }
-    catch {
-        return new Set();
-    }
-}
-export function preservedAgentFrontmatterFields(agent, cfg) {
-    const fields = readAgentFrontmatterFields(agent.filePath);
-    const changed = (...names) => {
-        for (const name of names)
-            fields.delete(name);
-    };
-    if (hasKey(cfg, "name"))
-        changed("name");
-    if (hasKey(cfg, "package"))
-        changed("package");
-    if (hasKey(cfg, "description"))
-        changed("description");
-    if (hasKey(cfg, "advertise"))
-        changed("advertise");
-    if (hasKey(cfg, "aliases"))
-        changed("alias", "aliases");
-    if (hasKey(cfg, "systemPrompt"))
-        changed("systemPrompt");
-    if (hasKey(cfg, "runner"))
-        changed("runner");
-    if (hasKey(cfg, "model"))
-        changed("model");
-    if (hasKey(cfg, "tools"))
-        changed("tools");
-    if (hasKey(cfg, "excludeTools"))
-        changed("excludeTools");
-    if (hasKey(cfg, "skills"))
-        changed("skill", "skills");
-    if (hasKey(cfg, "skillPath"))
-        changed("skillPath");
-    if (hasKey(cfg, "extensions"))
-        changed("extensions");
-    if (hasKey(cfg, "subagentOnlyExtensions"))
-        changed("subagentOnlyExtensions");
-    if (hasKey(cfg, "mutationTools"))
-        changed("mutationTools");
-    if (hasKey(cfg, "thinking")) {
-        changed("thinking");
-        if (cfg.thinking === "off")
-            fields.add("thinking");
-    }
-    if (hasKey(cfg, "systemPromptMode")) {
-        changed("systemPromptMode");
-        fields.add("systemPromptMode");
-    }
-    if (hasKey(cfg, "inheritProjectContext")) {
-        changed("inheritProjectContext");
-        fields.add("inheritProjectContext");
-    }
-    if (hasKey(cfg, "inheritGlobalContext")) {
-        changed("inheritGlobalContext");
-        fields.add("inheritGlobalContext");
-    }
-    if (hasKey(cfg, "inheritSkills")) {
-        changed("inheritSkills");
-        fields.add("inheritSkills");
-    }
-    if (hasKey(cfg, "defaultContext"))
-        changed("defaultContext");
-    if (hasKey(cfg, "async"))
-        changed("async");
-    if (hasKey(cfg, "timeoutMs"))
-        changed("timeoutMs");
-    if (hasKey(cfg, "acceptance"))
-        changed("acceptance");
-    if (hasKey(cfg, "acceptanceRole"))
-        changed("acceptanceRole");
-    if (hasKey(cfg, "output"))
-        changed("output");
-    if (hasKey(cfg, "outputMode"))
-        changed("outputMode");
-    if (hasKey(cfg, "reads"))
-        changed("defaultReads");
-    if (hasKey(cfg, "progress"))
-        changed("defaultProgress");
-    if (hasKey(cfg, "maxSubagentDepth"))
-        changed("maxSubagentDepth");
-    if (hasKey(cfg, "toolBudget"))
-        changed("toolBudget");
-    return fields;
-}
-function parseTools(raw) {
-    const tools = [];
-    const mcpDirectTools = [];
-    for (const item of parseCsv(raw)) {
-        if (item.startsWith("mcp:")) {
-            const direct = item.slice(4).trim();
-            if (direct)
-                mcpDirectTools.push(direct);
-        }
-        else
-            tools.push(item);
-    }
-    return {
-        ...(tools.length ? { tools } : {}),
-        ...(mcpDirectTools.length ? { mcpDirectTools } : {}),
-    };
-}
-function applyAgentConfig(target, cfg) {
-    if (hasKey(cfg, "advertise")) {
-        if (cfg.advertise === "")
-            delete target.advertise;
-        else if (typeof cfg.advertise === "boolean")
-            target.advertise = cfg.advertise;
-        else
-            return "config.advertise must be a boolean or empty string when provided.";
-    }
-    if (hasKey(cfg, "aliases")) {
-        if (cfg.aliases === false || cfg.aliases === "")
-            delete target.aliases;
-        else if (typeof cfg.aliases === "string") {
-            const aliases = parseCsv(cfg.aliases).filter((alias) => alias !== target.name);
-            if (aliases.length)
-                target.aliases = aliases;
-            else
-                delete target.aliases;
-        }
-        else if (Array.isArray(cfg.aliases) && cfg.aliases.every((entry) => typeof entry === "string")) {
-            const aliases = [...new Set(cfg.aliases.map((entry) => entry.trim()).filter(Boolean).filter((alias) => alias !== target.name))];
-            if (aliases.length)
-                target.aliases = aliases;
-            else
-                delete target.aliases;
-        }
-        else
-            return "config.aliases must be a comma-separated string, string array, or false when provided.";
-    }
-    if (hasKey(cfg, "systemPrompt")) {
-        if (cfg.systemPrompt === false || cfg.systemPrompt === "")
-            target.systemPrompt = "";
-        else if (typeof cfg.systemPrompt === "string")
-            target.systemPrompt = cfg.systemPrompt;
-        else
-            return "config.systemPrompt must be a string or false when provided.";
-    }
-    if (hasKey(cfg, "runner")) {
-        if (cfg.runner === false || cfg.runner === "")
-            delete target.runner;
-        else if (cfg.runner && typeof cfg.runner === "object" && !Array.isArray(cfg.runner)) {
-            const runner = cfg.runner;
-            if (runner.type === "pi" && Object.keys(runner).every((key) => key === "type"))
-                target.runner = { type: "pi" };
-            else if (runner.type === "external-cli" && typeof runner.command === "string" && runner.command.trim()
-                && (runner.args === undefined || (Array.isArray(runner.args) && runner.args.every((arg) => typeof arg === "string")))
-                && (runner.adapter === undefined || isCodeOwnedExternalCliAdapterId(runner.adapter))
-                && (runner.adapter === undefined || runner.args === undefined || runner.args.length === 0)
-                && (runner.promptDelivery === undefined || runner.promptDelivery === "stdin")
-                && Object.keys(runner).every((key) => ["type", "adapter", "command", "args", "promptDelivery"].includes(key))) {
-                const runnerArgs = Array.isArray(runner.args) ? runner.args.filter((arg) => typeof arg === "string") : undefined;
-                target.runner = { type: "external-cli", ...(isCodeOwnedExternalCliAdapterId(runner.adapter) ? { adapter: runner.adapter } : {}), command: runner.command.trim(), ...(runnerArgs?.length ? { args: runnerArgs } : {}), ...(runner.promptDelivery ? { promptDelivery: "stdin" } : {}) };
-            }
-            else if (runner.type === "external-job" && typeof runner.provider === "string" && runner.provider.trim() === runner.provider && runner.provider
-                && (runner.options === undefined || (runner.options && typeof runner.options === "object" && !Array.isArray(runner.options) && isJsonSerializable(runner.options)))
-                && Object.keys(runner).every((key) => ["type", "provider", "options"].includes(key))) {
-                target.runner = { type: "external-job", provider: runner.provider, ...(runner.options ? { options: runner.options } : {}) };
-            }
-            else
-                return `config.runner must be { type: 'pi' }, { type: 'external-cli', adapter?: ${CODE_OWNED_EXTERNAL_CLI_ADAPTER_LABEL}, command: string, args?: string[], promptDelivery?: 'stdin' }, or { type: 'external-job', provider: string, options?: object }.`;
-        }
-        else
-            return "config.runner must be an object, false, or empty string when provided.";
-    }
-    if (hasKey(cfg, "launcher"))
-        return "config.launcher is not supported by agent management; edit the agent file's 'launcher' frontmatter directly.";
-    if (target.launcher && (target.runner?.type === "external-cli" || target.runner?.type === "external-job")) {
-        return `config.runner type '${target.runner.type}' cannot be combined with this agent's 'launcher' frontmatter.`;
-    }
-    if (hasKey(cfg, "model")) {
-        if (cfg.model === false || cfg.model === "")
-            delete target.model;
-        else if (typeof cfg.model === "string") {
-            const model = cfg.model.trim();
-            if (model)
-                target.model = model;
-            else
-                delete target.model;
-        }
-        else
-            return "config.model must be a string or false when provided.";
-    }
-    if (hasKey(cfg, "fallbackModels"))
-        return "config.fallbackModels was removed; configure one model instead.";
-    if (hasKey(cfg, "tools")) {
-        if (cfg.tools === false || cfg.tools === "") {
-            delete target.tools;
-            delete target.mcpDirectTools;
-        }
-        else if (typeof cfg.tools === "string") {
-            const parsed = parseTools(cfg.tools);
-            if (parsed.tools)
-                target.tools = parsed.tools;
-            else
-                delete target.tools;
-            if (parsed.mcpDirectTools)
-                target.mcpDirectTools = parsed.mcpDirectTools;
-            else
-                delete target.mcpDirectTools;
-        }
-        else
-            return "config.tools must be a comma-separated string or false when provided.";
-    }
-    if (hasKey(cfg, "excludeTools")) {
-        if (cfg.excludeTools === false || cfg.excludeTools === "")
-            delete target.excludeTools;
-        else if (typeof cfg.excludeTools === "string") {
-            const excludeTools = parseCsv(cfg.excludeTools);
-            if (excludeTools.length)
-                target.excludeTools = [...new Set(excludeTools)];
-            else
-                delete target.excludeTools;
-        }
-        else if (Array.isArray(cfg.excludeTools) && cfg.excludeTools.every((entry) => typeof entry === "string")) {
-            const excludeTools = [...new Set(cfg.excludeTools.map((entry) => entry.trim()).filter(Boolean))];
-            if (excludeTools.length)
-                target.excludeTools = excludeTools;
-            else
-                delete target.excludeTools;
-        }
-        else
-            return "config.excludeTools must be a comma-separated string, string array, or false when provided.";
-    }
-    if (hasKey(cfg, "skills")) {
-        if (cfg.skills === false || cfg.skills === "")
-            delete target.skills;
-        else if (typeof cfg.skills === "string") {
-            const skills = parseCsv(cfg.skills);
-            if (skills.length)
-                target.skills = skills;
-            else
-                delete target.skills;
-        }
-        else
-            return "config.skills must be a comma-separated string or false when provided.";
-    }
-    if (hasKey(cfg, "skillPath")) {
-        if (cfg.skillPath === false || cfg.skillPath === "")
-            delete target.skillPath;
-        else if (typeof cfg.skillPath === "string") {
-            const skillPath = parseCsv(cfg.skillPath);
-            if (skillPath.length)
-                target.skillPath = skillPath;
-            else
-                delete target.skillPath;
-        }
-        else if (Array.isArray(cfg.skillPath) && cfg.skillPath.every((entry) => typeof entry === "string")) {
-            const skillPath = [...new Set(cfg.skillPath.map((entry) => entry.trim()).filter(Boolean))];
-            if (skillPath.length)
-                target.skillPath = skillPath;
-            else
-                delete target.skillPath;
-        }
-        else
-            return "config.skillPath must be a comma-separated string, string array, or false when provided.";
-    }
-    if (hasKey(cfg, "extensions")) {
-        if (cfg.extensions === false)
-            delete target.extensions;
-        else if (cfg.extensions === "")
-            target.extensions = [];
-        else if (typeof cfg.extensions === "string")
-            target.extensions = parseCsv(cfg.extensions);
-        else
-            return "config.extensions must be a comma-separated string, empty string, or false when provided.";
-    }
-    if (hasKey(cfg, "subagentOnlyExtensions")) {
-        if (cfg.subagentOnlyExtensions === false)
-            delete target.subagentOnlyExtensions;
-        else if (cfg.subagentOnlyExtensions === "")
-            target.subagentOnlyExtensions = [];
-        else if (typeof cfg.subagentOnlyExtensions === "string")
-            target.subagentOnlyExtensions = parseCsv(cfg.subagentOnlyExtensions);
-        else
-            return "config.subagentOnlyExtensions must be a comma-separated string, empty string, or false when provided.";
-    }
-    if (hasKey(cfg, "mutationTools")) {
-        if (cfg.mutationTools === false)
-            delete target.mutationTools;
-        else if (cfg.mutationTools === "")
-            target.mutationTools = [];
-        else if (typeof cfg.mutationTools === "string")
-            target.mutationTools = parseCsv(cfg.mutationTools);
-        else
-            return "config.mutationTools must be a comma-separated string, empty string, or false when provided.";
-    }
-    if (hasKey(cfg, "thinking")) {
-        if (cfg.thinking === false || cfg.thinking === "")
-            delete target.thinking;
-        else if (typeof cfg.thinking === "string") {
-            const thinking = cfg.thinking.trim();
-            if (thinking)
-                target.thinking = thinking;
-            else
-                delete target.thinking;
-        }
-        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;
-        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.";
-        target.inheritProjectContext = cfg.inheritProjectContext;
-    }
-    if (hasKey(cfg, "inheritGlobalContext")) {
-        if (typeof cfg.inheritGlobalContext !== "boolean")
-            return "config.inheritGlobalContext must be a boolean when provided.";
-        target.inheritGlobalContext = cfg.inheritGlobalContext;
-    }
-    if (hasKey(cfg, "inheritSkills")) {
-        if (typeof cfg.inheritSkills !== "boolean")
-            return "config.inheritSkills must be a boolean when provided.";
-        target.inheritSkills = cfg.inheritSkills;
-    }
-    if (hasKey(cfg, "defaultContext")) {
-        if (cfg.defaultContext === false || cfg.defaultContext === "")
-            delete target.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, "async")) {
-        if (cfg.async === "")
-            delete target.defaultAsync;
-        else if (typeof cfg.async === "boolean")
-            target.defaultAsync = cfg.async;
-        else
-            return "config.async must be a boolean or empty string when provided.";
-    }
-    if (hasKey(cfg, "timeoutMs")) {
-        if (cfg.timeoutMs === false || cfg.timeoutMs === "")
-            delete target.defaultTimeoutMs;
-        else if (typeof cfg.timeoutMs === "number" && Number.isInteger(cfg.timeoutMs) && cfg.timeoutMs > 0)
-            target.defaultTimeoutMs = cfg.timeoutMs;
-        else
-            return "config.timeoutMs must be a positive integer or false when provided.";
-    }
-    if (hasKey(cfg, "acceptance")) {
-        if (cfg.acceptance === "")
-            delete target.defaultAcceptance;
-        else {
-            const errors = validateAcceptanceInput(cfg.acceptance, "config.acceptance");
-            if (errors.length > 0)
-                return errors.join(" ");
-            target.defaultAcceptance = cfg.acceptance;
-        }
-    }
-    if (hasKey(cfg, "acceptanceRole")) {
-        if (cfg.acceptanceRole === false || cfg.acceptanceRole === "")
-            delete target.acceptanceRole;
-        else if (cfg.acceptanceRole === "read-only" || cfg.acceptanceRole === "writer")
-            target.acceptanceRole = cfg.acceptanceRole;
-        else
-            return "config.acceptanceRole must be 'read-only', 'writer', or false when provided.";
-    }
-    if (hasKey(cfg, "output")) {
-        if (cfg.output === false || cfg.output === "")
-            delete target.output;
-        else if (typeof cfg.output === "string")
-            target.output = cfg.output;
-        else
-            return "config.output must be a string or false when provided.";
-    }
-    if (hasKey(cfg, "outputMode")) {
-        if (cfg.outputMode === "inline" || cfg.outputMode === "file-only")
-            target.outputMode = cfg.outputMode;
-        else
-            return "config.outputMode must be 'inline' or 'file-only' when provided.";
-    }
-    if (hasKey(cfg, "reads")) {
-        if (cfg.reads === false || cfg.reads === "")
-            delete target.defaultReads;
-        else if (typeof cfg.reads === "string") {
-            const reads = parseCsv(cfg.reads);
-            if (reads.length)
-                target.defaultReads = reads;
-            else
-                delete target.defaultReads;
-        }
-        else
-            return "config.reads must be a comma-separated string or false when provided.";
-    }
-    if (hasKey(cfg, "progress")) {
-        if (typeof cfg.progress !== "boolean")
-            return "config.progress must be a boolean when provided.";
-        target.defaultProgress = cfg.progress;
-    }
-    if (hasKey(cfg, "maxSubagentDepth")) {
-        if (cfg.maxSubagentDepth === false || cfg.maxSubagentDepth === "")
-            delete target.maxSubagentDepth;
-        else if (typeof cfg.maxSubagentDepth === "number" && Number.isInteger(cfg.maxSubagentDepth) && cfg.maxSubagentDepth >= 0) {
-            target.maxSubagentDepth = cfg.maxSubagentDepth;
-        }
-        else
-            return "config.maxSubagentDepth must be an integer >= 0 or false when provided.";
-    }
-    if (hasKey(cfg, "toolBudget")) {
-        if (cfg.toolBudget === false || cfg.toolBudget === "")
-            delete target.toolBudget;
-        else {
-            const validation = validateToolBudgetConfig(cfg.toolBudget, "config.toolBudget");
-            if (validation.error)
-                return validation.error;
-            target.toolBudget = cfg.toolBudget;
-        }
-    }
-    if (target.runner?.type === "external-cli" || target.runner?.type === "external-job") {
-        const unsupported = [
-            target.tools?.length || target.mcpDirectTools?.length ? "tools" : undefined,
-            target.excludeTools?.length ? "excludeTools" : undefined,
-            target.model ? "model" : undefined,
-            target.thinking ? "thinking" : undefined,
-            target.extensions?.length ? "extensions" : undefined,
-            target.subagentOnlyExtensions?.length ? "subagentOnlyExtensions" : undefined,
-            target.mutationTools?.length ? "mutationTools" : undefined,
-            target.skills?.length || target.skillPath?.length ? "skills" : undefined,
-            target.maxSubagentDepth !== undefined ? "maxSubagentDepth" : undefined,
-            target.toolBudget ? "toolBudget" : undefined,
-        ].filter((field) => Boolean(field));
-        if (unsupported.length > 0)
-            return `config.runner type '${target.runner.type}' does not support Pi-only fields: ${unsupported.join(", ")}.`;
-    }
-    return undefined;
-}
-function resolveTarget(name, matches, cwd, ctx, scopeHint) {
-    const distinctNames = [...new Set(matches.map((match) => match.name))];
-    if (distinctNames.length > 1)
-        return result(`Ambiguous agent alias or name '${name}': ${distinctNames.sort((a, b) => a.localeCompare(b)).join(", ")}`, true);
-    const mutable = matches.filter((match) => isMutableSource(match.source));
-    if (mutable.length === 0) {
-        if (matches.length > 0)
-            return result(`Agent '${name}' is read-only and cannot be modified. Create a same-named agent in user or project scope to override it.`, true);
-        return result(`Agent '${name}' not found. Available: ${availableAgentNames(ctx, cwd).join(", ") || "none"}.`, true);
-    }
-    if (mutable.length === 1)
-        return mutable[0];
-    const scope = asDisambiguationScope(scopeHint);
-    if (!scope) {
-        const paths = mutable.map((match) => `${match.source}: ${match.filePath}`).join("\n");
-        return result(`Agent '${name}' exists in both scopes. Specify agentScope: 'user' or 'project'.\n${paths}`, true);
-    }
-    const scoped = mutable.filter((match) => match.source === scope);
-    if (scoped.length === 0)
-        return result(`Agent '${name}' not found in scope '${scope}'.`, true);
-    if (scoped.length > 1)
-        return result(`Multiple agents named '${name}' found in scope '${scope}': ${scoped.map((match) => match.filePath).join(", ")}`, true);
-    return scoped[0];
-}
-function renamePath(currentPath, newName, scope, ctx) {
-    if (nameExistsInScope(ctx, scope, newName, currentPath))
-        return { error: `Name '${newName}' already exists in ${scope} scope.` };
-    const filePath = path.join(path.dirname(currentPath), `${newName}.md`);
-    if (fs.existsSync(filePath) && filePath !== currentPath)
-        return { error: `File already exists at ${filePath} but is not a valid agent definition. Remove or rename it first.` };
-    fs.renameSync(currentPath, filePath);
-    return { filePath };
-}
-function packageSourceLabel(agent) {
-    if (!agent.packageSourceName)
-        return agent.source;
-    return agent.packageSourceVersion ? `${agent.packageSourceName}@${agent.packageSourceVersion}` : agent.packageSourceName;
-}
-function errorMessage(error) {
-    return error instanceof Error ? error.message : String(error);
-}
-function registeredExternalJobProviderStatus() {
-    try {
-        return { ok: true, names: new Set(listExternalJobProviders().map((provider) => provider.name)) };
-    }
-    catch (error) {
-        return { ok: false, error: errorMessage(error) };
-    }
-}
-function externalJobProviderSuffix(provider, names) {
-    if (!names)
-        return "?";
-    return names.has(provider) ? "✓" : "missing";
-}
-/** A placed agent checks only local ssh; machine catalog and remote CLI validation happen at launch. */
-function externalCliAvailabilityKey(command, machine) {
-    return machine ? `ssh@${machine}` : command;
-}
-function externalCliAvailabilityForAgents(agents) {
-    const availability = new Map();
-    for (const agent of agents) {
-        const runner = agent.runner;
-        if (runner?.type !== "external-cli")
-            continue;
-        const key = externalCliAvailabilityKey(runner.command, agent.machine);
-        if (availability.has(key))
-            continue;
-        availability.set(key, resolveExternalCliBinaryAvailability(agent.machine ? "ssh" : runner.command, process.env));
-    }
-    return availability;
-}
-function runnerListBadge(agent, providerNames, externalCliAvailability) {
-    if (agent.runner?.type === "external-job")
-        return `external-job:${agent.runner.provider} ${externalJobProviderSuffix(agent.runner.provider, providerNames)}`;
-    if (agent.runner?.type === "external-cli") {
-        const placed = agent.machine ? `${agent.runner.command} @ ${agent.machine}` : agent.runner.command;
-        const availability = externalCliAvailability?.get(externalCliAvailabilityKey(agent.runner.command, agent.machine));
-        if (!availability)
-            return `external-cli:${placed}`;
-        if (agent.machine)
-            return `external-cli:${placed} saved Herdr placement; transport ${availability.available ? "✓" : "missing"}; machine not preflighted`;
-        return `external-cli:${placed} ${availability.available ? "✓" : "missing"}`;
-    }
-    if (agent.machine)
-        return `machine: ${agent.machine} (saved Herdr placement)`;
-    return undefined;
-}
-function agentListMetadata(agent, providerNames, externalCliAvailability) {
-    const source = agent.source === "package" ? packageSourceLabel(agent) : agent.source;
-    return [
-        source,
-        runnerListBadge(agent, providerNames, externalCliAvailability),
-        agent.defaultContext ? `context: ${agent.defaultContext}` : undefined,
-        agent.aliases?.length ? `aliases: ${agent.aliases.join(", ")}` : undefined,
-    ].filter((part) => Boolean(part)).join(", ");
-}
-function formatAgentListLine(agent, providerNames) {
-    return `- ${agent.name} (${agentListMetadata(agent, providerNames)}): ${agent.description}`;
-}
-function formatAgentCapabilitiesLine(agent, providerNames, externalCliAvailability) {
-    const declaredTools = [
-        ...(agent.tools ?? []),
-        ...(agent.mcpDirectTools ?? []).map((tool) => `mcp:${tool}`),
-    ];
-    let tools = "none";
-    if (agent.tools === undefined && agent.mcpDirectTools === undefined) {
-        tools = "default/ambient";
-    }
-    else if (declaredTools.length > 0) {
-        tools = declaredTools.join(", ");
-    }
-    if (agent.excludeTools?.length)
-        tools = `${tools}; excludes: ${agent.excludeTools.join(", ")}`;
-    let model = "inherits current session";
-    if (agent.model !== undefined) {
-        model = agent.model;
-        if (agent.modelProvider && !agent.model.includes("/"))
-            model = `${agent.modelProvider}/${agent.model}`;
-    }
-    const thinking = agent.thinking === false ? "off" : agent.thinking ?? "default";
-    const machine = agent.machine ? `; Machine: ${agent.machine} (saved Herdr placement)` : "";
-    const acceptance = formatAcceptanceSummary(agent);
-    return `- ${agent.name} (${agentListMetadata(agent, providerNames, externalCliAvailability)}): Description: ${previewDisplayText(agent.description, 240)}; Tools: ${tools}; Model: ${model}; Thinking: ${thinking}${machine}${acceptance ? `; ${acceptance}` : ""}`;
-}
-function formatAcceptanceSummary(agent) {
-    const policy = agent.defaultAcceptance;
-    const summary = [];
-    if (policy === false)
-        summary.push("Acceptance: disabled");
-    else if (typeof policy === "string")
-        summary.push(`Acceptance: ${policy}`);
-    else if (policy) {
-        const modifiers = [
-            ...(policy.evidence ?? []),
-            ...(policy.verify ?? []).map((command) => `verify: ${formatAcceptanceDisplayLabel(command.id)}`),
-            ...(policy.criteria?.length ? [`criteria: ${policy.criteria.length}`] : []),
-            ...(policy.stopRules?.length ? [`stopRules: ${policy.stopRules.length}`] : []),
-        ];
-        if (policy.review === false)
-            modifiers.push("review: off");
-        else if (policy.review) {
-            const displayReview = policy.review.agent
-                ? { ...policy.review, agent: formatAcceptanceDisplayLabel(policy.review.agent) }
-                : policy.review;
-            modifiers.push(`review: ${formatReviewGateLabel(displayReview)}`);
-        }
-        if (policy.report)
-            modifiers.push(`report: ${policy.report}`);
-        summary.push(`Acceptance: ${policy.level ?? "auto"}${modifiers.length > 0 ? ` (${modifiers.join(", ")})` : ""}`);
-    }
-    if (agent.acceptanceRole)
-        summary.push(`Acceptance role: ${agent.acceptanceRole}`);
-    return summary.length > 0 ? summary.join("; ") : undefined;
-}
-function formatAcceptanceDisplayLabel(value) {
-    return JSON.stringify(previewDisplayText(value, 80));
-}
-const EXTERNAL_JOB_CAPABILITIES = { stop: false, steer: false, resume: false, structuredOutput: false, toolEvents: false };
-const PI_AGENT_RUNNER = { type: "pi" };
-function listOrEmpty(values) {
-    return values ?? [];
-}
-function agentCapabilityRunner(agent, providerNames, externalCliAvailability) {
-    const runner = agent.runner;
-    if (!runner || runner.type === "pi")
-        return PI_AGENT_RUNNER;
-    if (runner.type === "external-cli") {
-        const availability = externalCliAvailability.get(externalCliAvailabilityKey(runner.command, agent.machine));
-        return {
-            type: "external-cli",
-            adapter: runner.adapter,
-            command: runner.command,
-            ...(agent.machine ? { machine: agent.machine } : {}),
-            ...availability,
-            capabilities: resolveExternalCliRunnerStatus(runner).capabilities,
-        };
-    }
-    return { type: "external-job", provider: runner.provider, available: providerNames?.has(runner.provider), capabilities: EXTERNAL_JOB_CAPABILITIES };
-}
-function agentCapabilityTools(agent) {
-    return {
-        ambient: agent.tools === undefined && agent.mcpDirectTools === undefined,
-        names: listOrEmpty(agent.tools),
-        ...(agent.excludeTools !== undefined ? { excludeTools: [...agent.excludeTools] } : {}),
-        mcpDirectTools: listOrEmpty(agent.mcpDirectTools),
-        mutationTools: agent.mutationTools,
-    };
-}
-function agentCapabilityRow(agent, options) {
-    return {
-        name: agent.name,
-        description: previewDisplayText(agent.description, 1000),
-        source: agent.source,
-        executable: options.executable,
-        restrictionSources: options.executable ? undefined : options.restrictionSources ?? [],
-        aliases: agent.aliases ? [...agent.aliases] : undefined,
-        runner: agentCapabilityRunner(agent, options.providerNames, options.externalCliAvailability),
-        tools: agentCapabilityTools(agent),
-        model: presentDetails({ value: agent.model, thinking: agent.thinking }),
-        execution: presentDetails({ defaultAsync: agent.defaultAsync, timeoutMs: agent.defaultTimeoutMs }),
-        acceptance: presentDetails({ policy: agent.defaultAcceptance, role: agent.acceptanceRole }),
-        output: presentDetails({ path: agent.output, mode: agent.outputMode }),
-        extensions: presentDetails({ names: agent.extensions, subagentOnly: agent.subagentOnlyExtensions, skills: agent.skills }),
-    };
-}
-function agentCapabilitiesSnapshot(input) {
-    return {
-        agents: [
-            ...input.agents.map((agent) => agentCapabilityRow(agent, { executable: true, providerNames: input.providerNames, externalCliAvailability: input.externalCliAvailability })),
-            ...input.restrictedAgents.map((agent) => agentCapabilityRow(agent, { executable: false, providerNames: input.providerNames, externalCliAvailability: input.externalCliAvailability, restrictionSources: input.restrictedSources })),
-        ],
-        restrictedCount: input.restrictedAgents.length,
-        ...(input.restrictedSources?.length ? { capabilityCeilingSources: [...input.restrictedSources] } : {}),
-    };
-}
-function providerNames(status) {
-    return status.ok ? status.names : undefined;
-}
-function appendRestrictedAgentLines(input) {
-    if (input.agents.length === 0)
-        return;
-    input.lines.push("", `Restricted agents (not executable in this session${input.sources?.length ? `; capability ceiling: ${input.sources.join(", ")}` : ""}):`, ...input.agents.map((agent) => input.formatLine(agent, input.providerNames)));
-}
-function appendExternalJobRegistryLine(lines, agents, status) {
-    if (status.ok || !agents.some((agent) => agent.runner?.type === "external-job"))
-        return;
-    lines.push("", `External-job provider registry unavailable: ${status.error}`);
-}
-function appendAgentDiagnosticLines(lines, diagnostics) {
-    if (!diagnostics?.length)
-        return;
-    lines.push("", "Invalid agent definitions:", ...diagnostics.map((diagnostic) => `- ${diagnostic.name ?? diagnostic.filePath} (${diagnostic.source}): ${diagnostic.error}`));
-}
-function agentCapabilityDetails(input) {
-    if (!input.capabilityMode)
-        return undefined;
-    return {
-        agentCapabilities: jsonDetails(agentCapabilitiesSnapshot({
-            agents: input.agents,
-            restrictedAgents: input.restrictedAgents,
-            providerNames: input.providerNames,
-            externalCliAvailability: input.externalCliAvailability,
-            restrictedSources: input.restrictedSources,
-        })),
-    };
-}
-function formatAgentListSections(agents, providerNames, formatLine = formatAgentListLine) {
-    if (agents.length === 0)
-        return ["- (none)"];
-    const sections = [
-        ["package", "Package agents"],
-        ["user", "User agents"],
-        ["project", "Project agents"],
-        ["runtime", "Runtime agents"],
-        ["builtin", "Builtin agents"],
-    ];
-    const lines = [];
-    for (const [source, label] of sections) {
-        const matches = agents.filter((agent) => agent.source === source);
-        if (matches.length === 0)
-            continue;
-        if (lines.length > 0)
-            lines.push("");
-        lines.push(label, ...matches.map((agent) => formatLine(agent, providerNames)));
-    }
-    return lines;
-}
-function formatRunnerDetail(agent, providerNames) {
-    if (!agent.runner)
-        return undefined;
-    if (agent.runner.type === "external-job")
-        return `Runner: external-job via ${agent.runner.provider} ${externalJobProviderSuffix(agent.runner.provider, providerNames)}`;
-    if (agent.runner.type === "external-cli")
-        return `Runner: external-cli ${agent.runner.command}`;
-    return `Runner: ${JSON.stringify(agent.runner)}`;
-}
-function formatAgentDetail(agent) {
-    const tools = [...(agent.tools ?? []), ...(agent.mcpDirectTools ?? []).map((t) => `mcp:${t}`)];
-    const providerStatus = registeredExternalJobProviderStatus();
-    const lines = [`Agent: ${agent.name} (${agent.source})`, `Path: ${agent.filePath}`, `Description: ${agent.description}`];
-    if (agent.source === "package" && agent.packageSourceName) {
-        lines.push(`Source package: ${packageSourceLabel(agent)}`);
-        if (agent.packageSourceRoot)
-            lines.push(`Package root: ${agent.packageSourceRoot}`);
-    }
-    if (agent.packageName) {
-        lines.push(`Local name: ${frontmatterNameForConfig(agent)}`);
-        lines.push(`Package: ${agent.packageName}`);
-    }
-    if (agent.aliases?.length)
-        lines.push(`Aliases: ${agent.aliases.join(", ")}`);
-    if (agent.model)
-        lines.push(`Model: ${agent.model}`);
-    if (tools.length)
-        lines.push(`Tools: ${tools.join(", ")}`);
-    if (agent.excludeTools?.length)
-        lines.push(`Excluded tools: ${agent.excludeTools.join(", ")}`);
-    if (agent.skills?.length)
-        lines.push(`Skills: ${agent.skills.join(", ")}`);
-    if (agent.skillPath?.length)
-        lines.push(`Skill paths: ${agent.skillPath.join(", ")}`);
-    lines.push(`System prompt mode: ${agent.systemPromptMode}`);
-    const runnerDetail = formatRunnerDetail(agent, providerStatus.ok ? providerStatus.names : undefined);
-    if (runnerDetail) {
-        lines.push(runnerDetail);
-        if (agent.runner?.type === "external-job" && !providerStatus.ok)
-            lines.push(`External-job provider registry unavailable: ${providerStatus.error}`);
-        if (agent.runner?.type === "external-job" && agent.runner.options)
-            lines.push(`Runner options: ${JSON.stringify(agent.runner.options)}`);
-    }
-    lines.push(`Inherit project context: ${agent.inheritProjectContext ? "true" : "false"}`);
-    lines.push(`Inherit global context: ${agent.inheritGlobalContext ? "true" : "false"}`);
-    lines.push(`Inherit skills: ${agent.inheritSkills ? "true" : "false"}`);
-    if (agent.defaultContext)
-        lines.push(`Default context: ${agent.defaultContext}`);
-    if (agent.defaultAsync !== undefined)
-        lines.push(`Async: ${agent.defaultAsync ? "true" : "false"}`);
-    if (agent.defaultTimeoutMs !== undefined)
-        lines.push(`Timeout: ${agent.defaultTimeoutMs}ms`);
-    if (agent.defaultAcceptance !== undefined)
-        lines.push(`Acceptance: ${typeof agent.defaultAcceptance === "object" ? JSON.stringify(agent.defaultAcceptance) : String(agent.defaultAcceptance)}`);
-    if (agent.acceptanceRole)
-        lines.push(`Acceptance role: ${agent.acceptanceRole}`);
-    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.subagentOnlyExtensions !== undefined)
-        lines.push(`Subagent-only extensions: ${agent.subagentOnlyExtensions.length ? agent.subagentOnlyExtensions.join(", ") : "(none)"}`);
-    if (agent.mutationTools !== undefined)
-        lines.push(`Mutation tools: ${agent.mutationTools.length ? agent.mutationTools.join(", ") : "(none)"}`);
-    if (agent.thinking)
-        lines.push(`Thinking: ${agent.thinking}`);
-    if (agent.output)
-        lines.push(`Output: ${agent.output}`);
-    if (agent.outputMode)
-        lines.push(`Output mode: ${agent.outputMode}`);
-    if (agent.defaultReads?.length)
-        lines.push(`Reads: ${agent.defaultReads.join(", ")}`);
-    if (agent.defaultProgress)
-        lines.push("Progress: true");
-    if (agent.maxSubagentDepth !== undefined)
-        lines.push(`Max subagent depth: ${agent.maxSubagentDepth}`);
-    if (agent.toolBudget)
-        lines.push(`Tool budget: ${JSON.stringify(agent.toolBudget)}`);
-    if (agent.memory)
-        lines.push(`Memory: ${agent.memory.scope} scope, path: ${agent.memory.path}`);
-    if (agent.systemPrompt.trim())
-        lines.push("", "System Prompt:", agent.systemPrompt);
-    return lines.join("\n");
-}
-export function handleList(params, ctx) {
-    const scope = normalizeListScope(params.agentScope) ?? "both";
-    const d = discoverCatalog(ctx, ctx.cwd, ctx.model?.provider);
-    let scopedAgents = effectiveAgentsForScope(scope, d, ctx.runtimeAgentOwner, ctx.model?.provider);
-    scopedAgents = scopedAgents
-        .sort((a, b) => a.name.localeCompare(b.name));
-    const capabilityCeiling = resolveCurrentSubagentCapabilityCeiling(ctx.currentSessionId);
-    const visibleAgents = scopedAgents.filter((a) => !a.disabled);
-    const agents = visibleAgents.filter((a) => isAgentAllowedByCapabilityCeiling(a.name, capabilityCeiling));
-    const restrictedAgents = visibleAgents.filter((a) => !isAgentAllowedByCapabilityCeiling(a.name, capabilityCeiling));
-    const restrictedSources = capabilityCeilingAgentRestrictionSources(capabilityCeiling);
-    const proactiveSuggestions = buildProactiveSkillSubagentRecommendationLines({
-        agents,
-        ...(ctx.config?.proactiveSkillSubagents !== undefined ? { config: ctx.config.proactiveSkillSubagents } : {}),
-        discoverAvailableSkills: () => discoverAvailableSkills(ctx.cwd),
-    });
-    const providerStatus = registeredExternalJobProviderStatus();
-    const providerNameSet = providerNames(providerStatus);
-    const capabilityMode = params.capabilities === true;
-    const externalCliAvailability = capabilityMode ? externalCliAvailabilityForAgents([...agents, ...restrictedAgents]) : undefined;
-    const formatLine = capabilityMode
-        ? (agent, names) => formatAgentCapabilitiesLine(agent, names, externalCliAvailability)
-        : formatAgentListLine;
-    const lines = [
-        capabilityMode ? "Executable agents (capabilities):" : "Executable agents:",
-        ...formatAgentListSections(agents, providerNameSet, formatLine),
-    ];
-    appendRestrictedAgentLines({ lines, agents: restrictedAgents, sources: restrictedSources, providerNames: providerNameSet, formatLine });
-    appendExternalJobRegistryLine(lines, [...agents, ...restrictedAgents], providerStatus);
-    appendAgentDiagnosticLines(lines, d.agentDiagnostics);
-    if (proactiveSuggestions.length)
-        lines.push("", ...proactiveSuggestions);
-    return result(lines.join("\n"), false, agentCapabilityDetails({
-        capabilityMode,
-        agents,
-        restrictedAgents,
-        providerNames: providerNameSet,
-        externalCliAvailability: externalCliAvailability ?? new Map(),
-        restrictedSources,
-    }));
-}
-function formatModelSource(agent, currentModel) {
-    if (agent.override && agent.model !== agent.override.base.model) {
-        return `${agent.override.scope} override`;
-    }
-    if (agent.modelSource?.type === "subagents.defaultModel" && agent.model === agent.modelSource.model) {
-        return `${agent.modelSource.scope} defaultModel`;
-    }
-    if (agent.model)
-        return `${agent.source} agent config`;
-    if (currentModel)
-        return "inherits current session model";
-    return "inherit requested, but no current session model is available";
-}
-function handleModels(params, ctx) {
-    const requestedAgent = params.agent?.trim();
-    const scope = normalizeListScope(params.agentScope);
-    if (!scope)
-        return result("agentScope must be 'user', 'project', or 'both' for models.", true);
-    const discovered = discoverCatalog(ctx, ctx.cwd, ctx.model?.provider);
-    const effectiveAgents = effectiveAgentsForScope(scope, discovered, ctx.runtimeAgentOwner, ctx.model?.provider)
-        .sort((a, b) => a.name.localeCompare(b.name));
-    const availableModels = ctx.modelRegistry.getAvailable().map(toModelInfo);
-    const currentModel = ctx.model ? { provider: ctx.model.provider, id: ctx.model.id } : undefined;
-    const preferredProvider = ctx.model?.provider;
-    const capabilityCeiling = resolveCurrentSubagentCapabilityCeiling(ctx.currentSessionId);
-    let selectedAgents = effectiveAgents;
-    if (requestedAgent) {
-        const matches = findAgentsInDiscovery(requestedAgent, discovered, scope, ctx.runtimeAgentOwner, ctx.model?.provider);
-        const diagnostics = diagnosticsForScope(discovered.agentDiagnostics, scope);
-        const normalizedName = sanitizeName(requestedAgent);
-        const diagnostic = findBlockingAgentDiagnostic(requestedAgent, matches, diagnostics)
-            ?? (normalizedName !== requestedAgent ? findBlockingAgentDiagnostic(normalizedName, matches, diagnostics) : undefined);
-        if (diagnostic)
-            return result(`Agent '${params.agent}' has invalid configuration: ${diagnostic.error}`, true);
-        const distinctNames = [...new Set(matches.map((agent) => agent.name))];
-        if (distinctNames.length > 1)
-            return result(`Ambiguous agent alias or name '${params.agent}': ${distinctNames.sort((a, b) => a.localeCompare(b)).join(", ")}`, true);
-        if (!matches.length) {
-            return result(`Agent '${params.agent}' not found. Available: ${availableAgentNamesFromDiscovery(discovered, ctx.runtimeAgentOwner, ctx.model?.provider).join(", ") || "none"}.`, true);
-        }
-        selectedAgents = [matches[0]];
-    }
-    const lines = [
-        requestedAgent ? "Subagent model" : "Subagent models",
-        "",
-        ...(requestedAgent ? [] : [
-            "Current session model:",
-            `  ${currentModel ? `${currentModel.provider}/${currentModel.id}` : "(unavailable)"}`,
-            "",
-        ]),
-    ];
-    const modelEntries = selectedAgents.flatMap((agent) => requestedAgent
-        ? [{ agent, name: requestedAgent }]
-        : [{ agent, name: agent.name }, ...(agent.aliases ?? []).map((name) => ({ agent, name }))]);
-    for (const { agent, name } of modelEntries) {
-        const resolvedModel = resolveSubagentModelOverride(agent.model, currentModel, availableModels, agent.modelProvider ?? preferredProvider);
-        const effectiveThinking = resolveEffectiveThinking(resolvedModel, agent.thinking)
-            ?? (agent.thinking === false ? "off" : undefined);
-        const source = `${formatModelSource(agent, currentModel)}${agent.disabled ? "; disabled" : ""}${isAgentAllowedByCapabilityCeiling(agent.name, capabilityCeiling) ? "" : "; restricted"}`;
-        if (requestedAgent) {
-            lines.push(`Agent: ${requestedAgent}`);
-            lines.push("Effective model:");
-            lines.push(`  ${resolvedModel ?? "(unresolved)"}`);
-            lines.push(`Source: ${source}`);
-            lines.push(`Thinking: ${effectiveThinking ?? "default"}`);
-            if (agent.override) {
-                lines.push("Override file:");
-                lines.push(`  ${agent.override.path}`);
-            }
-            if (agent.model && resolvedModel && agent.model !== resolvedModel) {
-                lines.push("Requested model setting:");
-                lines.push(`  ${agent.model}`);
-            }
-            if (agent.disabled)
-                lines.push("Disabled: true");
-            if (!isAgentAllowedByCapabilityCeiling(agent.name, capabilityCeiling))
-                lines.push("Restricted: true");
-            lines.push("Current session model:");
-            lines.push(`  ${currentModel ? `${currentModel.provider}/${currentModel.id}` : "(unavailable)"}`);
-            break;
-        }
-        lines.push(name);
-        lines.push("  model:");
-        lines.push(`    ${resolvedModel ?? "(unresolved)"}`);
-        lines.push(`  source: ${source}`);
-        lines.push(`  thinking: ${effectiveThinking ?? "default"}`);
-        if (agent.override) {
-            lines.push("  override file:");
-            lines.push(`    ${agent.override.path}`);
-        }
-        if (agent.model && resolvedModel && agent.model !== resolvedModel) {
-            lines.push("  requested model setting:");
-            lines.push(`    ${agent.model}`);
-        }
-        lines.push("");
-    }
-    const availableFullIds = availableModels.map((m) => m.fullId).sort();
-    if (!requestedAgent && availableFullIds.length > 0) {
-        lines.push("Available models in this session's registry (copy an exact provider/id when passing model):");
-        lines.push("");
-        const shown = availableFullIds.slice(0, 80);
-        for (const fullId of shown)
-            lines.push(`  ${fullId}`);
-        if (availableFullIds.length > shown.length)
-            lines.push(`  ... and ${availableFullIds.length - shown.length} more`);
-        lines.push("");
-        lines.push("Use an exact provider/id from this list when you pass model; bare ids resolve only when unique in the registry.");
-    }
-    if (!requestedAgent)
-        appendAgentDiagnosticLines(lines, diagnosticsForScope(discovered.agentDiagnostics, scope));
-    return result(lines.join("\n"));
-}
-function handleGet(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for get.", true);
-    const scope = normalizeListScope(params.agentScope);
-    if (!scope)
-        return result("agentScope must be 'user', 'project', or 'both' for get.", true);
-    const discovered = discoverCatalog(ctx, ctx.cwd, ctx.model?.provider);
-    const matches = findAgentsInDiscovery(params.agent, discovered, scope);
-    const diagnostics = diagnosticsForScope(discovered.agentDiagnostics, scope);
-    const rawName = params.agent.trim();
-    const normalizedName = sanitizeName(rawName);
-    const diagnostic = findBlockingAgentDiagnostic(rawName, matches, diagnostics)
-        ?? (normalizedName !== rawName ? findBlockingAgentDiagnostic(normalizedName, matches, diagnostics) : undefined);
-    if (diagnostic)
-        return result(`Agent '${params.agent}' has invalid configuration: ${diagnostic.error}`, true);
-    const distinctNames = [...new Set(matches.map((agent) => agent.name))];
-    if (distinctNames.length > 1)
-        return result(`Ambiguous agent alias or name '${params.agent}': ${distinctNames.sort((a, b) => a.localeCompare(b)).join(", ")}`, true);
-    if (!matches.length) {
-        return result(`Agent '${params.agent}' not found. Available: ${availableAgentNamesFromDiscovery(discovered).join(", ") || "none"}.`, true);
-    }
-    return result(matches.map(formatAgentDetail).join("\n\n"));
-}
-export function handleCreate(params, ctx) {
-    const parsedConfig = configObject(params.config);
-    if (parsedConfig.status === "error")
-        return result(parsedConfig.message, true);
-    if (parsedConfig.status === "missing")
-        return result("config required for create.", true);
-    const cfg = parsedConfig.value;
-    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);
-    const parsedPackage = parsePackageConfig(cfg.package);
-    if (parsedPackage.error)
-        return result(parsedPackage.error, true);
-    const runtimeName = buildRuntimeName(name, parsedPackage.packageName);
-    const scopeRaw = cfg.scope ?? "user";
-    if (scopeRaw !== "user" && scopeRaw !== "project")
-        return result("config.scope must be 'user' or 'project'.", true);
-    const scope = scopeRaw;
-    if (hasKey(cfg, "steps"))
-        return result("Durable chain definitions were removed; use a workflow script or /prompt-workflow for repeatable workflows.", true);
-    const d = discoverCatalog(ctx);
-    const projectConfigDir = getProjectConfigDir(ctx.cwd);
-    const targetDir = scope === "user" ? d.userDir : d.projectDir ?? path.join(projectConfigDir, "agents");
-    fs.mkdirSync(targetDir, { recursive: true });
-    if (nameExistsInScope(ctx, scope, runtimeName))
-        return result(`Name '${runtimeName}' already exists in ${scope} scope. Use update instead.`, true);
-    const targetPath = path.join(targetDir, `${runtimeName}.md`);
-    if (fs.existsSync(targetPath))
-        return result(`File already exists at ${targetPath} but is not a valid agent definition. Remove or rename it first.`, true);
-    const warnings = [];
-    if (d.builtin.some((a) => a.name === runtimeName))
-        warnings.push(`Note: this shadows the builtin agent '${runtimeName}'.`);
-    const agent = {
-        name: runtimeName,
-        localName: name,
-        ...(parsedPackage.packageName !== undefined ? { packageName: parsedPackage.packageName } : {}),
-        description: cfg.description.trim(),
-        source: scope,
-        filePath: targetPath,
-        systemPrompt: "",
-        systemPromptMode: defaultSystemPromptMode(name),
-        inheritProjectContext: defaultInheritProjectContext(name),
-        inheritGlobalContext: false,
-        inheritSkills: defaultInheritSkills(),
-    };
-    const applyError = applyAgentConfig(agent, cfg);
-    if (applyError)
-        return result(applyError, true);
-    const profileError = validateCodeOwnedProfileRunner(agent);
-    if (profileError)
-        return result(profileError, true);
-    const mw = modelWarning(ctx, agent.model);
-    if (mw)
-        warnings.push(mw);
-    const sw = skillsWarning(ctx.cwd, agent);
-    if (sw)
-        warnings.push(sw);
-    fs.writeFileSync(targetPath, serializeAgent(agent), "utf-8");
-    ctx.onAgentsChanged?.();
-    return result([`Created agent '${runtimeName}' at ${targetPath}.`, ...warnings].join("\n"));
-}
-export function handleUpdate(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for update.", true);
-    const parsedConfig = configObject(params.config);
-    if (parsedConfig.status === "error")
-        return result(parsedConfig.message, true);
-    if (parsedConfig.status === "missing")
-        return result("config required for update.", true);
-    const cfg = parsedConfig.value;
-    if (hasKey(cfg, "steps"))
-        return result("Durable chain definitions were removed; use a workflow script or /prompt-workflow for repeatable workflows.", true);
-    const warnings = [];
-    const scopeHint = asDisambiguationScope(params.agentScope);
-    const targetOrError = resolveTarget(params.agent, findAgents(params.agent, ctx, scopeHint ?? "both"), ctx.cwd, ctx, scopeHint);
-    if ("content" in targetOrError)
-        return targetOrError;
-    const target = targetOrError;
-    if (target.source !== "user" && target.source !== "project")
-        return result(`Cannot update ${target.source} agent '${target.name}'. Eject it to user or project scope first.`, true);
-    let updated;
-    try {
-        updated = editableAgentConfig(target);
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        return result(`Could not reread agent definition ${target.filePath} before updating '${target.name}': ${message}`, true);
-    }
-    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);
-    let newLocalName = target.localName ?? frontmatterNameForConfig(target);
-    if (hasKey(cfg, "name")) {
-        newLocalName = sanitizeName(cfg.name);
-        if (!newLocalName)
-            return result("config.name is invalid after sanitization.", true);
-    }
-    let newPackageName = target.packageName;
-    if (hasKey(cfg, "package")) {
-        const parsedPackage = parsePackageConfig(cfg.package);
-        if (parsedPackage.error)
-            return result(parsedPackage.error, true);
-        newPackageName = parsedPackage.packageName;
-    }
-    const applyError = applyAgentConfig(updated, cfg);
-    if (applyError)
-        return result(applyError, true);
-    const preserveFrontmatterFields = preservedAgentFrontmatterFields(target, cfg);
-    updated.localName = newLocalName;
-    if (newPackageName !== undefined)
-        updated.packageName = newPackageName;
-    else
-        delete updated.packageName;
-    updated.name = buildRuntimeName(newLocalName, newPackageName);
-    const profileError = validateCodeOwnedProfileRunner(updated);
-    if (profileError)
-        return result(profileError, true);
-    if (hasKey(cfg, "description"))
-        updated.description = cfg.description.trim();
-    if (hasKey(cfg, "model")) {
-        const mw = modelWarning(ctx, updated.model);
-        if (mw)
-            warnings.push(mw);
-    }
-    if (hasKey(cfg, "skills") || hasKey(cfg, "skillPath")) {
-        const sw = skillsWarning(ctx.cwd, updated);
-        if (sw)
-            warnings.push(sw);
-    }
-    if (updated.name !== oldName) {
-        const renamed = renamePath(target.filePath, updated.name, target.source, ctx);
-        if (renamed.error)
-            return result(renamed.error, true);
-        updated.filePath = renamed.filePath;
-    }
-    fs.writeFileSync(updated.filePath, serializeAgent(updated, { preserveFrontmatterFields }), "utf-8");
-    ctx.onAgentsChanged?.();
-    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"));
-}
-function handleDelete(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for delete.", true);
-    const scopeHint = asDisambiguationScope(params.agentScope);
-    const targetOrError = resolveTarget(params.agent, findAgents(params.agent, ctx, scopeHint ?? "both"), ctx.cwd, ctx, scopeHint);
-    if ("content" in targetOrError)
-        return targetOrError;
-    const target = targetOrError;
-    fs.unlinkSync(target.filePath);
-    ctx.onAgentsChanged?.();
-    return result(`Deleted agent '${target.name}' at ${target.filePath}.`);
-}
-function handleEject(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for eject.", true);
-    const raw = params.agent.trim();
-    const sanitized = sanitizeName(raw);
-    const parsedScope = actionScope(params.agentScope, "eject");
-    if (parsedScope.error)
-        return parsedScope.error;
-    const scope = parsedScope.scope;
-    const d = discoverCatalog(ctx);
-    const source = [...d.package, ...d.builtin].find((a) => a.name === raw || a.name === sanitized);
-    if (!source) {
-        return result(`Agent '${raw}' not found or is not a bundled/package agent. eject copies a builtin or package agent to ${scope} scope so it can be customized. Available: ${availableAgentNames(ctx).join(", ") || "none"}.`, true);
-    }
-    const runtimeName = source.name;
-    const existingCustom = (scope === "user" ? d.user : d.project).find((a) => a.name === runtimeName);
-    if (existingCustom) {
-        return result(`Agent '${runtimeName}' is already a custom ${scope} agent at ${existingCustom.filePath}. Edit it with { action: "update", agent: "${runtimeName}" } or delete it first.`, true);
-    }
-    if (nameExistsInScope(ctx, scope, runtimeName)) {
-        return result(`An agent named '${runtimeName}' already exists in ${scope} scope. Remove or rename it first.`, true);
-    }
-    const projectConfigDir = getProjectConfigDir(ctx.cwd);
-    const targetDir = scope === "user" ? d.userDir : d.projectDir ?? path.join(projectConfigDir, "agents");
-    fs.mkdirSync(targetDir, { recursive: true });
-    const targetPath = path.join(targetDir, `${runtimeName}.md`);
-    if (fs.existsSync(targetPath)) {
-        return result(`File already exists at ${targetPath} but is not a valid agent definition. Remove or rename it first.`, true);
-    }
-    let content;
-    try {
-        content = fs.readFileSync(source.filePath, "utf-8");
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        return result(`Failed to read source agent at ${source.filePath}: ${message}`, true);
-    }
-    fs.writeFileSync(targetPath, content, "utf-8");
-    ctx.onAgentsChanged?.();
-    return result(`Ejected agent '${runtimeName}' from ${source.source} to ${scope} scope at ${targetPath}. Edit it there to customize; it shadows the bundled ${source.source} agent of the same name.`);
-}
-function handleDisable(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for disable.", true);
-    const raw = params.agent.trim();
-    const parsedScope = actionScope(params.agentScope, "disable");
-    if (parsedScope.error)
-        return parsedScope.error;
-    const scope = parsedScope.scope;
-    const d = discoverCatalog(ctx);
-    if (scope === "project" && d.projectSettingsPath === null) {
-        return result("Project override is not available here: no project config root (.pi or .agents) was found above the cwd. Use agentScope: 'user' or run from inside a project.", true);
-    }
-    const effective = resolveEffectiveAgent(d, raw);
-    if (effective.error)
-        return result(effective.error, true);
-    if (!effective.agent) {
-        return result(`Agent '${raw}' not found. Available: ${availableAgentNames(ctx).join(", ") || "none"}.`, true);
-    }
-    const runtimeName = effective.agent.name;
-    const settingsPath = mergeBuiltinAgentOverride(ctx.cwd, runtimeName, scope, { disabled: true });
-    const after = resolveEffectiveAgent(discoverCatalog(ctx), raw).agent;
-    if (after?.disabled === true) {
-        ctx.onAgentsChanged?.();
-        return result(`Disabled agent '${runtimeName}' via ${scope} settings override at ${settingsPath}. It is now hidden from runtime discovery and { action: "list" }.`);
-    }
-    return result(`Wrote a disabled override for '${runtimeName}' at ${settingsPath}, but the agent is still enabled. A higher-precedence ${after?.override?.scope ?? "project"} override is likely winning. Try agentScope: '${after?.override?.scope ?? "project"}'.`, true);
-}
-function handleEnable(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for enable.", true);
-    const raw = params.agent.trim();
-    const parsedScope = actionScope(params.agentScope, "enable");
-    if (parsedScope.error)
-        return parsedScope.error;
-    const scope = parsedScope.scope;
-    const d = discoverCatalog(ctx);
-    if (scope === "project" && d.projectSettingsPath === null) {
-        return result("Project override is not available here: no project config root (.pi or .agents) was found above the cwd. Use agentScope: 'user' or run from inside a project.", true);
-    }
-    const effective = resolveEffectiveAgent(d, raw);
-    if (effective.error)
-        return result(effective.error, true);
-    if (!effective.agent) {
-        return result(`Agent '${raw}' not found. Available: ${availableAgentNames(ctx).join(", ") || "none"}.`, true);
-    }
-    const runtimeName = effective.agent.name;
-    const { path: settingsPath, removed } = removeBuiltinAgentOverrideFields(ctx.cwd, runtimeName, scope, ["disabled"]);
-    const after = resolveEffectiveAgent(discoverCatalog(ctx), raw).agent;
-    if (after && after.disabled !== true) {
-        if (removed)
-            ctx.onAgentsChanged?.();
-        if (removed)
-            return result(`Enabled agent '${runtimeName}' (removed disabled override at ${settingsPath}).`);
-        return result(`Agent '${runtimeName}' is already enabled.`);
-    }
-    if (after?.override?.scope && after.override.scope !== scope) {
-        return result(`Agent '${runtimeName}' is still disabled via a ${after.override.scope} scope override at ${after.override.path}. Specify agentScope: '${after.override.scope}' to enable it.`, true);
-    }
-    return result(`Agent '${runtimeName}' is still disabled after removing the ${scope} disabled override. It may be hidden via subagents.disableBuiltins in ${after?.override?.scope ?? scope} settings at ${after?.override?.path ?? settingsPath}.`, true);
-}
-function handleReset(params, ctx) {
-    if (!params.agent)
-        return result("Specify 'agent' for reset.", true);
-    const raw = params.agent.trim();
-    const sanitized = sanitizeName(raw);
-    const parsedScope = actionScope(params.agentScope, "reset");
-    if (parsedScope.error)
-        return parsedScope.error;
-    const scope = parsedScope.scope;
-    const d = discoverCatalog(ctx);
-    if (scope === "project" && d.projectSettingsPath === null) {
-        return result("Project override is not available here: no project config root (.pi or .agents) was found above the cwd. Use agentScope: 'user' or run from inside a project.", true);
-    }
-    const bundled = [...d.package, ...d.builtin].find((a) => a.name === raw || a.name === sanitized);
-    if (!bundled) {
-        const custom = [...d.user, ...d.project].find((a) => a.name === raw || a.name === sanitized);
-        if (custom) {
-            return result(`Agent '${raw}' has no bundled default to reset to. Use { action: "delete", agent: "${custom.name}" } to remove the custom ${custom.source} agent.`, true);
-        }
-        return result(`Agent '${raw}' not found. Available: ${availableAgentNames(ctx).join(", ") || "none"}.`, true);
-    }
-    const runtimeName = bundled.name;
-    const custom = (scope === "user" ? d.user : d.project).find((a) => a.name === raw || a.name === sanitized);
-    const lines = [];
-    if (custom) {
-        fs.unlinkSync(custom.filePath);
-        lines.push(`Deleted custom ${scope} agent file at ${custom.filePath}.`);
-    }
-    const overrideRemoval = removeBuiltinAgentOverride(ctx.cwd, runtimeName, scope, { preserveMachine: true });
-    if (overrideRemoval.removed)
-        lines.push(`${overrideRemoval.machinePreserved ? "Cleared customization in" : "Removed"} ${scope} settings override at ${overrideRemoval.path}.${overrideRemoval.machinePreserved ? " Retained machine placement." : ""}`);
-    if (lines.length === 0) {
-        const otherScope = scope === "user" ? "project" : "user";
-        const otherCustom = (otherScope === "user" ? d.user : d.project).find((a) => a.name === raw || a.name === sanitized);
-        const hasOtherOverride = bundled.override?.scope === otherScope;
-        const note = (otherCustom || hasOtherOverride)
-            ? ` Customization exists in ${otherScope} scope; specify agentScope: '${otherScope}' to reset it.`
-            : "";
-        return result(`Agent '${runtimeName}' has no ${scope} customization to reset.${note} It is at its bundled ${bundled.source} default.`);
-    }
-    lines.push(`Reset agent '${runtimeName}' to its bundled ${bundled.source} default.`);
-    ctx.onAgentsChanged?.();
-    return result(lines.join("\n"));
-}
-export function handleManagementAction(action, params, ctx) {
-    switch (action) {
-        case "list": return handleList(params, ctx);
-        case "get": return handleGet(params, ctx);
-        case "models": return handleModels(params, ctx);
-        case "create": return handleCreate(params, ctx);
-        case "update": return handleUpdate(params, ctx);
-        case "delete": return handleDelete(params, ctx);
-        case "eject": return handleEject(params, ctx);
-        case "disable": return handleDisable(params, ctx);
-        case "enable": return handleEnable(params, ctx);
-        case "reset": return handleReset(params, ctx);
-        default: return result(`Unknown action: ${action}`, true);
-    }
-}
-//# sourceMappingURL=agent-management.js.map