@oh-my-pi/pi-coding-agent 15.1.2 → 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/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/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/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