pi-better-subagents 0.1.29 → 0.1.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-subagents",
-  "version": "0.1.29",
+  "version": "0.1.31",
   "description": "Pi extension for detached, sandboxed subagent runs that keep the foreground session free.",
   "license": "MIT",
   "type": "module",

package/shared-navigator.ts

@@ -1,4 +1,5 @@
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+import { truncateToWidth as piTruncateToWidth, visibleWidth as piVisibleWidth } from "@earendil-works/pi-tui";
 import type { Component } from "@earendil-works/pi-tui";
 import { createRenderScheduler, type RenderScheduler } from "./shared-render-scheduler.ts";
 
@@ -179,7 +180,10 @@ export function ensureBackgroundWorkNavigator(ctx: ExtensionContext, deps: HostD
   if (!isNavigatorUiAvailable(ctx)) return;
   const s = state();
   s.uiCtx = ctx;
-  s.deps = deps;
+  s.deps = {
+    ...deps,
+    createTranscriptComponent: deps.createTranscriptComponent ?? s.deps?.createTranscriptComponent,
+  };
   try { (ctx.ui as any).setWidget?.(MAIN_LIST_WIDGET_KEY, undefined); } catch { /* ignore */ }
   s.mainListWidgetInstalled = false;
   s.mainListRequestRender = undefined;
@@ -277,10 +281,10 @@ function refreshMainListWidget(): void {
   }) : undefined;
   const changed = nextSignature !== s.lastMainListSignature;
   s.lastMainListSignature = nextSignature;
-  s.lastMainListLines = rows.length ? buildMainListLines(rows, MAIN_LIST_FALLBACK_WIDTH, deps.truncate, themeFg(ctx), {
+  s.lastMainListLines = rows.length ? ["", ...buildMainListLines(rows, MAIN_LIST_FALLBACK_WIDTH, deps.truncate, themeFg(ctx), {
     selectedId: s.mainListFocused ? s.mainListSelectedId : undefined,
     focused: s.mainListFocused === true,
-  }) : undefined;
+  })] : undefined;
   if (!rows.length) {
     if (!s.mainListWidgetInstalled) return;
     try { (ctx.ui as any).setWidget?.(MAIN_LIST_WIDGET_KEY, undefined); } catch { /* ignore */ }
@@ -320,10 +324,10 @@ function createMainListWidget(tui: { requestRender?(): void }, theme: unknown, d
         state().lastMainListLines = undefined;
         return [];
       }
-      const lines = buildMainListLines(rows, renderWidth(width), deps.truncate, themeFgFromTheme(theme), {
+      const lines = ["", ...buildMainListLines(rows, renderWidth(width), deps.truncate, themeFgFromTheme(theme), {
         selectedId: state().mainListFocused ? state().mainListSelectedId : undefined,
         focused: state().mainListFocused === true,
-      });
+      })];
       state().lastMainListLines = lines;
       return lines;
     },
@@ -1266,7 +1270,8 @@ export function wrapLogText(text: string, width: number): string[] {
       continue;
     }
     const sourceIndent = source.match(/^[ \t]*/)?.[0] ?? "";
-    const indent = sourceIndent.replace(/\t/g, "  ");
+    // Leave room for a full-width grapheme even on deeply indented log lines.
+    const indent = sourceIndent.replace(/\t/g, "  ").slice(0, max - 2);
     let remaining = source.slice(sourceIndent.length);
     const contentWidth = Math.max(1, max - visibleWidth(indent));
     if (!remaining) {
@@ -1299,33 +1304,25 @@ function lastDelimiterBreak(value: string): number {
   return last;
 }
 
+const GRAPHEME_SEGMENTER = new Intl.Segmenter(undefined, { granularity: "grapheme" });
+
 function splitVisiblePrefix(value: string, width: number): [string, string] {
   const max = Math.max(0, Math.floor(width));
   let visible = 0;
-  let index = 0;
-  while (index < value.length && visible < max) {
-    if (value[index] === "\u001b" || value[index] === "\u009b") {
-      const match = value.slice(index).match(ANSI_RE);
-      if (match?.index === 0) {
-        index += match[0].length;
-        continue;
-      }
-    }
-    if (value[index] === "<") {
-      const close = value.indexOf(">", index);
-      if (close !== -1) {
-        const tag = value.slice(index, close + 1);
-        if (/^<\/?[a-zA-Z][\w-]*>$/.test(tag) || tag === "</>") {
-          index = close + 1;
-          continue;
-        }
-      }
+  let end = 0;
+  for (const { segment, index } of GRAPHEME_SEGMENTER.segment(value)) {
+    if (index < end) continue;
+    const formatting = value.slice(index).match(FORMATTING_PREFIX_RE);
+    if (formatting) {
+      end = index + formatting[0].length;
+      continue;
     }
-    const codePoint = value.codePointAt(index)!;
-    index += codePoint > 0xFFFF ? 2 : 1;
-    visible += 1;
+    const cells = piVisibleWidth(segment);
+    if (visible + cells > max) break;
+    visible += cells;
+    end = index + segment.length;
   }
-  return [value.slice(0, index), value.slice(index)];
+  return [value.slice(0, end), value.slice(end)];
 }
 
 function cycleLogTailRows(current: number): number {
@@ -1384,44 +1381,19 @@ function safeTruncate(line: string, width: number, truncate: (s: string, width:
 
 const ANSI_RE = new RegExp("[\\u001B\\u009B][[\\]()#;?]*(?:(?:(?:[a-zA-Z\\d]*(?:;[a-zA-Z\\d]*)*)?\\u0007)|(?:(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PR-TZcf-ntqry=><~]))", "g");
 
+const FORMATTING_PREFIX_RE = new RegExp(`^(?:${ANSI_RE.source}|<\\/?[a-zA-Z][\\w-]*>|</>)`);
+
 function visibleWidth(value: string): number {
-  return String(value ?? "")
+  const cleaned = String(value ?? "")
     .replace(ANSI_RE, "")
     .replace(/<\/?[a-zA-Z][\w-]*>/g, "")
-    .replace(/<\/>/g, "")
-    .length;
+    .replace(/<\/>/g, "");
+  return piVisibleWidth(cleaned);
 }
 
 function truncateVisible(value: string, width: number): string {
   const str = String(value ?? "");
   const max = Math.max(0, Math.floor(width || 0));
   if (visibleWidth(str) <= max) return str;
-  let out = "";
-  let vis = 0;
-  let i = 0;
-  while (i < str.length && vis < max) {
-    if (str[i] === "\u001b" || str[i] === "\u009b") {
-      const match = str.slice(i).match(ANSI_RE);
-      if (match && match.index === 0) {
-        out += match[0];
-        i += match[0].length;
-        continue;
-      }
-    }
-    if (str[i] === "<") {
-      const close = str.indexOf(">", i);
-      if (close !== -1) {
-        const tag = str.slice(i, close + 1);
-        if (/^<\/?[a-zA-Z][\w-]*>$/.test(tag) || tag === "</>") {
-          out += tag;
-          i = close + 1;
-          continue;
-        }
-      }
-    }
-    out += str[i];
-    vis += 1;
-    i += 1;
-  }
-  return out;
+  return piTruncateToWidth(str, max);
 }