agentgui 1.0.602 → 1.0.604

package/package.json

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

package/static/js/client.js

@@ -765,10 +765,15 @@ class AgentGUIClient {
       this._serverProcessingEstimate = 0.7 * this._serverProcessingEstimate + 0.3 * serverTime;
     }
 
-    // Always subscribe to the session so blocks are not lost regardless of which conversation is active
+    // Subscribe to the session so blocks are not lost, but skip conversation
+    // re-subscribe when resumed=true to prevent infinite loop (server sends
+    // streaming_start on subscribe when activeExecutions exists, which would
+    // trigger another subscribe here, looping forever)
     if (this.wsManager.isConnected) {
       this.wsManager.subscribeToSession(data.sessionId);
-      this.wsManager.sendMessage({ type: 'subscribe', conversationId: data.conversationId });
+      if (!data.resumed) {
+        this.wsManager.sendMessage({ type: 'subscribe', conversationId: data.conversationId });
+      }
     }
 
     // If this streaming event is for a different conversation than what we are viewing,

package/static/js/streaming-renderer.js

@@ -751,6 +751,8 @@ class StreamingRenderer {
     if (block.id) details.dataset.toolUseId = block.id;
     details.classList.add(this._getBlockTypeClass('tool_use'));
     details.classList.add(this._getToolColorClass(toolName));
+    const normalizedForOpen = toolName.replace(/^mcp__.*?__/, '');
+    if (normalizedForOpen === 'TodoWrite') details.open = true;
     const summary = document.createElement('summary');
     summary.className = 'folded-tool-bar';
     const displayName = this.getToolUseDisplayName(toolName);
@@ -1228,17 +1230,8 @@ class StreamingRenderer {
     const content = block.content || '';
     const toolName = block.tool_name || block.name || '';
 
-    // Special handling for TodoWrite: render directly without success wrapper
-    // Detect by tool name OR by content structure (todos array)
-    if ((toolName.includes('TodoWrite') || (typeof content === 'object' && Array.isArray(content?.todos))) && typeof content === 'object' && content.todos && Array.isArray(content.todos)) {
-      const statusIcons = { completed: '✅', in_progress: '⚙', pending: '☐' };
-      const completedCount = content.todos.filter(t => t.status === 'completed').length;
-      const totalCount = content.todos.length;
-      const items = content.todos.map(t => `<div class="todo-item"><span class="todo-status">${statusIcons[t.status] || '&#9744;'}</span><span class="todo-text">${this.escapeHtml(t.content || '')}</span></div>`).join('');
-      const div = document.createElement('div');
-      div.className = 'block-tool-result';
-      div.innerHTML = `<details class="folded-tool" open><summary class="folded-tool-bar" style="cursor:pointer;padding:0.5rem;background:var(--color-bg-secondary);border-radius:0.25rem;user-select:none"><span style="font-weight:600;font-size:0.9rem">📋 Tasks</span><span style="margin-left:0.5rem;font-size:0.8rem;color:var(--color-text-secondary)">${completedCount}/${totalCount} complete</span></summary><div class="folded-tool-body tool-param-todos" style="padding:0.75rem">${items}</div></details>`;
-      return div;
+    if (toolName.includes('TodoWrite') || (typeof content === 'object' && Array.isArray(content?.todos))) {
+      return null;
     }
 
     const contentStr = typeof content === 'string' ? content : JSON.stringify(content, null, 2);