pi-better-background-tasks 0.2.20 → 0.4.0

package/src/shared-navigator.ts

@@ -106,7 +106,7 @@ export const CLOSE_CONFIRM_STATUS_KEY = "background-work-close";
 export const MAIN_LIST_WIDGET_KEY = "background-work-list";
 export const DETAIL_TICK_MS = 10_000;
 export const CLOSE_ARM_MS = 3000;
-export const DEFAULT_LOG_TAIL_ROWS = 10;
+export const DEFAULT_LOG_TAIL_ROWS = 25;
 export const LOG_TAIL_ROW_CHOICES = [10, 25] as const;
 const MAIN_LIST_FALLBACK_WIDTH = 100;
 const DETAIL_OVERLAY_BOTTOM_MARGIN_ROWS = 0;
@@ -542,7 +542,10 @@ function stableFacts(facts: string[] | undefined): string[] {
 
 function rowSummary(row: InternalRow): string {
   if (row.providerId === "subagents") {
+    const attention = row.facts?.map(singleLine).filter(Boolean);
+    if (attention?.length) return `${singleLine(row.status)} · ${attention.join(" · ")}`;
     const parts: string[] = [];
+    if (row.statusTone !== "running" && row.kind !== "main agent") parts.push(singleLine(row.status));
     if (row.model) parts.push(row.effort ? `${singleLine(row.model)} ${singleLine(row.effort)}` : singleLine(row.model));
     if (row.tool) parts.push(`tool ${singleLine(row.tool)}`);
     if (row.tokens) parts.push(singleLine(row.tokens));
@@ -928,7 +931,7 @@ function createOverlayComponent(
         transcriptDetail = null;
         transcriptComponent = null;
         contentLines = mode === "detail"
-          ? buildDetailLines(detail, width, deps.truncate, fg, { expandedSections, logTailRows, minRows: detailRows, bottomFooter: false })
+          ? buildDetailLines(detail, width, deps.truncate, fg, { expandedSections, logTailRows, minRows: detailRows, maxRows: detailRows, bottomFooter: false })
           : buildListLines(overlayState, width, deps.truncate, fg);
       }
       if (mode !== "detail") return contentLines;
@@ -1024,14 +1027,11 @@ function buildTranscriptDetailLines(
   ];
   const transcriptHeader = ["", dim(section(`transcript · latest ${tailRows} rows`, width), fg)];
   if (detail.transcriptDiagnostic) transcriptHeader.push(`   ${dim(detail.transcriptDiagnostic, fg)}`);
+  // Metadata always renders; the transcript tail takes only the rows left over (the tail size is a cap, not a guarantee).
+  lines.push(...optionalMetadata);
   if (options.maxRows !== undefined) {
-    const fixedRows = lines.length + transcriptHeader.length + 1;
-    const transcriptBudget = Math.max(1, options.maxRows - fixedRows);
+    const transcriptBudget = Math.max(1, options.maxRows - lines.length - transcriptHeader.length - 1);
     shownTranscriptLines = shownTranscriptLines.slice(-transcriptBudget);
-    const metadataBudget = Math.max(0, options.maxRows - fixedRows - shownTranscriptLines.length);
-    lines.push(...optionalMetadata.slice(0, metadataBudget));
-  } else {
-    lines.push(...optionalMetadata);
   }
   lines.push(...transcriptHeader);
   lines.push(...shownTranscriptLines);
@@ -1120,7 +1120,7 @@ function buildDetailLines(
   width: number,
   truncate: (s: string, width: number) => string,
   fg: (color: string, value: string) => string,
-  options: { expandedSections?: Set<string>; logTailRows?: number; minRows?: number; bottomFooter?: boolean } = {},
+  options: { expandedSections?: Set<string>; logTailRows?: number; minRows?: number; maxRows?: number; bottomFooter?: boolean } = {},
 ): string[] {
   if (!detail) {
     const lines = [rule("Work unavailable", width), dim("   ← back · Esc close", fg), ""];
@@ -1184,7 +1184,9 @@ function buildDetailLines(
     const wrapped = /log/i.test(detail.evidence.label)
       ? wrapLogText(body, width - 6)
       : wrapEvidenceText(body, width - 6);
-    const shown = tailEvidence ? wrapped.slice(-tailRows) : wrapped;
+    let shown = tailEvidence ? wrapped.slice(-tailRows) : wrapped;
+    // Metadata above always renders; a tail that outgrows the remaining rows drops its oldest rows so the newest stay on screen (+1 for the trailing blank).
+    if (tailEvidence && options.maxRows !== undefined) shown = shown.slice(-Math.max(1, options.maxRows - lines.length - 1));
     for (const raw of shown) lines.push(raw ? `   ${raw}` : "   ");
   }
   lines.push("");