Fix duplicate SetEnabled call and improve macOS Python detection

Author: dsarno

MCPForUnity/Editor/Dependencies/PlatformDetectors/MacOSPlatformDetector.cs

@@ -25,59 +25,31 @@ public override DependencyStatus DetectPython()
 
             try
             {
-                // 1. Check explicit common Homebrew/system paths first to avoid PATH ambiguity
-                // This prioritizes Homebrew versions (often newer) over system versions
-                var candidatePaths = new[]
-                {
-                    "/opt/homebrew/bin/python3",
-                    "/usr/local/bin/python3",
-                    "/opt/homebrew/bin/python3.13",
-                    "/opt/homebrew/bin/python3.12",
-                    "/opt/homebrew/bin/python3.11",
-                    "/opt/homebrew/bin/python3.10",
-                    "/usr/local/bin/python3.13",
-                    "/usr/local/bin/python3.12",
-                    "/usr/local/bin/python3.11",
-                    "/usr/local/bin/python3.10"
-                };
-
-                foreach (var path in candidatePaths)
+                // 1. Try 'which' command with augmented PATH (prioritizing Homebrew)
+                if (TryFindInPath("python3", out string pathResult) ||
+                    TryFindInPath("python", out pathResult))
                 {
-                    if (File.Exists(path) && TryValidatePython(path, out string v, out string p))
+                    if (TryValidatePython(pathResult, out string version, out string fullPath))
                     {
                         status.IsAvailable = true;
-                        status.Version = v;
-                        status.Path = p;
-                        status.Details = $"Found Python {v} at {p}";
+                        status.Version = version;
+                        status.Path = fullPath;
+                        status.Details = $"Found Python {version} at {fullPath}";
                         return status;
                     }
                 }
 
-                // 2. Try running python directly from PATH (fallback)
-                if (TryValidatePython("python3", out string version, out string fullPath) ||
-                    TryValidatePython("python", out version, out fullPath))
+                // 2. Fallback: Try running python directly from PATH
+                if (TryValidatePython("python3", out string v, out string p) ||
+                    TryValidatePython("python", out v, out p))
                 {
                     status.IsAvailable = true;
-                    status.Version = version;
-                    status.Path = fullPath;
-                    status.Details = $"Found Python {version} in PATH";
+                    status.Version = v;
+                    status.Path = p;
+                    status.Details = $"Found Python {v} in PATH";
                     return status;
                 }
 
-                // 3. Fallback: try 'which' command
-                if (TryFindInPath("python3", out string pathResult) ||
-                    TryFindInPath("python", out pathResult))
-                {
-                    if (TryValidatePython(pathResult, out version, out fullPath))
-                    {
-                        status.IsAvailable = true;
-                        status.Version = version;
-                        status.Path = fullPath;
-                        status.Details = $"Found Python {version} in PATH";
-                        return status;
-                    }
-                }
-
                 status.ErrorMessage = "Python not found in PATH or standard locations";
                 status.Details = "Install Python 3.10+ via Homebrew ('brew install python3') and ensure it's in your PATH.";
             }

MCPForUnity/Editor/Windows/Components/Connection/McpConnectionSection.cs

@@ -187,8 +187,6 @@ public void UpdateConnectionStatus()
 
                 unityPortField.SetEnabled(true);
 
-                unityPortField.SetEnabled(true);
-
                 healthStatusLabel.text = HealthStatusUnknown;
                 healthIndicator.RemoveFromClassList("healthy");
                 healthIndicator.RemoveFromClassList("warning");