Remove old UI and do lots of cleanup (#340)

* Remove legacy UI and correct priority ordering of menu items

* Remove old UI screen

Users now have the new UI alone, less confusing and more predictable

* Remove unused config files

* Remove test for window that doesn't exist

* Remove unused code

* Remove dangling .meta file

* refactor: remove client configuration step from setup wizard

* refactor: remove menu item attributes and manual window actions from Python tool sync

* feat: update minimum Python version requirement from 3.10 to 3.11

The docs have 3.12. However, feature wise it seems that 3.11 is required

* fix: replace emoji warning symbol with unicode character in setup wizard dialogs

* docs: reorganize images into docs/images directory and update references

* docs: add UI preview image to README

* docs: add run_test function and resources section to available tools list

The recent changes should close #311

* fix: add SystemRoot env var to Windows config to support Python path resolution

Closes #315

* refactor: consolidate package installation and detection into unified lifecycle manager

Duplicate code for pretty much no reason, as they both initialized there was a small chance of a race condition as well. Consolidating made sense here

* Doc fixes from CodeRabbit

* Excellent bug catch from CodeRabbit

* fix: preserve existing environment variables when updating codex server config

* Update docs so the paths match the original name

* style: fix list indentation in README-DEV.md development docs

* refactor: simplify env table handling in CodexConfigHelper by removing preservation logic

* refactor: simplify configuration logic by removing redundant change detection

Always overwrite configs

* feat: ensure config directory exists before writing config files

* feat: persist server installation errors and show retry UI instead of auto-marking as handled

* refactor: consolidate configuration helpers by merging McpConfigFileHelper into McpConfigurationHelper

* Small fixes from CodeRabbit

* Remove test because we overwrite Codex configs

* Remove unused function

* feat: improve server cleanup and process handling on Windows

- Added DeleteDirectoryWithRetry helper to handle Windows file locking with retries and readonly attribute clearing
- Implemented KillWindowsUvProcesses to safely terminate Python processes in virtual environments using WMIC
- Extended TryKillUvForPath to work on Windows, preventing file handle locks during server deletion
- Improved error messages to be more descriptive about file locking issues
- Replaced direct Directory.Delete calls with

* fix: improve TCP socket cleanup to prevent CLOSE_WAIT states

- Added proper socket shutdown sequence using Socket.Shutdown() before closing connections
- Enhanced error handling with specific catches for SocketException vs general exceptions
- Added debug logging for socket shutdown errors to help diagnose connection issues
- Restructured HandleClientAsync to ensure socket cleanup happens in the correct order
- Implemented proper socket teardown in both client handling and connection cleanup paths

Author: msanatan

MCPForUnity/Editor/Data/DefaultServerConfig.cs

@@ -126,7 +126,7 @@ private static void GenerateRecommendations(DependencyCheckResult result, IPlatf
             {
                 if (dep.Name == "Python")
                 {
-                    result.RecommendedActions.Add($"Install Python 3.10+ from: {detector.GetPythonInstallUrl()}");
+                    result.RecommendedActions.Add($"Install Python 3.11+ from: {detector.GetPythonInstallUrl()}");
                 }
                 else if (dep.Name == "UV Package Manager")
                 {

MCPForUnity/Editor/Dependencies/DependencyManager.cs

@@ -62,7 +62,7 @@ public override DependencyStatus DetectPython()
                     }
                 }
 
-                status.ErrorMessage = "Python not found. Please install Python 3.10 or later.";
+                status.ErrorMessage = "Python not found. Please install Python 3.11 or later.";
                 status.Details = "Checked common installation paths including system, snap, and user-local locations.";
             }
             catch (Exception ex)
@@ -144,10 +144,10 @@ private bool TryValidatePython(string pythonPath, out string version, out string
                     version = output.Substring(7); // Remove "Python " prefix
                     fullPath = pythonPath;
 
-                    // Validate minimum version (Python 4+ or Python 3.10+)
+                    // Validate minimum version (Python 4+ or Python 3.11+)
                     if (TryParseVersion(version, out var major, out var minor))
                     {
-                        return major > 3 || (major >= 3 && minor >= 10);
+                        return major > 3 || (major >= 3 && minor >= 11);
                     }
                 }
             }

MCPForUnity/Editor/Dependencies/PlatformDetectors/LinuxPlatformDetector.cs

@@ -35,8 +35,7 @@ public override DependencyStatus DetectPython()
                     "/opt/homebrew/bin/python3",
                     "/Library/Frameworks/Python.framework/Versions/3.13/bin/python3",
                     "/Library/Frameworks/Python.framework/Versions/3.12/bin/python3",
-                    "/Library/Frameworks/Python.framework/Versions/3.11/bin/python3",
-                    "/Library/Frameworks/Python.framework/Versions/3.10/bin/python3"
+                    "/Library/Frameworks/Python.framework/Versions/3.11/bin/python3"
                 };
 
                 foreach (var candidate in candidates)
@@ -65,7 +64,7 @@ public override DependencyStatus DetectPython()
                     }
                 }
 
-                status.ErrorMessage = "Python not found. Please install Python 3.10 or later.";
+                status.ErrorMessage = "Python not found. Please install Python 3.11 or later.";
                 status.Details = "Checked common installation paths including Homebrew, Framework, and system locations.";
             }
             catch (Exception ex)
@@ -144,10 +143,10 @@ private bool TryValidatePython(string pythonPath, out string version, out string
                     version = output.Substring(7); // Remove "Python " prefix
                     fullPath = pythonPath;
 
-                    // Validate minimum version (Python 4+ or Python 3.10+)
+                    // Validate minimum version (Python 4+ or Python 3.11+)
                     if (TryParseVersion(version, out var major, out var minor))
                     {
-                        return major > 3 || (major >= 3 && minor >= 10);
+                        return major > 3 || (major >= 3 && minor >= 11);
                     }
                 }
             }

MCPForUnity/Editor/Dependencies/PlatformDetectors/MacOSPlatformDetector.cs

@@ -68,7 +68,7 @@ public override DependencyStatus DetectPython()
                     }
                 }
 
-                status.ErrorMessage = "Python not found. Please install Python 3.10 or later.";
+                status.ErrorMessage = "Python not found. Please install Python 3.11 or later.";
                 status.Details = "Checked common installation paths and PATH environment variable.";
             }
             catch (Exception ex)
@@ -132,10 +132,10 @@ private bool TryValidatePython(string pythonPath, out string version, out string
                     version = output.Substring(7); // Remove "Python " prefix
                     fullPath = pythonPath;
 
-                    // Validate minimum version (Python 4+ or Python 3.10+)
+                    // Validate minimum version (Python 4+ or Python 3.11+)
                     if (TryParseVersion(version, out var major, out var minor))
                     {
-                        return major > 3 || (major >= 3 && minor >= 10);
+                        return major > 3 || (major >= 3 && minor >= 11);
                     }
                 }
             }

MCPForUnity/Editor/Dependencies/PlatformDetectors/WindowsPlatformDetector.cs

@@ -3,6 +3,7 @@
 using System.IO;
 using System.Linq;
 using MCPForUnity.External.Tommy;
+using MCPForUnity.Editor.Services;
 
 namespace MCPForUnity.Editor.Helpers
 {
@@ -26,10 +27,10 @@ public static bool IsCodexConfigured(string pythonDir)
                 string toml = File.ReadAllText(configPath);
                 if (!TryParseCodexServer(toml, out _, out var args)) return false;
 
-                string dir = McpConfigFileHelper.ExtractDirectoryArg(args);
+                string dir = McpConfigurationHelper.ExtractDirectoryArg(args);
                 if (string.IsNullOrEmpty(dir)) return false;
 
-                return McpConfigFileHelper.PathsEqual(dir, pythonDir);
+                return McpConfigurationHelper.PathsEqual(dir, pythonDir);
             }
             catch
             {
@@ -125,6 +126,8 @@ private static TomlTable TryParseToml(string toml)
         /// <summary>
         /// Creates a TomlTable for the unityMCP server configuration
         /// </summary>
+        /// <param name="uvPath">Path to uv executable</param>
+        /// <param name="serverSrc">Path to server source directory</param>
         private static TomlTable CreateUnityMcpTable(string uvPath, string serverSrc)
         {
             var unityMCP = new TomlTable();
@@ -137,6 +140,15 @@ private static TomlTable CreateUnityMcpTable(string uvPath, string serverSrc)
             argsArray.Add(new TomlString { Value = "server.py" });
             unityMCP["args"] = argsArray;
 
+            // Add Windows-specific environment configuration, see: https://github.com/CoplayDev/unity-mcp/issues/315
+            var platformService = MCPServiceLocator.Platform;
+            if (platformService.IsWindows())
+            {
+                var envTable = new TomlTable { IsInline = true };
+                envTable["SystemRoot"] = new TomlString { Value = platformService.GetSystemRoot() };
+                unityMCP["env"] = envTable;
+            }
+
             return unityMCP;
         }