pi-crew 0.9.51 → 0.9.53

package/src/extension/management.ts

@@ -1,571 +0,0 @@
-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 { serializeAgent } from "../agents/agent-serializer.ts";
-import { discoverAgents, invalidateAgentDiscoveryCache } from "../agents/discover-agents.ts";
-import type { PiTeamsConfig } from "../config/config.ts";
-import type { TeamToolParamsValue } from "../schema/team-tool-schema.ts";
-import { atomicWriteFile } from "../state/atomic-write.ts";
-import { allTeams, discoverTeams, invalidateTeamDiscoveryCache } 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 { projectCrewRoot, userPiRoot } from "../utils/paths.ts";
-import { allWorkflows, discoverWorkflows, invalidateWorkflowDiscoveryCache } 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";
-import { type PiTeamsToolResult, toolResult } from "./tool-result.ts";
-
-/**
- * Invalidate all discovery caches that may have read the on-disk resource
- * file we are about to (or just did) write. Called once per create/update/delete
- * to keep the 5 s TTL caches in sync without forcing a per-render full rescan.
- */
-function invalidateResourceCaches(): void {
-	invalidateAgentDiscoveryCache();
-	invalidateTeamDiscoveryCache();
-	invalidateWorkflowDiscoveryCache();
-}
-
-interface ManagementContext {
-	cwd: string;
-	config?: PiTeamsConfig;
-}
-
-type MutableSource = "user" | "project";
-
-type MutableResource = AgentConfig | TeamConfig | WorkflowConfig;
-
-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 {
-	const base = scope === "user" ? userPiRoot() : projectCrewRoot(ctx.cwd);
-	if (resource === "agent") return path.join(base, "agents");
-	if (resource === "team") return path.join(base, "teams");
-	return path.join(base, "workflows");
-}
-
-function extensionFor(resource: "agent" | "team" | "workflow"): string {
-	if (resource === "agent") return ".md";
-	if (resource === "team") return ".team.md";
-	return ".workflow.md";
-}
-
-function backupFile(filePath: string): string {
-	// Include milliseconds and a short random suffix to prevent collision
-	// when multiple backups happen within the same second.
-	const ts = new Date().toISOString().replace(/[-:.TZ]/g, "");
-	const random = crypto.randomUUID().slice(0, 8);
-	const backupPath = `${filePath}.bak-${ts.slice(0, 17)}-${random}`;
-	fs.copyFileSync(filePath, backupPath);
-	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 parseStringArray(value: unknown): string[] | undefined {
-	if (typeof value === "string")
-		return value
-			.split(",")
-			.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 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,
-		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;
-}
-
-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." };
-	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.` };
-		const obj = item as Record<string, unknown>;
-		const name = requireString(obj.name, `config.roles[${i}].name`);
-		if (name.error) return { error: name.error };
-		const agent = requireString(obj.agent, `config.roles[${i}].agent`);
-		if (agent.error) return { error: agent.error };
-		roles.push({
-			name: sanitizeName(name.value!),
-			agent: sanitizeName(agent.value!),
-			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
-					? obj.maxConcurrency
-					: undefined,
-		});
-	}
-	return { roles };
-}
-
-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." };
-	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.` };
-		const obj = item as Record<string, unknown>;
-		const id = requireString(obj.id, `config.steps[${i}].id`);
-		if (id.error) return { error: id.error };
-		const role = requireString(obj.role, `config.steps[${i}].role`);
-		if (role.error) return { error: role.error };
-		// SECURITY: Sanitize task field to prevent markdown injection in workflow markdown.
-		// Strip characters that could create headings, code blocks, or links.
-		// Defense-in-depth — the task field is stored in YAML frontmatter, not executed.
-		const rawTask = obj.task;
-		const task =
-			typeof rawTask === "string"
-				? rawTask
-						.replace(/^#{1,6}\s+/gm, "")
-						.replace(/```/g, "``")
-						.replace(/\n{3,}/g, "\n\n")
-						.slice(0, 4000)
-				: "{goal}";
-		steps.push({
-			id: sanitizeName(id.value!),
-			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,
-			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,
-			verify: typeof obj.verify === "boolean" ? obj.verify : undefined,
-		});
-	}
-	return { steps };
-}
-
-function parseWorkflowMaxConcurrency(value: unknown): number | 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[] {
-	const normalized = sanitizeName(name);
-	const sourceMatches = (item: { name: string; source: ResourceSource }) =>
-		(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];
-		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];
-		return pool.filter(sourceMatches);
-	}
-	{
-		const discovery = discoverWorkflows(ctx.cwd);
-		const pool =
-			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[] {
-	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 (resource === "workflow") {
-		for (const team of allTeams(discoverTeams(ctx.cwd))) {
-			if (team.defaultWorkflow === name) refs.push(`team '${team.name}' defaultWorkflow`);
-		}
-	}
-	return refs;
-}
-
-function escapeRegex(str: string): string {
-	return str.replace(/[.*+?^${}()|[\\]\\]/g, "\\$&");
-}
-
-function walkTsFiles(dir: string): string[] {
-	const results: string[] = [];
-	if (!fs.existsSync(dir)) return results;
-	for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
-		const fullPath = path.join(dir, entry.name);
-		if (entry.isDirectory()) {
-			results.push(...walkTsFiles(fullPath));
-		} else if (entry.name.endsWith(".ts") || entry.name.endsWith(".md")) {
-			results.push(fullPath);
-		}
-	}
-	return results;
-}
-
-function updateReferencesForRename(
-	ctx: ManagementContext,
-	resource: "agent" | "team" | "workflow",
-	oldName: string,
-	newName: string,
-	scope: MutableSource,
-	dryRun: boolean,
-): string[] {
-	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)) {
-		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);
-			nextTeam = { ...team, roles };
-		}
-		if (resource === "workflow" && team.defaultWorkflow === oldName) {
-			updated = true;
-			nextTeam = { ...team, defaultWorkflow: newName };
-		}
-		if (!updated) continue;
-		changed.push(team.filePath);
-		if (!dryRun) {
-			backupFile(team.filePath);
-			atomicWriteFile(team.filePath, serializeTeam(nextTeam));
-		}
-	}
-	// 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)) {
-		let updated = false;
-		const newSteps = workflow.steps.map((step) => {
-			if (step.role === oldName) {
-				updated = true;
-				return { ...step, role: newName };
-			}
-			return step;
-		});
-		if (!updated) continue;
-		changed.push(workflow.filePath);
-		if (!dryRun) {
-			backupFile(workflow.filePath);
-			atomicWriteFile(workflow.filePath, serializeWorkflow({ ...workflow, steps: newSteps }));
-		}
-	}
-	// L12 fix: update agent references in 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 newContent = content.replace(agentPattern, `$1${newName}$2`);
-			if (newContent !== content) {
-				changed.push(fixture);
-				if (!dryRun) {
-					atomicWriteFile(fixture, newContent);
-				}
-			}
-		}
-	}
-	return changed;
-}
-
-function resolveMutable(ctx: ManagementContext, params: TeamToolParamsValue): { resource?: MutableResource; error?: PiTeamsToolResult } {
-	if (!params.resource)
-		return {
-			error: result("resource is required for update/delete.", "error", true),
-		};
-	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),
-		};
-	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),
-		};
-	if (matches.length > 1)
-		return {
-			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);
-	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 name = sanitizeName(nameValue.value!);
-	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);
-	fs.mkdirSync(path.dirname(filePath), { recursive: true });
-
-	let content: string;
-	if (params.resource === "agent") {
-		const agent: AgentConfig = {
-			name,
-			description: descriptionValue.value!,
-			source: scope,
-			filePath,
-			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,
-			tools: parseStringArray(cfg.tools),
-			extensions: hasOwn(cfg, "extensions") ? (parseStringArray(cfg.extensions) ?? []) : undefined,
-			skills: parseStringArray(cfg.skills),
-			systemPromptMode: cfg.systemPromptMode === "append" ? "append" : "replace",
-			inheritProjectContext: cfg.inheritProjectContext === true,
-			inheritSkills: cfg.inheritSkills === true,
-			routing: parseRouting(cfg),
-		};
-		content = serializeAgent(agent);
-	} else if (params.resource === "team") {
-		const parsedRoles = parseRoles(cfg.roles);
-		if (parsedRoles.error) return result(parsedRoles.error, "error", true);
-		content = serializeTeam({
-			name,
-			description: descriptionValue.value!,
-			source: scope,
-			filePath,
-			roles: parsedRoles.roles!,
-			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
-					? cfg.maxConcurrency
-					: undefined,
-			routing: parseRouting(cfg),
-		});
-	} else {
-		const parsedSteps = parseSteps(cfg.steps);
-		if (parsedSteps.error) return result(parsedSteps.error, "error", true);
-		content = serializeWorkflow({
-			name,
-			description: descriptionValue.value!,
-			source: scope,
-			filePath,
-			maxConcurrency: parseWorkflowMaxConcurrency(cfg.maxConcurrency),
-			steps: parsedSteps.steps!,
-		});
-	}
-
-	if (params.dryRun) return result(`[dry-run] Would create ${params.resource} '${name}' at ${filePath}:\n\n${content}`);
-	try {
-		atomicWriteFile(filePath, content);
-	} catch (writeError) {
-		return result(
-			`Failed to create ${params.resource}: ${writeError instanceof Error ? writeError.message : String(writeError)}`,
-			"error",
-			true,
-		);
-	}
-	// F17/F15: bust the discovery caches so future powerbar/scheduler reads see
-	// the new file immediately (instead of after the 5 s TTL expires).
-	invalidateResourceCaches();
-	return result(`Created ${params.resource} '${name}' at ${filePath}.`);
-}
-
-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 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);
-
-	let content: string;
-	if (params.resource === "agent") {
-		const agent = current as AgentConfig;
-		content = serializeAgent({
-			...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,
-			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,
-			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,
-			routing: parseRouting(cfg, agent.routing),
-		});
-	} else if (params.resource === "team") {
-		const team = current as TeamConfig;
-		let roles = team.roles;
-		if (hasOwn(cfg, "roles")) {
-			const parsedRoles = parseRoles(cfg.roles);
-			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,
-			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,
-			maxConcurrency:
-				typeof cfg.maxConcurrency === "number" && Number.isInteger(cfg.maxConcurrency) && cfg.maxConcurrency > 0
-					? cfg.maxConcurrency
-					: team.maxConcurrency,
-			routing: parseRouting(cfg, team.routing),
-		});
-	} else {
-		const workflow = current as WorkflowConfig;
-		let steps = workflow.steps;
-		if (hasOwn(cfg, "steps")) {
-			const parsedSteps = parseSteps(cfg.steps);
-			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,
-			steps,
-		});
-	}
-
-	const referenceUpdates = params.updateReferences
-		? updateReferencesForRename(ctx, params.resource!, current.name, nextName, source, true)
-		: [];
-	if (params.dryRun) {
-		return result(
-			[
-				`[dry-run] Would update ${params.resource} at ${current.filePath}:`,
-				"",
-				content,
-				...(referenceUpdates.length
-					? ["", "Would update references in:", ...referenceUpdates.map((filePath) => `- ${filePath}`)]
-					: []),
-			].join("\n"),
-		);
-	}
-	const backupPath = backupFile(current.filePath);
-	try {
-		if (nextPath !== current.filePath) {
-			try {
-				fs.renameSync(current.filePath, nextPath);
-			} catch (renameError) {
-				if ((renameError as NodeJS.ErrnoException).code === "EXDEV") {
-					fs.copyFileSync(current.filePath, nextPath);
-					fs.unlinkSync(current.filePath);
-				} else {
-					throw renameError;
-				}
-			}
-		}
-		atomicWriteFile(nextPath, content);
-	} catch (updateError) {
-		return result(
-			`Failed to update ${params.resource}: ${updateError instanceof Error ? updateError.message : String(updateError)}`,
-			"error",
-			true,
-		);
-	}
-	// F17/F15: bust the discovery caches so the renamed/updated resource is
-	// visible immediately. updateReferencesForRename may have rewritten peer
-	// files too, so we invalidate AFTER that pass completes.
-	const updatedRefs = params.updateReferences
-		? updateReferencesForRename(ctx, params.resource!, current.name, nextName, source, false)
-		: [];
-	invalidateResourceCaches();
-	return result(
-		[
-			`Updated ${params.resource} at ${nextPath}. Backup: ${backupPath}.`,
-			...(updatedRefs.length ? ["Updated references:", ...updatedRefs.map((filePath) => `- ${filePath}`)] : []),
-		].join("\n"),
-	);
-}
-
-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);
-	const resolved = resolveMutable(ctx, params);
-	if (resolved.error) return resolved.error;
-	const refs = findReferences(ctx, params.resource!, resolved.resource!.name);
-	if (refs.length > 0 && !params.force) {
-		return result(
-			`${params.resource} '${resolved.resource!.name}' is still referenced. Use force: true to delete anyway.\n${refs.map((ref) => `- ${ref}`).join("\n")}`,
-			"error",
-			true,
-		);
-	}
-	if (params.dryRun)
-		return result(
-			`[dry-run] Would delete ${params.resource} at ${resolved.resource!.filePath}.${refs.length ? `\nReferences:\n${refs.map((ref) => `- ${ref}`).join("\n")}` : ""}`,
-		);
-	const backupPath = backupFile(resolved.resource!.filePath);
-	try {
-		fs.unlinkSync(resolved.resource!.filePath);
-	} catch (deleteError) {
-		return result(
-			`Failed to delete ${params.resource}: ${deleteError instanceof Error ? deleteError.message : String(deleteError)}`,
-			"error",
-			true,
-		);
-	}
-	// F17/F15: bust the discovery caches so the deletion is visible immediately.
-	invalidateResourceCaches();
-	return result(`Deleted ${params.resource} at ${resolved.resource!.filePath}. Backup: ${backupPath}.`);
-}

package/src/extension/message-renderers.ts

@@ -1,108 +0,0 @@
-/**
- * Custom message renderers for pi-crew session entries (Round 13 UX).
- *
- * pi-crew emits CustomMessageEntry rows via `pi.appendEntry()` for run
- * lifecycle events (crew:run-started, crew:run-completed,
- * crew:resume-directive). Without a registered renderer these display as
- * raw JSON in the conversation. These renderers give them a clean,
- * crew-branded look using the active theme.
- */
-
-import type { ExtensionAPI, MessageRenderOptions, Theme } from "@earendil-works/pi-coding-agent";
-import { Text } from "@earendil-works/pi-tui";
-
-interface CrewMessageDetails {
-	runId?: string;
-	team?: string;
-	workflow?: string;
-	agent?: string;
-	goal?: string;
-	status?: string;
-	taskCount?: number;
-	timestamp?: number;
-}
-
-type MessageLike = {
-	content: string | Array<{ type: string; text?: string }>;
-	details?: CrewMessageDetails;
-};
-
-function extractText(message: MessageLike): string {
-	if (typeof message.content === "string") return message.content;
-	return (message.content ?? []).map((c) => c.text ?? "").join("");
-}
-
-function statusLevel(status: string | undefined): "success" | "error" | "warning" | "muted" {
-	switch (status) {
-		case "completed":
-			return "success";
-		case "failed":
-		case "blocked":
-			return "error";
-		case "cancelled":
-			return "warning";
-		default:
-			return "muted";
-	}
-}
-
-function statusIcon(status: string | undefined): string {
-	switch (status) {
-		case "completed":
-			return "✅";
-		case "failed":
-		case "blocked":
-			return "❌";
-		case "cancelled":
-			return "🚫";
-		case "running":
-		case "planning":
-			return "🚀";
-		default:
-			return "•";
-	}
-}
-
-/** Truncate a string to maxLen chars with an ellipsis. */
-function truncate(text: string, maxLen: number): string {
-	return text.length > maxLen ? `${text.slice(0, maxLen - 1)}…` : text;
-}
-
-/** Render crew:run-started entries as a branded launch line. */
-export function renderRunStarted(message: MessageLike, _options: MessageRenderOptions, theme: Theme): Text {
-	const details = message.details ?? {};
-	const goal = details.goal ? truncate(details.goal, 70) : "";
-	const team = details.team ?? details.agent ?? "direct";
-	const workflow = details.workflow ?? "default";
-	const text = `🚀 crew run ${details.runId ?? ""} started — ${team}/${workflow}${goal ? ` — ${goal}` : ""}`;
-	return new Text(theme.fg("accent", theme.bold("crew ")) + theme.fg("text", text), 0, 0);
-}
-
-/** Render crew:run-completed entries with a status-colored summary. */
-export function renderRunCompleted(message: MessageLike, _options: MessageRenderOptions, theme: Theme): Text {
-	const details = message.details ?? {};
-	const status = details.status;
-	const level = statusLevel(status);
-	const icon = statusIcon(status);
-	const goal = details.goal ? truncate(details.goal, 60) : "";
-	const tasks = details.taskCount !== undefined ? ` · ${details.taskCount} tasks` : "";
-	const text = `${icon} crew run ${details.runId ?? ""} ${status ?? "finished"}${tasks}${goal ? ` — ${goal}` : ""}`;
-	return new Text(theme.fg(level, theme.bold("crew ")) + theme.fg(level, text), 0, 0);
-}
-
-/** Render crew:resume-directive entries as an informational system note. */
-export function renderResumeDirective(message: MessageLike, _options: MessageRenderOptions, theme: Theme): Text {
-	const text = extractText(message) || "Context compacted — resuming in-flight crew work.";
-	return new Text(theme.fg("muted", theme.bold("crew ") + text), 0, 0);
-}
-
-/** Register all crew message renderers. Safe to call once at extension load. */
-export function registerCrewMessageRenderers(pi: { registerMessageRenderer?: ExtensionAPI["registerMessageRenderer"] }): void {
-	// Optional chaining guards against older Pi versions (and test stubs)
-	// without registerMessageRenderer.
-	// The renderers return Text (a Component) — cast through never to match
-	// the MessageRenderer<T> signature which expects Component | undefined.
-	pi.registerMessageRenderer?.("crew:run-started", renderRunStarted as never);
-	pi.registerMessageRenderer?.("crew:run-completed", renderRunCompleted as never);
-	pi.registerMessageRenderer?.("crew:resume-directive", renderResumeDirective as never);
-}