@ryan_nookpi/pi-extension-subagent 0.5.5 → 0.5.6

package/README.md

@@ -68,7 +68,7 @@ Run `/subagents`, `subagent agents`, or the `list-agents` tool to inspect discov
 
 ### Optional starter pack
 
-If no agents are found, `/subagents` can offer an optional, opinionated starter pack. It copies nine agent templates, two example workflow skills, and missing global `subagent` settings. Existing files and configured values are not overwritten.
+If no agents are found, `/subagents` can offer an optional, opinionated starter pack. It copies eight agent templates, two example workflow skills, and missing global `subagent` settings. Existing files and configured values are not overwritten.
 
 The starter pack is not required. Decline it if you prefer to define agents manually. It fills a missing `claudeRuntime` setting with `cli`; without that setting, the extension default is `sdk`.
 

package/failure-telemetry.ts

@@ -5,6 +5,7 @@ export type SubagentErrorClass =
 	| "overloaded"
 	| "rate_limit"
 	| "tool_error"
+	| "network_error"
 	| "aborted"
 	| "process_error"
 	| "unknown";
@@ -64,6 +65,9 @@ const TOOL_ERROR_PATTERNS = [
 	/invalid tool/i,
 ];
 
+const NETWORK_PATTERNS = [/\bwebsocket\b/i, /\bconnection error\b/i];
+const PROCESS_SIGNAL_TERMINATION_PATTERN = /\bprocess terminated by signal\b/i;
+
 export function classifySubagentFailure(input: FailureTelemetryInput): SubagentErrorClass | undefined {
 	if (!input.failed) return undefined;
 	const text = [input.errorMessage, input.stderr, input.output].filter(Boolean).join("\n");
@@ -73,6 +77,14 @@ export function classifySubagentFailure(input: FailureTelemetryInput): SubagentE
 	if (OVERLOADED_PATTERNS.some((pattern) => pattern.test(text))) return "overloaded";
 	if (RATE_LIMIT_PATTERNS.some((pattern) => pattern.test(text))) return "rate_limit";
 	if (TOOL_ERROR_PATTERNS.some((pattern) => pattern.test(text))) return "tool_error";
+	const signalTerminated = PROCESS_SIGNAL_TERMINATION_PATTERN.test(input.stderr ?? "");
+	if (
+		!signalTerminated &&
+		(NETWORK_PATTERNS.some((pattern) => pattern.test(text)) ||
+			/^terminated\.?$/i.test(input.errorMessage?.trim() ?? ""))
+	) {
+		return "network_error";
+	}
 	if ((input.exitCode ?? 0) !== 0) return "process_error";
 	return "unknown";
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryan_nookpi/pi-extension-subagent",
-  "version": "0.5.5",
+  "version": "0.5.6",
   "description": "Asynchronous subagent delegation for pi with run, batch, chain, and continuation workflows.",
   "license": "MIT",
   "repository": {

package/retry.ts

@@ -34,9 +34,13 @@ function buildFailureText(result: SingleResult): string {
 		.join("\n");
 }
 
+const PROCESS_SIGNAL_TERMINATION_PATTERN = /\bprocess terminated by signal\b/i;
+
 const TRANSIENT_FAILURE_PATTERNS: RegExp[] = [
 	/\bnetwork\b/i,
+	/\bwebsocket\b/i,
 	/\bfetch failed\b/i,
+	/\bconnection error\b/i,
 	/\bconnection reset\b/i,
 	/\bsocket hang up\b/i,
 	/\btemporar(?:y|ily) unavailable\b/i,
@@ -70,12 +74,18 @@ function matchesTransientPattern(text: string): boolean {
 
 export function diagnoseRetryableResult(result: SingleResult): RetryDecision {
 	if (result.stopReason === "aborted") return { retryable: false };
+	if (PROCESS_SIGNAL_TERMINATION_PATTERN.test(result.stderr)) return { retryable: false };
 
 	const finalOutput = getFinalOutput(result.messages).trim();
 	if (result.exitCode === 0 && result.stopReason !== "error" && finalOutput) {
 		return { retryable: false };
 	}
 
+	const errorMessage = result.errorMessage?.trim();
+	if (errorMessage && /^terminated\.?$/i.test(errorMessage)) {
+		return { retryable: true, reason: errorMessage };
+	}
+
 	const failureText = buildFailureText(result);
 	if (!failureText) return { retryable: false };
 	if (!matchesTransientPattern(failureText)) return { retryable: false };

package/starter-pack.ts

@@ -11,7 +11,6 @@ export const STARTER_AGENT_NAMES = [
 	"reviewer",
 	"searcher",
 	"security-auditor",
-	"simplifier",
 	"verifier",
 	"worker",
 ] as const;
@@ -220,7 +219,7 @@ export async function offerStarterPackIfEmpty(
 
 	const accepted = await ctx.ui.confirm(
 		"Install starter subagents?",
-		"No subagent definitions were found. Install 9 portable English agents, the stress-interview and self-healing skills, and missing subagent settings? Existing files and configured values will not be overwritten.",
+		"No subagent definitions were found. Install 8 portable English agents, the stress-interview and self-healing skills, and missing subagent settings? Existing files and configured values will not be overwritten.",
 	);
 	if (!accepted) return { status: "declined", discovery };
 

package/tool-execute.ts

@@ -49,6 +49,7 @@ import {
 import { clearPendingGroupCompletion, upsertPendingGroupCompletion } from "./group-pending.js";
 import { enqueueSubagentInvocation } from "./invocation-queue.js";
 import { appendDisplayTaskUpdate, getSessionFileSize } from "./persisted-session.js";
+import { invokeWithAutoRetry, MAX_SUBAGENT_AUTO_RETRIES } from "./retry.js";
 import {
 	clearFinishedRuns,
 	evictStaleFinishedGroups,
@@ -1416,44 +1417,60 @@ export function createSubagentToolExecute(pi: ExtensionAPI, store: SubagentStore
 			let claudeCheckpointSent = !!runState.claudeSessionId;
 			const abortSignal = runState.abortController?.signal;
 			const recordActivity = createRunActivityRecorder(pi, runState, (model) => resolveContextWindow(ctx, model));
-			return enqueueSubagentInvocation(async () => {
-				if (runState.removed || abortSignal?.aborted) throw new Error("Subagent was aborted");
-				return runSingleAgent(
-					ctx.cwd,
-					agents,
-					runState.agent,
-					taskForAgent,
-					runState.pipelineStepIndex,
-					abortSignal,
-					(partial) => {
-						if (runState.removed || store.disposed) return;
-						const current = partial.details?.results?.[0];
-						if (!current) return;
-						updateRunFromResult(runState, current);
-						recordActivity();
-						if (shouldRunAsync && !claudeCheckpointSent && runState.claudeSessionId) {
-							claudeCheckpointSent = true;
-							pi.sendMessage(buildRunStartMessage(runState, "started"), {
-								deliverAs: "followUp",
-								triggerTurn: false,
-							});
-						}
-						updateCommandRunsWidget(store);
-					},
-					(results) => makeDetails(mode, results),
-					{
-						sessionFile: runState.sessionFile,
-						resumeSessionId: runState.claudeSessionId,
-						sidecarSessionFile: runState.sessionFile,
-						persistedSessionBaseOffset: runState.persistedSessionBaseOffset,
-						onDiagnostic: createRunDiagnosticSink(pi, runState),
-					},
-				);
-			}).then((result) =>
-				runState.removed || abortSignal?.aborted
+			return invokeWithAutoRetry({
+				maxRetries: MAX_SUBAGENT_AUTO_RETRIES,
+				signal: abortSignal,
+				onRetryScheduled: ({ retryIndex, maxRetries, delayMs, reason }) => {
+					runState.retryCount = retryIndex;
+					runState.lastRetryReason = reason;
+					runState.lastActivityAt = Date.now();
+					runState.lastLine = `Auto-retrying ${retryIndex}/${maxRetries} in ${Math.ceil(delayMs / 1000)}s: ${reason}`;
+					runState.lastOutput = runState.lastLine;
+					updateCommandRunsWidget(store);
+				},
+				invoke: () => {
+					runState.persistedSessionBaseOffset = getSessionFileSize(runState.sessionFile);
+					return enqueueSubagentInvocation(async () => {
+						if (runState.removed || abortSignal?.aborted) throw new Error("Subagent was aborted");
+						return runSingleAgent(
+							ctx.cwd,
+							agents,
+							runState.agent,
+							taskForAgent,
+							runState.pipelineStepIndex,
+							abortSignal,
+							(partial) => {
+								if (runState.removed || store.disposed) return;
+								const current = partial.details?.results?.[0];
+								if (!current) return;
+								updateRunFromResult(runState, current);
+								recordActivity();
+								if (shouldRunAsync && !claudeCheckpointSent && runState.claudeSessionId) {
+									claudeCheckpointSent = true;
+									pi.sendMessage(buildRunStartMessage(runState, "started"), {
+										deliverAs: "followUp",
+										triggerTurn: false,
+									});
+								}
+								updateCommandRunsWidget(store);
+							},
+							(results) => makeDetails(mode, results),
+							{
+								sessionFile: runState.sessionFile,
+								resumeSessionId: runState.claudeSessionId,
+								sidecarSessionFile: runState.sessionFile,
+								persistedSessionBaseOffset: runState.persistedSessionBaseOffset,
+								onDiagnostic: createRunDiagnosticSink(pi, runState),
+							},
+						);
+					});
+				},
+			}).then(({ result, retryCount }) => {
+				runState.retryCount = retryCount;
+				return runState.removed || abortSignal?.aborted
 					? finalizeRunError(runState, new Error("Subagent was aborted"))
-					: finalizeRunState(runState, result),
-			);
+					: finalizeRunState(runState, result);
+			});
 		}
 
 		if (hasSingle) {

package/seeds/agents/simplifier.md

@@ -1,68 +0,0 @@
----
-name: simplifier
-description: Code simplification specialist that improves clarity while preserving behavior
-runtime: pi
-thinking: high
-tools: read,grep,find,ls,bash,edit,write
----
-
-<system_prompt agent="simplifier">
-  <identity>
-    You simplify recently modified code for clarity, consistency, and maintainability without changing observable behavior.
-  </identity>
-
-  <scope_rule>
-    <rule>Only simplify the requested or clearly identified changed scope.</rule>
-    <rule>Do not broaden into unrelated cleanup or architecture changes.</rule>
-    <rule>Preserve outputs, side effects, public contracts, and data flow.</rule>
-    <rule>Handle each supplied finding independently; one blocked item must not stop unrelated safe items.</rule>
-  </scope_rule>
-
-  <principles>
-    <rule>Prefer explicit, readable control flow over clever compression.</rule>
-    <rule>Reduce unnecessary nesting, indirection, and dead intermediates.</rule>
-    <rule>Keep abstractions that improve organization, testing, or reuse.</rule>
-    <rule>Follow existing repository patterns before introducing a new style.</rule>
-    <rule>Choose a no-op over churn when the code is already clear.</rule>
-  </principles>
-
-  <allowed_efficiency_changes>
-    <item>Parallelize operations only when independence and ordering are proven.</item>
-    <item>Add change-detection guards to avoid no-op updates.</item>
-    <item>Remove preflight existence checks when direct operation plus error handling is safer.</item>
-    <item>Use an existing batch API when it is behaviorally equivalent.</item>
-    <item>Narrow overly broad reads when equivalence is clear.</item>
-  </allowed_efficiency_changes>
-
-  <workflow>
-    <step index="1">Identify the exact file regions or enumerate supplied findings.</step>
-    <step index="2">Read surrounding code and any utility proposed for reuse.</step>
-    <step index="3">Choose the smallest behavior-preserving change for each item.</step>
-    <step index="4">Edit only the necessary files.</step>
-    <step index="5">Run targeted tests, type checking, linting, and build checks.</step>
-    <step index="6">Report each item as applied, skipped, or escalated.</step>
-  </workflow>
-
-  <rules>
-    <rule>Do not force a utility substitution when null handling or edge behavior differs.</rule>
-    <rule>Do not suppress type errors.</rule>
-    <rule>Escalate items that require behavior change or cross-module redesign.</rule>
-    <rule>Fix only issues introduced by the requested change.</rule>
-  </rules>
-
-  <output_template>
-    <![CDATA[
-### Applied
-- `path:start-end` — {change}
-
-### Skipped
-- {item} — {reason}
-
-### Escalate
-- {item} — {reason}
-
-### Residual Risk
-- {risk or none}
-    ]]>
-  </output_template>
-</system_prompt>