agentgui 1.0.391 → 1.0.393

package/.prd

@@ -28,10 +28,10 @@ Transform AgentGUI into a fully ACP (Agent Connect Protocol) v0.2.3 compliant se
 - Run cancellation working
 - Event stream format compliant with ACP spec
 
-### ✅ WAVE 4: UI Fixes & Optimization (COMPLETED - Already Implemented)
-- **4.1** Thread Sidebar UI Consistency: Agent/model persistence working correctly via `applyAgentAndModelSelection()`
-- **4.2** WebSocket Optimization: Adaptive batching (16-200ms), subscription targeting, rate limiting all implemented
-- **4.3** Duplicate Displays: No duplicates found - all displays serve appropriate purposes
+### ✅ WAVE 4: UI Fixes & Optimization (COMPLETED - Enhanced)
+- **4.1** Thread Sidebar UI Consistency: Fixed agentId vs agentType inconsistency, sidebar now correctly uses `agentId`, model column confirmed in database, agent/model restore on page reload working
+- **4.2** WebSocket Optimization: Added message deduplication via `wsLastMessages` Map and `createMessageKey()` function, prevents identical consecutive messages, adaptive batching and rate limiting already present
+- **4.3** Duplicate Displays: Removed redundant agent/model from conversation headers (3 locations) and streaming start event, kept authoritative displays in sidebar and input selectors only
 
 ---
 

package/package.json

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

package/server.js

@@ -1825,6 +1825,12 @@ const server = http.createServer(async (req, res) => {
       return;
     }
 
+    if (pathOnly === '/api/ws-stats' && req.method === 'GET') {
+      const stats = wsOptimizer.getStats();
+      sendJSON(req, res, 200, stats);
+      return;
+    }
+
     if (pathOnly === '/api/agents/search' && req.method === 'POST') {
       const body = await parseBody(req);
       const result = queries.searchAgents(discoveredAgents, body);
@@ -3732,6 +3738,7 @@ wss.on('connection', (ws, req) => {
     ws.on('close', () => {
       if (ws.terminalProc) { try { ws.terminalProc.kill(); } catch(e) {} ws.terminalProc = null; }
       syncClients.delete(ws);
+      wsOptimizer.removeClient(ws);
       for (const sub of ws.subscriptions) {
         const idx = subscriptionIndex.get(sub);
         if (idx) { idx.delete(ws); if (idx.size === 0) subscriptionIndex.delete(sub); }
@@ -3749,68 +3756,15 @@ const BROADCAST_TYPES = new Set([
   'model_download_progress', 'stt_progress', 'tts_setup_progress', 'voice_list'
 ]);
 
-const wsBatchQueues = new Map();
-const wsLastMessages = new Map();
-const BATCH_BY_TIER = { excellent: 16, good: 32, fair: 50, poor: 100, bad: 200 };
-
-const TIER_ORDER = ['excellent', 'good', 'fair', 'poor', 'bad'];
-function getBatchInterval(ws) {
-  const tier = ws.latencyTier || 'good';
-  const trend = ws.latencyTrend;
-  if (trend === 'rising' || trend === 'falling') {
-    const idx = TIER_ORDER.indexOf(tier);
-    if (trend === 'rising' && idx < TIER_ORDER.length - 1) return BATCH_BY_TIER[TIER_ORDER[idx + 1]] || 32;
-    if (trend === 'falling' && idx > 0) return BATCH_BY_TIER[TIER_ORDER[idx - 1]] || 32;
-  }
-  return BATCH_BY_TIER[tier] || 32;
-}
-
-function flushWsBatch(ws) {
-  const queue = wsBatchQueues.get(ws);
-  if (!queue || queue.msgs.length === 0) return;
-  if (ws.readyState !== 1) { wsBatchQueues.delete(ws); wsLastMessages.delete(ws); return; }
-  if (queue.msgs.length === 1) {
-    ws.send(queue.msgs[0]);
-  } else {
-    ws.send('[' + queue.msgs.join(',') + ']');
-  }
-  queue.msgs.length = 0;
-  queue.timer = null;
-}
-
-function createMessageKey(event) {
-  return `${event.type}:${event.sessionId || ''}:${event.conversationId || ''}:${event.status || ''}`;
-}
-
-function sendToClient(ws, data) {
-  if (ws.readyState !== 1) return;
-
-  const event = JSON.parse(data);
-  const msgKey = createMessageKey(event);
-  const lastKey = wsLastMessages.get(ws);
-
-  if (msgKey === lastKey) {
-    return;
-  }
-
-  wsLastMessages.set(ws, msgKey);
-
-  let queue = wsBatchQueues.get(ws);
-  if (!queue) { queue = { msgs: [], timer: null }; wsBatchQueues.set(ws, queue); }
-  queue.msgs.push(data);
-  if (!queue.timer) {
-    queue.timer = setTimeout(() => flushWsBatch(ws), getBatchInterval(ws));
-  }
-}
+const wsOptimizer = new WSOptimizer();
 
 function broadcastSync(event) {
-  const data = JSON.stringify(event);
   const isBroadcast = BROADCAST_TYPES.has(event.type);
 
-  // Send to WebSocket clients
+  // Send to WebSocket clients using optimizer
   if (syncClients.size > 0) {
     if (isBroadcast) {
-      for (const ws of syncClients) sendToClient(ws, data);
+      for (const ws of syncClients) wsOptimizer.sendToClient(ws, event);
     } else {
       const targets = new Set();
       if (event.sessionId) {
@@ -3821,7 +3775,7 @@ function broadcastSync(event) {
         const subs = subscriptionIndex.get(`conv-${event.conversationId}`);
         if (subs) for (const ws of subs) targets.add(ws);
       }
-      for (const ws of targets) sendToClient(ws, data);
+      for (const ws of targets) wsOptimizer.sendToClient(ws, event);
     }
   }
 
@@ -3838,6 +3792,7 @@ const heartbeatInterval = setInterval(() => {
   syncClients.forEach(ws => {
     if (!ws.isAlive) {
       syncClients.delete(ws);
+      wsOptimizer.removeClient(ws);
       return ws.terminate();
     }
     ws.isAlive = false;