@masslessai/push-todo 3.10.7 → 3.10.9

package/lib/daemon-health.js

@@ -8,7 +8,7 @@
  */
 
 import { existsSync, readFileSync, writeFileSync, mkdirSync, unlinkSync } from 'fs';
-import { spawn } from 'child_process';
+import { spawn, execFileSync } from 'child_process';
 import { homedir } from 'os';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
@@ -116,6 +116,30 @@ function formatUptime(startedAt) {
   }
 }
 
+/**
+ * Kill any orphaned daemon processes not tracked by PID file.
+ * Prevents zombie daemons from racing the real daemon for tasks.
+ */
+function killOrphanedDaemons() {
+  try {
+    const trackedPid = existsSync(PID_FILE)
+      ? parseInt(readFileSync(PID_FILE, 'utf8').trim(), 10)
+      : null;
+    const output = execFileSync('pgrep', ['-f', 'node.*daemon\\.js'], {
+      encoding: 'utf8',
+      timeout: 5000
+    }).trim();
+    for (const line of output.split('\n')) {
+      const pid = parseInt(line.trim(), 10);
+      if (!isNaN(pid) && pid !== trackedPid && pid !== process.pid) {
+        try { process.kill(pid, 'SIGTERM'); } catch {}
+      }
+    }
+  } catch {
+    // pgrep not found or no matches — both fine
+  }
+}
+
 /**
  * Start the daemon process.
  * Same as Python's start_daemon().
@@ -126,6 +150,9 @@ export function startDaemon() {
     return true;
   }
 
+  // Kill any zombie daemons before starting fresh
+  killOrphanedDaemons();
+
   mkdirSync(PUSH_DIR, { recursive: true });
 
   const daemonScript = join(__dirname, 'daemon.js');
@@ -138,7 +165,12 @@ export function startDaemon() {
     const child = spawn(process.execPath, [daemonScript], {
       detached: true,
       stdio: ['ignore', 'ignore', 'ignore'],
-      env: { ...process.env, PUSH_DAEMON: '1' }
+      env: (() => {
+        const env = { ...process.env, PUSH_DAEMON: '1' };
+        delete env.CLAUDECODE;  // Strip to avoid leaking into Claude child processes
+        delete env.CLAUDE_CODE_ENTRYPOINT;  // Strip to avoid any entrypoint-based guards
+        return env;
+      })()
     });
 
     writeFileSync(PID_FILE, String(child.pid));

package/lib/daemon.js

@@ -92,6 +92,7 @@ const taskDetails = new Map();  // displayNumber -> details
 const completedToday = [];
 const taskLastOutput = new Map(); // displayNumber -> timestamp
 const taskStdoutBuffer = new Map(); // displayNumber -> lines[]
+const taskStderrBuffer = new Map(); // displayNumber -> lines[]
 const taskProjectPaths = new Map(); // displayNumber -> projectPath
 let daemonStartTime = null;
 
@@ -1380,6 +1381,7 @@ IMPORTANT:
       env: (() => {
         const env = { ...process.env, PUSH_TASK_ID: task.id, PUSH_DISPLAY_NUMBER: String(displayNumber) };
         delete env.CLAUDECODE;  // Strip to avoid "nested session" guard in Claude Code
+        delete env.CLAUDE_CODE_ENTRYPOINT;  // Strip to avoid any entrypoint-based guards
         return env;
       })()
     });
@@ -1395,6 +1397,20 @@ IMPORTANT:
     runningTasks.set(displayNumber, taskInfo);
     taskLastOutput.set(displayNumber, Date.now());
     taskStdoutBuffer.set(displayNumber, []);
+    taskStderrBuffer.set(displayNumber, []);
+
+    // Monitor stderr (critical for diagnosing fast exits)
+    child.stderr.on('data', (data) => {
+      const lines = data.toString().split('\n');
+      for (const line of lines) {
+        if (line.trim()) {
+          const buffer = taskStderrBuffer.get(displayNumber) || [];
+          buffer.push(line);
+          if (buffer.length > 20) buffer.shift();
+          taskStderrBuffer.set(displayNumber, buffer);
+        }
+      }
+    });
 
     // Monitor stdout
     child.stdout.on('data', (data) => {
@@ -1579,7 +1595,11 @@ async function handleTaskCompletion(displayNumber, exitCode) {
       sessionId
     });
   } else {
-    const stderr = taskInfo.process.stderr?.read()?.toString() || '';
+    const stderrLines = taskStderrBuffer.get(displayNumber) || [];
+    const stderr = stderrLines.join('\n');
+    if (stderr) {
+      log(`Task #${displayNumber} stderr: ${stderr.slice(0, 500)}`);
+    }
 
     // Ask Claude to explain what went wrong (needs worktree path)
     const failureSummary = extractSemanticSummary(worktreePath, sessionId);
@@ -1614,6 +1634,7 @@ async function handleTaskCompletion(displayNumber, exitCode) {
   taskDetails.delete(displayNumber);
   taskLastOutput.delete(displayNumber);
   taskStdoutBuffer.delete(displayNumber);
+  taskStderrBuffer.delete(displayNumber);
   taskProjectPaths.delete(displayNumber);
   updateStatusFile();
 }
@@ -1757,6 +1778,7 @@ async function checkTimeouts() {
     taskDetails.delete(displayNumber);
     taskLastOutput.delete(displayNumber);
     taskStdoutBuffer.delete(displayNumber);
+    taskStderrBuffer.delete(displayNumber);
     taskProjectPaths.delete(displayNumber);
     cleanupWorktree(displayNumber, projectPath);
   }
@@ -1797,10 +1819,13 @@ function checkAndApplyUpdate() {
 
       // Spawn new daemon from updated code, then exit
       const daemonScript = join(__dirname, 'daemon.js');
+      const selfUpdateEnv = { ...process.env, PUSH_DAEMON: '1' };
+      delete selfUpdateEnv.CLAUDECODE;  // Strip to avoid leaking into Claude child processes
+      delete selfUpdateEnv.CLAUDE_CODE_ENTRYPOINT;  // Strip to avoid any entrypoint-based guards
       const child = spawn(process.execPath, [daemonScript], {
         detached: true,
         stdio: ['ignore', 'ignore', 'ignore'],
-        env: { ...process.env, PUSH_DAEMON: '1' }
+        env: selfUpdateEnv
       });
       writeFileSync(PID_FILE, String(child.pid));
       child.unref();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@masslessai/push-todo",
-  "version": "3.10.7",
+  "version": "3.10.9",
   "description": "Voice tasks from Push iOS app for Claude Code",
   "type": "module",
   "bin": {