@oh-my-pi/pi-coding-agent 15.1.1 → 15.1.3

package/src/edit/modes/patch.ts

@@ -1578,16 +1578,16 @@ export async function computePatchDiff(
 
 export const patchEditEntrySchema = z
 	.object({
-		op: z.enum(["create", "delete", "update"]).optional().describe("Operation (default: update)"),
-		rename: z.string().describe("New path for move").optional(),
-		diff: z.string().describe("Diff hunks (update) or full content (create)").optional(),
+		op: z.enum(["create", "delete", "update"]).optional().describe("operation (default update)"),
+		rename: z.string().describe("new path for move").optional(),
+		diff: z.string().describe("diff hunks or full content for create").optional(),
 	})
 	.strict();
 
 export const patchEditSchema = z
 	.object({
-		path: z.string().describe("file path for edits"),
-		edits: z.array(patchEditEntrySchema).min(1).describe("Patch operations"),
+		path: z.string().describe("file path"),
+		edits: z.array(patchEditEntrySchema).min(1).describe("patch operations"),
 	})
 	.strict();
 

package/src/edit/modes/replace.ts

@@ -978,16 +978,16 @@ export function findContextLine(
 
 export const replaceEditEntrySchema = z
 	.object({
-		old_text: z.string().describe("Text to find (fuzzy whitespace matching enabled)"),
-		new_text: z.string().describe("Replacement text"),
-		all: z.boolean().describe("Replace all occurrences (default: unique match required)").optional(),
+		old_text: z.string().describe("text to find"),
+		new_text: z.string().describe("replacement text"),
+		all: z.boolean().describe("replace all occurrences").optional(),
 	})
 	.strict();
 
 export const replaceEditSchema = z
 	.object({
-		path: z.string().describe("file path for edits"),
-		edits: z.array(replaceEditEntrySchema).min(1).describe("Replacements"),
+		path: z.string().describe("file path"),
+		edits: z.array(replaceEditEntrySchema).min(1).describe("replacements"),
 	})
 	.strict();
 

package/src/edit/normalize.ts

@@ -23,7 +23,7 @@ export function detectLineEnding(content: string): LineEnding {
 
 /** Normalize all line endings to LF */
 export function normalizeToLF(text: string): string {
-	return text.replace(/\r\n/g, "\n").replace(/\r/g, "\n");
+	return text.replace(/\r\n?/g, "\n");
 }
 
 /** Restore line endings to the specified type */

package/src/edit/renderer.ts

@@ -1,9 +1,10 @@
 /**
  * Edit tool renderer and LSP batching helpers.
  */
-import { sanitizeText } from "@oh-my-pi/pi-natives";
+
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Text, visibleWidth, wrapTextWithAnsi } from "@oh-my-pi/pi-tui";
+import { sanitizeText } from "@oh-my-pi/pi-utils";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
 import type { FileDiagnosticsResult } from "../lsp";
 import { renderDiff as renderDiffColored } from "../modes/components/diff";

package/src/edit/streaming.ts

@@ -13,7 +13,7 @@
  * the injected `editMode` rather than probing argument shape.
  */
 
-import { sanitizeText } from "@oh-my-pi/pi-natives";
+import { sanitizeText } from "@oh-my-pi/pi-utils";
 import {
 	ABORT_MARKER,
 	BEGIN_PATCH_MARKER,

package/src/eval/index.ts

@@ -1,6 +1,4 @@
 export * from "./backend";
 export { default as jsBackend } from "./js";
-export * from "./parse";
 export { default as pythonBackend } from "./py";
-export * from "./sniff";
 export * from "./types";

package/src/eval/js/shared/runtime.ts

@@ -137,6 +137,13 @@ export class JsRuntime {
 			},
 			__omp_import__: async (source: string, options?: ImportCallOptions) => {
 				const target = resolveImportSpecifier(this.#cwd, source);
+				// Always invalidate cached module records for user-owned source files so edits
+				// between cells are picked up. Bun ignores query-string busting on `file:` URLs
+				// but honors `delete require.cache[absPath]`; bare specifiers and URL schemes are
+				// left alone to keep package identity stable across cells.
+				if (isLocalPathSpecifier(source) && path.isAbsolute(target)) {
+					delete require.cache[target];
+				}
 				return options !== undefined ? await import(target, options) : await import(target);
 			},
 			__omp_emit_status__: (op: string, data: Record<string, unknown> = {}) => {
@@ -193,3 +200,21 @@ function resolveImportSpecifier(cwd: string, source: string): string {
 		return source;
 	}
 }
+
+/**
+ * Returns true when the original specifier is a relative or absolute filesystem path
+ * (i.e. user-owned source the agent is iterating on). Bare specifiers and URL schemes
+ * are excluded — `node:` built-ins cannot be reloaded, and busting bare packages would
+ * defeat module identity for every cell while bringing no editing benefit.
+ */
+function isLocalPathSpecifier(source: string): boolean {
+	return (
+		source.startsWith("./") ||
+		source.startsWith("../") ||
+		source === "." ||
+		source === ".." ||
+		source.startsWith("/") ||
+		source.startsWith("~/") ||
+		/^[a-zA-Z]:[\\/]/.test(source)
+	);
+}

package/src/eval/py/kernel.ts

@@ -626,7 +626,7 @@ export class PythonKernel {
 		const exitedPromise = this.#exitedPromise;
 		const timeout = new Promise<null>(resolve => {
 			const timer = setTimeout(() => resolve(null), Math.max(0, timeoutMs));
-			(timer as { unref?: () => void }).unref?.();
+			timer.unref?.();
 		});
 		return Promise.race([exitedPromise.then(code => code as number | null), timeout]);
 	}

package/src/exa/researcher.ts

@@ -14,9 +14,9 @@ const researcherStartTool = createExaTool(
 	"Start Deep Research",
 	"Start an asynchronous deep research task using Exa's researcher. Returns a task_id for polling completion.",
 	z.object({
-		query: z.string().describe("Research query to investigate"),
-		depth: z.number().int().min(1).max(5).describe("Research depth (1-5, default: 3)").optional(),
-		breadth: z.number().int().min(1).max(5).describe("Research breadth (1-5, default: 3)").optional(),
+		query: z.string().describe("research query"),
+		depth: z.number().int().min(1).max(5).describe("research depth (1-5)").optional(),
+		breadth: z.number().int().min(1).max(5).describe("research breadth (1-5)").optional(),
 	}),
 	"deep_researcher_start",
 	{ formatResponse: false },
@@ -27,7 +27,7 @@ const researcherPollTool = createExaTool(
 	"Poll Research Status",
 	"Poll the status of an asynchronous research task. Returns status (pending|running|completed|failed) and result if completed.",
 	z.object({
-		task_id: z.string().describe("Task ID returned from exa_researcher_start"),
+		task_id: z.string().describe("task id"),
 	}),
 	"deep_researcher_check",
 	{ formatResponse: false },

package/src/exa/search.ts

@@ -30,28 +30,16 @@ Parameters:
 - num_results: Maximum number of results to return (default: 10, max: 100)`,
 
 	z.object({
-		query: z.string().describe("Search query"),
-		type: z
-			.enum(["keyword", "neural", "auto"])
-			.describe("Search type - neural (semantic), keyword (exact), or auto")
-			.optional(),
-		include_domains: z.array(z.string()).describe("Only include results from these domains").optional(),
-		exclude_domains: z.array(z.string()).describe("Exclude results from these domains").optional(),
-		start_published_date: z
-			.string()
-			.describe("Filter results published after this date (ISO 8601 format)")
-			.optional(),
-		end_published_date: z.string().describe("Filter results published before this date (ISO 8601 format)").optional(),
-		use_autoprompt: z.boolean().describe("Let Exa optimize your query automatically (default: true)").optional(),
-		text: z.boolean().describe("Include page text content in results (costs more, default: false)").optional(),
-		highlights: z.boolean().describe("Include highlighted relevant snippets (default: false)").optional(),
-		num_results: z
-			.number()
-			.int()
-			.min(1)
-			.max(100)
-			.describe("Maximum number of results to return (default: 10, max: 100)")
-			.optional(),
+		query: z.string().describe("search query"),
+		type: z.enum(["keyword", "neural", "auto"]).describe("search type").optional(),
+		include_domains: z.array(z.string()).describe("include domains").optional(),
+		exclude_domains: z.array(z.string()).describe("exclude domains").optional(),
+		start_published_date: z.string().describe("published after (iso 8601)").optional(),
+		end_published_date: z.string().describe("published before (iso 8601)").optional(),
+		use_autoprompt: z.boolean().describe("autoprompt").optional(),
+		text: z.boolean().describe("include page text").optional(),
+		highlights: z.boolean().describe("include highlights").optional(),
+		num_results: z.number().int().min(1).max(100).describe("max results (1-100)").optional(),
 	}),
 	"web_search_exa",
 );

package/src/exa/websets.ts

@@ -53,8 +53,8 @@ const websetCreateTool = createWebsetTool(
 	"Create Webset",
 	"Create a new webset collection for organizing web content.",
 	z.object({
-		name: z.string().describe("Name of the webset"),
-		description: z.string().describe("Optional description").optional(),
+		name: z.string().describe("webset name"),
+		description: z.string().describe("description").optional(),
 	}),
 	"create_webset",
 );
@@ -72,7 +72,7 @@ const websetGetTool = createWebsetTool(
 	"Get Webset",
 	"Get details of a specific webset by ID.",
 	z.object({
-		id: z.string().describe("Webset ID"),
+		id: z.string().describe("webset id"),
 	}),
 	"get_webset",
 );
@@ -82,9 +82,9 @@ const websetUpdateTool = createWebsetTool(
 	"Update Webset",
 	"Update a webset's name or description.",
 	z.object({
-		id: z.string().describe("Webset ID"),
-		name: z.string().describe("New name").optional(),
-		description: z.string().describe("New description").optional(),
+		id: z.string().describe("webset id"),
+		name: z.string().describe("new name").optional(),
+		description: z.string().describe("new description").optional(),
 	}),
 	"update_webset",
 );
@@ -94,7 +94,7 @@ const websetDeleteTool = createWebsetTool(
 	"Delete Webset",
 	"Delete a webset and all its contents.",
 	z.object({
-		id: z.string().describe("Webset ID"),
+		id: z.string().describe("webset id"),
 	}),
 	"delete_webset",
 );
@@ -105,9 +105,9 @@ const websetItemsListTool = createWebsetTool(
 	"List Webset Items",
 	"List items in a webset with optional pagination.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		limit: z.number().describe("Number of items to return").optional(),
-		offset: z.number().describe("Pagination offset").optional(),
+		webset_id: z.string().describe("webset id"),
+		limit: z.number().describe("max items").optional(),
+		offset: z.number().describe("offset").optional(),
 	}),
 	"list_webset_items",
 );
@@ -117,8 +117,8 @@ const websetItemGetTool = createWebsetTool(
 	"Get Webset Item",
 	"Get a specific item from a webset.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		item_id: z.string().describe("Item ID"),
+		webset_id: z.string().describe("webset id"),
+		item_id: z.string().describe("item id"),
 	}),
 	"get_item",
 );
@@ -129,8 +129,8 @@ const websetSearchCreateTool = createWebsetTool(
 	"Create Webset Search",
 	"Create a new search within a webset.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		query: z.string().describe("Search query"),
+		webset_id: z.string().describe("webset id"),
+		query: z.string().describe("search query"),
 	}),
 	"create_search",
 );
@@ -140,8 +140,8 @@ const websetSearchGetTool = createWebsetTool(
 	"Get Webset Search",
 	"Get the status and results of a webset search.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		search_id: z.string().describe("Search ID"),
+		webset_id: z.string().describe("webset id"),
+		search_id: z.string().describe("search id"),
 	}),
 	"get_search",
 );
@@ -151,8 +151,8 @@ const websetSearchCancelTool = createWebsetTool(
 	"Cancel Webset Search",
 	"Cancel a running webset search.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		search_id: z.string().describe("Search ID"),
+		webset_id: z.string().describe("webset id"),
+		search_id: z.string().describe("search id"),
 	}),
 	"cancel_search",
 );
@@ -163,9 +163,9 @@ const websetEnrichmentCreateTool = createWebsetTool(
 	"Create Enrichment",
 	"Create a new enrichment task for a webset.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		name: z.string().describe("Enrichment name"),
-		prompt: z.string().describe("Enrichment prompt"),
+		webset_id: z.string().describe("webset id"),
+		name: z.string().describe("enrichment name"),
+		prompt: z.string().describe("enrichment prompt"),
 	}),
 	"create_enrichment",
 );
@@ -175,8 +175,8 @@ const websetEnrichmentGetTool = createWebsetTool(
 	"Get Enrichment",
 	"Get the status and results of an enrichment task.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		enrichment_id: z.string().describe("Enrichment ID"),
+		webset_id: z.string().describe("webset id"),
+		enrichment_id: z.string().describe("enrichment id"),
 	}),
 	"get_enrichment",
 );
@@ -186,10 +186,10 @@ const websetEnrichmentUpdateTool = createWebsetTool(
 	"Update Enrichment",
 	"Update an enrichment's name or prompt.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		enrichment_id: z.string().describe("Enrichment ID"),
-		name: z.string().describe("New name").optional(),
-		prompt: z.string().describe("New prompt").optional(),
+		webset_id: z.string().describe("webset id"),
+		enrichment_id: z.string().describe("enrichment id"),
+		name: z.string().describe("new name").optional(),
+		prompt: z.string().describe("new prompt").optional(),
 	}),
 	"update_enrichment",
 );
@@ -199,8 +199,8 @@ const websetEnrichmentDeleteTool = createWebsetTool(
 	"Delete Enrichment",
 	"Delete an enrichment task.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		enrichment_id: z.string().describe("Enrichment ID"),
+		webset_id: z.string().describe("webset id"),
+		enrichment_id: z.string().describe("enrichment id"),
 	}),
 	"delete_enrichment",
 );
@@ -210,8 +210,8 @@ const websetEnrichmentCancelTool = createWebsetTool(
 	"Cancel Enrichment",
 	"Cancel a running enrichment task.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		enrichment_id: z.string().describe("Enrichment ID"),
+		webset_id: z.string().describe("webset id"),
+		enrichment_id: z.string().describe("enrichment id"),
 	}),
 	"cancel_enrichment",
 );
@@ -222,8 +222,8 @@ const websetMonitorCreateTool = createWebsetTool(
 	"Create Monitor",
 	"Create a monitoring task for a webset with optional webhook notifications.",
 	z.object({
-		webset_id: z.string().describe("Webset ID"),
-		webhook_url: z.string().describe("Webhook URL for notifications").optional(),
+		webset_id: z.string().describe("webset id"),
+		webhook_url: z.string().describe("webhook url").optional(),
 	}),
 	"create_monitor",
 );

package/src/extensibility/typebox.ts

@@ -12,7 +12,7 @@
  * the agent pipeline:
  *
  *   - `isZodSchema()` keys off the Zod `_zod` marker that every schema carries.
- *   - `zodToWireSchema()` emits the same draft-7 JSON Schema providers expect
+ *   - `zodToWireSchema()` emits the same draft 2020-12 JSON Schema providers expect
  *     from TypeBox-authored tools (defaulted fields treated as optional, etc.).
  *
  * The surface intentionally covers only the common TypeBox builders. Plugins

package/src/goals/tools/goal-tool.ts

@@ -15,9 +15,9 @@ import { completionBudgetReport, remainingTokens } from "../runtime";
 import type { Goal, GoalStatus, GoalToolDetails } from "../state";
 
 const goalSchema = z.object({
-	op: z.union([z.literal("create"), z.literal("get"), z.literal("complete")]).describe("Goal operation."),
-	objective: z.string().describe("Goal objective. Required when op=create.").optional(),
-	token_budget: z.number().int().describe("Optional positive token budget. Only honored when op=create.").optional(),
+	op: z.enum(["create", "get", "complete"]).describe("goal operation"),
+	objective: z.string().describe("goal objective").optional(),
+	token_budget: z.number().int().describe("token budget").optional(),
 });
 
 export type GoalToolInput = z.infer<typeof goalSchema>;

package/src/hashline/index.ts

@@ -11,4 +11,3 @@ export * from "./prefixes";
 export * from "./recovery";
 export * from "./stream";
 export * from "./types";
-export * from "./utils";

package/src/hashline/input.ts

@@ -2,7 +2,6 @@ import * as path from "node:path";
 import { ABORT_MARKER, BEGIN_PATCH_MARKER, END_PATCH_MARKER, FILE_HEADER_PREFIX } from "./constants";
 import { HL_EDIT_SEP } from "./hash";
 import type { SplitHashlineOptions } from "./types";
-import { stripTrailingCarriageReturn } from "./utils";
 
 export interface HashlineInputSection {
 	path: string;
@@ -64,8 +63,7 @@ function stripLeadingBlankLines(input: string): string {
 }
 
 export function containsRecognizableHashlineOperations(input: string): boolean {
-	for (const rawLine of input.split("\n")) {
-		const line = stripTrailingCarriageReturn(rawLine);
+	for (const line of input.split(/\r?\n/)) {
 		if (/^[+<=-]\s+/.test(line) || line.startsWith(HL_EDIT_SEP)) return true;
 	}
 	return false;
@@ -74,8 +72,8 @@ export function containsRecognizableHashlineOperations(input: string): boolean {
 function normalizeFallbackInput(input: string, options: SplitHashlineOptions): string {
 	const stripped = input.startsWith("\uFEFF") ? input.slice(1) : input;
 	const hasExplicitHeader = stripped
-		.split("\n")
-		.some(rawLine => parseHashlineHeaderLine(stripTrailingCarriageReturn(rawLine), options.cwd) !== null);
+		.split(/\r?\n/)
+		.some(rawLine => parseHashlineHeaderLine(rawLine, options.cwd) !== null);
 	if (hasExplicitHeader) return input;
 
 	if (!options.path || !containsRecognizableHashlineOperations(input)) return input;
@@ -91,8 +89,8 @@ export function splitHashlineInput(input: string, options: SplitHashlineOptions
 
 export function splitHashlineInputs(input: string, options: SplitHashlineOptions = {}): HashlineInputSection[] {
 	const stripped = stripLeadingBlankLines(normalizeFallbackInput(input, options));
-	const lines = stripped.split("\n");
-	const firstLine = stripTrailingCarriageReturn(lines[0] ?? "");
+	const lines = stripped.split(/\r?\n/);
+	const firstLine = lines[0] ?? "";
 
 	if (parseHashlineHeaderLine(firstLine, options.cwd) === null) {
 		const preview = JSON.stringify(firstLine.slice(0, 120));
@@ -108,13 +106,12 @@ export function splitHashlineInputs(input: string, options: SplitHashlineOptions
 
 	const flush = () => {
 		if (currentPath.length === 0) return;
-		const hasOps = currentLines.some(rawLine => stripTrailingCarriageReturn(rawLine).trim().length > 0);
+		const hasOps = currentLines.some(line => line.trim().length > 0);
 		if (hasOps) sections.push({ path: currentPath, diff: currentLines.join("\n") });
 		currentLines = [];
 	};
 
-	for (const rawLine of lines) {
-		const line = stripTrailingCarriageReturn(rawLine);
+	for (const line of lines) {
 		if (line.trimEnd() === END_PATCH_MARKER || line.trimEnd() === ABORT_MARKER) break;
 		if (isPatchEnvelopeMarker(line)) continue;
 		const header = parseHashlineHeaderLine(line, options.cwd);
@@ -123,7 +120,7 @@ export function splitHashlineInputs(input: string, options: SplitHashlineOptions
 			currentPath = header.path;
 			currentLines = [];
 		} else {
-			currentLines.push(rawLine);
+			currentLines.push(line);
 		}
 	}
 	flush();

package/src/hashline/parser.ts

@@ -1,7 +1,6 @@
 import { ABORT_MARKER, ABORT_WARNING, BEGIN_PATCH_MARKER, END_PATCH_MARKER, RANGE_INTERIOR_HASH } from "./constants";
 import { describeAnchorExamples, HL_EDIT_SEP, HL_HASH_CAPTURE_RE_RAW } from "./hash";
 import type { Anchor, HashlineCursor, HashlineEdit } from "./types";
-import { stripTrailingCarriageReturn } from "./utils";
 
 const LID_CAPTURE_RE = new RegExp(`^${HL_HASH_CAPTURE_RE_RAW}$`);
 
@@ -85,9 +84,9 @@ function collectPayload(
 	const payload: string[] = [];
 	let index = startIndex;
 	while (index < lines.length) {
-		const line = stripTrailingCarriageReturn(lines[index]);
+		const line = lines[index];
 		if (line.startsWith(HL_EDIT_SEP)) {
-			payload.push(line.slice(1));
+			payload.push(line.slice(1).trimEnd());
 			index++;
 			continue;
 		}
@@ -100,11 +99,10 @@ function collectPayload(
 		// when inserting a blank line.
 		if (line.length === 0) {
 			let lookahead = index + 1;
-			while (lookahead < lines.length && stripTrailingCarriageReturn(lines[lookahead]).length === 0) {
+			while (lookahead < lines.length && lines[lookahead].length === 0) {
 				lookahead++;
 			}
-			const followedByPayload =
-				lookahead < lines.length && stripTrailingCarriageReturn(lines[lookahead]).startsWith(HL_EDIT_SEP);
+			const followedByPayload = lookahead < lines.length && lines[lookahead].startsWith(HL_EDIT_SEP);
 			const acceptBareBlank = requirePayload && payload.length === 0;
 			if (followedByPayload || acceptBareBlank) {
 				for (let j = index; j < lookahead; j++) payload.push("");
@@ -127,7 +125,7 @@ export function parseHashline(diff: string): HashlineEdit[] {
 export function parseHashlineWithWarnings(diff: string): { edits: HashlineEdit[]; warnings: string[] } {
 	const edits: HashlineEdit[] = [];
 	const warnings: string[] = [];
-	const lines = diff.split("\n");
+	const lines = diff.split(/\r?\n/);
 	let editIndex = 0;
 
 	const pushInsert = (cursor: HashlineCursor, text: string, lineNum: number) => {
@@ -136,7 +134,7 @@ export function parseHashlineWithWarnings(diff: string): { edits: HashlineEdit[]
 
 	for (let i = 0; i < lines.length; ) {
 		const lineNum = i + 1;
-		const line = stripTrailingCarriageReturn(lines[i]);
+		const line = lines[i];
 
 		if (line.trim().length === 0) {
 			i++;

package/src/index.ts

@@ -2,9 +2,6 @@ import { HookEditorComponent, HookInputComponent, HookSelectorComponent } from "
 
 // Core session management
 
-// TypeBox helper for string enums (convenience for custom tools)
-// Re-export from pi-ai which uses the correct enum-based schema format
-export { StringEnum } from "@oh-my-pi/pi-ai";
 // Re-export TUI components for custom tool rendering
 export { Container, Markdown, Spacer, Text } from "@oh-my-pi/pi-tui";
 // Logging