From 80e1173ef7907e978e36314a0d936de418be2903 Mon Sep 17 00:00:00 2001
From: Daniel Sauer <81422812+sauerdaniel@users.noreply.github.com>
Date: Tue, 13 Jan 2026 19:38:34 +0100
Subject: [PATCH] fix(mcp): close existing client before reassignment to
 prevent leaks (#8253)

---
 packages/opencode/src/mcp/index.ts | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/packages/opencode/src/mcp/index.ts b/packages/opencode/src/mcp/index.ts
index aca0c6631..4e0968391 100644
--- a/packages/opencode/src/mcp/index.ts
+++ b/packages/opencode/src/mcp/index.ts
@@ -266,6 +266,13 @@ export namespace MCP {
         status: s.status,
       }
     }
+    // Close existing client if present to prevent memory leaks
+    const existingClient = s.clients[name]
+    if (existingClient) {
+      await existingClient.close().catch((error) => {
+        log.error("Failed to close existing MCP client", { name, error })
+      })
+    }
     s.clients[name] = result.mcpClient
     s.status[name] = result.status
 
@@ -523,6 +530,13 @@ export namespace MCP {
     const s = await state()
     s.status[name] = result.status
     if (result.mcpClient) {
+      // Close existing client if present to prevent memory leaks
+      const existingClient = s.clients[name]
+      if (existingClient) {
+        await existingClient.close().catch((error) => {
+          log.error("Failed to close existing MCP client", { name, error })
+        })
+      }
       s.clients[name] = result.mcpClient
     }
   }