myrlin-workbook 0.9.20 → 0.9.22

package/logs/server.pid

@@ -1 +1 @@
-45572
+67796

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "myrlin-workbook",
-  "version": "0.9.20",
+  "version": "0.9.22",
   "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

@@ -189,15 +189,8 @@ if (!process.env.CWM_NO_OPEN) {
   }
 }
 
-// ─── Memory Watchdog ──────────────────────────────────────
-// Monitor RSS and kill idle PTY sessions when memory exceeds threshold.
-// 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
+// ─── RSS Logger ───────────────────────────────────────────
+// Periodic RSS logging so we can trace memory in server.log
 const _rssLogger = setInterval(() => {
   const mem = process.memoryUsage();
   const ptyManager = getPtyManager();
@@ -208,45 +201,6 @@ const _rssLogger = setInterval(() => {
 }, 60000);
 _rssLogger.unref();
 
-const _memoryWatchdog = setInterval(() => {
-  const rssMB = Math.round(process.memoryUsage().rss / 1024 / 1024);
-  if (rssMB < MEMORY_WARN_MB) return;
-
-  const { logWarning } = require('./crash-logger');
-  const ptyManager = getPtyManager();
-  if (!ptyManager) return;
-
-  const sessions = ptyManager.listSessions();
-  const isCritical = rssMB >= MEMORY_CRITICAL_MB;
-  const label = isCritical ? 'CRITICAL' : 'WARNING';
-  try { console.log(`[Memory ${label}] RSS=${rssMB}MB, ${sessions.length} PTY sessions`); } catch (_) {}
-  logWarning('server', `Memory ${label}: RSS=${rssMB}MB, ${sessions.length} PTY sessions`);
-
-  // Kill PTY sessions with zero connected WebSocket clients first
-  let killed = 0;
-  for (const s of sessions) {
-    if (s.clientCount === 0) {
-      ptyManager.killSession(s.sessionId);
-      killed++;
-    }
-  }
-
-  // In critical mode, also kill sessions that have been alive longest
-  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++;
-  }
-
-  if (killed > 0) {
-    try { console.log(`[Memory] Killed ${killed} PTY session(s) to free memory`); } catch (_) {}
-  }
-
-  // Force GC if available (node --expose-gc)
-  if (global.gc) global.gc();
-}, MEMORY_CHECK_INTERVAL);
-_memoryWatchdog.unref();
-
 // ─── Graceful Shutdown ─────────────────────────────────────
 
 process.on('SIGINT', () => {

package/src/supervisor.js

@@ -46,7 +46,7 @@ if (process.argv.includes('--daemon')) {
     // 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`;
+    const cmd = `cmd.exe /c start /b "" "${nodeExe}" --max-old-space-size=4096 ${scriptArgs} >> "${logFile}" 2>&1`;
     execSync(cmd, { stdio: 'ignore', windowsHide: true });
 
     // The PID isn't directly available from start /b. Write a marker so we
@@ -73,7 +73,7 @@ if (process.argv.includes('--daemon')) {
     // 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], {
+    const child = spawn(nodeExe, ['--max-old-space-size=4096', __filename, ...childArgs], {
       stdio: ['ignore', out, err],
       detached: true,
       env: { ...process.env },
@@ -115,7 +115,7 @@ function startChild() {
   lastStartTime = Date.now();
   console.log(`[supervisor] Starting GUI server (attempt ${consecutiveRestarts + 1})...`);
 
-  child = spawn(process.execPath, ['--max-old-space-size=1024', guiScript, ...guiArgs], {
+  child = spawn(process.execPath, ['--max-old-space-size=4096', guiScript, ...guiArgs], {
     stdio: 'inherit',
     env: { ...process.env, CWM_NO_OPEN: consecutiveRestarts > 0 ? '1' : '' },
   });

package/src/web/pty-manager.js

@@ -108,11 +108,6 @@ 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 = 5;
-
 class PtySessionManager {
   constructor() {
     this.sessions = new Map(); // sessionId -> PtySession
@@ -137,13 +132,6 @@ class PtySessionManager {
       return existing;
     }
 
-    // Enforce max live sessions to prevent OOM from too many ConPTY handles
-    const aliveCount = [...this.sessions.values()].filter(s => s.alive).length;
-    if (aliveCount >= MAX_PTY_SESSIONS) {
-      console.log(`[PTY] Rejected spawn for ${sessionId}: ${aliveCount} live sessions (max ${MAX_PTY_SESSIONS})`);
-      return null;
-    }
-
     // ── Defense-in-depth: validate all user-controlled inputs ──
     // Primary validation happens at the API/WebSocket boundary (server.js, pty-server.js).
     // This is a secondary gate to catch any bypass or future code path that skips validation.

package/src/web/public/app.js

@@ -9058,6 +9058,36 @@ class CWMApp {
      TERMINAL GRID VIEW
      ═══════════════════════════════════════════════════════════ */
 
+  /**
+   * Show a disconnected placeholder in a terminal pane slot.
+   * Preserves session info in terminalPanes[] for layout saves.
+   * Click reconnects via openTerminalInPane().
+   */
+  _showDisconnectedPlaceholder(slotIdx, sessionId, sessionName, spawnOpts) {
+    // Store placeholder so saveCurrentGroupPanes preserves the mapping
+    this.terminalPanes[slotIdx] = { sessionId, sessionName, spawnOpts: spawnOpts || {}, _disconnected: true };
+    const paneEl = document.getElementById(`term-pane-${slotIdx}`);
+    if (!paneEl) return;
+    paneEl.hidden = false;
+    paneEl.classList.remove('terminal-pane-empty');
+    const titleEl = paneEl.querySelector('.terminal-pane-title');
+    if (titleEl) titleEl.textContent = sessionName || sessionId;
+    const closeBtn = paneEl.querySelector('.terminal-pane-close');
+    if (closeBtn) closeBtn.hidden = false;
+    const container = document.getElementById(`term-container-${slotIdx}`);
+    if (container) {
+      container.innerHTML = `<div style="display:flex;flex-direction:column;align-items:center;justify-content:center;height:100%;gap:8px;color:var(--overlay0);font-size:13px;cursor:pointer;" class="reconnect-prompt">
+        <span style="font-size:20px;">&#x1F50C;</span>
+        <span>${this.escapeHtml(sessionName || sessionId)}</span>
+        <span style="font-size:11px;opacity:0.7;">Click to connect</span>
+      </div>`;
+      container.querySelector('.reconnect-prompt').addEventListener('click', () => {
+        this.openTerminalInPane(slotIdx, sessionId, sessionName, spawnOpts);
+      });
+    }
+    this.updateTerminalGridLayout();
+  }
+
   openTerminalInPane(slotIdx, sessionId, sessionName, spawnOpts) {
     // Check localStorage for a previously saved name for this session
     const savedTitle = this.getProjectSessionTitle(sessionId);
@@ -12048,10 +12078,12 @@ class CWMApp {
             const uploadBtn = paneEl.querySelector('.terminal-pane-upload');
             if (uploadBtn) uploadBtn.hidden = false;
           }
-          // Reattach xterm DOM
+          // Reattach xterm DOM, or re-render placeholder for disconnected panes
           if (cached.domFragments[i]) {
             const termContainer = document.getElementById(`term-container-${i}`);
             if (termContainer) termContainer.appendChild(cached.domFragments[i]);
+          } else if (cached.panes[i]._disconnected) {
+            this._showDisconnectedPlaceholder(i, cached.panes[i].sessionId, cached.panes[i].sessionName, cached.panes[i].spawnOpts);
           }
         }
       }
@@ -12071,12 +12103,15 @@ class CWMApp {
         }
       });
     } else {
-      // No cache, create fresh connections (first time opening this group)
+      // No cache: show disconnected placeholders instead of spawning all PTYs.
+      // Each PTY spawns a Claude process (~150MB), so eagerly connecting all
+      // panes across all tab groups would consume gigabytes of system memory.
+      // Users click a pane to connect on demand.
       const group = this._tabGroups.find(g => g.id === groupId);
       if (group && group.panes) {
         group.panes.forEach(p => {
           if (p.sessionId && !this.terminalPanes[p.slot]) {
-            this.openTerminalInPane(p.slot, p.sessionId, p.sessionName || 'Terminal', p.spawnOpts || {});
+            this._showDisconnectedPlaceholder(p.slot, p.sessionId, p.sessionName || 'Terminal', p.spawnOpts || {});
           }
         });
       }