From 88cb60efe1463f66c929b852eff44806ebff05e9 Mon Sep 17 00:00:00 2001
From: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com>
Date: Wed, 11 Dec 2024 16:18:43 -0800
Subject: [PATCH] Fix server restart UX

---
 src/services/mcp/McpHub.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/services/mcp/McpHub.ts b/src/services/mcp/McpHub.ts
index 53e2ff3..9a5677b 100644
--- a/src/services/mcp/McpHub.ts
+++ b/src/services/mcp/McpHub.ts
@@ -222,7 +222,10 @@ export class McpHub {
 					if (connection) {
 						// NOTE: we do not set server status to "disconnected" because stderr logs do not necessarily mean the server crashed or disconnected, it could just be informational. In fact when the server first starts up, it immediately logs "<name> server running on stdio" to stderr.
 						this.appendErrorMessage(connection, errorOutput)
-						await this.notifyWebviewOfServerChanges()
+						// Only need to update webview right away if it's already disconnected
+						if (connection.server.status === "disconnected") {
+							await this.notifyWebviewOfServerChanges()
+						}
 					}
 				})
 			} else {