Fix issue where stray MCP connection state is left after closing connection (#292)

cmsparks · web-flow · commit aacf8375ccaf · 2025-05-21T09:46:35.000+05:30
diff --git a/.changeset/rude-glasses-know.md b/.changeset/rude-glasses-know.md
@@ -0,0 +1,5 @@
+---
+"agents": patch
+---
+
+Fix issue where stray MCP connection state is left after closing connection
diff --git a/packages/agents/src/mcp/client.ts b/packages/agents/src/mcp/client.ts
@@ -232,6 +232,7 @@ export class MCPClientManager {
       throw new Error(`Connection with id "${id}" does not exist.`);
     }
     await this.mcpConnections[id].client.close();
+    delete this.mcpConnections[id];
   }
 
   /**

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"agents": patch
 +---
++
 +Fix issue where stray MCP connection state is left after closing connection
Original file line number	Diff line number	Diff line change
`@@ -232,6 +232,7 @@ export class MCPClientManager {`
`232`	`232`	throw new Error(`Connection with id "${id}" does not exist.`);
`233`	`233`	`}`
`234`	`234`	`await this.mcpConnections[id].client.close();`
	`235`	`+ delete this.mcpConnections[id];`
`235`	`236`	`}`
`236`	`237`
`237`	`238`	`/**`