Fix manage prefabs (#405)

msanatan · web-flow · commit adfc6f5e84aa · 2025-11-28T18:47:11.000-04:00
* Standardize import ordering and whitespace in plugin

The whitespace gave a warning in the asset store submission

* Fix manage_prefab tool structure

* Fix manage_editor actions

* Add get_component singular to manage_gameobject

* Improve uv cache clear error handling with lock detection and combined output

Replace simple stderr-only error reporting with combined stdout/stderr output. Add detection for "currently in-use" lock errors with helpful hint about waiting or using --force flag. Provide fallback message when command fails with no output.

* Improve error message formatting in uv cache clear failure logging
diff --git a/MCPForUnity/Editor/Dependencies/PlatformDetectors/PlatformDetectorBase.cs b/MCPForUnity/Editor/Dependencies/PlatformDetectors/PlatformDetectorBase.cs
@@ -54,7 +54,7 @@ protected bool TryFindUvInPath(out string uvPath, out string version)
 
             // Try common uv command names
             var commands = new[] { "uvx", "uv" };
-            
+
             foreach (var cmd in commands)
             {
                 try
diff --git a/MCPForUnity/Editor/Helpers/CodexConfigHelper.cs b/MCPForUnity/Editor/Helpers/CodexConfigHelper.cs
@@ -2,8 +2,8 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
-using MCPForUnity.External.Tommy;
 using MCPForUnity.Editor.Services;
+using MCPForUnity.External.Tommy;
 using UnityEditor;
 
 namespace MCPForUnity.Editor.Helpers
diff --git a/MCPForUnity/Editor/Helpers/ConfigJsonBuilder.cs b/MCPForUnity/Editor/Helpers/ConfigJsonBuilder.cs
@@ -1,10 +1,10 @@
-using Newtonsoft.Json;
 using System;
 using System.Collections.Generic;
-using Newtonsoft.Json.Linq;
-using MCPForUnity.Editor.Models;
-using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Constants;
+using MCPForUnity.Editor.Helpers;
+using MCPForUnity.Editor.Models;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
 using UnityEditor;
 
 namespace MCPForUnity.Editor.Helpers
diff --git a/MCPForUnity/Editor/Helpers/ExecPath.cs b/MCPForUnity/Editor/Helpers/ExecPath.cs
@@ -2,10 +2,10 @@
 using System.Diagnostics;
 using System.IO;
 using System.Linq;
-using System.Text;
 using System.Runtime.InteropServices;
-using UnityEditor;
+using System.Text;
 using MCPForUnity.Editor.Constants;
+using UnityEditor;
 
 namespace MCPForUnity.Editor.Helpers
 {
diff --git a/MCPForUnity/Editor/Helpers/GameObjectSerializer.cs b/MCPForUnity/Editor/Helpers/GameObjectSerializer.cs
@@ -2,11 +2,11 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
+using MCPForUnity.Runtime.Serialization; // For Converters
 using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using UnityEditor;
 using UnityEngine;
-using MCPForUnity.Runtime.Serialization; // For Converters
 
 namespace MCPForUnity.Editor.Helpers
 {
diff --git a/MCPForUnity/Editor/Helpers/HttpEndpointUtility.cs b/MCPForUnity/Editor/Helpers/HttpEndpointUtility.cs
@@ -1,7 +1,7 @@
 using System;
-using UnityEditor;
-using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Config;
+using MCPForUnity.Editor.Constants;
+using UnityEditor;
 
 namespace MCPForUnity.Editor.Helpers
 {
diff --git a/MCPForUnity/Editor/Helpers/McpLog.cs b/MCPForUnity/Editor/Helpers/McpLog.cs
@@ -1,6 +1,6 @@
+using MCPForUnity.Editor.Constants;
 using UnityEditor;
 using UnityEngine;
-using MCPForUnity.Editor.Constants;
 
 namespace MCPForUnity.Editor.Helpers
 {
diff --git a/MCPForUnity/Editor/Helpers/PortManager.cs b/MCPForUnity/Editor/Helpers/PortManager.cs
@@ -1,14 +1,14 @@
 using System;
 using System.IO;
-using UnityEditor;
 using System.Net;
 using System.Net.Sockets;
 using System.Security.Cryptography;
 using System.Text;
 using System.Threading;
+using MCPForUnity.Editor.Constants;
 using Newtonsoft.Json;
+using UnityEditor;
 using UnityEngine;
-using MCPForUnity.Editor.Constants;
 
 namespace MCPForUnity.Editor.Helpers
 {
diff --git a/MCPForUnity/Editor/Helpers/Response.cs b/MCPForUnity/Editor/Helpers/Response.cs
@@ -2,8 +2,8 @@
 
 namespace MCPForUnity.Editor.Helpers
 {
-public interface IMcpResponse
-{
+    public interface IMcpResponse
+    {
         [JsonProperty("success")]
         bool Success { get; }
     }
diff --git a/MCPForUnity/Editor/Helpers/TelemetryHelper.cs b/MCPForUnity/Editor/Helpers/TelemetryHelper.cs
@@ -1,9 +1,9 @@
 using System;
 using System.Collections.Generic;
 using System.Threading;
-using UnityEngine;
 using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Services.Transport.Transports;
+using UnityEngine;
 
 namespace MCPForUnity.Editor.Helpers
 {
diff --git a/MCPForUnity/Editor/Migrations/LegacyServerSrcMigration.cs b/MCPForUnity/Editor/Migrations/LegacyServerSrcMigration.cs
@@ -1,7 +1,7 @@
 using System;
+using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Services;
-using MCPForUnity.Editor.Constants;
 using UnityEditor;
 using UnityEngine;
 
diff --git a/MCPForUnity/Editor/Migrations/StdIoVersionMigration.cs b/MCPForUnity/Editor/Migrations/StdIoVersionMigration.cs
@@ -1,14 +1,14 @@
 using System;
 using System.IO;
+using System.Linq;
 using MCPForUnity.Editor.Clients;
+using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Models;
 using MCPForUnity.Editor.Services;
 using Newtonsoft.Json.Linq;
 using UnityEditor;
 using UnityEngine;
-using MCPForUnity.Editor.Constants;
-using System.Linq;
 
 namespace MCPForUnity.Editor.Migrations
 {
diff --git a/MCPForUnity/Editor/Resources/Tests/GetTests.cs b/MCPForUnity/Editor/Resources/Tests/GetTests.cs
@@ -1,9 +1,9 @@
 using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
-using Newtonsoft.Json.Linq;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Services;
+using Newtonsoft.Json.Linq;
 using UnityEditor.TestTools.TestRunner.Api;
 
 namespace MCPForUnity.Editor.Resources.Tests
diff --git a/MCPForUnity/Editor/Services/BridgeControlService.cs b/MCPForUnity/Editor/Services/BridgeControlService.cs
@@ -1,11 +1,11 @@
 
 using System;
 using System.Threading.Tasks;
-using UnityEditor;
 using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Services.Transport;
 using MCPForUnity.Editor.Services.Transport.Transports;
+using UnityEditor;
 
 namespace MCPForUnity.Editor.Services
 {
diff --git a/MCPForUnity/Editor/Services/HttpBridgeReloadHandler.cs b/MCPForUnity/Editor/Services/HttpBridgeReloadHandler.cs
@@ -1,10 +1,10 @@
 using System;
 using System.Threading.Tasks;
-using UnityEditor;
 using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Services.Transport;
 using MCPForUnity.Editor.Windows;
+using UnityEditor;
 
 namespace MCPForUnity.Editor.Services
 {
diff --git a/MCPForUnity/Editor/Services/PackageUpdateService.cs b/MCPForUnity/Editor/Services/PackageUpdateService.cs
@@ -1,9 +1,9 @@
 using System;
 using System.Net;
+using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using Newtonsoft.Json.Linq;
 using UnityEditor;
-using MCPForUnity.Editor.Constants;
 
 namespace MCPForUnity.Editor.Services
 {
diff --git a/MCPForUnity/Editor/Services/ServerManagementService.cs b/MCPForUnity/Editor/Services/ServerManagementService.cs
@@ -41,15 +41,24 @@ public bool ClearUvxCache()
                     McpLog.Debug($"uv cache cleared successfully: {stdout}");
                     return true;
                 }
-                else
-                {
-                    string errorMessage = string.IsNullOrEmpty(stderr)
-                        ? "Unknown error"
-                        : stderr;
+                string combinedOutput = string.Join(
+                    Environment.NewLine,
+                    new[] { stderr, stdout }.Where(s => !string.IsNullOrWhiteSpace(s)).Select(s => s.Trim()));
 
-                    McpLog.Error($"Failed to clear uv cache using '{uvCommand} {args}': {errorMessage}. Ensure uv is installed, available on PATH, or set an override in Advanced Settings.");
-                    return false;
+                string lockHint = (!string.IsNullOrEmpty(combinedOutput) &&
+                                   combinedOutput.IndexOf("currently in-use", StringComparison.OrdinalIgnoreCase) >= 0)
+                    ? "Another uv process may be holding the cache lock; wait a moment and try again or clear with '--force' from a terminal."
+                    : string.Empty;
+
+                if (string.IsNullOrEmpty(combinedOutput))
+                {
+                    combinedOutput = "Command failed with no output. Ensure uv is installed, on PATH, or set an override in Advanced Settings.";
                 }
+
+                McpLog.Error(
+                    $"Failed to clear uv cache using '{uvCommand} {args}'. " +
+                    $"Details: {combinedOutput}{(string.IsNullOrEmpty(lockHint) ? string.Empty : " Hint: " + lockHint)}");
+                return false;
             }
             catch (Exception ex)
             {
diff --git a/MCPForUnity/Editor/Services/StdioBridgeReloadHandler.cs b/MCPForUnity/Editor/Services/StdioBridgeReloadHandler.cs
@@ -1,9 +1,9 @@
 using System;
-using UnityEditor;
 using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Services.Transport;
 using MCPForUnity.Editor.Services.Transport.Transports;
+using UnityEditor;
 
 namespace MCPForUnity.Editor.Services
 {
diff --git a/MCPForUnity/Editor/Services/Transport/Transports/StdioBridgeHost.cs b/MCPForUnity/Editor/Services/Transport/Transports/StdioBridgeHost.cs
@@ -1,22 +1,22 @@
 using System;
-using System.Collections.Generic;
 using System.Collections.Concurrent;
+using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 using System.Net;
 using System.Net.Sockets;
 using System.Threading;
 using System.Threading.Tasks;
-using Newtonsoft.Json;
-using Newtonsoft.Json.Linq;
-using UnityEditor;
-using UnityEngine;
 using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Models;
+using MCPForUnity.Editor.Services.Transport;
 using MCPForUnity.Editor.Tools;
 using MCPForUnity.Editor.Tools.Prefabs;
-using MCPForUnity.Editor.Services.Transport;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using UnityEditor;
+using UnityEngine;
 
 namespace MCPForUnity.Editor.Services.Transport.Transports
 {
diff --git a/MCPForUnity/Editor/Setup/SetupWindowService.cs b/MCPForUnity/Editor/Setup/SetupWindowService.cs
@@ -1,10 +1,10 @@
 using System;
+using MCPForUnity.Editor.Config;
+using MCPForUnity.Editor.Constants;
 using MCPForUnity.Editor.Dependencies;
 using MCPForUnity.Editor.Dependencies.Models;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Windows;
-using MCPForUnity.Editor.Constants;
-using MCPForUnity.Editor.Config;
 using UnityEditor;
 using UnityEngine;
 
diff --git a/MCPForUnity/Editor/Tools/ManageEditor.cs b/MCPForUnity/Editor/Tools/ManageEditor.cs
@@ -1,8 +1,8 @@
 using System;
+using MCPForUnity.Editor.Helpers;
 using Newtonsoft.Json.Linq;
 using UnityEditor;
 using UnityEditorInternal; // Required for tag management
-using MCPForUnity.Editor.Helpers;
 
 namespace MCPForUnity.Editor.Tools
 {
diff --git a/MCPForUnity/Editor/Tools/ManageGameObject.cs b/MCPForUnity/Editor/Tools/ManageGameObject.cs
@@ -3,6 +3,8 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
+using MCPForUnity.Editor.Helpers; // For Response class
+using MCPForUnity.Runtime.Serialization;
 using Newtonsoft.Json; // Added for JsonSerializationException
 using Newtonsoft.Json.Linq;
 using UnityEditor;
@@ -11,8 +13,6 @@
 using UnityEditorInternal;
 using UnityEngine;
 using UnityEngine.SceneManagement;
-using MCPForUnity.Editor.Helpers; // For Response class
-using MCPForUnity.Runtime.Serialization;
 
 namespace MCPForUnity.Editor.Tools
 {
diff --git a/MCPForUnity/Editor/Tools/ManageScene.cs b/MCPForUnity/Editor/Tools/ManageScene.cs
@@ -2,12 +2,12 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
+using MCPForUnity.Editor.Helpers; // For Response class
 using Newtonsoft.Json.Linq;
 using UnityEditor;
 using UnityEditor.SceneManagement;
 using UnityEngine;
 using UnityEngine.SceneManagement;
-using MCPForUnity.Editor.Helpers; // For Response class
 
 namespace MCPForUnity.Editor.Tools
 {
diff --git a/MCPForUnity/Editor/Tools/ManageShader.cs b/MCPForUnity/Editor/Tools/ManageShader.cs
@@ -2,10 +2,10 @@
 using System.IO;
 using System.Linq;
 using System.Text.RegularExpressions;
+using MCPForUnity.Editor.Helpers;
 using Newtonsoft.Json.Linq;
 using UnityEditor;
 using UnityEngine;
-using MCPForUnity.Editor.Helpers;
 
 namespace MCPForUnity.Editor.Tools
 {
diff --git a/MCPForUnity/Editor/Tools/ReadConsole.cs b/MCPForUnity/Editor/Tools/ReadConsole.cs
@@ -2,11 +2,11 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
+using MCPForUnity.Editor.Helpers; // For Response class
 using Newtonsoft.Json.Linq;
 using UnityEditor;
 using UnityEditorInternal;
 using UnityEngine;
-using MCPForUnity.Editor.Helpers; // For Response class
 
 namespace MCPForUnity.Editor.Tools
 {
diff --git a/MCPForUnity/Editor/Windows/Components/ClientConfig/McpClientConfigSection.cs b/MCPForUnity/Editor/Windows/Components/ClientConfig/McpClientConfigSection.cs
@@ -4,13 +4,13 @@
 using System.IO;
 using System.Linq;
 using System.Runtime.InteropServices;
-using UnityEditor;
-using UnityEngine;
-using UnityEngine.UIElements;
 using MCPForUnity.Editor.Clients;
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Models;
 using MCPForUnity.Editor.Services;
+using UnityEditor;
+using UnityEngine;
+using UnityEngine.UIElements;
 
 namespace MCPForUnity.Editor.Windows.Components.ClientConfig
 {
diff --git a/MCPForUnity/Editor/Windows/MCPForUnityEditorWindow.cs b/MCPForUnity/Editor/Windows/MCPForUnityEditorWindow.cs
@@ -4,8 +4,8 @@
 using UnityEditor;
 using UnityEngine;
 using UnityEngine.UIElements;
-using MCPForUnity.Editor.Helpers;
-using MCPForUnity.Editor.Services;
+using MCPForUnity.Editor.Helpers;
+using MCPForUnity.Editor.Services;
 using MCPForUnity.Editor.Windows.Components.Settings;
 using MCPForUnity.Editor.Windows.Components.Connection;
 using MCPForUnity.Editor.Windows.Components.ClientConfig;
@@ -19,7 +19,7 @@ public class MCPForUnityEditorWindow : EditorWindow
         private McpConnectionSection connectionSection;
         private McpClientConfigSection clientConfigSection;
 
-        private static readonly HashSet<MCPForUnityEditorWindow> OpenWindows = new();
+        private static readonly HashSet<MCPForUnityEditorWindow> OpenWindows = new();
 
         public static void ShowWindow()
         {
@@ -102,8 +102,8 @@ public void CreateGUI()
             {
                 var clientConfigRoot = clientConfigTree.Instantiate();
                 sectionsContainer.Add(clientConfigRoot);
-                clientConfigSection = new McpClientConfigSection(clientConfigRoot);
-            }
+                clientConfigSection = new McpClientConfigSection(clientConfigRoot);
+            }
 
             // Initial updates
             RefreshAllData();
diff --git a/MCPForUnity/Editor/Windows/MCPSetupWindow.cs b/MCPForUnity/Editor/Windows/MCPSetupWindow.cs
diff --git a/Server/src/services/tools/manage_editor.py b/Server/src/services/tools/manage_editor.py
diff --git a/Server/src/services/tools/manage_gameobject.py b/Server/src/services/tools/manage_gameobject.py
diff --git a/Server/src/services/tools/manage_prefabs.py b/Server/src/services/tools/manage_prefabs.py

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ protected bool TryFindUvInPath(out string uvPath, out string version)`
`54`	`54`
`55`	`55`	`// Try common uv command names`
`56`	`56`	`var commands = new[] { "uvx", "uv" };`
`57`		`-`
	`57`	`+`
`58`	`58`	`foreach (var cmd in commands)`
`59`	`59`	`{`
`60`	`60`	`try`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`using System;`
`2`		`-using UnityEditor;`
`3`		`-using MCPForUnity.Editor.Constants;`
`4`	`2`	`using MCPForUnity.Editor.Config;`
	`3`	`+using MCPForUnity.Editor.Constants;`
	`4`	`+using UnityEditor;`
`5`	`5`
`6`	`6`	`namespace MCPForUnity.Editor.Helpers`
`7`	`7`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
	`1`	`+using MCPForUnity.Editor.Constants;`
`1`	`2`	`using UnityEditor;`
`2`	`3`	`using UnityEngine;`
`3`		`-using MCPForUnity.Editor.Constants;`
`4`	`4`
`5`	`5`	`namespace MCPForUnity.Editor.Helpers`
`6`	`6`	`{`
Original file line number	Diff line number	Diff line change
`@@ -2,8 +2,8 @@`
`2`	`2`
`3`	`3`	`namespace MCPForUnity.Editor.Helpers`
`4`	`4`	`{`
`5`		`-public interface IMcpResponse`
`6`		`-{`
	`5`	`+ public interface IMcpResponse`
	`6`	`+ {`
`7`	`7`	`[JsonProperty("success")]`
`8`	`8`	`bool Success { get; }`
`9`	`9`	`}`