myrlin-workbook 0.9.18 → 0.9.20

package/logs/server.pid

@@ -1 +1 @@
-21156
+45572

package/package.json

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

package/src/web/public/app.js

@@ -9110,11 +9110,33 @@ class CWMApp {
       });
     };
 
-    // Auto-close pane on fatal connection error (max retries exhausted or server error).
-    // Prevents dead panes from occupying grid space in the terminal layout.
-    tp.onFatalError = () => {
+    // On fatal connection error, show disconnected state but PRESERVE pane info.
+    // The session ID, name, and spawnOpts stay in the layout so the user can
+    // reconnect by clicking, and layout saves don't lose the session mapping.
+    tp.onFatalError = (failedSessionId) => {
       const idx = this.terminalPanes.indexOf(tp);
-      if (idx !== -1) this.closeTerminalPane(idx);
+      if (idx === -1) return;
+      // Stash session info before disposing so we can offer reconnect
+      const sid = tp.sessionId;
+      const sName = tp.sessionName;
+      const opts = { ...(tp.spawnOpts || {}) };
+      tp.dispose();
+      // Replace the TerminalPane with a lightweight placeholder that preserves
+      // the session mapping for layout saves (saveCurrentGroupPanes reads these).
+      this.terminalPanes[idx] = { sessionId: sid, sessionName: sName, spawnOpts: opts, _disconnected: true };
+      const paneEl = document.getElementById(`term-pane-${idx}`);
+      if (!paneEl) return;
+      const container = document.getElementById(`term-container-${idx}`);
+      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;">&#x26A0;</span>
+          <span>${this.escapeHtml(sName || sid)}</span>
+          <span style="font-size:11px;opacity:0.7;">Disconnected. Click to reconnect.</span>
+        </div>`;
+        container.querySelector('.reconnect-prompt').addEventListener('click', () => {
+          this.openTerminalInPane(idx, sid, sName, opts);
+        });
+      }
     };
 
     // Enable auto-trust if the setting is on
@@ -12083,12 +12105,16 @@ class CWMApp {
 
     group.panes = [];
     for (let i = 0; i < CWMApp.MAX_PANES; i++) {
-      if (this.terminalPanes[i]) {
+      const tp = this.terminalPanes[i];
+      // Save both live TerminalPanes and disconnected placeholders.
+      // Disconnected placeholders have { sessionId, sessionName, spawnOpts, _disconnected: true }
+      // and MUST be preserved so layout restores don't lose session mappings.
+      if (tp && tp.sessionId) {
         group.panes.push({
           slot: i,
-          sessionId: this.terminalPanes[i].sessionId,
-          sessionName: this.terminalPanes[i].sessionName,
-          spawnOpts: this.terminalPanes[i].spawnOpts || {},
+          sessionId: tp.sessionId,
+          sessionName: tp.sessionName,
+          spawnOpts: tp.spawnOpts || {},
         });
       }
     }