@arhen/pi-core-subagent 1.3.53 → 1.3.55

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@arhen/pi-core-subagent",
-	"version": "1.3.53",
+	"version": "1.3.55",
 	"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

@@ -60,8 +60,12 @@ function taskStatsWithUsage(task: TaskSnapshot): string {
 	const usage = formatUsage(task.usage);
 	return `${stats}${usage ? ` · ${usage}` : ""}`;
 }
+export function modelTag(task: TaskSnapshot): string {
+	const ref = [task.provider, task.model, task.thinking].filter(Boolean).join("/");
+	return ref ? ` [${ref}]` : "";
+}
 export function taskLine(task: TaskSnapshot): string {
-	return `${statusIcon(task.status)} ${task.agent} · ${taskStatsWithUsage(task)} · ${taskTimer(task)}`;
+	return `${statusIcon(task.status)} ${task.agent}${modelTag(task)} · ${taskStatsWithUsage(task)} · ${taskTimer(task)}`;
 }
 export function colorNums(text: string, theme: Theme): string {
 	return text.replace(/((?:\d+(?:\.\d+)?[a-zA-Z]*)+)|([^\d]+)/g, (_m, num?: string, rest?: string) =>
@@ -71,15 +75,16 @@ export function colorNums(text: string, theme: Theme): string {
 function themedTaskLine(task: TaskSnapshot, theme: Theme, activity = ""): string {
 	const tail = `${taskStatsWithUsage(task)} · ${taskTimer(task)}`;
 
+	const tag = theme.fg("dim", modelTag(task));
 	const gate =
 		task.status === "queued" && task.needs?.length ? `${theme.fg("muted", `↳ waits ${task.needs.join(", ")}`)} · ` : "";
 	if (TERMINAL.includes(task.status)) {
-		return theme.fg("dim", `${statusIcon(task.status)} ${task.agent} · ${tail}`);
+		return theme.fg("dim", `${statusIcon(task.status)} ${task.agent}${tag} · ${tail}`);
 	}
 
 	pulsePhase += 1;
 	const name = isTalking(task) ? theme.fg(pulsePhase % 2 === 0 ? "accent" : "dim", `${task.agent} ⇄`) : task.agent;
-	return `${statusIcon(task.status)} ${name} · ${gate}${activity}${colorNums(tail, theme)}`;
+	return `${statusIcon(task.status)} ${name}${tag} · ${gate}${activity}${colorNums(tail, theme)}`;
 }
 const ARG_KEYS = ["pattern", "query", "command", "path", "file_path", "filePath", "url", "name", "subject", "task"];
 export function describeCall(toolName: string, args: unknown, cwd?: string): string {
@@ -114,7 +119,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 +145,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 });
 	}
@@ -825,6 +827,7 @@ export class SubagentManager {
 					task: input.task,
 
 					model: model?.id ?? input.model,
+					provider: model?.provider,
 					thinking,
 					tools,
 				},
@@ -878,7 +881,13 @@ export class SubagentManager {
 			this.updateTask(
 				run,
 				task,
-				{ status: "running", sessionId: child.sessionId, sessionFile: child.sessionFile },
+				{
+					status: "running",
+					sessionId: child.sessionId,
+					sessionFile: child.sessionFile,
+					provider: child.model?.provider ?? task.provider,
+					thinking: child.thinkingLevel,
+				},
 				ctx,
 				onUpdate,
 			);

package/src/types.ts

@@ -33,6 +33,7 @@ export interface TaskSnapshot {
 	notifiedParent?: boolean;
 	error?: string;
 	model?: string;
+	provider?: string;
 	modelNote?: string;
 	toolsNote?: string;
 	thinking?: string;