agentgui 1.0.423 → 1.0.425

package/lib/ws-handlers-session.js

@@ -111,7 +111,8 @@ export function register(router, deps) {
     if (!agent || agent.protocol !== 'acp') return { subAgents: [] };
     try {
       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 })) };
+      const list = Array.isArray(data) ? data : (data?.agents || []);
+      return { subAgents: list.map(a => ({ id: a.agent_id || a.id, name: a.metadata?.ref?.name || a.name || a.agent_id || a.id })) };
     } catch (_) { return { subAgents: [] }; }
   });
 
@@ -143,12 +144,21 @@ export function register(router, deps) {
       } catch (_) {}
       try {
         const data = await acpFetch(port, '/agents/search', {});
+        const list = Array.isArray(data) ? data : (data?.agents || []);
         const allModels = new Map();
-        for (const a of (data?.agents || [])) {
+        for (const a of list) {
+          const agentId = a.agent_id || a.id;
+          const name = a.metadata?.ref?.name || a.name || agentId;
+          if (agentId && name) allModels.set(agentId, name);
           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);
           }
+          try {
+            const desc = await acpFetch(port, `/agents/${agentId}/descriptor`);
+            const enumVals = desc?.specs?.config?.properties?.model?.enum || desc?.specs?.input?.properties?.model?.enum || [];
+            for (const v of enumVals) allModels.set(v, v);
+          } catch (_) {}
         }
         if (allModels.size > 0) {
           const models = Array.from(allModels.entries()).map(([id, label]) => ({ id, label }));
@@ -157,6 +167,23 @@ export function register(router, deps) {
         }
       } catch (_) {}
     }
+    const acpAgents = discoveredAgents.filter(x => x.protocol === 'acp');
+    for (const acpAgent of acpAgents) {
+      const port = acpAgent.acpPort || 8080;
+      try {
+        const desc = await acpFetch(port, `/agents/${p.id}/descriptor`);
+        if (desc) {
+          const enumVals = desc?.specs?.config?.properties?.model?.enum || desc?.specs?.input?.properties?.model?.enum || [];
+          if (enumVals.length > 0) {
+            const models = enumVals.map(v => ({ id: v, label: v }));
+            modelCache.set(p.id, { models, ts: Date.now() });
+            return { models };
+          }
+          const name = desc?.metadata?.ref?.name;
+          if (name) { const models = [{ id: p.id, label: name }]; 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.423",
+  "version": "1.0.425",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "server.js",

package/server.js

@@ -568,18 +568,7 @@ async function getModelsForAgent(agentId) {
   let models = null;
 
   if (agentId === 'claude-code') {
-    const hardcodedModels = [
-      { id: 'haiku', label: 'Haiku (Default)' },
-      { id: 'sonnet', label: 'Sonnet' },
-      { id: 'opus', label: 'Opus' },
-    ];
-
-    const apiModels = await fetchClaudeModelsFromAPI();
-    if (apiModels) {
-      models = [...hardcodedModels, ...apiModels.slice(1)];
-    } else {
-      models = hardcodedModels;
-    }
+    models = await fetchClaudeModelsFromAPI();
   } else if (agentId === 'gemini') {
     models = await fetchGeminiModelsFromAPI();
   }
@@ -632,88 +621,8 @@ async function getModelsForAgent(agentId) {
     }
   }
 
-  // Fallback default models for agents that fail to retrieve models
-  const fallbackModels = {
-    'claude-code': [
-      { id: 'haiku', label: 'Haiku (Default)' },
-      { id: 'sonnet', label: 'Sonnet' },
-      { id: 'opus', label: 'Opus' }
-    ],
-    'gemini': [
-      { id: 'gemini-1.5-pro', label: 'Gemini 1.5 Pro' },
-      { id: 'gemini-1.5-flash', label: 'Gemini 1.5 Flash' },
-      { id: 'gemini-2.0-pro', label: 'Gemini 2.0 Pro' },
-      { id: 'gemini-2.0-flash', label: 'Gemini 2.0 Flash' }
-    ],
-    'opencode': [
-      { id: 'claude-sonnet-4-20250514', label: 'Claude Sonnet 4' },
-      { id: 'claude-haiku-4-20250514', label: 'Claude Haiku 4' },
-      { id: 'gpt-4-turbo', label: 'GPT-4 Turbo' },
-      { id: 'gemini-pro', label: 'Gemini Pro' }
-    ],
-    'kilo': [
-      { id: 'claude-sonnet-4', label: 'Claude Sonnet 4' },
-      { id: 'gemini-2.0-flash', label: 'Gemini 2.0 Flash' },
-      { id: 'gpt-4', label: 'GPT-4' }
-    ],
-    'goose': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'goose-pro', label: 'Goose Pro' }
-    ],
-    'openhands': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'openhands-1', label: 'OpenHands 1' }
-    ],
-    'augment': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'augment-pro', label: 'Augment Pro' }
-    ],
-    'cline': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'cline-1', label: 'Cline 1' }
-    ],
-    'kimi': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'kimi-pro', label: 'Kimi Pro' }
-    ],
-    'qwen': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'qwen-7b', label: 'Qwen 7B' },
-      { id: 'qwen-14b', label: 'Qwen 14B' }
-    ],
-    'codex': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'code-davinci-002', label: 'Code Davinci 002' },
-      { id: 'code-cushman-001', label: 'Code Cushman 001' }
-    ],
-    'mistral': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'mistral-small', label: 'Mistral Small' },
-      { id: 'mistral-medium', label: 'Mistral Medium' },
-      { id: 'mistral-large', label: 'Mistral Large' }
-    ],
-    'kiro': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'kiro-1', label: 'Kiro 1' }
-    ],
-    'fast-agent': [
-      { id: 'default', label: 'Default Model' },
-      { id: 'fast-agent-1', label: 'Fast Agent 1' }
-    ]
-  };
-
-  if (fallbackModels[agentId]) {
-    const models = fallbackModels[agentId];
-    modelCache.set(agentId, { models, timestamp: Date.now() });
-    return models;
-  }
-
-  // Default fallback for any other agent
-  const defaultFallback = [
-    { id: 'default', label: 'Default Model' }
-  ];
-  modelCache.set(agentId, { models: defaultFallback, timestamp: Date.now() });
-  return defaultFallback;
+  modelCache.set(agentId, { models: [], timestamp: Date.now() });
+  return [];
 }
 
 const GEMINI_SCOPES = [