agentgui 1.0.990 → 1.0.992

package/lib/acp-sdk-manager.js

@@ -179,12 +179,6 @@ export function getPort(agentId) {
   return acpMachine.isHealthy(agentId) ? (ACP_TOOLS.find(t => t.id === agentId)?.port || null) : null;
 }
 
-export function getRunningPorts() {
-  const ports = {};
-  for (const tool of ACP_TOOLS) { if (acpMachine.isHealthy(tool.id)) ports[tool.id] = tool.port; }
-  return ports;
-}
-
 export async function restart(agentId) {
   const tool = ACP_TOOLS.find(t => t.id === agentId);
   if (!tool) return false;

package/lib/acp-server-machine.js

@@ -160,10 +160,6 @@ export function getBackoffDelay(toolId) {
   return calcBackoff(purgeOldRestarts(s.context.restarts));
 }
 
-export function getMachineActors() {
-  return actors;
-}
-
 export function stopAll() {
   for (const [, actor] of actors) actor.stop();
   actors.clear();

package/lib/agent-descriptors.js

@@ -43,6 +43,3 @@ export function getAgentDescriptor(agentId) {
   return agentDescriptorCache.get(agentId) || null;
 }
 
-export function getAllDescriptors() {
-  return Object.fromEntries(agentDescriptorCache);
-}

package/lib/agent-discovery.js

@@ -23,13 +23,6 @@ const BINARIES = [
   { cmd: 'hermes', id: 'hermes', name: 'Hermes Agent', icon: 'h', protocol: 'acp' },
 ];
 
-const CLI_WRAPPERS = [
-  { id: 'cli-opencode', name: 'OpenCode', icon: 'O', protocol: 'cli-wrapper', acpId: 'opencode' },
-  { id: 'cli-gemini', name: 'Gemini', icon: 'G', protocol: 'cli-wrapper', acpId: 'gemini' },
-  { id: 'cli-kilo', name: 'Kilo', icon: 'K', protocol: 'cli-wrapper', acpId: 'kilo' },
-  { id: 'cli-codex', name: 'Codex', icon: 'X', protocol: 'cli-wrapper', acpId: 'codex' },
-];
-
 export function findCommand(cmd, rootDir) {
   if (!cmd) return null;
   const isWindows = os.platform() === 'win32';
@@ -79,19 +72,8 @@ export function discoverAgents(rootDir) {
     }
   }
 
-  console.log('[discoverAgents] Found agents:', agents.map(a => a.id).join(', '));
-  for (const wrapper of CLI_WRAPPERS) {
-    if (agents.some(a => a.id === wrapper.acpId)) {
-      console.log(`[discoverAgents] Adding CLI wrapper for ${wrapper.id}`);
-      agents.push(wrapper);
-    } else {
-      console.log(`[discoverAgents] Skipping CLI wrapper ${wrapper.id} (ACP agent ${wrapper.acpId} not found)`);
-    }
-  }
-  const wrappedAcpIds = new Set(CLI_WRAPPERS.filter(w => agents.some(a => a.id === w.acpId)).map(w => w.acpId));
-  const filtered = agents.filter(a => !wrappedAcpIds.has(a.id));
-  console.log('[discoverAgents] Final agent count:', filtered.length, 'Agent IDs:', filtered.map(a => a.id).join(', '));
-  return filtered;
+  console.log('[discoverAgents] Final agent count:', agents.length, 'Agent IDs:', agents.map(a => a.id).join(', '));
+  return agents;
 }
 
 export async function initializeAgentDiscovery(discoveredAgents, rootDir, logError) {

package/lib/asset-server.js

@@ -124,40 +124,3 @@ export function serveFile(filePath, res, req, { compressAndSend, acceptsEncoding
     });
   });
 }
-
-export function createChunkBatcher(queries, debugLog) {
-  const pending = [];
-  let timer = null;
-  const BATCH_SIZE = 10;
-  const BATCH_INTERVAL = 50;
-
-  function flush() {
-    if (pending.length === 0) return;
-    const batch = pending.splice(0);
-    try {
-      const tx = queries._db ? queries._db.transaction(() => {
-        for (const c of batch) queries.createChunk(c.sessionId, c.conversationId, c.sequence, c.type, c.data);
-      }) : null;
-      if (tx) { tx(); } else {
-        for (const c of batch) {
-          try { queries.createChunk(c.sessionId, c.conversationId, c.sequence, c.type, c.data); } catch (e) { debugLog(`[chunk] ${e.message}`); }
-        }
-      }
-    } catch (err) {
-      debugLog(`[chunk-batch] Batch write failed: ${err.message}`);
-      for (const c of batch) {
-        try { queries.createChunk(c.sessionId, c.conversationId, c.sequence, c.type, c.data); } catch (_) {}
-      }
-    }
-  }
-
-  function add(sessionId, conversationId, sequence, blockType, blockData) {
-    pending.push({ sessionId, conversationId, sequence, type: blockType, data: blockData });
-    if (pending.length >= BATCH_SIZE) { if (timer) { clearTimeout(timer); timer = null; } flush(); }
-    else if (!timer) { timer = setTimeout(() => { timer = null; flush(); }, BATCH_INTERVAL); }
-  }
-
-  function drain() { if (timer) { clearTimeout(timer); timer = null; } flush(); }
-
-  return { add, drain };
-}

package/lib/claude-runner-run.js

@@ -43,6 +43,4 @@ export async function runClaudeWithStreaming(prompt, cwd, agentId = 'claude-code
   return agent.run(prompt, cwd, enhancedConfig);
 }
 
-export function getRegisteredAgents() { return registry.list(); }
-export function isAgentRegistered(agentId) { return registry.has(agentId); }
 export default runClaudeWithStreaming;

package/lib/execution-machine.js

@@ -140,17 +140,6 @@ export function snapshot(convId) {
   return actor ? actor.getSnapshot() : null;
 }
 
-export function isStreaming(convId) {
-  const s = snapshot(convId);
-  return s ? s.value === 'streaming' || s.value === 'rate_limited' : false;
-}
-
-export function isActive(convId) {
-  const s = snapshot(convId);
-  if (!s) return false;
-  return s.value === 'streaming' || s.value === 'rate_limited' || s.value === 'draining';
-}
-
 export function getContext(convId) {
   const s = snapshot(convId);
   return s ? s.context : null;

package/lib/gm-agent-configs.js

@@ -64,14 +64,3 @@ export async function installGMAgentConfigs() {
   });
   log('results: ' + summary.join(', '));
 }
-
-export async function forceReinstallGMAgentConfigs() {
-  log('force reinstalling all agent configs');
-  const results = await Promise.allSettled(GM_PACKAGES.map(p => runInstaller(p.pkg)));
-  const summary = GM_PACKAGES.map((p, i) => {
-    const r = results[i];
-    const ok = r.status === 'fulfilled' && r.value;
-    return p.pkg + ': ' + (ok ? 'ok' : 'failed');
-  });
-  log('results: ' + summary.join(', '));
-}

package/lib/process-message.js

@@ -1,4 +1,4 @@
-export function createProcessMessage({ queries, activeExecutions, rateLimitState, execMachine, broadcastSync, runClaudeWithStreaming, cleanupExecution, discoveredAgents, STARTUP_CWD, buildSystemPrompt, parseRateLimitResetTime, touchACP, getJsonlWatcher, debugLog, logError, scheduleRetry, drainMessageQueue, createEventHandler }) {
+export function createProcessMessage({ queries, activeExecutions, rateLimitState, execMachine, broadcastSync, runClaudeWithStreaming, cleanupExecution, STARTUP_CWD, buildSystemPrompt, parseRateLimitResetTime, touchACP, getJsonlWatcher, debugLog, logError, scheduleRetry, drainMessageQueue, createEventHandler }) {
   async function processMessageWithStreaming(conversationId, messageId, sessionId, content, agentId, model, subAgent) {
     const startTime = Date.now();
     touchACP(agentId);
@@ -31,9 +31,7 @@ export function createProcessMessage({ queries, activeExecutions, rateLimitState
     // Resolve agent before building stateRef so isJsonlBacked can be set correctly.
     // claude-code (protocol: direct) writes JSONL -> JsonlParser owns event broadcasting.
     // All other agents (protocol: acp) rely solely on onEvent for streaming.
-    let resolvedAgentId = agentId || 'claude-code';
-    const wrapperAgent = discoveredAgents.find(a => a.id === resolvedAgentId && a.protocol === 'cli-wrapper' && a.acpId);
-    if (wrapperAgent) resolvedAgentId = wrapperAgent.acpId;
+    const resolvedAgentId = agentId || 'claude-code';
     const isJsonlBacked = resolvedAgentId === 'claude-code';
     // stateRef tracks eventCount (for session response metadata) and resumeSessionId
     const stateRef = { eventCount: 0, resumeSessionId: conv?.claudeSessionId || null };

package/lib/server-utils.js

@@ -45,18 +45,10 @@ export function makeGetModelsForAgent(deps) {
         { id: 'opus', label: 'Opus' }
       ];
     } else {
-      // The client sends canonical registry ids ('opencode', 'kilo', 'codex'),
-      // but discoverAgents() replaces each found ACP agent with its cli-wrapper
-      // entry whose own id is prefixed ('cli-opencode') and whose acpId carries
-      // the original id. Match on acpId too so model discovery resolves the
-      // wrapper instead of finding nothing and silently returning [] models.
-      const agent = discoveredAgents.find(a => a.id === agentId || a.acpId === agentId);
+      const agent = discoveredAgents.find(a => a.id === agentId);
       if (agent?.protocol === 'acp') {
         await ensureRunning(agentId);
         try { models = await queryACPModels(agentId); } catch (_) {}
-      } else if (agent?.protocol === 'cli-wrapper' && agent.acpId) {
-        await ensureRunning(agent.acpId);
-        try { models = await queryACPModels(agent.acpId); } catch (_) {}
       }
     }
     modelCache.set(agentId, { models, timestamp: Date.now() });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.990",
+  "version": "1.0.992",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "electron/main.js",

package/test.js

@@ -340,13 +340,13 @@ await ok('cross-tab-storage: "updated in another tab" banner on stale load', ()
   }
 });
 
-await ok('agents.models: resolves cli-wrapper by acpId (registry id, not wrapper id)', async () => {
-  // The client picker sends canonical registry ids ('opencode'), but
-  // discoverAgents() stores only the cli-wrapper entry ('cli-opencode',
-  // acpId:'opencode'). getModelsForAgent must resolve via acpId, else model
-  // discovery silently returns [] for every ACP agent (witnessed regression).
+await ok('agents.models: ACP agent reaches ensureRunning + queryACPModels (not empty)', async () => {
+  // discoveredAgents and the client both use the canonical registry id
+  // ('opencode', protocol 'acp'). getModelsForAgent must start the ACP server
+  // and query it - a regression once silently returned [] for every ACP agent
+  // because of an id-scheme mismatch.
   const { makeGetModelsForAgent } = await import('./lib/server-utils.js');
-  const discoveredAgents = [{ id: 'cli-opencode', protocol: 'cli-wrapper', acpId: 'opencode' }];
+  const discoveredAgents = [{ id: 'opencode', protocol: 'acp' }];
   const calls = [];
   const getModels = makeGetModelsForAgent({
     modelCache: new Map(),
@@ -355,7 +355,7 @@ await ok('agents.models: resolves cli-wrapper by acpId (registry id, not wrapper
     queryACPModels: async (id) => { calls.push(['query', id]); return [{ id: 'm1', label: 'M1' }]; },
   });
   const models = await getModels('opencode');
-  assert.deepEqual(calls, [['ensure', 'opencode'], ['query', 'opencode']], 'should ensureRunning + query with acpId');
+  assert.deepEqual(calls, [['ensure', 'opencode'], ['query', 'opencode']], 'should ensureRunning + query the ACP agent');
   assert.equal(models.length, 1, 'should return the ACP-provided models, not an empty list');
 });