agentgui 1.0.671 → 1.0.673

package/lib/claude-runner.js

@@ -625,7 +625,7 @@ registry.register({
   closeStdin: true, // close stdin so claude doesn't wait for input in --print mode
   useJsonRpcStdin: false,
   supportedFeatures: ['streaming', 'resume', 'system-prompt', 'permissions-skip', 'steering'],
-  spawnEnv: { MAX_THINKING_TOKENS: '0', AGENTGUI_SUBPROCESS: '1', CLAUDECODE: undefined },
+  spawnEnv: { MAX_THINKING_TOKENS: '0', AGENTGUI_SUBPROCESS: '1' },
 
   buildArgs(prompt, config) {
     const {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.671",
+  "version": "1.0.673",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "server.js",

package/server.js

@@ -3237,7 +3237,7 @@ function serveFile(filePath, res, req) {
         'Content-Type': contentType,
         'Content-Length': stats.size,
         'ETag': etag,
-        'Cache-Control': ['.js', '.css'].includes(ext) ? 'no-cache' : 'public, max-age=3600, must-revalidate'
+        'Cache-Control': ['.js', '.css'].includes(ext) ? 'public, max-age=0, must-revalidate' : 'public, max-age=3600, must-revalidate'
       };
       if (acceptsEncoding(req, 'br') && stats.size > 860) {
         const stream = fs.createReadStream(filePath);
@@ -4151,7 +4151,8 @@ wsRouter.onLegacy((data, ws) => {
       try { ws.terminalProc.kill(); } catch(e) {}
     }
     try {
-      const pty = require('node-pty');
+      const _req = createRequire(import.meta.url);
+      const pty = _req('node-pty');
       const shell = process.env.SHELL || '/bin/bash';
       const cwd = data.cwd || process.env.STARTUP_CWD || process.env.HOME || '/';
       const proc = pty.spawn(shell, [], {
@@ -4178,7 +4179,6 @@ wsRouter.onLegacy((data, ws) => {
       sendWs(ws, ({ type: 'terminal_started', timestamp: Date.now() }));
     } catch (e) {
       console.error('[TERMINAL] Failed to spawn PTY, falling back to pipes:', e.message);
-      const { spawn } = require('child_process');
       const shell = process.env.SHELL || '/bin/bash';
       const cwd = data.cwd || process.env.STARTUP_CWD || process.env.HOME || '/';
       const proc = spawn(shell, ['-i'], { cwd, env: { ...process.env, TERM: 'xterm-256color', COLORTERM: 'truecolor' }, stdio: ['pipe', 'pipe', 'pipe'] });

package/static/js/client.js

@@ -1156,6 +1156,8 @@ class AgentGUIClient {
       if (indicator) {
         indicator.innerHTML = `<span style="color:var(--color-error);">Error: ${this.escapeHtml(data.error || 'Unknown error')}</span>`;
       }
+      // Remove all thinking blocks on error
+      streamingEl.querySelectorAll('.block-thinking').forEach(block => block.remove());
     } else {
       const outputEl3 = document.getElementById('output');
       const messagesEl3 = outputEl3 && outputEl3.querySelector('.conversation-messages');
@@ -1219,6 +1221,9 @@ class AgentGUIClient {
       const prevTextEl = streamingEl.querySelector('.streaming-text-current');
       if (prevTextEl) prevTextEl.classList.remove('streaming-text-current');
 
+      // Remove all thinking blocks (block-thinking elements)
+      streamingEl.querySelectorAll('.block-thinking').forEach(block => block.remove());
+
       const ts = document.createElement('div');
       ts.className = 'message-timestamp';
       ts.textContent = new Date().toLocaleString();

package/static/js/streaming-renderer.js

@@ -1232,7 +1232,9 @@ class StreamingRenderer {
     const details = document.createElement('details');
     details.className = 'folded-tool' + (isError ? ' folded-tool-error' : ' folded-tool-success');
     details.dataset.eventType = 'tool_result';
-    if (!isError) details.open = true;
+    // Only open by default if the content is an image
+    const isImageContent = contentStr.includes('data:image/') || (block.content && block.content.type === 'base64');
+    if (!isError && isImageContent) details.open = true;
     if (block.tool_use_id) details.dataset.toolUseId = block.tool_use_id;
     details.classList.add(this._getBlockTypeClass('tool_result'));
 

package/static/js/terminal.js

@@ -1,39 +1,31 @@
 (function() {
-  var ws = null;
   var term = null;
   var fitAddon = null;
   var termActive = false;
   var BASE = window.__BASE_URL || '';
-
-  function getWsUrl() {
-    var proto = location.protocol === 'https:' ? 'wss:' : 'ws:';
-    return proto + '//' + location.host + BASE + '/sync';
-  }
+  var _wsListener = null;
 
   function getCwd() {
     try {
-      // Try conversation manager first
       if (window.conversationManager && window.conversationManager.activeId) {
         var mgr = window.conversationManager;
-        var id = mgr.activeId;
-        if (mgr.conversations) {
-          var conv = mgr.conversations.find(function(c) { return c.id === id; });
-          if (conv && conv.workingDirectory) return conv.workingDirectory;
-        }
+        var conv = mgr.conversations && mgr.conversations.find(function(c) { return c.id === mgr.activeId; });
+        if (conv && conv.workingDirectory) return conv.workingDirectory;
       }
-      // Fallback to global currentConversation
-      if (window.currentConversation) {
-        var convId = window.currentConversation;
-        if (window.conversationManager && window.conversationManager.conversations) {
-          var convList = window.conversationManager.conversations;
-          var match = convList.find(function(c) { return c.id === convId; });
-          if (match && match.workingDirectory) return match.workingDirectory;
-        }
+      if (window.currentConversation && window.conversationManager && window.conversationManager.conversations) {
+        var match = window.conversationManager.conversations.find(function(c) { return c.id === window.currentConversation; });
+        if (match && match.workingDirectory) return match.workingDirectory;
       }
     } catch (_) {}
     return undefined;
   }
 
+  function wsSend(obj) {
+    if (window.wsManager && window.wsManager.send) {
+      window.wsManager.send(obj);
+    }
+  }
+
   function ensureTerm() {
     var output = document.getElementById('terminalOutput');
     if (!output) return false;
@@ -60,16 +52,12 @@
     fitAddon.fit();
 
     term.onData(function(data) {
-      if (ws && ws.readyState === WebSocket.OPEN) {
-        var encoded = btoa(unescape(encodeURIComponent(data)));
-        ws.send(JSON.stringify({ type: 'terminal_input', data: encoded }));
-      }
+      var encoded = btoa(unescape(encodeURIComponent(data)));
+      wsSend({ type: 'terminal_input', data: encoded });
     });
 
     term.onResize(function(size) {
-      if (ws && ws.readyState === WebSocket.OPEN) {
-        ws.send(JSON.stringify({ type: 'terminal_resize', cols: size.cols, rows: size.rows }));
-      }
+      wsSend({ type: 'terminal_resize', cols: size.cols, rows: size.rows });
     });
 
     var resizeTimer;
@@ -78,57 +66,42 @@
         try { fitAddon.fit(); } catch(_) {}
         clearTimeout(resizeTimer);
         resizeTimer = setTimeout(function() {
-          if (term && ws && ws.readyState === WebSocket.OPEN) {
-            ws.send(JSON.stringify({ type: 'terminal_resize', cols: term.cols, rows: term.rows }));
-          }
+          if (term) wsSend({ type: 'terminal_resize', cols: term.cols, rows: term.rows });
         }, 200);
       }
     });
 
-    output.addEventListener('click', function() {
+    document.getElementById('terminalOutput').addEventListener('click', function() {
       if (term && term.focus) term.focus();
     });
 
     return true;
   }
 
-  function connectAndStart() {
-    var cwd = getCwd();
-    if (ws && ws.readyState === WebSocket.OPEN) {
-      var dims = term ? { cols: term.cols, rows: term.rows } : { cols: 80, rows: 24 };
-      ws.send(JSON.stringify({ type: 'terminal_start', cwd: cwd, cols: dims.cols, rows: dims.rows }));
-      setTimeout(function() { if (term && term.focus) term.focus(); }, 100);
-      return;
-    }
-    if (ws && ws.readyState === WebSocket.CONNECTING) {
-      return;
-    }
-
-    ws = new WebSocket(getWsUrl());
-    ws.onopen = function() {
-      var dims = term ? { cols: term.cols, rows: term.rows } : { cols: 80, rows: 24 };
-      ws.send(JSON.stringify({ type: 'terminal_start', cwd: cwd, cols: dims.cols, rows: dims.rows }));
-      setTimeout(function() { if (term && term.focus) term.focus(); }, 100);
-    };
-    ws.onmessage = function(e) {
-      try {
-        var msg = JSON.parse(e.data);
-        if (msg.type === 'terminal_output' && term) {
-          var raw = msg.encoding === 'base64'
-            ? decodeURIComponent(escape(atob(msg.data)))
-            : msg.data;
-          term.write(raw);
-        } else if (msg.type === 'terminal_exit' && term) {
-          term.write('\r\n[Process exited with code ' + msg.code + ']\r\n');
-          if (termActive) setTimeout(connectAndStart, 2000);
-        }
-      } catch(_) {}
-    };
-    ws.onclose = function() {
-      ws = null;
-      if (termActive) setTimeout(connectAndStart, 2000);
+  function installWsListener() {
+    if (_wsListener || !window.wsManager) return;
+    _wsListener = function(msg) {
+      if (!termActive) return;
+      if (msg.type === 'terminal_output' && term) {
+        var raw = msg.encoding === 'base64'
+          ? decodeURIComponent(escape(atob(msg.data)))
+          : msg.data;
+        term.write(raw);
+      } else if (msg.type === 'terminal_exit' && term) {
+        term.write('\r\n[Process exited with code ' + msg.code + ']\r\n');
+        if (termActive) setTimeout(startSession, 2000);
+      }
     };
-    ws.onerror = function() {};
+    window.wsManager.on('message', _wsListener);
+  }
+
+  function startSession() {
+    if (!window.wsManager) return;
+    installWsListener();
+    var cwd = getCwd();
+    var dims = term ? { cols: term.cols, rows: term.rows } : { cols: 80, rows: 24 };
+    wsSend({ type: 'terminal_start', cwd: cwd, cols: dims.cols, rows: dims.rows });
+    setTimeout(function() { if (term && term.focus) term.focus(); }, 100);
   }
 
   function startTerminal() {
@@ -137,19 +110,21 @@
       return;
     }
     termActive = true;
-    connectAndStart();
+    if (window.wsManager && window.wsManager.isConnected) {
+      startSession();
+    } else if (window.wsManager) {
+      var onConnected = function() {
+        window.wsManager.off('connected', onConnected);
+        startSession();
+      };
+      window.wsManager.on('connected', onConnected);
+    }
     setTimeout(function() { if (fitAddon) try { fitAddon.fit(); } catch(_) {} }, 100);
   }
 
   function stopTerminal() {
     termActive = false;
-    if (ws && ws.readyState === WebSocket.OPEN) {
-      ws.send(JSON.stringify({ type: 'terminal_stop' }));
-    }
-    if (ws) {
-      ws.close();
-      ws = null;
-    }
+    wsSend({ type: 'terminal_stop' });
   }
 
   function initTerminalEarly() {
@@ -172,7 +147,7 @@
         return;
       }
       termActive = true;
-      connectAndStart();
+      startSession();
       setTimeout(function() { if (fitAddon) try { fitAddon.fit(); } catch(_) {} }, 50);
       setTimeout(function() { if (fitAddon) try { fitAddon.fit(); } catch(_) {} }, 300);
     } else if (termActive) {
@@ -180,15 +155,12 @@
     }
   });
 
-  // Restart terminal in the new conversation's cwd when conversation switches while terminal is active
-  window.addEventListener('conversation-changed', function(e) {
+  window.addEventListener('conversation-changed', function() {
     if (!termActive) return;
     var cwd = getCwd();
-    if (ws && ws.readyState === WebSocket.OPEN) {
-      var dims = term ? { cols: term.cols, rows: term.rows } : { cols: 80, rows: 24 };
-      ws.send(JSON.stringify({ type: 'terminal_start', cwd: cwd, cols: dims.cols, rows: dims.rows }));
-      if (term) term.write('\r\n\x1b[33m[Switched to: ' + (cwd || '/') + ']\x1b[0m\r\n');
-    }
+    var dims = term ? { cols: term.cols, rows: term.rows } : { cols: 80, rows: 24 };
+    wsSend({ type: 'terminal_start', cwd: cwd, cols: dims.cols, rows: dims.rows });
+    if (term) term.write('\r\n\x1b[33m[Switched to: ' + (cwd || '/') + ']\x1b[0m\r\n');
   });
 
   window.terminalModule = {