agentgui 1.0.769 → 1.0.770

package/CLAUDE.md

@@ -50,6 +50,8 @@ lib/routes-speech.js   Speech/TTS HTTP route handlers (stt, tts, voices, speech-
 lib/routes-oauth.js    OAuth HTTP route handlers (gemini-oauth/*, codex-oauth/*)
 lib/routes-tools.js    Tool management HTTP route handlers (list, install, update, history, refresh)
 lib/routes-util.js     Utility HTTP route handlers (clone, folders, git, home, version, import)
+lib/routes-conversations.js  Conversation CRUD HTTP route handlers (list, create, get, update, delete, archive, restore)
+lib/routes-debug.js    Debug/backup/restore/ws-stats HTTP route handlers
 lib/routes-threads.js  Thread CRUD HTTP route handlers (ACP v0.2.3 thread API)
 lib/ws-protocol.js     WebSocket RPC router (WsRouter class)
 lib/ws-optimizer.js    Per-client priority queue for WS event batching

package/lib/routes-conversations.js

@@ -0,0 +1,87 @@
+import path from 'path';
+import os from 'os';
+
+function expandTilde(p) { return p && p.startsWith('~') ? path.join(os.homedir(), p.slice(1)) : p; }
+
+export function register(deps) {
+  const { sendJSON, parseBody, queries, activeExecutions, broadcastSync } = deps;
+  const routes = {};
+
+  routes['GET /api/conversations'] = async (req, res) => {
+    const conversations = queries.getConversationsList();
+    const activeSessionConvIds = new Set(queries.getActiveSessionConversationIds());
+    for (const conv of conversations) {
+      if (conv.isStreaming && !activeExecutions.has(conv.id) && !activeSessionConvIds.has(conv.id)) conv.isStreaming = 0;
+    }
+    sendJSON(req, res, 200, { conversations });
+  };
+
+  routes['POST /api/conversations'] = async (req, res) => {
+    const body = await parseBody(req);
+    const normalizedWorkingDir = body.workingDirectory ? path.resolve(expandTilde(body.workingDirectory)) : null;
+    const conversation = queries.createConversation(body.agentId, body.title, normalizedWorkingDir, body.model || null);
+    queries.createEvent('conversation.created', { agentId: body.agentId, workingDirectory: conversation.workingDirectory, model: conversation.model }, conversation.id);
+    broadcastSync({ type: 'conversation_created', conversation });
+    sendJSON(req, res, 201, { conversation });
+  };
+
+  routes['GET /api/conversations/archived'] = async (req, res) => {
+    sendJSON(req, res, 200, { conversations: queries.getArchivedConversations() });
+  };
+
+  routes['_match'] = (method, pathOnly) => {
+    const key = `${method} ${pathOnly}`;
+    if (routes[key]) return routes[key];
+    let m;
+    if ((m = pathOnly.match(/^\/api\/conversations\/([^/]+)$/))) {
+      if (method === 'GET') return (req, res) => handleGetConv(req, res, m[1]);
+      if (method === 'POST' || method === 'PUT') return (req, res) => handleUpdateConv(req, res, m[1]);
+      if (method === 'DELETE') return (req, res) => handleDeleteConv(req, res, m[1]);
+    }
+    if (method === 'POST' && (m = pathOnly.match(/^\/api\/conversations\/([^/]+)\/archive$/)))
+      return (req, res) => handleArchive(req, res, m[1]);
+    if (method === 'POST' && (m = pathOnly.match(/^\/api\/conversations\/([^/]+)\/restore$/)))
+      return (req, res) => handleRestore(req, res, m[1]);
+    return null;
+  };
+
+  async function handleGetConv(req, res, id) {
+    const conv = queries.getConversation(id);
+    if (!conv) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
+    const latestSession = queries.getLatestSession(id);
+    sendJSON(req, res, 200, { conversation: conv, isActivelyStreaming: activeExecutions.has(id), latestSession });
+  }
+
+  async function handleUpdateConv(req, res, id) {
+    const body = await parseBody(req);
+    if (body.workingDirectory) body.workingDirectory = path.resolve(expandTilde(body.workingDirectory));
+    const conv = queries.updateConversation(id, body);
+    if (!conv) { sendJSON(req, res, 404, { error: 'Conversation not found' }); return; }
+    queries.createEvent('conversation.updated', body, id);
+    broadcastSync({ type: 'conversation_updated', conversation: conv });
+    sendJSON(req, res, 200, { conversation: conv });
+  }
+
+  async function handleDeleteConv(req, res, id) {
+    const deleted = queries.deleteConversation(id);
+    if (!deleted) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
+    broadcastSync({ type: 'conversation_deleted', conversationId: id });
+    sendJSON(req, res, 200, { deleted: true });
+  }
+
+  async function handleArchive(req, res, id) {
+    const conv = queries.archiveConversation(id);
+    if (!conv) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
+    broadcastSync({ type: 'conversation_deleted', conversationId: id });
+    sendJSON(req, res, 200, { conversation: conv });
+  }
+
+  async function handleRestore(req, res, id) {
+    const conv = queries.restoreConversation(id);
+    if (!conv) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
+    broadcastSync({ type: 'conversation_created', conversation: conv });
+    sendJSON(req, res, 200, { conversation: conv });
+  }
+
+  return routes;
+}

package/package.json

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

package/server.js

@@ -24,6 +24,7 @@ import { register as registerUtilRoutes } from './lib/routes-util.js';
 import { register as registerToolRoutes } from './lib/routes-tools.js';
 import { register as registerThreadRoutes } from './lib/routes-threads.js';
 import { register as registerDebugRoutes } from './lib/routes-debug.js';
+import { register as registerConvRoutes } from './lib/routes-conversations.js';
 import { startCodexOAuth, exchangeCodexOAuthCode, handleCodexOAuthCallback, getCodexOAuthStatus, getCodexOAuthState, CODEX_HOME, CODEX_AUTH_FILE } from './lib/oauth-codex.js';
 import { WSOptimizer } from './lib/ws-optimizer.js';
 import { WsRouter } from './lib/ws-protocol.js';
@@ -546,94 +547,8 @@ const server = http.createServer(async (req, res) => {
       return;
     }
 
-    if (pathOnly === '/api/conversations' && req.method === 'GET') {
-      const conversations = queries.getConversationsList();
-      // Filter out stale streaming state using a single bulk query instead of N+1 per-conversation queries
-      const activeSessionConvIds = new Set(queries.getActiveSessionConversationIds());
-      for (const conv of conversations) {
-        if (conv.isStreaming && !activeExecutions.has(conv.id) && !activeSessionConvIds.has(conv.id)) {
-          conv.isStreaming = 0;
-        }
-      }
-            sendJSON(req, res, 200, { conversations });
-      return;
-    }
-
-    if (pathOnly === '/api/conversations' && req.method === 'POST') {
-      const body = await parseBody(req);
-      // Normalize working directory to avoid Windows path issues; expand ~ to home
-      const expandTilde = p => p && p.startsWith('~') ? path.join(os.homedir(), p.slice(1)) : p;
-      const normalizedWorkingDir = body.workingDirectory ? path.resolve(expandTilde(body.workingDirectory)) : null;
-      const conversation = queries.createConversation(body.agentId, body.title, normalizedWorkingDir, body.model || null);
-      queries.createEvent('conversation.created', { agentId: body.agentId, workingDirectory: conversation.workingDirectory, model: conversation.model }, conversation.id);
-      broadcastSync({ type: 'conversation_created', conversation });
-            sendJSON(req, res, 201, { conversation });
-      return;
-    }
-
-    const convMatch = pathOnly.match(/^\/api\/conversations\/([^/]+)$/);
-    if (convMatch) {
-      if (req.method === 'GET') {
-        const conv = queries.getConversation(convMatch[1]);
-        if (!conv) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
-
-        const latestSession = queries.getLatestSession(convMatch[1]);
-        const isActivelyStreaming = activeExecutions.has(convMatch[1]);
-
-                sendJSON(req, res, 200, {
-          conversation: conv,
-          isActivelyStreaming,
-          latestSession
-        });
-        return;
-      }
-
-      if (req.method === 'POST' || req.method === 'PUT') {
-        const body = await parseBody(req);
-        // Normalize working directory if present to avoid Windows path issues; expand ~ to home
-        if (body.workingDirectory) {
-          const expandTilde = p => p && p.startsWith('~') ? path.join(os.homedir(), p.slice(1)) : p;
-          body.workingDirectory = path.resolve(expandTilde(body.workingDirectory));
-        }
-        const conv = queries.updateConversation(convMatch[1], body);
-        if (!conv) { sendJSON(req, res, 404, { error: 'Conversation not found' }); return; }
-        queries.createEvent('conversation.updated', body, convMatch[1]);
-        broadcastSync({ type: 'conversation_updated', conversation: conv });
-                sendJSON(req, res, 200, { conversation: conv });
-        return;
-      }
-
-      if (req.method === 'DELETE') {
-        const deleted = queries.deleteConversation(convMatch[1]);
-        if (!deleted) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
-        broadcastSync({ type: 'conversation_deleted', conversationId: convMatch[1] });
-                sendJSON(req, res, 200, { deleted: true });
-        return;
-      }
-    }
-
-    if (pathOnly === '/api/conversations/archived' && req.method === 'GET') {
-      sendJSON(req, res, 200, { conversations: queries.getArchivedConversations() });
-      return;
-    }
-
-    const archiveMatch = pathOnly.match(/^\/api\/conversations\/([^/]+)\/archive$/);
-    if (archiveMatch && req.method === 'POST') {
-      const conv = queries.archiveConversation(archiveMatch[1]);
-      if (!conv) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
-      broadcastSync({ type: 'conversation_deleted', conversationId: archiveMatch[1] });
-      sendJSON(req, res, 200, { conversation: conv });
-      return;
-    }
-
-    const restoreMatch = pathOnly.match(/^\/api\/conversations\/([^/]+)\/restore$/);
-    if (restoreMatch && req.method === 'POST') {
-      const conv = queries.restoreConversation(restoreMatch[1]);
-      if (!conv) { sendJSON(req, res, 404, { error: 'Not found' }); return; }
-      broadcastSync({ type: 'conversation_created', conversation: conv });
-      sendJSON(req, res, 200, { conversation: conv });
-      return;
-    }
+    const convHandler = _convRoutes._match(req.method, pathOnly);
+    if (convHandler) { await convHandler(req, res); return; }
 
     const messagesMatch = pathOnly.match(/^\/api\/conversations\/([^/]+)\/messages$/);
     if (messagesMatch) {
@@ -2850,6 +2765,7 @@ const _utilRoutes = registerUtilRoutes({ sendJSON, parseBody, queries, STARTUP_C
 const _toolRoutes = registerToolRoutes({ sendJSON, parseBody, queries, broadcastSync, logError, toolManager });
 const _threadRoutes = registerThreadRoutes({ sendJSON, parseBody, queries });
 const _debugRoutes = registerDebugRoutes({ sendJSON, queries, activeExecutions, messageQueues, syncClients, wsOptimizer, _errLogPath });
+const _convRoutes = registerConvRoutes({ sendJSON, parseBody, queries, activeExecutions, broadcastSync });
 
 registerConvHandlers(wsRouter, {
   queries, activeExecutions, rateLimitState,