@henryqw/pi-subagent 2.5.1 → 2.9.5

package/CONTEXT.md

@@ -17,8 +17,8 @@ Provide validated user Roles, shared task-model Pi launch policy, generic manage
 
 ## Invariants
 
-- One Delegated Task creates one ephemeral child process and no saved session. Its soft deadline is 10 minutes; active model/tool execution or activity within the last minute grants one 5-minute grace period before a hard stop.
-- Up to four active ephemeral `delegate_task` children run per Main; excess calls wait FIFO. Queued calls do not start a child or consume child timeout. Managed Herdr workers are unaffected.
+- One Delegated Task creates one ephemeral child process and no saved session. Default timeouts: 10-minute soft deadline; active model/tool execution or activity within the last minute grants one 5-minute grace period before a hard stop. Every value is configurable via the `timeout` object in `~/.pi/agent/config/pi-subagent.json` (`softMinutes`, `graceMinutes`, `activeWindowSeconds`).
+- Up to five active ephemeral `delegate_task` children run per Main by default, configurable via `maxSubagents` in `~/.pi/agent/config/pi-subagent.json` or the `PI_SUBAGENT_MAX_SUBAGENTS` environment variable; excess calls wait FIFO. Queued calls do not start a child or consume child timeout. Managed Herdr workers are unaffected.
 - Ambient child extensions and Skills stay disabled; Role explicitly selects extension sources and named Skills. Pi loads Skills supplied by those extension packages or their resource discovery. Omitted Role tools use Pi's effective `defaultTools`; an explicit list sets base tools while loaded extension tools activate automatically.
 - Role Skill names resolve through Main's effective Pi Skill registry; unavailable names warn and skip without blocking delegation.
 - Main selects Role and may override Model Class per task; omitted class uses shared `pi-subagent/delegateTask` assignment, initially `balanced`. Library callers select Role plus their own shared task ID.

package/README.md

@@ -27,7 +27,7 @@ An explicit `model` (`provider/modelId`) overrides `modelClass` and resolves aga
 
 `modelClass` is `fast`, `balanced`, `frontier`, or `fav`. Omitted class uses the shared `pi-subagent/delegateTask` assignment, which defaults to `balanced`. Primary route is resolved against current scoped text models; fallback is tried only before launch. If no route is usable, delegation rejects with `Run /task-models`. A started child is never retried.
 
-Main splits broad work into independent bounded tasks and keeps integration and cross-cutting decisions. Each `task` states its objective, exact scope and exclusions, relevant context and constraints, expected deliverable, and validation. Each call uses the least capable `modelClass` that can reliably complete its task. Independent sibling calls can run concurrently; concurrent edit tasks must own non-overlapping files. Up to four active ephemeral `delegate_task` children run per Main; excess calls wait FIFO. Queued calls do not start a child or consume child timeout. Managed Herdr workers are unaffected.
+Main splits broad work into independent bounded tasks and keeps integration and cross-cutting decisions. Each `task` states its objective, exact scope and exclusions, relevant context and constraints, expected deliverable, and validation. Each call uses the least capable `modelClass` that can reliably complete its task. Independent sibling calls can run concurrently; concurrent edit tasks must own non-overlapping files. Up to five active ephemeral `delegate_task` subagents run per Main; excess calls wait FIFO. Configure the cap with `"maxSubagents"` (positive integer) in `~/.pi/agent/config/pi-subagent.json`, or override per session with the `PI_SUBAGENT_MAX_SUBAGENTS` environment variable. Child timeouts are configurable with a `"timeout"` object: `{ "softMinutes": 20, "graceMinutes": 10, "activeWindowSeconds": 90 }` (all keys optional; defaults 10/5/60). Invalid config values fall back to the default with a warning; an invalid environment variable fails fast. Queued calls do not start a child or consume child timeout. Managed Herdr workers are unaffected.
 
 Each call starts one isolated child (`pi --mode json -p --no-session`). Ambient extensions and Skills are off. Role/caller extensions load; those packages' tools and Skills auto-load, plus any extra `skills` names. Child uses the delegated working directory and Main's project approval. Abort kills the child process group. An inactive child times out after 10 minutes; current model/tool execution or activity in the last minute grants one 5-minute grace period, then the child stops. Streaming output is capped at 50 KiB. Unused JSON event types are discarded before payload buffering; consumed or unclassifiable events above 1 MiB fail delegation.
 

package/extensions/config.ts

@@ -0,0 +1,131 @@
+import { readFileSync } from "node:fs";
+import { join } from "node:path";
+import { getAgentDir } from "@earendil-works/pi-coding-agent";
+
+export interface SubagentTimeoutConfig {
+	/** Soft deadline in minutes; the child is asked to stop after this. */
+	softMinutes?: number;
+	/** Extra minutes past the soft deadline before a stuck child is killed. */
+	graceMinutes?: number;
+	/** Activity window in seconds that qualifies an active child for grace. */
+	activeWindowSeconds?: number;
+}
+
+export interface SubagentConfig {
+	maxSubagents?: number;
+	timeout?: SubagentTimeoutConfig;
+}
+
+export interface LoadedSubagentConfig {
+	config: SubagentConfig;
+	/** Human-readable problems when the file exists but is partly unusable; the file is never rewritten. */
+	error?: string;
+}
+
+const positive = (value: unknown): value is number =>
+	typeof value === "number" && Number.isFinite(value) && value > 0;
+
+// Node clamps setTimeout delays above 2^31 - 1 ms to 1 ms, which would kill
+// every child immediately instead of applying the configured deadline.
+const MAX_TIMER_DELAY_MS = 2_147_483_647;
+// Defaults for fields the user left unset, used when bounding the combined
+// soft + grace hard-deadline delay.
+const DEFAULT_SOFT_MINUTES = 10;
+const DEFAULT_GRACE_MINUTES = 5;
+const TIMEOUT_FIELDS: Array<[keyof SubagentTimeoutConfig, number, string]> = [
+	["softMinutes", 60_000, "minutes"],
+	["graceMinutes", 60_000, "minutes"],
+	["activeWindowSeconds", 1_000, "seconds"],
+];
+
+/**
+ * Required single-extension config path form (AGENTS.md); agentDir is
+ * injectable so tests can point at a temp directory.
+ */
+export const configPath = (agentDir = getAgentDir()): string =>
+	join(agentDir, "config", "pi-subagent.json");
+
+/**
+ * Read the optional user config at `<agentDir>/config/pi-subagent.json`.
+ * Treated as untrusted user data: malformed files are preserved untouched and
+ * reported instead of crashing the session; callers fall back to defaults.
+ */
+export function readSubagentConfig(agentDir = getAgentDir()): LoadedSubagentConfig {
+	const path = configPath(agentDir);
+	let raw: string;
+	try {
+		raw = readFileSync(path, "utf8");
+	} catch (error) {
+		if ((error as NodeJS.ErrnoException).code === "ENOENT") return { config: {} };
+		throw error;
+	}
+	let parsed: unknown;
+	try {
+		parsed = JSON.parse(raw);
+	} catch (error) {
+		return {
+			config: {},
+			error: `${path} is not valid JSON (${error instanceof Error ? error.message : String(error)}); using defaults.`,
+		};
+	}
+	if (!parsed || typeof parsed !== "object" || Array.isArray(parsed)) {
+		return { config: {}, error: `${path} must contain a JSON object; using defaults.` };
+	}
+	const record = parsed as Record<string, unknown>;
+	const problems: string[] = [];
+	const config: SubagentConfig = {};
+	for (const key of Object.keys(record)) {
+		if (key !== "maxSubagents" && key !== "timeout") {
+			problems.push(`unknown config key ${JSON.stringify(key)}; expected maxSubagents, timeout`);
+		}
+	}
+
+	if (record.maxSubagents !== undefined) {
+		if (typeof record.maxSubagents === "number" && Number.isSafeInteger(record.maxSubagents) && record.maxSubagents >= 1) {
+			config.maxSubagents = record.maxSubagents;
+		} else {
+			problems.push(`maxSubagents must be a safe integer >= 1, got ${JSON.stringify(record.maxSubagents)}`);
+		}
+	}
+
+	if (record.timeout !== undefined) {
+		if (!record.timeout || typeof record.timeout !== "object" || Array.isArray(record.timeout)) {
+			problems.push(`timeout must be a JSON object, got ${JSON.stringify(record.timeout)}`);
+		} else {
+			const timeoutRecord = record.timeout as Record<string, unknown>;
+			const timeout: SubagentTimeoutConfig = {};
+			for (const [key, unitMs, unit] of TIMEOUT_FIELDS) {
+				const value = timeoutRecord[key];
+				if (value === undefined) continue;
+				if (!positive(value)) {
+					problems.push(`timeout.${key} must be a positive number of ${unit}, got ${JSON.stringify(value)}`);
+				} else if (value * unitMs > MAX_TIMER_DELAY_MS) {
+					problems.push(`timeout.${key} exceeds the maximum supported delay of ${MAX_TIMER_DELAY_MS} ms, got ${JSON.stringify(value)} ${unit}`);
+				} else {
+					(timeout as Record<string, number>)[key] = value;
+				}
+			}
+			for (const key of Object.keys(timeoutRecord)) {
+				if (!TIMEOUT_FIELDS.some(([known]) => known === key)) {
+					problems.push(`unknown timeout.${key}; expected ${TIMEOUT_FIELDS.map(([known]) => known).join(", ")}`);
+				}
+			}
+			if (Object.keys(timeout).length) config.timeout = timeout;
+		}
+	}
+
+	// The hard deadline schedules softMs + graceMs in one timer; the combination
+	// must stay inside Node's limit even when each field fits alone. Falling back
+	// to defaults drops the whole timeout object, matching other invalid values.
+	if (config.timeout) {
+		const hardDeadlineMinutes =
+			(config.timeout.softMinutes ?? DEFAULT_SOFT_MINUTES) +
+			(config.timeout.graceMinutes ?? DEFAULT_GRACE_MINUTES);
+		if (hardDeadlineMinutes * 60_000 > MAX_TIMER_DELAY_MS) {
+			problems.push(`timeout softMinutes + graceMinutes must stay within ${MAX_TIMER_DELAY_MS} ms combined, got ${hardDeadlineMinutes} minutes`);
+			delete config.timeout;
+		}
+	}
+
+	return { config, error: problems.length ? `${path}: ${problems.join("; ")}; using defaults.` : undefined };
+}

package/extensions/subagent.ts

@@ -4,6 +4,7 @@ import { basename } from "node:path";
 import { StringEnum } from "@earendil-works/pi-ai";
 import { type ExtensionAPI, type ExtensionContext, type Theme } from "@earendil-works/pi-coding-agent";
 import { type Component, truncateToWidth, type TUI, visibleWidth } from "@earendil-works/pi-tui";
+import { readSubagentConfig, type SubagentTimeoutConfig } from "./config.ts";
 import {
 	availableTaskModels,
 	modelReference,
@@ -20,7 +21,6 @@ const MODEL_CLASSES = PROFILE_NAMES;
 const SUBAGENT_TASK = "pi-subagent/delegateTask";
 const MAX_OUTPUT_BYTES = 50 * 1024;
 const MAX_JSON_EVENT_BYTES = 1024 * 1024;
-const MAX_ACTIVE_CHILDREN = 4;
 const CONSUMED_JSON_EVENTS = new Set(["message_start", "message_update", "message_end"]);
 const JSON_EVENT_TYPE = /^\s*\{\s*"type"\s*:\s*"([^"\\]+)"/;
 const WIDGET_KEY = "subagent-status";
@@ -35,6 +35,16 @@ const DEFAULT_TIMEOUT_POLICY = {
 const SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
 
 type TimeoutPolicy = typeof DEFAULT_TIMEOUT_POLICY;
+
+/** Merge validated config-file timeout fields over defaults; absent keys keep defaults. */
+export function resolveTimeoutPolicy(partial: SubagentTimeoutConfig | undefined): TimeoutPolicy {
+	const policy: TimeoutPolicy = { ...DEFAULT_TIMEOUT_POLICY };
+	if (!partial) return policy;
+	if (partial.softMinutes !== undefined) policy.softMs = partial.softMinutes * 60_000;
+	if (partial.graceMinutes !== undefined) policy.graceMs = partial.graceMinutes * 60_000;
+	if (partial.activeWindowSeconds !== undefined) policy.activeWindowMs = partial.activeWindowSeconds * 1_000;
+	return policy;
+}
 type ModelClass = ProfileName;
 class SubagentTimeoutError extends Error {}
 type ChildResult = {
@@ -429,6 +439,9 @@ const Parameters = Type.Object({
 	modelClass: Type.Optional(StringEnum(MODEL_CLASSES, {
 		description: "Classify task complexity: fast for narrow lookups or mechanical edits; balanced for normal bounded work; frontier for ambiguous, cross-cutting, or high-risk reasoning; fav for the user's favorite model when they ask for it. Defaults to the shared pi-subagent/delegateTask assignment.",
 	})),
+	background: Type.Optional(Type.Boolean({
+		description: "Run without blocking: returns a task ID immediately and delivers the outcome as a message when the Subagent settles. Prefer blocking delegation whenever the parent needs the result to continue.",
+	})),
 });
 
 function resolveDesignatedRoute(ctx: ExtensionContext, reference: string): ResolvedTaskRoute {
@@ -441,6 +454,8 @@ function resolveDesignatedRoute(ctx: ExtensionContext, reference: string): Resol
 	return { model, thinkingLevel: levels.includes("medium") ? "medium" : levels.at(-1)! };
 }
 
+const BACKGROUND_RESULT_TYPE = "subagent-background-result";
+
 const roleSummary = (): string => {
 	try {
 		const roles = loadRoles();
@@ -452,14 +467,47 @@ const roleSummary = (): string => {
 
 export default function subagentExtension(
 	pi: ExtensionAPI,
-	timeoutPolicy: TimeoutPolicy = DEFAULT_TIMEOUT_POLICY,
+	overrideTimeoutPolicy?: TimeoutPolicy,
 ): void {
 	const widgetItems = new Map<string, WidgetItem>();
+	// Each child is a full Pi process issuing its own model calls; cap parallel
+	// spend. Precedence: PI_SUBAGENT_MAX_SUBAGENTS env > config/pi-subagent.json
+	// maxSubagents > default 5. Invalid config falls back to the default and is
+	// reported once the UI exists; an invalid env value fails fast.
+	const loadedConfig = readSubagentConfig();
+	const startupWarnings = [loadedConfig.error].filter((message): message is string => message !== undefined);
+	let maxActiveSubagents = loadedConfig.config.maxSubagents ?? 5;
+	const maxSubagentsRaw = process.env.PI_SUBAGENT_MAX_SUBAGENTS;
+	if (maxSubagentsRaw !== undefined) {
+		// Reject "2workers", "1.5", "1e3" — parseInt would silently accept prefixes —
+		// and digit strings that overflow to Infinity, which would disable the cap.
+		if (!/^\d+$/.test(maxSubagentsRaw) || !/^[1-9]\d*$/.test(maxSubagentsRaw)) {
+			throw new Error(`PI_SUBAGENT_MAX_SUBAGENTS must be a positive integer, got ${JSON.stringify(maxSubagentsRaw)}.`);
+		}
+		const parsed = Number.parseInt(maxSubagentsRaw, 10);
+		if (!Number.isSafeInteger(parsed)) {
+			throw new Error(`PI_SUBAGENT_MAX_SUBAGENTS exceeds the supported range, got ${JSON.stringify(maxSubagentsRaw)}.`);
+		}
+		maxActiveSubagents = parsed;
+	}
+	let backgroundSequence = 0;
+	// Explicit policy argument (tests/embedders) wins; otherwise resolve from
+	// config file over defaults.
+	const timeoutPolicy: TimeoutPolicy = overrideTimeoutPolicy ?? resolveTimeoutPolicy(loadedConfig.config.timeout);
+	// Background children outlive the launching tool call, so they get their own
+	// abort signal: tied to the session, not to the turn that started them.
+	const backgroundTasks = new Map<string, AbortController>();
+	// Latest known session context; refreshed on session lifecycle and model
+	// changes so queued background launches resolve against effective state.
+	let latestCtx: ExtensionContext | undefined;
+	// Bumped by session_start and session_shutdown; background tasks may only
+	// deliver into the exact session that launched them.
+	let sessionEpoch = 0;
 	let activeChildren = 0;
 	const queuedChildren: Array<() => void> = [];
 	const acquireChildPermit = (signal: AbortSignal | undefined): Promise<void> => {
 		if (signal?.aborted) return Promise.reject(new Error("Subagent was aborted."));
-		if (activeChildren < MAX_ACTIVE_CHILDREN) {
+		if (activeChildren < maxActiveSubagents) {
 			activeChildren++;
 			return Promise.resolve();
 		}
@@ -561,7 +609,10 @@ export default function subagentExtension(
 	};
 
 	pi.on("session_start", (_event, ctx) => {
+		sessionEpoch += 1;
+		latestCtx = ctx;
 		ensureWidget(ctx);
+		for (const warning of startupWarnings.splice(0)) ctx.ui.notify(warning, "warning");
 	});
 	pi.on("session_shutdown", (_event, ctx) => {
 		stopWidgetTimer();
@@ -569,18 +620,54 @@ export default function subagentExtension(
 		activeTui = undefined;
 		widgetInstalled = false;
 		if (ctx.hasUI) ctx.ui.setWidget(WIDGET_KEY, undefined);
+		// Invalidate every outstanding background delivery: the aborting session
+		// is gone, and a later-settling child must not reach the next session.
+		sessionEpoch += 1;
+		for (const controller of backgroundTasks.values()) controller.abort();
+		backgroundTasks.clear();
+	});
+	// btw-style context refresh: model_select carries the new model on the event,
+	// agent_settled delivers the freshest full context after each turn.
+	pi.on("model_select", (event, ctx) => {
+		latestCtx = { ...ctx, model: event.model } as ExtensionContext;
 	});
+	pi.on("agent_settled", (_event, ctx) => {
+		latestCtx = ctx;
+	});
+
+	const reportBackground = async (
+		launchEpoch: number,
+		taskId: string,
+		details: { role: string; model?: string; thinkingLevel?: string },
+		outcome: "completed" | "failed" | "aborted",
+		text: string,
+	): Promise<void> => {
+		if (launchEpoch !== sessionEpoch) return;
+		// Custom messages convert to user-role LLM messages, so the parent agent
+		// sees the outcome on its next turn without a forced turn now.
+		try {
+			pi.sendMessage({
+				customType: BACKGROUND_RESULT_TYPE,
+				content: `Background subagent ${taskId} (${details.role}) ${outcome}.\n\n${capOutput(text)}`,
+				display: true,
+				details: { ...details, taskId, outcome },
+			}, { triggerTurn: false });
+		} catch {
+			// Session may already be gone; the widget row still shows the outcome.
+		}
+	};
 
 	pi.registerTool({
 		name: "delegate_task",
 		label: "Subagent",
-		description: `Delegate one bounded, independently executable task to one isolated Pi Subagent. Roles: ${roleSummary()}. Choose fast for narrow work, balanced for normal work, frontier for ambiguous and high-risk work, or fav when the user asks for their favorite model; omit modelClass to use shared task-model settings. When the user designates a specific model, pass it as provider/modelId in model.`,
+		description: `Delegate one bounded, independently executable task to one isolated Pi Subagent. Roles: ${roleSummary()}. Choose fast for narrow work, balanced for normal work, frontier for ambiguous and high-risk work, or fav when the user asks for their favorite model; omit modelClass to use shared task-model settings. When the user designates a specific model, pass it as provider/modelId in model. Set background only when the user explicitly wants the task to run without blocking; background tasks report results later and cannot be waited on.`,
 		promptSnippet: "Delegate one bounded, independently executable task to an isolated role",
 		promptGuidelines: [
 			"Before calling delegate_task, split broad work into the smallest independent bounded tasks; keep integration and cross-cutting decisions in Main.",
 			"Each delegate_task task must state its objective, exact scope and exclusions, relevant context and constraints, expected deliverable, and validation; never pass the parent request unchanged.",
 			"Use fav when the user explicitly asks for their favorite model. Otherwise, choose the least capable modelClass that can reliably complete the task: fast for narrow work, balanced for normal work, and frontier only for ambiguous, cross-cutting, or high-risk work.",
 			"Submit independent delegate_task calls together for parallel execution. Parallel edits must own non-overlapping files; otherwise sequence them. Use the minimum number of Subagents needed.",
+			"Use background: true only when the user explicitly asks for non-blocking delegation (for example \"keep working while this runs\"); the result arrives as a message after the current turn and the parent must not assume it is available yet.",
 		],
 		parameters: Parameters,
 		async execute(toolCallId, params, signal, onUpdate, ctx) {
@@ -594,28 +681,99 @@ export default function subagentExtension(
 			if (params.modelClass !== undefined && !isModelClass(params.modelClass)) {
 				throw new Error("delegate_task modelClass must be fast, balanced, frontier, or fav.");
 			}
-			const launch = params.model !== undefined
-				? createRoleLaunch(pi, ctx, { role, route: resolveDesignatedRoute(ctx, cleanText(params.model, "model", "delegate_task")) })
+			// Resolve against the latest known session context: a task queued past
+			// the cap must pick up model or Codex account changes that happened
+			// while it waited.
+			const launchCtx = () => latestCtx ?? ctx;
+			const resolveLaunch = () => params.model !== undefined
+				? createRoleLaunch(pi, launchCtx(), { role, route: resolveDesignatedRoute(launchCtx(), cleanText(params.model, "model", "delegate_task")) })
 				: params.modelClass === undefined
-					? resolveRoleLaunch(pi, ctx, { role, taskId: SUBAGENT_TASK })
-					: createRoleLaunch(pi, ctx, { role, route: resolveTaskRoute(ctx, params.modelClass) });
-			const modelReferenceValue = modelReference(launch.model);
-			const thinkingLevel = launch.thinkingLevel;
-			if (launch.missingSkills.length) {
-				ctx.ui.notify(
-					`Subagent role ${role.name} skipped unavailable Pi skills: ${launch.missingSkills.join(", ")}.`,
-					"warning",
-				);
+					? resolveRoleLaunch(pi, launchCtx(), { role, taskId: SUBAGENT_TASK })
+					: createRoleLaunch(pi, launchCtx(), { role, route: resolveTaskRoute(launchCtx(), params.modelClass) });
+			const notifyMissingSkills = (launch: ReturnType<typeof resolveLaunch>) => {
+				if (launch.missingSkills.length) {
+					ctx.ui.notify(
+						`Subagent role ${role.name} skipped unavailable Pi skills: ${launch.missingSkills.join(", ")}.`,
+						"warning",
+					);
+				}
+			};
+
+			if (params.background) {
+				const taskId = `bg-${++backgroundSequence}-${Date.now().toString(36)}`;
+				const controller = new AbortController();
+				backgroundTasks.set(taskId, controller);
+				// Freeze the launching session now: a task that settles after a
+				// reload must not deliver into whichever session is active then.
+				const launchEpoch = sessionEpoch;
+				void (async () => {
+					let acquired = false;
+					let widgetStatus: Exclude<WidgetStatus, "working"> = "failure";
+					// Role is known up front; model/thinking join after launch resolution.
+					let details: { role: string; model?: string; thinkingLevel?: string } = { role: role.name };
+					try {
+						await acquireChildPermit(controller.signal);
+						acquired = true;
+						// Resolve resources only once launched: a task queued past the
+						// cap must not start with model routes or skills resolved before
+						// registries or accounts changed while it waited.
+						const launch = resolveLaunch();
+						notifyMissingSkills(launch);
+						details = { role: role.name, model: modelReference(launch.model), thinkingLevel: launch.thinkingLevel };
+						startWidgetItem(taskId, role.name, launch.model.id, launch.thinkingLevel, task, ctx);
+						const result = await runPi(
+							["--mode", "json", "-p", ...launch.args, `Task: ${task}`],
+							ctx.cwd,
+							controller.signal,
+							undefined,
+							(tokens) => updateWidgetTokens(taskId, tokens),
+							timeoutPolicy,
+						);
+						const failed = result.exitCode !== 0 || result.stopReason === "error" || result.stopReason === "aborted";
+						widgetStatus = result.stopReason === "aborted" ? "aborted" : failed ? "failure" : "success";
+						const text = capOutput(failed
+							? result.errorMessage || result.stderr.trim() || result.output || `Subagent exited with code ${result.exitCode}.`
+							: result.output || "(no output)");
+						await reportBackground(
+							launchEpoch,
+							taskId,
+							details,
+							result.stopReason === "aborted" ? "aborted" : failed ? "failed" : "completed",
+							text,
+						);
+					} catch (error) {
+						const aborted = controller.signal.aborted && !(error instanceof SubagentTimeoutError);
+						widgetStatus = aborted ? "aborted" : "failure";
+						await reportBackground(
+							launchEpoch,
+							taskId,
+							details,
+							aborted ? "aborted" : "failed",
+							capOutput(error instanceof Error ? error.message : String(error)),
+						);
+					} finally {
+						if (acquired) releaseChildPermit();
+						finishWidgetItem(taskId, widgetStatus);
+						backgroundTasks.delete(taskId);
+					}
+				})();
+				return {
+					content: [{ type: "text" as const, text: `Background subagent ${taskId} started (${role.name}). The outcome arrives as a message when the task settles; keep working or end your turn.` }],
+					details: { role: role.name, taskId, background: true },
+				};
 			}
 
-			const args = ["--mode", "json", "-p", ...launch.args, `Task: ${task}`];
+			const launch = resolveLaunch();
+			notifyMissingSkills(launch);
+			const modelReferenceValue = modelReference(launch.model);
+			const details = { role: role.name, model: modelReferenceValue, thinkingLevel: launch.thinkingLevel };
+
 			await acquireChildPermit(signal);
 			let widgetStatus: Exclude<WidgetStatus, "working"> = "failure";
 			try {
-				startWidgetItem(toolCallId, role.name, launch.model.id, thinkingLevel, task, ctx);
-				const details = { role: role.name, model: modelReferenceValue, thinkingLevel };
+				startWidgetItem(toolCallId, role.name, launch.model.id, launch.thinkingLevel, task, ctx);
 				const result = await runPi(
-					args,
+					["--mode", "json", "-p", ...launch.args, `Task: ${task}`],
 					ctx.cwd,
 					signal,
 					(text) => onUpdate?.({ content: [{ type: "text", text }], details }),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-subagent",
-  "version": "2.5.1",
+  "version": "2.9.5",
   "description": "Delegate one task to an isolated Pi role with explicit extensions and skills.",
   "keywords": [
     "pi-package",