fix: only check MCP configuration for selected client on window focus

msanatan · msanatan · commit 80b7d811b0f3 · 2025-08-01T23:11:04.000-04:00
diff --git a/UnityMcpBridge/Editor/Windows/UnityMcpEditorWindow.cs b/UnityMcpBridge/Editor/Windows/UnityMcpEditorWindow.cs
@@ -58,10 +58,10 @@ private void OnEnable()
         
         private void OnFocus()
         {
-            // Refresh configuration status when window gains focus
-            foreach (McpClient mcpClient in mcpClients.clients)
+            if (mcpClients.clients.Count > 0 && selectedClientIndex < mcpClients.clients.Count)
             {
-                CheckMcpConfiguration(mcpClient);
+                McpClient selectedClient = mcpClients.clients[selectedClientIndex];
+                CheckMcpConfiguration(selectedClient);
             }
             Repaint();
         }

Original file line number	Diff line number	Diff line change
`@@ -58,10 +58,10 @@ private void OnEnable()`
`58`	`58`
`59`	`59`	`private void OnFocus()`
`60`	`60`	`{`
`61`		`- // Refresh configuration status when window gains focus`
`62`		`- foreach (McpClient mcpClient in mcpClients.clients)`
	`61`	`+ if (mcpClients.clients.Count > 0 && selectedClientIndex < mcpClients.clients.Count)`
`63`	`62`	`{`
`64`		`- CheckMcpConfiguration(mcpClient);`
	`63`	`+ McpClient selectedClient = mcpClients.clients[selectedClientIndex];`
	`64`	`+ CheckMcpConfiguration(selectedClient);`
`65`	`65`	`}`
`66`	`66`	`Repaint();`
`67`	`67`	`}`