agentgui 1.0.149 → 1.0.151

package/lib/claude-runner.js

@@ -139,7 +139,24 @@ class AgentRunner {
     });
   }
 
-  async runACP(prompt, cwd, config = {}) {
+  async runACP(prompt, cwd, config = {}, _retryCount = 0) {
+    const maxRetries = config.maxRetries ?? 1;
+    try {
+      return await this._runACPOnce(prompt, cwd, config);
+    } catch (err) {
+      const isEmptyExit = err.message && err.message.includes('ACP exited with code');
+      const isBinaryError = err.code === 'ENOENT' || (err.message && err.message.includes('ENOENT'));
+      if ((isEmptyExit || isBinaryError) && _retryCount < maxRetries) {
+        const delay = Math.min(1000 * Math.pow(2, _retryCount), 5000);
+        console.error(`[${this.id}] ACP attempt ${_retryCount + 1} failed: ${err.message}. Retrying in ${delay}ms...`);
+        await new Promise(r => setTimeout(r, delay));
+        return this.runACP(prompt, cwd, config, _retryCount + 1);
+      }
+      throw err;
+    }
+  }
+
+  async _runACPOnce(prompt, cwd, config = {}) {
     return new Promise((resolve, reject) => {
       const {
         timeout = 300000,
@@ -147,11 +164,10 @@ class AgentRunner {
         onError = null
       } = config;
 
-      // Use adapter if required (e.g., for Claude Code via Zed adapter)
       const cmd = this.requiresAdapter && this.adapterCommand ? this.adapterCommand : this.command;
       const baseArgs = this.requiresAdapter && this.adapterCommand ? this.adapterArgs : ['acp'];
       const args = [...baseArgs];
-      
+
       const proc = spawn(cmd, args, { cwd });
 
       if (config.onPid) {
@@ -163,6 +179,7 @@ class AgentRunner {
       let sessionId = null;
       let requestId = 0;
       let initialized = false;
+      let stderrText = '';
 
       const timeoutHandle = setTimeout(() => {
         timedOut = true;
@@ -170,12 +187,9 @@ class AgentRunner {
         reject(new Error(`${this.name} ACP timeout after ${timeout}ms`));
       }, timeout);
 
-      // ACP protocol handler
       const handleMessage = (message) => {
-        // Normalize ACP message to common format
         const normalized = this.protocolHandler(message, { sessionId, initialized });
         if (!normalized) {
-          // Check for initialization response
           if (message.id === 1 && message.result) {
             initialized = true;
           }
@@ -217,13 +231,13 @@ class AgentRunner {
 
       proc.stderr.on('data', (chunk) => {
         const errorText = chunk.toString();
+        stderrText += errorText;
         console.error(`[${this.id}] stderr:`, errorText);
         if (onError) {
           try { onError(errorText); } catch (e) {}
         }
       });
 
-      // Send ACP initialize request (protocolVersion must be an integer per ACP spec)
       const initRequest = {
         jsonrpc: '2.0',
         id: ++requestId,
@@ -245,12 +259,10 @@ class AgentRunner {
 
       let sessionCreated = false;
 
-      // Wait for initialization then create session and send prompt
       const checkInitAndSend = () => {
         if (initialized && !sessionCreated) {
           sessionCreated = true;
-          
-          // Step 1: Create a new session
+
           const sessionRequest = {
             jsonrpc: '2.0',
             id: ++requestId,
@@ -269,14 +281,11 @@ class AgentRunner {
       let promptId = null;
       let completed = false;
 
-      // Handle session creation response and send prompt
       const originalHandler = handleMessage;
       const enhancedHandler = (message) => {
-        // Check for session/new response
         if (message.id && message.result && message.result.sessionId) {
           sessionId = message.result.sessionId;
-          
-          // Step 2: Send the prompt
+
           promptId = ++requestId;
           const promptRequest = {
             jsonrpc: '2.0',
@@ -290,8 +299,7 @@ class AgentRunner {
           proc.stdin.write(JSON.stringify(promptRequest) + '\n');
           return;
         }
-        
-        // Check for prompt response (end of turn)
+
         if (message.id === promptId && message.result && message.result.stopReason) {
           completed = true;
           clearTimeout(timeoutHandle);
@@ -299,11 +307,10 @@ class AgentRunner {
           resolve({ outputs, sessionId });
           return;
         }
-        
+
         originalHandler(message);
       };
 
-      // Override the message handler
       buffer = '';
       proc.stdout.removeAllListeners('data');
       proc.stdout.on('data', (chunk) => {
@@ -317,12 +324,11 @@ class AgentRunner {
           if (line.trim()) {
             try {
               const message = JSON.parse(line);
-              
-              // Check for initialization response
+
               if (message.id === 1 && message.result) {
                 initialized = true;
               }
-              
+
               enhancedHandler(message);
             } catch (e) {
               console.error(`[${this.id}] JSON parse error:`, line.substring(0, 100));
@@ -340,7 +346,8 @@ class AgentRunner {
         if (code === 0 || outputs.length > 0) {
           resolve({ outputs, sessionId });
         } else {
-          reject(new Error(`${this.name} ACP exited with code ${code}`));
+          const detail = stderrText ? `: ${stderrText.substring(0, 200)}` : '';
+          reject(new Error(`${this.name} ACP exited with code ${code}${detail}`));
         }
       });
 
@@ -386,12 +393,15 @@ class AgentRegistry {
   }
 
   listACPAvailable() {
-    // Check which agents are actually installed
-    const { execSync } = require('child_process');
+    const { spawnSync } = require('child_process');
     return this.list().filter(agent => {
       try {
-        execSync(`which ${agent.command} 2>/dev/null`, { encoding: 'utf-8' });
-        return true;
+        const which = spawnSync('which', [agent.command], { encoding: 'utf-8', timeout: 3000 });
+        if (which.status !== 0) return false;
+        const binPath = (which.stdout || '').trim();
+        if (!binPath) return false;
+        const check = spawnSync(binPath, ['--version'], { encoding: 'utf-8', timeout: 10000 });
+        return check.status === 0 && (check.stdout || '').trim().length > 0;
       } catch {
         return false;
       }

package/package.json

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

package/static/index.html

@@ -7,10 +7,14 @@
   <title>AgentGUI</title>
   <link rel="icon" type="image/svg+xml" href="data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 100 100'%3E%3Crect width='100' height='100' rx='20' fill='%233b82f6'/%3E%3Ctext x='50' y='68' font-size='50' font-family='sans-serif' font-weight='bold' fill='white' text-anchor='middle'%3EG%3C/text%3E%3C/svg%3E">
 
+  <link rel="preconnect" href="https://cdn.jsdelivr.net" crossorigin>
+  <link rel="preconnect" href="https://cdnjs.cloudflare.com" crossorigin>
   <link href="https://cdn.jsdelivr.net/npm/rippleui@1.12.1/dist/css/styles.css" rel="stylesheet">
-  <link href="https://cdn.jsdelivr.net/npm/prismjs@1.29.0/themes/prism-dark.css" rel="stylesheet">
-  <link href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/styles/github-dark.min.css" rel="stylesheet">
-  <script src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/highlight.min.js"></script>
+  <link rel="preload" href="https://cdn.jsdelivr.net/npm/prismjs@1.29.0/themes/prism-dark.css" as="style" onload="this.onload=null;this.rel='stylesheet'">
+  <noscript><link href="https://cdn.jsdelivr.net/npm/prismjs@1.29.0/themes/prism-dark.css" rel="stylesheet"></noscript>
+  <link rel="preload" href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/styles/github-dark.min.css" as="style" onload="this.onload=null;this.rel='stylesheet'">
+  <noscript><link href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/styles/github-dark.min.css" rel="stylesheet"></noscript>
+  <script defer src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/highlight.min.js"></script>
 
   <style>
     *, *::before, *::after { box-sizing: border-box; }
@@ -1158,6 +1162,19 @@
       font-size: 0.9rem;
     }
 
+    .block-text + .block-text {
+      margin-top: -0.75rem;
+      padding-top: 0;
+      border-top-left-radius: 0;
+      border-top-right-radius: 0;
+    }
+
+    .block-text:has(+ .block-text) {
+      margin-bottom: 0;
+      border-bottom-left-radius: 0;
+      border-bottom-right-radius: 0;
+    }
+
     .block-code {
       margin-bottom: 0.75rem;
       border-radius: 0.5rem;
@@ -2060,16 +2077,16 @@
     </div>
   </div>
 
-  <script src="/gm/js/event-processor.js"></script>
-  <script src="/gm/js/streaming-renderer.js"></script>
-  <script src="/gm/js/websocket-manager.js"></script>
-  <script src="/gm/js/event-filter.js"></script>
-  <script src="/gm/js/syntax-highlighter.js"></script>
-  <script src="/gm/js/ui-components.js"></script>
-  <script src="/gm/js/conversations.js"></script>
-  <script src="/gm/js/client.js"></script>
+  <script defer src="/gm/js/event-processor.js"></script>
+  <script defer src="/gm/js/streaming-renderer.js"></script>
+  <script defer src="/gm/js/websocket-manager.js"></script>
+  <script defer src="/gm/js/event-filter.js"></script>
+  <script defer src="/gm/js/syntax-highlighter.js"></script>
+  <script defer src="/gm/js/ui-components.js"></script>
+  <script defer src="/gm/js/conversations.js"></script>
+  <script defer src="/gm/js/client.js"></script>
   <script type="module" src="/gm/js/voice.js"></script>
-  <script src="/gm/js/features.js"></script>
+  <script defer src="/gm/js/features.js"></script>
 
   <script>
     const savedTheme = localStorage.getItem('theme') || 'light';

package/static/js/streaming-renderer.js

@@ -135,12 +135,17 @@ class StreamingRenderer {
     const lastTime = this.dedupMap.get(key);
     const now = Date.now();
 
-    // Deduplicate within 100ms window
     if (lastTime && (now - lastTime) < 100) {
       return true;
     }
 
     this.dedupMap.set(key, now);
+    if (this.dedupMap.size > 5000) {
+      const cutoff = now - 1000;
+      for (const [k, t] of this.dedupMap) {
+        if (t < cutoff) this.dedupMap.delete(k);
+      }
+    }
     return false;
   }
 
@@ -358,17 +363,25 @@ class StreamingRenderer {
    * Render text block with semantic HTML
    */
   renderBlockText(block, context) {
-    const div = document.createElement('div');
-    div.className = 'block-text';
-
     const text = block.text || '';
-    if (this.containsHtmlTags(text)) {
-      div.innerHTML = this.sanitizeHtml(text);
-      div.classList.add('html-content');
-    } else {
-      div.innerHTML = this.parseAndRenderMarkdown(text);
+    const isHtml = this.containsHtmlTags(text);
+    const cached = this.renderCache.get(text);
+    const html = cached || (isHtml ? this.sanitizeHtml(text) : this.parseAndRenderMarkdown(text));
+
+    if (!cached && this.renderCache.size < 2000) {
+      this.renderCache.set(text, html);
+    }
+
+    const lastChild = this.outputContainer && this.outputContainer.lastElementChild;
+    if (lastChild && lastChild.classList.contains('block-text') && !isHtml && !lastChild.classList.contains('html-content')) {
+      lastChild.innerHTML += html;
+      return null;
     }
 
+    const div = document.createElement('div');
+    div.className = 'block-text';
+    if (isHtml) div.classList.add('html-content');
+    div.innerHTML = html;
     return div;
   }
 
@@ -447,11 +460,9 @@ class StreamingRenderer {
 
     const preStyle = "background:#1e293b;padding:1rem;border-radius:0 0 0.375rem 0.375rem;overflow-x:auto;font-family:'Monaco','Menlo','Ubuntu Mono',monospace;font-size:0.875rem;line-height:1.6;color:#e2e8f0;border:1px solid #334155;border-top:none;margin:0";
     const codeContainer = document.createElement('div');
+    codeContainer.innerHTML = `<pre style="${preStyle}"><code>${this.escapeHtml(code)}</code></pre>`;
     if (typeof hljs !== 'undefined') {
-      const result = hljs.highlightAuto(code);
-      codeContainer.innerHTML = `<pre style="${preStyle}"><code class="hljs">${result.value}</code></pre>`;
-    } else {
-      codeContainer.innerHTML = `<pre style="${preStyle}"><code>${this.escapeHtml(code)}</code></pre>`;
+      this.lazyHighlight(codeContainer, code);
     }
 
     details.appendChild(summary);
@@ -1785,13 +1796,39 @@ class StreamingRenderer {
    * Auto-scroll to bottom of container
    */
   autoScroll() {
-    if (this.scrollContainer) {
-      try {
-        this.scrollContainer.scrollTop = this.scrollContainer.scrollHeight;
-      } catch (e) {
-        // Ignore scroll errors
+    if (this._scrollRafPending) return;
+    this._scrollRafPending = true;
+    requestAnimationFrame(() => {
+      this._scrollRafPending = false;
+      if (this.scrollContainer) {
+        try { this.scrollContainer.scrollTop = this.scrollContainer.scrollHeight; } catch (_) {}
       }
+    });
+  }
+
+  lazyHighlight(container, code) {
+    if (!this._hlObserver) {
+      this._hlObserver = new IntersectionObserver((entries) => {
+        for (const entry of entries) {
+          if (!entry.isIntersecting) continue;
+          const el = entry.target;
+          const raw = el._rawCode;
+          if (!raw) continue;
+          this._hlObserver.unobserve(el);
+          try {
+            const codeEl = el.querySelector('code');
+            if (codeEl && typeof hljs !== 'undefined') {
+              const result = hljs.highlightAuto(raw);
+              codeEl.classList.add('hljs');
+              codeEl.innerHTML = result.value;
+            }
+          } catch (_) {}
+          delete el._rawCode;
+        }
+      }, { rootMargin: '200px' });
     }
+    container._rawCode = code;
+    this._hlObserver.observe(container);
   }
 
   updateVirtualScroll() {