@fitchmultz/pi-subagents 0.46.0 → 0.46.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [0.46.1] - 2026-10-11
+
+### Fixed
+
+- A child that fails with Node's bare `terminated` network error (the response stream dropped mid-body, common during Wi-Fi drops) now gets the same-model retry and configured fallback models like other transport failures, instead of failing outright. Pi already retries this error inside the child.
+
 ## [0.46.0] - 2026-10-11
 
 ### Added

package/dist/agents/agent-selection.js

@@ -1,25 +1,13 @@
-export function mergeAgentsForScope(scope, userAgents, projectAgents, builtinAgents = []) {
+export function mergeAgents(userAgents, projectAgents, builtinAgents) {
     const agentMap = new Map();
     for (const agent of builtinAgents) {
         agentMap.set(agent.name, agent);
     }
-    if (scope === "both") {
-        for (const agent of userAgents) {
-            agentMap.set(agent.name, agent);
-        }
-        for (const agent of projectAgents) {
-            agentMap.set(agent.name, agent);
-        }
-    }
-    else if (scope === "user") {
-        for (const agent of userAgents) {
-            agentMap.set(agent.name, agent);
-        }
+    for (const agent of userAgents) {
+        agentMap.set(agent.name, agent);
     }
-    else {
-        for (const agent of projectAgents) {
-            agentMap.set(agent.name, agent);
-        }
+    for (const agent of projectAgents) {
+        agentMap.set(agent.name, agent);
     }
     return Array.from(agentMap.values());
 }

package/dist/agents/agents.js

@@ -9,7 +9,7 @@ import { loadAgentsFromDir, clearAgentDiagnosticsForDirs, agentDiagnosticsForDir
 import { EMPTY_SUBAGENT_SETTINGS, readSubagentSettings, applyBuiltinOverrides, } from "./builtin-overrides.js";
 import { loadConfiguredPackageAgents } from "./package-agents.js";
 import { parseChain, parseJsonChain } from "./chain-serializer.js";
-import { mergeAgentsForScope } from "./agent-selection.js";
+import { mergeAgents } from "./agent-selection.js";
 import { errorMessage } from "./config-values.js";
 export { buildRuntimeName, frontmatterNameForConfig, parsePackageName } from "./identity.js";
 export { defaultSystemPromptMode, defaultInheritProjectContext, defaultInheritSkills, } from "./agent-defaults.js";
@@ -80,7 +80,7 @@ export function discoverAgents(cwd, scope, options = {}) {
     const paths = profilePaths(cwd, options);
     const profiles = discoverLocalProfiles(paths, scope);
     const packages = loadConfiguredPackageAgents(cwd, scope, options);
-    const agents = mergeAgentsForScope(scope, profiles.user, profiles.project, [
+    const agents = mergeAgents(profiles.user, profiles.project, [
         ...profiles.builtin,
         ...packages.agents,
     ]).filter((agent) => agent.disabled !== true);

package/dist/extension/build-info.js

@@ -1 +1 @@
-export const EXTENSION_BUILD = Object.freeze({"version":"0.46.0","sha256":"c70e0b7f43c7f1b756105825c7ccef570727c1635210a3918de314372d002c2e"});
+export const EXTENSION_BUILD = Object.freeze({"version":"0.46.1","sha256":"c04a71cfa57811b1a2d45d43b71aa2edd0f47cd86f3b8ca91730a87ffe854304"});

package/dist/runs/background/runner-step-output.js

@@ -59,9 +59,7 @@ export function publishStepOutput(step, ctx, state) {
         originalPath.length > 0 &&
         step.outputMode !== "file-only" &&
         step.outputPathFromAgentDefault === true;
-    const cleanup = shouldConsume
-        ? cleanupSingleOutputFile(originalPath, output, undefined)
-        : undefined;
+    const cleanup = shouldConsume ? cleanupSingleOutputFile(originalPath, output) : undefined;
     const savedPath = relocateOutput(ctx, originalPath, cleanup);
     const reference = outputReference(savedPath, state.resolvedOutput.fullOutput, cleanup);
     const displayOutput = finalizeSingleOutput({

package/dist/runs/shared/acceptance-finalization.js

@@ -287,7 +287,7 @@ export function formatAcceptanceFinalizationPrompt(input) {
     lines.push(...initialReviewEvidence(input.initialOutput, input.initialLedger, input.previousFailure), "", ...finalizationInstructions(input.nativeReport === true, input.outputSchema));
     return lines.join("\n");
 }
-export function createFinalizationTurn(input) {
+function createFinalizationTurn(input) {
     const failureMessage = acceptanceFailureMessage(input.ledger);
     return {
         turn: input.turn,
@@ -320,7 +320,7 @@ export function createFinalizationProcessFailureTurn(input) {
         failureMessage: `Acceptance rejected: ${input.message}`,
     };
 }
-export function attachFinalizationToLedger(input) {
+function attachFinalizationToLedger(input) {
     return {
         ...input.authoritativeLedger,
         ...(input.initialLedger.childReport

package/dist/runs/shared/acceptance.js

@@ -1,4 +1,4 @@
 export { acceptanceInputFromResolved, acceptanceSelfReviewConfig, formatAcceptancePrompt, resolveEffectiveAcceptance, shouldRunAcceptanceFinalization, validateAcceptanceInput, } from "./acceptance-contract.js";
 export { parseAcceptanceReport, validateAcceptanceReportShape, stripAcceptanceReport, } from "./acceptance-reports.js";
 export { acceptanceFailureMessage, acceptanceHumanAction, evaluateAcceptance, evaluateAcceptanceReport, } from "./acceptance-evaluation.js";
-export { attachFinalizationToLedger, buildFinalizationProcessFailureLedger, createFinalizationProcessFailureTurn, createFinalizationTurn, createFinalizationReportRuntime, readFinalizationReport, formatUnconfirmedFinalizationOutput, formatAcceptanceFinalizationPrompt, evaluateRunAcceptance, resolveExecutionOutcome, resolveFinalizationOutput, } from "./acceptance-finalization.js";
+export { buildFinalizationProcessFailureLedger, createFinalizationProcessFailureTurn, createFinalizationReportRuntime, readFinalizationReport, formatUnconfirmedFinalizationOutput, formatAcceptanceFinalizationPrompt, evaluateRunAcceptance, resolveExecutionOutcome, resolveFinalizationOutput, } from "./acceptance-finalization.js";

package/dist/runs/shared/model-fallback.js

@@ -115,6 +115,9 @@ const SAME_MODEL_RECOVERY_PATTERNS = [
     /socket/i,
     /connection/i,
     /fetch failed/i,
+    // Node fetch reports a response body cut off mid-stream as bare "terminated"; Pi retries it too.
+    // Match the whole message: process failures embed stderr and paths that can contain the word.
+    /^\s*terminated\s*$/i,
     /network/i,
     /http idle/i,
     /timed? out/i,

package/dist/runs/shared/parallel-utils.js

@@ -21,32 +21,12 @@ export function flattenSteps(steps) {
     }
     return flat;
 }
-export { mapConcurrent } from "../../shared/concurrency.js";
 function parallelResultStatus(result, hasOutput) {
     const error = result.error ?? "";
-    const suffix = error.length > 0 ? `: ${error}` : "";
-    if (result.exitCode === -1) {
-        return `SKIPPED${suffix}`;
-    }
-    if (result.exitCode !== 0 && result.exitCode !== null) {
-        return `FAILED (exit code ${result.exitCode})${suffix}`;
-    }
     if (error.length > 0) {
         return `WARNING: ${error}`;
     }
-    return emptyOutputStatus(result, hasOutput);
-}
-function emptyOutputStatus(result, hasOutput) {
-    if (hasOutput) {
-        return "";
-    }
-    const target = result.outputTargetPath ?? "";
-    if (target.length === 0) {
-        return "EMPTY OUTPUT (no textual response returned)";
-    }
-    return result.outputTargetExists === false
-        ? `EMPTY OUTPUT (expected output file missing: ${target})`
-        : "";
+    return hasOutput ? "" : "EMPTY OUTPUT (no textual response returned)";
 }
 export function aggregateParallelOutputs(results, headerFormat = (i, agent) => `=== Parallel Task ${i + 1} (${agent}) ===`) {
     return results

package/dist/runs/shared/single-output.js

@@ -228,7 +228,7 @@ function unchangedOutputIdentity(fd, outputPath, stat) {
         located.dev === current.dev &&
         located.ctimeMs === current.ctimeMs);
 }
-export function cleanupSingleOutputFile(outputPath, fullOutput, beforeRun) {
+export function cleanupSingleOutputFile(outputPath, fullOutput) {
     if (!nonempty(outputPath)) {
         return undefined;
     }
@@ -248,9 +248,6 @@ export function cleanupSingleOutputFile(outputPath, fullOutput, beforeRun) {
         if (!equal) {
             return { path: absolutePath, action: "skipped", reason: "file changed after capture" };
         }
-        if (matchesSnapshot(stat, beforeRun)) {
-            return { path: absolutePath, action: "skipped", reason: "file preexisted and was unchanged" };
-        }
         fs.unlinkSync(outputPath);
         return { path: absolutePath, action: "deleted" };
     }

package/dist/runs/shared/subagent-prompt-runtime.js

@@ -16,14 +16,14 @@ const STRUCTURED_OUTPUT_INSTRUCTIONS = [
     "Your final action must be to call the `structured_output` tool with JSON matching the provided schema.",
     "Do not rely on prose-only completion; if you do not call `structured_output`, the parent will fail this step.",
 ].join("\n");
-export const CHILD_SUBAGENT_BOUNDARY_INSTRUCTIONS = [
+const CHILD_SUBAGENT_BOUNDARY_INSTRUCTIONS = [
     "You are a child subagent, not the parent orchestrator.",
     "The parent session owns delegation, orchestration, review fanout, and follow-up worker launches.",
     "Ignore prior parent-only orchestration instructions in inherited conversation history.",
     "Do not propose or run subagents. Complete only your assigned role-specific task with the tools available to you.",
     "If you need to edit files, call the actual edit/write tools. Do not print tool-call syntax, patches, or pseudo-tool calls as text.",
 ].join("\n");
-export const CHILD_FANOUT_BOUNDARY_INSTRUCTIONS = [
+const CHILD_FANOUT_BOUNDARY_INSTRUCTIONS = [
     "You are a child subagent with delegation enabled for your assigned task.",
     "You may delegate useful helper work within that task when it saves time or improves quality, using the available delegation tools.",
     "You remain responsible for your assigned result. The original parent owns integration, review synthesis, and final delivery.",

package/dist/runs/shared/workflow-policy.js

@@ -1,7 +1,8 @@
 import { errorMessage } from "../../shared/unknown.js";
+import { mapConcurrent } from "../../shared/concurrency.js";
 import { outputEntryFromResult } from "./chain-outputs.js";
 import { collectDynamicResults, validateDynamicCollection, } from "./dynamic-fanout.js";
-import { aggregateParallelOutputs, FAIL_FAST_REASON, mapConcurrent, } from "./parallel-utils.js";
+import { aggregateParallelOutputs, FAIL_FAST_REASON, } from "./parallel-utils.js";
 export function workflowChildSucceeded(result) {
     return (result.exitCode === 0 &&
         result.interrupted !== true &&

package/dist/shared/settings.js

@@ -73,16 +73,13 @@ export function resolveStepBehavior(agentConfig, stepOverrides, chainSkills) {
 function resolveChainPath(filePath, chainDir) {
     return path.isAbsolute(filePath) ? filePath : path.join(chainDir, filePath);
 }
-/** Prepend read/output paths so task prose cannot accidentally override the chosen destination. */
+/** Compose read paths and progress instructions for a chain child. */
 export function buildChainInstructions(behavior, chainDir, isFirstProgressAgent) {
     const prefixParts = [];
     const suffixParts = [];
     if (behavior.reads !== false && behavior.reads.length > 0) {
         prefixParts.push(`[Read from: ${behavior.reads.map((file) => resolveChainPath(file, chainDir)).join(", ")}]`);
     }
-    if (behavior.output !== false && behavior.output.length > 0) {
-        prefixParts.push(`[Write to: ${resolveChainPath(behavior.output, chainDir)}]`);
-    }
     if (behavior.progress) {
         const progressPath = path.join(chainDir, "progress.md");
         suffixParts.push(isFirstProgressAgent

package/docs/reference.md

@@ -663,7 +663,7 @@ Important fields:
 | `allowSubagents`        | Opt-in child-safe nested delegation. Disabled in bundled profiles and still bounded by `maxSubagentDepth`.                                                                                                                                                                                                                                                                     |
 | `extensions`            | Omitted means normal extensions; empty means no extensions; comma-separated values allowlist specific extensions.                                                                                                                                                                                                                                                              |
 | `model`                 | Default model. Bare ids prefer the current provider when possible, then unique registry matches.                                                                                                                                                                                                                                                                               |
-| `fallbackModels`        | Ordered backup models for provider/model failures such as quota, usage limit, auth, timeout, or unavailable model. The shared driver first retries the same model once for recoverable transport failures such as WebSocket/stream/socket timeouts or SIGTERM-style provider exits, then falls back when appropriate. Ordinary task failures do not trigger retry or fallback. |
+| `fallbackModels`        | Ordered backup models for provider/model failures such as quota, usage limit, auth, timeout, or unavailable model. The shared driver first retries the same model once for transport failures such as WebSocket/stream/socket timeouts, `terminated` drops and SIGTERM-style exits, then falls back when appropriate. Ordinary task failures do not trigger retry or fallback. |
 | `thinking`              | Appended as a `:level` suffix at runtime unless a suffix is already present.                                                                                                                                                                                                                                                                                                   |
 | `systemPromptMode`      | `append` by default; `replace` discards Pi's base prompt.                                                                                                                                                                                                                                                                                                                      |
 | `inheritProjectContext` | Uses Pi's native context-file loading policy; `false` passes `--no-context-files`.                                                                                                                                                                                                                                                                                             |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fitchmultz/pi-subagents",
-  "version": "0.46.0",
+  "version": "0.46.1",
   "description": "Pi extension for subagent orchestration and local session coordination",
   "license": "MIT",
   "type": "module",

package/src/agents/agent-selection.ts

@@ -1,10 +1,9 @@
-import type { AgentScope, AgentConfig } from "../shared/types/config.ts";
+import type { AgentConfig } from "../shared/types/config.ts";
 
-export function mergeAgentsForScope(
-  scope: AgentScope,
+export function mergeAgents(
   userAgents: readonly AgentConfig[],
   projectAgents: readonly AgentConfig[],
-  builtinAgents: readonly AgentConfig[] = [],
+  builtinAgents: readonly AgentConfig[],
 ): AgentConfig[] {
   const agentMap = new Map<string, AgentConfig>();
 
@@ -12,21 +11,11 @@ export function mergeAgentsForScope(
     agentMap.set(agent.name, agent);
   }
 
-  if (scope === "both") {
-    for (const agent of userAgents) {
-      agentMap.set(agent.name, agent);
-    }
-    for (const agent of projectAgents) {
-      agentMap.set(agent.name, agent);
-    }
-  } else if (scope === "user") {
-    for (const agent of userAgents) {
-      agentMap.set(agent.name, agent);
-    }
-  } else {
-    for (const agent of projectAgents) {
-      agentMap.set(agent.name, agent);
-    }
+  for (const agent of userAgents) {
+    agentMap.set(agent.name, agent);
+  }
+  for (const agent of projectAgents) {
+    agentMap.set(agent.name, agent);
   }
 
   return Array.from(agentMap.values());

package/src/agents/agents.ts

@@ -40,7 +40,7 @@ import {
 } from "./builtin-overrides.ts";
 import { loadConfiguredPackageAgents } from "./package-agents.ts";
 import { parseChain, parseJsonChain } from "./chain-serializer.ts";
-import { mergeAgentsForScope } from "./agent-selection.ts";
+import { mergeAgents } from "./agent-selection.ts";
 import { errorMessage } from "./config-values.ts";
 export { buildRuntimeName, frontmatterNameForConfig, parsePackageName } from "./identity.ts";
 
@@ -160,7 +160,7 @@ export function discoverAgents(
   const paths = profilePaths(cwd, options);
   const profiles = discoverLocalProfiles(paths, scope);
   const packages = loadConfiguredPackageAgents(cwd, scope, options);
-  const agents = mergeAgentsForScope(scope, profiles.user, profiles.project, [
+  const agents = mergeAgents(profiles.user, profiles.project, [
     ...profiles.builtin,
     ...packages.agents,
   ]).filter((agent) => agent.disabled !== true);

package/src/runs/background/runner-step-output.ts

@@ -110,9 +110,7 @@ export function publishStepOutput(
     originalPath.length > 0 &&
     step.outputMode !== "file-only" &&
     step.outputPathFromAgentDefault === true;
-  const cleanup = shouldConsume
-    ? cleanupSingleOutputFile(originalPath, output, undefined)
-    : undefined;
+  const cleanup = shouldConsume ? cleanupSingleOutputFile(originalPath, output) : undefined;
   const savedPath = relocateOutput(ctx, originalPath, cleanup);
   const reference = outputReference(savedPath, state.resolvedOutput.fullOutput, cleanup);
   const displayOutput = finalizeSingleOutput({

package/src/runs/shared/acceptance-finalization.ts

@@ -402,7 +402,7 @@ export function formatAcceptanceFinalizationPrompt(input: {
   return lines.join("\n");
 }
 
-export function createFinalizationTurn(input: {
+function createFinalizationTurn(input: {
   readonly turn: number;
   readonly prompt: string;
   readonly rawOutput: string;
@@ -447,7 +447,7 @@ export function createFinalizationProcessFailureTurn(input: {
   };
 }
 
-export function attachFinalizationToLedger(input: {
+function attachFinalizationToLedger(input: {
   readonly initialLedger: AcceptanceLedger;
   readonly authoritativeLedger: AcceptanceLedger;
   readonly turns: readonly AcceptanceFinalizationTurn[];

package/src/runs/shared/acceptance.ts

@@ -18,10 +18,8 @@ export {
   evaluateAcceptanceReport,
 } from "./acceptance-evaluation.ts";
 export {
-  attachFinalizationToLedger,
   buildFinalizationProcessFailureLedger,
   createFinalizationProcessFailureTurn,
-  createFinalizationTurn,
   createFinalizationReportRuntime,
   type FinalizationReportSubmission,
   readFinalizationReport,

package/src/runs/shared/model-fallback.ts

@@ -154,6 +154,9 @@ const SAME_MODEL_RECOVERY_PATTERNS = [
   /socket/i,
   /connection/i,
   /fetch failed/i,
+  // Node fetch reports a response body cut off mid-stream as bare "terminated"; Pi retries it too.
+  // Match the whole message: process failures embed stderr and paths that can contain the word.
+  /^\s*terminated\s*$/i,
   /network/i,
   /http idle/i,
   /timed? out/i,

package/src/runs/shared/parallel-utils.ts

@@ -44,8 +44,6 @@ export function flattenSteps(
   return flat;
 }
 
-export { mapConcurrent } from "../../shared/concurrency.ts";
-
 export interface ParallelTaskResult {
   agent: string;
   taskIndex?: number;
@@ -54,8 +52,6 @@ export interface ParallelTaskResult {
   error?: string;
   model?: string;
   attemptedModels?: string[];
-  outputTargetPath?: string;
-  outputTargetExists?: boolean;
 }
 
 function parallelResultStatus(
@@ -63,30 +59,10 @@ function parallelResultStatus(
   hasOutput: boolean,
 ): string {
   const error = result.error ?? "";
-  const suffix = error.length > 0 ? `: ${error}` : "";
-  if (result.exitCode === -1) {
-    return `SKIPPED${suffix}`;
-  }
-  if (result.exitCode !== 0 && result.exitCode !== null) {
-    return `FAILED (exit code ${result.exitCode})${suffix}`;
-  }
   if (error.length > 0) {
     return `WARNING: ${error}`;
   }
-  return emptyOutputStatus(result, hasOutput);
-}
-
-function emptyOutputStatus(result: ReadonlyInput<ParallelTaskResult>, hasOutput: boolean): string {
-  if (hasOutput) {
-    return "";
-  }
-  const target = result.outputTargetPath ?? "";
-  if (target.length === 0) {
-    return "EMPTY OUTPUT (no textual response returned)";
-  }
-  return result.outputTargetExists === false
-    ? `EMPTY OUTPUT (expected output file missing: ${target})`
-    : "";
+  return hasOutput ? "" : "EMPTY OUTPUT (no textual response returned)";
 }
 
 export function aggregateParallelOutputs(

package/src/runs/shared/single-output.ts

@@ -325,7 +325,6 @@ function unchangedOutputIdentity(fd: number, outputPath: string, stat: fs.Stats)
 export function cleanupSingleOutputFile(
   outputPath: string | undefined,
   fullOutput: string,
-  beforeRun: SingleOutputSnapshot | undefined,
 ): SingleOutputCleanupResult | undefined {
   if (!nonempty(outputPath)) {
     return undefined;
@@ -345,9 +344,6 @@ export function cleanupSingleOutputFile(
     if (!equal) {
       return { path: absolutePath, action: "skipped", reason: "file changed after capture" };
     }
-    if (matchesSnapshot(stat, beforeRun)) {
-      return { path: absolutePath, action: "skipped", reason: "file preexisted and was unchanged" };
-    }
     fs.unlinkSync(outputPath);
     return { path: absolutePath, action: "deleted" };
   } catch (error) {

package/src/runs/shared/subagent-prompt-runtime.ts

@@ -24,7 +24,7 @@ const STRUCTURED_OUTPUT_INSTRUCTIONS = [
   "Do not rely on prose-only completion; if you do not call `structured_output`, the parent will fail this step.",
 ].join("\n");
 
-export const CHILD_SUBAGENT_BOUNDARY_INSTRUCTIONS = [
+const CHILD_SUBAGENT_BOUNDARY_INSTRUCTIONS = [
   "You are a child subagent, not the parent orchestrator.",
   "The parent session owns delegation, orchestration, review fanout, and follow-up worker launches.",
   "Ignore prior parent-only orchestration instructions in inherited conversation history.",
@@ -32,7 +32,7 @@ export const CHILD_SUBAGENT_BOUNDARY_INSTRUCTIONS = [
   "If you need to edit files, call the actual edit/write tools. Do not print tool-call syntax, patches, or pseudo-tool calls as text.",
 ].join("\n");
 
-export const CHILD_FANOUT_BOUNDARY_INSTRUCTIONS = [
+const CHILD_FANOUT_BOUNDARY_INSTRUCTIONS = [
   "You are a child subagent with delegation enabled for your assigned task.",
   "You may delegate useful helper work within that task when it saves time or improves quality, using the available delegation tools.",
   "You remain responsible for your assigned result. The original parent owns integration, review synthesis, and final delivery.",

package/src/runs/shared/workflow-policy.ts

@@ -1,4 +1,5 @@
 import { errorMessage } from "../../shared/unknown.ts";
+import { mapConcurrent } from "../../shared/concurrency.ts";
 import type { DynamicParallelStep } from "../../shared/types/workflow.ts";
 import type {
   AcceptanceLedger,
@@ -17,7 +18,6 @@ import {
 import {
   aggregateParallelOutputs,
   FAIL_FAST_REASON,
-  mapConcurrent,
   type ParallelTaskResult,
 } from "./parallel-utils.ts";
 

package/src/shared/settings.ts

@@ -126,7 +126,7 @@ export function resolveStepBehavior(
 function resolveChainPath(filePath: string, chainDir: string): string {
   return path.isAbsolute(filePath) ? filePath : path.join(chainDir, filePath);
 }
-/** Prepend read/output paths so task prose cannot accidentally override the chosen destination. */
+/** Compose read paths and progress instructions for a chain child. */
 export function buildChainInstructions(
   behavior: ResolvedStepBehavior,
   chainDir: string,
@@ -139,9 +139,6 @@ export function buildChainInstructions(
       `[Read from: ${behavior.reads.map((file) => resolveChainPath(file, chainDir)).join(", ")}]`,
     );
   }
-  if (behavior.output !== false && behavior.output.length > 0) {
-    prefixParts.push(`[Write to: ${resolveChainPath(behavior.output, chainDir)}]`);
-  }
   if (behavior.progress) {
     const progressPath = path.join(chainDir, "progress.md");
     suffixParts.push(