@ryan_nookpi/pi-extension-subagent 0.5.0 → 0.5.1

package/agents.ts

@@ -140,25 +140,30 @@ function loadAgentsFromDir(dir: string, source: "user" | "project", options: Loa
 			continue;
 		}
 
-		const { frontmatter, body } = parseFrontmatter<Record<string, string>>(content);
-		if (!frontmatter.name || !frontmatter.description) continue;
-
-		const tools = normalizeTools(frontmatter.tools, format);
-		const model = normalizeModel(frontmatter.model, format);
-		const thinking = normalizeThinkingLevel(frontmatter.thinking);
-		const runtime: AgentRuntime = frontmatter.runtime === "claude" ? "claude" : "pi";
-
-		agents.push({
-			name: frontmatter.name,
-			description: frontmatter.description,
-			tools,
-			model,
-			thinking,
-			systemPrompt: attachCommonSubagentRule(body, runtime),
-			source,
-			filePath,
-			runtime,
-		});
+		try {
+			const { frontmatter, body } = parseFrontmatter<Record<string, unknown>>(content);
+			if (typeof frontmatter.name !== "string" || typeof frontmatter.description !== "string") continue;
+			if (!frontmatter.name || !frontmatter.description) continue;
+
+			const tools = normalizeTools(frontmatter.tools, format);
+			const model = normalizeModel(typeof frontmatter.model === "string" ? frontmatter.model : undefined, format);
+			const thinking = normalizeThinkingLevel(
+				typeof frontmatter.thinking === "string" ? frontmatter.thinking : undefined,
+			);
+			const runtime: AgentRuntime = frontmatter.runtime === "claude" ? "claude" : "pi";
+
+			agents.push({
+				name: frontmatter.name,
+				description: frontmatter.description,
+				tools,
+				model,
+				thinking,
+				systemPrompt: attachCommonSubagentRule(body, runtime),
+				source,
+				filePath,
+				runtime,
+			});
+		} catch {}
 	}
 
 	return agents;

package/commands.ts

@@ -1250,6 +1250,8 @@ export function registerAll(pi: ExtensionAPI, store: SubagentStore): SubagentReg
 				runState.model = undefined;
 				runState.retryCount = 0;
 				runState.lastRetryReason = undefined;
+				runState.errorClass = undefined;
+				runState.autoAbortReason = undefined;
 				runState.removed = false;
 				runState.deliveryMode = deliveryMode;
 				runState.turnCount = Math.max(DEFAULT_TURN_COUNT, runState.turnCount || DEFAULT_TURN_COUNT) + 1;

package/package.json

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

package/tool-execute.ts

@@ -1254,6 +1254,10 @@ export function createSubagentToolExecute(pi: ExtensionAPI, store: SubagentStore
 				runState.lastOutput = "";
 				runState.usage = undefined;
 				runState.model = undefined;
+				runState.retryCount = 0;
+				runState.lastRetryReason = undefined;
+				runState.errorClass = undefined;
+				runState.autoAbortReason = undefined;
 				runState.removed = false;
 				runState.turnCount = Math.max(DEFAULT_TURN_COUNT, runState.turnCount || DEFAULT_TURN_COUNT) + 1;
 				runState.contextMode = runState.contextMode ?? (config.inheritMainContext ? "main" : "sub");
@@ -1979,7 +1983,14 @@ export function createSubagentToolExecute(pi: ExtensionAPI, store: SubagentStore
 						pipeline.stepRunIds.push(runState.id);
 						chainLaunches.push(toLaunchSummary(runState, "chain"));
 
-						const finalized = await launchRunInBackground(runState, taskForAgent);
+						let finalized: FinalizedRun;
+						try {
+							finalized = await launchRunInBackground(runState, taskForAgent);
+						} catch (error: unknown) {
+							finalized = finalizeRunError(runState, error);
+						} finally {
+							clearRunAbortState(runState);
+						}
 						if (runState.removed) {
 							terminalStatus = finalized.isError ? "error" : "stopped";
 							pipeline.stepResults.push({

package/utils/agent-utils.ts

@@ -88,12 +88,10 @@ export const CLAUDE_MODEL_ALIAS_MAP: Record<string, string> = {
  * Normalize a comma-separated tool list according to the given format.
  * For "claude" format, maps Claude tool names to pi equivalents.
  */
-export function normalizeTools(rawTools: string | undefined, format: "pi" | "claude"): string[] | undefined {
-	if (!rawTools) return undefined;
-
-	const parsed = rawTools
-		.split(",")
-		.map((t) => t.trim())
+export function normalizeTools(rawTools: unknown, format: "pi" | "claude"): string[] | undefined {
+	const parsed = (Array.isArray(rawTools) ? rawTools : typeof rawTools === "string" ? rawTools.split(",") : [])
+		.filter((tool): tool is string => typeof tool === "string")
+		.map((tool) => tool.trim())
 		.filter(Boolean);
 
 	if (parsed.length === 0) return undefined;