agentgui 1.0.421 → 1.0.423

package/lib/ws-handlers-session.js

@@ -60,6 +60,14 @@ function checkAgentAuth(agent) {
   return s;
 }
 
+async function acpFetch(port, path, body = null) {
+  const opts = { headers: { 'Content-Type': 'application/json' }, signal: AbortSignal.timeout(3000) };
+  if (body !== null) { opts.method = 'POST'; opts.body = JSON.stringify(body); }
+  const res = await fetch(`http://localhost:${port}${path}`, opts);
+  if (!res.ok) return null;
+  return res.json();
+}
+
 export function register(router, deps) {
   const { db, discoveredAgents, getModelsForAgent, modelCache,
     getAgentDescriptor, activeScripts, broadcastSync, startGeminiOAuth,
@@ -88,51 +96,23 @@ export function register(router, deps) {
   router.handle('sess.exec', (p) => {
     const limit = Math.min(parseInt(p.limit || '1000'), 5000);
     const offset = Math.max(parseInt(p.offset || '0'), 0);
-    const filterType = p.filterType || null;
-    const data = {
-      sessionId: p.id, events: [], total: 0, limit, offset, hasMore: false,
-      metadata: { status: 'pending', startTime: Date.now(), duration: 0, eventCount: 0 }
-    };
-    if (filterType) data.events = data.events.filter(e => e.type === filterType);
-    return data;
+    return { sessionId: p.id, events: [], total: 0, limit, offset, hasMore: false, metadata: { status: 'pending', startTime: Date.now(), duration: 0, eventCount: 0 } };
   });
 
-  router.handle('agent.ls', () => {
-    // Get local agents only (avoid external HTTP calls in WebSocket to prevent recursion)
-    const localAgents = discoveredAgents.map(agent => ({
-      id: agent.id,
-      name: agent.name,
-      icon: agent.icon,
-      path: agent.path,
-      protocol: agent.protocol || 'unknown',
-      description: agent.description || '',
-      status: 'available'
-    }));
-    
-    return { agents: localAgents };
-  });
+  router.handle('agent.ls', () => ({
+    agents: discoveredAgents.map(a => ({
+      id: a.id, name: a.name, icon: a.icon, path: a.path,
+      protocol: a.protocol || 'unknown', description: a.description || '', status: 'available'
+    }))
+  }));
 
   router.handle('agent.subagents', async (p) => {
     const agent = discoveredAgents.find(x => x.id === p.id);
-    if (!agent) return { subAgents: [] };
+    if (!agent || agent.protocol !== 'acp') return { subAgents: [] };
     try {
-      const port = agent.acpPort || 8080;
-      const res = await fetch(`http://localhost:${port}/agents/search`, {
-        method: 'POST',
-        headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({}),
-        signal: AbortSignal.timeout(3000)
-      });
-      if (!res.ok) return { subAgents: [] };
-      const data = await res.json();
-      const subAgents = (data.agents || []).map(a => ({
-        id: a.agent_id || a.id,
-        name: a.metadata?.ref?.name || a.name || a.agent_id || a.id
-      }));
-      return { subAgents };
-    } catch (_) {
-      return { subAgents: [] };
-    }
+      const data = await acpFetch(agent.acpPort || 8080, '/agents/search', {});
+      return { subAgents: (data?.agents || []).map(a => ({ id: a.agent_id || a.id, name: a.metadata?.ref?.name || a.name || a.agent_id || a.id })) };
+    } catch (_) { return { subAgents: [] }; }
   });
 
   router.handle('agent.get', (p) => {
@@ -150,6 +130,33 @@ export function register(router, deps) {
   router.handle('agent.models', async (p) => {
     const cached = modelCache.get(p.id);
     if (cached && (Date.now() - cached.ts) < 300000) return { models: cached.models };
+    const agent = discoveredAgents.find(x => x.id === p.id);
+    if (agent?.protocol === 'acp') {
+      const port = agent.acpPort || 8080;
+      try {
+        const data = await acpFetch(port, '/v1/models');
+        const models = (data?.data || data?.models || []).map(m => ({
+          id: m.id || m.model_id,
+          label: m.name || m.display_name || m.id || m.model_id
+        })).filter(m => m.id);
+        if (models.length > 0) { modelCache.set(p.id, { models, ts: Date.now() }); return { models }; }
+      } catch (_) {}
+      try {
+        const data = await acpFetch(port, '/agents/search', {});
+        const allModels = new Map();
+        for (const a of (data?.agents || [])) {
+          for (const m of (a.metadata?.models || a.specs?.models || [])) {
+            const id = m.id || m;
+            if (id) allModels.set(id, m.name || m.label || id);
+          }
+        }
+        if (allModels.size > 0) {
+          const models = Array.from(allModels.entries()).map(([id, label]) => ({ id, label }));
+          modelCache.set(p.id, { models, ts: Date.now() });
+          return { models };
+        }
+      } catch (_) {}
+    }
     try {
       const models = await getModelsForAgent(p.id);
       modelCache.set(p.id, { models, ts: Date.now() });

package/package.json

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

package/static/js/client.js

@@ -363,6 +363,7 @@ class AgentGUIClient {
       this.ui.cliSelector.addEventListener('change', () => {
         if (!this._agentLocked) {
           this.loadSubAgentsForCli(this.ui.cliSelector.value);
+          this.loadModelsForAgent(this.ui.cliSelector.value);
           this.saveAgentAndModelToConversation();
         }
       });
@@ -1874,6 +1875,7 @@ class AgentGUIClient {
           .map(a => `<option value="${a.id}">${a.name}</option>`)
           .join('');
         this.ui.agentSelector.style.display = 'inline-block';
+        this.loadModelsForAgent(this.getEffectiveAgentId());
       }
     } catch (_) {
       // No sub-agents available for this CLI tool — keep hidden
@@ -1951,6 +1953,9 @@ class AgentGUIClient {
     this._agentLocked = false;
     if (this.ui.cliSelector) {
       this.ui.cliSelector.disabled = false;
+      if (this.ui.cliSelector.options.length > 0) {
+        this.ui.cliSelector.style.display = 'inline-block';
+      }
     }
     if (this.ui.agentSelector) {
       this.ui.agentSelector.disabled = false;