Skip to content

Commit 4dce901

Browse files
built-by-asclaude
andcommitted
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 <[email protected]>
1 parent d687523 commit 4dce901

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

main.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,7 @@ ipcMain.handle("list-mcp-servers", async (_event, sessionId: string) => {
704704
}
705705
});
706706

707-
ipcMain.handle("add-mcp-server", async (_event, _sessionId: string, name: string, config: any) => {
707+
ipcMain.handle("add-mcp-server", async (_event, name: string, config: any) => {
708708
try {
709709
await addMcpServer(name, config);
710710
} catch (error) {
@@ -713,7 +713,7 @@ ipcMain.handle("add-mcp-server", async (_event, _sessionId: string, name: string
713713
}
714714
});
715715

716-
ipcMain.handle("remove-mcp-server", async (_event, _sessionId: string, name: string) => {
716+
ipcMain.handle("remove-mcp-server", async (_event, name: string) => {
717717
try {
718718
await removeMcpServer(name);
719719
} catch (error) {
@@ -722,7 +722,7 @@ ipcMain.handle("remove-mcp-server", async (_event, _sessionId: string, name: str
722722
}
723723
});
724724

725-
ipcMain.handle("get-mcp-server-details", async (_event, _sessionId: string, name: string) => {
725+
ipcMain.handle("get-mcp-server-details", async (_event, name: string) => {
726726
try {
727727
return await getMcpServerDetails(name);
728728
} catch (error) {

renderer.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -949,7 +949,7 @@ function renderMcpServers() {
949949
e.stopPropagation();
950950
if (confirm(`Remove MCP server "${server.name}"?`)) {
951951
try {
952-
await ipcRenderer.invoke("remove-mcp-server", activeSessionId, server.name);
952+
await ipcRenderer.invoke("remove-mcp-server", server.name);
953953
await loadMcpServers();
954954
} catch (error) {
955955
alert(`Failed to remove server: ${error}`);
@@ -978,7 +978,7 @@ async function showMcpServerDetails(name: string) {
978978
detailsModal?.classList.remove("hidden");
979979

980980
try {
981-
const details = await ipcRenderer.invoke("get-mcp-server-details", activeSessionId, name);
981+
const details = await ipcRenderer.invoke("get-mcp-server-details", name);
982982

983983
if (detailsContent) {
984984
let html = "";
@@ -1139,7 +1139,7 @@ addMcpBtn?.addEventListener("click", async () => {
11391139
addMcpBtn.classList.add("opacity-50", "cursor-not-allowed");
11401140

11411141
try {
1142-
await ipcRenderer.invoke("add-mcp-server", activeSessionId, name, config);
1142+
await ipcRenderer.invoke("add-mcp-server", name, config);
11431143
await loadMcpServers();
11441144
mcpModal?.classList.add("hidden");
11451145
} catch (error) {
@@ -1168,7 +1168,7 @@ removeMcpDetailsBtn?.addEventListener("click", async () => {
11681168

11691169
if (confirm(`Remove MCP server "${serverName}"?`)) {
11701170
try {
1171-
await ipcRenderer.invoke("remove-mcp-server", activeSessionId, serverName);
1171+
await ipcRenderer.invoke("remove-mcp-server", serverName);
11721172
mcpDetailsModal?.classList.add("hidden");
11731173
await loadMcpServers();
11741174
} catch (error) {

0 commit comments

Comments
 (0)