Remove unused sessionId parameter from MCP handlers

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 <noreply@anthropic.com>
This commit is contained in:
Amrit Subramanian 2025-10-07 23:20:26 -04:00
parent d687523e4d
commit 4dce901166
2 changed files with 7 additions and 7 deletions

View file

@ -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 { try {
await addMcpServer(name, config); await addMcpServer(name, config);
} catch (error) { } 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 { try {
await removeMcpServer(name); await removeMcpServer(name);
} catch (error) { } 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 { try {
return await getMcpServerDetails(name); return await getMcpServerDetails(name);
} catch (error) { } catch (error) {

View file

@ -949,7 +949,7 @@ function renderMcpServers() {
e.stopPropagation(); e.stopPropagation();
if (confirm(`Remove MCP server "${server.name}"?`)) { if (confirm(`Remove MCP server "${server.name}"?`)) {
try { try {
await ipcRenderer.invoke("remove-mcp-server", activeSessionId, server.name); await ipcRenderer.invoke("remove-mcp-server", server.name);
await loadMcpServers(); await loadMcpServers();
} catch (error) { } catch (error) {
alert(`Failed to remove server: ${error}`); alert(`Failed to remove server: ${error}`);
@ -978,7 +978,7 @@ async function showMcpServerDetails(name: string) {
detailsModal?.classList.remove("hidden"); detailsModal?.classList.remove("hidden");
try { try {
const details = await ipcRenderer.invoke("get-mcp-server-details", activeSessionId, name); const details = await ipcRenderer.invoke("get-mcp-server-details", name);
if (detailsContent) { if (detailsContent) {
let html = ""; let html = "";
@ -1139,7 +1139,7 @@ addMcpBtn?.addEventListener("click", async () => {
addMcpBtn.classList.add("opacity-50", "cursor-not-allowed"); addMcpBtn.classList.add("opacity-50", "cursor-not-allowed");
try { try {
await ipcRenderer.invoke("add-mcp-server", activeSessionId, name, config); await ipcRenderer.invoke("add-mcp-server", name, config);
await loadMcpServers(); await loadMcpServers();
mcpModal?.classList.add("hidden"); mcpModal?.classList.add("hidden");
} catch (error) { } catch (error) {
@ -1168,7 +1168,7 @@ removeMcpDetailsBtn?.addEventListener("click", async () => {
if (confirm(`Remove MCP server "${serverName}"?`)) { if (confirm(`Remove MCP server "${serverName}"?`)) {
try { try {
await ipcRenderer.invoke("remove-mcp-server", activeSessionId, serverName); await ipcRenderer.invoke("remove-mcp-server", serverName);
mcpDetailsModal?.classList.add("hidden"); mcpDetailsModal?.classList.add("hidden");
await loadMcpServers(); await loadMcpServers();
} catch (error) { } catch (error) {