myrlin-workbook 0.9.28 → 0.9.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "myrlin-workbook",
-  "version": "0.9.28",
+  "version": "0.9.30",
   "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/scripts/postinstall.js

@@ -1,37 +1,77 @@
-#!/usr/bin/env node
-/**
- * Postinstall script: fix node-pty spawn-helper permissions on macOS/Linux.
- * The prebuilt spawn-helper binary ships without execute permission (644),
- * causing posix_spawnp failures. This sets it to 755.
- * See: https://github.com/therealarthur/myrlin-workbook/issues/4
- */
-'use strict';
-
-if (process.platform === 'win32') {
-  process.exit(0);
-}
-
-const { execSync } = require('child_process');
-const path = require('path');
-const fs = require('fs');
-
-const prebuildsDir = path.join(__dirname, '..', 'node_modules', 'node-pty', 'prebuilds');
-
-if (!fs.existsSync(prebuildsDir)) {
-  process.exit(0);
-}
-
-try {
-  // Find all spawn-helper binaries across platform dirs and chmod +x them
-  const dirs = fs.readdirSync(prebuildsDir, { withFileTypes: true })
-    .filter(d => d.isDirectory());
-
-  for (const dir of dirs) {
-    const helper = path.join(prebuildsDir, dir.name, 'spawn-helper');
-    if (fs.existsSync(helper)) {
-      execSync(`chmod +x "${helper}"`, { stdio: 'ignore' });
-    }
-  }
-} catch (_) {
-  // Non-fatal: if chmod fails, user can still manually fix
-}
+#!/usr/bin/env node
+/**
+ * Postinstall script: fix node-pty spawn-helper permissions on macOS/Linux.
+ *
+ * The prebuilt spawn-helper binary in node-pty's package ships without
+ * execute permission (mode 644 instead of 755), causing posix_spawnp to
+ * fail with "posix_spawnp failed" on first PTY spawn.
+ *
+ * This is a known node-pty packaging issue. We work around it here by
+ * locating node-pty (wherever npm/npx hoisted it), finding all prebuilt
+ * spawn-helper binaries across platform dirs, and setting them to 755.
+ *
+ * See: https://github.com/therealarthur/myrlin-workbook/issues/4
+ */
+'use strict';
+
+if (process.platform === 'win32') {
+  process.exit(0);
+}
+
+const path = require('path');
+const fs = require('fs');
+
+/**
+ * Locate node-pty's package directory using require.resolve.
+ * Works regardless of where npm/npx/yarn placed it (hoisted, nested,
+ * pnp-virtual, etc.). Returns null if node-pty isn't installed.
+ */
+function findNodePtyDir() {
+  try {
+    const ptyMain = require.resolve('node-pty', { paths: [path.join(__dirname, '..')] });
+    let dir = path.dirname(ptyMain);
+    // Walk up until we find package.json with "name": "node-pty"
+    for (let i = 0; i < 8; i++) {
+      const pkg = path.join(dir, 'package.json');
+      if (fs.existsSync(pkg)) {
+        try {
+          const json = JSON.parse(fs.readFileSync(pkg, 'utf8'));
+          if (json && json.name === 'node-pty') return dir;
+        } catch (_) {}
+      }
+      const parent = path.dirname(dir);
+      if (parent === dir) break;
+      dir = parent;
+    }
+  } catch (_) {}
+  return null;
+}
+
+const ptyDir = findNodePtyDir();
+if (!ptyDir) {
+  // node-pty not installed yet (rare during postinstall); skip silently.
+  process.exit(0);
+}
+
+const prebuildsDir = path.join(ptyDir, 'prebuilds');
+if (!fs.existsSync(prebuildsDir)) {
+  process.exit(0);
+}
+
+try {
+  const platforms = fs.readdirSync(prebuildsDir, { withFileTypes: true })
+    .filter(d => d.isDirectory());
+
+  for (const platform of platforms) {
+    const helper = path.join(prebuildsDir, platform.name, 'spawn-helper');
+    if (fs.existsSync(helper)) {
+      try {
+        fs.chmodSync(helper, 0o755);
+      } catch (_) {
+        // Non-fatal: runtime fix will catch this on first PTY spawn
+      }
+    }
+  }
+} catch (_) {
+  // Non-fatal
+}

package/src/web/pty-manager.js

@@ -11,10 +11,52 @@
  *   - Scrollback is capped at ~100KB total characters
  */
 
-const pty = require('node-pty');
 const fs = require('fs');
 const os = require('os');
 const path = require('path');
+
+// Ensure node-pty's prebuilt spawn-helper is executable BEFORE requiring node-pty.
+// node-pty's prebuild ships with mode 644 instead of 755, causing posix_spawnp
+// to fail on macOS/Linux. The postinstall script handles this in normal installs
+// but doesn't run with --ignore-scripts or in some npx caches. This runtime
+// fallback covers those cases. See: https://github.com/therealarthur/myrlin-workbook/issues/4
+if (process.platform !== 'win32') {
+  try {
+    const ptyMain = require.resolve('node-pty');
+    let dir = path.dirname(ptyMain);
+    for (let i = 0; i < 8; i++) {
+      const pkg = path.join(dir, 'package.json');
+      if (fs.existsSync(pkg)) {
+        try {
+          const json = JSON.parse(fs.readFileSync(pkg, 'utf8'));
+          if (json && json.name === 'node-pty') break;
+        } catch (_) {}
+      }
+      const parent = path.dirname(dir);
+      if (parent === dir) { dir = null; break; }
+      dir = parent;
+    }
+    if (dir) {
+      const prebuildsDir = path.join(dir, 'prebuilds');
+      if (fs.existsSync(prebuildsDir)) {
+        for (const p of fs.readdirSync(prebuildsDir)) {
+          const helper = path.join(prebuildsDir, p, 'spawn-helper');
+          if (fs.existsSync(helper)) {
+            try {
+              const stat = fs.statSync(helper);
+              // Only chmod if not already executable, avoids unnecessary syscalls
+              if ((stat.mode & 0o111) === 0) fs.chmodSync(helper, 0o755);
+            } catch (_) {}
+          }
+        }
+      }
+    }
+  } catch (_) {
+    // node-pty not yet resolvable; require() below will throw with a clearer error
+  }
+}
+
+const pty = require('node-pty');
 const { getStore } = require('../state/store');
 
 /**

package/src/web/public/app.js

@@ -3109,10 +3109,16 @@ class CWMApp {
           this.showToast('Select or create a project first', 'warning');
           return;
         }
-        // Use project folder name as friendly default name instead of raw UUID
-        const projectName = projectPath ? projectPath.split('\\').pop() || projectPath.split('/').pop() || sessionName : sessionName;
-        const shortId = sessionName.length > 8 ? sessionName.substring(0, 8) : sessionName;
-        const friendlyName = projectName + ' (' + shortId + ')';
+        // Prefer custom title from Claude session, fall back to folder name + short UUID
+        const customTitle = this.getProjectSessionTitle(sessionName);
+        let friendlyName;
+        if (customTitle) {
+          friendlyName = customTitle;
+        } else {
+          const projectName = projectPath ? projectPath.split('\\').pop() || projectPath.split('/').pop() || sessionName : sessionName;
+          const shortId = sessionName.length > 8 ? sessionName.substring(0, 8) : sessionName;
+          friendlyName = projectName + ' (' + shortId + ')';
+        }
         this.api('POST', '/api/sessions', {
           name: friendlyName,
           workspaceId: this.state.activeWorkspace.id,
@@ -7230,8 +7236,10 @@ class CWMApp {
           !p.dirExists ? '<span class="discover-badge discover-badge-missing">missing</span>' : '',
         ].filter(Boolean).join(' ');
 
-        const latestSessionId = p.sessions && p.sessions.length > 0 ? p.sessions[0].name : '';
-        return `<div class="discover-row" data-path="${this.escapeHtml(p.realPath)}" data-name="${this.escapeHtml(name)}" data-session-id="${this.escapeHtml(latestSessionId)}">
+        const latestSession = p.sessions && p.sessions.length > 0 ? p.sessions[0] : null;
+        const latestSessionId = latestSession ? latestSession.name : '';
+        const latestSessionTitle = latestSession ? (latestSession.title || '') : '';
+        return `<div class="discover-row" data-path="${this.escapeHtml(p.realPath)}" data-name="${this.escapeHtml(name)}" data-session-id="${this.escapeHtml(latestSessionId)}" data-session-title="${this.escapeHtml(latestSessionTitle)}">
           <div class="discover-check">
             <input type="checkbox" class="discover-cb" ${p.dirExists ? 'checked' : ''} ${!p.dirExists ? 'disabled' : ''}>
           </div>
@@ -7293,7 +7301,7 @@ class CWMApp {
         const cb = row.querySelector('.discover-cb');
         if (cb && cb.checked) {
           selected.push({
-            name: row.dataset.name,
+            name: row.dataset.sessionTitle || row.dataset.name,
             path: row.dataset.path,
             sessionId: row.dataset.sessionId || '',
           });
@@ -9546,9 +9554,9 @@ class CWMApp {
         const path = p.realPath || '';
         // Match against project name, encoded name, or path
         if (name.toLowerCase().includes(query) || encoded.toLowerCase().includes(query) || path.toLowerCase().includes(query)) return true;
-        // Match against any session ID/name within this project
+        // Match against any session ID, title, or Claude custom-title within this project
         const allSessions = p.sessions || [];
-        return allSessions.some(s => (s.name || '').toLowerCase().includes(query));
+        return allSessions.some(s => (s.name || '').toLowerCase().includes(query) || (s.title || '').toLowerCase().includes(query));
       });
     }
 
@@ -9579,8 +9587,9 @@ class CWMApp {
         if (!projectMatches) {
           sessions = sessions.filter(s => {
             const sName = (s.name || '').toLowerCase();
+            const sClaudeTitle = (s.title || '').toLowerCase();
             const sTitle = (this.getProjectSessionTitle(s.name) || '').toLowerCase();
-            return sName.includes(query) || sTitle.includes(query);
+            return sName.includes(query) || sClaudeTitle.includes(query) || sTitle.includes(query);
           });
         }
       }
@@ -9588,13 +9597,19 @@ class CWMApp {
       // Build session sub-items
       const sessionItems = sessions.map(s => {
         const sessName = s.name || 'unnamed';
+        const claudeTitle = s.title || null;
+        if (claudeTitle && !this.getProjectSessionTitle(sessName)) {
+          const titles = JSON.parse(localStorage.getItem('cwm_projectSessionTitles') || '{}');
+          titles[sessName] = claudeTitle;
+          localStorage.setItem('cwm_projectSessionTitles', JSON.stringify(titles));
+        }
         const storedTitle = this.getProjectSessionTitle(sessName);
-        const displayName = storedTitle || (sessName.length > 24 ? sessName.substring(0, 24) + '...' : sessName);
+        const displayName = storedTitle || claudeTitle || (sessName.length > 24 ? sessName.substring(0, 24) + '...' : sessName);
         const sessSize = s.size ? this.formatSize(s.size) : '';
         const sessTime = s.modified ? this.relativeTime(s.modified) : '';
-        // Tooltip: show title + session ID so user sees both on hover
-        const tooltip = storedTitle
-          ? `${storedTitle}\n\nSession: ${sessName}`
+        const effectiveTitle = storedTitle || claudeTitle;
+        const tooltip = effectiveTitle
+          ? `${effectiveTitle}\n\nSession: ${sessName}`
           : sessName;
         return `<div class="project-session-item" draggable="true" data-session-name="${this.escapeHtml(sessName)}" data-project-path="${this.escapeHtml(p.realPath || '')}" data-project-encoded="${this.escapeHtml(encoded)}" title="${this.escapeHtml(tooltip)}">
           <span class="project-session-name">${this.escapeHtml(displayName)}</span>

package/src/web/server.js

@@ -1646,6 +1646,13 @@ app.get('/api/discover', requireAuth, (req, res) => {
         // skip if can't read
       }
 
+      // Extract the last custom-title from each JSONL file
+      const sessionTitles = {};
+      for (const sf of sessionFiles) {
+        const title = extractCustomTitle(path.join(projectDir, sf.name + '.jsonl'));
+        if (title) sessionTitles[sf.name] = title;
+      }
+
       // Check for CLAUDE.md
       let hasClaudeMd = false;
       try {
@@ -1672,7 +1679,7 @@ app.get('/api/discover', requireAuth, (req, res) => {
         sessionCount: sessionFiles.length,
         totalSize,
         lastActive: sessionFiles.length > 0 ? sessionFiles[0].modified : null,
-        sessions: sessionFiles.map(s => ({ name: s.name, modified: s.modified, size: s.size })),
+        sessions: sessionFiles.map(s => ({ name: s.name, modified: s.modified, size: s.size, title: sessionTitles[s.name] || null })),
       });
     }
 
@@ -1801,48 +1808,34 @@ function greedyFsWalk(root, tokens) {
  */
 function getOriginalPathFromJsonl(projectDir) {
   try {
-    const files = fs.readdirSync(projectDir);
-    // Find the first .jsonl file (skip subdirectories)
-    const jsonlFile = files.find(f => {
+    const jsonlFiles = fs.readdirSync(projectDir).filter(f => {
       if (!f.endsWith('.jsonl')) return false;
-      try {
-        return !fs.statSync(path.join(projectDir, f)).isDirectory();
-      } catch (_) {
-        return false;
-      }
+      try { return !fs.statSync(path.join(projectDir, f)).isDirectory(); } catch (_) { return false; }
     });
-    if (!jsonlFile) return null;
-
-    const jsonlPath = path.join(projectDir, jsonlFile);
 
-    // Only read first 4KB to find cwd field (avoids loading large files)
-    const fd = fs.openSync(jsonlPath, 'r');
-    const buffer = Buffer.alloc(4096);
-    let content;
-    try {
-      const bytesRead = fs.readSync(fd, buffer, 0, 4096, 0);
-      content = buffer.toString('utf-8', 0, bytesRead);
-    } finally {
-      fs.closeSync(fd);
-    }
-
-    // Parse first 3 lines to find cwd field
-    const lines = content.split('\n').slice(0, 3);
-    for (const line of lines) {
-      if (!line.trim()) continue;
+    // Try each file until one yields a cwd — stub sessions may lack it
+    for (const jsonlFile of jsonlFiles) {
       try {
-        const record = JSON.parse(line);
-        if (record.cwd && typeof record.cwd === 'string') {
-          return record.cwd;
+        const fd = fs.openSync(path.join(projectDir, jsonlFile), 'r');
+        let content;
+        try {
+          const buffer = Buffer.alloc(16384);
+          const bytesRead = fs.readSync(fd, buffer, 0, 16384, 0);
+          content = buffer.toString('utf-8', 0, bytesRead);
+        } finally {
+          fs.closeSync(fd);
         }
-      } catch (_) {
-        // Skip invalid JSON lines
-        continue;
-      }
+
+        for (const line of content.split('\n')) {
+          if (!line.includes('"cwd"')) continue;
+          try {
+            const record = JSON.parse(line);
+            if (record.cwd && typeof record.cwd === 'string') return record.cwd;
+          } catch (_) { continue; }
+        }
+      } catch (_) { continue; }
     }
-  } catch (_) {
-    // Silently fail and return null
-  }
+  } catch (_) {}
   return null;
 }
 
@@ -2524,7 +2517,7 @@ app.post('/api/search-conversations', requireAuth, async (req, res) => {
  */
 app.get('/api/keys/anthropic', requireAuth, (req, res) => {
   const store = getStore();
-  const key = (store.getState().settings || {}).anthropicApiKey || '';
+  const key = (store.state.settings || {}).anthropicApiKey || '';
   if (!key) return res.json({ configured: false, masked: null });
   const masked = '...' + key.slice(-8);
   return res.json({ configured: true, masked });
@@ -2560,7 +2553,7 @@ app.post('/api/ai/punctuate', requireAuth, async (req, res) => {
   }
 
   const store = getStore();
-  const apiKey = (store.getState().settings || {}).anthropicApiKey || '';
+  const apiKey = (store.state.settings || {}).anthropicApiKey || '';
   if (!apiKey) {
     return res.status(400).json({ error: 'No Anthropic API key configured' });
   }
@@ -2612,7 +2605,7 @@ app.post('/api/ai/find-session', requireAuth, async (req, res) => {
   }
 
   const store = getStore();
-  const apiKey = (store.getState().settings || {}).anthropicApiKey || '';
+  const apiKey = (store.state.settings || {}).anthropicApiKey || '';
 
   // Gather all metadata
   const workspaces = store.getAllWorkspacesList();
@@ -2631,6 +2624,7 @@ app.post('/api/ai/find-session', requireAuth, async (req, res) => {
         const name = getProjectDisplayName(d.name, realPath);
         let sessionCount = 0;
         let lastActive = null;
+        const sessionTitles = [];
         try {
           const files = fs.readdirSync(projectDir).filter(f => f.endsWith('.jsonl'));
           sessionCount = files.length;
@@ -2639,9 +2633,11 @@ app.post('/api/ai/find-session', requireAuth, async (req, res) => {
               const stat = fs.statSync(path.join(projectDir, f));
               if (!lastActive || stat.mtime > new Date(lastActive)) lastActive = stat.mtime;
             } catch (_) {}
+            const title = extractCustomTitle(path.join(projectDir, f));
+            if (title) sessionTitles.push(title);
           }
         } catch (_) {}
-        return { encodedName: d.name, name, path: realPath, sessionCount, lastActive };
+        return { encodedName: d.name, name, path: realPath, sessionCount, lastActive, sessionTitles };
       });
     } catch (_) {}
   }
@@ -2658,7 +2654,8 @@ app.post('/api/ai/find-session', requireAuth, async (req, res) => {
     })),
     discoveredProjects: discoveredProjects.map(p => ({
       encodedName: p.encodedName, name: p.name, path: p.path,
-      sessionCount: p.sessionCount, lastActive: p.lastActive
+      sessionCount: p.sessionCount, lastActive: p.lastActive,
+      sessionTitles: p.sessionTitles
     }))
   };
 
@@ -2722,9 +2719,9 @@ app.post('/api/ai/find-session', requireAuth, async (req, res) => {
       if (score > 0.2) scored.push({ type: 'session', id: s.id, confidence: score, summary: 'Matched by name, topic, or path' });
     }
     for (const p of discoveredProjects) {
-      const text = `${p.name} ${p.path}`.toLowerCase();
+      const text = `${p.name} ${p.path} ${(p.sessionTitles || []).join(' ')}`.toLowerCase();
       const score = terms.filter(t => text.includes(t)).length / terms.length;
-      if (score > 0.2) scored.push({ type: 'project', id: p.encodedName, confidence: score, summary: 'Matched by project name or path' });
+      if (score > 0.2) scored.push({ type: 'project', id: p.encodedName, confidence: score, summary: 'Matched by project name, path, or session title' });
     }
 
     scored.sort((a, b) => b.confidence - a.confidence);
@@ -7243,6 +7240,30 @@ function getSearchableFiles() {
  * @param {string} sessionId - Fallback UUID
  * @returns {string} A human-readable session name
  */
+function extractCustomTitle(jsonlPath) {
+  try {
+    const fd = fs.openSync(jsonlPath, 'r');
+    try {
+      const size = fs.fstatSync(fd).size;
+      const tailSize = Math.min(131072, size);
+      const buf = Buffer.alloc(tailSize);
+      fs.readSync(fd, buf, 0, tailSize, size - tailSize);
+      const lines = buf.toString('utf8').split('\n');
+      for (let i = lines.length - 1; i >= 0; i--) {
+        if (lines[i].includes('"custom-title"')) {
+          try {
+            const obj = JSON.parse(lines[i]);
+            if (obj.customTitle) return obj.customTitle;
+          } catch (_) {}
+        }
+      }
+    } finally {
+      fs.closeSync(fd);
+    }
+  } catch (_) {}
+  return null;
+}
+
 function extractSessionName(filePath, sessionId) {
   try {
     // Read just the first 10KB to find the first meaningful message
@@ -8054,4 +8075,5 @@ module.exports = {
   startServer,
   getPtyManager,
   structuredError,
+  extractCustomTitle,
 };