agentgui 1.0.603 → 1.0.605

package/lib/ws-handlers-util.js

@@ -16,7 +16,8 @@ export function register(router, deps) {
   router.handle('folders', (p) => {
     const folderPath = p.path || STARTUP_CWD;
     try {
-      const expanded = folderPath.startsWith('~') ? folderPath.replace('~', os.homedir()) : folderPath;
+      const raw = folderPath.startsWith('~') ? folderPath.replace('~', os.homedir()) : folderPath;
+      const expanded = path.resolve(raw);
       const entries = fs.readdirSync(expanded, { withFileTypes: true });
       const folders = entries
         .filter(e => e.isDirectory() && !e.name.startsWith('.'))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.603",
+  "version": "1.0.605",
   "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/conversations.js

@@ -191,7 +191,8 @@ class ConversationManager {
 
       this.folderBrowser.listEl.innerHTML = '';
 
-      if (dirPath !== '~' && dirPath !== '/' && dirPath !== this.folderBrowser.homePath) {
+      const isAtRoot = dirPath === '~' || dirPath === '/' || dirPath === this.folderBrowser.homePath || /^[A-Za-z]:[\\\/]?$/.test(dirPath);
+      if (!isAtRoot) {
         const parentPath = this.getParentPath(dirPath);
         const upItem = document.createElement('li');
         upItem.className = 'folder-list-item';
@@ -212,7 +213,8 @@ class ConversationManager {
         li.addEventListener('click', () => {
           const expandedBase = dirPath === '~' ? this.folderBrowser.homePath : dirPath;
           const separator = expandedBase.includes('\\') ? '\\' : '/';
-          const newPath = expandedBase + separator + folder.name;
+          const base = expandedBase.replace(/[\/\\]+$/, '');
+          const newPath = base + separator + folder.name;
           this.loadFolders(newPath);
         });
         this.folderBrowser.listEl.appendChild(li);
@@ -244,17 +246,25 @@ class ConversationManager {
 
     const expanded = dirPath === '~' ? this.folderBrowser.homePath : dirPath;
     const parts = pathSplit(expanded);
-    const separator = expanded.includes('\\') ? '\\' : '/';
+    const isWin = expanded.includes('\\') || /^[A-Za-z]:/.test(expanded);
+    const separator = isWin ? '\\' : '/';
+    const rootPath = isWin && parts[0] && /^[A-Za-z]:$/.test(parts[0]) ? parts[0] + separator : separator;
 
     let html = '';
-    html += `<span class="folder-breadcrumb-segment" data-path="${separator}">${separator} </span>`;
+    html += `<span class="folder-breadcrumb-segment" data-path="${this.escapeHtml(rootPath)}">${this.escapeHtml(rootPath)} </span>`;
 
     let accumulated = '';
+    const isDriveLetter = isWin && parts[0] && /^[A-Za-z]:$/.test(parts[0]);
     for (let i = 0; i < parts.length; i++) {
-      accumulated += separator + parts[i];
+      if (i === 0 && isDriveLetter) {
+        accumulated = parts[0];
+      } else {
+        accumulated += separator + parts[i];
+      }
+      const segPath = (i === 0 && isDriveLetter) ? rootPath : accumulated;
       const isLast = i === parts.length - 1;
       html += `<span class="folder-breadcrumb-separator">${separator}</span>`;
-      html += `<span class="folder-breadcrumb-segment${isLast ? '' : ''}" data-path="${this.escapeHtml(accumulated)}">${this.escapeHtml(parts[i])}</span>`;
+      html += `<span class="folder-breadcrumb-segment${isLast ? '' : ''}" data-path="${this.escapeHtml(segPath)}">${this.escapeHtml(parts[i])}</span>`;
     }
 
     this.folderBrowser.breadcrumbEl.innerHTML = html;