pi-bar 0.3.19 → 0.3.21

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.19/assets/screenshot.png)
+![pi-bar screenshot, three rows showing green, yellow, and red context usage](https://cdn.jsdelivr.net/npm/pi-bar@0.3.21/assets/screenshot.png)
 
 ## Why use it?
 

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(
@@ -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));
@@ -741,7 +777,8 @@ class FooterTldrEngine {
 			);
 			if (!this.isCurrentJob(job) || response.stopReason !== "stop") return;
 
-			const text = extractTextContent(response.content) ?? "";
+			const rawText = extractTextContent(response.content) ?? "";
+			const text = sanitizeTldrText(rawText);
 			if (!text) return;
 			const checkpoint = {
 				activityIndex: job.activityIndex,
@@ -826,13 +863,13 @@ class FooterTldrEngine {
 				{
 					type: "text",
 					text: [
-						"Previous generated TLDR checkpoints:",
+						"Prior TLDRs (context only, do not copy phrasing):",
 						previousCheckpointLines(this.acceptedCheckpoints),
 						"",
-						"New raw activity since the latest accepted checkpoint:",
+						"New activity to summarize:",
 						...rawActivities.map(formatRawActivity),
 						"",
-						`Write the next TLDR through activity ${job.activityIndex}.`,
+						"Write the next TLDR.",
 					].join("\n"),
 				},
 			],
@@ -848,14 +885,16 @@ function checkpointSystemPrompt(job: TldrCheckpointJob): string {
 			: "Start with a present-tense -ing verb.";
 	return `Write one plain-English TLDR for a terminal coding agent.
 Use the prior TLDRs for context and the new activity for the update.
-Summarize only activity up to the requested index.
+Summarize the current state of work; do not narrate the history.
 If context is sparse, still summarize the available activity.
 Never ask for more information or say there is not enough context.
 Return one concise status fragment under ${TLDR_TARGET_SUMMARY_CHARS} characters.
 Omit subjects like "the agent" or "it".
 Prefer verb + direct object. Include outcome only if important.
 Do not address the user.
-Plain text only; no markdown, JSON, code, bullets, quotes, or file/tool names.
+Do not mention activity numbers, indexes, checkpoints, prior TLDRs, or phrases like "through activity".
+Output only the status fragment itself. No prefixes, labels, bullets, or quotes.
+Plain text only; no markdown, JSON, code, file paths, or tool names.
 ${tenseInstruction}`;
 }
 
@@ -863,12 +902,28 @@ function previousCheckpointLines(checkpoints: readonly TldrCheckpoint[]): string
 	if (checkpoints.length === 0) return "none";
 	return checkpoints
 		.slice(-MAX_CONTEXT_CHECKPOINTS)
-		.map((checkpoint) => `- Through activity ${checkpoint.activityIndex}: ${checkpoint.text}`)
+		.map((checkpoint) => `- ${sanitizeTldrText(checkpoint.text)}`)
 		.join("\n");
 }
 
 function formatRawActivity(activity: TldrActivity): string {
-	return `[${activity.index}] ${activity.text}`;
+	return `- ${activity.text}`;
+}
+
+const LEAKED_PREFIX_PATTERN =
+	/^\s*(?:[-*•]\s*)?(?:(?:through\s+activity|activity|checkpoint|tldr|summary)\s*[#:\-—–]?\s*\d*[:\.\-—–]?\s*)+/i;
+
+const LEADING_PUNCT_PATTERN = /^[\s\-—–•*:#.,;]+/;
+
+function sanitizeTldrText(text: string): string {
+	let cleaned = text.trim();
+	let previous: string;
+	do {
+		previous = cleaned;
+		cleaned = cleaned.replace(LEAKED_PREFIX_PATTERN, "").trim();
+		cleaned = cleaned.replace(LEADING_PUNCT_PATTERN, "").trim();
+	} while (cleaned !== previous && cleaned.length > 0);
+	return cleaned || text.trim();
 }
 
 function shouldShowStatus(key: string, filter: StatusFilter): boolean {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-bar",
-  "version": "0.3.19",
+  "version": "0.3.21",
   "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.19/assets/screenshot.png"
+    "image": "https://cdn.jsdelivr.net/npm/pi-bar@0.3.21/assets/screenshot.png"
   }
 }