@mjasnikovs/pi-task 0.14.0 → 0.14.2

package/dist/remote/ui-script.js

@@ -53,6 +53,7 @@ export function clientScript(wsUrl) {
     let autoScroll = true;
     let reconnectDelay = 1000;
     let reconnectAnim = null;
+    let reconnectTimer = null;
     let ws = null;
 
     const BT = String.fromCharCode(96);
@@ -784,8 +785,15 @@ export function clientScript(wsUrl) {
     });
 
     function connect() {
-      ws = new WebSocket(WS_URL);
-      ws.addEventListener('open', () => {
+      // Capture the socket locally so a superseded socket's late events (a
+      // delayed 'close' after we already opened a replacement) can't touch the
+      // overlay or schedule a second reconnect — every handler bails unless it's
+      // still the current socket.
+      const sock = new WebSocket(WS_URL);
+      ws = sock;
+      sock.addEventListener('open', () => {
+        if (ws !== sock) return;
+        if (reconnectTimer) { clearTimeout(reconnectTimer); reconnectTimer = null; }
         if (reconnectAnim) { clearInterval(reconnectAnim); reconnectAnim = null; }
         reconnectOverlay.classList.remove('visible');
         reconnectDelay = 1000;
@@ -797,10 +805,12 @@ export function clientScript(wsUrl) {
         // endpoint, so a redundant re-POST is harmless.
         if (notifyEnabled()) { subscribePush().catch(function () {}); }
       });
-      ws.addEventListener('message', (e) => {
+      sock.addEventListener('message', (e) => {
+        if (ws !== sock) return;
         try { handleMsg(JSON.parse(e.data)); } catch {}
       });
-      ws.addEventListener('close', () => {
+      sock.addEventListener('close', () => {
+        if (ws !== sock) return;
         setEnabled(false);
         reconnectOverlay.classList.add('visible');
         // Animate the same braille spinner used elsewhere, with a live countdown.
@@ -816,9 +826,31 @@ export function clientScript(wsUrl) {
         if (reconnectAnim) clearInterval(reconnectAnim);
         paint();
         reconnectAnim = setInterval(paint, 90);
-        setTimeout(() => { reconnectDelay = Math.min(reconnectDelay * 2, 30000); connect(); }, reconnectDelay);
+        if (reconnectTimer) clearTimeout(reconnectTimer);
+        reconnectTimer = setTimeout(() => {
+          reconnectTimer = null;
+          reconnectDelay = Math.min(reconnectDelay * 2, 30000);
+          connect();
+        }, reconnectDelay);
       });
     }
 
+    // Reconnect immediately instead of waiting out the exponential backoff. A
+    // phone that backgrounds the PWA throttles our retry timer and the radio
+    // drops, so by the time it foregrounds reconnectDelay can be pinned at 30s —
+    // leaving the user staring at a spinner (over an already-updated question)
+    // while the server is reachable RIGHT NOW. Returning to the tab, regaining
+    // network, or refocusing the window should all retry at once. No-op if a
+    // socket is already open or a connect is in flight.
+    function connectNow() {
+      if (ws && (ws.readyState === WebSocket.CONNECTING || ws.readyState === WebSocket.OPEN)) return;
+      if (reconnectTimer) { clearTimeout(reconnectTimer); reconnectTimer = null; }
+      reconnectDelay = 1000; // a deliberate return shouldn't inherit a stale 30s backoff
+      connect();
+    }
+    document.addEventListener('visibilitychange', () => { if (!document.hidden) connectNow(); });
+    window.addEventListener('online', connectNow);
+    window.addEventListener('focus', connectNow);
+
     connect();`;
 }

package/dist/task/enforce-guidelines.d.ts

@@ -74,6 +74,13 @@ export declare function classifyEnforceChildFailure(r: EnforceChildResult): stri
  * names of any new (untracked) files. Non-destructive — it does not touch the
  * index, so the later `git add -A` in gitCommitAll still stages everything
  * (including fixes the enforcement child makes).
+ *
+ * The `.pi-tasks/` directory is excluded from both git commands. Those task
+ * files are committable (tracked) by design, so they show up in `git diff HEAD`
+ * and `git ls-files --others` — git does not honor the fd/ripgrep `.ignore` that
+ * keeps them out of the worker's find/grep discovery. Without this pathspec the
+ * enforce child is handed its own TASK_*.md / TASK_AUTO_*.md bookkeeping as
+ * "changes to verify" and edits them, corrupting the front matter mid-run.
  */
 export declare function captureDiff(cwd: string, signal?: AbortSignal, spawnFn?: SpawnFn): Promise<string>;
 export interface EnforcementDeps {

package/dist/task/enforce-guidelines.js

@@ -20,6 +20,7 @@ import * as fsp from 'node:fs/promises';
 import * as path from 'node:path';
 import { runChildDefault } from '../shared/child-process.js';
 import { USER_CANCELLED } from './child-runner.js';
+import { TASKS_DIR_NAME } from './task-types.js';
 /** Filenames discovered in the working directory (cwd only — no tree walk). */
 export const GUIDELINE_FILENAMES = ['AGENTS.md', 'CLAUDE.md'];
 /** Tools the fix pass is allowed: read/search to inspect, edit/write to fix. */
@@ -131,11 +132,28 @@ export function classifyEnforceChildFailure(r) {
  * names of any new (untracked) files. Non-destructive — it does not touch the
  * index, so the later `git add -A` in gitCommitAll still stages everything
  * (including fixes the enforcement child makes).
+ *
+ * The `.pi-tasks/` directory is excluded from both git commands. Those task
+ * files are committable (tracked) by design, so they show up in `git diff HEAD`
+ * and `git ls-files --others` — git does not honor the fd/ripgrep `.ignore` that
+ * keeps them out of the worker's find/grep discovery. Without this pathspec the
+ * enforce child is handed its own TASK_*.md / TASK_AUTO_*.md bookkeeping as
+ * "changes to verify" and edits them, corrupting the front matter mid-run.
  */
 export async function captureDiff(cwd, signal, spawnFn) {
+    // `:(exclude)<dir>` is a git pathspec that drops everything under the tasks
+    // directory from the result, leaving only real source changes to verify.
+    const excludeTasks = `:(exclude)${TASKS_DIR_NAME}`;
     const run = (args) => runChildDefault({ command: 'git', args }, cwd, signal, { mode: 'text' }, spawnFn);
-    const tracked = await run(['diff', 'HEAD']);
-    const untracked = await run(['ls-files', '--others', '--exclude-standard']);
+    const tracked = await run(['diff', 'HEAD', '--', '.', excludeTasks]);
+    const untracked = await run([
+        'ls-files',
+        '--others',
+        '--exclude-standard',
+        '--',
+        '.',
+        excludeTasks
+    ]);
     const parts = [];
     if (tracked.exitCode === 0 && tracked.stdout.trim().length > 0)
         parts.push(tracked.stdout.trim());

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.14.0",
+    "version": "0.14.2",
     "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",