Merge branch 'RooCodeInc:main' into webview-leaks

Author: xyOz-dev

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Roo Code Changelog
 
+## [3.19.3] - 2025-06-02
+
+- Fix SSE MCP Invocation - Fixed SSE connection issue in McpHub.ts by ensuring transport.start override only applies to stdio transports, allowing SSE and streamable-http transports to retain their original start methods (thanks @taylorwilsdon!)
+
 ## [3.19.2] - 2025-06-01
 
 - Add support for Streamable HTTP Transport MCP servers (thanks @taylorwilsdon!)

src/package.json

@@ -3,7 +3,7 @@
 	"displayName": "%extension.displayName%",
 	"description": "%extension.description%",
 	"publisher": "RooVeterinaryInc",
-	"version": "3.19.2",
+	"version": "3.19.3",
 	"icon": "assets/icons/icon.png",
 	"galleryBanner": {
 		"color": "#617A91",

src/services/mcp/McpHub.ts

@@ -598,13 +598,23 @@ export class McpHub {
 					}
 					await this.notifyWebviewOfServerChanges()
 				}
+
+				transport.onclose = async () => {
+					const connection = this.findConnection(name, source)
+					if (connection) {
+						connection.server.status = "disconnected"
+					}
+					await this.notifyWebviewOfServerChanges()
+				}
 			} else {
-				// Correctly placed "unsupported type" else block
 				// Should not happen if validateServerConfig is correct
 				throw new Error(`Unsupported MCP server type: ${(configInjected as any).type}`)
 			}
-			// transport.start assignment moved after all type-specific initializations
-			transport.start = async () => {} // No-op now, .connect() won't fail
+
+			// Only override transport.start for stdio transports that have already been started
+			if (configInjected.type === "stdio") {
+				transport.start = async () => {}
+			}
 
 			const connection: McpConnection = {
 				server: {