@arhen/pi-core-subagent 1.3.53 → 1.3.54

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@arhen/pi-core-subagent",
-	"version": "1.3.53",
+	"version": "1.3.54",
 	"type": "module",
 	"description": "pi extension: fast in-process subagents with a dependency-graph scheduler (needs edges gate tasks and carry upstream output into dependent prompts), plus background runs, intercom and agent-to-agent mailbox. Leader defines agents inline.",
 	"license": "MIT",

package/src/format.ts

@@ -114,7 +114,8 @@ export function compactLines(run: RunSnapshot): string[] {
 	if (run.tasks.length > MAX_TASKS) lines.push(`… +${run.tasks.length - MAX_TASKS} more`);
 	return lines;
 }
-const WIDGET_MAX_LINES = 10;
+// header + 4 task rows; live tasks are ordered first, so the cap only ever hides finished work
+const WIDGET_MAX_LINES = 5;
 
 export class SubagentsWidget implements Component {
 	constructor(
@@ -139,17 +140,21 @@ export class SubagentsWidget implements Component {
 			),
 		];
 		const budget = WIDGET_MAX_LINES - 1;
+		// live tasks first so the budget never hides work in progress
+		const all = runs.flatMap((run) => run.tasks);
+		const ordered = [
+			...all.filter((t) => !TERMINAL.includes(t.status)),
+			...all.filter((t) => TERMINAL.includes(t.status)),
+		];
 		let shown = 0;
-		outer: for (const run of runs) {
-			for (const task of run.tasks) {
-				if (shown >= budget) break outer;
-				shown += 1;
-				const activity = task.lastActivity ? `${this.theme.fg("dim", `→ ${task.lastActivity}`)} · ` : "";
+		for (const task of ordered) {
+			if (shown >= budget) break;
+			shown += 1;
+			const activity = task.lastActivity ? `${this.theme.fg("dim", `→ ${task.lastActivity}`)} · ` : "";
 
-				lines.push(
-					truncateToWidth(`${this.theme.fg("dim", "├─")} ${themedTaskLine(task, this.theme, activity)}`, width, "…"),
-				);
-			}
+			lines.push(
+				truncateToWidth(`${this.theme.fg("dim", "├─")} ${themedTaskLine(task, this.theme, activity)}`, width, "…"),
+			);
 		}
 		const hidden = total - shown;
 		if (hidden > 0) {

package/src/index.ts

@@ -140,6 +140,7 @@ export default function (pi: ExtensionAPI) {
 			"Define each agent inline: invented name, focused system prompt, read-only by default (write:true to edit). A matched agent file takes over (see description); matching is by description, not name — name the agent whatever fits the goal.",
 			"Right after spawning, call subagent_status(runId) ONCE before any other work — a child that died on spawn (or never started) is invisible until far later otherwise. If it shows a task failed/never started, fix or respawn immediately.",
 			"Never block with nothing to do: if you have no work left after spawning, end your turn — completion notifies you and wakes a fresh turn with the results. await_subagent/autoAwait while idle only burns time and tokens.",
+			"A failed task interrupts you immediately as a steering message — handle it in the same turn (resume, swap model, re-dispatch) instead of finishing the plan on a broken intermediate result. Completes and aborts queue as follow-ups.",
 			"autoAwait:true only when this SAME turn must consume the result immediately. await_subagent is for syncing with your own parallel work — not the default follow-up to a spawn.",
 			"A task that failed mid-work (provider error, rate limit, timeout) keeps its session file and branch: resume_subagent(runId, taskId, model?) revives it with full context — prefer that over respawning. Respawn only when it never started (no session file).",
 		],

package/src/manager.ts

@@ -437,7 +437,9 @@ export class SubagentManager {
 	}
 
 	private deliverMode(kind: string, task: TaskSnapshot): "steer" | "followUp" {
-		return isStartupFailure(task, kind) ? "steer" : "followUp";
+		// failures interrupt the leader's turn: it must decide immediately (resume, swap model, respawn)
+		// instead of finding out after its own turn ended. Aborts are user-initiated, so they queue.
+		return kind === "failed" || isStartupFailure(task, kind) ? "steer" : "followUp";
 	}
 
 	private notifyTask(run: RunSnapshot, task: TaskSnapshot, kind: "completed" | "failed" | "aborted"): void {
@@ -470,7 +472,7 @@ export class SubagentManager {
 				? `A subagent is asking you a question (task ${extra?.taskId}): ${extra?.question ?? ""}\nReply with reply_subagent(runId: "${run.id}", taskId: "${extra?.taskId}", message: ...).`
 				: makeNotice(run, kind);
 		try {
-			this.pi.sendUserMessage(body, { deliverAs: "followUp" });
+			this.pi.sendUserMessage(body, { deliverAs: kind === "failed" ? "steer" : "followUp" });
 		} catch {}
 		this.emit("subagent:notification", { runId: run.id, kind, body });
 	}