Start MCP server after binary download

Wait for DownloadServerBinaryIfNeeded() to complete before scheduling the MCP server auto-start. Adds a ContinueWith-based check and a new ActionStartServerIfNeeded helper that subscribes to EditorApplication.update only when binaries are available (skipping auto-start in CI or on download failure). Moves the auto-start logic into McpServerManager and removes the previous EnvironmentUtils.IsCi gating and delayed ConnectIfNeeded scheduling in Startup to centralize async startup and avoid blocking during domain reloads.

Author: IvanMurzak

Unity-MCP-Plugin/Assets/root/Editor/Scripts/McpServerManager.cs

@@ -69,10 +69,25 @@ static McpServerManager()
             // Register for editor quit to clean up the server process
             EditorApplication.quitting += OnEditorQuitting;
 
+            DownloadServerBinaryIfNeeded()
+                .ContinueWith(task =>
+                {
+                    if (!task.Result)
+                        return; // No binaries available (either in CI or failed to download), skip auto-start
+
+                    EditorApplication.update += ActionStartServerIfNeeded;
+                });
+
             // Check if server process is still running (e.g., after domain reload)
             CheckExistingProcess();
         }
 
+        static void ActionStartServerIfNeeded()
+        {
+            EditorApplication.update -= ActionStartServerIfNeeded;
+            StartServerIfNeeded();
+        }
+
         #region Binary Metadata
 
         public const string ExecutableName = "unity-mcp-server";

Unity-MCP-Plugin/Assets/root/Editor/Scripts/Startup.cs

@@ -28,19 +28,14 @@ static Startup()
             UnityMcpPlugin.Instance.BuildMcpPluginIfNeeded();
             UnityMcpPlugin.Instance.AddUnityLogCollectorIfNeeded(() => new BufferedFileLogStorage());
 
-            // Defer connection to avoid blocking during domain reload.
-            // Starting async SignalR connections during [InitializeOnLoad] can cause
-            // Unity to freeze because async continuations may run on the main thread
-            // while it's still processing the domain reload.
-            if (!EnvironmentUtils.IsCi())
-                EditorApplication.delayCall += () => UnityMcpPlugin.ConnectIfNeeded();
-
-            McpServerManager.DownloadServerBinaryIfNeeded();
-
-            // Defer MCP server auto-start to avoid blocking during domain reload
-            // and to ensure configuration is fully loaded
-            if (!EnvironmentUtils.IsCi())
-                EditorApplication.update += ActionStartServerIfNeeded;
+            McpServerManager.DownloadServerBinaryIfNeeded()
+                .ContinueWith(task =>
+                {
+                    if (!task.Result)
+                        return; // No binaries available (either in CI or already exist), skip auto-start
+
+                    EditorApplication.update += ActionStartServerIfNeeded;
+                });
 
             if (Application.dataPath.Contains(" "))
                 Debug.LogError("The project path contains spaces, which may cause issues during usage of AI Game Developer. Please consider the move the project to a folder without spaces.");