pi-better-background-tasks 0.2.14 → 0.2.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-background-tasks",
-  "version": "0.2.14",
+  "version": "0.2.16",
   "description": "Pi extension for durable background shell tasks, watchers, logs, and status inspection.",
   "license": "MIT",
   "type": "module",

package/src/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";
 
@@ -280,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 */ }
@@ -323,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;
     },
@@ -1269,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) {
@@ -1302,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 {
@@ -1387,44 +1381,86 @@ 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;
-        }
-      }
+  return piTruncateToWidth(str, max);
+}
+
+/**
+ * Render one provider detail with the overlay line builder and the main-list
+ * row formatter. Callers must already be inside the process that registered
+ * the provider. `listedIds` is the live list; `rowLine` uses the same
+ * formatter on the detail payload when the live list has filtered the run.
+ */
+export function renderRegisteredWorkDetail(providerId: string, id: string, width = 100): {
+  detail: BackgroundWorkDetail | null;
+  lines: string[];
+  rowLine: string | null;
+  listLines: string[] | null;
+  listedIds: string[];
+} {
+  const s = state();
+  const deps = s.deps;
+  const ctx = s.uiCtx;
+  const truncate = deps?.truncate ?? ((value: string, limit: number) => {
+    const text = String(value ?? "");
+    return text.length > limit ? text.slice(0, Math.max(0, limit)) : text;
+  });
+  const fg = ctx ? themeFg(ctx) : (_color: string, value: string) => value;
+  const provider = s.providers.get(providerId);
+  let listed: BackgroundWorkRow[] = [];
+  try { listed = provider?.listRows(Date.now()) ?? []; } catch { listed = []; }
+  const detail = detailFor(rowKey(providerId, id), Date.now(), { logTailLines: 40 });
+  const listLines = s.lastMainListLines ?? null;
+  const listedIds = listed.map((row) => row.id);
+  if (!detail) return { detail: null, lines: [], rowLine: null, listLines, listedIds };
+  const theme = (ctx?.ui as { theme?: unknown } | undefined)?.theme;
+  let lines: string[];
+  try {
+    if (detail.transcript && deps?.createTranscriptComponent) {
+      const transcriptLines = deps.createTranscriptComponent(detail, theme).render(width) ?? [];
+      lines = buildTranscriptDetailLines(detail, transcriptLines, width, truncate, fg, { logTailRows: 12 });
+    } else {
+      lines = buildDetailLines(detail, width, truncate, fg, { logTailRows: 12 });
     }
-    out += str[i];
-    vis += 1;
-    i += 1;
+  } catch {
+    lines = buildDetailLines(detail, width, truncate, fg, { logTailRows: 12 });
   }
-  return out;
+  const modelLine = detail.metadata.find((item) => item.label === "model")?.value ?? "";
+  const effortMatch = modelLine.match(/ · effort (.+)$/);
+  const elapsed = detail.metadata.find((item) => item.label === "elapsed")?.value ?? "-";
+  const row: InternalRow = {
+    providerId,
+    id: detail.id,
+    name: detail.title,
+    model: effortMatch ? modelLine.slice(0, effortMatch.index) : modelLine,
+    ...(effortMatch ? { effort: effortMatch[1] } : {}),
+    status: detail.status,
+    statusTone: detail.statusTone,
+    kind: "subagent",
+    elapsed,
+    primary: "",
+    sortStartedAt: 0,
+    navigatorId: rowKey(providerId, detail.id),
+    providerLabel: provider?.label ?? providerId,
+  };
+  return {
+    detail,
+    lines,
+    rowLine: formatMainListRow(row, true, fg, width),
+    listLines,
+    listedIds,
+  };
 }