@mjasnikovs/pi-task 0.13.5 → 0.13.6

package/dist/remote/bridge.js

@@ -1,4 +1,4 @@
-import { broadcast as wsBroadcast, hasConnectedClients } from './broadcast.js';
+import { broadcast as wsBroadcast } from './broadcast.js';
 import { pushNotify } from './push.js';
 import { setPrompt, clearPrompt } from './session-state.js';
 const g = globalThis;
@@ -56,9 +56,10 @@ export class SessionUI {
             allowSkip: spec.allowSkip
         };
         setPrompt(prompt);
-        // Reaches a backgrounded/suspended phone, which the in-page UI can't.
-        if (!hasConnectedClients())
-            void pushNotify('pi needs your input', spec.question, 'pi-prompt').catch(() => { });
+        // Reaches a backgrounded/suspended phone, which the in-page UI can't. The
+        // service worker drops the banner if a window is visible+focused (sw.ts),
+        // so we always push and let delivery-time visibility decide.
+        void pushNotify('pi needs your input', spec.question, 'pi-prompt').catch(() => { });
         // Local: resolves to a value/undefined, or undefined on abort. Swallow
         // the rejection some implementations throw on abort so it never leaks.
         const local = this.ctx.hasUI ?

package/dist/remote/broadcast.d.ts

@@ -3,4 +3,3 @@ export declare function addClient(ws: WebSocket): void;
 export declare function removeClient(ws: WebSocket): void;
 export declare function broadcast(msg: unknown): void;
 export declare function sendTo(ws: WebSocket, msg: unknown): void;
-export declare function hasConnectedClients(): boolean;

package/dist/remote/broadcast.js

@@ -22,10 +22,3 @@ export function sendTo(ws, msg) {
         ws.send(JSON.stringify(msg));
     }
 }
-export function hasConnectedClients() {
-    for (const ws of clients) {
-        if (ws.readyState === ws.OPEN)
-            return true;
-    }
-    return false;
-}

package/dist/remote/events.js

@@ -1,7 +1,6 @@
 import { setAgentIdle } from './state.js';
 import { pushNotify } from './push.js';
 import { publishNotify } from './bridge.js';
-import { hasConnectedClients } from './broadcast.js';
 import { agentStart, appendText, textEnd, startTool, updateTool, endTool, agentEnd, addUserTurn, addError, addSystemNote } from './session-state.js';
 /** Mirror pi agent events into the authoritative SessionState. Each handler
  *  drives a mutator, which updates the snapshot AND broadcasts the live delta. */
@@ -23,8 +22,10 @@ export function setupEvents(pi) {
             if (errorMessage || ae.reason === 'error') {
                 const message = errorMessage || 'Request failed';
                 addError(message);
-                if (!hasConnectedClients())
-                    void pushNotify('Agent error', message, 'pi-error').catch(() => { });
+                // Always push; the service worker suppresses the banner when a
+                // window is actually visible+focused (sw.ts), which is the only
+                // reliable foreground signal — an open WebSocket is not one.
+                void pushNotify('Agent error', message, 'pi-error').catch(() => { });
             }
         }
     });
@@ -43,8 +44,7 @@ export function setupEvents(pi) {
     pi.on('agent_end', (_event, ctx) => {
         setAgentIdle(true);
         agentEnd(ctx.getContextUsage());
-        if (!hasConnectedClients())
-            void pushNotify('Task finished', '', 'pi-end').catch(() => { });
+        void pushNotify('Task finished', '', 'pi-end').catch(() => { });
     });
     pi.on('input', (event, _ctx) => {
         if (event.source === 'interactive' && typeof event.text === 'string') {

package/dist/remote/server.js

@@ -121,21 +121,6 @@ export async function startServer(onMessage, getHtml) {
         handle.onFirstConnect = null;
         // One authoritative snapshot — the client replaces its whole view with it.
         sendTo(ws, snapshot());
-        // Heartbeat: ping every 30 s; terminate if the client doesn't respond.
-        // This catches phones that close/sleep without sending a TCP FIN, so the
-        // server's client-set stays accurate and push notifications fire correctly.
-        let alive = true;
-        const heartbeat = setInterval(() => {
-            if (!alive) {
-                ws.terminate();
-                return;
-            }
-            alive = false;
-            ws.ping();
-        }, 10_000);
-        ws.on('pong', () => {
-            alive = true;
-        });
         ws.on('message', data => {
             let msg;
             try {
@@ -157,7 +142,6 @@ export async function startServer(onMessage, getHtml) {
             onMessage(msg.text);
         });
         ws.on('close', () => {
-            clearInterval(heartbeat);
             removeClient(ws);
         });
     });

package/dist/task/phases.js

@@ -397,12 +397,23 @@ export async function phaseGrill(deps, ctx, widgetState, refined, research) {
             if (a === undefined)
                 throw new Error(USER_CANCELLED);
             const typed = a.trim();
+            // Two-option mode labels the choices "A:"/"B:", so a user (local TUI
+            // or remote "Manual answer") naturally types the bare letter to pick.
+            // Map it back to the option's full text — storing the literal "A"
+            // leaves the next grill-gen call a dangling reference it can't decode.
+            const twoOption = plainSuggested !== undefined && plainAlt !== undefined;
             if (typed.length === 0 && plainSuggested) {
                 answer = plainSuggested;
             }
             else if (typed.length === 0) {
                 answer = '(skipped)';
             }
+            else if (twoOption && /^a[.)]?$/i.test(typed)) {
+                answer = plainSuggested;
+            }
+            else if (twoOption && /^b[.)]?$/i.test(typed)) {
+                answer = plainAlt;
+            }
             else {
                 answer = typed;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.13.5",
+    "version": "0.13.6",
     "description": "Deterministic spec-orchestration for local models, with a bundled real-time remote web view and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",