myrlin-workbook 0.9.17 → 0.9.19

package/logs/server.pid

@@ -1 +1 @@
-21156
+49156

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "myrlin-workbook",
-  "version": "0.9.17",
+  "version": "0.9.19",
   "description": "Browser-based project manager for Claude Code sessions - session discovery, multi-terminal, cost tracking, docs, and kanban board",
   "main": "src/index.js",
   "bin": {

package/src/gui.js

@@ -191,10 +191,22 @@ if (!process.env.CWM_NO_OPEN) {
 
 // ─── Memory Watchdog ──────────────────────────────────────
 // Monitor RSS and kill idle PTY sessions when memory exceeds threshold.
-// This prevents the OS from silently OOM-killing the entire process tree.
-const MEMORY_CHECK_INTERVAL = 30000; // Check every 30 seconds
-const MEMORY_WARN_MB = 350;          // Warn and kill idle PTYs
-const MEMORY_CRITICAL_MB = 450;      // Force-kill all clientless PTYs
+// ConPTY on Windows allocates significant native memory (outside V8 heap)
+// so RSS can spike well beyond --max-old-space-size.
+const MEMORY_CHECK_INTERVAL = 15000; // Check every 15 seconds
+const MEMORY_WARN_MB = 200;          // Start killing idle (zero-client) PTYs
+const MEMORY_CRITICAL_MB = 350;      // Aggressively kill PTYs to survive
+
+// Periodic RSS logging so we can trace memory trajectory in server.log
+const _rssLogger = setInterval(() => {
+  const mem = process.memoryUsage();
+  const ptyManager = getPtyManager();
+  const ptySessions = ptyManager ? ptyManager.listSessions().length : 0;
+  try {
+    console.log(`[RSS] ${Math.round(mem.rss / 1024 / 1024)}MB heap=${Math.round(mem.heapUsed / 1024 / 1024)}/${Math.round(mem.heapTotal / 1024 / 1024)}MB pty=${ptySessions}`);
+  } catch (_) {}
+}, 60000);
+_rssLogger.unref();
 
 const _memoryWatchdog = setInterval(() => {
   const rssMB = Math.round(process.memoryUsage().rss / 1024 / 1024);
@@ -220,8 +232,8 @@ const _memoryWatchdog = setInterval(() => {
   }
 
   // In critical mode, also kill sessions that have been alive longest
-  if (isCritical && killed === 0 && sessions.length > 2) {
-    const oldest = sessions.sort((a, b) => (a.createdAt || 0) - (b.createdAt || 0));
+  if (isCritical && killed === 0 && sessions.length > 1) {
+    const oldest = [...sessions].sort((a, b) => (a.createdAt || 0) - (b.createdAt || 0));
     ptyManager.killSession(oldest[0].sessionId);
     killed++;
   }

package/src/supervisor.js

@@ -33,25 +33,57 @@ if (process.argv.includes('--daemon')) {
   const logDir = path.join(__dirname, '..', 'logs');
   if (!fs.existsSync(logDir)) fs.mkdirSync(logDir, { recursive: true });
   const logFile = path.join(logDir, 'server.log');
-  const out = fs.openSync(logFile, 'a');
-  const err = fs.openSync(logFile, 'a');
+  const pidFile = path.join(logDir, 'server.pid');
 
   // Strip --daemon from args so the child runs in foreground (supervised) mode
   const childArgs = process.argv.slice(2).filter(a => a !== '--daemon');
-
-  const child = spawn(process.execPath, [__filename, ...childArgs], {
-    stdio: ['ignore', out, err],
-    detached: true,
-    env: { ...process.env },
-  });
-
-  // Write PID file so other tools can find/stop the server
-  const pidFile = path.join(logDir, 'server.pid');
-  fs.writeFileSync(pidFile, String(child.pid), 'utf8');
-
-  child.unref();
-  console.log(`[supervisor] Daemonized server (PID ${child.pid}), logs at ${logFile}`);
-  process.exit(0);
+  const nodeExe = process.execPath;
+  const scriptArgs = [__filename, ...childArgs].map(a => `"${a}"`).join(' ');
+
+  if (process.platform === 'win32') {
+    // On Windows, Node's detached:true still inherits the console session's
+    // Job Object. When the parent shell (Git Bash, cmd, Claude Code) exits,
+    // Windows kills the entire job group. Use cmd.exe /c start to create a
+    // process in a completely new console session, then redirect its output.
+    const { execSync } = require('child_process');
+    const cmd = `cmd.exe /c start /b "" "${nodeExe}" --max-old-space-size=1024 ${scriptArgs} >> "${logFile}" 2>&1`;
+    execSync(cmd, { stdio: 'ignore', windowsHide: true });
+
+    // The PID isn't directly available from start /b. Write a marker so we
+    // can find it via tasklist. Wait briefly for the process to appear.
+    setTimeout(() => {
+      try {
+        const { execSync: es } = require('child_process');
+        const psCmd = `powershell.exe -NoProfile -Command "Get-CimInstance Win32_Process | Where-Object { $_.CommandLine -like '*supervisor.js*' -and $_.CommandLine -notlike '*--daemon*' } | Select-Object -ExpandProperty ProcessId"`;
+        const out = es(psCmd, { encoding: 'utf8', timeout: 10000 });
+        const pids = out.trim().split('\n').map(l => l.trim()).filter(Boolean);
+        if (pids.length > 0) {
+          const pid = pids[pids.length - 1];
+          fs.writeFileSync(pidFile, pid, 'utf8');
+          console.log(`[supervisor] Daemonized server (PID ${pid}), logs at ${logFile}`);
+        } else {
+          console.log(`[supervisor] Daemonized server, logs at ${logFile}`);
+        }
+      } catch (_) {
+        console.log(`[supervisor] Daemonized server, logs at ${logFile}`);
+      }
+      process.exit(0);
+    }, 2000);
+  } else {
+    // Unix: standard detach with file descriptors
+    const out = fs.openSync(logFile, 'a');
+    const err = fs.openSync(logFile, 'a');
+    const child = spawn(nodeExe, ['--max-old-space-size=1024', __filename, ...childArgs], {
+      stdio: ['ignore', out, err],
+      detached: true,
+      env: { ...process.env },
+    });
+    fs.writeFileSync(pidFile, String(child.pid), 'utf8');
+    child.unref();
+    console.log(`[supervisor] Daemonized server (PID ${child.pid}), logs at ${logFile}`);
+    process.exit(0);
+  }
+  return; // Guard: don't fall through to supervisor logic while waiting
 }
 
 // ─── EPIPE Protection ────────────────────────────────────

package/src/web/pty-manager.js

@@ -111,7 +111,7 @@ class PtySession {
 // Maximum number of live PTY sessions. Each one is a ConPTY handle + a Claude
 // process (100-200MB each). Beyond this limit, new spawns are rejected to
 // prevent the OS from OOM-killing the server process tree.
-const MAX_PTY_SESSIONS = 10;
+const MAX_PTY_SESSIONS = 5;
 
 class PtySessionManager {
   constructor() {