@oh-my-pi/pi-coding-agent 15.0.0 → 15.0.2

package/src/utils/image-resize.ts

@@ -1,11 +1,10 @@
 import type { ImageContent } from "@oh-my-pi/pi-ai";
-import { ImageFormat, PhotonImage, SamplingFilter } from "@oh-my-pi/pi-natives";
 
 export interface ImageResizeOptions {
-	maxWidth?: number; // Default: 1568
-	maxHeight?: number; // Default: 1568
-	maxBytes?: number; // Default: 500KB
-	jpegQuality?: number; // Default: 75
+	maxWidth?: number;
+	maxHeight?: number;
+	maxBytes?: number;
+	jpegQuality?: number;
 }
 
 export interface ResizedImage {
@@ -24,12 +23,12 @@ export interface ResizedImage {
 const DEFAULT_MAX_BYTES = 500 * 1024;
 
 const DEFAULT_OPTIONS: Required<ImageResizeOptions> = {
-	// 1568px — Anthropic downscales anything larger; OpenAI tiles at 768px;
-	// sending bigger pixels wastes bandwidth the model never sees.
+	// Anthropic's "internal recommended size" — Claude internally caps images at
+	// 1568px on the longest edge before vision processing.
 	maxWidth: 1568,
 	maxHeight: 1568,
 	maxBytes: DEFAULT_MAX_BYTES,
-	jpegQuality: 75,
+	jpegQuality: 80,
 };
 
 /** Pick the smallest of N encoded buffers. */
@@ -48,43 +47,34 @@ Buffer.prototype.toBase64 = function (this: Buffer) {
 /**
  * Resize and recompress an image to fit within the specified max dimensions and file size.
  *
- * Defaults target Anthropic's internal 1568px downscale threshold and produce small
- * lossy JPEG output suitable for tool-call payloads (~100–500KB typical).
- *
  * Strategy:
- * 1. Fast path — if input already fits dimensions AND is at <=25% of byte budget,
- *    return as-is. Avoids re-encoding tiny icons/diagrams.
- * 2. Resize to maxWidth/maxHeight, encode both PNG and JPEG at default quality,
- *    pick whichever is smaller. PNG wins for line art / few-color UI; JPEG wins
- *    for photographic content.
- * 3. If still too large, JPEG-only quality ladder (PNG quality is a no-op).
- * 4. If still too large, progressively reduce dimensions and retry the JPEG ladder.
- * 5. Last resort: ship the smallest variant produced.
+ *  1. Probe metadata. If already within all limits, return original.
+ *  2. Resize to fit max dimensions and encode at high quality across PNG/JPEG/WebP — return smallest.
+ *  3. If still too large, walk a lossy JPEG/WebP quality ladder.
+ *  4. If still too large, walk a dimension-scale ladder × quality ladder.
+ *  5. If still too large, return the smallest variant produced.
  *
- * On any decode failure, returns the original bytes unchanged with wasResized=false.
+ * Backed by `Bun.Image`: a chainable native pipeline that runs decode/transform/encode
+ * off the JS thread when the terminal (`.bytes()`) is awaited.
  */
 export async function resizeImage(img: ImageContent, options?: ImageResizeOptions): Promise<ResizedImage> {
 	const opts = { ...DEFAULT_OPTIONS, ...options };
 	const inputBuffer = Buffer.from(img.data, "base64");
 
 	try {
-		const image = await PhotonImage.parse(inputBuffer);
-
-		const originalWidth = image.width;
-		const originalHeight = image.height;
-		const format = img.mimeType?.split("/")[1] ?? "png";
+		const { width: originalWidth, height: originalHeight, format } = await new Bun.Image(inputBuffer).metadata();
+		const sourceMime = img.mimeType ?? `image/${format}`;
 
-		// Check if already within all limits (dimensions AND size)
-		const originalSize = inputBuffer.length;
-		// Fast path: skip if already within dimensions AND well under budget.
+		// Fast path: already within dimensions AND well under budget.
 		// Threshold is 1/4 of budget — if already that compact, don't re-encode.
 		// Avoids wasted work on tiny icons/diagrams while ensuring larger PNGs
 		// still get JPEG-compressed.
+		const originalSize = inputBuffer.length;
 		const comfortableSize = opts.maxBytes / 4;
 		if (originalWidth <= opts.maxWidth && originalHeight <= opts.maxHeight && originalSize <= comfortableSize) {
 			return {
 				buffer: inputBuffer,
-				mimeType: img.mimeType ?? `image/${format}`,
+				mimeType: sourceMime,
 				originalWidth,
 				originalHeight,
 				width: originalWidth,
@@ -117,14 +107,11 @@ export async function resizeImage(img: ImageContent, options?: ImageResizeOption
 			height: number,
 			quality: number,
 		): Promise<{ buffer: Uint8Array; mimeType: string }> {
-			const resized = await image.resize(width, height, SamplingFilter.Lanczos3);
-
 			const [pngBuffer, jpegBuffer, webpBuffer] = await Promise.all([
-				resized.encode(ImageFormat.PNG, quality),
-				resized.encode(ImageFormat.JPEG, quality),
-				resized.encode(ImageFormat.WEBP, quality),
+				new Bun.Image(inputBuffer).resize(width, height).png().bytes(),
+				new Bun.Image(inputBuffer).resize(width, height).jpeg({ quality }).bytes(),
+				new Bun.Image(inputBuffer).resize(width, height).webp({ quality }).bytes(),
 			]);
-
 			return pickSmallest(
 				{ buffer: pngBuffer, mimeType: "image/png" },
 				{ buffer: jpegBuffer, mimeType: "image/jpeg" },
@@ -140,10 +127,9 @@ export async function resizeImage(img: ImageContent, options?: ImageResizeOption
 			height: number,
 			quality: number,
 		): Promise<{ buffer: Uint8Array; mimeType: string }> {
-			const resized = await image.resize(width, height, SamplingFilter.Lanczos3);
 			const [jpegBuffer, webpBuffer] = await Promise.all([
-				resized.encode(ImageFormat.JPEG, quality),
-				resized.encode(ImageFormat.WEBP, quality),
+				new Bun.Image(inputBuffer).resize(width, height).jpeg({ quality }).bytes(),
+				new Bun.Image(inputBuffer).resize(width, height).webp({ quality }).bytes(),
 			]);
 			return pickSmallest(
 				{ buffer: jpegBuffer, mimeType: "image/jpeg" },
@@ -159,7 +145,7 @@ export async function resizeImage(img: ImageContent, options?: ImageResizeOption
 		let finalWidth = targetWidth;
 		let finalHeight = targetHeight;
 
-		// First attempt: resize to target, try both PNG and JPEG, pick smaller
+		// First attempt: resize to target, try PNG/JPEG/WebP, pick smallest
 		best = await encodeSmallest(targetWidth, targetHeight, opts.jpegQuality);
 
 		if (best.buffer.length <= opts.maxBytes) {
@@ -264,9 +250,12 @@ export function formatDimensionNote(result: ResizedImage): string | undefined {
 	if (!result.wasResized) {
 		return undefined;
 	}
-
+	if (!result.originalWidth || !result.originalHeight || !result.width || !result.height) {
+		return undefined;
+	}
+	if (result.width === result.originalWidth && result.height === result.originalHeight) {
+		return undefined;
+	}
 	const scale = result.originalWidth / result.width;
-	return `[Image: original ${result.originalWidth}x${result.originalHeight}, displayed at ${result.width}x${
-		result.height
-	}. Multiply coordinates by ${scale.toFixed(2)} to map to original image.]`;
+	return `[Image: original ${result.originalWidth}x${result.originalHeight}, displayed at ${result.width}x${result.height}. Multiply coordinates by ${scale.toFixed(2)} to map to original image.]`;
 }

package/src/vim/parser.ts

@@ -124,10 +124,6 @@ export function parseKeySequences(sequences: string[]): VimKeyToken[] {
 	return tokens;
 }
 
-export function tokensToReplay(tokens: readonly VimKeyToken[]): string[] {
-	return tokens.map(token => token.value);
-}
-
 export function replayTokens(values: readonly string[]): VimKeyToken[] {
 	return values.map((value, index) => ({
 		value,
@@ -136,16 +132,3 @@ export function replayTokens(values: readonly string[]): VimKeyToken[] {
 		offset: index,
 	}));
 }
-
-export function formatVimError(error: unknown): string {
-	if (!(error instanceof VimError)) {
-		return error instanceof Error ? error.message : String(error);
-	}
-
-	const base = error.message;
-	if (!error.location) {
-		return base;
-	}
-
-	return `${base} (sequence ${error.location.sequenceIndex + 1}, token ${error.location.offset + 1})`;
-}

package/src/vim/render.ts

@@ -13,7 +13,7 @@ import type {
 
 export const VIM_OPEN_VIEWPORT_LINES = 80;
 export const VIM_DEFAULT_VIEWPORT_LINES = 10;
-export const VIM_TAB_DISPLAY = "→";
+const VIM_TAB_DISPLAY = "→";
 const VIM_INLINE_CURSOR = "▏";
 
 const VIM_VIEWPORT_WIDTH = 140;

package/src/vim/types.ts

@@ -164,7 +164,7 @@ export function clonePosition(position: Position): Position {
 	return { line: position.line, col: position.col };
 }
 
-export function comparePositions(left: Position, right: Position): number {
+function comparePositions(left: Position, right: Position): number {
 	if (left.line !== right.line) {
 		return left.line - right.line;
 	}

package/src/web/search/providers/anthropic.ts

@@ -38,6 +38,7 @@ export interface AnthropicSearchParams {
 	max_tokens?: number;
 	/** Sampling temperature (0–1). Lower = more focused/factual. */
 	temperature?: number;
+	signal?: AbortSignal;
 }
 
 /**
@@ -86,6 +87,7 @@ async function callSearch(
 	systemPrompt?: string,
 	maxTokens?: number,
 	temperature?: number,
+	signal?: AbortSignal,
 ): Promise<AnthropicApiResponse> {
 	const url = buildAnthropicUrl(auth);
 	const headers = buildAnthropicSearchHeaders(auth);
@@ -116,6 +118,7 @@ async function callSearch(
 		method: "POST",
 		headers,
 		body: JSON.stringify(body),
+		signal,
 	});
 
 	if (!response.ok) {
@@ -253,6 +256,7 @@ export async function searchAnthropic(params: AnthropicSearchParams): Promise<Se
 		params.system_prompt,
 		params.max_tokens,
 		params.temperature,
+		params.signal,
 	);
 
 	const result = parseResponse(response);
@@ -281,6 +285,7 @@ export class AnthropicProvider extends SearchProvider {
 			num_results: params.numSearchResults ?? params.limit,
 			max_tokens: params.maxOutputTokens,
 			temperature: params.temperature,
+			signal: params.signal,
 		});
 	}
 }

package/src/web/search/providers/exa.ts

@@ -29,6 +29,7 @@ export interface ExaSearchParams {
 	exclude_domains?: string[];
 	start_published_date?: string;
 	end_published_date?: string;
+	signal?: AbortSignal;
 }
 
 interface ExaSearchResult {
@@ -179,6 +180,7 @@ async function callExaSearch(apiKey: string, params: ExaSearchParams): Promise<E
 			"x-api-key": apiKey,
 		},
 		body: JSON.stringify(body),
+		signal: params.signal,
 	});
 
 	if (!response.ok) {
@@ -259,6 +261,7 @@ export class ExaProvider extends SearchProvider {
 		return searchExa({
 			query: params.query,
 			num_results: params.numSearchResults ?? params.limit,
+			signal: params.signal,
 		});
 	}
 }

package/src/web/search/providers/gemini.ts

@@ -5,15 +5,11 @@
  * Requires OAuth credentials stored in agent.db for provider "google-gemini-cli" or "google-antigravity".
  * Returns synthesized answers with citations and source metadata from grounding chunks.
  */
-import {
-	ANTIGRAVITY_SYSTEM_INSTRUCTION,
-	extractRetryDelay,
-	getAntigravityUserAgent,
-	getGeminiCliHeaders,
-} from "@oh-my-pi/pi-ai";
+import { ANTIGRAVITY_SYSTEM_INSTRUCTION, getAntigravityUserAgent, getGeminiCliHeaders } from "@oh-my-pi/pi-ai";
 import { refreshAntigravityToken } from "@oh-my-pi/pi-ai/utils/oauth/google-antigravity";
 import { refreshGoogleCloudToken } from "@oh-my-pi/pi-ai/utils/oauth/google-gemini-cli";
-import { getAgentDbPath } from "@oh-my-pi/pi-utils";
+import { fetchWithRetry, getAgentDbPath } from "@oh-my-pi/pi-utils";
+
 import { AgentStorage } from "../../../session/agent-storage";
 import type { SearchCitation, SearchResponse, SearchSource } from "../../../web/search/types";
 import { SearchProviderError } from "../../../web/search/types";
@@ -43,6 +39,7 @@ export interface GeminiSearchParams extends GeminiToolParams {
 	max_output_tokens?: number;
 	/** Sampling temperature (0–1). Lower = more focused/factual. */
 	temperature?: number;
+	signal?: AbortSignal;
 }
 
 export function buildGeminiRequestTools(params: GeminiToolParams): Array<Record<string, Record<string, unknown>>> {
@@ -239,6 +236,7 @@ async function callGeminiSearch(
 	maxOutputTokens?: number,
 	temperature?: number,
 	toolParams: GeminiToolParams = {},
+	signal?: AbortSignal,
 ): Promise<{
 	answer: string;
 	sources: SearchSource[];
@@ -303,98 +301,43 @@ async function callGeminiSearch(
 		}
 		(requestBody.request as Record<string, unknown>).generationConfig = generationConfig;
 	}
-	let response: Response | undefined;
-	let rateLimitTimeSpent = 0;
-	let lastError: Error | undefined;
-
-	for (let endpointIndex = 0; endpointIndex < endpoints.length; endpointIndex++) {
-		const url = `${endpoints[endpointIndex]}/v1internal:streamGenerateContent?alt=sse`;
-
-		for (let attempt = 0; attempt <= MAX_RETRIES; attempt++) {
-			try {
-				response = await fetch(url, {
-					method: "POST",
-					headers: {
-						Authorization: `Bearer ${auth.accessToken}`,
-						"Content-Type": "application/json",
-						Accept: "text/event-stream",
-						...headers,
-					},
-					body: JSON.stringify(requestBody),
-				});
-			} catch (error) {
-				if (attempt < MAX_RETRIES) {
-					await Bun.sleep(BASE_DELAY_MS * 2 ** attempt);
-					continue;
-				}
-
-				if (auth.isAntigravity && endpointIndex < endpoints.length - 1) {
-					break;
-				}
-
-				throw error;
-			}
-
-			if (response.ok) {
-				break;
-			}
-
-			const errorText = await response.text();
-			const canRefreshAuth =
-				response.status === 401 ||
-				response.status === 403 ||
-				(response.status === 400 &&
-					/api key not valid|invalid credentials|invalid authentication/i.test(errorText));
-			if (canRefreshAuth && attempt === 0 && (await refreshGeminiAuth(auth))) {
-				continue;
-			}
-			const isRetryableStatus =
-				response.status === 429 ||
-				response.status === 500 ||
-				response.status === 502 ||
-				response.status === 503 ||
-				response.status === 504;
-
-			if (isRetryableStatus && attempt < MAX_RETRIES) {
-				const serverDelay = extractRetryDelay(errorText, response);
-				if (response.status === 429) {
-					if (serverDelay && rateLimitTimeSpent + serverDelay <= RATE_LIMIT_BUDGET_MS) {
-						rateLimitTimeSpent += serverDelay;
-						await Bun.sleep(serverDelay);
-						continue;
-					}
-					if (!serverDelay) {
-						await Bun.sleep(BASE_DELAY_MS * 2 ** attempt);
-						continue;
-					}
-				} else {
-					await Bun.sleep(serverDelay ?? BASE_DELAY_MS * 2 ** attempt);
-					continue;
-				}
-			}
-
-			lastError = new SearchProviderError(
-				"gemini",
-				`Gemini Cloud Code API error (${response.status}): ${errorText}`,
-				response.status,
-			);
-
-			if (auth.isAntigravity && isRetryableStatus && endpointIndex < endpoints.length - 1) {
-				break;
-			}
-
-			throw lastError;
-		}
+	const buildInit = (): RequestInit => ({
+		method: "POST",
+		headers: {
+			Authorization: `Bearer ${auth.accessToken}`,
+			"Content-Type": "application/json",
+			Accept: "text/event-stream",
+			...headers,
+		},
+		body: JSON.stringify(requestBody),
+		signal,
+	});
+	const urlFor = (attempt: number) =>
+		`${endpoints[Math.min(attempt, endpoints.length - 1)]}/v1internal:streamGenerateContent?alt=sse`;
+
+	let response = await fetchWithRetry(urlFor, {
+		...buildInit(),
+		maxAttempts: MAX_RETRIES + 1,
+		defaultDelayMs: attempt => BASE_DELAY_MS * 2 ** attempt,
+		maxDelayMs: RATE_LIMIT_BUDGET_MS,
+	});
 
-		if (response?.ok) {
-			break;
+	if (!response.ok) {
+		const errorText = await response.clone().text();
+		const canRefreshAuth =
+			response.status === 401 ||
+			response.status === 403 ||
+			(response.status === 400 && /api key not valid|invalid credentials|invalid authentication/i.test(errorText));
+		if (canRefreshAuth && (await refreshGeminiAuth(auth))) {
+			response = await fetchWithRetry(urlFor, {
+				...buildInit(),
+				maxAttempts: MAX_RETRIES + 1,
+				defaultDelayMs: attempt => BASE_DELAY_MS * 2 ** attempt,
+				maxDelayMs: RATE_LIMIT_BUDGET_MS,
+			});
 		}
 	}
 
-	if (!response) {
-		throw new SearchProviderError("gemini", "Gemini API request failed", 500);
-	}
-
 	if (!response.ok) {
 		const errorText = await response.text();
 		throw new SearchProviderError(
@@ -560,6 +503,7 @@ export async function searchGemini(params: GeminiSearchParams): Promise<SearchRe
 			code_execution: params.code_execution,
 			url_context: params.url_context,
 		},
+		params.signal,
 	);
 
 	let sources = result.sources;
@@ -599,6 +543,7 @@ export class GeminiProvider extends SearchProvider {
 			google_search: params.googleSearch,
 			code_execution: params.codeExecution,
 			url_context: params.urlContext,
+			signal: params.signal,
 		});
 	}
 }

package/src/web/search/providers/jina.ts

@@ -17,6 +17,7 @@ const JINA_SEARCH_URL = "https://s.jina.ai";
 export interface JinaSearchParams {
 	query: string;
 	num_results?: number;
+	signal?: AbortSignal;
 }
 
 interface JinaSearchResult {
@@ -33,13 +34,14 @@ export function findApiKey(): string | null {
 }
 
 /** Call Jina Reader search API. */
-async function callJinaSearch(apiKey: string, query: string): Promise<JinaSearchResponse> {
+async function callJinaSearch(apiKey: string, query: string, signal?: AbortSignal): Promise<JinaSearchResponse> {
 	const requestUrl = `${JINA_SEARCH_URL}/${encodeURIComponent(query)}`;
 	const response = await fetch(requestUrl, {
 		headers: {
 			Accept: "application/json",
 			Authorization: `Bearer ${apiKey}`,
 		},
+		signal,
 	});
 
 	if (!response.ok) {
@@ -58,7 +60,7 @@ export async function searchJina(params: JinaSearchParams): Promise<SearchRespon
 		throw new Error("JINA_API_KEY not found. Set it in environment or .env file.");
 	}
 
-	const response = await callJinaSearch(apiKey, params.query);
+	const response = await callJinaSearch(apiKey, params.query, params.signal);
 	const sources: SearchSource[] = [];
 
 	for (const result of response) {
@@ -91,6 +93,7 @@ export class JinaProvider extends SearchProvider {
 		return searchJina({
 			query: params.query,
 			num_results: params.numSearchResults ?? params.limit,
+			signal: params.signal,
 		});
 	}
 }

package/src/web/search/providers/zai.ts

@@ -20,6 +20,7 @@ const DEFAULT_NUM_RESULTS = 10;
 export interface ZaiSearchParams {
 	query: string;
 	num_results?: number;
+	signal?: AbortSignal;
 }
 
 interface ZaiSearchResult {
@@ -55,7 +56,7 @@ export async function findApiKey(): Promise<string | null> {
 	return findCredential(getEnvApiKey("zai"), "zai");
 }
 
-async function callZaiTool(apiKey: string, args: Record<string, unknown>): Promise<unknown> {
+async function callZaiTool(apiKey: string, args: Record<string, unknown>, signal?: AbortSignal): Promise<unknown> {
 	const response = await fetch(ZAI_MCP_URL, {
 		method: "POST",
 		headers: {
@@ -72,6 +73,7 @@ async function callZaiTool(apiKey: string, args: Record<string, unknown>): Promi
 				arguments: args,
 			},
 		}),
+		signal,
 	});
 
 	if (!response.ok) {
@@ -157,7 +159,7 @@ async function callZaiSearch(apiKey: string, params: ZaiSearchParams): Promise<u
 	let lastError: unknown;
 	for (let i = 0; i < attempts.length; i++) {
 		try {
-			return await callZaiTool(apiKey, attempts[i]);
+			return await callZaiTool(apiKey, attempts[i], params.signal);
 		} catch (error) {
 			lastError = error;
 			const isLastAttempt = i === attempts.length - 1;
@@ -302,6 +304,7 @@ export class ZaiProvider extends SearchProvider {
 		return searchZai({
 			query: params.query,
 			num_results: params.numSearchResults ?? params.limit,
+			signal: params.signal,
 		});
 	}
 }

package/src/prompts/tools/exit-plan-mode.md

@@ -1,6 +0,0 @@
-Submits a finalized implementation plan for user approval.
-
-Write the plan to `local://PLAN.md` first, then call this with `title` (e.g. `WP_MIGRATION_PLAN`); on approval the file is renamed to `local://<title>.md` and full tool access is restored.
-- Use only after planning implementation steps; not for pure research.
-- NEVER call before the plan file exists.
-- NEVER use `ask` to request plan approval — this tool does that.

package/src/tools/exit-plan-mode.ts

@@ -1,97 +0,0 @@
-import * as fs from "node:fs/promises";
-import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
-import { isEnoent, prompt } from "@oh-my-pi/pi-utils";
-import { type Static, Type } from "@sinclair/typebox";
-import exitPlanModeDescription from "../prompts/tools/exit-plan-mode.md" with { type: "text" };
-import type { ToolSession } from ".";
-import { resolvePlanPath } from "./plan-mode-guard";
-import { ToolError } from "./tool-errors";
-
-const exitPlanModeSchema = Type.Object({
-	title: Type.String({ description: "final plan title", examples: ["WP_MIGRATION_PLAN"] }),
-});
-
-type ExitPlanModeParams = Static<typeof exitPlanModeSchema>;
-
-function normalizePlanTitle(title: string): { title: string; fileName: string } {
-	const trimmed = title.trim();
-	if (!trimmed) {
-		throw new ToolError("Title is required and must not be empty.");
-	}
-
-	if (trimmed.includes("/") || trimmed.includes("\\") || trimmed.includes("..")) {
-		throw new ToolError("Title must not contain path separators or '..'.");
-	}
-
-	const withExtension = trimmed.toLowerCase().endsWith(".md") ? trimmed : `${trimmed}.md`;
-	if (!/^[A-Za-z0-9_-]+\.md$/.test(withExtension)) {
-		throw new ToolError("Title may only contain letters, numbers, underscores, or hyphens.");
-	}
-
-	const normalizedTitle = withExtension.slice(0, -3);
-	return { title: normalizedTitle, fileName: withExtension };
-}
-
-export interface ExitPlanModeDetails {
-	planFilePath: string;
-	planExists: boolean;
-	title: string;
-	finalPlanFilePath: string;
-}
-
-export class ExitPlanModeTool implements AgentTool<typeof exitPlanModeSchema, ExitPlanModeDetails> {
-	readonly name = "exit_plan_mode";
-	readonly label = "ExitPlanMode";
-	readonly description: string;
-	readonly parameters = exitPlanModeSchema;
-	readonly strict = true;
-	readonly concurrency = "exclusive";
-	readonly intent = (): string => "present plan";
-
-	constructor(private readonly session: ToolSession) {
-		this.description = prompt.render(exitPlanModeDescription);
-	}
-
-	async execute(
-		_toolCallId: string,
-		params: ExitPlanModeParams,
-		_signal?: AbortSignal,
-		_onUpdate?: AgentToolUpdateCallback<ExitPlanModeDetails>,
-		_context?: AgentToolContext,
-	): Promise<AgentToolResult<ExitPlanModeDetails>> {
-		const state = this.session.getPlanModeState?.();
-		if (!state?.enabled) {
-			throw new ToolError("Plan mode is not active.");
-		}
-
-		const normalized = normalizePlanTitle(params.title);
-		const finalPlanFilePath = `local://${normalized.fileName}`;
-		const resolvedPlanPath = resolvePlanPath(this.session, state.planFilePath);
-		resolvePlanPath(this.session, finalPlanFilePath);
-		let planExists = false;
-		try {
-			const stat = await fs.stat(resolvedPlanPath);
-			planExists = stat.isFile();
-		} catch (error) {
-			if (!isEnoent(error)) {
-				throw error;
-			}
-		}
-
-		if (!planExists) {
-			throw new ToolError(
-				`Plan file not found at ${state.planFilePath}. Write the finalized plan to ${state.planFilePath} before calling exit_plan_mode.`,
-			);
-		}
-
-		return {
-			content: [{ type: "text", text: "Plan ready for approval." }],
-			details: {
-				planFilePath: state.planFilePath,
-				planExists,
-				title: normalized.title,
-				finalPlanFilePath,
-			},
-		};
-	}
-}