pi-bar 0.3.18 → 0.3.20

package/README.md

@@ -9,7 +9,7 @@
 claude-opus-4.7  ❯  think:med  ❯  2.6% / 1.0M  ❯  Inspecting package structure  ❯  plan:active
 ```
 
-![pi-bar screenshot, three rows showing green, yellow, and red context usage](https://cdn.jsdelivr.net/npm/pi-bar@0.3.18/assets/screenshot.png)
+![pi-bar screenshot, three rows showing green, yellow, and red context usage](https://cdn.jsdelivr.net/npm/pi-bar@0.3.20/assets/screenshot.png)
 
 ## Why use it?
 
@@ -48,7 +48,7 @@ Allowed segments are `model`, `thinking`, `context`, `tldr`, and `extensions`. T
 
 ### Configure live TLDR
 
-pi-bar shows a concise live TLDR of the current task in the footer. It generates TLDRs from recent prompt, assistant, and deterministic code-compressed tool facts (for example `bash ... ok`, `grep ... 4 matches`, `edit file`, or `custom_tool key=value`). It resets when you navigate the session tree so stale summaries do not follow you across branches. Set `PI_BAR_SHOW` without `tldr` to hide it.
+pi-bar shows a concise live TLDR of the current task in the footer. It generates TLDRs from recent prompt, assistant, and deterministic code-compressed tool facts (for example `bash ... ok`, `grep ... 4 matches`, `read file`, or `custom_tool key=value ok`). It resets when you navigate the session tree so stale summaries do not follow you across branches. Set `PI_BAR_SHOW` without `tldr` to hide it.
 
 Override the TLDR model with:
 

package/extensions/status-footer.ts

@@ -53,6 +53,7 @@ type TldrActivity = {
 	activityType: TldrActivityType;
 	displayPriority: TldrDisplayPriority;
 	text: string;
+	toolCallId?: string;
 };
 type TldrCheckpoint = {
 	activityIndex: number;
@@ -365,12 +366,18 @@ function usefulLineCount(text: string): number {
 		.filter(Boolean).length;
 }
 
+const ERROR_LINE_PATTERN = /(error|failed|TS\d+|E[A-Z]+\d*|Command exited|denied|refused|aborted|panic|fatal|exception)/i;
+
 function errorSummary(text: string | undefined): string | undefined {
 	if (!text) return undefined;
-	const compacted = compactText(text);
-	const exitMatch = compacted.match(/Command exited with code \d+/);
-	if (exitMatch) return truncateText(exitMatch[0], MAX_TOOL_RESULT_ERROR_CHARS);
-	return truncateText(compacted, MAX_TOOL_RESULT_ERROR_CHARS);
+	const cleaned = text.replace(ANSI_PATTERN, "");
+	const lines = cleaned
+		.split("\n")
+		.map((line) => line.trim())
+		.filter(Boolean);
+	if (lines.length === 0) return undefined;
+	const matched = lines.find((line) => ERROR_LINE_PATTERN.test(line)) ?? lines[lines.length - 1];
+	return truncateText(compactText(matched), MAX_TOOL_RESULT_ERROR_CHARS);
 }
 
 function summarizeToolResult(
@@ -388,7 +395,7 @@ function summarizeToolResult(
 		case "bash":
 			return undefined;
 		case "read":
-			return path ? `${lineCount || "file"} lines from ${path}` : undefined;
+			return path ? `read ${path}` : undefined;
 		case "edit":
 			return path ? `edited ${path}` : undefined;
 		case "write":
@@ -400,7 +407,7 @@ function summarizeToolResult(
 		case "ls":
 			return lineCount ? `${lineCount} entries` : undefined;
 		default:
-			return lineCount ? `${lineCount} output lines` : undefined;
+			return undefined;
 	}
 }
 
@@ -442,20 +449,32 @@ class TldrFactCollector {
 		const record = message as Record<string, unknown>;
 		if (record.role !== "assistant") return undefined;
 		const text = extractTextContent(record.content as readonly unknown[] | undefined);
-		return text
-			? this.addActivity(
-					"assistant_update",
-					"normal",
-					`assistant: ${tailText(text, MAX_ASSISTANT_UPDATE_CHARS)}`,
-				)
-			: undefined;
+		if (!text) return undefined;
+
+		// Streaming emits many partial updates per message. Keep only the latest by
+		// dropping the previous assistant_update so the model sees one accurate snapshot.
+		const last = this.activities[this.activities.length - 1];
+		if (last?.activityType === "assistant_update") {
+			this.activities.pop();
+		}
+
+		return this.addActivity(
+			"assistant_update",
+			"normal",
+			`assistant: ${tailText(text, MAX_ASSISTANT_UPDATE_CHARS)}`,
+		);
 	}
 
-	recordToolCall(event: { toolName: string; input?: Record<string, unknown> }): TldrActivity {
+	recordToolCall(event: {
+		toolName: string;
+		input?: Record<string, unknown>;
+		toolCallId?: string;
+	}): TldrActivity {
 		return this.addActivity(
 			"tool_call",
 			"normal",
 			`tool: ${compactToolInput(event.toolName, event.input)}`,
+			event.toolCallId,
 		);
 	}
 
@@ -465,6 +484,7 @@ class TldrFactCollector {
 		isError?: boolean;
 		content?: readonly unknown[];
 		details?: unknown;
+		toolCallId?: string;
 	}): TldrActivity {
 		const isError = Boolean(event.isError);
 		const resultText = summarizeToolResult(
@@ -475,12 +495,25 @@ class TldrFactCollector {
 		);
 		const status = isError ? "error" : "ok";
 		const tool = compactToolInput(event.toolName, event.input);
+
+		// Result fact subsumes the prior tool_call fact for the same call. Drop the
+		// matching tool_call so the model sees one combined fact instead of two.
+		if (event.toolCallId) {
+			const idx = this.activities.findIndex(
+				(activity) =>
+					activity.activityType === "tool_call" &&
+					activity.toolCallId === event.toolCallId,
+			);
+			if (idx !== -1) this.activities.splice(idx, 1);
+		}
+
 		return this.addActivity(
 			"tool_result",
 			"normal",
 			resultText
 				? `result: ${tool} ${status}; ${resultText}`
 				: `result: ${tool} ${status}`,
+			event.toolCallId,
 		);
 	}
 
@@ -538,12 +571,14 @@ class TldrFactCollector {
 		activityType: TldrActivityType,
 		displayPriority: TldrDisplayPriority,
 		text: string,
+		toolCallId?: string,
 	): TldrActivity {
 		const activity = {
 			index: this.nextIndex,
 			activityType,
 			displayPriority,
 			text: truncateText(text, MAX_ACTIVITY_TEXT_CHARS),
+			toolCallId,
 		};
 		this.nextIndex++;
 		this.activities.push(activity);
@@ -600,7 +635,7 @@ class FooterTldrEngine {
 
 	recordToolCall(
 		ctx: ExtensionContext,
-		event: { toolName: string; input?: Record<string, unknown> },
+		event: { toolName: string; input?: Record<string, unknown>; toolCallId?: string },
 	): void {
 		this.enqueue(ctx, this.facts.recordToolCall(event));
 	}
@@ -613,6 +648,7 @@ class FooterTldrEngine {
 			isError?: boolean;
 			content?: readonly unknown[];
 			details?: unknown;
+			toolCallId?: string;
 		},
 	): void {
 		this.enqueue(ctx, this.facts.recordToolResult(event));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-bar",
-  "version": "0.3.18",
+  "version": "0.3.20",
   "description": "Pi footer/statusline showing active model, thinking level, context pressure, live TLDR, and extension statuses.",
   "type": "module",
   "keywords": [
@@ -51,6 +51,6 @@
     "extensions": [
       "./extensions/status-footer.ts"
     ],
-    "image": "https://cdn.jsdelivr.net/npm/pi-bar@0.3.18/assets/screenshot.png"
+    "image": "https://cdn.jsdelivr.net/npm/pi-bar@0.3.20/assets/screenshot.png"
   }
 }