pi-crew 0.9.16 → 0.9.18

package/src/extension/management.ts

@@ -1,11 +1,7 @@
 import * as crypto from "node:crypto";
 import * as fs from "node:fs";
 import * as path from "node:path";
-import type {
-	AgentConfig,
-	ResourceSource,
-	RoutingMetadata,
-} from "../agents/agent-config.ts";
+import type { AgentConfig, ResourceSource, RoutingMetadata } from "../agents/agent-config.ts";
 import { serializeAgent } from "../agents/agent-serializer.ts";
 import { discoverAgents } from "../agents/discover-agents.ts";
 import type { PiTeamsConfig } from "../config/config.ts";
@@ -13,21 +9,10 @@ import type { TeamToolParamsValue } from "../schema/team-tool-schema.ts";
 import { allTeams, discoverTeams } from "../teams/discover-teams.ts";
 import type { TeamConfig, TeamRole } from "../teams/team-config.ts";
 import { serializeTeam } from "../teams/team-serializer.ts";
-import {
-	hasOwn,
-	parseConfigObject,
-	requireString,
-	sanitizeName,
-} from "../utils/names.ts";
+import { hasOwn, parseConfigObject, requireString, sanitizeName } from "../utils/names.ts";
 import { projectCrewRoot, userPiRoot } from "../utils/paths.ts";
-import {
-	allWorkflows,
-	discoverWorkflows,
-} from "../workflows/discover-workflows.ts";
-import type {
-	WorkflowConfig,
-	WorkflowStep,
-} from "../workflows/workflow-config.ts";
+import { allWorkflows, discoverWorkflows } from "../workflows/discover-workflows.ts";
+import type { WorkflowConfig, WorkflowStep } from "../workflows/workflow-config.ts";
 import { serializeWorkflow } from "../workflows/workflow-serializer.ts";
 import { enforceDestructiveIntent } from "./team-tool/intent-policy.ts";
 import type { TeamToolDetails } from "./team-tool-types.ts";
@@ -42,19 +27,11 @@ type MutableSource = "user" | "project";
 
 type MutableResource = AgentConfig | TeamConfig | WorkflowConfig;
 
-function result(
-	text: string,
-	status: TeamToolDetails["status"] = "ok",
-	isError = false,
-): PiTeamsToolResult {
+function result(text: string, status: TeamToolDetails["status"] = "ok", isError = false): PiTeamsToolResult {
 	return toolResult(text, { action: "management", status }, isError);
 }
 
-function scopeDir(
-	ctx: ManagementContext,
-	resource: "agent" | "team" | "workflow",
-	scope: MutableSource,
-): string {
+function scopeDir(ctx: ManagementContext, resource: "agent" | "team" | "workflow", scope: MutableSource): string {
 	const base = scope === "user" ? userPiRoot() : projectCrewRoot(ctx.cwd);
 	if (resource === "agent") return path.join(base, "agents");
 	if (resource === "team") return path.join(base, "teams");
@@ -77,16 +54,8 @@ function backupFile(filePath: string): string {
 	return backupPath;
 }
 
-function targetPath(
-	ctx: ManagementContext,
-	resource: "agent" | "team" | "workflow",
-	scope: MutableSource,
-	name: string,
-): string {
-	return path.join(
-		scopeDir(ctx, resource, scope),
-		`${name}${extensionFor(resource)}`,
-	);
+function targetPath(ctx: ManagementContext, resource: "agent" | "team" | "workflow", scope: MutableSource, name: string): string {
+	return path.join(scopeDir(ctx, resource, scope), `${name}${extensionFor(resource)}`);
 }
 
 function parseStringArray(value: unknown): string[] | undefined {
@@ -96,58 +65,31 @@ function parseStringArray(value: unknown): string[] | undefined {
 			.map((entry) => entry.trim())
 			.filter(Boolean);
 	if (Array.isArray(value))
-		return value
-			.filter(
-				(entry): entry is string =>
-					typeof entry === "string" && entry.trim().length > 0,
-			)
-			.map((entry) => entry.trim());
+		return value.filter((entry): entry is string => typeof entry === "string" && entry.trim().length > 0).map((entry) => entry.trim());
 	return undefined;
 }
 
-function parseRouting(
-	value: Record<string, unknown>,
-	fallback?: RoutingMetadata,
-): RoutingMetadata | undefined {
+function parseRouting(value: Record<string, unknown>, fallback?: RoutingMetadata): RoutingMetadata | undefined {
 	const routing = {
-		triggers: hasOwn(value, "triggers")
-			? parseStringArray(value.triggers)
-			: fallback?.triggers,
-		useWhen: hasOwn(value, "useWhen")
-			? parseStringArray(value.useWhen)
-			: fallback?.useWhen,
-		avoidWhen: hasOwn(value, "avoidWhen")
-			? parseStringArray(value.avoidWhen)
-			: fallback?.avoidWhen,
-		cost:
-			value.cost === "free" ||
-			value.cost === "cheap" ||
-			value.cost === "expensive"
-				? value.cost
-				: fallback?.cost,
+		triggers: hasOwn(value, "triggers") ? parseStringArray(value.triggers) : fallback?.triggers,
+		useWhen: hasOwn(value, "useWhen") ? parseStringArray(value.useWhen) : fallback?.useWhen,
+		avoidWhen: hasOwn(value, "avoidWhen") ? parseStringArray(value.avoidWhen) : fallback?.avoidWhen,
+		cost: value.cost === "free" || value.cost === "cheap" || value.cost === "expensive" ? value.cost : fallback?.cost,
 		category: hasOwn(value, "category")
 			? typeof value.category === "string" && value.category.trim()
 				? value.category.trim()
 				: undefined
 			: fallback?.category,
 	};
-	return routing.triggers ||
-		routing.useWhen ||
-		routing.avoidWhen ||
-		routing.cost ||
-		routing.category
-		? routing
-		: undefined;
+	return routing.triggers || routing.useWhen || routing.avoidWhen || routing.cost || routing.category ? routing : undefined;
 }
 
 function parseRoles(value: unknown): { roles?: TeamRole[]; error?: string } {
-	if (!Array.isArray(value) || value.length === 0)
-		return { error: "config.roles must be a non-empty array." };
+	if (!Array.isArray(value) || value.length === 0) return { error: "config.roles must be a non-empty array." };
 	const roles: TeamRole[] = [];
 	for (let i = 0; i < value.length; i++) {
 		const item = value[i];
-		if (!item || typeof item !== "object" || Array.isArray(item))
-			return { error: `config.roles[${i}] must be an object.` };
+		if (!item || typeof item !== "object" || Array.isArray(item)) return { error: `config.roles[${i}] must be an object.` };
 		const obj = item as Record<string, unknown>;
 		const name = requireString(obj.name, `config.roles[${i}].name`);
 		if (name.error) return { error: name.error };
@@ -156,15 +98,10 @@ function parseRoles(value: unknown): { roles?: TeamRole[]; error?: string } {
 		roles.push({
 			name: sanitizeName(name.value!),
 			agent: sanitizeName(agent.value!),
-			description:
-				typeof obj.description === "string"
-					? obj.description.trim()
-					: undefined,
+			description: typeof obj.description === "string" ? obj.description.trim() : undefined,
 			model: typeof obj.model === "string" ? obj.model.trim() : undefined,
 			maxConcurrency:
-				typeof obj.maxConcurrency === "number" &&
-				Number.isInteger(obj.maxConcurrency) &&
-				obj.maxConcurrency > 0
+				typeof obj.maxConcurrency === "number" && Number.isInteger(obj.maxConcurrency) && obj.maxConcurrency > 0
 					? obj.maxConcurrency
 					: undefined,
 		});
@@ -176,13 +113,11 @@ function parseSteps(value: unknown): {
 	steps?: WorkflowStep[];
 	error?: string;
 } {
-	if (!Array.isArray(value) || value.length === 0)
-		return { error: "config.steps must be a non-empty array." };
+	if (!Array.isArray(value) || value.length === 0) return { error: "config.steps must be a non-empty array." };
 	const steps: WorkflowStep[] = [];
 	for (let i = 0; i < value.length; i++) {
 		const item = value[i];
-		if (!item || typeof item !== "object" || Array.isArray(item))
-			return { error: `config.steps[${i}] must be an object.` };
+		if (!item || typeof item !== "object" || Array.isArray(item)) return { error: `config.steps[${i}] must be an object.` };
 		const obj = item as Record<string, unknown>;
 		const id = requireString(obj.id, `config.steps[${i}].id`);
 		if (id.error) return { error: id.error };
@@ -205,23 +140,13 @@ function parseSteps(value: unknown): {
 			role: sanitizeName(role.value!),
 			task,
 			dependsOn: parseStringArray(obj.dependsOn),
-			parallelGroup:
-				typeof obj.parallelGroup === "string"
-					? obj.parallelGroup.trim()
-					: undefined,
-			output:
-				obj.output === false
-					? false
-					: typeof obj.output === "string"
-						? obj.output.trim()
-						: undefined,
+			parallelGroup: typeof obj.parallelGroup === "string" ? obj.parallelGroup.trim() : undefined,
+			output: obj.output === false ? false : typeof obj.output === "string" ? obj.output.trim() : undefined,
 			reads: obj.reads === false ? false : parseStringArray(obj.reads),
 			model: typeof obj.model === "string" ? obj.model.trim() : undefined,
 			skills: obj.skills === false ? false : parseStringArray(obj.skills),
-			progress:
-				typeof obj.progress === "boolean" ? obj.progress : undefined,
-			worktree:
-				typeof obj.worktree === "boolean" ? obj.worktree : undefined,
+			progress: typeof obj.progress === "boolean" ? obj.progress : undefined,
+			worktree: typeof obj.worktree === "boolean" ? obj.worktree : undefined,
 			verify: typeof obj.verify === "boolean" ? obj.verify : undefined,
 		});
 	}
@@ -229,75 +154,49 @@ function parseSteps(value: unknown): {
 }
 
 function parseWorkflowMaxConcurrency(value: unknown): number | undefined {
-	if (typeof value !== "number" || !Number.isInteger(value) || value < 1)
-		return undefined;
+	if (typeof value !== "number" || !Number.isInteger(value) || value < 1) return undefined;
 	return value;
 }
 
-function findResource(
-	ctx: ManagementContext,
-	resource: "agent" | "team" | "workflow",
-	name: string,
-	scope?: string,
-): MutableResource[] {
+function findResource(ctx: ManagementContext, resource: "agent" | "team" | "workflow", name: string, scope?: string): MutableResource[] {
 	const normalized = sanitizeName(name);
 	const sourceMatches = (item: { name: string; source: ResourceSource }) =>
-		(scope === "user" || scope === "project"
-			? item.source === scope
-			: item.source !== "builtin") && item.name === normalized;
+		(scope === "user" || scope === "project" ? item.source === scope : item.source !== "builtin") && item.name === normalized;
 	// Search in the correct scope array directly to avoid allAgents shadowing issue.
 	if (resource === "agent") {
 		const discovery = discoverAgents(ctx.cwd);
 		const pool =
-			scope === "user"
-				? discovery.user
-				: scope === "project"
-					? discovery.project
-					: [...discovery.builtin, ...discovery.user];
+			scope === "user" ? discovery.user : scope === "project" ? discovery.project : [...discovery.builtin, ...discovery.user];
 		return pool.filter(sourceMatches);
 	}
 	if (resource === "team") {
 		const discovery = discoverTeams(ctx.cwd);
 		const pool =
-			scope === "user"
-				? discovery.user
-				: scope === "project"
-					? discovery.project
-					: [...discovery.builtin, ...discovery.user];
+			scope === "user" ? discovery.user : scope === "project" ? discovery.project : [...discovery.builtin, ...discovery.user];
 		return pool.filter(sourceMatches);
 	}
 	{
 		const discovery = discoverWorkflows(ctx.cwd);
 		const pool =
-			scope === "user"
-				? discovery.user
-				: scope === "project"
-					? discovery.project
-					: [...discovery.builtin, ...discovery.user];
+			scope === "user" ? discovery.user : scope === "project" ? discovery.project : [...discovery.builtin, ...discovery.user];
 		return pool.filter(sourceMatches);
 	}
 }
 
 // Note: only checks agent→team references and defaultWorkflow. Does not detect
 // workflow-step→agent/team references or team name in workflow metadata.
-function findReferences(
-	ctx: ManagementContext,
-	resource: "agent" | "team" | "workflow",
-	name: string,
-): string[] {
+function findReferences(ctx: ManagementContext, resource: "agent" | "team" | "workflow", name: string): string[] {
 	const refs: string[] = [];
 	if (resource === "agent") {
 		for (const team of allTeams(discoverTeams(ctx.cwd))) {
 			for (const role of team.roles) {
-				if (role.agent === name)
-					refs.push(`team '${team.name}' role '${role.name}'`);
+				if (role.agent === name) refs.push(`team '${team.name}' role '${role.name}'`);
 			}
 		}
 	}
 	if (resource === "workflow") {
 		for (const team of allTeams(discoverTeams(ctx.cwd))) {
-			if (team.defaultWorkflow === name)
-				refs.push(`team '${team.name}' defaultWorkflow`);
+			if (team.defaultWorkflow === name) refs.push(`team '${team.name}' defaultWorkflow`);
 		}
 	}
 	return refs;
@@ -332,18 +231,12 @@ function updateReferencesForRename(
 	if (oldName === newName) return [];
 	if (resource !== "agent" && resource !== "workflow") return [];
 	const changed: string[] = [];
-	for (const team of allTeams(discoverTeams(ctx.cwd)).filter(
-		(candidate) => candidate.source === scope,
-	)) {
+	for (const team of allTeams(discoverTeams(ctx.cwd)).filter((candidate) => candidate.source === scope)) {
 		let updated = false;
 		let nextTeam = team;
 		if (resource === "agent") {
-			const roles = team.roles.map((role) =>
-				role.agent === oldName ? { ...role, agent: newName } : role,
-			);
-			updated = roles.some(
-				(role, index) => role.agent !== team.roles[index]!.agent,
-			);
+			const roles = team.roles.map((role) => (role.agent === oldName ? { ...role, agent: newName } : role));
+			updated = roles.some((role, index) => role.agent !== team.roles[index]!.agent);
 			nextTeam = { ...team, roles };
 		}
 		if (resource === "workflow" && team.defaultWorkflow === oldName) {
@@ -359,9 +252,7 @@ function updateReferencesForRename(
 	}
 	// L12 fix: also update workflow step role references when renaming agents.
 	// Workflow files use `role:` to reference agent roles, not agent names.
-	for (const workflow of allWorkflows(discoverWorkflows(ctx.cwd)).filter(
-		(w) => w.source === scope,
-	)) {
+	for (const workflow of allWorkflows(discoverWorkflows(ctx.cwd)).filter((w) => w.source === scope)) {
 		let updated = false;
 		const newSteps = workflow.steps.map((step) => {
 			if (step.role === oldName) {
@@ -374,28 +265,16 @@ function updateReferencesForRename(
 		changed.push(workflow.filePath);
 		if (!dryRun) {
 			backupFile(workflow.filePath);
-			fs.writeFileSync(
-				workflow.filePath,
-				serializeWorkflow({ ...workflow, steps: newSteps }),
-				"utf-8",
-			);
+			fs.writeFileSync(workflow.filePath, serializeWorkflow({ ...workflow, steps: newSteps }), "utf-8");
 		}
 	}
 	// L12 fix: update agent references in test fixtures.
-	const testDir =
-		scope === "user"
-			? path.join(projectCrewRoot(ctx.cwd), "test")
-			: path.join(ctx.cwd, "test", "fixtures");
+	const testDir = scope === "user" ? path.join(projectCrewRoot(ctx.cwd), "test") : path.join(ctx.cwd, "test", "fixtures");
 	if (fs.existsSync(testDir)) {
 		for (const fixture of walkTsFiles(testDir)) {
 			const content = fs.readFileSync(fixture, "utf-8");
 			if (!content.includes(oldName)) continue;
-			const agentPattern = new RegExp(
-				"([\"'\\`]agent[=\"':\\s]*)" +
-					escapeRegex(oldName) +
-					"([\"'\\`]|\\s)",
-				"g",
-			);
+			const agentPattern = new RegExp("([\"'\\`]agent[=\"':\\s]*)" + escapeRegex(oldName) + "([\"'\\`]|\\s)", "g");
 			const newContent = content.replace(agentPattern, `$1${newName}$2`);
 			if (newContent !== content) {
 				changed.push(fixture);
@@ -408,80 +287,42 @@ function updateReferencesForRename(
 	return changed;
 }
 
-function resolveMutable(
-	ctx: ManagementContext,
-	params: TeamToolParamsValue,
-): { resource?: MutableResource; error?: PiTeamsToolResult } {
+function resolveMutable(ctx: ManagementContext, params: TeamToolParamsValue): { resource?: MutableResource; error?: PiTeamsToolResult } {
 	if (!params.resource)
 		return {
-			error: result(
-				"resource is required for update/delete.",
-				"error",
-				true,
-			),
+			error: result("resource is required for update/delete.", "error", true),
 		};
-	const name =
-		params.resource === "agent"
-			? params.agent
-			: params.resource === "team"
-				? params.team
-				: params.workflow;
+	const name = params.resource === "agent" ? params.agent : params.resource === "team" ? params.team : params.workflow;
 	if (!name)
 		return {
-			error: result(
-				`${params.resource} name is required.`,
-				"error",
-				true,
-			),
+			error: result(`${params.resource} name is required.`, "error", true),
 		};
 	const matches = findResource(ctx, params.resource, name, params.scope);
 	if (matches.length === 0)
 		return {
-			error: result(
-				`${params.resource} '${name}' not found in mutable user/project scopes.`,
-				"error",
-				true,
-			),
+			error: result(`${params.resource} '${name}' not found in mutable user/project scopes.`, "error", true),
 		};
 	if (matches.length > 1)
 		return {
-			error: result(
-				`${params.resource} '${name}' exists in multiple scopes. Specify scope: 'user' or 'project'.`,
-				"error",
-				true,
-			),
+			error: result(`${params.resource} '${name}' exists in multiple scopes. Specify scope: 'user' or 'project'.`, "error", true),
 		};
 	return { resource: matches[0] };
 }
 
-export function handleCreate(
-	params: TeamToolParamsValue,
-	ctx: ManagementContext,
-): PiTeamsToolResult {
-	if (!params.resource)
-		return result("resource is required for create.", "error", true);
+export function handleCreate(params: TeamToolParamsValue, ctx: ManagementContext): PiTeamsToolResult {
+	if (!params.resource) return result("resource is required for create.", "error", true);
 	const parsed = parseConfigObject(params.config);
 	if (parsed.error) return result(parsed.error, "error", true);
 	const cfg = parsed.value!;
 	const nameValue = requireString(cfg.name, "config.name");
 	if (nameValue.error) return result(nameValue.error, "error", true);
-	const descriptionValue = requireString(
-		cfg.description,
-		"config.description",
-	);
-	if (descriptionValue.error)
-		return result(descriptionValue.error, "error", true);
+	const descriptionValue = requireString(cfg.description, "config.description");
+	if (descriptionValue.error) return result(descriptionValue.error, "error", true);
 	const name = sanitizeName(nameValue.value!);
-	if (!name)
-		return result(
-			"config.name is invalid after sanitization.",
-			"error",
-			true,
-		);
+	if (!name) return result("config.name is invalid after sanitization.", "error", true);
 	const scope = cfg.scope === "project" ? "project" : "user";
 	const filePath = targetPath(ctx, params.resource, scope, name);
-	if (fs.existsSync(filePath))
-		return result(`File already exists: ${filePath}`, "error", true);
+	if (fs.existsSync(filePath)) return result(`File already exists: ${filePath}`, "error", true);
 	fs.mkdirSync(path.dirname(filePath), { recursive: true });
 
 	let content: string;
@@ -491,19 +332,14 @@ export function handleCreate(
 			description: descriptionValue.value!,
 			source: scope,
 			filePath,
-			systemPrompt:
-				typeof cfg.systemPrompt === "string" ? cfg.systemPrompt : "",
+			systemPrompt: typeof cfg.systemPrompt === "string" ? cfg.systemPrompt : "",
 			model: typeof cfg.model === "string" ? cfg.model : undefined,
 			fallbackModels: parseStringArray(cfg.fallbackModels),
-			thinking:
-				typeof cfg.thinking === "string" ? cfg.thinking : undefined,
+			thinking: typeof cfg.thinking === "string" ? cfg.thinking : undefined,
 			tools: parseStringArray(cfg.tools),
-			extensions: hasOwn(cfg, "extensions")
-				? (parseStringArray(cfg.extensions) ?? [])
-				: undefined,
+			extensions: hasOwn(cfg, "extensions") ? (parseStringArray(cfg.extensions) ?? []) : undefined,
 			skills: parseStringArray(cfg.skills),
-			systemPromptMode:
-				cfg.systemPromptMode === "append" ? "append" : "replace",
+			systemPromptMode: cfg.systemPromptMode === "append" ? "append" : "replace",
 			inheritProjectContext: cfg.inheritProjectContext === true,
 			inheritSkills: cfg.inheritSkills === true,
 			routing: parseRouting(cfg),
@@ -518,16 +354,10 @@ export function handleCreate(
 			source: scope,
 			filePath,
 			roles: parsedRoles.roles!,
-			defaultWorkflow:
-				typeof cfg.defaultWorkflow === "string"
-					? sanitizeName(cfg.defaultWorkflow)
-					: undefined,
-			workspaceMode:
-				cfg.workspaceMode === "worktree" ? "worktree" : "single",
+			defaultWorkflow: typeof cfg.defaultWorkflow === "string" ? sanitizeName(cfg.defaultWorkflow) : undefined,
+			workspaceMode: cfg.workspaceMode === "worktree" ? "worktree" : "single",
 			maxConcurrency:
-				typeof cfg.maxConcurrency === "number" &&
-				Number.isInteger(cfg.maxConcurrency) &&
-				cfg.maxConcurrency > 0
+				typeof cfg.maxConcurrency === "number" && Number.isInteger(cfg.maxConcurrency) && cfg.maxConcurrency > 0
 					? cfg.maxConcurrency
 					: undefined,
 			routing: parseRouting(cfg),
@@ -545,10 +375,7 @@ export function handleCreate(
 		});
 	}
 
-	if (params.dryRun)
-		return result(
-			`[dry-run] Would create ${params.resource} '${name}' at ${filePath}:\n\n${content}`,
-		);
+	if (params.dryRun) return result(`[dry-run] Would create ${params.resource} '${name}' at ${filePath}:\n\n${content}`);
 	try {
 		fs.writeFileSync(filePath, content, "utf-8");
 	} catch (writeError) {
@@ -561,29 +388,18 @@ export function handleCreate(
 	return result(`Created ${params.resource} '${name}' at ${filePath}.`);
 }
 
-export function handleUpdate(
-	params: TeamToolParamsValue,
-	ctx: ManagementContext,
-): PiTeamsToolResult {
+export function handleUpdate(params: TeamToolParamsValue, ctx: ManagementContext): PiTeamsToolResult {
 	const resolved = resolveMutable(ctx, params);
 	if (resolved.error) return resolved.error;
 	const parsed = parseConfigObject(params.config);
 	if (parsed.error) return result(parsed.error, "error", true);
 	const cfg = parsed.value!;
 	const current = resolved.resource!;
-	const nextName = hasOwn(cfg, "name")
-		? sanitizeName(String(cfg.name ?? ""))
-		: current.name;
-	if (!nextName)
-		return result(
-			"config.name is invalid after sanitization.",
-			"error",
-			true,
-		);
+	const nextName = hasOwn(cfg, "name") ? sanitizeName(String(cfg.name ?? "")) : current.name;
+	if (!nextName) return result("config.name is invalid after sanitization.", "error", true);
 	const source = current.source === "project" ? "project" : "user";
 	const nextPath = targetPath(ctx, params.resource!, source, nextName);
-	if (nextPath !== current.filePath && fs.existsSync(nextPath))
-		return result(`Target file already exists: ${nextPath}`, "error", true);
+	if (nextPath !== current.filePath && fs.existsSync(nextPath)) return result(`Target file already exists: ${nextPath}`, "error", true);
 
 	let content: string;
 	if (params.resource === "agent") {
@@ -592,50 +408,22 @@ export function handleUpdate(
 			...agent,
 			name: nextName,
 			filePath: nextPath,
-			description:
-				typeof cfg.description === "string" && cfg.description.trim()
-					? cfg.description.trim()
-					: agent.description,
-			systemPrompt:
-				typeof cfg.systemPrompt === "string"
-					? cfg.systemPrompt
-					: agent.systemPrompt,
-			model: hasOwn(cfg, "model")
-				? typeof cfg.model === "string" && cfg.model.trim()
-					? cfg.model.trim()
-					: undefined
-				: agent.model,
-			fallbackModels: hasOwn(cfg, "fallbackModels")
-				? parseStringArray(cfg.fallbackModels)
-				: agent.fallbackModels,
+			description: typeof cfg.description === "string" && cfg.description.trim() ? cfg.description.trim() : agent.description,
+			systemPrompt: typeof cfg.systemPrompt === "string" ? cfg.systemPrompt : agent.systemPrompt,
+			model: hasOwn(cfg, "model") ? (typeof cfg.model === "string" && cfg.model.trim() ? cfg.model.trim() : undefined) : agent.model,
+			fallbackModels: hasOwn(cfg, "fallbackModels") ? parseStringArray(cfg.fallbackModels) : agent.fallbackModels,
 			thinking: hasOwn(cfg, "thinking")
 				? typeof cfg.thinking === "string" && cfg.thinking.trim()
 					? cfg.thinking.trim()
 					: undefined
 				: agent.thinking,
-			tools: hasOwn(cfg, "tools")
-				? parseStringArray(cfg.tools)
-				: agent.tools,
-			extensions: hasOwn(cfg, "extensions")
-				? (parseStringArray(cfg.extensions) ?? [])
-				: agent.extensions,
-			skills: hasOwn(cfg, "skills")
-				? parseStringArray(cfg.skills)
-				: agent.skills,
+			tools: hasOwn(cfg, "tools") ? parseStringArray(cfg.tools) : agent.tools,
+			extensions: hasOwn(cfg, "extensions") ? (parseStringArray(cfg.extensions) ?? []) : agent.extensions,
+			skills: hasOwn(cfg, "skills") ? parseStringArray(cfg.skills) : agent.skills,
 			systemPromptMode:
-				cfg.systemPromptMode === "append"
-					? "append"
-					: cfg.systemPromptMode === "replace"
-						? "replace"
-						: agent.systemPromptMode,
-			inheritProjectContext:
-				typeof cfg.inheritProjectContext === "boolean"
-					? cfg.inheritProjectContext
-					: agent.inheritProjectContext,
-			inheritSkills:
-				typeof cfg.inheritSkills === "boolean"
-					? cfg.inheritSkills
-					: agent.inheritSkills,
+				cfg.systemPromptMode === "append" ? "append" : cfg.systemPromptMode === "replace" ? "replace" : agent.systemPromptMode,
+			inheritProjectContext: typeof cfg.inheritProjectContext === "boolean" ? cfg.inheritProjectContext : agent.inheritProjectContext,
+			inheritSkills: typeof cfg.inheritSkills === "boolean" ? cfg.inheritSkills : agent.inheritSkills,
 			routing: parseRouting(cfg, agent.routing),
 		});
 	} else if (params.resource === "team") {
@@ -643,34 +431,23 @@ export function handleUpdate(
 		let roles = team.roles;
 		if (hasOwn(cfg, "roles")) {
 			const parsedRoles = parseRoles(cfg.roles);
-			if (parsedRoles.error)
-				return result(parsedRoles.error, "error", true);
+			if (parsedRoles.error) return result(parsedRoles.error, "error", true);
 			roles = parsedRoles.roles!;
 		}
 		content = serializeTeam({
 			...team,
 			name: nextName,
 			filePath: nextPath,
-			description:
-				typeof cfg.description === "string" && cfg.description.trim()
-					? cfg.description.trim()
-					: team.description,
+			description: typeof cfg.description === "string" && cfg.description.trim() ? cfg.description.trim() : team.description,
 			roles,
 			defaultWorkflow: hasOwn(cfg, "defaultWorkflow")
 				? typeof cfg.defaultWorkflow === "string"
 					? sanitizeName(cfg.defaultWorkflow)
 					: undefined
 				: team.defaultWorkflow,
-			workspaceMode:
-				cfg.workspaceMode === "worktree"
-					? "worktree"
-					: cfg.workspaceMode === "single"
-						? "single"
-						: team.workspaceMode,
+			workspaceMode: cfg.workspaceMode === "worktree" ? "worktree" : cfg.workspaceMode === "single" ? "single" : team.workspaceMode,
 			maxConcurrency:
-				typeof cfg.maxConcurrency === "number" &&
-				Number.isInteger(cfg.maxConcurrency) &&
-				cfg.maxConcurrency > 0
+				typeof cfg.maxConcurrency === "number" && Number.isInteger(cfg.maxConcurrency) && cfg.maxConcurrency > 0
 					? cfg.maxConcurrency
 					: team.maxConcurrency,
 			routing: parseRouting(cfg, team.routing),
@@ -680,34 +457,21 @@ export function handleUpdate(
 		let steps = workflow.steps;
 		if (hasOwn(cfg, "steps")) {
 			const parsedSteps = parseSteps(cfg.steps);
-			if (parsedSteps.error)
-				return result(parsedSteps.error, "error", true);
+			if (parsedSteps.error) return result(parsedSteps.error, "error", true);
 			steps = parsedSteps.steps!;
 		}
 		content = serializeWorkflow({
 			...workflow,
 			name: nextName,
 			filePath: nextPath,
-			description:
-				typeof cfg.description === "string" && cfg.description.trim()
-					? cfg.description.trim()
-					: workflow.description,
-			maxConcurrency: hasOwn(cfg, "maxConcurrency")
-				? parseWorkflowMaxConcurrency(cfg.maxConcurrency)
-				: workflow.maxConcurrency,
+			description: typeof cfg.description === "string" && cfg.description.trim() ? cfg.description.trim() : workflow.description,
+			maxConcurrency: hasOwn(cfg, "maxConcurrency") ? parseWorkflowMaxConcurrency(cfg.maxConcurrency) : workflow.maxConcurrency,
 			steps,
 		});
 	}
 
 	const referenceUpdates = params.updateReferences
-		? updateReferencesForRename(
-				ctx,
-				params.resource!,
-				current.name,
-				nextName,
-				source,
-				true,
-			)
+		? updateReferencesForRename(ctx, params.resource!, current.name, nextName, source, true)
 		: [];
 	if (params.dryRun) {
 		return result(
@@ -716,13 +480,7 @@ export function handleUpdate(
 				"",
 				content,
 				...(referenceUpdates.length
-					? [
-							"",
-							"Would update references in:",
-							...referenceUpdates.map(
-								(filePath) => `- ${filePath}`,
-							),
-						]
+					? ["", "Would update references in:", ...referenceUpdates.map((filePath) => `- ${filePath}`)]
 					: []),
 			].join("\n"),
 		);
@@ -750,36 +508,20 @@ export function handleUpdate(
 		);
 	}
 	const updatedRefs = params.updateReferences
-		? updateReferencesForRename(
-				ctx,
-				params.resource!,
-				current.name,
-				nextName,
-				source,
-				false,
-			)
+		? updateReferencesForRename(ctx, params.resource!, current.name, nextName, source, false)
 		: [];
 	return result(
 		[
 			`Updated ${params.resource} at ${nextPath}. Backup: ${backupPath}.`,
-			...(updatedRefs.length
-				? [
-						"Updated references:",
-						...updatedRefs.map((filePath) => `- ${filePath}`),
-					]
-				: []),
+			...(updatedRefs.length ? ["Updated references:", ...updatedRefs.map((filePath) => `- ${filePath}`)] : []),
 		].join("\n"),
 	);
 }
 
-export function handleDelete(
-	params: TeamToolParamsValue,
-	ctx: ManagementContext,
-): PiTeamsToolResult {
+export function handleDelete(params: TeamToolParamsValue, ctx: ManagementContext): PiTeamsToolResult {
 	const intentError = enforceDestructiveIntent("delete", params, ctx.config);
 	if (intentError) return intentError;
-	if (!params.confirm)
-		return result("delete requires confirm: true.", "error", true);
+	if (!params.confirm) return result("delete requires confirm: true.", "error", true);
 	const resolved = resolveMutable(ctx, params);
 	if (resolved.error) return resolved.error;
 	const refs = findReferences(ctx, params.resource!, resolved.resource!.name);
@@ -804,7 +546,5 @@ export function handleDelete(
 			true,
 		);
 	}
-	return result(
-		`Deleted ${params.resource} at ${resolved.resource!.filePath}. Backup: ${backupPath}.`,
-	);
+	return result(`Deleted ${params.resource} at ${resolved.resource!.filePath}. Backup: ${backupPath}.`);
 }