pi-better-background-tasks 0.2.0 → 0.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-background-tasks",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "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 {

package/src/tools.ts

@@ -23,12 +23,16 @@ const SshSchema = Type.Object({
   identity_file: Type.Optional(Type.String({ description: "SSH identity file path." })),
   jump: Type.Optional(Type.String({ description: "SSH jump host passed with -J." })),
   options: Type.Optional(Type.Record(Type.String(), Type.String(), { description: "Additional SSH -o key/value options. Agent-safe defaults remain enforced." })),
+}, {
+  description: "Structured SSH connection for remote background tasks. Set ssh instead of wrapping command in a hand-written ssh command; command is the remote command, and Pi keeps durable local logs, status, callbacks, and remote stop semantics.",
 });
 
 const RemoteSchema = Type.Object({
   session: Type.Optional(Type.Union([Type.Literal("tmux"), Type.Literal("direct")], { description: "Remote session mode. SSH spawn defaults to durable tmux. Watch always uses direct one-shot polls. Explicit direct spawn has weaker stop semantics." })),
   install_tmux: Type.Optional(Type.Boolean({ description: "Allow SSH spawn to install tmux non-interactively when missing. Defaults true in tmux mode and is ignored for watch and direct spawn." })),
   workdir: Type.Optional(Type.String({ description: "Remote working directory for the spawned command." })),
+}, {
+  description: "Remote execution controls used with ssh. Omit session for SSH spawn to get the durable tmux default; SSH watch runs direct one-shot polls regardless of session and does not install tmux.",
 });
 
 const CommandFields = {
@@ -117,7 +121,7 @@ export function registerTools(pi: ExtensionAPI): void {
   pi.registerTool({
     name: "bg_task_spawn",
     label: "BG Spawn",
-    description: "Start a long-running background process and return immediately with its task id. For remote work, prefer structured ssh: spawn defaults to a remote tmux session with durable local logs and real remote stop. If tmux is missing, the preset attempts to install tmux non-interactively and fails closed with operator guidance when setup cannot proceed. Explicit remote.session=direct skips tmux, but direct mode has weaker stop semantics and may leave the remote process running. Never wait or poll in the foreground.",
+    description: "Start a long-running background process and return immediately with its task id. For remote work, prefer structured ssh: pass ssh:{host,user} and put the remote command in command; spawn defaults to a remote tmux session with durable local logs and real remote stop. If tmux is missing, the preset attempts to install tmux non-interactively and fails closed with operator guidance when setup cannot proceed. Explicit remote.session=direct skips tmux, but direct mode has weaker stop semantics and may leave the remote process running. Never wait or poll in the foreground.",
     parameters: SpawnParams,
     async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
       activeSession = getCallbackOrigin(ctx);
@@ -130,7 +134,7 @@ export function registerTools(pi: ExtensionAPI): void {
   pi.registerTool({
     name: "bg_task_watch",
     label: "BG Watch",
-    description: "Poll a command in the background until success_when, failure_when, or timeout matches. For remote work, pass structured ssh fields and provide the remote command in command; each interval opens a direct one-shot SSH poll without tmux installation. Returns immediately with its task id. Default timeout 900 seconds; pass timeout_seconds:0 to disable.",
+    description: "Poll a command in the background until success_when, failure_when, or timeout matches. For remote work, prefer structured ssh: pass ssh:{host,user} and provide the remote command in command; each interval opens a direct one-shot SSH poll without tmux installation. Returns immediately with its task id. Default timeout 900 seconds; pass timeout_seconds:0 to disable.",
     parameters: WatchParams,
     async execute(_toolCallId, params, _signal, _onUpdate, ctx) {
       activeSession = getCallbackOrigin(ctx);
@@ -190,7 +194,7 @@ export function registerTools(pi: ExtensionAPI): void {
   pi.registerTool({
     name: "bg_task",
     label: "BG Task",
-    description: "Action wrapper for background tasks: spawn, watch, list, status, log, stop, or clear. For remote work, pass structured ssh fields and provide the remote command in command. SSH spawn defaults to durable tmux; SSH watches use direct one-shot polls without tmux installation; remote.session=direct is a weaker-stop spawn escape hatch. Spawn/watch return immediately; do not poll in foreground. For action:status, default compact output and use verbose:true only for full metadata. For action:log, default compact tail and use tail_lines:0 only for explicit full logs.",
+    description: "Action wrapper for background tasks: spawn, watch, list, status, log, stop, or clear. For remote work, prefer structured ssh: pass ssh:{host,user} and provide the remote command in command. SSH spawn defaults to durable tmux; SSH watches use direct one-shot polls without tmux installation; remote.session=direct is a weaker-stop spawn escape hatch. Spawn/watch return immediately; do not poll in foreground. For action:status, default compact output and use verbose:true only for full metadata. For action:log, default compact tail and use tail_lines:0 only for explicit full logs.",
     parameters: ActionParams,
     renderResult(result: unknown, options: unknown, theme: unknown) {
       return renderBackgroundTaskLogDisplay(result, options, theme);