pi-subagents 0.11.10 → 0.11.12

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## [Unreleased]
 
+## [0.11.12] - 2026-03-28
+
+### Changed
+- Tool history (`recentTools`) in execution progress is now chronological (oldest first) and uncapped, replacing the old newest-first order with a 5-entry cap. Affects all execution paths (tool, slash commands, chains, parallel, async, delegation). Both single-task and chain-step render paths in `render.ts` now consistently use `slice(-3)` for most-recent display.
+- Removed 50ms throttle on execution progress updates. `onUpdate` now fires immediately on every tool start, tool end, message end, and tool result. Affects all execution paths.
+- Delegation bridge now passes through full `recentOutputLines` arrays, `recentTools` history, and resolved `model` to prompt-template consumers, replacing the old stripped-down single-line updates.
+
+## [0.11.11] - 2026-03-23
+
+### Changed
+- Updated for pi 0.62.0 compatibility. `Skill.source` replaced with `Skill.sourceInfo` for skill provenance, `Widget` type replaced with `Component`. Bumped devDependencies to `^0.62.0`.
+
 ## [0.11.10] - 2026-03-21
 
 ### Changed

package/execution.ts

@@ -139,48 +139,15 @@ export async function runSync(
 		closeJsonlWriter = () => jsonlWriter.close();
 		let buf = "";
 
-		// Throttled update mechanism - consolidates all updates
-		let lastUpdateTime = 0;
-		let updatePending = false;
-		let pendingTimer: ReturnType<typeof setTimeout> | null = null;
 		let processClosed = false;
-		const UPDATE_THROTTLE_MS = 50; // Reduced from 75ms for faster responsiveness
 
-		const scheduleUpdate = () => {
+		const fireUpdate = () => {
 			if (!onUpdate || processClosed) return;
-			const now = Date.now();
-			const elapsed = now - lastUpdateTime;
-
-			if (elapsed >= UPDATE_THROTTLE_MS) {
-				// Enough time passed, update immediately
-				// Clear any pending timer to avoid double-updates
-				if (pendingTimer) {
-					clearTimeout(pendingTimer);
-					pendingTimer = null;
-				}
-				lastUpdateTime = now;
-				updatePending = false;
-				progress.durationMs = now - startTime;
-				onUpdate({
-					content: [{ type: "text", text: getFinalOutput(result.messages) || "(running...)" }],
-					details: { mode: "single", results: [result], progress: [progress] },
-				});
-			} else if (!updatePending) {
-				// Schedule update for later
-				updatePending = true;
-				pendingTimer = setTimeout(() => {
-					pendingTimer = null;
-					if (updatePending && !processClosed) {
-						updatePending = false;
-						lastUpdateTime = Date.now();
-						progress.durationMs = Date.now() - startTime;
-						onUpdate({
-							content: [{ type: "text", text: getFinalOutput(result.messages) || "(running...)" }],
-							details: { mode: "single", results: [result], progress: [progress] },
-						});
-					}
-				}, UPDATE_THROTTLE_MS - elapsed);
-			}
+			progress.durationMs = Date.now() - startTime;
+			onUpdate({
+				content: [{ type: "text", text: getFinalOutput(result.messages) || "(running...)" }],
+				details: { mode: "single", results: [result], progress: [progress] },
+			});
 		};
 
 		const processLine = (line: string) => {
@@ -195,25 +162,20 @@ export async function runSync(
 					progress.toolCount++;
 					progress.currentTool = evt.toolName;
 					progress.currentToolArgs = extractToolArgsPreview((evt.args || {}) as Record<string, unknown>);
-					// Tool start is important - update immediately by forcing throttle reset
-					lastUpdateTime = 0;
-					scheduleUpdate();
+					fireUpdate();
 				}
 
 				if (evt.type === "tool_execution_end") {
 					if (progress.currentTool) {
-						progress.recentTools.unshift({
+						progress.recentTools.push({
 							tool: progress.currentTool,
 							args: progress.currentToolArgs || "",
 							endMs: now,
 						});
-						if (progress.recentTools.length > 5) {
-							progress.recentTools.pop();
-						}
 					}
 					progress.currentTool = undefined;
 					progress.currentToolArgs = undefined;
-					scheduleUpdate();
+					fireUpdate();
 				}
 
 				if (evt.type === "message_end" && evt.message) {
@@ -245,7 +207,7 @@ export async function runSync(
 							}
 						}
 					}
-					scheduleUpdate();
+					fireUpdate();
 				}
 				if (evt.type === "tool_result_end" && evt.message) {
 					result.messages.push(evt.message);
@@ -262,7 +224,7 @@ export async function runSync(
 							progress.recentOutput.splice(0, progress.recentOutput.length - 50);
 						}
 					}
-					scheduleUpdate();
+					fireUpdate();
 				}
 			} catch {
 				// Non-JSON stdout lines are expected; only structured events are parsed.
@@ -276,19 +238,12 @@ export async function runSync(
 			const lines = buf.split("\n");
 			buf = lines.pop() || "";
 			lines.forEach(processLine);
-
-			// Also schedule an update on data received (handles streaming output)
-			scheduleUpdate();
 		});
 		proc.stderr.on("data", (d) => {
 			stderrBuf += d.toString();
 		});
 		proc.on("close", (code) => {
 			processClosed = true;
-			if (pendingTimer) {
-				clearTimeout(pendingTimer);
-				pendingTimer = null;
-			}
 			if (buf.trim()) processLine(buf);
 			if (code !== 0 && stderrBuf.trim() && !result.error) {
 				result.error = stderrBuf.trim();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-subagents",
-  "version": "0.11.10",
+  "version": "0.11.12",
   "description": "Pi extension for delegating tasks to subagents with chains, parallel execution, and TUI clarification",
   "author": "Nico Bailon",
   "license": "MIT",
@@ -47,8 +47,8 @@
   },
   "devDependencies": {
     "@marcfargas/pi-test-harness": "^0.5.0",
-    "@mariozechner/pi-agent-core": "^0.54.0",
-    "@mariozechner/pi-ai": "^0.54.0",
-    "@mariozechner/pi-coding-agent": "^0.54.0"
+    "@mariozechner/pi-agent-core": "^0.62.0",
+    "@mariozechner/pi-ai": "^0.62.0",
+    "@mariozechner/pi-coding-agent": "^0.62.0"
   }
 }

package/prompt-template-bridge.ts

@@ -41,6 +41,9 @@ export interface PromptTemplateDelegationTaskProgress {
 	currentTool?: string;
 	currentToolArgs?: string;
 	recentOutput?: string;
+	recentOutputLines?: string[];
+	recentTools?: Array<{ tool: string; args: string }>;
+	model?: string;
 	toolCount?: number;
 	durationMs?: number;
 	tokens?: number;
@@ -51,6 +54,9 @@ export interface PromptTemplateDelegationUpdate {
 	currentTool?: string;
 	currentToolArgs?: string;
 	recentOutput?: string;
+	recentOutputLines?: string[];
+	recentTools?: Array<{ tool: string; args: string }>;
+	model?: string;
 	toolCount?: number;
 	durationMs?: number;
 	tokens?: number;
@@ -71,6 +77,7 @@ interface PromptTemplateBridgeResult {
 			messages?: unknown[];
 			exitCode?: number;
 			error?: string;
+			model?: string;
 		}>;
 		progress?: Array<{
 			index?: number;
@@ -79,6 +86,7 @@ interface PromptTemplateBridgeResult {
 			currentTool?: string;
 			currentToolArgs?: string;
 			recentOutput?: string[];
+			recentTools?: Array<{ tool?: string; args?: string }>;
 			toolCount?: number;
 			durationMs?: number;
 			tokens?: number;
@@ -154,17 +162,62 @@ export function firstTextContent(content: unknown): string | undefined {
 	return undefined;
 }
 
+function filterRecentOutput(lines: string[] | undefined): string[] | undefined {
+	if (!lines || lines.length === 0) return undefined;
+	const filtered = lines.filter((line) => typeof line === "string" && line.trim() && line.trim() !== "(running...)");
+	if (filtered.length === 0) return undefined;
+	return filtered;
+}
+
+function sanitizeRecentTools(
+	tools: Array<{ tool?: string; args?: string }> | undefined,
+): Array<{ tool: string; args: string }> | undefined {
+	if (!tools || tools.length === 0) return undefined;
+	const sanitized = tools
+		.filter((entry) => typeof entry.tool === "string" && entry.tool.trim().length > 0)
+		.map((entry) => ({
+			tool: entry.tool as string,
+			args: typeof entry.args === "string" ? entry.args : String(entry.args ?? ""),
+		}));
+	return sanitized.length > 0 ? sanitized : undefined;
+}
+
+function resolveProgressModel(
+	update: PromptTemplateBridgeResult,
+	entry: { index?: number; agent?: string },
+): string | undefined {
+	const results = update.details?.results;
+	if (!results || results.length === 0) return undefined;
+	if (typeof entry.index === "number" && entry.index >= 0) {
+		const byIndex = results[entry.index];
+		if (typeof byIndex?.model === "string") return byIndex.model;
+	}
+	if (entry.agent) {
+		const byAgent = results.find((result) => result.agent === entry.agent && typeof result.model === "string");
+		if (byAgent?.model) return byAgent.model;
+	}
+	const firstWithModel = results.find((result) => typeof result.model === "string");
+	return firstWithModel?.model;
+}
+
 function toDelegationUpdate(requestId: string, update: PromptTemplateBridgeResult): PromptTemplateDelegationUpdate | undefined {
 	const progress = update.details?.progress?.[0];
 	const taskProgress = update.details?.progress?.map((entry) => {
 		const lastOutput = entry.recentOutput?.[entry.recentOutput.length - 1];
+		const safeLastOutput =
+			typeof lastOutput === "string" && lastOutput.trim() && lastOutput !== "(running...)"
+				? lastOutput
+				: undefined;
 		return {
 			index: entry.index,
 			agent: entry.agent ?? "delegate",
 			status: entry.status,
 			currentTool: entry.currentTool,
 			currentToolArgs: entry.currentToolArgs,
-			recentOutput: lastOutput && lastOutput !== "(running...)" ? lastOutput : undefined,
+			recentOutput: safeLastOutput,
+			recentOutputLines: filterRecentOutput(entry.recentOutput),
+			recentTools: sanitizeRecentTools(entry.recentTools),
+			model: resolveProgressModel(update, entry),
 			toolCount: entry.toolCount,
 			durationMs: entry.durationMs,
 			tokens: entry.tokens,
@@ -172,11 +225,18 @@ function toDelegationUpdate(requestId: string, update: PromptTemplateBridgeResul
 	});
 	if (!progress && (!taskProgress || taskProgress.length === 0)) return undefined;
 	const lastOutput = progress?.recentOutput?.[progress.recentOutput.length - 1];
+	const safeLastOutput =
+		typeof lastOutput === "string" && lastOutput.trim() && lastOutput !== "(running...)"
+			? lastOutput
+			: undefined;
 	return {
 		requestId,
 		currentTool: progress?.currentTool,
 		currentToolArgs: progress?.currentToolArgs,
-		recentOutput: lastOutput && lastOutput !== "(running...)" ? lastOutput : undefined,
+		recentOutput: safeLastOutput,
+		recentOutputLines: filterRecentOutput(progress?.recentOutput),
+		recentTools: sanitizeRecentTools(progress?.recentTools),
+		model: progress ? resolveProgressModel(update, progress) : undefined,
 		toolCount: progress?.toolCount,
 		durationMs: progress?.durationMs,
 		tokens: progress?.tokens,

package/render.ts

@@ -4,7 +4,7 @@
 
 import type { AgentToolResult } from "@mariozechner/pi-agent-core";
 import { getMarkdownTheme, type ExtensionContext } from "@mariozechner/pi-coding-agent";
-import { Container, Markdown, Spacer, Text, truncateToWidth, visibleWidth, type Widget } from "@mariozechner/pi-tui";
+import { Container, Markdown, Spacer, Text, visibleWidth, type Component } from "@mariozechner/pi-tui";
 import {
 	type AsyncJobState,
 	type Details,
@@ -179,7 +179,7 @@ export function renderSubagentResult(
 	result: AgentToolResult<Details>,
 	_options: { expanded: boolean },
 	theme: Theme,
-): Widget {
+): Component {
 	const d = result.details;
 	if (!d || !d.results.length) {
 		const t = result.content[0];
@@ -447,7 +447,7 @@ export function renderSubagentResult(
 			}
 			// Recent tools
 			if (rProg.recentTools?.length) {
-				for (const t of rProg.recentTools.slice(0, 3)) {
+				for (const t of rProg.recentTools.slice(-3)) {
 					const maxArgsLen = Math.max(40, w - 30);
 					const argsPreview = t.args.length > maxArgsLen
 						? `${t.args.slice(0, maxArgsLen)}...`

package/skills.ts

@@ -194,32 +194,24 @@ function buildSkillPaths(cwd: string): string[] {
 	return [...new Set([...defaultSkillPaths, ...packagePaths, ...settingsPaths])];
 }
 
-function inferSkillSource(rawSource: unknown, filePath: string, cwd: string): SkillSource {
-	const source = typeof rawSource === "string" ? rawSource : "";
+function inferSkillSource(sourceInfo: { source: string; scope: string }, filePath: string, cwd: string): SkillSource {
+	const { scope, source } = sourceInfo;
+
+	if (scope === "project" && source === "local") return "project";
+	if (scope === "user" && source === "local") return "user";
+
+	// Fallback: infer from file path when sourceInfo isn't specific enough
+	// (e.g. scope === "temporary" for skills loaded via explicit skillPaths)
 	const projectRoot = path.resolve(cwd, CONFIG_DIR);
 	const isProjectScoped = isWithinPath(filePath, projectRoot);
+	if (isProjectScoped) return "project";
+
 	const isUserScoped = isWithinPath(filePath, AGENT_DIR);
+	if (isUserScoped) return "user";
+
 	const globalRoot = getGlobalNpmRoot();
-	const isGlobalPackage = globalRoot ? isWithinPath(filePath, globalRoot) : false;
-
-	if (source === "project") return "project";
-	if (source === "user") return "user";
-	if (source === "settings") {
-		if (isProjectScoped) return "project-settings";
-		if (isUserScoped) return "user-settings";
-		return "unknown";
-	}
-	if (source === "package") {
-		if (isProjectScoped) return "project-package";
-		if (isUserScoped || isGlobalPackage) return "user-package";
-		return "unknown";
-	}
-	if (source === "extension") return "extension";
-	if (source === "builtin") return "builtin";
+	if (globalRoot && isWithinPath(filePath, globalRoot)) return "user-package";
 
-	if (isProjectScoped) return "project";
-	if (isUserScoped) return "user";
-	if (isGlobalPackage) return "user-package";
 	return "unknown";
 }
 
@@ -247,7 +239,7 @@ function getCachedSkills(cwd: string): CachedSkillEntry[] {
 		const entry: CachedSkillEntry = {
 			name: skill.name,
 			filePath: skill.filePath,
-			source: inferSkillSource((skill as { source?: unknown }).source, skill.filePath, cwd),
+			source: inferSkillSource(skill.sourceInfo, skill.filePath, cwd),
 			description: skill.description,
 			order: i,
 		};