pi-agent-flow 0.4.3 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-agent-flow",
-  "version": "0.4.3",
+  "version": "0.4.4",
   "description": "Flow-state delegation extension for Pi coding agent.",
   "type": "module",
   "main": "index.ts",

package/render-utils.ts

@@ -87,6 +87,17 @@ export function getTruncationBudget(prefixLength: number): number {
 	return Math.max(width - prefixLength, 1);
 }
 
+/** Fixed content budget for collapsed-line text (dir/exe/log). */
+export const CONTENT_MAX = 40;
+
+/**
+ * Compute how many visible chars of content fit after a prefix,
+ * using the fixed CONTENT_MAX budget. Floor of 8 to keep things readable.
+ */
+export function contentBudget(prefixVisibleLen: number): number {
+	return Math.max(CONTENT_MAX - prefixVisibleLen, 8);
+}
+
 /**
  * Truncate an ANSI-colored string to at most `max` visible characters,
  * preserving ANSI codes in the kept portions. Does not inject reset codes
@@ -95,14 +106,14 @@ export function getTruncationBudget(prefixLength: number): number {
 function truncateAnsi(text: string, max: number): string {
 	if (visibleLength(text) <= max) return text;
 
-	if (max < 6) {
-		// Not enough room for " ... " — just truncate without ellipsis
+	if (max < 3) {
+		// Not enough room for '…' — just truncate without ellipsis
 		const { raw } = takeVisible(text, max);
 		return raw;
 	}
 
 	const head = Math.ceil(max * 0.6);
-	const tail = max - head - 5; // 5 = " ... ".length
+	const tail = max - head - 1; // 1 = '…'.length
 
 	// Walk through the string, collecting raw chars until we've consumed
 	// `count` visible characters. ANSI sequences are copied through without
@@ -160,7 +171,7 @@ function truncateAnsi(text: string, max: number): string {
 
 	const tailRaw = takeVisibleFromEnd(text, tail);
 
-	return headResult.raw + " ... " + tailRaw;
+	return headResult.raw + "\u2026" + tailRaw;
 }
 
 export function truncateChars(text: string, max: number): string {

package/render.ts

@@ -22,7 +22,7 @@ import {
 	isFlowError,
 	isFlowSuccess,
 } from "./types.js";
-import { formatFixedTokens, formatCompactStats, formatFlowTypeName } from "./render-utils.js";
+import { formatFixedTokens, formatCompactStats, formatFlowTypeName, truncateChars, contentBudget } from "./render-utils.js";
 
 function shortenPath(p: string): string {
 	const home = os.homedir();
@@ -235,23 +235,28 @@ function renderFlowCollapsed(
 
 	// dir: line (intent/objective)
 	if (r.intent) {
-		container.addChild(new TruncatedText(`${theme.fg("dim", "├─ dir:")} ${theme.fg("dim", r.intent)}`, 0, 0));
+		const dirContent = truncateChars(r.intent, contentBudget(10));
+		container.addChild(new TruncatedText(`${theme.fg("dim", "├─ dir:")} ${theme.fg("dim", dirContent)}`, 0, 0));
 	}
 
 	// exe: line (last tool call)
 	const lastTool = getLastToolCall(r.messages);
 	if (lastTool) {
 		const exeStr = formatFlowToolCall(lastTool.name, lastTool.args, theme.fg.bind(theme));
-		container.addChild(new TruncatedText(`${theme.fg("dim", "├─ exe:")} ${exeStr}`, 0, 0));
+		const exeContent = truncateChars(exeStr, contentBudget(10));
+		container.addChild(new TruncatedText(`${theme.fg("dim", "├─ exe:")} ${exeContent}`, 0, 0));
 	}
 
 	// log: line (last assistant text or streaming)
 	if (flowOutput) {
-		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("dim", flowOutput)}`, 0, 0));
+		const logContent = truncateChars(flowOutput, contentBudget(10));
+		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("dim", logContent)}`, 0, 0));
 	} else if (streamingText) {
-		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("dim", streamingText)}`, 0, 0));
+		const logContent = truncateChars(streamingText, contentBudget(10));
+		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("dim", logContent)}`, 0, 0));
 	} else if (error && r.errorMessage) {
-		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("error", r.errorMessage)}`, 0, 0));
+		const logContent = truncateChars(r.errorMessage, contentBudget(10));
+		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("error", logContent)}`, 0, 0));
 	} else {
 		container.addChild(new TruncatedText(`${theme.fg("dim", "└─ log:")} ${theme.fg("dim", "[n/a]")}`, 0, 0));
 	}
@@ -371,22 +376,26 @@ function renderActivityPanel(
 
 		// dir: line (intent/objective)
 		if (r.intent) {
-			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "├─ dir:")} ${theme.fg("dim", r.intent)}`, 0, 0));
+			const dirContent = truncateChars(r.intent, contentBudget(10));
+			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "├─ dir:")} ${theme.fg("dim", dirContent)}`, 0, 0));
 		}
 
 		// exe: line (last tool call)
 		const lastTool = getLastToolCall(r.messages);
 		if (lastTool) {
 			const exeStr = formatFlowToolCall(lastTool.name, lastTool.args, theme.fg.bind(theme));
-			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "├─ exe:")} ${exeStr}`, 0, 0));
+			const exeContent = truncateChars(exeStr, contentBudget(10));
+			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "├─ exe:")} ${exeContent}`, 0, 0));
 		}
 
 		// log: line (last assistant text)
 		const lastText = getLastAssistantText(r.messages);
 		if (lastText) {
-			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "└─ log:")} ${theme.fg("dim", lastText)}`, 0, 0));
+			const logContent = truncateChars(lastText, contentBudget(10));
+			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "└─ log:")} ${theme.fg("dim", logContent)}`, 0, 0));
 		} else if (error && r.errorMessage) {
-			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "└─ log:")} ${theme.fg("error", r.errorMessage)}`, 0, 0));
+			const logContent = truncateChars(r.errorMessage, contentBudget(10));
+			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "└─ log:")} ${theme.fg("error", logContent)}`, 0, 0));
 		} else {
 			container.addChild(new TruncatedText(`${theme.fg("dim", indent + "└─ log:")} ${theme.fg("dim", "[n/a]")}`, 0, 0));
 		}