@oh-my-pi/pi-coding-agent 13.18.0 → 14.0.2

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

@@ -1,8 +1,7 @@
 import * as fs from "node:fs/promises";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
-import { isEnoent } from "@oh-my-pi/pi-utils";
+import { isEnoent, prompt } from "@oh-my-pi/pi-utils";
 import { type Static, Type } from "@sinclair/typebox";
-import { renderPromptTemplate } from "../config/prompt-templates";
 import exitPlanModeDescription from "../prompts/tools/exit-plan-mode.md" with { type: "text" };
 import type { ToolSession } from ".";
 import { resolvePlanPath } from "./plan-mode-guard";
@@ -46,9 +45,10 @@ export class ExitPlanModeTool implements AgentTool<typeof exitPlanModeSchema, Ex
 	readonly description: string;
 	readonly parameters = exitPlanModeSchema;
 	readonly strict = true;
+	readonly concurrency = "exclusive";
 
 	constructor(private readonly session: ToolSession) {
-		this.description = renderPromptTemplate(exitPlanModeDescription);
+		this.description = prompt.render(exitPlanModeDescription);
 	}
 
 	async execute(

package/src/tools/fetch.ts

@@ -4,7 +4,7 @@ import type { AgentToolResult } from "@oh-my-pi/pi-agent-core";
 import type { ImageContent, TextContent } from "@oh-my-pi/pi-ai";
 import { htmlToMarkdown } from "@oh-my-pi/pi-natives";
 import { type Component, Text } from "@oh-my-pi/pi-tui";
-import { ptree, truncate } from "@oh-my-pi/pi-utils";
+import { $which, ptree, truncate } from "@oh-my-pi/pi-utils";
 import { parseHTML } from "linkedom";
 import type { Settings } from "../config/settings";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
@@ -23,7 +23,7 @@ import { convertWithMarkit, fetchBinary } from "../web/scrapers/utils";
 import { applyListLimit } from "./list-limit";
 import { formatStyledArtifactReference, type OutputMeta } from "./output-meta";
 import { formatExpandHint, getDomain } from "./render-utils";
-import { ToolAbortError } from "./tool-errors";
+import { ToolAbortError, ToolError } from "./tool-errors";
 import { toolResult } from "./tool-result";
 import { clampTimeout } from "./tool-timeouts";
 
@@ -94,7 +94,7 @@ const MAX_INLINE_IMAGE_OUTPUT_BYTES = 0.75 * 1024 * 1024;
  * Check if a command exists (cross-platform)
  */
 function hasCommand(cmd: string): boolean {
-	return Boolean(Bun.which(cmd));
+	return Boolean($which(cmd));
 }
 
 /**
@@ -137,6 +137,70 @@ export function isReadableUrlPath(value: string): boolean {
 	return /^https?:\/\//i.test(value) || /^www\./i.test(value);
 }
 
+const URL_LINE_RANGE_RE = /^L(\d+)(?:-L?(\d+))?$/i;
+
+export interface ParsedReadUrlTarget {
+	path: string;
+	raw: boolean;
+	offset?: number;
+	limit?: number;
+}
+
+export function parseReadUrlTarget(readPath: string, sel?: string): ParsedReadUrlTarget | null {
+	const embedded = sel ? undefined : tryExtractEmbeddedUrlSelector(readPath);
+	const urlPath = embedded?.path ?? readPath;
+	if (!isReadableUrlPath(urlPath)) {
+		return null;
+	}
+
+	const selector = sel ?? embedded?.sel;
+	const raw = selector === "raw";
+	const lineMatch = selector ? URL_LINE_RANGE_RE.exec(selector) : null;
+	if (lineMatch) {
+		const startLine = Number.parseInt(lineMatch[1]!, 10);
+		if (startLine < 1) {
+			throw new ToolError("L0 is invalid; lines are 1-indexed. Use sel=L1.");
+		}
+		const endLine = lineMatch[2] ? Number.parseInt(lineMatch[2], 10) : undefined;
+		if (endLine !== undefined && endLine < startLine) {
+			throw new ToolError(`Invalid range L${startLine}-L${endLine}: end must be >= start.`);
+		}
+		return {
+			path: urlPath,
+			raw: false,
+			offset: startLine,
+			limit: endLine !== undefined ? endLine - startLine + 1 : undefined,
+		};
+	}
+
+	return { path: urlPath, raw };
+}
+
+function tryExtractEmbeddedUrlSelector(readPath: string): { path: string; sel?: string } | null {
+	const lastColonIndex = readPath.lastIndexOf(":");
+	if (lastColonIndex <= 0) {
+		return null;
+	}
+
+	const candidateSelector = readPath.slice(lastColonIndex + 1);
+	const isEmbeddedSelector = candidateSelector === "raw" || URL_LINE_RANGE_RE.test(candidateSelector);
+	if (!isEmbeddedSelector) {
+		return null;
+	}
+
+	const basePath = readPath.slice(0, lastColonIndex);
+	if (!isReadableUrlPath(basePath)) {
+		return null;
+	}
+
+	try {
+		new URL(basePath.startsWith("http://") || basePath.startsWith("https://") ? basePath : `https://${basePath}`);
+		return { path: basePath, sel: candidateSelector };
+	} catch {
+		return null;
+	}
+}
+
 /**
  * Normalize MIME type (lowercase, strip charset/params)
  */

package/src/tools/find.ts

@@ -4,10 +4,9 @@ import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallb
 import { FileType, type GlobMatch, glob } from "@oh-my-pi/pi-natives";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text } from "@oh-my-pi/pi-tui";
-import { isEnoent, untilAborted } from "@oh-my-pi/pi-utils";
+import { isEnoent, prompt, untilAborted } from "@oh-my-pi/pi-utils";
 import type { Static } from "@sinclair/typebox";
 import { Type } from "@sinclair/typebox";
-import { renderPromptTemplate } from "../config/prompt-templates";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
 import type { Theme } from "../modes/theme/theme";
 import findDescription from "../prompts/tools/find.md" with { type: "text" };
@@ -86,7 +85,7 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 		options?: FindToolOptions,
 	) {
 		this.#customOps = options?.operations;
-		this.description = renderPromptTemplate(findDescription);
+		this.description = prompt.render(findDescription);
 	}
 
 	async execute(
@@ -234,8 +233,8 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 				});
 			};
 			const onMatch = onUpdate
-				? (match: GlobMatch | null) => {
-						if (signal?.aborted || !match) return;
+				? (err: Error | null, match: GlobMatch | null) => {
+						if (err || signal?.aborted || !match) return;
 						let relativePath = match.path;
 						if (!relativePath) return;
 						if (match.fileType === FileType.Dir && !relativePath.endsWith("/")) {

package/src/tools/fs-cache-invalidation.ts

@@ -1,10 +1,12 @@
 import { invalidateFsScanCache } from "@oh-my-pi/pi-natives";
+import { invalidateChunkCache } from "../edit/modes/chunk";
 
 /**
  * Invalidate shared filesystem scan caches after a content write/update.
  */
 export function invalidateFsScanAfterWrite(path: string): void {
 	invalidateFsScanCache(path);
+	invalidateChunkCache(path);
 }
 
 /**
@@ -12,6 +14,7 @@ export function invalidateFsScanAfterWrite(path: string): void {
  */
 export function invalidateFsScanAfterDelete(path: string): void {
 	invalidateFsScanCache(path);
+	invalidateChunkCache(path);
 }
 
 /**
@@ -22,7 +25,9 @@ export function invalidateFsScanAfterDelete(path: string): void {
  */
 export function invalidateFsScanAfterRename(oldPath: string, newPath: string): void {
 	invalidateFsScanCache(oldPath);
+	invalidateChunkCache(oldPath);
 	if (newPath !== oldPath) {
 		invalidateFsScanCache(newPath);
+		invalidateChunkCache(newPath);
 	}
 }

package/src/tools/gemini-image.ts

@@ -1,13 +1,20 @@
 import * as os from "node:os";
 import * as path from "node:path";
 import { getAntigravityHeaders, getEnvApiKey, StringEnum } from "@oh-my-pi/pi-ai";
-import { $env, isEnoent, ptree, readSseJson, Snowflake, untilAborted } from "@oh-my-pi/pi-utils";
+import {
+	$env,
+	isEnoent,
+	parseImageMetadata,
+	prompt,
+	ptree,
+	readSseJson,
+	Snowflake,
+	untilAborted,
+} from "@oh-my-pi/pi-utils";
 import { type Static, Type } from "@sinclair/typebox";
 import type { ModelRegistry } from "../config/model-registry";
-import { renderPromptTemplate } from "../config/prompt-templates";
 import type { CustomTool } from "../extensibility/custom-tools/types";
 import geminiImageDescription from "../prompts/tools/gemini-image.md" with { type: "text" };
-import { detectSupportedImageMimeTypeFromFile } from "../utils/mime";
 import { resolveReadPath } from "./path-utils";
 
 const DEFAULT_MODEL = "gemini-3-pro-image-preview";
@@ -417,7 +424,8 @@ async function loadImageFromPath(imagePath: string, cwd: string): Promise<Inline
 			throw new Error(`Image file too large: ${imagePath}`);
 		}
 
-		const mimeType = await detectSupportedImageMimeTypeFromFile(resolved);
+		const metadata = parseImageMetadata(buffer);
+		const mimeType = metadata?.mimeType;
 		if (!mimeType) {
 			throw new Error(`Unsupported image type: ${imagePath}`);
 		}
@@ -599,7 +607,7 @@ async function parseAntigravitySseForImage(response: Response, signal?: AbortSig
 export const geminiImageTool: CustomTool<typeof geminiImageSchema, GeminiImageToolDetails> = {
 	name: "generate_image",
 	label: "GenerateImage",
-	description: renderPromptTemplate(geminiImageDescription),
+	description: prompt.render(geminiImageDescription),
 	parameters: geminiImageSchema,
 	async execute(_toolCallId, params, _onUpdate, ctx, signal) {
 		return untilAborted(signal, async () => {