@oh-my-pi/pi-coding-agent 13.0.0 → 13.0.2

package/src/prompts/tools/task.md

@@ -19,7 +19,6 @@ Subagents have no access to your conversation history. They don't know:
 Subagents CAN grep the parent conversation file for supplementary details.
 
 For large intermediate outputs (long traces, JSON payloads, temporary analysis snapshots), you SHOULD write them to `local://<path>` and pass the path in task context instead of inlining bulky text.
----
 
 ## Parameters
 
@@ -128,9 +127,9 @@ Use structure every assignment:
 - Patterns/APIs to use; reference files if applicable
 
 ## Edge Cases / Don't Break
-- Tricky case 1: ...
-- Tricky case 2: ...
-- Existing behavior must survive: ...
+- Tricky case 1: …
+- Tricky case 2: …
+- Existing behavior must survive: …
 
 ## Acceptance (task-local)
 - Expected behavior or observable result
@@ -145,11 +144,11 @@ Use structure every assignment:
 - "Migrate to N-API."
 - "Fix the bug in streaming."
 - "Update all constructors in `src/**/*.ts`."
-**Vague context** — forces agent invent conventions:
+  **Vague context** — forces agent invent conventions:
 - "Use existing patterns."
 - "Follow conventions."
 - "No WASM."
-**Redundant context** — wastes tokens repeating what subagents already have:
+  **Redundant context** — wastes tokens repeating what subagents already have:
 - Restating AGENTS.md rules (coding style, import conventions, formatting commands, logger usage, etc.)
 - Repeating project constraints from context files
 - Listing tool/framework preferences already documented in the repo
@@ -173,7 +172,7 @@ Micromanaging (you think, agent types):
 ```
 assignment: "In src/api/handler.ts, line 47, change `throw err` to `throw new ApiError(err.message, 500)`.
 On line 63, wrap fetch call try/catch return 502 on failure.
-On line 89, add null check before accessing resp.body..."
+On line 89, add null check before accessing resp.body…"
 ```
 
 Delegating (agent thinks within constraints):
@@ -192,9 +191,9 @@ First style wastes your time, brittle if code shifts. Second gives agent room to
 <example type="bad" label="Duplicated context inflates tokens">
 <tasks>
   <task name="Grep">
-    <description>Port grep module from WASM to N-API...</description>
-    <assignment>Port grep module from WASM to N-API... (same blob repeated)</assignment>
-</task>
+    <description>Port grep module from WASM to N-API…</description>
+    <assignment>Port grep module from WASM to N-API… (same blob repeated)</assignment>
+  </task>
 </tasks>
 </example>
 
@@ -210,7 +209,7 @@ Do not touch TS bindings or downstream consumers — separate phase.
 - MUST use `#[napi]` attribute macro on all exports
 - MUST return `napi::Result<T>` for fallible ops; never panic
 - MUST use `spawn_blocking` for filesystem I/O or >1ms work
-...
+  …
 
 ## Acceptance (global)
 - Caller verifies after all tasks: `cargo test -p pi-natives` and `cargo build -p pi-natives` with no warnings
@@ -227,22 +226,22 @@ Do not touch TS bindings or downstream consumers — separate phase.
 
 ## Change
 - Implement three N-API exports in grep.rs:
-  - `search(pattern: JsString, path: JsString, env: Env) -> napi::Result<Vec<Match>>`
-...
+  - `search(pattern: JsString, path: JsString, env: Env) → napi::Result<Vec<Match>>`
+…
 
 ## Acceptance (task-local)
 - Three functions exported with correct signatures (caller verifies build after all tasks)
-</assignment>
-</task>
+    </assignment>
+  </task>
 
   <task name="PortHighlight">
     <description>Port highlight module to N-API</description>
     <assignment>
 ## Target
 - Files: `src/highlight.rs`, `src/lib.rs` (registration only)
-...
-</assignment>
-</task>
+  …
+    </assignment>
+  </task>
 </tasks>
 </example>
 ---
@@ -254,7 +253,7 @@ Each task MUST have small, well-defined scope — **at most 3–5 files**.
 - File paths use globs (`src/**/*.ts`) instead of explicit names
 - Assignment says "update all" / "migrate everything" / "refactor across"
 - Scope covers entire package or directory tree
-**Fix:** You MUST enumerate files first (grep/glob discovery), then fan out one task per file or small cluster.
+  **Fix:** You MUST enumerate files first (grep/glob discovery), then fan out one task per file or small cluster.
 ---
 
 ## Parallelization

package/src/task/index.ts

@@ -134,6 +134,7 @@ function renderDescription(
 export class TaskTool implements AgentTool<TaskSchema, TaskToolDetails, Theme> {
 	readonly name = "task";
 	readonly label = "Task";
+	readonly strict = true;
 	readonly parameters: TaskSchema;
 	readonly renderCall = renderCall;
 	readonly renderResult = renderResult;

package/src/tools/ask.ts

@@ -246,6 +246,7 @@ export class AskTool implements AgentTool<typeof askSchema, AskToolDetails> {
 	readonly label = "Ask";
 	readonly description: string;
 	readonly parameters = askSchema;
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(askDescription);

package/src/tools/bash.ts

@@ -91,6 +91,7 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
 	readonly description: string;
 	readonly parameters: BashToolSchema;
 	readonly concurrency = "exclusive";
+	readonly strict = true;
 	readonly #asyncEnabled: boolean;
 
 	constructor(private readonly session: ToolSession) {

package/src/tools/browser.ts

@@ -484,6 +484,7 @@ export class BrowserTool implements AgentTool<typeof browserSchema, BrowserToolD
 	readonly label = "Puppeteer";
 	readonly description: string;
 	readonly parameters = browserSchema;
+	readonly strict = true;
 	#browser: Browser | null = null;
 	#page: Page | null = null;
 	#currentHeadless: boolean | null = null;

package/src/tools/calculator.ts

@@ -399,6 +399,7 @@ export class CalculatorTool implements AgentTool<typeof calculatorSchema, Calcul
 	readonly label = "Calc";
 	readonly description: string;
 	readonly parameters = calculatorSchema;
+	readonly strict = true;
 
 	constructor(_session: ToolSession) {
 		this.description = renderPromptTemplate(calculatorDescription);

package/src/tools/cancel-job.ts

@@ -20,6 +20,7 @@ export class CancelJobTool implements AgentTool<typeof cancelJobSchema, CancelJo
 	readonly label = "CancelJob";
 	readonly description: string;
 	readonly parameters = cancelJobSchema;
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(cancelJobDescription);

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

@@ -45,6 +45,7 @@ export class ExitPlanModeTool implements AgentTool<typeof exitPlanModeSchema, Ex
 	readonly label = "ExitPlanMode";
 	readonly description: string;
 	readonly parameters = exitPlanModeSchema;
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(exitPlanModeDescription);

package/src/tools/fetch.ts

@@ -856,6 +856,7 @@ export class FetchTool implements AgentTool<typeof fetchSchema, FetchToolDetails
 	readonly label = "Fetch";
 	readonly description: string;
 	readonly parameters = fetchSchema;
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(fetchDescription);

package/src/tools/find.ts

@@ -127,6 +127,7 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 	readonly label = "Find";
 	readonly description: string;
 	readonly parameters = findSchema;
+	readonly strict = true;
 
 	readonly #customOps?: FindOperations;
 

package/src/tools/grep.ts

@@ -60,6 +60,7 @@ export class GrepTool implements AgentTool<typeof grepSchema, GrepToolDetails> {
 	readonly label = "Grep";
 	readonly description: string;
 	readonly parameters = grepSchema;
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		const displayMode = resolveFileDisplayMode(session);

package/src/tools/notebook.ts

@@ -65,6 +65,7 @@ export class NotebookTool implements AgentTool<typeof notebookSchema, NotebookTo
 	readonly description =
 		"Completely replaces the contents of a specific cell in a Jupyter notebook (.ipynb file) with new source. Jupyter notebooks are interactive documents that combine code, text, and visualizations, commonly used for data analysis and scientific computing. The notebook_path parameter must be an absolute path, not a relative path. The cell_number is 0-indexed. Use edit_mode=insert to add a new cell at the index specified by cell_number. Use edit_mode=delete to delete the cell at the index specified by cell_number.";
 	readonly parameters = notebookSchema;
+	readonly strict = true;
 	readonly concurrency = "exclusive";
 
 	constructor(private readonly session: ToolSession) {}

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

@@ -19,7 +19,7 @@ export function resolvePlanPath(session: ToolSession, targetPath: string): strin
 export function enforcePlanModeWrite(
 	session: ToolSession,
 	targetPath: string,
-	options?: { rename?: string; op?: "create" | "update" | "delete" },
+	options?: { move?: string; op?: "create" | "update" | "delete" },
 ): void {
 	const state = session.getPlanModeState?.();
 	if (!state?.enabled) return;
@@ -27,7 +27,7 @@ export function enforcePlanModeWrite(
 	const resolvedTarget = resolvePlanPath(session, targetPath);
 	const resolvedPlan = resolvePlanPath(session, state.planFilePath);
 
-	if (options?.rename) {
+	if (options?.move) {
 		throw new ToolError("Plan mode: renaming files is not allowed.");
 	}
 

package/src/tools/poll-jobs.ts

@@ -39,6 +39,7 @@ export class PollJobsTool implements AgentTool<typeof pollJobsSchema, PollJobsTo
 	readonly label = "PollJobs";
 	readonly description: string;
 	readonly parameters = pollJobsSchema;
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(pollJobsDescription);

package/src/tools/python.ts

@@ -148,6 +148,7 @@ export class PythonTool implements AgentTool<typeof pythonSchema> {
 	readonly description: string;
 	readonly parameters = pythonSchema;
 	readonly concurrency = "exclusive";
+	readonly strict = true;
 
 	readonly #proxyExecutor?: PythonProxyExecutor;
 

package/src/tools/read.ts

@@ -558,6 +558,7 @@ export class ReadTool implements AgentTool<typeof readSchema, ReadToolDetails> {
 	readonly description: string;
 	readonly parameters = readSchema;
 	readonly nonAbortable = true;
+	readonly strict = true;
 
 	readonly #autoResizeImages: boolean;
 

package/src/tools/ssh.ts

@@ -122,6 +122,7 @@ export class SshTool implements AgentTool<typeof sshSchema, SSHToolDetails> {
 	readonly label = "SSH";
 	readonly parameters = sshSchema;
 	readonly concurrency = "exclusive";
+	readonly strict = true;
 
 	readonly #allowedHosts: Set<string>;
 

package/src/tools/submit-result.ts

@@ -59,6 +59,7 @@ export class SubmitResultTool implements AgentTool<TObject, SubmitResultDetails>
 		"Finish the task with structured JSON output. Call exactly once at the end of the task.\n\n" +
 		"If you cannot complete the task, call with status='aborted' and an error message.";
 	readonly parameters: TObject;
+	readonly strict = true;
 
 	readonly #validate?: ValidateFunction;
 	readonly #schemaError?: string;

package/src/tools/todo-write.ts

@@ -289,6 +289,7 @@ export class TodoWriteTool implements AgentTool<typeof todoWriteSchema, TodoWrit
 	readonly description: string;
 	readonly parameters = todoWriteSchema;
 	readonly concurrency = "exclusive";
+	readonly strict = true;
 
 	constructor(private readonly session: ToolSession) {
 		this.description = renderPromptTemplate(todoWriteDescription);

package/src/tools/write.ts

@@ -75,6 +75,7 @@ export class WriteTool implements AgentTool<typeof writeSchema, WriteToolDetails
 	readonly description: string;
 	readonly parameters = writeSchema;
 	readonly nonAbortable = true;
+	readonly strict = true;
 	readonly concurrency = "exclusive";
 
 	readonly #writethrough: WritethroughCallback;

package/src/web/search/index.ts

@@ -258,6 +258,7 @@ export class SearchTool implements AgentTool<typeof webSearchSchema, SearchRende
 	readonly label = "Web Search";
 	readonly description: string;
 	readonly parameters = webSearchSchema;
+	readonly strict = true;
 
 	constructor(_session: ToolSession) {
 		this.description = renderPromptTemplate(webSearchDescription);