@linxiraos/pi-tui 1.4.0 → 1.4.1

package/src/tools/default-renderer.ts

@@ -104,21 +104,22 @@ function buildDefaultToolBody(input: DefaultToolRenderInput, uiTheme: Theme, con
 			body.push(uiTheme.fg("dim", "(no output)"));
 		} else {
 			let renderedAsJson = false;
-			if (textContent.startsWith("{") || textContent.startsWith("[")) {
+			if (isTreeSafeJson(textContent)) {
 				try {
 					const parsed = JSON.parse(textContent);
-					const maxDepth = options.expanded ? JSON_TREE_MAX_DEPTH_EXPANDED : JSON_TREE_MAX_DEPTH_COLLAPSED;
-					const maxLines = options.expanded ? JSON_TREE_MAX_LINES_EXPANDED : JSON_TREE_MAX_LINES_COLLAPSED;
-					const maxScalarLen = options.expanded ? JSON_TREE_SCALAR_LEN_EXPANDED : JSON_TREE_SCALAR_LEN_COLLAPSED;
-					const tree = renderJsonTreeLines(parsed, uiTheme, maxDepth, maxLines, maxScalarLen);
+					const tree = renderJsonTreeLines(parsed, uiTheme, {
+						maxDepth: options.expanded ? JSON_TREE_MAX_DEPTH_EXPANDED : JSON_TREE_MAX_DEPTH_COLLAPSED,
+						maxLines: options.expanded ? JSON_TREE_MAX_LINES_EXPANDED : JSON_TREE_MAX_LINES_COLLAPSED,
+						maxScalarLen: options.expanded ? JSON_TREE_SCALAR_LEN_EXPANDED : JSON_TREE_SCALAR_LEN_COLLAPSED,
+						// A result's own `i` or `__partialJson` keys are data, not call metadata.
+						hiddenRootKeys: [],
+					});
 
-					if (tree.lines.length > 0) {
+					// An expanded tree that hit a cap would show less than the raw output, so
+					// Ctrl+O falls through to the uncapped raw view instead.
+					if (tree.lines.length > 0 && !(options.expanded && tree.truncated)) {
 						body.push(...tree.lines);
-						if (!options.expanded) {
-							body.push(formatExpandHint(uiTheme, options.expanded, true));
-						} else if (tree.truncated) {
-							body.push(uiTheme.fg("dim", "…"));
-						}
+						if (!options.expanded) body.push(formatExpandHint(uiTheme, options.expanded, true));
 						renderedAsJson = true;
 					}
 				} catch {
@@ -126,13 +127,14 @@ function buildDefaultToolBody(input: DefaultToolRenderInput, uiTheme: Theme, con
 				}
 			}
 			if (!renderedAsJson) {
+				// No expanded cap: the terminal's own disclosure clamp is the only
+				// bound, so Ctrl+O reveals every row the collapsed view promised.
 				body.push(
 					...formatOutputPaneLines(
 						{
 							lines: textContent.split("\n"),
 							expanded: options.expanded,
 							collapsedMaxLines: 4,
-							expandedMaxLines: 12,
 							styleLine: line =>
 								clipCollapsed(styleToolOutputLine(line, uiTheme), contentWidth, options.expanded),
 							showExpandHintWhenUncapped: true,
@@ -157,14 +159,37 @@ export function formatDefaultToolExecution(
 	return [renderStatusLine(status, uiTheme), ...buildDefaultToolBody(input, uiTheme, contentWidth)].join("\n");
 }
 
+/**
+ * Serialized JSON past this size keeps the raw view: the tree caps out at a few
+ * hundred rows anyway, and parsing and indexing a huge payload only to discard
+ * the tree can stall the UI.
+ */
+const JSON_TREE_MAX_CHARS = 64 * 1024;
+/** An integer literal of 16+ digits may exceed `Number.MAX_SAFE_INTEGER` and change when parsed. */
+const UNSAFE_INTEGER_RE = /(?<![\w.])-?\d{16,}(?![\w.])/;
+
+/**
+ * Whether a tool result can render as a JSON tree without losing anything the
+ * raw text shows: a bracketed payload small enough to index, with no integer a
+ * double would round (64-bit ids).
+ */
+function isTreeSafeJson(text: string): boolean {
+	return (
+		(text.startsWith("{") || text.startsWith("[")) &&
+		text.length <= JSON_TREE_MAX_CHARS &&
+		!UNSAFE_INTEGER_RE.test(text)
+	);
+}
+
 /** Inline args summary budget in characters (a data cap, not a width). */
 const NATIVE_ARGS_SUMMARY_CHARS = 160;
-/** Result text kept for the native card body. */
-const NATIVE_RESULT_MAX_CHARS = 64 * 1024;
 
 /**
  * TSP view of the generic fallback card: the tool label with a one-line args
  * summary as the head target; the result as a JSON tree or raw `ansi` output.
+ * The complete output goes over the wire — the card-level disclosure clamp the
+ * terminal applies is the only bound, so an expanded card keeps every row. JSON
+ * the tree's depth, node or scalar caps would cut keeps the raw path instead.
  */
 export function describeDefaultToolExecution(input: DefaultToolRenderInput): NativeToolView {
 	const { result } = input;
@@ -175,15 +200,27 @@ export function describeDefaultToolExecution(input: DefaultToolRenderInput): Nat
 			: undefined;
 	const tool: NativeToolHead = { title: input.label, target: summary || undefined, targetKind: "text" };
 	if (!result) return { tool };
-	const output = result.output.trimEnd().slice(0, NATIVE_RESULT_MAX_CHARS);
+	const output = result.output.trimEnd();
 	const tone = result.skipped ? "info" : result.isError ? "error" : undefined;
 	if (!output) return { tool, tone, body: [noteText("(no output)")] };
-	if (output.startsWith("{") || output.startsWith("[")) {
+	if (isTreeSafeJson(output)) {
+		let parsed: unknown;
 		try {
-			return { tool, tone, body: [describeJsonTree(JSON.parse(output))] };
+			parsed = JSON.parse(output);
 		} catch {
 			// Not JSON: shown as raw output below.
 		}
+		if (parsed !== undefined) {
+			let truncated = false;
+			const tree = describeJsonTree(parsed, {
+				// A result's own `i` or `__partialJson` keys are data, not call metadata.
+				hiddenRootKeys: [],
+				onTruncated: () => {
+					truncated = true;
+				},
+			});
+			if (!truncated) return { tool, tone, body: [tree] };
+		}
 	}
 	return { tool, tone, body: [ansi(output)] };
 }

package/src/tools/eval.d.ts

@@ -61,7 +61,7 @@ export interface EvalCellResult {
     durationMs?: number;
     exitCode?: number;
     statusEvents?: EvalStatusEvent[];
-    /** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
+    /** Evictable events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
     statusEventsElided?: number;
 }
 /** Tool result detail object surfaced to the UI/transcript. */
@@ -69,7 +69,7 @@ export interface EvalToolDetails {
     cells?: EvalCellResult[];
     images?: ImageContent[];
     statusEvents?: EvalStatusEvent[];
-    /** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
+    /** Evictable events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
     statusEventsElided?: number;
     isError?: boolean;
     meta?: OutputMeta;
@@ -104,7 +104,7 @@ interface EvalRenderContext {
     timeout?: number;
 }
 /**
- * Coalescing key of a progress-snapshot event: `agent` and `judge_batch`
+ * Coalescing key of a progress-snapshot event: `agent`, `judge_batch`, and `mcp_progress`
  * events keyed by `id`, where only the newest snapshot matters. Discrete
  * actions (everything else) have no key. Shared by {@link upsertStatusEvent}
  * and the executors' display collectors so both coalesce identically.
@@ -114,12 +114,12 @@ export declare function statusEventKey(event: {
     [key: string]: unknown;
 }): string | undefined;
 /**
- * Discrete status events a log keeps. A cell polling a helper in a loop emits
+ * Evictable status events a log keeps. A cell polling a helper in a loop emits
  * one event per call (hundreds of thousands), and every live update and the
  * persisted tool result would otherwise carry them all.
  */
 export declare const MAX_STATUS_EVENTS = 200;
-/** A status event list plus the count of discrete events dropped from its front. */
+/** A status event list plus the count of evictable events dropped from its front. */
 export type StatusEventLog = Pick<EvalCellResult, "statusEvents" | "statusEventsElided">;
 /**
  * Append or replace a status event. Progress snapshots (see
@@ -129,9 +129,10 @@ export type StatusEventLog = Pick<EvalCellResult, "statusEvents" | "statusEvents
 export declare function upsertStatusEvent(events: EvalStatusEvent[], event: EvalStatusEvent): void;
 /**
  * {@link upsertStatusEvent} into a bounded log. Past {@link MAX_STATUS_EVENTS},
- * the oldest discrete event is dropped and counted. Snapshots stay, since agent
- * cards read them, and so does the newest committed `todo` result, which the
- * session todo panel refreshes from. Renderers already show only the newest
+ * the oldest discrete event or MCP progress snapshot is dropped and counted.
+ * Agent/judgment-batch snapshots stay, since agent cards read them, and so does
+ * the newest committed `todo` result, which the session todo panel refreshes
+ * from. Renderers already show only the newest
  * events behind an "… N earlier" row.
  */
 export declare function recordStatusEvent(log: StatusEventLog, event: EvalStatusEvent): void;

package/src/tools/eval.ts

@@ -35,6 +35,7 @@ import {
 	renderJsonTreeLines,
 } from "./json-tree";
 import { formatStyledTruncationWarning, stripOutputNotice } from "./output-meta";
+import { formatMCPProgress, parseMCPProgress } from "./mcp";
 import { splitPathAndSel } from "./read";
 import {
 	DEFAULT_TERMINAL_PREVIEW_LINES,
@@ -108,7 +109,7 @@ export interface EvalCellResult {
 	durationMs?: number;
 	exitCode?: number;
 	statusEvents?: EvalStatusEvent[];
-	/** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
+	/** Evictable events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
 	statusEventsElided?: number;
 }
 
@@ -117,7 +118,7 @@ export interface EvalToolDetails {
 	cells?: EvalCellResult[];
 	images?: ImageContent[];
 	statusEvents?: EvalStatusEvent[];
-	/** Discrete events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
+	/** Evictable events dropped from the front of `statusEvents` (see {@link recordStatusEvent}). */
 	statusEventsElided?: number;
 	isError?: boolean;
 	meta?: OutputMeta;
@@ -222,26 +223,29 @@ function pushCodeCellLines(
 type AgentEventStatus = "pending" | "running" | "completed" | "failed" | "aborted";
 
 /**
- * Coalescing key of a progress-snapshot event: `agent` and `judge_batch`
+ * Coalescing key of a progress-snapshot event: `agent`, `judge_batch`, and `mcp_progress`
  * events keyed by `id`, where only the newest snapshot matters. Discrete
  * actions (everything else) have no key. Shared by {@link upsertStatusEvent}
  * and the executors' display collectors so both coalesce identically.
  */
 export function statusEventKey(event: { op: string; [key: string]: unknown }): string | undefined {
-	if ((event.op === "agent" || event.op === "judge_batch") && typeof event.id === "string") {
+	if (
+		(event.op === "agent" || event.op === "judge_batch" || event.op === "mcp_progress") &&
+		typeof event.id === "string"
+	) {
 		return `${event.op}\0${event.id}`;
 	}
 	return undefined;
 }
 
 /**
- * Discrete status events a log keeps. A cell polling a helper in a loop emits
+ * Evictable status events a log keeps. A cell polling a helper in a loop emits
  * one event per call (hundreds of thousands), and every live update and the
  * persisted tool result would otherwise carry them all.
  */
 export const MAX_STATUS_EVENTS = 200;
 
-/** A status event list plus the count of discrete events dropped from its front. */
+/** A status event list plus the count of evictable events dropped from its front. */
 export type StatusEventLog = Pick<EvalCellResult, "statusEvents" | "statusEventsElided">;
 
 /**
@@ -263,9 +267,10 @@ export function upsertStatusEvent(events: EvalStatusEvent[], event: EvalStatusEv
 
 /**
  * {@link upsertStatusEvent} into a bounded log. Past {@link MAX_STATUS_EVENTS},
- * the oldest discrete event is dropped and counted. Snapshots stay, since agent
- * cards read them, and so does the newest committed `todo` result, which the
- * session todo panel refreshes from. Renderers already show only the newest
+ * the oldest discrete event or MCP progress snapshot is dropped and counted.
+ * Agent/judgment-batch snapshots stay, since agent cards read them, and so does
+ * the newest committed `todo` result, which the session todo panel refreshes
+ * from. Renderers already show only the newest
  * events behind an "… N earlier" row.
  */
 export function recordStatusEvent(log: StatusEventLog, event: EvalStatusEvent): void {
@@ -273,7 +278,9 @@ export function recordStatusEvent(log: StatusEventLog, event: EvalStatusEvent):
 	upsertStatusEvent(events, event);
 	if (events.length <= MAX_STATUS_EVENTS) return;
 	const keep = events.findLastIndex(e => e.op === "todo" && e.committed === true);
-	const oldest = events.findIndex((e, i) => i !== keep && statusEventKey(e) === undefined);
+	const oldest = events.findIndex(
+		(e, i) => i !== keep && (e.op === "mcp_progress" || statusEventKey(e) === undefined),
+	);
 	if (oldest < 0) return;
 	events.splice(oldest, 1);
 	log.statusEventsElided = (log.statusEventsElided ?? 0) + 1;
@@ -414,6 +421,12 @@ function formatStatusEvent(event: EvalStatusEvent, theme: Theme): string {
 	}
 
 	switch (op) {
+		case "mcp_progress": {
+			if (data.tool) parts.push(previewLine(sanitizeText(String(data.tool)), STATUS_VALUE_WIDTH));
+			const progress = parseMCPProgress(data);
+			if (progress) parts.push(formatMCPProgress(progress, STATUS_VALUE_WIDTH));
+			break;
+		}
 		case "read":
 			parts.push(`${data.chars ?? data.bytes ?? 0} chars`);
 			if (data.path) parts.push(`from ${shortenPath(String(data.path))}`);
@@ -693,20 +706,25 @@ const STATUS_VALUE_WIDTH = 200;
 
 /**
  * Plain one-line summary of a status event: op plus its scalar fields, each
- * collapsed to one line. A `preview` (file or command text a helper returned)
- * never joins the line: an expanded cell shows it under the line as a code
- * block holding its first {@link PREVIEW_LIMITS.OUTPUT_EXPANDED} lines, folded
- * to {@link PREVIEW_LIMITS.COLLAPSED_LINES}.
+ * collapsed to one line. MCP progress shows its tool and then the shared
+ * {@link formatMCPProgress} text; its call id is internal. A `preview` (file or
+ * command text a helper returned) never joins the line: an expanded cell shows
+ * it under the line as a code block holding its first
+ * {@link PREVIEW_LIMITS.OUTPUT_EXPANDED} lines, folded to
+ * {@link PREVIEW_LIMITS.COLLAPSED_LINES}.
  */
 function describeStatusEvent(event: EvalStatusEvent, expanded: boolean): NativeNode {
 	const spans: TspSpan[] = [span(event.op, "accent")];
+	const mcpProgress = event.op === "mcp_progress" ? parseMCPProgress(event) : undefined;
 	for (const key in event) {
 		if (key === "op" || key === "preview" || key === "resolvedThinkingLevel") continue;
+		if (event.op === "mcp_progress" && key !== "tool") continue;
 		const value = event[key];
 		if (typeof value === "string" || typeof value === "number" || typeof value === "boolean") {
 			spans.push(span(` ${key}=`, "muted"), span(previewLine(sanitizeText(String(value)), STATUS_VALUE_WIDTH)));
 		}
 	}
+	if (mcpProgress) spans.push(span(` ${formatMCPProgress(mcpProgress, STATUS_VALUE_WIDTH)}`));
 	const line = text(spans, { truncate: "end", role: "zeta.tool.eval.status" });
 	const preview = expanded ? eventString(event.preview)?.trimEnd() : undefined;
 	if (!preview) return line;

package/src/tools/find.ts

@@ -256,7 +256,7 @@ export const findToolRenderer = {
 			},
 			uiTheme,
 		);
-		return new Text(text, 1, 0);
+		return new Text(text, 0, 0);
 	},
 
 	renderResult(
@@ -269,7 +269,7 @@ export const findToolRenderer = {
 		const text = result.content?.find(c => c.type === "text")?.text ?? "";
 
 		if (result.isError) {
-			return new Text(formatErrorMessage(text || "Unknown error", uiTheme), 1, 0);
+			return new Text(formatErrorMessage(text || "Unknown error", uiTheme), 0, 0);
 		}
 
 		// Streaming progress: the tool reports the phase it is in.
@@ -285,7 +285,7 @@ export const findToolRenderer = {
 				},
 				uiTheme,
 			);
-			return new Text(header, 1, 0);
+			return new Text(header, 0, 0);
 		}
 
 		const { hits, stats, threshold } = details;
@@ -317,7 +317,7 @@ export const findToolRenderer = {
 					: "No relevant passages found";
 			const lines = [header, formatEmptyMessage(empty, uiTheme)];
 			for (const failure of stats.failures) lines.push(uiTheme.fg("warning", failure));
-			return new Text(lines.join("\n"), 1, 0);
+			return new Text(lines.join("\n"), 0, 0);
 		}
 
 		const header = renderStatusLine(
@@ -359,7 +359,6 @@ export const findToolRenderer = {
 				}
 				return [header, ...hitLines, ...extra].map(line => truncateToWidth(line, width, Ellipsis.Omit));
 			},
-			{ paddingX: 1 },
 		);
 	},
 	describeCall(args: FindRenderArgs): NativeToolView {

package/src/tools/glob.ts

@@ -151,7 +151,7 @@ export const globToolRenderer = {
 			},
 			uiTheme,
 		);
-		return new Text(text, 1, 0);
+		return new Text(text, 0, 0);
 	},
 
 	renderResult(
@@ -165,7 +165,7 @@ export const globToolRenderer = {
 
 		if (result.isError || details?.error) {
 			const errorText = details?.error || result.content?.find(c => c.type === "text")?.text || "Unknown error";
-			return new Text(formatErrorMessage(errorText, uiTheme), 1, 0);
+			return new Text(formatErrorMessage(errorText, uiTheme), 0, 0);
 		}
 
 		const hasDetailedData = details?.fileCount !== undefined;
@@ -178,7 +178,7 @@ export const globToolRenderer = {
 				textContent.includes("No files found") ||
 				textContent.trim() === ""
 			) {
-				return new Text(formatEmptyMessage("No files found", uiTheme), 1, 0);
+				return new Text(formatEmptyMessage("No files found", uiTheme), 0, 0);
 			}
 
 			const lines = textContent.split("\n").filter(l => l.trim());
@@ -207,7 +207,6 @@ export const globToolRenderer = {
 					);
 					return [header, ...listLines].map(l => truncateToWidth(l, width, Ellipsis.Omit));
 				},
-				{ paddingX: 1 },
 			);
 		}
 
@@ -244,7 +243,7 @@ export const globToolRenderer = {
 			);
 			const lines = [header, formatEmptyMessage(emptyLabel, uiTheme)];
 			if (missingNote) lines.push(missingNote);
-			return new Text(lines.join("\n"), 1, 0);
+			return new Text(lines.join("\n"), 0, 0);
 		}
 		const meta: string[] = [formatCount("file", fileCount)];
 		if (details?.scopePath) meta.push(`in ${details.scopePath}`);
@@ -298,7 +297,6 @@ export const globToolRenderer = {
 				);
 				return [header, ...fileLines, ...extraLines].map(l => truncateToWidth(l, width, Ellipsis.Omit));
 			},
-			{ paddingX: 1 },
 		);
 	},
 	describeCall(args: GlobRenderArgs): NativeToolView {

package/src/tools/grep.ts

@@ -365,7 +365,7 @@ export const grepToolRenderer = {
 			{ icon: "pending", title: "Grep", titleColor: "toolTitle", description: args.pattern || "?", meta },
 			uiTheme,
 		);
-		return new Text(text, 1, 0);
+		return new Text(text, 0, 0);
 	},
 
 	renderResult(
@@ -422,7 +422,6 @@ export const grepToolRenderer = {
 					);
 					return [header, ...listLines].map(l => truncateToWidth(l, width, Ellipsis.Omit));
 				},
-				{ paddingX: 1 },
 			);
 		}
 		const linkTargets = grepLinkTargets(details, options.renderContext?.rawDetails);
@@ -456,7 +455,7 @@ export const grepToolRenderer = {
 			);
 			const lines = [header, formatEmptyMessage(tuiText("gpNoMatches", "No matches found"), uiTheme)];
 			if (missingNote) lines.push(missingNote);
-			return new Text(lines.join("\n"), 1, 0);
+			return new Text(lines.join("\n"), 0, 0);
 		}
 
 		const summaryParts = [formatCount("match", matchCount), formatCount("file", fileCount)];
@@ -514,7 +513,6 @@ export const grepToolRenderer = {
 				);
 				return [header, ...matchLines, ...extraLines].map(l => truncateToWidth(l, width, Ellipsis.Omit));
 			},
-			{ paddingX: 1 },
 		);
 	},
 	describeCall(args: GrepRenderArgs): NativeToolView {

package/src/tools/irc.d.ts

@@ -19,6 +19,12 @@ export interface IrcMessage {
      * ping-pong forever.
      */
     wakeRelay?: boolean;
+    /**
+     * One leg of an `all` broadcast. Like relays, broadcasts never draw an
+     * automatic wake-turn reply: every woken idle peer would otherwise answer
+     * the broadcaster, mostly to say the message was not for it.
+     */
+    broadcast?: boolean;
 }
 /** Delivery outcome for one peer recipient. */
 export interface IrcDeliveryReceipt {

package/src/tools/irc.ts

@@ -21,6 +21,12 @@ export interface IrcMessage {
 	 * ping-pong forever.
 	 */
 	wakeRelay?: boolean;
+	/**
+	 * One leg of an `all` broadcast. Like relays, broadcasts never draw an
+	 * automatic wake-turn reply: every woken idle peer would otherwise answer
+	 * the broadcaster, mostly to say the message was not for it.
+	 */
+	broadcast?: boolean;
 }
 
 /** Delivery outcome for one peer recipient. */