pi-better-harness 0.1.18 → 0.1.20

package/node_modules/pi-better-background-tasks/package.json

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

package/node_modules/pi-better-background-tasks/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);

package/node_modules/pi-better-goal/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-goal",
-  "version": "0.1.20",
+  "version": "0.1.21",
   "description": "Pi extension for goal tracking with background-aware continuation.",
   "license": "MIT",
   "type": "module",

package/node_modules/pi-better-goal/src/index.ts

@@ -164,7 +164,7 @@ export default function (pi: ExtensionAPI): void {
   let continuationQueuedFor: string | null = null;
   let idleContinuationTimer: ReturnType<typeof setTimeout> | undefined;
   let idleContinuationSignature = "";
-  let refreshGoalWidget: (() => void) | undefined;
+  let refreshGoalWidget: ((force?: boolean) => void) | undefined;
   let lastAgentEvidence: ContinuationEvidence | null = null;
 
   const getGoal = (ctx: ExtensionContext): GoalSnapshot | null => currentGoalSnapshot(ctx);
@@ -180,21 +180,53 @@ export default function (pi: ExtensionAPI): void {
     appendContinuationState(createContinuationState(goal.goalId));
   };
 
+  const isForegroundBusy = (ctx: ExtensionContext): boolean =>
+    foregroundRunning || !ctx.isIdle();
+
+  /**
+   * Chat `notify` appends lines above the Working/bash status region. While the
+   * agent is streaming, that height change desyncs Pi main-screen differential
+   * rendering and stacks "Working..." / "Elapsed Xs" into scrollback. Keep
+   * mid-stream feedback out of the transcript; use the footer status instead.
+   */
+  const notifyGoal = (
+    ctx: ExtensionContext,
+    message: string,
+    type: "info" | "warning" | "error" = "info",
+  ): void => {
+    if (!ctx.hasUI) {
+      return;
+    }
+    if (isForegroundBusy(ctx) && type !== "error") {
+      try {
+        ctx.ui.setStatus(EXTENSION_NAME, message);
+      } catch {
+        // Footer status is best-effort only.
+      }
+      return;
+    }
+    ctx.ui.notify(message, type);
+  };
+
   const setGoal = (goal: GoalSnapshot, ctx: ExtensionContext, source: GoalEntrySource): void => {
+    const previous = getGoal(ctx);
+    const wasVisible = previous !== null && isGoalClockVisible(previous);
     pi.appendEntry(EXTENSION_NAME, goalSetEntry(goal, source));
     continuationQueuedFor = null;
     backgroundDrainTracker = null;
     clearIdleContinuation();
-    refreshGoalWidget?.();
+    // Force a full redraw when the dock height changes (absent ↔ visible clock).
+    refreshGoalWidget?.(!wasVisible || !isGoalClockVisible(goal));
   };
 
   const clearGoal = (ctx: ExtensionContext, source: GoalEntrySource): void => {
     const current = getGoal(ctx);
+    const wasVisible = current !== null && isGoalClockVisible(current);
     pi.appendEntry(EXTENSION_NAME, goalClearEntry(current?.goalId ?? null, source));
     continuationQueuedFor = null;
     backgroundDrainTracker = null;
     clearIdleContinuation();
-    refreshGoalWidget?.();
+    refreshGoalWidget?.(wasVisible);
   };
 
   const queueGoalContinuation = (goal: GoalSnapshot): void => {
@@ -395,8 +427,24 @@ export default function (pi: ExtensionAPI): void {
     ctx.ui.setWidget(
       EXTENSION_NAME,
       (tui, theme) => {
-        const scheduler = createRenderScheduler(() => tui.requestRender());
-        const refresh = (): void => scheduler.request();
+        const requestRender = (force = false): void => {
+          try {
+            tui.requestRender(force);
+          } catch {
+            // UI paint is best-effort only.
+          }
+        };
+        const scheduler = createRenderScheduler(() => requestRender(false));
+        const refresh = (force = false): void => {
+          if (force) {
+            scheduler.cancel();
+            // Height transitions must reset main-screen differential state so
+            // Working/Elapsed lines keep rewriting in place instead of stacking.
+            requestRender(true);
+            return;
+          }
+          scheduler.request();
+        };
         refreshGoalWidget = refresh;
 
         return {
@@ -444,49 +492,53 @@ export default function (pi: ExtensionAPI): void {
 
       if (!trimmed) {
         const continuation = current ? currentContinuationState(ctx, current.goalId) : null;
+        // Inspection is always explicit user intent; show the full summary even while busy.
         ctx.ui.notify(formatGoal(current, continuation, observeGoalStall(current, continuation, { foregroundRunning })), "info");
         return;
       }
 
       if (trimmed === "pause") {
         if (!current || current.status !== "active") {
-          ctx.ui.notify("Only active goals can be paused.", "warning");
+          notifyGoal(ctx, "Only active goals can be paused.", "warning");
           return;
         }
         setGoal(goalWithStatus(current, "paused"), ctx, "command");
-        ctx.ui.notify("Goal paused.", "info");
+        notifyGoal(ctx, "Goal paused.");
         return;
       }
 
       if (trimmed === "resume") {
         if (!current || current.status !== "paused") {
-          ctx.ui.notify("Only paused goals can be resumed.", "warning");
+          notifyGoal(ctx, "Only paused goals can be resumed.", "warning");
           return;
         }
         const goal = goalWithStatus(current, "active");
         setGoal(goal, ctx, "command");
         queueGoalContinuation(goal);
-        ctx.ui.notify("Goal resumed.", "info");
+        notifyGoal(ctx, "Goal resumed.");
         return;
       }
 
       if (trimmed === "clear") {
         clearGoal(ctx, "command");
-        ctx.ui.notify("Goal cleared.", "info");
+        notifyGoal(ctx, "Goal cleared.");
         return;
       }
 
       if (trimmed === "complete") {
         if (!current) {
-          ctx.ui.notify("No goal is set.", "warning");
+          notifyGoal(ctx, "No goal is set.", "warning");
           return;
         }
         setGoal(goalWithStatus(current, "complete"), ctx, "command");
-        ctx.ui.notify("Goal marked complete.", "info");
+        notifyGoal(ctx, "Goal marked complete.");
         return;
       }
 
-      if (current && current.status !== "complete" && ctx.hasUI) {
+      // Confirm dialogs swap the editor and reflow the dock. While the agent is
+      // streaming that desyncs main-screen differential paints, so replace
+      // silently mid-turn and only confirm when idle.
+      if (current && current.status !== "complete" && ctx.hasUI && !isForegroundBusy(ctx)) {
         const replace = await ctx.ui.confirm(
           "Replace active goal?",
           `Current goal: ${current.objective}`,
@@ -498,9 +550,9 @@ export default function (pi: ExtensionAPI): void {
 
       try {
         const goal = startOrReplaceGoal(trimmed, null, ctx, "command");
-        ctx.ui.notify(`Goal set: ${goal.objective}`, "info");
+        notifyGoal(ctx, `Goal set: ${goal.objective}`);
       } catch (error) {
-        ctx.ui.notify(error instanceof Error ? error.message : String(error), "error");
+        notifyGoal(ctx, error instanceof Error ? error.message : String(error), "error");
       }
     },
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-harness",
-  "version": "0.1.18",
+  "version": "0.1.20",
   "description": "Pi extension bundle for subagents, durable background tasks, and goal tracking.",
   "license": "MIT",
   "type": "module",
@@ -43,8 +43,8 @@
     "test": "node --test test/*.test.mjs"
   },
   "dependencies": {
-    "pi-better-background-tasks": "0.2.0",
-    "pi-better-goal": "0.1.20",
+    "pi-better-background-tasks": "0.2.1",
+    "pi-better-goal": "0.1.21",
     "pi-better-subagents": "0.1.19"
   },
   "bundledDependencies": [