@oh-my-pi/pi-coding-agent 9.6.3 → 9.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "9.6.3",
+	"version": "9.7.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -79,12 +79,12 @@
 		"test": "bun test"
 	},
 	"dependencies": {
-		"@oh-my-pi/omp-stats": "9.6.3",
-		"@oh-my-pi/pi-agent-core": "9.6.3",
-		"@oh-my-pi/pi-ai": "9.6.3",
-		"@oh-my-pi/pi-natives": "9.6.3",
-		"@oh-my-pi/pi-tui": "9.6.3",
-		"@oh-my-pi/pi-utils": "9.6.3",
+		"@oh-my-pi/omp-stats": "9.7.0",
+		"@oh-my-pi/pi-agent-core": "9.7.0",
+		"@oh-my-pi/pi-ai": "9.7.0",
+		"@oh-my-pi/pi-natives": "9.7.0",
+		"@oh-my-pi/pi-tui": "9.7.0",
+		"@oh-my-pi/pi-utils": "9.7.0",
 		"@openai/agents": "^0.4.4",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",

package/src/exec/bash-executor.ts

@@ -105,7 +105,6 @@ async function executeBashOnce(
 		env: finalEnv,
 		signal: options?.signal,
 		timeout: options?.timeout,
-		detached: true,
 	});
 
 	// Pump streams - errors during abort/timeout are expected

package/src/exec/shell-session.ts

@@ -254,7 +254,6 @@ class ShellSession {
 		this.#child = ptree.spawn([this.config.shell, ...args], {
 			stdin: "pipe",
 			env: this.config.env,
-			detached: !IS_WINDOWS,
 		});
 
 		if (this.#child.proc.exitCode !== null) {
@@ -502,11 +501,7 @@ class ShellSession {
 				child.proc.kill("SIGINT");
 				return;
 			}
-			if (child.isProcessGroup) {
-				process.kill(-child.pid, "SIGINT");
-			} else {
-				process.kill(child.pid, "SIGINT");
-			}
+			process.kill(child.pid, "SIGINT");
 		} catch {}
 	}
 

package/src/ipy/gateway-coordinator.ts

@@ -265,8 +265,8 @@ async function startGatewayProcess(
 			stdin: "ignore",
 			stdout: "pipe",
 			stderr: "pipe",
-			detached: true,
 			windowsHide: true,
+			detached: true,
 			env: kernelEnv,
 		},
 	);
@@ -296,13 +296,13 @@ async function startGatewayProcess(
 		await Bun.sleep(100);
 	}
 
-	await procmgr.terminate({ target: gatewayProcess, group: true });
+	gatewayProcess.kill();
 	throw new Error("Gateway startup timeout");
 }
 
 async function killGateway(pid: number, context: string): Promise<void> {
 	try {
-		await procmgr.terminate({ target: pid, group: true });
+		await procmgr.terminate({ target: pid });
 	} catch (err) {
 		logger.warn("Failed to kill shared gateway process", {
 			error: err instanceof Error ? err.message : String(err),

package/src/ipy/kernel.ts

@@ -490,7 +490,6 @@ export class PythonKernel {
 				{
 					cwd: options.cwd,
 					env: kernelEnv,
-					detached: true,
 				},
 			);
 

package/src/lsp/client.ts

@@ -411,7 +411,6 @@ export async function getOrCreateClient(config: ServerConfig, cwd: string, initT
 
 		const proc = ptree.spawn([command, ...args], {
 			cwd,
-			detached: true,
 			stdin: "pipe",
 			env: env ? { ...process.env, ...env } : undefined,
 		});

package/src/main.ts

@@ -385,7 +385,8 @@ async function buildSessionOptions(
 			process.exit(1);
 		}
 		options.model = model;
-		settings.setModelRole("default", `${model.provider}/${model.id}`);
+		const currentRoles = settings.get("modelRoles") as Record<string, string>;
+		settings.override("modelRoles", { ...currentRoles, default: `${model.provider}/${model.id}` });
 	} else if (scopedModels.length > 0 && !parsed.continue && !parsed.resume) {
 		const remembered = settings.getModelRole("default");
 		if (remembered) {
@@ -637,9 +638,10 @@ export async function main(args: string[]) {
 	const planModel = parsed.plan ?? process.env.OMP_PLAN_MODEL;
 	if (smolModel || slowModel || planModel) {
 		const currentRoles = settings.get("modelRoles") as Record<string, string>;
-		if (smolModel) settings.override("modelRoles", { ...currentRoles, smol: smolModel });
-		if (slowModel) settings.override("modelRoles", { ...currentRoles, slow: slowModel });
-		if (planModel) settings.override("modelRoles", { ...currentRoles, plan: planModel });
+		if (smolModel) currentRoles.smol = smolModel;
+		if (slowModel) currentRoles.slow = slowModel;
+		if (planModel) currentRoles.plan = planModel;
+		settings.override("modelRoles", currentRoles);
 	}
 
 	await initTheme(settings.get("theme"), isInteractive, settings.get("symbolPreset"), settings.get("colorBlindMode"));

package/src/prompts/system/system-prompt.md

@@ -290,6 +290,31 @@ These are comfort. They are not clarity.
 {{appendSystemPrompt}}
 {{/if}}
 
+{{#if isCoordinator}}
+{{#has tools "task"}}
+<parallel_reflex>
+When the work forks, you fork.
+
+Notice the sequential habit:
+- The comfort of doing one thing at a time
+- The illusion that order means correctness
+- The assumption that you must finish A before starting B
+  **Triggers requiring Task tool:**
+- Editing 4+ files with no dependencies between edits
+- Investigating 2+ independent subsystems or questions
+- Any work that decomposes into pieces that don't need each other's results
+
+<critical>
+Sequential requires justification.
+If you cannot articulate why B depends on A's result, they are parallel.
+</critical>
+
+Do not carry the whole problem in one skull.
+Split the load. Bring back facts. Then cut code.
+</parallel_reflex>
+{{/has}}
+{{/if}}
+
 <stakes>
 This is not practice.
 
@@ -325,23 +350,4 @@ The question is not "Does this work?"
 but "Under what conditions does this work, and what happens outside them?"
 
 Write what you can defend.
-</critical>
-
-{{#if isCoordinator}}
-{{#has tools "task"}}
-<critical id="coordinator">
-Your context window is limited—especially the output. Work in discrete steps and run each step using Task tool. Avoid putting substantial work in the main context when possible. Run multiple tasks in parallel whenever possible.
-
-## Triggers requiring Task tool
-- Editing 4+ files with no dependencies → `Task`
-- Investigating 2+ independent questions → `Task`
-- Any work that decomposes into pieces that don't need each other's results → `Task`
-
-Sequential requires justification.
-If you cannot articulate why B depends on A's result, they are parallel.
-
-Do not carry the whole problem in one skull.
-Split the load. Bring back facts. Then synthesize.
-</critical>
-{{/has}}
-{{/if}}
+</critical>

package/src/tools/find.ts

@@ -238,8 +238,8 @@ export class FindTool implements AgentTool<typeof findSchema, FindToolDetails> {
 				});
 			};
 			const onMatch = onUpdate
-				? (match: FindMatch) => {
-						if (signal?.aborted) return;
+				? (match: FindMatch | null) => {
+						if (signal?.aborted || !match) return;
 						let relativePath = match.path;
 						if (!relativePath) return;
 						if (match.fileType === "dir" && !relativePath.endsWith("/")) {

package/src/tools/read.ts

@@ -497,7 +497,6 @@ async function convertWithMarkitdown(
 		allowNonZero: true,
 		allowAbort: true,
 		stderr: "buffer",
-		detached: true,
 	});
 
 	if (result.exitError?.aborted) {

package/src/web/scrapers/utils.ts

@@ -54,7 +54,6 @@ export async function convertWithMarkitdown(
 			signal,
 			allowNonZero: true,
 			stderr: "full",
-			detached: true,
 		});
 		if (!result.ok) {
 			return {