@pi-archimedes/subagent 1.8.3 → 1.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-archimedes/subagent",
-  "version": "1.8.3",
+  "version": "1.9.0",
   "type": "module",
   "keywords": [
     "pi-package"
@@ -11,7 +11,7 @@
   ],
   "main": "./src/index.ts",
   "dependencies": {
-    "@pi-archimedes/core": "1.8.3"
+    "@pi-archimedes/core": "1.9.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-ai": ">=0.1.0",

package/src/compact.ts

@@ -1,5 +1,5 @@
 import { Text } from "@earendil-works/pi-tui";
-import type { SubagentDetails, SubagentProgress, SubagentResult, SubagentToolResult } from "./types.js";
+import type { SubagentDetails, SubagentProgress, SubagentResult, SubagentToolCall, SubagentToolResult } from "./types.js";
 import { formatTokens, formatDuration, formatCost, truncLine, buildStatsLine, buildAgentLabel } from "./format.js";
 
 type Theme = { fg: (token: string, text: string) => string; bold: (text: string) => string };
@@ -14,7 +14,7 @@ interface ActivityData {
   finalOutput: string | undefined;
   status: "running" | "completed" | "failed" | undefined;
   error: string | undefined;
-  toolCalls?: string[] | undefined;
+  toolCalls?: (SubagentToolCall | string)[] | undefined;
 }
 
 export function buildActivityLine(
@@ -35,7 +35,7 @@ export function buildActivityLine(
     return theme.fg("error", "✗ Failed");
   }
 
-  // Running: show the current tool with live duration
+  // Running: show the current tool with live duration (grey while running)
   if (data.currentTool) {
     const arrow = theme.fg("muted", "↳ ");
     const argsPreview = data.currentToolArgs
@@ -44,7 +44,7 @@ export function buildActivityLine(
     const durationPart = data.currentToolStartedAt
       ? " | " + formatDuration(Date.now() - data.currentToolStartedAt)
       : "";
-    let line = theme.fg("syntaxFunction", data.currentTool);
+    let line = theme.fg("muted", data.currentTool);
     if (argsPreview) {
       line += theme.fg("dim", ": " + argsPreview);
     }
@@ -55,9 +55,21 @@ export function buildActivityLine(
   }
 
   // Running, no active tool: show the most recently completed tool call
+  // Color only the tool name green (success) or red (error)
   if (data.toolCalls && data.toolCalls.length > 0) {
     const lastCall = data.toolCalls[data.toolCalls.length - 1];
-    if (lastCall) return theme.fg("muted", "↳ " + lastCall);
+    if (lastCall) {
+      if (typeof lastCall === "string") {
+        return theme.fg("dim", "↳ " + truncLine(lastCall, 60));
+      }
+      const color = lastCall.error ? "error" : "success";
+      const arrow = theme.fg("muted", "↳ ");
+      const name = theme.fg(color, lastCall.name);
+      const argsPart = lastCall.argsPreview
+        ? theme.fg("dim", ": " + truncLine(lastCall.argsPreview, 60))
+        : "";
+      return arrow + name + argsPart;
+    }
   }
 
   // Running, no tool history: show first line of streamed output if any

package/src/expanded.ts

@@ -1,9 +1,25 @@
 import { Text } from "@earendil-works/pi-tui";
-import type { SubagentDetails, SubagentProgress, SubagentResult } from "./types.js";
+import type { SubagentDetails, SubagentProgress, SubagentResult, SubagentToolCall } from "./types.js";
 import { formatTokens, formatDuration, truncLine, buildStatsLine, buildAgentLabel } from "./format.js";
 
 type Theme = { fg: (token: string, text: string) => string; bold: (text: string) => string };
 
+// ── Helpers ─────────────────────────────────────────────────────────────────
+
+function formatToolCall(call: SubagentToolCall | string, theme: Theme): string {
+  // Backward compat: old persisted sessions may have toolCalls as string[]
+  if (typeof call === "string") {
+    return theme.fg("dim", "↳ " + call);
+  }
+  const arrow = theme.fg("muted", "↳ ");
+  const color = call.error ? "error" : "success";
+  const name = theme.fg(color, call.name);
+  const argsPart = call.argsPreview
+    ? theme.fg("dim", ": " + call.argsPreview)
+    : "";
+  return arrow + name + argsPart;
+}
+
 // ── Expanded completed result ───────────────────────────────────────────────
 
 export function buildExpandedText(
@@ -43,7 +59,7 @@ export function buildExpandedText(
   if (toolCalls && toolCalls.length > 0) {
     lines.push("");
     for (const call of toolCalls) {
-      lines.push(theme.fg("dim", "↳ " + call));
+      lines.push(formatToolCall(call, theme));
     }
   }
 
@@ -116,7 +132,7 @@ export function renderProgressExpanded(
   if (progress.toolCalls && progress.toolCalls.length > 0) {
     lines.push("");
     for (const call of progress.toolCalls) {
-      lines.push(theme.fg("dim", "↳ " + call));
+      lines.push(formatToolCall(call, theme));
     }
   }
 
@@ -129,7 +145,7 @@ export function renderProgressExpanded(
     const durationPart = progress.currentToolStartedAt
       ? " | " + formatDuration(Date.now() - progress.currentToolStartedAt)
       : "";
-    let line = theme.fg("syntaxFunction", progress.currentTool);
+    let line = theme.fg("muted", progress.currentTool);
     if (argsPreview) {
       line += theme.fg("dim", ": " + argsPreview);
     }
@@ -192,7 +208,7 @@ export function buildProgressExpandedText(
   if (progress.toolCalls && progress.toolCalls.length > 0) {
     lines.push("");
     for (const call of progress.toolCalls) {
-      lines.push(theme.fg("dim", "↳ " + call));
+      lines.push(formatToolCall(call, theme));
     }
   }
 
@@ -204,7 +220,7 @@ export function buildProgressExpandedText(
     const durationPart = progress.currentToolStartedAt
       ? " | " + formatDuration(Date.now() - progress.currentToolStartedAt)
       : "";
-    let line = theme.fg("syntaxFunction", progress.currentTool);
+    let line = theme.fg("muted", progress.currentTool);
     if (argsPreview) {
       line += theme.fg("dim", ": " + argsPreview);
     }

package/src/handlers.ts

@@ -50,11 +50,16 @@ export function extractArgsPreview(args: unknown): string {
 export function handleToolStart(state: StreamState, event: JsonEvent): void {
   state.toolCount++;
   state.currentTool = event.toolName as string;
-  state.currentToolArgs = JSON.stringify(event.args);
+  // Use extractArgsPreview instead of JSON.stringify for readable display
+  const argsPreview = extractArgsPreview(event.args);
+  state.currentToolArgs = argsPreview;
   state.currentToolStartedAt = Date.now();
   // Record tool call with args preview
-  const argsPreview = extractArgsPreview(event.args);
-  state.toolCalls.push(`${state.currentTool}: ${argsPreview}`);
+  state.toolCalls.push({
+    name: state.currentTool,
+    argsPreview,
+    error: false,
+  });
   if (state.toolCalls.length > TOOL_CALLS_MAX) {
     state.toolCalls.splice(0, state.toolCalls.length - TOOL_CALLS_MAX);
   }
@@ -75,6 +80,14 @@ export function handleToolEnd(state: StreamState): void {
  */
 export function handleToolResult(state: StreamState, event: JsonEvent): void {
   const result = event.result as Record<string, unknown> | undefined;
+
+  // Mark the last tool call as errored if the result indicates an error
+  const lastCall = state.toolCalls[state.toolCalls.length - 1];
+  const isError = event.isError === true || result?.isError === true;
+  if (lastCall && lastCall.name === (event.toolName as string) && isError) {
+    lastCall.error = true;
+  }
+
   if (!result) return;
 
   const toolName = (event.toolName as string) ?? "tool";

package/src/types.ts

@@ -7,6 +7,12 @@ export interface SubagentUsage {
   turns: number;
 }
 
+export interface SubagentToolCall {
+  name: string;
+  argsPreview: string;
+  error: boolean;
+}
+
 export interface SubagentProgress {
   agent: string;
   status: "running" | "completed" | "failed";
@@ -27,8 +33,8 @@ export interface SubagentProgress {
   output: string | undefined;
   /** Last N lines of assistant text for live display */
   recentOutput: string[] | undefined;
-  /** History of tool calls: "toolName: args_preview" */
-  toolCalls: string[] | undefined;
+  /** History of tool calls with status tracking */
+  toolCalls: SubagentToolCall[] | undefined;
 }
 
 export interface SubagentResult {
@@ -70,7 +76,7 @@ export interface StreamState {
   model: string | undefined;
   accumulatedOutput: string[];
   recentOutput: string[];
-  toolCalls: string[];
+  toolCalls: SubagentToolCall[];
   finalOutput: string | undefined;
 }