pi-pretty-tui 0.3.2 → 0.3.3

package/README.md

@@ -38,6 +38,12 @@ Click activity rows, tool titles or thought entries to expand and collapse detai
 
 Settings persist across sessions. Disabling the extension restores Pi's native appearance.
 
+### Repeated errors
+
+In clean and compact, adjacent standalone errors can share the first complete error and a `[×2]` / `[×3]` count even when trace, request or correlation IDs change. Source type, title and surrounding text must match exactly; only standalone hexadecimal IDs (16–128 characters containing a letter) or UUIDs may differ. Short codes, numeric quantities and prefixed IDs such as `msg_...` / `rs_...` remain exact.
+
+Other transcript entries break the run; errors with thoughts, tool calls or visible answer text are not folded. In fullscreen, click the first error to expand or collapse the run. Expanded errors show every complete original error, aligned with the same prefix and without numbering; new retries preserve the expanded state. Session records remain unchanged. You can also use `/pretty-tui full` to see every occurrence and its original IDs.
+
 ## Copying
 
 Start Pi in **fullscreen mode** to use mouse controls:

package/README.zh-CN.md

@@ -38,6 +38,12 @@ pi install npm:pi-pretty-tui
 
 设置会持久保存。禁用插件后恢复 Pi 原生外观。
 
+### 重复错误
+
+clean 和 compact 中，相邻的独立错误即使 trace、request 或 correlation ID 不同，也可合并显示第一条完整错误及 `[×2]` / `[×3]` 次数。来源类型、标题和周围文本必须完全一致；仅独立的长十六进制 ID（16–128 位且含字母）或 UUID 可以变化。短错误码、纯数字值及 `msg_...` / `rs_...` 等带前缀的标识仍要求一致。
+
+其他会话条目会打断合并；带思考、工具调用或可见回答的错误不参与折叠。全屏模式下，点击第一条错误可展开或收起；展开后每条完整原始错误使用相同前缀、左侧对齐，不加序号，新追加的重复错误保留展开状态。会话记录保持不变，也可使用 `/pretty-tui full` 查看每次错误及原始 ID。
+
 ## 复制操作
 
 以 **fullscreen 全屏模式**启动 Pi，即可使用鼠标复制入口：

package/extensions/index.ts

@@ -89,6 +89,24 @@ const loadConfig = (path: string): PrettyTuiConfig => {
   }
 };
 
+/** Stable text affixes around opaque IDs, without fuzzy similarity chaining. */
+const errorAffixKey = (text: string): string => {
+  const parts: (string | ["hex" | "uuid"])[] = [];
+  let cursor = 0;
+  // Whole tokens only: keep semantic identifiers such as msg_<hex> and
+  // rs_<hex>, short codes, ordinary words, and numeric quantities exact.
+  const opaqueId = /(?<![\w-])(?:[\da-f]{8}-[\da-f]{4}-[\da-f]{4}-[\da-f]{4}-[\da-f]{12}|[\da-f]{16,128})(?![\w-])/giu;
+  for (const match of text.matchAll(opaqueId)) {
+    const kind = match[0].includes("-") ? "uuid" : "hex";
+    if (kind === "hex" && !/[a-f]/iu.test(match[0])) continue;
+    parts.push(text.slice(cursor, match.index), [kind]);
+    cursor = match.index + match[0].length;
+  }
+  parts.push(text.slice(cursor));
+  // Typed slots cannot collide with a literal placeholder in an error.
+  return JSON.stringify(parts);
+};
+
 /** Polished user messages, tool calls, and list rendering for Pi's TUI. */
 export default function prettyTui(pi: ExtensionAPI) {
   const cwd = process.cwd();
@@ -111,6 +129,21 @@ export default function prettyTui(pi: ExtensionAPI) {
   let currentPromptEditor: any;
   let errorChatContainer: any;
   const runtimeErrorDescriptors = new WeakMap<object, { source: string; title: string; content: string }>();
+  const expandedErrorRuns = new WeakSet<object>();
+  const errorDisclosureRows = new WeakMap<object, { start: number; end: number }>();
+  const errorDisclosureHit = (component: object, event: any) => {
+    const rows = errorDisclosureRows.get(component);
+    return renderMode !== "full" && Boolean(rows && event.y >= rows.start && event.y < rows.end);
+  };
+  const handleErrorDisclosureMouse = (component: object, event: any) => {
+    if (event.button !== "left" || !errorDisclosureHit(component, event)) return undefined;
+    if (event.type === "press") return { handled: true };
+    if (event.type !== "click") return undefined;
+    if (expandedErrorRuns.has(component)) expandedErrorRuns.delete(component);
+    else expandedErrorRuns.add(component);
+    currentTui?.requestRender?.();
+    return { handled: true };
+  };
   const errorDescriptor = (component: any) => {
     const runtime = runtimeErrorDescriptors.get(component);
     if (runtime) return runtime;
@@ -124,15 +157,21 @@ export default function prettyTui(pi: ExtensionAPI) {
     const children: any[] = errorChatContainer?.children ?? [];
     const index = children.indexOf(component);
     const descriptor = errorDescriptor(component);
+    const key = descriptor && errorAffixKey(descriptor.content);
     const same = (other: any) => {
       const value = errorDescriptor(other);
       return descriptor && value && value.source === descriptor.source &&
-        value.title === descriptor.title && value.content === descriptor.content;
+        value.title === descriptor.title && errorAffixKey(value.content) === key;
     };
     if (index < 0 || !descriptor) return { hidden: false, count: 1, compact: false };
+    // Only the visible run head needs a count; hidden retries must not rescan
+    // the whole suffix and repeatedly fingerprint every later error.
+    const hidden = index > 0 && Boolean(same(children[index - 1]));
     let count = 1;
-    while (index + count < children.length && same(children[index + count])) count++;
-    return { hidden: index > 0 && Boolean(same(children[index - 1])), count,
+    if (!hidden) {
+      while (index + count < children.length && same(children[index + count])) count++;
+    }
+    return { hidden, count,
       compact: index > 0 && Boolean(errorDescriptor(children[index - 1])) };
   };
   const runningColor = (theme: any, text: string): string =>
@@ -605,23 +644,45 @@ export default function prettyTui(pi: ExtensionAPI) {
 
   const renderStandaloneError = (member: ActivityMember, width: number, component?: any): string[] => {
     const run = component && renderMode !== "full" ? errorRun(component) : { hidden: false, count: 1, compact: false };
+    if (component) errorDisclosureRows.delete(component);
     if (run.hidden) return [];
     const theme = defaultActivityTheme();
     const style = activityUpdateStyle(member, theme);
     const original = component ? errorDescriptor(component) : undefined;
-    const title = stripTerminalSequences(original?.title ?? member.updateTitle ?? "Error");
-    const content = stripTerminalSequences(original?.content || member.updateContent || "");
-    const heading = style.icon + style.title(title);
-    const prefix = heading + (content ? theme.fg("dim", " · ") : "");
-    const text = prefix + theme.fg("muted", content) + (run.count > 1 ? theme.fg("dim", ` ×${run.count}`) : "");
-    // Wrap rather than truncate; all original detail lines remain accessible.
-    const continuation = " ".repeat(Math.min(visibleWidth(prefix), Math.max(0, Math.floor(width / 3))));
-    const rows: string[] = [];
-    for (const line of text.split("\n")) {
-      const wrapped = wrapTextWithAnsi(line, Math.max(1, width - visibleWidth(continuation)));
-      rows.push(...wrapped.map((row, index) => (rows.length || index ? continuation : "") + row));
+    const renderError = (titleText: string, contentText: string, count = 1): string[] => {
+      const title = stripTerminalSequences(titleText);
+      const content = stripTerminalSequences(contentText);
+      const heading = style.icon + style.title(title);
+      const prefix = heading + (content ? theme.fg("dim", " · ") : "");
+      const text = prefix + theme.fg("muted", content) + (count > 1 ? theme.fg("dim", component && expandedErrorRuns.has(component) ? " [Collapse]" : ` [×${count}]`) : "");
+      // Wrap rather than truncate; all original detail lines remain accessible.
+      const continuation = " ".repeat(Math.min(visibleWidth(prefix), Math.max(0, Math.floor(width / 3))));
+      const rows: string[] = [];
+      for (const line of text.split("\n")) {
+        const wrapped = wrapTextWithAnsi(line, Math.max(1, width - visibleWidth(continuation)));
+        rows.push(...wrapped.map((row, index) => (rows.length || index ? continuation : "") + row));
+      }
+      return rows.map((line) => truncateToWidth(line, Math.max(1, width), ""));
+    };
+    const rows = [
+      ...(run.compact ? [] : [""]),
+      ...renderError(original?.title ?? member.updateTitle ?? "Error", original?.content || member.updateContent || "", run.count),
+    ];
+    if (component && run.count > 1) {
+      errorDisclosureRows.set(component, { start: run.compact ? 0 : 1, end: rows.length });
+      mouseClickHitTests.set(component, (event) => errorDisclosureHit(component, event));
+      if (expandedErrorRuns.has(component)) {
+        // The visible head owns the expanded projection. Original retry
+        // components stay untouched and hidden, and only the head counts them.
+        const children = errorChatContainer?.children ?? [];
+        const index = children.indexOf(component);
+        for (const retry of children.slice(index + 1, index + run.count)) {
+          const descriptor = errorDescriptor(retry);
+          if (descriptor) rows.push(...renderError(descriptor.title, descriptor.content));
+        }
+      }
     }
-    return [...(run.compact ? [] : [""]), ...rows.map((line) => truncateToWidth(line, Math.max(1, width), ""))];
+    return rows;
   };
 
   const renderPendingActivityUpdate = (member: ActivityMember, width: number, component?: any): string[] => {
@@ -1405,10 +1466,15 @@ export default function prettyTui(pi: ExtensionAPI) {
       }
       if (renderMode === "compact") {
         const terminal = assistantTerminalUpdate(this[originalMessageKey] ?? this.lastMessage);
-        if (terminal) output.push(...renderPendingActivityUpdate(terminal, width, this));
+        if (terminal) {
+          const offset = output.length;
+          output.push(...renderPendingActivityUpdate(terminal, width, this));
+          const disclosure = errorDisclosureRows.get(this);
+          if (disclosure) errorDisclosureRows.set(this, { start: disclosure.start + offset, end: disclosure.end + offset });
+        }
       }
       panelAssistantViews.set(this, { rows, width: innerWidth, height: nativeLines.length });
-      mouseClickHitTests.set(this, (event) => thoughtRows.has(event.y));
+      mouseClickHitTests.set(this, (event) => thoughtRows.has(event.y) || errorDisclosureHit(this, event));
       return renderMode === "compact" && width < 8
         ? output.map((line) => isNativeImageRow(line) ? line : truncateToWidth(line, Math.max(1, width), "")) : output;
     };
@@ -1457,6 +1523,7 @@ export default function prettyTui(pi: ExtensionAPI) {
     };
 
     const renderAssistantTranscript = function (this: any, width: number): string[] {
+      errorDisclosureRows.delete(this);
       const message = this[originalMessageKey] ?? this.lastMessage;
       if (
         message &&
@@ -1582,6 +1649,11 @@ export default function prettyTui(pi: ExtensionAPI) {
     };
 
     const handleAssistantTranscriptMouse = function (this: any, event: any) {
+      if (errorDescriptor(this)) {
+        patchedRender.call(this, event.width);
+        const disclosure = handleErrorDisclosureMouse(this, event);
+        if (disclosure) return disclosure;
+      }
       const message = this[originalMessageKey] ?? this.lastMessage;
       if (renderMode !== "clean") {
         patchedRender.call(this, event.width);
@@ -1750,7 +1822,7 @@ export default function prettyTui(pi: ExtensionAPI) {
     const component: Component = {
       render: (width: number) => renderActivityUpdateProjection(group, member, width, component),
       invalidate() {},
-      handleMouse: (event: any) => handleActivityUpdateMouse(group, member, event),
+      handleMouse: (event: any) => handleErrorDisclosureMouse(component, event) ?? handleActivityUpdateMouse(group, member, event),
     } as Component;
     if (severity === "error") runtimeErrorDescriptors.set(component, {
       source: "runtime:error", title, content,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-pretty-tui",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "A beautiful, polished TUI for the Pi coding agent",
   "license": "MIT",
   "author": "Kainan Yang <ykn0309@whu.edu.cn>",