@ferris1225/pi-subagents 0.16.0 → 0.16.1

package/README.md

@@ -351,7 +351,7 @@ agent's default — its frontmatter `thinking`, else the global default). The gl
 | `agentScope` | `user`, `project`, or `both`; controls which user/project agent directories are discovered. |
 | `maxConcurrency` | Max sub-agent processes running at once (1–16, default 4), and the max tasks one parallel `subagent` call accepts. Extra work waits in the queue. |
 | `maxFixRounds` | Auto-fix rounds when a reviewer returns `REVIEW_FAIL`: the extension dispatches a `worker` (briefed with the review's concrete findings) then a `reviewer` re-review, repeating up to this many times before waking the main agent with the full chain. `0` disables it (the main agent handles fixes itself). Default 2. The reviewer stays read-only and in its own context; the loop is orchestrated by the extension, not by the reviewer. |
-| `idleTimeoutSec` | Idle timeout in seconds: a sub-agent whose stdout (JSON event stream) goes silent for this long is terminated and retried with the fallback model (if one is available). `0` disables the idle watchdog. Default 90. Unlike the total timeout, this only fires when the child produces no output at all — a long but active run is never interrupted. |
+| `idleTimeoutSec` | Idle timeout in seconds: a sub-agent whose stdout (JSON event stream) goes silent for this long is terminated and retried with the fallback model (if one is available). `0` disables the idle watchdog. Default 90. This only fires when the child produces no output at all — a long but active run is never interrupted. |
 
 ### Configuration migration
 
@@ -382,10 +382,10 @@ At runtime, if an agent's model fails at the provider level before producing any
 model id, auth, thinking level, quota, ...), the run is retried **once** with the main window's
 current model. This per-run degradation is never persisted — a transient provider hiccup must
 not silently downgrade the configured model — and it does not apply to task-level failures
-(the model worked, the task failed), aborts, or total timeouts. Idle timeouts (the child's
-stdout goes silent for `idleTimeoutSec` seconds) are treated as model-level failures and do
-trigger the fallback, since a stalled SSE stream is usually a provider-side issue. Results
-carry a `model fell back from …` note when it happened.
+(the model worked, the task failed) or aborts. Idle timeouts (the child's stdout goes silent
+for `idleTimeoutSec` seconds) are treated as model-level failures and do trigger the fallback,
+since a stalled SSE stream is usually a provider-side issue. Results carry a `model fell back
+from …` note when it happened.
 
 Thinking strength uses this precedence: `agentThinkingLevels` entry → agent frontmatter `thinking` → `thinkingLevel` default.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ferris1225/pi-subagents",
-  "version": "0.16.0",
+  "version": "0.16.1",
   "description": "Focused sub-agent delegation for pi: explore / worker / reviewer agents in isolated context, with proactive dispatch injection and per-agent model selection.",
   "type": "module",
   "license": "MIT",

package/src/index.ts

@@ -12,7 +12,6 @@
  * runaway recursion and keeps child context windows clean.
  */
 
-import type { AgentToolResult } from "@earendil-works/pi-agent-core";
 import { getAgentDir, type ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Text, truncateToWidth } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
@@ -31,7 +30,6 @@ import { buildDelegationDirective } from "./prompt.ts";
 import { runSetup } from "./setup.ts";
 import {
 	currentSubagentDepth,
-	getFinalOutput,
 	getResultOutput,
 	isFailedResult,
 	reviewVerdict,
@@ -217,7 +215,7 @@ export default function (pi: ExtensionAPI): void {
 		],
 		parameters: SubagentParams,
 
-		async execute(_toolCallId, params, signal, onUpdate, ctx) {
+		async execute(_toolCallId, params, signal, _onUpdate, ctx) {
 			monitor.beginTurn();
 			let config = await loadConfig(configPath);
 			// Pick up concurrency changes from /subagents-setup without a restart.

package/src/spawn.ts

@@ -4,8 +4,7 @@
  * written to a temp file and passed via `--append-system-prompt` (which accepts a
  * file path). The task itself is sent through the child's stdin pipe, not another
  * temp file or command-line argument. Child stdout is a JSON-lines event stream;
- * we accumulate assistant messages from `message_end` events and stream partial
- * output back via onUpdate.
+ * we accumulate assistant messages from `message_end` events.
  *
  * Adapted from the official pi example `examples/extensions/subagent`.
  */
@@ -16,7 +15,6 @@ import { mkdtemp, rm, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { basename, join } from "node:path";
 import { StringDecoder } from "node:string_decoder";
-import type { AgentToolResult } from "@earendil-works/pi-agent-core";
 import type { Message } from "@earendil-works/pi-ai";
 import type { AgentConfig, AgentSource } from "./agents.ts";
 import { DEFAULT_THINKING_LEVEL, type ThinkingLevel } from "./config.ts";
@@ -28,8 +26,6 @@ import { DEFAULT_THINKING_LEVEL, type ThinkingLevel } from "./config.ts";
 /** Default thinking level for sub-agents. pi clamps it to the resolved model's support. */
 export const SUBAGENT_THINKING_LEVEL: ThinkingLevel = DEFAULT_THINKING_LEVEL;
 export const DEPTH_ENV_VAR = "PI_SUBAGENT_DEPTH";
-/** No default deadline: sub-agents may run until completion or explicit cancellation. */
-export const SUBAGENT_TIMEOUT_MS = 0;
 export const SUBAGENT_KILL_GRACE_MS = 5_000;
 /** Default idle watchdog: terminate a child whose stdout goes silent for this
  * many milliseconds. 0 disables it. The actual value comes from config
@@ -70,8 +66,6 @@ export interface SubagentDetails {
 	background?: boolean;
 }
 
-export type OnUpdateCallback = (partial: AgentToolResult<SubagentDetails>) => void;
-
 export type SubagentLiveEvent =
 	| { kind: "status"; status: "queued" | "running" | "done" | "failed" }
 	| { kind: "usage"; usage: UsageStats; model?: string }
@@ -165,7 +159,6 @@ export function isModelLevelFailure(result: SingleResult): boolean {
 	if (result.errorMessage?.includes("idle timeout")) return true;
 	// The model produced text: the failure belongs to the task, not the model.
 	if (getFinalOutput(result.messages)) return false;
-	if (result.errorMessage?.includes("timed out")) return false;
 	// Require evidence the failure came from the model/provider (an error
 	// message or stderr), not from the child process failing to start.
 	return result.messages.length > 0 || result.stderr.trim().length > 0;
@@ -178,26 +171,6 @@ export function getResultOutput(result: SingleResult): string {
 	return getFinalOutput(result.messages) || "(no output)";
 }
 
-export async function mapWithConcurrencyLimit<TIn, TOut>(
-	items: TIn[],
-	concurrency: number,
-	fn: (item: TIn, index: number) => Promise<TOut>,
-): Promise<TOut[]> {
-	if (items.length === 0) return [];
-	const limit = Math.max(1, Math.min(concurrency, items.length));
-	const results: TOut[] = new Array(items.length);
-	let nextIndex = 0;
-	const workers = new Array(limit).fill(null).map(async () => {
-		while (true) {
-			const current = nextIndex++;
-			if (current >= items.length) return;
-			results[current] = await fn(items[current], current);
-		}
-	});
-	await Promise.all(workers);
-	return results;
-}
-
 async function writePromptToTempFile(agentName: string, prompt: string): Promise<{ dir: string; filePath: string }> {
 	const dir = await mkdtemp(join(tmpdir(), "pi-subagents-"));
 	const safeName = agentName.replace(/[^\w.-]+/g, "_");
@@ -261,13 +234,10 @@ export interface RunSingleOptions {
 	cwd?: string;
 	/** Thinking level passed to the child pi process. */
 	thinkingLevel?: ThinkingLevel;
-	/** Optional total timeout; zero (the default) disables it. Intended for tests and controlled callers. */
-	timeoutMs?: number;
 	/** Idle timeout in ms: terminate the child if its stdout produces no activity
 	 * for this duration. 0 (the default) disables the idle watchdog. */
 	idleTimeoutMs?: number;
 	signal?: AbortSignal;
-	onUpdate?: OnUpdateCallback;
 	onLive?: (e: SubagentLiveEvent) => void;
 	makeDetails: (results: SingleResult[]) => SubagentDetails;
 	env?: NodeJS.ProcessEnv;
@@ -281,10 +251,8 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 		task,
 		cwd,
 		thinkingLevel = SUBAGENT_THINKING_LEVEL,
-		timeoutMs = SUBAGENT_TIMEOUT_MS,
 		idleTimeoutMs = SUBAGENT_DEFAULT_IDLE_TIMEOUT_MS,
 		signal,
-		onUpdate,
 		onLive,
 		makeDetails,
 	} = options;
@@ -324,13 +292,6 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 		thinking: thinkingLevel,
 	};
 
-	const emitUpdate = (): void => {
-		onUpdate?.({
-			content: [{ type: "text", text: getFinalOutput(currentResult.messages) || "(running...)" }],
-			details: makeDetails([currentResult]),
-		});
-	};
-
 	try {
 		if (agent.systemPrompt.trim()) {
 			const tmp = await writePromptToTempFile(agent.name, agent.systemPrompt);
@@ -340,7 +301,6 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 		}
 
 		let wasAborted = false;
-		let timedOut = false;
 
 		// Increment depth so nested sub-agents can be guarded against runaway recursion.
 		const childDepth = currentSubagentDepth(options.env) + 1;
@@ -361,7 +321,6 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 			let closed = false;
 			let termSent = false;
 			let forceKillTimer: ReturnType<typeof setTimeout> | undefined;
-			let timeoutTimer: ReturnType<typeof setTimeout> | undefined;
 			let abortHandler: (() => void) | undefined;
 			let lastActivityAt = Date.now();
 			let idleTimer: ReturnType<typeof setInterval> | undefined;
@@ -370,7 +329,6 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 				if (closed) return;
 				closed = true;
 				if (forceKillTimer) clearTimeout(forceKillTimer);
-				if (timeoutTimer) clearTimeout(timeoutTimer);
 				if (idleTimer) clearInterval(idleTimer);
 				if (signal && abortHandler) signal.removeEventListener("abort", abortHandler);
 				resolve(code ?? 1);
@@ -461,12 +419,10 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 							onLive({ kind: "usage", usage: { ...currentResult.usage }, model: currentResult.model });
 						} catch { /* never throw from event handling */ }
 					}
-					emitUpdate();
 				}
 
 				if (event.type === "tool_result_end" && event.message) {
 					currentResult.messages.push(event.message as Message);
-					emitUpdate();
 				}
 			};
 			// Send the task through the child stdin pipe instead of the process
@@ -502,7 +458,6 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 				const failed =
 					code !== 0 ||
 					wasAborted ||
-					timedOut ||
 					(signal?.aborted ?? false) ||
 					currentResult.stopReason === "error" ||
 					currentResult.stopReason === "aborted";
@@ -526,22 +481,12 @@ export async function runSingleAgent(options: RunSingleOptions): Promise<SingleR
 				finish(1);
 			});
 
-			if (timeoutMs > 0) {
-				timeoutTimer = setTimeout(() => {
-					timedOut = true;
-					currentResult.stopReason = "error";
-					currentResult.errorMessage = `Subagent timed out after ${Math.ceil(timeoutMs / 1000)} seconds.`;
-					terminate();
-				}, timeoutMs);
-			}
-
 			if (idleTimeoutMs > 0) {
 				const checkInterval = Math.min(10_000, Math.floor(idleTimeoutMs / 3));
 				idleTimer = setInterval(() => {
 					if (closed) return;
 					if (Date.now() - lastActivityAt >= idleTimeoutMs) {
 						if (idleTimer) clearInterval(idleTimer);
-						timedOut = true;
 						currentResult.stopReason = "error";
 						currentResult.errorMessage = `Subagent idle timeout: no activity for ${Math.ceil(idleTimeoutMs / 1000)} seconds.`;
 						terminate();