pi-better-background-tasks 0.2.1 → 0.2.3

package/package.json

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

package/src/shared-navigator.ts

@@ -83,6 +83,7 @@ type NavigatorState = {
   deps?: HostDeps;
   lastHint?: string | null;
   lastMainListLines?: string[];
+  lastMainListSignature?: string;
   mainListWidgetInstalled?: boolean;
   mainListRequestRender?: () => void;
   mainListSelectedId?: string;
@@ -110,7 +111,6 @@ const MAIN_LIST_FALLBACK_WIDTH = 100;
 const DETAIL_OVERLAY_FOOTER_ROWS = 3;
 const EVIDENCE_SECTION_ID = "__evidence__";
 const RUNNING_DOT_GLYPH = "●";
-const RUNNING_DOT_FRAMES = ["dim", "accent", "accent", "dim"] as const;
 
 function state(): NavigatorState {
   const g = globalThis as typeof globalThis & { [GLOBAL_KEY]?: NavigatorState };
@@ -199,6 +199,7 @@ export function disposeBackgroundWorkNavigator(ctx?: ExtensionContext): void {
   }
   s.lastHint = undefined;
   s.lastMainListLines = undefined;
+  s.lastMainListSignature = undefined;
   s.mainListDeadlineScheduler?.dispose();
   s.mainListDeadlineScheduler = undefined;
   s.mainListWidgetInstalled = false;
@@ -263,6 +264,12 @@ function refreshMainListWidget(): void {
     s.mainListDeadlineScheduler?.schedule(nextExpiry - now);
   }
   syncMainListSelection(rows);
+  const nextSignature = rows.length ? mainListSignature(rows, {
+    selectedId: s.mainListFocused ? s.mainListSelectedId : undefined,
+    focused: s.mainListFocused === true,
+  }) : undefined;
+  const changed = nextSignature !== s.lastMainListSignature;
+  s.lastMainListSignature = nextSignature;
   s.lastMainListLines = rows.length ? buildMainListLines(rows, MAIN_LIST_FALLBACK_WIDTH, deps.truncate, themeFg(ctx), {
     selectedId: s.mainListFocused ? s.mainListSelectedId : undefined,
     focused: s.mainListFocused === true,
@@ -272,6 +279,7 @@ function refreshMainListWidget(): void {
     try { (ctx.ui as any).setWidget?.(MAIN_LIST_WIDGET_KEY, undefined); } catch { /* ignore */ }
     s.mainListWidgetInstalled = false;
     s.mainListRequestRender = undefined;
+    s.lastMainListSignature = undefined;
     return;
   }
   if (!s.mainListWidgetInstalled) {
@@ -280,7 +288,9 @@ function refreshMainListWidget(): void {
       s.mainListWidgetInstalled = true;
     } catch { /* ignore */ }
   }
-  try { s.mainListRequestRender?.(); } catch { /* ignore */ }
+  if (changed) {
+    try { s.mainListRequestRender?.(); } catch { /* ignore */ }
+  }
 }
 
 function themeFg(ctx: ExtensionContext): (color: string, value: string) => string {
@@ -481,9 +491,40 @@ function statusGlyph(row: InternalRow): string {
 }
 
 function statusIndicator(row: InternalRow, now = Date.now()): { glyph: string; color: string } {
+  void now;
   if (row.statusTone !== "running") return { glyph: statusGlyph(row), color: toneColor(row.statusTone, row.status) };
-  const frame = Math.floor(now / DETAIL_TICK_MS) % RUNNING_DOT_FRAMES.length;
-  return { glyph: RUNNING_DOT_GLYPH, color: RUNNING_DOT_FRAMES[frame]! };
+  return { glyph: RUNNING_DOT_GLYPH, color: "accent" };
+}
+
+function mainListSignature(rows: InternalRow[], options: { selectedId?: string; focused?: boolean } = {}): string {
+  return JSON.stringify({
+    focused: options.focused === true,
+    selectedId: options.focused === true ? options.selectedId ?? null : null,
+    rows: rows.map((row) => ({
+      navigatorId: row.navigatorId,
+      providerLabel: row.providerLabel,
+      parentRow: row.parentRow === true,
+      providerId: row.providerId,
+      id: row.id,
+      name: row.name ?? null,
+      model: row.model ?? null,
+      effort: row.effort ?? null,
+      tool: row.tool ?? null,
+      tokens: row.tokens ?? null,
+      command: row.command ?? null,
+      status: row.status,
+      statusTone: row.statusTone,
+      kind: row.kind,
+      primary: row.primary,
+      secondary: row.secondary ?? null,
+      facts: stableFacts(row.facts),
+      sortStartedAt: row.sortStartedAt,
+    })),
+  });
+}
+
+function stableFacts(facts: string[] | undefined): string[] {
+  return (facts ?? []).filter((fact) => !/\bleft$/.test(singleLine(fact)));
 }
 
 function rowSummary(row: InternalRow): string {
@@ -868,13 +909,13 @@ function createOverlayComponent(
           width,
           deps.truncate,
           fg,
-          { minRows: detailRows },
+          { minRows: detailRows, bottomFooter: false, logTailRows },
         );
       } else {
         transcriptDetail = null;
         transcriptComponent = null;
         contentLines = mode === "detail"
-          ? buildDetailLines(detail, width, deps.truncate, fg, { expandedSections, logTailRows, minRows: detailRows })
+          ? buildDetailLines(detail, width, deps.truncate, fg, { expandedSections, logTailRows, minRows: detailRows, bottomFooter: false })
           : buildListLines(overlayState, width, deps.truncate, fg);
       }
       if (mode !== "detail") return contentLines;
@@ -953,9 +994,11 @@ function buildTranscriptDetailLines(
   width: number,
   truncate: (s: string, width: number) => string,
   fg: (color: string, value: string) => string,
-  options: { minRows?: number } = {},
+  options: { minRows?: number; bottomFooter?: boolean; logTailRows?: number } = {},
 ): string[] {
   const actions = [...(detail.footerActions ?? ["x close"]), "Esc close"].join(" · ");
+  const tailRows = options.logTailRows ?? DEFAULT_LOG_TAIL_ROWS;
+  const shownTranscriptLines = transcriptLines.length ? transcriptLines.slice(-tailRows) : ["   (no transcript yet)"];
   const lines: string[] = [
     fg("accent", rule(detail.title, width)),
     dim(`   ← main · ${actions}`, fg),
@@ -964,10 +1007,11 @@ function buildTranscriptDetailLines(
   ];
   if (detail.subtitle) lines.push(`   summary  ${detail.subtitle}`);
   for (const item of detail.metadata) lines.push(`   ${item.label.padEnd(8, " ").slice(0, 8)} ${item.value}`);
-  lines.push("", dim(section("transcript", width), fg));
+  lines.push("", dim(section(`transcript · latest ${tailRows} rows`, width), fg));
   if (detail.transcriptDiagnostic) lines.push(`   ${dim(detail.transcriptDiagnostic, fg)}`);
-  lines.push(...(transcriptLines.length ? transcriptLines : ["   (no transcript yet)"]));
+  lines.push(...shownTranscriptLines);
   lines.push("");
+  if (options.bottomFooter === false) return lines.map((line) => safeTruncate(line, width, truncate));
   const footerLines = [dim(`   ← main · ${actions}`, fg), dim(rule("", width), fg)];
   padBeforeFooter(lines, footerLines.length, options.minRows);
   lines.push(...footerLines);
@@ -1051,10 +1095,11 @@ function buildDetailLines(
   width: number,
   truncate: (s: string, width: number) => string,
   fg: (color: string, value: string) => string,
-  options: { expandedSections?: Set<string>; logTailRows?: number; minRows?: number } = {},
+  options: { expandedSections?: Set<string>; logTailRows?: number; minRows?: number; bottomFooter?: boolean } = {},
 ): string[] {
   if (!detail) {
     const lines = [rule("Work unavailable", width), dim("   ← back · Esc close", fg), ""];
+    if (options.bottomFooter === false) return lines.map((line) => safeTruncate(line, width, truncate));
     const footerLines = [dim(rule("", width), fg)];
     padBeforeFooter(lines, footerLines.length, options.minRows);
     lines.push(...footerLines);
@@ -1108,14 +1153,17 @@ function buildDetailLines(
       for (const raw of shown) lines.push(raw ? `   ${raw}` : "   ");
     }
   } else {
-    const evidenceLabel = /log/i.test(detail.evidence.label) ? `${detail.evidence.label} · latest ${tailRows} rows` : detail.evidence.label;
+    const tailEvidence = isTailEvidence(detail);
+    const evidenceLabel = tailEvidence ? `${detail.evidence.label} · latest ${tailRows} rows` : detail.evidence.label;
     lines.push(dim(section(evidenceLabel, width), fg));
     const wrapped = /log/i.test(detail.evidence.label)
       ? wrapLogText(body, width - 6)
-      : body.split(/\r?\n/);
-    for (const raw of wrapped) lines.push(raw ? `   ${raw}` : "   ");
+      : wrapEvidenceText(body, width - 6);
+    const shown = tailEvidence ? wrapped.slice(-tailRows) : wrapped;
+    for (const raw of shown) lines.push(raw ? `   ${raw}` : "   ");
   }
   lines.push("");
+  if (options.bottomFooter === false) return lines.map((line) => safeTruncate(line, width, truncate));
   const footerLines = [dim(`   ← back · ${actions}`, fg), dim(rule("", width), fg)];
   padBeforeFooter(lines, footerLines.length, options.minRows);
   lines.push(...footerLines);
@@ -1141,7 +1189,11 @@ function applyDefaultExpandedSections(detail: BackgroundWorkDetail | null, expan
 }
 
 function isFoldableEvidence(detail: BackgroundWorkDetail): boolean {
-  return !/log/i.test(detail.evidence.label);
+  return !isTailEvidence(detail);
+}
+
+function isTailEvidence(detail: BackgroundWorkDetail): boolean {
+  return /log|transcript/i.test(detail.evidence.label);
 }
 
 function sectionHeader(label: string, width: number): string {