@mjasnikovs/pi-task 0.39.2 → 0.39.3

package/dist/task/widget.js

@@ -115,8 +115,6 @@ export function buildWidgetData(s) {
 }
 // ─── Widget lifecycle ────────────────────────────────────────────────────────
 export function startWidget(ctx, getState) {
-    if (!ctx.hasUI)
-        return () => { };
     // pi's `ctx.ui` getter calls `runner.assertActive()`, which THROWS once the
     // ctx goes stale (/reload, session replacement) — so the theme read belongs
     // INSIDE the guard, not one line above it. render() runs from a timer, and a
@@ -128,7 +126,12 @@ export function startWidget(ctx, getState) {
         if (stale)
             return;
         const s = getState();
-        const plain = s ? buildWidgetLines(s, undefined) : undefined; // un-themed for the wire; needs no ctx
+        // The wire half FIRST, and unguarded: it needs no ctx, so gating it on a
+        // local terminal that may not exist would leave a headless host's browser
+        // with no phase, no progress and no failure flash.
+        setTaskWidget(s ? buildWidgetLines(s, undefined) : undefined, s ? buildWidgetData(s) : null);
+        if (!ctx.hasUI)
+            return;
         try {
             const lines = s ? buildWidgetLines(s, ctx.ui.theme) : undefined;
             ctx.ui.setWidget(WIDGET_KEY, lines);
@@ -136,9 +139,7 @@ export function startWidget(ctx, getState) {
         catch {
             stale = true;
             clearInterval(timer);
-            return;
         }
-        setTaskWidget(plain, s ? buildWidgetData(s) : null);
     };
     // The timer is created BEFORE the first render so `timer` is always bound
     // when render's catch reaches for it. Calling render() first instead would
@@ -208,16 +209,15 @@ export function buildAutoLoaderData(s) {
  * (returns a no-op disposer) when there's no UI.
  */
 export function startAutoLoader(ctx, getState) {
-    if (!ctx.hasUI)
-        return () => { };
-    // Same staleness hazard as startWidget: the theme read must sit inside the
-    // guard, and a stale ctx stops the timer rather than throwing every tick.
+    // Same staleness hazard as startWidget, and the same wire-half-first order.
     let stale = false;
     const render = () => {
         if (stale)
             return;
         const s = getState();
-        const plain = s ? buildAutoLoaderLines(s, undefined) : undefined; // needs no ctx
+        setTaskWidget(s ? buildAutoLoaderLines(s, undefined) : undefined, s ? buildAutoLoaderData(s) : null);
+        if (!ctx.hasUI)
+            return;
         try {
             const lines = s ? buildAutoLoaderLines(s, ctx.ui.theme) : undefined;
             ctx.ui.setWidget(AUTO_WIDGET_KEY, lines);
@@ -225,9 +225,7 @@ export function startAutoLoader(ctx, getState) {
         catch {
             stale = true;
             clearInterval(timer);
-            return;
         }
-        setTaskWidget(plain, s ? buildAutoLoaderData(s) : null);
     };
     // The timer is created BEFORE the first render so `timer` is always bound
     // when render's catch reaches for it. Calling render() first instead would
@@ -277,40 +275,33 @@ export function buildImplData(s) {
     return d;
 }
 export function flashTerminalWidget(ctx, state, taskId, reason) {
-    if (!ctx.hasUI)
-        return;
-    // Same staleness hazard as the render timers: bail rather than throw.
-    let theme;
-    try {
-        theme = ctx.ui.theme;
-    }
-    catch {
-        return;
-    }
-    let line;
-    let clearMs;
-    if (state === 'cancelled') {
-        line = theme.fg('warning', `⚠ ${taskId} cancelled`);
-        clearMs = NOTIFY_CLEAR_MS;
-    }
-    else {
-        line = theme.fg('error', `✘ ${taskId} failed${reason ? ': ' + reason : ''}`);
-        clearMs = FAIL_CLEAR_MS;
-    }
     const plainLine = state === 'cancelled' ?
         `⚠ ${taskId} cancelled`
         : `✘ ${taskId} failed${reason ? ': ' + reason : ''}`;
-    try {
-        ctx.ui.setWidget(WIDGET_KEY, [line]);
-        setTaskWidget([plainLine]);
-    }
-    catch {
-        /* stale ctx */
+    const clearMs = state === 'cancelled' ? NOTIFY_CLEAR_MS : FAIL_CLEAR_MS;
+    setTaskWidget([plainLine]);
+    // The wire half is already done, so a missing or stale local ctx no longer
+    // costs the browser the flash as well. Same staleness hazard as the render
+    // timers: bail rather than throw.
+    if (ctx.hasUI) {
+        try {
+            const theme = ctx.ui.theme;
+            const tint = state === 'cancelled' ? 'warning' : 'error';
+            ctx.ui.setWidget(WIDGET_KEY, [theme.fg(tint, plainLine)]);
+        }
+        catch {
+            /* stale ctx */
+        }
     }
     setTimeout(() => {
+        // Wire first, for the same reason as the flash above: a ctx that went
+        // stale in the meantime would otherwise leave the browser showing the
+        // flash for the rest of the session.
+        setTaskWidget(undefined);
+        if (!ctx.hasUI)
+            return;
         try {
             ctx.ui.setWidget(WIDGET_KEY, undefined);
-            setTaskWidget(undefined);
         }
         catch {
             /* stale ctx */

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.39.2",
+    "version": "0.39.3",
     "description": "Deterministic task planning and spec-orchestration for local models — crash-safe /task pipelines with verify/enforce gates, a real-time remote web view, and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",