@oh-my-pi/pi-coding-agent 3.21.0 → 3.24.0

package/src/core/tools/lsp/render.ts

@@ -237,10 +237,7 @@ function renderDiagnostics(
 			}
 			const severityColor = severityToColor(item.severity);
 			const location = formatDiagnosticLocation(item.file, item.line, item.col, theme);
-			output += `\n ${theme.fg("dim", branch)} ${theme.fg(severityColor, location)} ${theme.fg(
-				"dim",
-				`[${item.severity}]`,
-			)}`;
+			output += `\n ${theme.fg("dim", branch)} ${theme.fg(severityColor, location)} ${theme.fg("dim", `[${item.severity}]`)}`;
 			if (item.message) {
 				output += `\n ${theme.fg("dim", detailPrefix)}${theme.fg(
 					"muted",
@@ -274,10 +271,7 @@ function renderDiagnostics(
 		output += `\n ${theme.fg("dim", branch)} ${theme.fg(severityColor, location)}${message}`;
 	}
 	if (remaining > 0) {
-		output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg(
-			"muted",
-			`${theme.format.ellipsis} ${remaining} more`,
-		)}`;
+		output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", `${theme.format.ellipsis} ${remaining} more`)}`;
 	}
 
 	return new Text(output, 0, 0);
@@ -332,10 +326,7 @@ function renderReferences(refMatch: RegExpMatchArray, lines: string[], expanded:
 					const isLastLoc = li === locsToShow.length - 1 && locs.length <= maxLocsPerFile;
 					const locBranch = isLastLoc ? theme.tree.last : theme.tree.branch;
 					const locCont = isLastLoc ? "   " : `${theme.tree.vertical}  `;
-					output += `\n ${theme.fg("dim", fileCont)}${theme.fg("dim", locBranch)} ${theme.fg(
-						"muted",
-						`line ${line}, col ${col}`,
-					)}`;
+					output += `\n ${theme.fg("dim", fileCont)}${theme.fg("dim", locBranch)} ${theme.fg("muted", `line ${line}, col ${col}`)}`;
 					if (expanded) {
 						const context = `at ${file}:${line}:${col}`;
 						output += `\n ${theme.fg("dim", fileCont)}${theme.fg("dim", locCont)}${theme.fg(
@@ -354,10 +345,7 @@ function renderReferences(refMatch: RegExpMatchArray, lines: string[], expanded:
 		}
 
 		if (files.length > maxFiles) {
-			output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg(
-				"muted",
-				formatMoreItems(files.length - maxFiles, "file", theme),
-			)}`;
+			output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", formatMoreItems(files.length - maxFiles, "file", theme))}`;
 		}
 
 		return output;
@@ -463,10 +451,7 @@ function renderSymbols(symbolsMatch: RegExpMatchArray, lines: string[], expanded
 		)}`;
 	}
 	if (topLevelCount > 3) {
-		output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg(
-			"muted",
-			`${theme.format.ellipsis} ${topLevelCount - 3} more`,
-		)}`;
+		output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", `${theme.format.ellipsis} ${topLevelCount - 3} more`)}`;
 	}
 
 	return new Text(output, 0, 0);
@@ -502,10 +487,7 @@ function renderGeneric(text: string, lines: string[], expanded: boolean, theme:
 
 	const firstLine = lines[0] || "No output";
 	const expandHint = formatExpandHint(false, lines.length > 1, theme);
-	let output = `${icon} ${theme.fg(
-		"dim",
-		truncate(firstLine, TRUNCATE_LENGTHS.TITLE, theme.format.ellipsis),
-	)}${expandHint}`;
+	let output = `${icon} ${theme.fg("dim", truncate(firstLine, TRUNCATE_LENGTHS.TITLE, theme.format.ellipsis))}${expandHint}`;
 
 	if (lines.length > 1) {
 		const previewLines = lines.slice(1, 4);
@@ -518,10 +500,7 @@ function renderGeneric(text: string, lines: string[], expanded: boolean, theme:
 			)}`;
 		}
 		if (lines.length > 4) {
-			output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg(
-				"muted",
-				formatMoreItems(lines.length - 4, "line", theme),
-			)}`;
+			output += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", formatMoreItems(lines.length - 4, "line", theme))}`;
 		}
 	}
 

package/src/core/tools/notebook.ts

@@ -4,6 +4,7 @@ import { Text } from "@oh-my-pi/pi-tui";
 import { Type } from "@sinclair/typebox";
 import type { Theme } from "../../modes/interactive/theme/theme";
 import type { RenderResultOptions } from "../custom-tools/types";
+import type { ToolSession } from "../sdk";
 import { untilAborted } from "../utils";
 import { resolveToCwd } from "./path-utils";
 import {
@@ -61,7 +62,7 @@ function splitIntoLines(content: string): string[] {
 	return content.split("\n").map((line, i, arr) => (i < arr.length - 1 ? `${line}\n` : line));
 }
 
-export function createNotebookTool(cwd: string): AgentTool<typeof notebookSchema> {
+export function createNotebookTool(session: ToolSession): AgentTool<typeof notebookSchema> {
 	return {
 		name: "notebook",
 		label: "Notebook",
@@ -79,7 +80,7 @@ export function createNotebookTool(cwd: string): AgentTool<typeof notebookSchema
 			}: { action: string; notebook_path: string; cell_index: number; content?: string; cell_type?: string },
 			signal?: AbortSignal,
 		) => {
-			const absolutePath = resolveToCwd(notebook_path, cwd);
+			const absolutePath = resolveToCwd(notebook_path, session.cwd);
 
 			return untilAborted(signal, async () => {
 				// Check if file exists
@@ -190,9 +191,6 @@ export function createNotebookTool(cwd: string): AgentTool<typeof notebookSchema
 	};
 }
 
-/** Default notebook tool using process.cwd() */
-export const notebookTool = createNotebookTool(process.cwd());
-
 // =============================================================================
 // TUI Renderer
 // =============================================================================

package/src/core/tools/output.ts

@@ -14,7 +14,7 @@ import { Type } from "@sinclair/typebox";
 import type { Theme } from "../../modes/interactive/theme/theme";
 import outputDescription from "../../prompts/tools/output.md" with { type: "text" };
 import type { RenderResultOptions } from "../custom-tools/types";
-import type { SessionContext } from "./index";
+import type { ToolSession } from "./index";
 import {
 	formatCount,
 	formatEmptyMessage,
@@ -120,10 +120,7 @@ function extractPreviewLines(content: string, maxLines: number): string[] {
 	return preview;
 }
 
-export function createOutputTool(
-	_cwd: string,
-	sessionContext?: SessionContext,
-): AgentTool<typeof outputSchema, OutputToolDetails> {
+export function createOutputTool(session: ToolSession): AgentTool<typeof outputSchema, OutputToolDetails> {
 	return {
 		name: "output",
 		label: "Output",
@@ -133,7 +130,7 @@ export function createOutputTool(
 			_toolCallId: string,
 			params: { ids: string[]; format?: "raw" | "json" | "stripped"; offset?: number; limit?: number },
 		): Promise<{ content: TextContent[]; details: OutputToolDetails }> => {
-			const sessionFile = sessionContext?.getSessionFile();
+			const sessionFile = session.getSessionFile();
 
 			if (!sessionFile) {
 				return {
@@ -253,9 +250,6 @@ export function createOutputTool(
 	};
 }
 
-/** Default output tool using process.cwd() - for backwards compatibility */
-export const outputTool = createOutputTool(process.cwd());
-
 // =============================================================================
 // TUI Renderer
 // =============================================================================
@@ -309,15 +303,9 @@ export const outputToolRenderer = {
 			const icon = uiTheme.styledSymbol("status.error", "error");
 			let text = `${icon} ${uiTheme.fg("error", `Error: Not found: ${details.notFound.join(", ")}`)}`;
 			if (details.availableIds?.length) {
-				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg(
-					"muted",
-					`Available: ${details.availableIds.join(", ")}`,
-				)}`;
+				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg("muted", `Available: ${details.availableIds.join(", ")}`)}`;
 			} else {
-				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg(
-					"muted",
-					"No outputs available in current session",
-				)}`;
+				text += `\n ${uiTheme.fg("dim", uiTheme.tree.last)} ${uiTheme.fg("muted", "No outputs available in current session")}`;
 			}
 			return new Text(text, 0, 0);
 		}

package/src/core/tools/read.ts

@@ -12,6 +12,7 @@ import { formatDimensionNote, resizeImage } from "../../utils/image-resize";
 import { detectSupportedImageMimeTypeFromFile } from "../../utils/mime";
 import { ensureTool } from "../../utils/tools-manager";
 import type { RenderResultOptions } from "../custom-tools/types";
+import type { ToolSession } from "../sdk";
 import { untilAborted } from "../utils";
 import { createLsTool } from "./ls";
 import { resolveReadPath, resolveToCwd } from "./path-utils";
@@ -340,14 +341,9 @@ export interface ReadToolDetails {
 	redirectedTo?: "ls";
 }
 
-export interface ReadToolOptions {
-	/** Whether to auto-resize images to 2000x2000 max. Default: true */
-	autoResizeImages?: boolean;
-}
-
-export function createReadTool(cwd: string, options?: ReadToolOptions): AgentTool<typeof readSchema> {
-	const autoResizeImages = options?.autoResizeImages ?? true;
-	const lsTool = createLsTool(cwd);
+export function createReadTool(session: ToolSession): AgentTool<typeof readSchema> {
+	const autoResizeImages = session.settings?.getImageAutoResize() ?? true;
+	const lsTool = createLsTool(session);
 	return {
 		name: "read",
 		label: "Read",
@@ -358,7 +354,7 @@ export function createReadTool(cwd: string, options?: ReadToolOptions): AgentToo
 			{ path: readPath, offset, limit }: { path: string; offset?: number; limit?: number },
 			signal?: AbortSignal,
 		) => {
-			const absolutePath = resolveReadPath(readPath, cwd);
+			const absolutePath = resolveReadPath(readPath, session.cwd);
 
 			return untilAborted(signal, async () => {
 				let isDirectory = false;
@@ -378,14 +374,12 @@ export function createReadTool(cwd: string, options?: ReadToolOptions): AgentToo
 					}
 				} catch (error) {
 					if (isNotFoundError(error)) {
-						const suggestions = await findReadPathSuggestions(readPath, cwd);
+						const suggestions = await findReadPathSuggestions(readPath, session.cwd);
 						let message = `File not found: ${readPath}`;
 
 						if (suggestions?.suggestions.length) {
 							const scopeLabel = suggestions.scopeLabel ? ` in ${suggestions.scopeLabel}` : "";
-							message += `\n\nClosest matches${scopeLabel}:\n${suggestions.suggestions
-								.map((match) => `- ${match}`)
-								.join("\n")}`;
+							message += `\n\nClosest matches${scopeLabel}:\n${suggestions.suggestions.map((match) => `- ${match}`).join("\n")}`;
 							if (suggestions.truncated) {
 								message += `\n[Search truncated to first ${MAX_FUZZY_CANDIDATES} paths. Refine the path if the match isn't listed.]`;
 							}
@@ -462,9 +456,7 @@ export function createReadTool(cwd: string, options?: ReadToolOptions): AgentToo
 						let outputText = truncation.content;
 
 						if (truncation.truncated) {
-							outputText += `\n\n[Document converted via markitdown. Output truncated to ${formatSize(
-								DEFAULT_MAX_BYTES,
-							)}]`;
+							outputText += `\n\n[Document converted via markitdown. Output truncated to ${formatSize(DEFAULT_MAX_BYTES)}]`;
 							details = { truncation };
 						}
 
@@ -562,9 +554,6 @@ export function createReadTool(cwd: string, options?: ReadToolOptions): AgentToo
 	};
 }
 
-/** Default read tool using process.cwd() - for backwards compatibility */
-export const readTool = createReadTool(process.cwd());
-
 // =============================================================================
 // TUI Renderer
 // =============================================================================

package/src/core/tools/review.ts

@@ -19,13 +19,6 @@ const PRIORITY_LABELS: Record<number, string> = {
 	3: "P3",
 };
 
-const _PRIORITY_DESCRIPTIONS: Record<number, string> = {
-	0: "Drop everything to fix. Blocking release, operations, or major usage.",
-	1: "Urgent. Should be addressed in the next cycle.",
-	2: "Normal. To be fixed eventually.",
-	3: "Low. Nice to have.",
-};
-
 // report_finding schema
 const ReportFindingParams = Type.Object({
 	title: Type.String({
@@ -62,8 +55,6 @@ export const reportFindingTool: AgentTool<typeof ReportFindingParams, ReportFind
 	label: "Report Finding",
 	description: "Report a code review finding. Use this for each issue found. Call submit_review when done.",
 	parameters: ReportFindingParams,
-	hidden: true,
-
 	async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
 		const { title, body, priority, confidence, file_path, line_start, line_end } = params;
 		const location = `${file_path}:${line_start}${line_end !== line_start ? `-${line_end}` : ""}`;
@@ -142,7 +133,6 @@ export const submitReviewTool: AgentTool<typeof SubmitReviewParams, SubmitReview
 	label: "Submit Review",
 	description: "Submit the final review verdict. Call this after all findings have been reported.",
 	parameters: SubmitReviewParams,
-	hidden: true,
 
 	async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
 		const { overall_correctness, explanation, confidence } = params;
@@ -206,14 +196,6 @@ export const submitReviewTool: AgentTool<typeof SubmitReviewParams, SubmitReview
 	},
 };
 
-export function createReportFindingTool(): AgentTool<typeof ReportFindingParams, ReportFindingDetails, Theme> {
-	return reportFindingTool;
-}
-
-export function createSubmitReviewTool(): AgentTool<typeof SubmitReviewParams, SubmitReviewDetails, Theme> {
-	return submitReviewTool;
-}
-
 // Re-export types for external use
 export type { ReportFindingDetails, SubmitReviewDetails };
 

package/src/core/tools/rulebook.ts

@@ -9,6 +9,7 @@
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
 import { Type } from "@sinclair/typebox";
 import type { Rule } from "../../capability/rule";
+import type { ToolSession } from "./index";
 
 export interface RulebookToolDetails {
 	type: "rulebook";
@@ -23,9 +24,14 @@ const rulebookSchema = Type.Object({
 
 /**
  * Create a rulebook tool with access to discovered rules.
- * @param rules - Array of discovered rules (non-TTSR rules with descriptions)
+ * Returns null if no rules available.
  */
-export function createRulebookTool(rules: Rule[]): AgentTool<typeof rulebookSchema> {
+export function createRulebookTool(session: ToolSession): AgentTool<typeof rulebookSchema> | null {
+	const rules = session.rulebookRules;
+	if (!rules || rules.length === 0) {
+		return null;
+	}
+
 	// Build lookup map for O(1) access
 	const ruleMap = new Map<string, Rule>();
 	for (const rule of rules) {

package/src/core/tools/task/agents.ts

@@ -5,7 +5,6 @@
  */
 
 // Embed agent markdown files at build time
-import browserMd from "../../../prompts/browser.md" with { type: "text" };
 import exploreMd from "../../../prompts/explore.md" with { type: "text" };
 import planMd from "../../../prompts/plan.md" with { type: "text" };
 import reviewerMd from "../../../prompts/reviewer.md" with { type: "text" };
@@ -13,11 +12,37 @@ import taskMd from "../../../prompts/task.md" with { type: "text" };
 import type { AgentDefinition, AgentSource } from "./types";
 
 const EMBEDDED_AGENTS: { name: string; content: string }[] = [
-	{ name: "browser.md", content: browserMd },
 	{ name: "explore.md", content: exploreMd },
 	{ name: "plan.md", content: planMd },
 	{ name: "reviewer.md", content: reviewerMd },
-	{ name: "task.md", content: taskMd },
+	{
+		name: "task.md",
+		content: `---
+name: task
+description: General-purpose subagent with full capabilities for delegated multi-step tasks
+spawns: explore
+model: default
+---
+${taskMd}`,
+	},
+	{
+		name: "quick_task.md",
+		content: `---
+name: quick_task
+description: Quick task for fast execution
+model: pi/smol
+---
+${taskMd}`,
+	},
+	{
+		name: "deep_task.md",
+		content: `---
+name: deep_task
+description: Deep task for comprehensive reasoning
+model: pi/slow
+---
+${taskMd}`,
+	},
 ];
 
 /**
@@ -88,16 +113,12 @@ function parseAgent(fileName: string, content: string, source: AgentSource): Age
 		spawns = "*";
 	}
 
-	const recursive =
-		frontmatter.recursive === undefined ? false : frontmatter.recursive === "true" || frontmatter.recursive === "1";
-
 	return {
 		name: frontmatter.name,
 		description: frontmatter.description,
 		tools: tools && tools.length > 0 ? tools : undefined,
 		spawns,
 		model: frontmatter.model,
-		recursive,
 		systemPrompt: body,
 		source,
 		filePath: `embedded:${fileName}`,

package/src/core/tools/task/discovery.ts

@@ -126,18 +126,12 @@ function loadAgentsFromDir(dir: string, source: AgentSource): AgentDefinition[]
 			spawns = "*";
 		}
 
-		const recursive =
-			frontmatter.recursive === undefined
-				? undefined
-				: frontmatter.recursive === "true" || frontmatter.recursive === "1";
-
 		agents.push({
 			name: frontmatter.name,
 			description: frontmatter.description,
 			tools: tools && tools.length > 0 ? tools : undefined,
 			spawns,
 			model: frontmatter.model,
-			recursive,
 			systemPrompt: body,
 			source,
 			filePath,