@oh-my-pi/pi-coding-agent 8.1.0 → 8.2.0

package/src/task/discovery.ts

@@ -11,13 +11,11 @@
  *
  * Agent files use markdown with YAML frontmatter.
  */
-
-import * as fs from "node:fs";
+import * as fs from "node:fs/promises";
 import * as path from "node:path";
-import { findAllNearestProjectConfigDirs, getConfigDirs } from "@oh-my-pi/pi-coding-agent/config";
-import { parseAgentFields } from "@oh-my-pi/pi-coding-agent/discovery/helpers";
-import { parseFrontmatter } from "@oh-my-pi/pi-coding-agent/utils/frontmatter";
-import { loadBundledAgents } from "./agents";
+import { logger } from "@oh-my-pi/pi-utils";
+import { findAllNearestProjectConfigDirs, getConfigDirs } from "../config";
+import { loadBundledAgents, parseAgent } from "./agents";
 import type { AgentDefinition, AgentSource } from "./types";
 
 /** Result of agent discovery */
@@ -29,55 +27,23 @@ export interface DiscoveryResult {
 /**
  * Load agents from a directory.
  */
-function loadAgentsFromDir(dir: string, source: AgentSource): AgentDefinition[] {
-	const agents: AgentDefinition[] = [];
-
-	if (!fs.existsSync(dir)) {
-		return agents;
-	}
-
-	let entries: fs.Dirent[];
-	try {
-		entries = fs.readdirSync(dir, { withFileTypes: true });
-	} catch {
-		return agents;
-	}
-
-	for (const entry of entries) {
-		if (!entry.name.endsWith(".md")) continue;
-
-		const filePath = path.resolve(dir, entry.name);
-
-		// Handle both regular files and symlinks
-		try {
-			if (!fs.statSync(filePath).isFile()) continue;
-		} catch {
-			continue;
-		}
-
-		let content: string;
-		try {
-			content = fs.readFileSync(filePath, "utf-8");
-		} catch {
-			continue;
-		}
-
-		const { frontmatter, body } = parseFrontmatter(content, { source: filePath });
-		const fields = parseAgentFields(frontmatter);
-
-		if (!fields) {
-			continue;
-		}
-
-		agents.push({
-			...fields,
-			systemPrompt: body,
-			source,
-			filePath,
+async function loadAgentsFromDir(dir: string, source: AgentSource): Promise<AgentDefinition[]> {
+	const entries = await fs.readdir(dir, { withFileTypes: true }).catch(() => []);
+	const files = entries
+		.filter(entry => (entry.isFile() || entry.isSymbolicLink()) && entry.name.endsWith(".md"))
+		.sort((a, b) => a.name.localeCompare(b.name))
+		.map(file => {
+			const filePath = path.join(dir, file.name);
+			return fs
+				.readFile(filePath, "utf-8")
+				.then(content => parseAgent(filePath, content, source, "warn"))
+				.catch(error => {
+					logger.warn("Failed to read agent file", { filePath, error });
+					return null;
+				});
 		});
-	}
 
-	return agents;
+	return (await Promise.all(files)).filter(Boolean) as AgentDefinition[];
 }
 
 /**
@@ -89,62 +55,59 @@ function loadAgentsFromDir(dir: string, source: AgentSource): AgentDefinition[]
  */
 export async function discoverAgents(cwd: string): Promise<DiscoveryResult> {
 	const resolvedCwd = path.resolve(cwd);
-	const agentSources = Array.from(new Set(getConfigDirs("", { project: false }).map((entry) => entry.source)));
+	const agentSources = Array.from(new Set(getConfigDirs("", { project: false }).map(entry => entry.source)));
 
 	// Get user directories (priority order: .omp, .pi, .claude, ...)
 	const userDirs = getConfigDirs("agents", { project: false })
-		.filter((entry) => agentSources.includes(entry.source))
-		.map((entry) => ({
+		.filter(entry => agentSources.includes(entry.source))
+		.map(entry => ({
 			...entry,
 			path: path.resolve(entry.path),
 		}));
 
 	// Get project directories by walking up from cwd (priority order)
 	const projectDirs = (await findAllNearestProjectConfigDirs("agents", resolvedCwd))
-		.filter((entry) => agentSources.includes(entry.source))
-		.map((entry) => ({
+		.filter(entry => agentSources.includes(entry.source))
+		.map(entry => ({
 			...entry,
 			path: path.resolve(entry.path),
 		}));
 
 	const orderedSources = agentSources.filter(
-		(source) =>
-			userDirs.some((entry) => entry.source === source) || projectDirs.some((entry) => entry.source === source),
+		source => userDirs.some(entry => entry.source === source) || projectDirs.some(entry => entry.source === source),
 	);
 
 	const orderedDirs: Array<{ dir: string; source: AgentSource }> = [];
 	for (const source of orderedSources) {
-		const project = projectDirs.find((entry) => entry.source === source);
+		const project = projectDirs.find(entry => entry.source === source);
 		if (project) orderedDirs.push({ dir: project.path, source: "project" });
-		const user = userDirs.find((entry) => entry.source === source);
+		const user = userDirs.find(entry => entry.source === source);
 		if (user) orderedDirs.push({ dir: user.path, source: "user" });
 	}
 
-	const agents: AgentDefinition[] = [];
 	const seen = new Set<string>();
-
-	for (const { dir, source } of orderedDirs) {
-		for (const agent of loadAgentsFromDir(dir, source)) {
-			if (seen.has(agent.name)) continue;
-			agents.push(agent);
+	const loadedAgents = (await Promise.all(orderedDirs.map(({ dir, source }) => loadAgentsFromDir(dir, source))))
+		.flat()
+		.filter(agent => {
+			if (seen.has(agent.name)) return false;
 			seen.add(agent.name);
-		}
-	}
+			return true;
+		});
 
-	for (const agent of loadBundledAgents()) {
-		if (seen.has(agent.name)) continue;
-		agents.push(agent);
+	const bundledAgents = loadBundledAgents().filter(agent => {
+		if (seen.has(agent.name)) return false;
 		seen.add(agent.name);
-	}
+		return true;
+	});
 
 	const projectAgentsDir = projectDirs.length > 0 ? projectDirs[0].path : null;
 
-	return { agents, projectAgentsDir };
+	return { agents: [...loadedAgents, ...bundledAgents], projectAgentsDir };
 }
 
 /**
  * Get an agent by name from discovered agents.
  */
 export function getAgent(agents: AgentDefinition[], name: string): AgentDefinition | undefined {
-	return agents.find((a) => a.name === name);
+	return agents.find(a => a.name === name);
 }

package/src/task/executor.ts

@@ -3,20 +3,19 @@
  *
  * Runs each subagent in a Bun Worker and forwards AgentEvents for progress tracking.
  */
-
 import path from "node:path";
 import type { AgentEvent, ThinkingLevel } from "@oh-my-pi/pi-agent-core";
-import type { ModelRegistry } from "@oh-my-pi/pi-coding-agent/config/model-registry";
-import { formatModelString, parseModelPattern } from "@oh-my-pi/pi-coding-agent/config/model-resolver";
-import { checkPythonKernelAvailability } from "@oh-my-pi/pi-coding-agent/ipy/kernel";
-import { LspTool } from "@oh-my-pi/pi-coding-agent/lsp";
-import type { LspParams } from "@oh-my-pi/pi-coding-agent/lsp/types";
-import { callTool } from "@oh-my-pi/pi-coding-agent/mcp/client";
-import type { MCPManager } from "@oh-my-pi/pi-coding-agent/mcp/manager";
-import type { AuthStorage } from "@oh-my-pi/pi-coding-agent/session/auth-storage";
-import { PythonTool, type PythonToolParams } from "@oh-my-pi/pi-coding-agent/tools/python";
-import type { EventBus } from "@oh-my-pi/pi-coding-agent/utils/event-bus";
 import type { ToolSession } from "..";
+import type { ModelRegistry } from "../config/model-registry";
+import { formatModelString, parseModelPattern } from "../config/model-resolver";
+import { checkPythonKernelAvailability } from "../ipy/kernel";
+import { LspTool } from "../lsp";
+import type { LspParams } from "../lsp/types";
+import { callTool } from "../mcp/client";
+import type { MCPManager } from "../mcp/manager";
+import type { AuthStorage } from "../session/auth-storage";
+import { PythonTool, type PythonToolParams } from "../tools/python";
+import type { EventBus } from "../utils/event-bus";
 import { subprocessToolRegistry } from "./subprocess-tool-registry";
 import {
 	type AgentDefinition,
@@ -171,7 +170,7 @@ function getUsageTokens(usage: unknown): number {
  * falling back to empty strings if not.
  */
 function extractMCPToolMetadata(mcpManager: MCPManager): MCPToolMetadata[] {
-	return mcpManager.getTools().map((tool) => {
+	return mcpManager.getTools().map(tool => {
 		// MCPTool and DeferredMCPTool have these properties
 		const mcpTool = tool as { mcpToolName?: string; mcpServerName?: string };
 		return {
@@ -264,7 +263,7 @@ export async function runSubprocess(options: ExecutorOptions): Promise<SingleRes
 
 	const pythonToolMode = options.settingsManager?.getPythonToolMode?.() ?? "ipy-only";
 	if (toolNames?.includes("exec")) {
-		const expanded = toolNames.filter((name) => name !== "exec");
+		const expanded = toolNames.filter(name => name !== "exec");
 		if (pythonToolMode === "bash-only") {
 			expanded.push("bash");
 		} else if (pythonToolMode === "ipy-only") {
@@ -706,11 +705,11 @@ export async function runSubprocess(options: ExecutorOptions): Promise<SingleRes
 		message: string;
 	}
 
-	const done = await new Promise<Extract<SubagentWorkerResponse, { type: "done" }>>((resolve) => {
+	const done = await new Promise<Extract<SubagentWorkerResponse, { type: "done" }>>(resolve => {
 		const cleanup = () => {
 			listenerController.abort();
 		};
-		finalize = (message) => {
+		finalize = message => {
 			if (resolved) return;
 			resolved = true;
 			cleanup();

package/src/task/index.ts

@@ -12,22 +12,23 @@
  *   - Progress tracking via JSON events
  *   - Session artifacts for debugging
  */
-
-import { mkdir, rm, stat } from "node:fs/promises";
-import { tmpdir } from "node:os";
+import * as fs from "node:fs/promises";
+import * as os from "node:os";
 import path from "node:path";
 import type { AgentTool, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import type { Usage } from "@oh-my-pi/pi-ai";
-import { renderPromptTemplate } from "@oh-my-pi/pi-coding-agent/config/prompt-templates";
-import type { Theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import taskDescriptionTemplate from "@oh-my-pi/pi-coding-agent/prompts/tools/task.md" with { type: "text" };
-import { AgentOutputManager } from "@oh-my-pi/pi-coding-agent/task/output-manager";
-import { formatDuration } from "@oh-my-pi/pi-coding-agent/tools/render-utils";
 import { $ } from "bun";
 import { nanoid } from "nanoid";
 import type { ToolSession } from "..";
+import { renderPromptTemplate } from "../config/prompt-templates";
+import type { Theme } from "../modes/theme/theme";
+import taskDescriptionTemplate from "../prompts/tools/task.md" with { type: "text" };
+import { formatDuration } from "../tools/render-utils";
+// Import review tools for side effects (registers subagent tool handlers)
+import "../tools/review";
 import { discoverAgents, getAgent } from "./discovery";
 import { runSubprocess } from "./executor";
+import { AgentOutputManager } from "./output-manager";
 import { mapWithConcurrencyLimit } from "./parallel";
 import { renderCall, renderResult } from "./render";
 import { renderTemplate } from "./template";
@@ -49,9 +50,6 @@ import {
 	getRepoRoot,
 } from "./worktree";
 
-// Import review tools for side effects (registers subagent tool handlers)
-import "../tools/review";
-
 /** Format byte count for display */
 function formatBytes(bytes: number): string {
 	if (bytes < 1024) return `${bytes}B`;
@@ -174,7 +172,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 		// Validate agent exists
 		const agent = getAgent(agents, agentName);
 		if (!agent) {
-			const available = agents.map((a) => a.name).join(", ") || "none";
+			const available = agents.map(a => a.name).join(", ") || "none";
 			return {
 				content: [
 					{
@@ -268,7 +266,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 				problems.push(`Missing task ids at indexes: ${missingTaskIndexes.join(", ")}`);
 			}
 			if (duplicateIds.length > 0) {
-				const details = duplicateIds.map((entry) => `${entry.id} (indexes ${entry.indexes.join(", ")})`).join("; ");
+				const details = duplicateIds.map(entry => `${entry.id} (indexes ${entry.indexes.join(", ")})`).join("; ");
 				problems.push(`Duplicate task ids detected (case-insensitive): ${details}`);
 			}
 			return {
@@ -308,9 +306,8 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 		// Derive artifacts directory
 		const sessionFile = this.session.getSessionFile();
 		const artifactsDir = sessionFile ? sessionFile.slice(0, -6) : null;
-		const tempArtifactsDir = artifactsDir ? null : path.join(tmpdir(), `omp-task-${nanoid()}`);
+		const tempArtifactsDir = artifactsDir ? null : path.join(os.tmpdir(), `omp-task-${nanoid()}`);
 		const effectiveArtifactsDir = artifactsDir || tempArtifactsDir!;
-		await mkdir(effectiveArtifactsDir, { recursive: true });
 
 		// Initialize progress tracking
 		const progressMap = new Map<number, AgentProgress>();
@@ -349,7 +346,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 
 			// Check spawn restrictions from parent
 			const parentSpawns = this.session.getSessionSpawns() ?? "*";
-			const allowedSpawns = parentSpawns.split(",").map((s) => s.trim());
+			const allowedSpawns = parentSpawns.split(",").map(s => s.trim());
 			const isSpawnAllowed = (): boolean => {
 				if (parentSpawns === "") return false; // Empty = deny all
 				if (parentSpawns === "*") return true; // Wildcard = allow all
@@ -372,11 +369,11 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 			// Allocate unique IDs across the session to prevent artifact collisions
 			const outputManager =
 				this.session.agentOutputManager ?? new AgentOutputManager(this.session.getArtifactsDir ?? (() => null));
-			const uniqueIds = await outputManager.allocateBatch(tasks.map((t) => t.id));
+			const uniqueIds = await outputManager.allocateBatch(tasks.map(t => t.id));
 			const tasksWithUniqueIds = tasks.map((t, i) => ({ ...t, id: uniqueIds[i] }));
 
 			// Build full prompts with context prepended
-			const tasksWithContext = tasksWithUniqueIds.map((t) => renderTemplate(context, t));
+			const tasksWithContext = tasksWithUniqueIds.map(t => renderTemplate(context, t));
 
 			// Initialize progress for all tasks
 			for (let i = 0; i < tasksWithContext.length; i++) {
@@ -419,7 +416,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 						enableLsp: false,
 						signal,
 						eventBus: undefined,
-						onProgress: (progress) => {
+						onProgress: progress => {
 							progressMap.set(index, {
 								...structuredClone(progress),
 								args: tasksWithContext[index]?.args,
@@ -459,7 +456,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 						enableLsp: false,
 						signal,
 						eventBus: undefined,
-						onProgress: (progress) => {
+						onProgress: progress => {
 							progressMap.set(index, {
 								...structuredClone(progress),
 								args: tasksWithContext[index]?.args,
@@ -565,29 +562,29 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 			let patchApplySummary = "";
 			let patchesApplied: boolean | null = null;
 			if (isIsolated) {
-				const patchesInOrder = results.map((result) => result.patchPath).filter(Boolean) as string[];
-				const missingPatch = results.some((result) => !result.patchPath);
+				const patchesInOrder = results.map(result => result.patchPath).filter(Boolean) as string[];
+				const missingPatch = results.some(result => !result.patchPath);
 				if (!repoRoot || missingPatch) {
 					patchesApplied = false;
 				} else {
 					const patchStats = await Promise.all(
-						patchesInOrder.map(async (patchPath) => ({
+						patchesInOrder.map(async patchPath => ({
 							patchPath,
-							size: (await stat(patchPath)).size,
+							size: (await fs.stat(patchPath)).size,
 						})),
 					);
-					const nonEmptyPatches = patchStats.filter((patch) => patch.size > 0).map((patch) => patch.patchPath);
+					const nonEmptyPatches = patchStats.filter(patch => patch.size > 0).map(patch => patch.patchPath);
 					if (nonEmptyPatches.length === 0) {
 						patchesApplied = true;
 					} else {
 						const patchTexts = await Promise.all(
-							nonEmptyPatches.map(async (patchPath) => Bun.file(patchPath).text()),
+							nonEmptyPatches.map(async patchPath => Bun.file(patchPath).text()),
 						);
-						const combinedPatch = patchTexts.map((text) => (text.endsWith("\n") ? text : `${text}\n`)).join("");
+						const combinedPatch = patchTexts.map(text => (text.endsWith("\n") ? text : `${text}\n`)).join("");
 						if (!combinedPatch.trim()) {
 							patchesApplied = true;
 						} else {
-							const combinedPatchPath = path.join(tmpdir(), `omp-task-combined-${nanoid()}.patch`);
+							const combinedPatchPath = path.join(os.tmpdir(), `omp-task-combined-${nanoid()}.patch`);
 							try {
 								await Bun.write(combinedPatchPath, combinedPatch);
 								const checkResult = await $`git apply --check --binary ${combinedPatchPath}`
@@ -604,7 +601,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 									patchesApplied = applyResult.exitCode === 0;
 								}
 							} finally {
-								await rm(combinedPatchPath, { force: true });
+								await fs.rm(combinedPatchPath, { force: true });
 							}
 						}
 					}
@@ -617,18 +614,18 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 						"<system-notification>Patches were not applied and must be handled manually.</system-notification>";
 					const patchList =
 						patchPaths.length > 0
-							? `\n\nPatch artifacts:\n${patchPaths.map((patch) => `- ${patch}`).join("\n")}`
+							? `\n\nPatch artifacts:\n${patchPaths.map(patch => `- ${patch}`).join("\n")}`
 							: "";
 					patchApplySummary = `\n\n${notification}${patchList}`;
 				}
 			}
 
 			// Build final output - match plugin format
-			const successCount = results.filter((r) => r.exitCode === 0).length;
-			const cancelledCount = results.filter((r) => r.aborted).length;
+			const successCount = results.filter(r => r.exitCode === 0).length;
+			const cancelledCount = results.filter(r => r.aborted).length;
 			const totalDuration = Date.now() - startTime;
 
-			const summaries = results.map((r) => {
+			const summaries = results.map(r => {
 				const status = r.aborted ? "cancelled" : r.exitCode === 0 ? "completed" : `failed (exit ${r.exitCode})`;
 				const output = r.output.trim() || r.stderr.trim() || "(no output)";
 				const preview = output.split("\n").slice(0, 5).join("\n");
@@ -638,10 +635,10 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 				return `[${r.agent}] ${status}${meta} ${r.id}\n${preview}`;
 			});
 
-			const outputIds = results.filter((r) => !r.aborted || r.output.trim()).map((r) => r.id);
+			const outputIds = results.filter(r => !r.aborted || r.output.trim()).map(r => r.id);
 			const outputHint =
 				outputIds.length > 0
-					? `\n\nUse read with agent:// for full logs: ${outputIds.map((id) => `agent://${id}`).join(", ")}`
+					? `\n\nUse read with agent:// for full logs: ${outputIds.map(id => `agent://${id}`).join(", ")}`
 					: "";
 			const schemaNote = schemaOverridden
 				? `\n\nNote: Agent '${agentName}' has a fixed output schema; your 'output' parameter was ignored.\nRequired schema: ${JSON.stringify(agent.output)}`
@@ -655,7 +652,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 			const shouldCleanupTempArtifacts =
 				tempArtifactsDir && (!isIsolated || patchesApplied === true || patchesApplied === null);
 			if (shouldCleanupTempArtifacts) {
-				await rm(tempArtifactsDir, { recursive: true, force: true });
+				await fs.rm(tempArtifactsDir, { recursive: true, force: true });
 			}
 
 			return {

package/src/task/output-manager.ts

@@ -6,8 +6,7 @@
  *
  * This enables reliable agent:// URL resolution and prevents artifact collisions.
  */
-
-import { readdir } from "node:fs/promises";
+import * as fs from "node:fs/promises";
 
 /**
  * Manages agent output ID allocation to ensure uniqueness.
@@ -37,7 +36,7 @@ export class AgentOutputManager {
 
 		let files: string[];
 		try {
-			files = await readdir(dir);
+			files = await fs.readdir(dir);
 		} catch {
 			return; // Directory doesn't exist yet
 		}
@@ -73,7 +72,7 @@ export class AgentOutputManager {
 	 */
 	async allocateBatch(ids: string[]): Promise<string[]> {
 		await this.#ensureInitialized();
-		return ids.map((id) => `${this.#nextId++}-${id}`);
+		return ids.map(id => `${this.#nextId++}-${id}`);
 	}
 
 	/**

package/src/task/parallel.ts

@@ -1,7 +1,6 @@
 /**
  * Parallel execution with concurrency control.
  */
-
 import { MAX_CONCURRENCY } from "./types";
 
 /** Result of parallel execution */

package/src/task/render.ts

@@ -4,10 +4,11 @@
  * Provides renderCall and renderResult functions for displaying
  * task execution in the terminal UI.
  */
-
 import path from "node:path";
-import type { RenderResultOptions } from "@oh-my-pi/pi-coding-agent/extensibility/custom-tools/types";
-import type { Theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
+import type { Component } from "@oh-my-pi/pi-tui";
+import { Container, Text } from "@oh-my-pi/pi-tui";
+import type { RenderResultOptions } from "../extensibility/custom-tools/types";
+import type { Theme } from "../modes/theme/theme";
 import {
 	formatBadge,
 	formatDuration,
@@ -15,17 +16,15 @@ import {
 	formatStatusIcon,
 	formatTokens,
 	truncate,
-} from "@oh-my-pi/pi-coding-agent/tools/render-utils";
+} from "../tools/render-utils";
 import {
 	type FindingPriority,
 	getPriorityInfo,
 	PRIORITY_LABELS,
 	type ReportFindingDetails,
 	type SubmitReviewDetails,
-} from "@oh-my-pi/pi-coding-agent/tools/review";
-import { renderStatusLine } from "@oh-my-pi/pi-coding-agent/tui";
-import type { Component } from "@oh-my-pi/pi-tui";
-import { Container, Text } from "@oh-my-pi/pi-tui";
+} from "../tools/review";
+import { renderStatusLine } from "../tui";
 import { subprocessToolRegistry } from "./subprocess-tool-registry";
 import type { AgentProgress, SingleResult, TaskParams, TaskToolDetails } from "./types";
 
@@ -79,7 +78,7 @@ function formatJsonScalar(value: unknown, theme: Theme): string {
 }
 
 function buildTreePrefix(ancestors: boolean[], theme: Theme): string {
-	return ancestors.map((hasNext) => (hasNext ? `${theme.tree.vertical}  ` : "   ")).join("");
+	return ancestors.map(hasNext => (hasNext ? `${theme.tree.vertical}  ` : "   ")).join("");
 }
 
 function renderJsonTreeLines(
@@ -295,7 +294,7 @@ function formatArgsInline(args: Record<string, string>, theme: Theme): string {
 
 /** Convert snake_case or kebab-case to Title Case */
 function humanizeKey(key: string): string {
-	return key.replace(/[-_]/g, " ").replace(/\b\w/g, (c) => c.toUpperCase());
+	return key.replace(/[-_]/g, " ").replace(/\b\w/g, c => c.toUpperCase());
 }
 
 function formatScalarInline(value: unknown, maxLen: number, theme: Theme): string {
@@ -512,8 +511,8 @@ function renderAgentProgress(
 			const completeData = progress.extractedToolData.complete as Array<{ data: unknown }> | undefined;
 			const reportFindingData = progress.extractedToolData.report_finding as ReportFindingDetails[] | undefined;
 			const reviewData = completeData
-				?.map((c) => c.data as SubmitReviewDetails)
-				.filter((d) => d && typeof d === "object" && "overall_correctness" in d);
+				?.map(c => c.data as SubmitReviewDetails)
+				.filter(d => d && typeof d === "object" && "overall_correctness" in d);
 			if (reviewData && reviewData.length > 0) {
 				const summary = reviewData[reviewData.length - 1];
 				const findings = reportFindingData ?? [];
@@ -697,8 +696,8 @@ function renderAgentResult(result: SingleResult, isLast: boolean, expanded: bool
 
 	// Extract review verdict from complete tool's data field if it matches SubmitReviewDetails
 	const reviewData = completeData
-		?.map((c) => c.data as SubmitReviewDetails)
-		.filter((d) => d && typeof d === "object" && "overall_correctness" in d);
+		?.map(c => c.data as SubmitReviewDetails)
+		.filter(d => d && typeof d === "object" && "overall_correctness" in d);
 	const submitReviewData = reviewData && reviewData.length > 0 ? reviewData : undefined;
 
 	if (submitReviewData && submitReviewData.length > 0) {
@@ -775,12 +774,12 @@ export function renderResult(
 	theme: Theme,
 ): Component {
 	const { expanded, isPartial, spinnerFrame } = options;
-	const fallbackText = result.content.find((c) => c.type === "text")?.text ?? "";
+	const fallbackText = result.content.find(c => c.type === "text")?.text ?? "";
 	const details = result.details;
 
 	if (!details) {
 		// Fallback to simple text
-		const text = result.content.find((c) => c.type === "text")?.text || "";
+		const text = result.content.find(c => c.type === "text")?.text || "";
 		return new Text(theme.fg("dim", truncate(text, 100, theme.format.ellipsis)), 0, 0);
 	}
 
@@ -800,8 +799,8 @@ export function renderResult(
 		});
 
 		// Summary line
-		const abortedCount = details.results.filter((r) => r.aborted).length;
-		const successCount = details.results.filter((r) => !r.aborted && r.exitCode === 0).length;
+		const abortedCount = details.results.filter(r => r.aborted).length;
+		const successCount = details.results.filter(r => !r.aborted && r.exitCode === 0).length;
 		const failCount = details.results.length - successCount - abortedCount;
 		let summary = `${theme.fg("dim", "Total:")} `;
 		if (abortedCount > 0) {
@@ -829,7 +828,7 @@ export function renderResult(
 	if (fallbackText.trim()) {
 		const summaryLines = fallbackText.split("\n");
 		const markerIndex = summaryLines.findIndex(
-			(line) => line.includes("<system-notification>") || line.startsWith("Applied patches:"),
+			line => line.includes("<system-notification>") || line.startsWith("Applied patches:"),
 		);
 		if (markerIndex >= 0) {
 			const extra = summaryLines.slice(markerIndex);
@@ -840,7 +839,7 @@ export function renderResult(
 		}
 	}
 
-	const indented = lines.map((line) => (line.length > 0 ? `   ${line}` : ""));
+	const indented = lines.map(line => (line.length > 0 ? `   ${line}` : ""));
 	return new Text(indented.join("\n"), 0, 0);
 }
 

package/src/task/subprocess-tool-registry.ts

@@ -6,9 +6,8 @@
  * - Trigger subprocess termination on completion
  * - Provide custom rendering for realtime/final display
  */
-
-import type { Theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import type { Component } from "@oh-my-pi/pi-tui";
+import type { Theme } from "../modes/theme/theme";
 
 /** Event from subprocess tool execution (parsed from JSONL) */
 export interface SubprocessToolEvent {

package/src/task/worker-protocol.ts

@@ -1,7 +1,7 @@
 import type { AgentEvent, ThinkingLevel } from "@oh-my-pi/pi-agent-core";
-import type { SerializedModelRegistry } from "@oh-my-pi/pi-coding-agent/config/model-registry";
-import type { Settings } from "@oh-my-pi/pi-coding-agent/config/settings-manager";
-import type { SerializedAuthStorage } from "@oh-my-pi/pi-coding-agent/session/auth-storage";
+import type { SerializedModelRegistry } from "../config/model-registry";
+import type { Settings } from "../config/settings-manager";
+import type { SerializedAuthStorage } from "../session/auth-storage";
 
 /**
  * MCP tool metadata passed from parent to worker for proxy tool creation.