nothumanallowed 14.3.4 → 14.3.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "14.3.4",
+  "version": "14.3.6",
   "description": "NotHumanAllowed — 38 AI agents, 80 tools, Studio (visual agentic workflows). Email, calendar, browser automation, screen capture, canvas, cron/heartbeat, Alexandria E2E messaging, GitHub, Notion, Slack, voice chat, free AI (Liara), 28 languages. Zero-dependency CLI.",
   "type": "module",
   "bin": {

package/src/commands/ui.mjs

@@ -38,6 +38,33 @@ export async function cmdUI(args) {
 
   const host = explicitHost || '127.0.0.1';
 
+  // Kill any existing process on this port (prevents EADDRINUSE)
+  try {
+    const { execSync } = await import('child_process');
+    if (process.platform === 'win32') {
+      // Windows: find PID on port and kill it
+      const out = execSync(`netstat -ano | findstr :${port} | findstr LISTENING`, { encoding: 'utf-8', timeout: 3000 }).trim();
+      const pid = out.split(/\s+/).pop();
+      if (pid && pid !== String(process.pid)) {
+        execSync(`taskkill /F /PID ${pid}`, { timeout: 3000, stdio: 'ignore' });
+        console.log(`  \x1b[33m!\x1b[0m Killed previous process on port ${port} (PID ${pid})`);
+      }
+    } else {
+      // Unix/Mac: lsof to find and kill
+      const out = execSync(`lsof -ti:${port} 2>/dev/null`, { encoding: 'utf-8', timeout: 3000 }).trim();
+      if (out) {
+        const pids = out.split('\n').filter((p) => p && p !== String(process.pid));
+        for (const pid of pids) {
+          try { process.kill(parseInt(pid), 'SIGTERM'); } catch {}
+        }
+        if (pids.length > 0) {
+          console.log(`  \x1b[33m!\x1b[0m Killed previous process on port ${port} (PID ${pids.join(', ')})`);
+          await new Promise((r) => setTimeout(r, 500)); // wait for port to free
+        }
+      }
+    }
+  } catch { /* port is free */ }
+
   // Auto-start ops daemon (Telegram + cron) if not already running
   if (!isRunning()) {
     const result = startDaemon();

package/src/constants.mjs

@@ -5,7 +5,7 @@ import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-export const VERSION = '14.3.4';
+export const VERSION = '14.3.6';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/server/routes/config.mjs

@@ -117,19 +117,18 @@ export function register(router) {
             }
           } catch { /* daemon restart is best-effort */ }
 
-          // Self-restart: launch a shell script that waits for us to die, then starts nha ui.
-          // This avoids the port-in-use race condition.
-          // Detect current port from the server's address
+          // Self-restart: write a temp script, run it with nohup, then exit.
+          // This survives parent death on macOS/Linux/Windows.
           let port = '3847';
           try { port = String(req.socket?.localPort || 3847); } catch {}
-          const { spawn } = await import('child_process');
-          const restartScript = `sleep 2 && nha ui --port ${port}`;
-          const child = spawn('sh', ['-c', restartScript], {
-            detached: true,
+          const os = await import('os');
+          const tmpScript = path.join(os.default.tmpdir(), '.nha-restart.sh');
+          fs.writeFileSync(tmpScript, `#!/bin/sh\nsleep 3\nnha ui --port ${port}\n`, { mode: 0o755 });
+          execSync(`nohup ${tmpScript} > /dev/null 2>&1 &`, {
+            timeout: 3000,
             stdio: 'ignore',
-            env: { ...process.env, NHA_RESTARTED: '1' },
+            shell: true,
           });
-          child.unref();
         } catch { /* ignore errors */ }
         process.exit(0);
       }, 1500);