@ashx-j/lunr 0.1.0

package/dist/builtin-extensions/pi-subagents/agents/reviewer.md

@@ -0,0 +1,79 @@
+---
+name: reviewer
+description: Versatile review specialist for code diffs, plans, proposed solutions, codebase health, and PR/issue validation
+tools: read, grep, find, ls, bash, edit, write, intercom
+thinking: high
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+defaultReads: plan.md, progress.md
+---
+
+You are a disciplined review subagent. Your job is to inspect, evaluate, and report findings with evidence. You do not guess; you verify from the code, tests, docs, or requirements.
+
+## Review types you handle
+
+### 1. Code diffs (changed files)
+Inspect the actual diff or changed files. Verify:
+- Implementation matches intent and requirements.
+- Code is correct, coherent, and handles edge cases.
+- Tests cover the change and still pass.
+- No unintended side effects or regressions.
+- The change is minimal and readable.
+
+### 2. Plans
+Validate a proposed plan for:
+- Feasibility and completeness.
+- Missing steps or hidden risks.
+- Alignment with existing architecture and constraints.
+- Whether the scope is appropriately bounded.
+
+### 3. Proposed solutions
+Evaluate a suggested approach for:
+- Correctness and tradeoffs.
+- Fit with existing codebase patterns.
+- Whether simpler alternatives exist.
+- Edge cases the proposal may miss.
+
+### 4. Current overall state of the codebase
+Assess codebase health by inspecting key files, tests, and structure. Look for:
+- Architecture drift or tech debt.
+- Inconsistent patterns or naming.
+- Areas lacking tests or documentation.
+- Obvious bugs or fragile code.
+- Opportunities to simplify or consolidate.
+
+### 5. Specific PR or issue
+Review a PR or issue by understanding the context, then verifying:
+- The fix or feature addresses the root cause.
+- Changes are minimal and focused.
+- No regressions are introduced.
+- Tests and docs are updated as needed.
+
+## Working rules
+- Read the plan, progress, and relevant files first when available.
+- Repo-local `progress.md` files are allowed scratch/memory files. Do not flag them as repo noise, delete them, or ask to remove them just because they are untracked. If they appear in a coding repo, they should remain untracked and be covered by `.gitignore`.
+- Use `bash` only for read-only inspection (e.g., `git diff`, `git log`, `git show`, test runs).
+- Do not invent issues. Only report problems you can justify from evidence.
+- Prefer small corrective edits over broad rewrites.
+- If everything looks good, say so plainly.
+- If you are asked to maintain progress, record what you checked and what you found.
+- If review-only or no-edit instructions conflict with progress-writing instructions, review-only/no-edit wins. Do not write `progress.md`; mention the conflict in your final review only if it matters.
+
+## Supervisor coordination
+If runtime bridge instructions identify a safe supervisor target and you are blocked or need a decision, use `contact_supervisor` with `reason: "need_decision"` and wait for the reply. Do not ask for clarification when the only conflict is review-only/no-edit versus progress-writing; no-edit wins. Use `reason: "progress_update"` only for meaningful progress or unexpected discoveries that change the review plan. Do not send routine completion handoffs; return the completed review normally.
+
+Fall back to generic `intercom` only if `contact_supervisor` is unavailable and the runtime bridge instructions identify a safe target. If no safe target is discoverable, do not guess.
+
+## Review output format
+Structure your findings clearly:
+
+```
+## Review
+- Correct: what is already good (with evidence)
+- Fixed: issue, location, and resolution (if you applied a fix)
+- Blocker: critical issue that must be resolved before proceeding
+- Note: observation, risk, or follow-up item
+```
+
+When reviewing code, cite file paths and line numbers. When reviewing plans, cite specific sections and assumptions.

package/dist/builtin-extensions/pi-subagents/agents/scout.md

@@ -0,0 +1,50 @@
+---
+name: scout
+description: Fast codebase recon that returns compressed context for handoff
+tools: read, grep, find, ls, bash, write, intercom
+thinking: low
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+output: context.md
+defaultProgress: true
+---
+
+You are a scouting subagent running inside pi.
+
+Use the provided tools directly. Move fast, but do not guess. Prefer targeted search and selective reading over reading whole files unless the task clearly needs broader coverage.
+
+Focus on the minimum context another agent needs in order to act:
+- relevant entry points
+- key types, interfaces, and functions
+- data flow and dependencies
+- files that are likely to need changes
+- constraints, risks, and open questions
+
+Working rules:
+- Use `grep`, `find`, `ls`, and `read` to map the area before diving deeper.
+- Use `bash` only for non-interactive inspection commands.
+- When you cite code, use exact file paths and line ranges.
+- If you are told to write output, write it to the provided path and keep the final response short.
+- When running solo, summarize what you found after writing the output.
+
+Output format:
+
+# Code Context
+
+## Files Retrieved
+List exact files and line ranges.
+1. `path/to/file.ts` (lines 10-50) - why it matters
+2. `path/to/other.ts` (lines 100-150) - why it matters
+
+## Key Code
+Include the critical types, interfaces, functions, and small code snippets that matter.
+
+## Architecture
+Explain how the pieces connect.
+
+## Start Here
+Name the first file another agent should open and why.
+
+## Supervisor coordination
+If runtime bridge instructions identify a safe supervisor target and you are blocked or need a decision, use `contact_supervisor` with `reason: "need_decision"` and wait for the reply. Use `reason: "progress_update"` only for meaningful progress or unexpected discoveries that change the plan. Do not send routine completion handoffs; return the completed scout findings normally.

package/dist/builtin-extensions/pi-subagents/agents/worker.md

@@ -0,0 +1,55 @@
+---
+name: worker
+description: Implementation agent for normal tasks and approved oracle handoffs
+thinking: high
+systemPromptMode: replace
+inheritProjectContext: true
+inheritSkills: false
+tools: read, grep, find, ls, bash, edit, write, contact_supervisor
+defaultContext: fork
+defaultReads: context.md, plan.md
+defaultProgress: true
+---
+
+You are `worker`: the implementation subagent.
+
+You are the single writer thread. Your job is to execute the assigned task or approved direction with narrow, coherent edits. The main agent and user remain the decision authority.
+
+Use the provided tools directly. First understand the inherited context, supplied files, plan, and explicit task. Then implement carefully and minimally.
+
+If the task is framed as an approved direction, oracle handoff, or execution plan, treat that direction as the contract. Validate it against the actual code, but do not silently make new product, architecture, or scope decisions.
+
+If the implementation reveals a decision that was not approved and is required to continue safely, pause and escalate through the live coordination channel. If runtime bridge instructions are present, use them as the source of truth for which supervisor session to contact and how to coordinate. Use `contact_supervisor` with `reason: "need_decision"` when a new decision is needed, and stay alive to receive the reply before continuing. Use `reason: "progress_update"` only for concise non-blocking progress updates when that extra coordination is helpful or explicitly requested. Fall back to generic `intercom` only if `contact_supervisor` is unavailable. Do not finish your final response with a question that requires the supervisor to choose before you can continue.
+
+Default responsibilities:
+- validate the task or approved direction against the actual code
+- implement the smallest correct change
+- follow existing patterns in the codebase
+- verify the result with appropriate checks when possible
+- keep `progress.md` accurate when asked to maintain it
+- report back clearly with changes, validation, risks, and next steps
+
+Working rules:
+- Prefer narrow, correct changes over broad rewrites.
+- Do not add speculative scaffolding or future-proofing unless explicitly required.
+- Do not leave placeholder code, TODOs, or silent scope changes.
+- Use `bash` for inspection, validation, and relevant tests.
+- If there is supplied context or a plan, read it first.
+- If implementation reveals a gap in the approved direction, pause and escalate with `contact_supervisor` and `reason: "need_decision"` instead of silently patching around it with an implicit decision.
+- If implementation reveals an unapproved product or architecture choice, use `contact_supervisor` with `reason: "need_decision"` and wait for the reply instead of deciding it yourself or returning a final choose-one answer.
+- If your delegated task expects code or file edits and you have not made those edits, do not return a success summary. Make the edits, contact the supervisor if blocked, or explicitly report that no edits were made.
+- If you send a blocked/progress update through `contact_supervisor`, keep it short and still return the full structured task result normally.
+- Do not send routine completion handoffs. Return the completed implementation summary normally when no coordination is needed.
+
+When running in a chain, expect instructions about:
+- which files to read first
+- where to maintain progress tracking
+- where to write output if a file target is provided
+
+Your final response should follow this shape:
+
+Implemented X.
+Changed files: Y.
+Validation: Z.
+Open risks/questions: R.
+Recommended next step: N.

package/dist/builtin-extensions/pi-subagents/index.d.ts

@@ -0,0 +1,2 @@
+export { default } from "./src/extension/index.ts";
+//# sourceMappingURL=index.d.ts.map

package/dist/builtin-extensions/pi-subagents/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/builtin-extensions/pi-subagents/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,OAAO,EAAE,MAAM,0BAA0B,CAAC","sourcesContent":["// @ts-nocheck\nexport { default } from \"./src/extension/index.ts\";\n"]}

package/dist/builtin-extensions/pi-subagents/index.js

@@ -0,0 +1,3 @@
+// @ts-nocheck
+export { default } from "./src/extension/index.js";
+//# sourceMappingURL=index.js.map

package/dist/builtin-extensions/pi-subagents/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/builtin-extensions/pi-subagents/index.ts"],"names":[],"mappings":"AAAA,cAAc;AACd,OAAO,EAAE,OAAO,EAAE,MAAM,0BAA0B,CAAC","sourcesContent":["// @ts-nocheck\nexport { default } from \"./src/extension/index.ts\";\n"]}

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

@@ -0,0 +1,20 @@
+import type { AgentToolResult } from "@earendil-works/pi-agent-core";
+import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+import type { Details, ExtensionConfig } from "../shared/types.ts";
+type ManagementContext = Pick<ExtensionContext, "cwd" | "modelRegistry"> & {
+    model?: ExtensionContext["model"];
+    config?: ExtensionConfig;
+};
+interface ManagementParams {
+    action?: string;
+    agent?: string;
+    chainName?: string;
+    agentScope?: string;
+    config?: unknown;
+}
+export declare function handleList(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details>;
+export declare function handleCreate(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details>;
+export declare function handleUpdate(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details>;
+export declare function handleManagementAction(action: string, params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details>;
+export {};
+//# sourceMappingURL=agent-management.d.ts.map

package/dist/builtin-extensions/pi-subagents/src/agents/agent-management.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-management.d.ts","sourceRoot":"","sources":["../../../../../src/builtin-extensions/pi-subagents/src/agents/agent-management.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,+BAA+B,CAAC;AACrE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,iCAAiC,CAAC;AAgCxE,OAAO,KAAK,EAAmB,OAAO,EAAE,eAAe,EAAoB,MAAM,oBAAoB,CAAC;AAKtG,KAAK,iBAAiB,GAAG,IAAI,CAAC,gBAAgB,EAAE,KAAK,GAAG,eAAe,CAAC,GAAG;IAAE,KAAK,CAAC,EAAE,gBAAgB,CAAC,OAAO,CAAC,CAAC;IAAC,MAAM,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC;AAE3I,UAAU,gBAAgB;IACzB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,OAAO,CAAC;CACjB;AA+kBD,wBAAgB,UAAU,CAAC,MAAM,EAAE,gBAAgB,EAAE,GAAG,EAAE,iBAAiB,GAAG,eAAe,CAAC,OAAO,CAAC,CA0BrG;AAiHD,wBAAgB,YAAY,CAAC,MAAM,EAAE,gBAAgB,EAAE,GAAG,EAAE,iBAAiB,GAAG,eAAe,CAAC,OAAO,CAAC,CA2DvG;AAED,wBAAgB,YAAY,CAAC,MAAM,EAAE,gBAAgB,EAAE,GAAG,EAAE,iBAAiB,GAAG,eAAe,CAAC,OAAO,CAAC,CA2GvG;AAwJD,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,gBAAgB,EAAE,GAAG,EAAE,iBAAiB,GAAG,eAAe,CAAC,OAAO,CAAC,CAcjI","sourcesContent":["// @ts-nocheck\nimport * as fs from \"node:fs\";\nimport * as path from \"node:path\";\nimport type { AgentToolResult } from \"@earendil-works/pi-agent-core\";\nimport type { ExtensionContext } from \"@earendil-works/pi-coding-agent\";\nimport {\n\ttype AgentConfig,\n\ttype AgentScope,\n\ttype AgentSource,\n\ttype ChainConfig,\n\ttype ChainStepConfig,\n\tBUILTIN_AGENT_NAMES,\n\tdefaultInheritProjectContext,\n\tdefaultInheritSkills,\n\tdefaultSystemPromptMode,\n\tdiscoverAgentsAll,\n\tbuildRuntimeName,\n\tfrontmatterNameForConfig,\n\tparsePackageName,\n\tmergeBuiltinAgentOverride,\n\tremoveBuiltinAgentOverride,\n\tremoveBuiltinAgentOverrideFields,\n} from \"./agents.ts\";\nimport { serializeAgent } from \"./agent-serializer.ts\";\nimport { mergeAgentsForScope } from \"./agent-selection.ts\";\nimport { serializeChain, serializeJsonChain } from \"./chain-serializer.ts\";\nimport { discoverAvailableSkills, resolveSkills } from \"./skills.ts\";\nimport {\n\tbuildProactiveSkillSubagentRecommendationLines,\n} from \"./proactive-skills.ts\";\nimport { parseFrontmatter } from \"./frontmatter.ts\";\nimport { toModelInfo } from \"../shared/model-info.ts\";\nimport { resolveSubagentModelOverride, type ParentModel } from \"../runs/shared/model-fallback.ts\";\nimport { validateToolBudgetConfig } from \"../runs/shared/tool-budget.ts\";\nimport { resolveTurnBudgetConfig } from \"../runs/shared/turn-budget.ts\";\nimport { validateAcceptanceInput } from \"../runs/shared/acceptance.ts\";\nimport type { AcceptanceInput, Details, ExtensionConfig, ToolBudgetConfig } from \"../shared/types.ts\";\nimport { getProjectConfigDir } from \"../shared/utils.ts\";\n\ntype ManagementAction = \"list\" | \"get\" | \"models\" | \"create\" | \"update\" | \"delete\" | \"eject\" | \"disable\" | \"enable\" | \"reset\";\ntype ManagementScope = \"user\" | \"project\";\ntype ManagementContext = Pick<ExtensionContext, \"cwd\" | \"modelRegistry\"> & { model?: ExtensionContext[\"model\"]; config?: ExtensionConfig };\n\ninterface ManagementParams {\n\taction?: string;\n\tagent?: string;\n\tchainName?: string;\n\tagentScope?: string;\n\tconfig?: unknown;\n}\n\nfunction result(text: string, isError = false): AgentToolResult<Details> {\n\treturn { content: [{ type: \"text\", text }], isError, details: { mode: \"management\", results: [] } };\n}\n\nfunction parseCsv(value: string): string[] {\n\treturn [...new Set(value.split(\",\").map((v) => v.trim()).filter(Boolean))];\n}\n\nfunction configObject(config: unknown): { value?: Record<string, unknown>; error?: string } {\n\tlet val = config;\n\tif (typeof val === \"string\") {\n\t\ttry {\n\t\t\tval = JSON.parse(val);\n\t\t} catch (error) {\n\t\t\tconst message = error instanceof Error ? error.message : String(error);\n\t\t\treturn { error: `config must be valid JSON: ${message}` };\n\t\t}\n\t}\n\tif (!val || typeof val !== \"object\" || Array.isArray(val)) return {};\n\treturn { value: val as Record<string, unknown> };\n}\n\nfunction hasKey(obj: Record<string, unknown>, key: string): boolean {\n\treturn Object.prototype.hasOwnProperty.call(obj, key);\n}\n\nfunction asDisambiguationScope(scope: unknown): ManagementScope | undefined {\n\tif (scope === \"user\" || scope === \"project\") return scope;\n\treturn undefined;\n}\n\nfunction actionScope(scope: unknown, action: ManagementAction): { scope?: ManagementScope; error?: AgentToolResult<Details> } {\n\tif (scope === undefined) return { scope: \"user\" };\n\tconst parsed = asDisambiguationScope(scope);\n\treturn parsed ? { scope: parsed } : { error: result(`agentScope must be 'user' or 'project' for ${action}.`, true) };\n}\n\nfunction normalizeListScope(scope: unknown): AgentScope | undefined {\n\tif (scope === undefined) return \"both\";\n\tif (scope === \"user\" || scope === \"project\" || scope === \"both\") return scope;\n\treturn undefined;\n}\n\nfunction sanitizeName(name: string): string {\n\treturn name.toLowerCase().trim().replace(/\\s+/g, \"-\").replace(/[^a-z0-9-]/g, \"\").replace(/-+/g, \"-\").replace(/^-+|-+$/g, \"\");\n}\n\nfunction parsePackageConfig(value: unknown): { packageName?: string; error?: string } {\n\treturn parsePackageName(value, \"config.package\");\n}\n\nfunction allAgents(d: { builtin: AgentConfig[]; package: AgentConfig[]; user: AgentConfig[]; project: AgentConfig[] }): AgentConfig[] {\n\treturn [...d.builtin, ...d.package, ...d.user, ...d.project];\n}\n\nfunction availableNames(cwd: string, kind: \"agent\" | \"chain\"): string[] {\n\tconst d = discoverAgentsAll(cwd);\n\tconst items = kind === \"agent\" ? allAgents(d) : d.chains;\n\treturn [...new Set(items.map((x) => x.name))].sort((a, b) => a.localeCompare(b));\n}\n\nfunction findAgents(name: string, cwd: string, scope: AgentScope = \"both\"): AgentConfig[] {\n\tconst d = discoverAgentsAll(cwd);\n\tconst raw = name.trim();\n\tconst sanitized = sanitizeName(raw);\n\treturn allAgents(d)\n\t\t.filter((a) => (scope === \"both\" || a.source === scope) && (a.name === raw || a.name === sanitized))\n\t\t.sort((a, b) => a.source.localeCompare(b.source));\n}\n\nfunction findChains(name: string, cwd: string, scope: AgentScope = \"both\"): ChainConfig[] {\n\tconst raw = name.trim();\n\tconst sanitized = sanitizeName(raw);\n\treturn discoverAgentsAll(cwd).chains\n\t\t.filter((c) => (scope === \"both\" || c.source === scope) && (c.name === raw || c.name === sanitized))\n\t\t.sort((a, b) => a.source.localeCompare(b.source));\n}\n\nconst AGENT_SOURCE_PRECEDENCE: Record<AgentSource, number> = { builtin: 0, package: 1, user: 2, project: 3 };\n\n// Returns the highest-precedence agent for a name (project > user > package > builtin,\n// matching mergeAgentsForScope for \"both\"), including disabled agents so disable/enable/reset\n// can locate agents that runtime discovery filters out.\nfunction pickEffectiveAgent(d: ReturnType<typeof discoverAgentsAll>, name: string): AgentConfig | undefined {\n\tconst raw = name.trim();\n\tconst sanitized = sanitizeName(raw);\n\tconst matches = allAgents(d).filter((a) => a.name === raw || a.name === sanitized);\n\tif (matches.length === 0) return undefined;\n\treturn matches.reduce((best, agent) => (AGENT_SOURCE_PRECEDENCE[agent.source] > AGENT_SOURCE_PRECEDENCE[best.source] ? agent : best));\n}\n\nfunction nameExistsInScope(cwd: string, scope: ManagementScope, name: string, excludePath?: string): boolean {\n\tconst d = discoverAgentsAll(cwd);\n\tfor (const a of scope === \"user\" ? d.user : d.project) {\n\t\tif (a.name === name && a.filePath !== excludePath) return true;\n\t}\n\tfor (const c of d.chains) {\n\t\tif (c.source === scope && c.name === name && c.filePath !== excludePath) return true;\n\t}\n\treturn false;\n}\n\nfunction isMutableSource(source: AgentSource): source is ManagementScope {\n\treturn source === \"user\" || source === \"project\";\n}\n\nfunction unknownChainAgents(cwd: string, steps: ChainStepConfig[]): string[] {\n\tconst d = discoverAgentsAll(cwd);\n\tconst known = new Set(allAgents(d).map((a) => a.name));\n\treturn [...new Set(steps.map((s) => s.agent).filter((a) => !known.has(a)))].sort((a, b) => a.localeCompare(b));\n}\n\nfunction chainStepWarnings(ctx: ManagementContext, steps: ChainStepConfig[]): string[] {\n\tconst warnings: string[] = [];\n\tconst available = new Set(discoverAvailableSkills(ctx.cwd).map((s) => s.name));\n\tfor (let i = 0; i < steps.length; i++) {\n\t\tconst s = steps[i]!;\n\t\tif (s.model) {\n\t\t\tconst found = ctx.modelRegistry.getAvailable().some((m) => `${m.provider}/${m.id}` === s.model || m.id === s.model);\n\t\t\tif (!found) warnings.push(`Warning: step ${i + 1} (${s.agent}): model '${s.model}' is not in the current model registry.`);\n\t\t}\n\t\tif (Array.isArray(s.skills) && s.skills.length > 0) {\n\t\t\tconst missing = s.skills.filter((sk) => !available.has(sk));\n\t\t\tif (missing.length) warnings.push(`Warning: step ${i + 1} (${s.agent}): skills not found: ${missing.join(\", \")}.`);\n\t\t}\n\t}\n\treturn warnings;\n}\n\nfunction modelWarning(ctx: ManagementContext, model: string | undefined): string | undefined {\n\tif (!model) return undefined;\n\tconst found = ctx.modelRegistry.getAvailable().some((m) => `${m.provider}/${m.id}` === model || m.id === model);\n\treturn found ? undefined : `Warning: model '${model}' is not in the current model registry.`;\n}\n\nfunction fallbackModelsWarning(ctx: ManagementContext, fallbackModels: string[] | undefined): string | undefined {\n\tif (!fallbackModels || fallbackModels.length === 0) return undefined;\n\tconst available = new Set(ctx.modelRegistry.getAvailable().flatMap((m) => [`${m.provider}/${m.id}`, m.id]));\n\tconst missing = fallbackModels.filter((model) => !available.has(model));\n\treturn missing.length ? `Warning: fallback models not in the current model registry: ${missing.join(\", \")}.` : undefined;\n}\n\nfunction skillsWarning(cwd: string, agent: Pick<AgentConfig, \"skills\" | \"skillPath\" | \"filePath\">): string | undefined {\n\tif (!agent.skills?.length) return undefined;\n\tconst { missing } = resolveSkills(\n\t\tagent.skills,\n\t\tcwd,\n\t\tagent.skillPath,\n\t\tagent.filePath ? path.dirname(agent.filePath) : cwd,\n\t);\n\treturn missing.length ? `Warning: skills not found: ${missing.join(\", \")}.` : undefined;\n}\n\nfunction editableAgentConfig(agent: AgentConfig): AgentConfig {\n\tconst base = agent.override?.base;\n\tif (!base) return { ...agent };\n\n\treturn {\n\t\t...agent,\n\t\tmodel: base.model,\n\t\tfallbackModels: base.fallbackModels ? [...base.fallbackModels] : undefined,\n\t\tthinking: base.thinking,\n\t\tsystemPromptMode: base.systemPromptMode,\n\t\tinheritProjectContext: base.inheritProjectContext,\n\t\tinheritSkills: base.inheritSkills,\n\t\tdefaultContext: base.defaultContext,\n\t\tacceptanceRole: base.acceptanceRole,\n\t\tdisabled: base.disabled,\n\t\tsystemPrompt: base.systemPrompt,\n\t\tskills: base.skills ? [...base.skills] : undefined,\n\t\tskillPath: base.skillPath ? [...base.skillPath] : undefined,\n\t\ttools: base.tools ? [...base.tools] : undefined,\n\t\tmcpDirectTools: base.mcpDirectTools ? [...base.mcpDirectTools] : undefined,\n\t\tsubagentOnlyExtensions: base.subagentOnlyExtensions ? [...base.subagentOnlyExtensions] : undefined,\n\t\tcompletionGuard: base.completionGuard,\n\t\toverride: undefined,\n\t};\n}\n\nfunction readAgentFrontmatterFields(filePath: string): Set<string> {\n\ttry {\n\t\tconst { frontmatter } = parseFrontmatter(fs.readFileSync(filePath, \"utf-8\"));\n\t\treturn new Set(Object.keys(frontmatter));\n\t} catch {\n\t\treturn new Set();\n\t}\n}\n\nfunction preservedAgentFrontmatterFields(agent: AgentConfig, cfg: Record<string, unknown>): Set<string> {\n\tconst fields = readAgentFrontmatterFields(agent.filePath);\n\tconst changed = (...names: string[]) => {\n\t\tfor (const name of names) fields.delete(name);\n\t};\n\n\tif (hasKey(cfg, \"name\")) changed(\"name\");\n\tif (hasKey(cfg, \"package\")) changed(\"package\");\n\tif (hasKey(cfg, \"description\")) changed(\"description\");\n\tif (hasKey(cfg, \"systemPrompt\")) changed(\"systemPrompt\");\n\tif (hasKey(cfg, \"model\")) changed(\"model\");\n\tif (hasKey(cfg, \"fallbackModels\")) changed(\"fallbackModels\");\n\tif (hasKey(cfg, \"tools\")) changed(\"tools\");\n\tif (hasKey(cfg, \"skills\")) changed(\"skill\", \"skills\");\n\tif (hasKey(cfg, \"skillPath\")) changed(\"skillPath\");\n\tif (hasKey(cfg, \"extensions\")) changed(\"extensions\");\n\tif (hasKey(cfg, \"subagentOnlyExtensions\")) changed(\"subagentOnlyExtensions\");\n\tif (hasKey(cfg, \"thinking\")) {\n\t\tchanged(\"thinking\");\n\t\tif (cfg.thinking === \"off\") fields.add(\"thinking\");\n\t}\n\tif (hasKey(cfg, \"systemPromptMode\")) {\n\t\tchanged(\"systemPromptMode\");\n\t\tfields.add(\"systemPromptMode\");\n\t}\n\tif (hasKey(cfg, \"inheritProjectContext\")) {\n\t\tchanged(\"inheritProjectContext\");\n\t\tfields.add(\"inheritProjectContext\");\n\t}\n\tif (hasKey(cfg, \"inheritSkills\")) {\n\t\tchanged(\"inheritSkills\");\n\t\tfields.add(\"inheritSkills\");\n\t}\n\tif (hasKey(cfg, \"defaultContext\")) changed(\"defaultContext\");\n\tif (hasKey(cfg, \"async\")) changed(\"async\");\n\tif (hasKey(cfg, \"timeoutMs\")) changed(\"timeoutMs\");\n\tif (hasKey(cfg, \"turnBudget\")) changed(\"turnBudget\");\n\tif (hasKey(cfg, \"acceptance\")) changed(\"acceptance\");\n\tif (hasKey(cfg, \"acceptanceRole\")) changed(\"acceptanceRole\");\n\tif (hasKey(cfg, \"output\")) changed(\"output\");\n\tif (hasKey(cfg, \"reads\")) changed(\"defaultReads\");\n\tif (hasKey(cfg, \"progress\")) changed(\"defaultProgress\");\n\tif (hasKey(cfg, \"maxSubagentDepth\")) changed(\"maxSubagentDepth\");\n\tif (hasKey(cfg, \"completionGuard\")) {\n\t\tchanged(\"completionGuard\");\n\t\tif (cfg.completionGuard === true) fields.add(\"completionGuard\");\n\t}\n\tif (hasKey(cfg, \"toolBudget\")) changed(\"toolBudget\");\n\n\treturn fields;\n}\n\nfunction parseStepList(raw: unknown): { steps?: ChainStepConfig[]; error?: string } {\n\tif (!Array.isArray(raw)) return { error: \"config.steps must be an array.\" };\n\tif (raw.length === 0) return { error: \"config.steps must include at least one step.\" };\n\tconst steps: ChainStepConfig[] = [];\n\tfor (let i = 0; i < raw.length; i++) {\n\t\tconst item = raw[i];\n\t\tif (!item || typeof item !== \"object\" || Array.isArray(item)) return { error: `config.steps[${i}] must be an object.` };\n\t\tconst s = item as Record<string, unknown>;\n\t\tif (typeof s.agent !== \"string\" || !s.agent.trim()) return { error: `config.steps[${i}].agent must be a non-empty string.` };\n\t\tconst step: ChainStepConfig = { agent: s.agent.trim(), task: typeof s.task === \"string\" ? s.task : \"\" };\n\t\tif (hasKey(s, \"phase\")) {\n\t\t\tif (typeof s.phase === \"string\") step.phase = s.phase;\n\t\t\telse return { error: `config.steps[${i}].phase must be a string.` };\n\t\t}\n\t\tif (hasKey(s, \"label\")) {\n\t\t\tif (typeof s.label === \"string\") step.label = s.label;\n\t\t\telse return { error: `config.steps[${i}].label must be a string.` };\n\t\t}\n\t\tif (hasKey(s, \"as\")) {\n\t\t\tif (typeof s.as === \"string\") step.as = s.as;\n\t\t\telse return { error: `config.steps[${i}].as must be a string.` };\n\t\t}\n\t\tif (hasKey(s, \"outputSchema\")) {\n\t\t\tif (typeof s.outputSchema === \"string\") step.outputSchema = s.outputSchema;\n\t\t\telse return { error: `config.steps[${i}].outputSchema must be a schema file path string for saved chains.` };\n\t\t}\n\t\tif (hasKey(s, \"output\")) {\n\t\t\tif (s.output === false) step.output = false;\n\t\t\telse if (typeof s.output === \"string\") step.output = s.output;\n\t\t\telse return { error: `config.steps[${i}].output must be a string or false.` };\n\t\t}\n\t\tif (hasKey(s, \"outputMode\")) {\n\t\t\tif (s.outputMode === \"inline\" || s.outputMode === \"file-only\") step.outputMode = s.outputMode;\n\t\t\telse return { error: `config.steps[${i}].outputMode must be 'inline' or 'file-only'.` };\n\t\t}\n\t\tif (hasKey(s, \"reads\")) {\n\t\t\tif (s.reads === false) step.reads = false;\n\t\t\telse if (Array.isArray(s.reads)) step.reads = s.reads.filter((v): v is string => typeof v === \"string\").map((v) => v.trim()).filter(Boolean);\n\t\t\telse return { error: `config.steps[${i}].reads must be an array or false.` };\n\t\t}\n\t\tif (hasKey(s, \"model\")) {\n\t\t\tif (typeof s.model === \"string\") step.model = s.model;\n\t\t\telse return { error: `config.steps[${i}].model must be a string.` };\n\t\t}\n\t\tif (hasKey(s, \"skills\")) {\n\t\t\tif (s.skills === false) step.skills = false;\n\t\t\telse if (Array.isArray(s.skills)) step.skills = s.skills.filter((v): v is string => typeof v === \"string\").map((v) => v.trim()).filter(Boolean);\n\t\t\telse return { error: `config.steps[${i}].skills must be an array or false.` };\n\t\t}\n\t\tif (hasKey(s, \"progress\")) {\n\t\t\tif (typeof s.progress === \"boolean\") step.progress = s.progress;\n\t\t\telse return { error: `config.steps[${i}].progress must be a boolean.` };\n\t\t}\n\t\tif (hasKey(s, \"toolBudget\")) {\n\t\t\tconst validation = validateToolBudgetConfig(s.toolBudget, `config.steps[${i}].toolBudget`);\n\t\t\tif (validation.error) return { error: validation.error };\n\t\t\tstep.toolBudget = s.toolBudget as ChainStepConfig[\"toolBudget\"];\n\t\t}\n\t\tsteps.push(step);\n\t}\n\treturn { steps };\n}\n\nfunction parseTools(raw: string): { tools?: string[]; mcpDirectTools?: string[] } {\n\tconst tools: string[] = [];\n\tconst mcpDirectTools: string[] = [];\n\tfor (const item of parseCsv(raw)) {\n\t\tif (item.startsWith(\"mcp:\")) {\n\t\t\tconst direct = item.slice(4).trim();\n\t\t\tif (direct) mcpDirectTools.push(direct);\n\t\t} else tools.push(item);\n\t}\n\treturn { tools: tools.length ? tools : undefined, mcpDirectTools: mcpDirectTools.length ? mcpDirectTools : undefined };\n}\n\nfunction applyAgentConfig(target: AgentConfig, cfg: Record<string, unknown>): string | undefined {\n\tif (hasKey(cfg, \"systemPrompt\")) {\n\t\tif (cfg.systemPrompt === false || cfg.systemPrompt === \"\") target.systemPrompt = \"\";\n\t\telse if (typeof cfg.systemPrompt === \"string\") target.systemPrompt = cfg.systemPrompt;\n\t\telse return \"config.systemPrompt must be a string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"model\")) {\n\t\tif (cfg.model === false || cfg.model === \"\") target.model = undefined;\n\t\telse if (typeof cfg.model === \"string\") target.model = cfg.model.trim() || undefined;\n\t\telse return \"config.model must be a string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"fallbackModels\")) {\n\t\tif (cfg.fallbackModels === false || cfg.fallbackModels === \"\") target.fallbackModels = undefined;\n\t\telse if (typeof cfg.fallbackModels === \"string\") {\n\t\t\tconst models = parseCsv(cfg.fallbackModels);\n\t\t\ttarget.fallbackModels = models.length ? models : undefined;\n\t\t} else if (Array.isArray(cfg.fallbackModels)) {\n\t\t\tconst models = cfg.fallbackModels\n\t\t\t\t.filter((value): value is string => typeof value === \"string\")\n\t\t\t\t.map((value) => value.trim())\n\t\t\t\t.filter(Boolean);\n\t\t\ttarget.fallbackModels = models.length ? [...new Set(models)] : undefined;\n\t\t} else return \"config.fallbackModels must be a comma-separated string, string array, or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"tools\")) {\n\t\tif (cfg.tools === false || cfg.tools === \"\") { target.tools = undefined; target.mcpDirectTools = undefined; }\n\t\telse if (typeof cfg.tools === \"string\") { const parsed = parseTools(cfg.tools); target.tools = parsed.tools; target.mcpDirectTools = parsed.mcpDirectTools; }\n\t\telse return \"config.tools must be a comma-separated string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"skills\")) {\n\t\tif (cfg.skills === false || cfg.skills === \"\") target.skills = undefined;\n\t\telse if (typeof cfg.skills === \"string\") { const skills = parseCsv(cfg.skills); target.skills = skills.length ? skills : undefined; }\n\t\telse return \"config.skills must be a comma-separated string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"skillPath\")) {\n\t\tif (cfg.skillPath === false || cfg.skillPath === \"\") target.skillPath = undefined;\n\t\telse if (typeof cfg.skillPath === \"string\") { const skillPath = parseCsv(cfg.skillPath); target.skillPath = skillPath.length ? skillPath : undefined; }\n\t\telse if (Array.isArray(cfg.skillPath) && cfg.skillPath.every((entry) => typeof entry === \"string\")) {\n\t\t\tconst skillPath = [...new Set(cfg.skillPath.map((entry) => entry.trim()).filter(Boolean))];\n\t\t\ttarget.skillPath = skillPath.length ? skillPath : undefined;\n\t\t} else return \"config.skillPath must be a comma-separated string, string array, or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"extensions\")) {\n\t\tif (cfg.extensions === false) target.extensions = undefined;\n\t\telse if (cfg.extensions === \"\") target.extensions = [];\n\t\telse if (typeof cfg.extensions === \"string\") target.extensions = parseCsv(cfg.extensions);\n\t\telse return \"config.extensions must be a comma-separated string, empty string, or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"subagentOnlyExtensions\")) {\n\t\tif (cfg.subagentOnlyExtensions === false) target.subagentOnlyExtensions = undefined;\n\t\telse if (cfg.subagentOnlyExtensions === \"\") target.subagentOnlyExtensions = [];\n\t\telse if (typeof cfg.subagentOnlyExtensions === \"string\") target.subagentOnlyExtensions = parseCsv(cfg.subagentOnlyExtensions);\n\t\telse return \"config.subagentOnlyExtensions must be a comma-separated string, empty string, or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"thinking\")) {\n\t\tif (cfg.thinking === false || cfg.thinking === \"\") target.thinking = undefined;\n\t\telse if (typeof cfg.thinking === \"string\") target.thinking = cfg.thinking.trim() || undefined;\n\t\telse return \"config.thinking must be a string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"systemPromptMode\")) {\n\t\tif (cfg.systemPromptMode === \"append\" || cfg.systemPromptMode === \"replace\") target.systemPromptMode = cfg.systemPromptMode;\n\t\telse return \"config.systemPromptMode must be 'append' or 'replace' when provided.\";\n\t}\n\tif (hasKey(cfg, \"inheritProjectContext\")) {\n\t\tif (typeof cfg.inheritProjectContext !== \"boolean\") return \"config.inheritProjectContext must be a boolean when provided.\";\n\t\ttarget.inheritProjectContext = cfg.inheritProjectContext;\n\t}\n\tif (hasKey(cfg, \"inheritSkills\")) {\n\t\tif (typeof cfg.inheritSkills !== \"boolean\") return \"config.inheritSkills must be a boolean when provided.\";\n\t\ttarget.inheritSkills = cfg.inheritSkills;\n\t}\n\tif (hasKey(cfg, \"defaultContext\")) {\n\t\tif (cfg.defaultContext === false || cfg.defaultContext === \"\") target.defaultContext = undefined;\n\t\telse if (cfg.defaultContext === \"fresh\" || cfg.defaultContext === \"fork\") target.defaultContext = cfg.defaultContext;\n\t\telse return \"config.defaultContext must be 'fresh', 'fork', or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"async\")) {\n\t\tif (cfg.async === \"\") target.defaultAsync = undefined;\n\t\telse if (typeof cfg.async === \"boolean\") target.defaultAsync = cfg.async;\n\t\telse return \"config.async must be a boolean or empty string when provided.\";\n\t}\n\tif (hasKey(cfg, \"timeoutMs\")) {\n\t\tif (cfg.timeoutMs === false || cfg.timeoutMs === \"\") target.defaultTimeoutMs = undefined;\n\t\telse if (typeof cfg.timeoutMs === \"number\" && Number.isInteger(cfg.timeoutMs) && cfg.timeoutMs > 0) target.defaultTimeoutMs = cfg.timeoutMs;\n\t\telse return \"config.timeoutMs must be a positive integer or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"turnBudget\")) {\n\t\tif (cfg.turnBudget === false || cfg.turnBudget === \"\") target.defaultTurnBudget = undefined;\n\t\telse {\n\t\t\tconst resolved = resolveTurnBudgetConfig(cfg.turnBudget, \"config.turnBudget\");\n\t\t\tif (resolved.error) return resolved.error;\n\t\t\ttarget.defaultTurnBudget = resolved.turnBudget;\n\t\t}\n\t}\n\tif (hasKey(cfg, \"acceptance\")) {\n\t\tif (cfg.acceptance === \"\") target.defaultAcceptance = undefined;\n\t\telse {\n\t\t\tconst errors = validateAcceptanceInput(cfg.acceptance, \"config.acceptance\");\n\t\t\tif (errors.length > 0) return errors.join(\" \");\n\t\t\ttarget.defaultAcceptance = cfg.acceptance as AcceptanceInput;\n\t\t}\n\t}\n\tif (hasKey(cfg, \"acceptanceRole\")) {\n\t\tif (cfg.acceptanceRole === false || cfg.acceptanceRole === \"\") target.acceptanceRole = undefined;\n\t\telse if (cfg.acceptanceRole === \"read-only\" || cfg.acceptanceRole === \"writer\") target.acceptanceRole = cfg.acceptanceRole;\n\t\telse return \"config.acceptanceRole must be 'read-only', 'writer', or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"output\")) {\n\t\tif (cfg.output === false || cfg.output === \"\") target.output = undefined;\n\t\telse if (typeof cfg.output === \"string\") target.output = cfg.output;\n\t\telse return \"config.output must be a string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"reads\")) {\n\t\tif (cfg.reads === false || cfg.reads === \"\") target.defaultReads = undefined;\n\t\telse if (typeof cfg.reads === \"string\") {\n\t\t\tconst reads = parseCsv(cfg.reads);\n\t\t\ttarget.defaultReads = reads.length ? reads : undefined;\n\t\t} else return \"config.reads must be a comma-separated string or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"progress\")) {\n\t\tif (typeof cfg.progress !== \"boolean\") return \"config.progress must be a boolean when provided.\";\n\t\ttarget.defaultProgress = cfg.progress;\n\t}\n\tif (hasKey(cfg, \"maxSubagentDepth\")) {\n\t\tif (cfg.maxSubagentDepth === false || cfg.maxSubagentDepth === \"\") target.maxSubagentDepth = undefined;\n\t\telse if (typeof cfg.maxSubagentDepth === \"number\" && Number.isInteger(cfg.maxSubagentDepth) && cfg.maxSubagentDepth >= 0) {\n\t\t\ttarget.maxSubagentDepth = cfg.maxSubagentDepth;\n\t\t} else return \"config.maxSubagentDepth must be an integer >= 0 or false when provided.\";\n\t}\n\tif (hasKey(cfg, \"completionGuard\")) {\n\t\tif (typeof cfg.completionGuard !== \"boolean\") return \"config.completionGuard must be a boolean when provided.\";\n\t\ttarget.completionGuard = cfg.completionGuard;\n\t}\n\tif (hasKey(cfg, \"toolBudget\")) {\n\t\tif (cfg.toolBudget === false || cfg.toolBudget === \"\") target.toolBudget = undefined;\n\t\telse {\n\t\t\tconst validation = validateToolBudgetConfig(cfg.toolBudget, \"config.toolBudget\");\n\t\t\tif (validation.error) return validation.error;\n\t\t\ttarget.toolBudget = cfg.toolBudget as ToolBudgetConfig;\n\t\t}\n\t}\n\treturn undefined;\n}\n\nfunction resolveTarget<T extends { source: AgentSource; filePath: string }>(\n\tkind: \"agent\" | \"chain\",\n\tname: string,\n\tmatches: T[],\n\tcwd: string,\n\tscopeHint?: string,\n): T | AgentToolResult<Details> {\n\tconst mutable = matches.filter((m): m is T & { source: ManagementScope } => isMutableSource(m.source));\n\tif (mutable.length === 0) {\n\t\tif (matches.length > 0) {\n\t\t\treturn result(`${kind === \"agent\" ? \"Agent\" : \"Chain\"} '${name}' is read-only and cannot be modified. Create a same-named ${kind} in user or project scope to override it.`, true);\n\t\t}\n\t\tconst available = availableNames(cwd, kind);\n\t\treturn result(`${kind === \"agent\" ? \"Agent\" : \"Chain\"} '${name}' not found. Available: ${available.join(\", \") || \"none\"}.`, true);\n\t}\n\tif (mutable.length === 1) return mutable[0]!;\n\tconst scope = asDisambiguationScope(scopeHint);\n\tif (!scope) {\n\t\tconst paths = mutable.map((m) => `${m.source}: ${m.filePath}`).join(\"\\n\");\n\t\treturn result(`${kind === \"agent\" ? \"Agent\" : \"Chain\"} '${name}' exists in both scopes. Specify agentScope: 'user' or 'project'.\\n${paths}`, true);\n\t}\n\tconst scoped = mutable.filter((m) => m.source === scope);\n\tif (scoped.length === 0) return result(`${kind === \"agent\" ? \"Agent\" : \"Chain\"} '${name}' not found in scope '${scope}'.`, true);\n\tif (scoped.length > 1) return result(`Multiple ${kind}s named '${name}' found in scope '${scope}': ${scoped.map((m) => m.filePath).join(\", \")}`, true);\n\treturn scoped[0]!;\n}\n\nfunction renamePath(\n\tkind: \"agent\" | \"chain\",\n\tcurrentPath: string,\n\tnewName: string,\n\tscope: ManagementScope,\n\tcwd: string,\n): { filePath?: string; error?: string } {\n\tif (nameExistsInScope(cwd, scope, newName, currentPath)) return { error: `Name '${newName}' already exists in ${scope} scope.` };\n\tconst ext = kind === \"agent\" ? \".md\" : currentPath.endsWith(\".chain.json\") ? \".chain.json\" : \".chain.md\";\n\tconst filePath = path.join(path.dirname(currentPath), `${newName}${ext}`);\n\tif (fs.existsSync(filePath) && filePath !== currentPath) {\n\t\treturn { error: `File already exists at ${filePath} but is not a valid ${kind} definition. Remove or rename it first.` };\n\t}\n\tfs.renameSync(currentPath, filePath);\n\treturn { filePath };\n}\n\nfunction formatAgentDetail(agent: AgentConfig): string {\n\tconst tools = [...(agent.tools ?? []), ...(agent.mcpDirectTools ?? []).map((t) => `mcp:${t}`)];\n\tconst lines: string[] = [`Agent: ${agent.name} (${agent.source})`, `Path: ${agent.filePath}`, `Description: ${agent.description}`];\n\tif (agent.packageName) {\n\t\tlines.push(`Local name: ${frontmatterNameForConfig(agent)}`);\n\t\tlines.push(`Package: ${agent.packageName}`);\n\t}\n\tif (agent.model) lines.push(`Model: ${agent.model}`);\n\tif (agent.fallbackModels?.length) lines.push(`Fallback models: ${agent.fallbackModels.join(\", \")}`);\n\tif (tools.length) lines.push(`Tools: ${tools.join(\", \")}`);\n\tif (agent.skills?.length) lines.push(`Skills: ${agent.skills.join(\", \")}`);\n\tif (agent.skillPath?.length) lines.push(`Skill paths: ${agent.skillPath.join(\", \")}`);\n\tlines.push(`System prompt mode: ${agent.systemPromptMode}`);\n\tlines.push(`Inherit project context: ${agent.inheritProjectContext ? \"true\" : \"false\"}`);\n\tlines.push(`Inherit skills: ${agent.inheritSkills ? \"true\" : \"false\"}`);\n\tif (agent.defaultContext) lines.push(`Default context: ${agent.defaultContext}`);\n\tif (agent.defaultAsync !== undefined) lines.push(`Async: ${agent.defaultAsync ? \"true\" : \"false\"}`);\n\tif (agent.defaultTimeoutMs !== undefined) lines.push(`Timeout: ${agent.defaultTimeoutMs}ms`);\n\tif (agent.defaultTurnBudget) lines.push(`Turn budget: ${JSON.stringify(agent.defaultTurnBudget)}`);\n\tif (agent.defaultAcceptance !== undefined) lines.push(`Acceptance: ${typeof agent.defaultAcceptance === \"object\" ? JSON.stringify(agent.defaultAcceptance) : String(agent.defaultAcceptance)}`);\n\tif (agent.acceptanceRole) lines.push(`Acceptance role: ${agent.acceptanceRole}`);\n\tif (agent.source === \"builtin\") lines.push(`Disabled: ${agent.disabled ? \"true\" : \"false\"}`);\n\tif (agent.extensions !== undefined) lines.push(`Extensions: ${agent.extensions.length ? agent.extensions.join(\", \") : \"(none)\"}`);\n\tif (agent.subagentOnlyExtensions !== undefined) lines.push(`Subagent-only extensions: ${agent.subagentOnlyExtensions.length ? agent.subagentOnlyExtensions.join(\", \") : \"(none)\"}`);\n\tif (agent.thinking) lines.push(`Thinking: ${agent.thinking}`);\n\tif (agent.output) lines.push(`Output: ${agent.output}`);\n\tif (agent.defaultReads?.length) lines.push(`Reads: ${agent.defaultReads.join(\", \")}`);\n\tif (agent.defaultProgress) lines.push(\"Progress: true\");\n\tif (agent.maxSubagentDepth !== undefined) lines.push(`Max subagent depth: ${agent.maxSubagentDepth}`);\n\tif (agent.completionGuard === false) lines.push(\"Completion guard: false\");\n\tif (agent.toolBudget) lines.push(`Tool budget: ${JSON.stringify(agent.toolBudget)}`);\n\tif (agent.memory) lines.push(`Memory: ${agent.memory.scope} scope, path: ${agent.memory.path}`);\n\tif (agent.systemPrompt.trim()) lines.push(\"\", \"System Prompt:\", agent.systemPrompt);\n\treturn lines.join(\"\\n\");\n}\n\nfunction formatChainStepDetail(step: ChainStepConfig, index: number): string[] {\n\tconst lines: string[] = [];\n\tif (step.expand || step.collect) {\n\t\tconst parallel = step.parallel && !Array.isArray(step.parallel) && typeof step.parallel === \"object\" ? step.parallel as { agent?: unknown; task?: unknown; label?: unknown; outputSchema?: unknown } : undefined;\n\t\tconst expand = step.expand && typeof step.expand === \"object\" ? step.expand as { from?: { output?: unknown; path?: unknown }; item?: unknown; key?: unknown; maxItems?: unknown; onEmpty?: unknown } : undefined;\n\t\tconst collect = step.collect && typeof step.collect === \"object\" ? step.collect as { as?: unknown; outputSchema?: unknown } : undefined;\n\t\tlines.push(`${index + 1}. Dynamic fanout${typeof collect?.as === \"string\" ? ` -> ${collect.as}` : \"\"}`);\n\t\tif (expand?.from) lines.push(`   Expand: ${String(expand.from.output ?? \"?\")}${String(expand.from.path ?? \"\")}`);\n\t\tif (typeof expand?.item === \"string\") lines.push(`   Item variable: ${expand.item}`);\n\t\tif (typeof expand?.key === \"string\") lines.push(`   Key: ${expand.key}`);\n\t\tif (typeof expand?.maxItems === \"number\") lines.push(`   Max items: ${expand.maxItems}`);\n\t\tif (typeof expand?.onEmpty === \"string\") lines.push(`   On empty: ${expand.onEmpty}`);\n\t\tif (parallel?.agent) lines.push(`   Agent: ${String(parallel.agent)}`);\n\t\tif (typeof parallel?.label === \"string\") lines.push(`   Label: ${parallel.label}`);\n\t\tif (typeof parallel?.task === \"string\" && parallel.task.trim()) lines.push(`   Task: ${parallel.task}`);\n\t\tif (parallel?.outputSchema) lines.push(\"   Structured output: true\");\n\t\tif (parallel && \"toolBudget\" in parallel) lines.push(`   Tool budget: ${JSON.stringify((parallel as { toolBudget?: unknown }).toolBudget)}`);\n\t\tif (collect?.outputSchema) lines.push(\"   Collect schema: true\");\n\t\tif (step.concurrency !== undefined) lines.push(`   Concurrency: ${step.concurrency}`);\n\t\tif (step.failFast !== undefined) lines.push(`   Fail fast: ${step.failFast ? \"true\" : \"false\"}`);\n\t\treturn lines;\n\t}\n\tlines.push(`${index + 1}. ${step.agent}`);\n\tif (step.task?.trim()) lines.push(`   Task: ${step.task}`);\n\tif (step.output === false) lines.push(\"   Output: false\");\n\telse if (step.output) lines.push(`   Output: ${step.output}`);\n\tif (step.outputMode) lines.push(`   Output mode: ${step.outputMode}`);\n\tif (step.toolBudget) lines.push(`   Tool budget: ${JSON.stringify(step.toolBudget)}`);\n\tif (step.reads === false) lines.push(\"   Reads: false\");\n\telse if (Array.isArray(step.reads) && step.reads.length > 0) lines.push(`   Reads: ${step.reads.join(\", \")}`);\n\tif (step.model) lines.push(`   Model: ${step.model}`);\n\tif (step.skills === false) lines.push(\"   Skills: false\");\n\telse if (Array.isArray(step.skills) && step.skills.length > 0) lines.push(`   Skills: ${step.skills.join(\", \")}`);\n\tif (step.progress !== undefined) lines.push(`   Progress: ${step.progress ? \"true\" : \"false\"}`);\n\treturn lines;\n}\n\nfunction formatChainDetail(chain: ChainConfig): string {\n\tconst lines: string[] = [`Chain: ${chain.name} (${chain.source})`, `Path: ${chain.filePath}`, `Description: ${chain.description}`];\n\tif (chain.packageName) {\n\t\tlines.push(`Local name: ${frontmatterNameForConfig(chain)}`);\n\t\tlines.push(`Package: ${chain.packageName}`);\n\t}\n\tlines.push(\"\", \"Steps:\");\n\tfor (let i = 0; i < chain.steps.length; i++) {\n\t\tlines.push(...formatChainStepDetail(chain.steps[i]!, i));\n\t}\n\treturn lines.join(\"\\n\");\n}\n\nexport function handleList(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tconst scope = normalizeListScope(params.agentScope) ?? \"both\";\n\tconst d = discoverAgentsAll(ctx.cwd);\n\tconst scopedAgents = mergeAgentsForScope(scope, d.user, d.project, d.builtin, d.package)\n\t\t.sort((a, b) => a.name.localeCompare(b.name));\n\tconst agents = scopedAgents.filter((a) => !a.disabled);\n\tconst chains = d.chains.filter((c) => scope === \"both\" || c.source === \"package\" || c.source === scope).sort((a, b) => a.name.localeCompare(b.name));\n\tconst diagnostics = d.chainDiagnostics.filter((entry) => scope === \"both\" || entry.source === scope);\n\tconst proactiveSuggestions = buildProactiveSkillSubagentRecommendationLines({\n\t\tagents,\n\t\tchains,\n\t\tconfig: ctx.config?.proactiveSkillSubagents,\n\t\tdiscoverAvailableSkills: () => discoverAvailableSkills(ctx.cwd),\n\t});\n\tconst lines = [\n\t\t\"Executable agents:\",\n\t\t...(agents.length\n\t\t\t? agents.map((a) => `- ${a.name} (${a.source}${a.defaultContext ? `, context: ${a.defaultContext}` : \"\"}): ${a.description}`)\n\t\t\t: [\"- (none)\"]),\n\t\t\"\",\n\t\t\"Chains:\",\n\t\t...(chains.length ? chains.map((c) => `- ${c.name} (${c.source}): ${c.description}`) : [\"- (none)\"]),\n\t\t...(proactiveSuggestions.length ? [\"\", ...proactiveSuggestions] : []),\n\t\t...(diagnostics.length ? [\"\", \"Chain diagnostics:\", ...diagnostics.map((entry) => `- ${entry.filePath}: ${entry.error}`)] : []),\n\t];\n\treturn result(lines.join(\"\\n\"));\n}\n\nfunction formatModelSource(agent: AgentConfig, currentModel: ParentModel | undefined): string {\n\tif (agent.override && agent.model !== agent.override.base.model) {\n\t\treturn `${agent.override.scope} override`;\n\t}\n\tif (agent.modelSource?.type === \"subagents.defaultModel\" && agent.model === agent.modelSource.model) {\n\t\treturn `${agent.modelSource.scope} defaultModel`;\n\t}\n\tif (agent.model) return \"builtin agent config\";\n\tif (currentModel) return \"inherits current session model\";\n\treturn \"inherit requested, but no current session model is available\";\n}\n\nfunction handleModels(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tconst requestedAgent = params.agent?.trim();\n\tif (requestedAgent && !(BUILTIN_AGENT_NAMES as readonly string[]).includes(requestedAgent)) {\n\t\treturn result(`Builtin agent '${requestedAgent}' not found. Available: ${BUILTIN_AGENT_NAMES.join(\", \")}.`, true);\n\t}\n\n\tconst discovered = discoverAgentsAll(ctx.cwd);\n\tconst builtinByName = new Map(discovered.builtin.map((agent) => [agent.name, agent]));\n\tconst availableModels = ctx.modelRegistry.getAvailable().map(toModelInfo);\n\tconst currentModel = ctx.model ? { provider: ctx.model.provider, id: ctx.model.id } : undefined;\n\tconst preferredProvider = ctx.model?.provider;\n\tconst names = requestedAgent ? [requestedAgent] : [...BUILTIN_AGENT_NAMES];\n\n\tif (requestedAgent) {\n\t\tconst agent = builtinByName.get(requestedAgent);\n\t\tif (!agent) return result(`Builtin agent '${requestedAgent}' not found.`, true);\n\t\tconst resolvedModel = resolveSubagentModelOverride(agent.model, currentModel, availableModels, preferredProvider);\n\t\tconst lines = [\n\t\t\t\"Builtin subagent model\",\n\t\t\t\"\",\n\t\t\t`Agent: ${requestedAgent}`,\n\t\t\t\"Effective model:\",\n\t\t\t`  ${resolvedModel ?? \"(unresolved)\"}`,\n\t\t\t`Source: ${formatModelSource(agent, currentModel)}`,\n\t\t];\n\t\tif (agent.override) {\n\t\t\tlines.push(\"Override file:\");\n\t\t\tlines.push(`  ${agent.override.path}`);\n\t\t}\n\t\tif (agent.model && resolvedModel && agent.model !== resolvedModel) {\n\t\t\tlines.push(\"Requested model setting:\");\n\t\t\tlines.push(`  ${agent.model}`);\n\t\t}\n\t\tif (agent.disabled) lines.push(\"Disabled: true\");\n\t\tlines.push(\"Current session model:\");\n\t\tlines.push(`  ${currentModel ? `${currentModel.provider}/${currentModel.id}` : \"(unavailable)\"}`);\n\t\treturn result(lines.join(\"\\n\"));\n\t}\n\n\tconst lines = [\n\t\t\"Builtin subagent models\",\n\t\t\"\",\n\t\t\"Current session model:\",\n\t\t`  ${currentModel ? `${currentModel.provider}/${currentModel.id}` : \"(unavailable)\"}`,\n\t\t\"\",\n\t];\n\n\tfor (const name of names) {\n\t\tconst agent = builtinByName.get(name);\n\t\tif (!agent) {\n\t\t\tlines.push(name);\n\t\t\tlines.push(\"  model:\");\n\t\t\tlines.push(\"    (builtin definition not found)\");\n\t\t\tlines.push(\"  source: missing\");\n\t\t\tlines.push(\"\");\n\t\t\tcontinue;\n\t\t}\n\t\tconst resolvedModel = resolveSubagentModelOverride(agent.model, currentModel, availableModels, preferredProvider);\n\t\tconst source = `${formatModelSource(agent, currentModel)}${agent.disabled ? \"; disabled\" : \"\"}`;\n\t\tlines.push(name);\n\t\tlines.push(\"  model:\");\n\t\tlines.push(`    ${resolvedModel ?? \"(unresolved)\"}`);\n\t\tlines.push(`  source: ${source}`);\n\t\tlines.push(\"\");\n\t}\n\n\treturn result(lines.join(\"\\n\"));\n}\n\nfunction handleGet(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent && !params.chainName) return result(\"Specify 'agent' or 'chainName' for get.\", true);\n\tconst hasBoth = Boolean(params.agent && params.chainName);\n\tconst blocks: string[] = [];\n\tlet anyFound = false;\n\tif (params.agent) {\n\t\tconst matches = findAgents(params.agent, ctx.cwd, \"both\");\n\t\tif (!matches.length) {\n\t\t\tconst msg = `Agent '${params.agent}' not found. Available: ${availableNames(ctx.cwd, \"agent\").join(\", \") || \"none\"}.`;\n\t\t\tif (!hasBoth) return result(msg, true);\n\t\t\tblocks.push(msg);\n\t\t} else {\n\t\t\tanyFound = true;\n\t\t\tblocks.push(...matches.map(formatAgentDetail));\n\t\t}\n\t}\n\tif (params.chainName) {\n\t\tconst matches = findChains(params.chainName, ctx.cwd, \"both\");\n\t\tif (!matches.length) {\n\t\t\tconst msg = `Chain '${params.chainName}' not found. Available: ${availableNames(ctx.cwd, \"chain\").join(\", \") || \"none\"}.`;\n\t\t\tif (!hasBoth) return result(msg, true);\n\t\t\tblocks.push(msg);\n\t\t} else {\n\t\t\tanyFound = true;\n\t\t\tblocks.push(...matches.map(formatChainDetail));\n\t\t}\n\t}\n\treturn result(blocks.join(\"\\n\\n\"), !anyFound);\n}\n\nexport function handleCreate(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tconst parsedConfig = configObject(params.config);\n\tif (parsedConfig.error) return result(parsedConfig.error, true);\n\tconst cfg = parsedConfig.value;\n\tif (!cfg) return result(\"config required for create.\", true);\n\tif (typeof cfg.name !== \"string\" || !cfg.name.trim()) return result(\"config.name is required and must be a non-empty string.\", true);\n\tif (typeof cfg.description !== \"string\" || !cfg.description.trim()) return result(\"config.description is required and must be a non-empty string.\", true);\n\tconst name = sanitizeName(cfg.name);\n\tif (!name) return result(\"config.name is invalid after sanitization. Use letters, numbers, spaces, or hyphens.\", true);\n\tconst parsedPackage = parsePackageConfig(cfg.package);\n\tif (parsedPackage.error) return result(parsedPackage.error, true);\n\tconst runtimeName = buildRuntimeName(name, parsedPackage.packageName);\n\tconst scopeRaw = cfg.scope ?? \"user\";\n\tif (scopeRaw !== \"user\" && scopeRaw !== \"project\") return result(\"config.scope must be 'user' or 'project'.\", true);\n\tconst scope = scopeRaw as ManagementScope;\n\tconst isChain = hasKey(cfg, \"steps\");\n\tconst d = discoverAgentsAll(ctx.cwd);\n\tconst projectConfigDir = getProjectConfigDir(ctx.cwd);\n\tconst targetDir = isChain\n\t\t? scope === \"user\" ? d.userChainDir : d.projectChainDir ?? path.join(projectConfigDir, \"chains\")\n\t\t: scope === \"user\" ? d.userDir : d.projectDir ?? path.join(projectConfigDir, \"agents\");\n\tfs.mkdirSync(targetDir, { recursive: true });\n\tif (nameExistsInScope(ctx.cwd, scope, runtimeName)) return result(`Name '${runtimeName}' already exists in ${scope} scope. Use update instead.`, true);\n\tconst targetPath = path.join(targetDir, isChain ? `${runtimeName}.chain.md` : `${runtimeName}.md`);\n\tif (fs.existsSync(targetPath)) return result(`File already exists at ${targetPath} but is not a valid ${isChain ? \"chain\" : \"agent\"} definition. Remove or rename it first.`, true);\n\tconst warnings: string[] = [];\n\tif (!isChain && d.builtin.some((a) => a.name === runtimeName)) warnings.push(`Note: this shadows the builtin agent '${runtimeName}'.`);\n\tif (isChain) {\n\t\tconst parsed = parseStepList(cfg.steps);\n\t\tif (parsed.error) return result(parsed.error, true);\n\t\tconst chain: ChainConfig = { name: runtimeName, localName: name, packageName: parsedPackage.packageName, description: cfg.description.trim(), source: scope, filePath: targetPath, steps: parsed.steps! };\n\t\tfs.writeFileSync(targetPath, serializeChain(chain), \"utf-8\");\n\t\tconst missing = unknownChainAgents(ctx.cwd, chain.steps);\n\t\tif (missing.length) warnings.push(`Warning: chain steps reference unknown agents: ${missing.join(\", \")}.`);\n\t\twarnings.push(...chainStepWarnings(ctx, chain.steps));\n\t\treturn result([`Created chain '${runtimeName}' at ${targetPath}.`, ...warnings].join(\"\\n\"));\n\t}\n\tconst agent: AgentConfig = {\n\t\tname: runtimeName,\n\t\tlocalName: name,\n\t\tpackageName: parsedPackage.packageName,\n\t\tdescription: cfg.description.trim(),\n\t\tsource: scope,\n\t\tfilePath: targetPath,\n\t\tsystemPrompt: \"\",\n\t\tsystemPromptMode: defaultSystemPromptMode(name),\n\t\tinheritProjectContext: defaultInheritProjectContext(name),\n\t\tinheritSkills: defaultInheritSkills(),\n\t};\n\tconst applyError = applyAgentConfig(agent, cfg);\n\tif (applyError) return result(applyError, true);\n\tconst mw = modelWarning(ctx, agent.model);\n\tif (mw) warnings.push(mw);\n\tconst fmw = fallbackModelsWarning(ctx, agent.fallbackModels);\n\tif (fmw) warnings.push(fmw);\n\tconst sw = skillsWarning(ctx.cwd, agent);\n\tif (sw) warnings.push(sw);\n\tfs.writeFileSync(targetPath, serializeAgent(agent), \"utf-8\");\n\treturn result([`Created agent '${runtimeName}' at ${targetPath}.`, ...warnings].join(\"\\n\"));\n}\n\nexport function handleUpdate(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent && !params.chainName) return result(\"Specify 'agent' or 'chainName' for update.\", true);\n\tif (params.agent && params.chainName) return result(\"Specify either 'agent' or 'chainName', not both.\", true);\n\tconst parsedConfig = configObject(params.config);\n\tif (parsedConfig.error) return result(parsedConfig.error, true);\n\tconst cfg = parsedConfig.value;\n\tif (!cfg) return result(\"config required for update.\", true);\n\tconst warnings: string[] = [];\n\tif (params.agent) {\n\t\tconst scopeHint = asDisambiguationScope(params.agentScope);\n\t\tconst targetOrError = resolveTarget(\"agent\", params.agent, findAgents(params.agent, ctx.cwd, scopeHint ?? \"both\"), ctx.cwd, params.agentScope);\n\t\tif (\"content\" in targetOrError) return targetOrError;\n\t\tconst target = targetOrError;\n\t\tconst updated = editableAgentConfig(target);\n\t\tconst oldName = target.name;\n\t\tif (hasKey(cfg, \"name\") && (typeof cfg.name !== \"string\" || !cfg.name.trim())) return result(\"config.name must be a non-empty string when provided.\", true);\n\t\tif (hasKey(cfg, \"description\") && (typeof cfg.description !== \"string\" || !cfg.description.trim())) return result(\"config.description must be a non-empty string when provided.\", true);\n\t\tlet newLocalName = target.localName ?? frontmatterNameForConfig(target);\n\t\tif (hasKey(cfg, \"name\")) {\n\t\t\tnewLocalName = sanitizeName(cfg.name as string);\n\t\t\tif (!newLocalName) return result(\"config.name is invalid after sanitization.\", true);\n\t\t}\n\t\tlet newPackageName = target.packageName;\n\t\tif (hasKey(cfg, \"package\")) {\n\t\t\tconst parsedPackage = parsePackageConfig(cfg.package);\n\t\t\tif (parsedPackage.error) return result(parsedPackage.error, true);\n\t\t\tnewPackageName = parsedPackage.packageName;\n\t\t}\n\t\tconst applyError = applyAgentConfig(updated, cfg);\n\t\tif (applyError) return result(applyError, true);\n\t\tconst preserveFrontmatterFields = preservedAgentFrontmatterFields(target, cfg);\n\t\tupdated.localName = newLocalName;\n\t\tupdated.packageName = newPackageName;\n\t\tupdated.name = buildRuntimeName(newLocalName, newPackageName);\n\t\tif (hasKey(cfg, \"description\")) updated.description = (cfg.description as string).trim();\n\t\tif (hasKey(cfg, \"model\")) {\n\t\t\tconst mw = modelWarning(ctx, updated.model);\n\t\t\tif (mw) warnings.push(mw);\n\t\t}\n\t\tif (hasKey(cfg, \"fallbackModels\")) {\n\t\t\tconst fmw = fallbackModelsWarning(ctx, updated.fallbackModels);\n\t\t\tif (fmw) warnings.push(fmw);\n\t\t}\n\t\tif (hasKey(cfg, \"skills\") || hasKey(cfg, \"skillPath\")) {\n\t\t\tconst sw = skillsWarning(ctx.cwd, updated);\n\t\t\tif (sw) warnings.push(sw);\n\t\t}\n\t\tif (updated.name !== oldName) {\n\t\t\tconst renamed = renamePath(\"agent\", target.filePath, updated.name, target.source, ctx.cwd);\n\t\t\tif (renamed.error) return result(renamed.error, true);\n\t\t\tupdated.filePath = renamed.filePath!;\n\t\t}\n\t\tfs.writeFileSync(updated.filePath, serializeAgent(updated, { preserveFrontmatterFields }), \"utf-8\");\n\t\tif (updated.name !== oldName) {\n\t\t\tconst refs = discoverAgentsAll(ctx.cwd).chains.filter((c) => c.steps.some((s) => s.agent === oldName)).map((c) => `${c.name} (${c.source})`);\n\t\t\tif (refs.length) warnings.push(`Warning: chains still reference '${oldName}': ${refs.join(\", \")}.`);\n\t\t}\n\t\tconst headline = updated.name === oldName\n\t\t\t? `Updated agent '${updated.name}' at ${updated.filePath}.`\n\t\t\t: `Updated agent '${oldName}' to '${updated.name}' at ${updated.filePath}.`;\n\t\treturn result([headline, ...warnings].join(\"\\n\"));\n\t}\n\tconst scopeHint = asDisambiguationScope(params.agentScope);\n\tconst targetOrError = resolveTarget(\"chain\", params.chainName!, findChains(params.chainName!, ctx.cwd, scopeHint ?? \"both\"), ctx.cwd, params.agentScope);\n\tif (\"content\" in targetOrError) return targetOrError;\n\tconst target = targetOrError;\n\tconst updated: ChainConfig = { ...target, steps: [...target.steps] };\n\tconst oldName = target.name;\n\tif (hasKey(cfg, \"name\") && (typeof cfg.name !== \"string\" || !cfg.name.trim())) return result(\"config.name must be a non-empty string when provided.\", true);\n\tif (hasKey(cfg, \"description\") && (typeof cfg.description !== \"string\" || !cfg.description.trim())) return result(\"config.description must be a non-empty string when provided.\", true);\n\tlet newLocalName = target.localName ?? frontmatterNameForConfig(target);\n\tif (hasKey(cfg, \"name\")) {\n\t\tnewLocalName = sanitizeName(cfg.name as string);\n\t\tif (!newLocalName) return result(\"config.name is invalid after sanitization.\", true);\n\t}\n\tlet newPackageName = target.packageName;\n\tif (hasKey(cfg, \"package\")) {\n\t\tconst parsedPackage = parsePackageConfig(cfg.package);\n\t\tif (parsedPackage.error) return result(parsedPackage.error, true);\n\t\tnewPackageName = parsedPackage.packageName;\n\t}\n\tlet parsedSteps: ChainStepConfig[] | undefined;\n\tif (hasKey(cfg, \"steps\")) {\n\t\tconst parsed = parseStepList(cfg.steps);\n\t\tif (parsed.error) return result(parsed.error, true);\n\t\tparsedSteps = parsed.steps!;\n\t}\n\tupdated.localName = newLocalName;\n\tupdated.packageName = newPackageName;\n\tupdated.name = buildRuntimeName(newLocalName, newPackageName);\n\tif (hasKey(cfg, \"description\")) updated.description = (cfg.description as string).trim();\n\tif (parsedSteps) {\n\t\tupdated.steps = parsedSteps;\n\t\tconst missing = unknownChainAgents(ctx.cwd, updated.steps);\n\t\tif (missing.length) warnings.push(`Warning: chain steps reference unknown agents: ${missing.join(\", \")}.`);\n\t\twarnings.push(...chainStepWarnings(ctx, updated.steps));\n\t}\n\tif (updated.name !== oldName) {\n\t\tconst renamed = renamePath(\"chain\", target.filePath, updated.name, target.source, ctx.cwd);\n\t\tif (renamed.error) return result(renamed.error, true);\n\t\tupdated.filePath = renamed.filePath!;\n\t}\n\tfs.writeFileSync(updated.filePath, updated.filePath.endsWith(\".chain.json\") ? serializeJsonChain(updated) : serializeChain(updated), \"utf-8\");\n\tconst headline = updated.name === oldName\n\t\t? `Updated chain '${updated.name}' at ${updated.filePath}.`\n\t\t: `Updated chain '${oldName}' to '${updated.name}' at ${updated.filePath}.`;\n\treturn result([headline, ...warnings].join(\"\\n\"));\n}\n\nfunction handleDelete(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent && !params.chainName) return result(\"Specify 'agent' or 'chainName' for delete.\", true);\n\tif (params.agent && params.chainName) return result(\"Specify either 'agent' or 'chainName', not both.\", true);\n\tconst scopeHint = asDisambiguationScope(params.agentScope);\n\tif (params.agent) {\n\t\tconst targetOrError = resolveTarget(\"agent\", params.agent, findAgents(params.agent, ctx.cwd, scopeHint ?? \"both\"), ctx.cwd, params.agentScope);\n\t\tif (\"content\" in targetOrError) return targetOrError;\n\t\tconst target = targetOrError;\n\t\tfs.unlinkSync(target.filePath);\n\t\tconst refs = discoverAgentsAll(ctx.cwd).chains.filter((c) => c.steps.some((s) => s.agent === target.name)).map((c) => `${c.name} (${c.source})`);\n\t\tconst lines = [`Deleted agent '${target.name}' at ${target.filePath}.`];\n\t\tif (refs.length) lines.push(`Warning: chains reference deleted agent '${target.name}': ${refs.join(\", \")}.`);\n\t\treturn result(lines.join(\"\\n\"));\n\t}\n\tconst targetOrError = resolveTarget(\"chain\", params.chainName!, findChains(params.chainName!, ctx.cwd, scopeHint ?? \"both\"), ctx.cwd, params.agentScope);\n\tif (\"content\" in targetOrError) return targetOrError;\n\tconst target = targetOrError;\n\tfs.unlinkSync(target.filePath);\n\treturn result(`Deleted chain '${target.name}' at ${target.filePath}.`);\n}\n\nfunction handleEject(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent) return result(\"Specify 'agent' for eject.\", true);\n\tconst raw = params.agent.trim();\n\tconst sanitized = sanitizeName(raw);\n\tconst parsedScope = actionScope(params.agentScope, \"eject\");\n\tif (parsedScope.error) return parsedScope.error;\n\tconst scope = parsedScope.scope!;\n\tconst d = discoverAgentsAll(ctx.cwd);\n\tconst source = [...d.package, ...d.builtin].find((a) => a.name === raw || a.name === sanitized);\n\tif (!source) {\n\t\treturn 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: ${availableNames(ctx.cwd, \"agent\").join(\", \") || \"none\"}.`, true);\n\t}\n\tconst runtimeName = source.name;\n\tconst existingCustom = (scope === \"user\" ? d.user : d.project).find((a) => a.name === runtimeName);\n\tif (existingCustom) {\n\t\treturn result(`Agent '${runtimeName}' is already a custom ${scope} agent at ${existingCustom.filePath}. Edit it with { action: \"update\", agent: \"${runtimeName}\" } or delete it first.`, true);\n\t}\n\tif (nameExistsInScope(ctx.cwd, scope, runtimeName)) {\n\t\treturn result(`An agent or chain named '${runtimeName}' already exists in ${scope} scope. Remove or rename it first.`, true);\n\t}\n\tconst projectConfigDir = getProjectConfigDir(ctx.cwd);\n\tconst targetDir = scope === \"user\" ? d.userDir : d.projectDir ?? path.join(projectConfigDir, \"agents\");\n\tfs.mkdirSync(targetDir, { recursive: true });\n\tconst targetPath = path.join(targetDir, `${runtimeName}.md`);\n\tif (fs.existsSync(targetPath)) {\n\t\treturn result(`File already exists at ${targetPath} but is not a valid agent definition. Remove or rename it first.`, true);\n\t}\n\tlet content: string;\n\ttry {\n\t\tcontent = fs.readFileSync(source.filePath, \"utf-8\");\n\t} catch (error) {\n\t\tconst message = error instanceof Error ? error.message : String(error);\n\t\treturn result(`Failed to read source agent at ${source.filePath}: ${message}`, true);\n\t}\n\tfs.writeFileSync(targetPath, content, \"utf-8\");\n\treturn 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.`);\n}\n\nfunction handleDisable(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent) return result(\"Specify 'agent' for disable.\", true);\n\tconst raw = params.agent.trim();\n\tconst parsedScope = actionScope(params.agentScope, \"disable\");\n\tif (parsedScope.error) return parsedScope.error;\n\tconst scope = parsedScope.scope!;\n\tconst d = discoverAgentsAll(ctx.cwd);\n\tif (scope === \"project\" && d.projectSettingsPath === null) {\n\t\treturn 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);\n\t}\n\tconst effective = pickEffectiveAgent(d, raw);\n\tif (!effective) {\n\t\treturn result(`Agent '${raw}' not found. Available: ${availableNames(ctx.cwd, \"agent\").join(\", \") || \"none\"}.`, true);\n\t}\n\tconst runtimeName = effective.name;\n\tconst settingsPath = mergeBuiltinAgentOverride(ctx.cwd, runtimeName, scope, { disabled: true });\n\tconst after = pickEffectiveAgent(discoverAgentsAll(ctx.cwd), raw);\n\tif (after?.disabled === true) {\n\t\treturn result(`Disabled agent '${runtimeName}' via ${scope} settings override at ${settingsPath}. It is now hidden from runtime discovery and { action: \"list\" }.`);\n\t}\n\treturn 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);\n}\n\nfunction handleEnable(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent) return result(\"Specify 'agent' for enable.\", true);\n\tconst raw = params.agent.trim();\n\tconst parsedScope = actionScope(params.agentScope, \"enable\");\n\tif (parsedScope.error) return parsedScope.error;\n\tconst scope = parsedScope.scope!;\n\tconst d = discoverAgentsAll(ctx.cwd);\n\tif (scope === \"project\" && d.projectSettingsPath === null) {\n\t\treturn 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);\n\t}\n\tconst effective = pickEffectiveAgent(d, raw);\n\tif (!effective) {\n\t\treturn result(`Agent '${raw}' not found. Available: ${availableNames(ctx.cwd, \"agent\").join(\", \") || \"none\"}.`, true);\n\t}\n\tconst runtimeName = effective.name;\n\tconst { path: settingsPath, removed } = removeBuiltinAgentOverrideFields(ctx.cwd, runtimeName, scope, [\"disabled\"]);\n\tconst after = pickEffectiveAgent(discoverAgentsAll(ctx.cwd), raw);\n\tif (after && after.disabled !== true) {\n\t\tif (removed) return result(`Enabled agent '${runtimeName}' (removed disabled override at ${settingsPath}).`);\n\t\treturn result(`Agent '${runtimeName}' is already enabled.`);\n\t}\n\tif (after?.override?.scope && after.override.scope !== scope) {\n\t\treturn 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);\n\t}\n\treturn 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);\n}\n\nfunction handleReset(params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tif (!params.agent) return result(\"Specify 'agent' for reset.\", true);\n\tconst raw = params.agent.trim();\n\tconst sanitized = sanitizeName(raw);\n\tconst parsedScope = actionScope(params.agentScope, \"reset\");\n\tif (parsedScope.error) return parsedScope.error;\n\tconst scope = parsedScope.scope!;\n\tconst d = discoverAgentsAll(ctx.cwd);\n\tif (scope === \"project\" && d.projectSettingsPath === null) {\n\t\treturn 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);\n\t}\n\tconst bundled = [...d.package, ...d.builtin].find((a) => a.name === raw || a.name === sanitized);\n\tif (!bundled) {\n\t\tconst custom = [...d.user, ...d.project].find((a) => a.name === raw || a.name === sanitized);\n\t\tif (custom) {\n\t\t\treturn result(`Agent '${raw}' has no bundled default to reset to. Use { action: \"delete\", agent: \"${custom.name}\" } to remove the custom ${custom.source} agent.`, true);\n\t\t}\n\t\treturn result(`Agent '${raw}' not found. Available: ${availableNames(ctx.cwd, \"agent\").join(\", \") || \"none\"}.`, true);\n\t}\n\tconst runtimeName = bundled.name;\n\tconst custom = (scope === \"user\" ? d.user : d.project).find((a) => a.name === raw || a.name === sanitized);\n\tconst lines: string[] = [];\n\tif (custom) {\n\t\tfs.unlinkSync(custom.filePath);\n\t\tlines.push(`Deleted custom ${scope} agent file at ${custom.filePath}.`);\n\t}\n\tconst overrideRemoval = removeBuiltinAgentOverride(ctx.cwd, runtimeName, scope);\n\tif (overrideRemoval.removed) lines.push(`Removed ${scope} settings override at ${overrideRemoval.path}.`);\n\tif (lines.length === 0) {\n\t\tconst otherScope = scope === \"user\" ? \"project\" : \"user\";\n\t\tconst otherCustom = (otherScope === \"user\" ? d.user : d.project).find((a) => a.name === raw || a.name === sanitized);\n\t\tconst hasOtherOverride = bundled.override?.scope === otherScope;\n\t\tconst note = (otherCustom || hasOtherOverride)\n\t\t\t? ` Customization exists in ${otherScope} scope; specify agentScope: '${otherScope}' to reset it.`\n\t\t\t: \"\";\n\t\treturn result(`Agent '${runtimeName}' has no ${scope} customization to reset.${note} It is at its bundled ${bundled.source} default.`);\n\t}\n\tlines.push(`Reset agent '${runtimeName}' to its bundled ${bundled.source} default.`);\n\treturn result(lines.join(\"\\n\"));\n}\n\nexport function handleManagementAction(action: string, params: ManagementParams, ctx: ManagementContext): AgentToolResult<Details> {\n\tswitch (action as ManagementAction) {\n\t\tcase \"list\": return handleList(params, ctx);\n\t\tcase \"get\": return handleGet(params, ctx);\n\t\tcase \"models\": return handleModels(params, ctx);\n\t\tcase \"create\": return handleCreate(params, ctx);\n\t\tcase \"update\": return handleUpdate(params, ctx);\n\t\tcase \"delete\": return handleDelete(params, ctx);\n\t\tcase \"eject\": return handleEject(params, ctx);\n\t\tcase \"disable\": return handleDisable(params, ctx);\n\t\tcase \"enable\": return handleEnable(params, ctx);\n\t\tcase \"reset\": return handleReset(params, ctx);\n\t\tdefault: return result(`Unknown action: ${action}`, true);\n\t}\n}\n"]}