From 4dce90116652c15c1f883c138d3ee3ecf29a2250 Mon Sep 17 00:00:00 2001 From: Amrit Subramanian Date: Tue, 7 Oct 2025 23:20:26 -0400 Subject: [PATCH] Remove unused sessionId parameter from MCP handlers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Since MCP commands now run in independent login shells via execAsync, the sessionId parameter is no longer needed. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- main.ts | 6 +++--- renderer.ts | 8 ++++---- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/main.ts b/main.ts index 721a4f3..af2e5fe 100644 --- a/main.ts +++ b/main.ts @@ -704,7 +704,7 @@ ipcMain.handle("list-mcp-servers", async (_event, sessionId: string) => { } }); -ipcMain.handle("add-mcp-server", async (_event, _sessionId: string, name: string, config: any) => { +ipcMain.handle("add-mcp-server", async (_event, name: string, config: any) => { try { await addMcpServer(name, config); } catch (error) { @@ -713,7 +713,7 @@ ipcMain.handle("add-mcp-server", async (_event, _sessionId: string, name: string } }); -ipcMain.handle("remove-mcp-server", async (_event, _sessionId: string, name: string) => { +ipcMain.handle("remove-mcp-server", async (_event, name: string) => { try { await removeMcpServer(name); } catch (error) { @@ -722,7 +722,7 @@ ipcMain.handle("remove-mcp-server", async (_event, _sessionId: string, name: str } }); -ipcMain.handle("get-mcp-server-details", async (_event, _sessionId: string, name: string) => { +ipcMain.handle("get-mcp-server-details", async (_event, name: string) => { try { return await getMcpServerDetails(name); } catch (error) { diff --git a/renderer.ts b/renderer.ts index 195b14d..c593187 100644 --- a/renderer.ts +++ b/renderer.ts @@ -949,7 +949,7 @@ function renderMcpServers() { e.stopPropagation(); if (confirm(`Remove MCP server "${server.name}"?`)) { try { - await ipcRenderer.invoke("remove-mcp-server", activeSessionId, server.name); + await ipcRenderer.invoke("remove-mcp-server", server.name); await loadMcpServers(); } catch (error) { alert(`Failed to remove server: ${error}`); @@ -978,7 +978,7 @@ async function showMcpServerDetails(name: string) { detailsModal?.classList.remove("hidden"); try { - const details = await ipcRenderer.invoke("get-mcp-server-details", activeSessionId, name); + const details = await ipcRenderer.invoke("get-mcp-server-details", name); if (detailsContent) { let html = ""; @@ -1139,7 +1139,7 @@ addMcpBtn?.addEventListener("click", async () => { addMcpBtn.classList.add("opacity-50", "cursor-not-allowed"); try { - await ipcRenderer.invoke("add-mcp-server", activeSessionId, name, config); + await ipcRenderer.invoke("add-mcp-server", name, config); await loadMcpServers(); mcpModal?.classList.add("hidden"); } catch (error) { @@ -1168,7 +1168,7 @@ removeMcpDetailsBtn?.addEventListener("click", async () => { if (confirm(`Remove MCP server "${serverName}"?`)) { try { - await ipcRenderer.invoke("remove-mcp-server", activeSessionId, serverName); + await ipcRenderer.invoke("remove-mcp-server", serverName); mcpDetailsModal?.classList.add("hidden"); await loadMcpServers(); } catch (error) {