Fix manage prefabs (#405)

* 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

MCPForUnity/Editor/Helpers/CodexConfigHelper.cs

@@ -2,8 +2,8 @@ using System;
 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

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

MCPForUnity/Editor/Helpers/ExecPath.cs

@@ -2,10 +2,10 @@ using System;
 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
 {

MCPForUnity/Editor/Helpers/GameObjectSerializer.cs

@@ -2,11 +2,11 @@ using System;
 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
 {

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
 {

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
 {

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 Newtonsoft.Json;
-using UnityEngine;
 using MCPForUnity.Editor.Constants;
+using Newtonsoft.Json;
+using UnityEditor;
+using UnityEngine;
 
 namespace MCPForUnity.Editor.Helpers
 {

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
 {

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;
 

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
 {

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

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
 {

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
 {

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
 {

MCPForUnity/Editor/Services/ServerManagementService.cs

@@ -41,15 +41,24 @@ namespace MCPForUnity.Editor.Services
                     McpLog.Debug($"uv cache cleared successfully: {stdout}");
                     return true;
                 }
-                else
+                string combinedOutput = string.Join(
-                {
+                    Environment.NewLine,
-                    string errorMessage = string.IsNullOrEmpty(stderr)
+                    new[] { stderr, stdout }.Where(s => !string.IsNullOrWhiteSpace(s)).Select(s => s.Trim()));
-                        ? "Unknown error"
-                        : stderr;
 
-                    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.");
+                string lockHint = (!string.IsNullOrEmpty(combinedOutput) &&
-                    return false;
+                                   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)
             {

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
 {

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 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 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.Tools;
-using MCPForUnity.Editor.Tools.Prefabs;
-using MCPForUnity.Editor.Services.Transport;
 
 namespace MCPForUnity.Editor.Services.Transport.Transports
 {

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;
 

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
 {

MCPForUnity/Editor/Tools/ManageGameObject.cs

@@ -3,6 +3,8 @@ using System;
 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 UnityEditor.SceneManagement;
 using UnityEditorInternal;
 using UnityEngine;
 using UnityEngine.SceneManagement;
-using MCPForUnity.Editor.Helpers; // For Response class
-using MCPForUnity.Runtime.Serialization;
 
 namespace MCPForUnity.Editor.Tools
 {

MCPForUnity/Editor/Tools/ManageScene.cs

@@ -2,12 +2,12 @@ using System;
 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
 {

MCPForUnity/Editor/Tools/ManageShader.cs

@@ -2,10 +2,10 @@ using System;
 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
 {

MCPForUnity/Editor/Tools/ReadConsole.cs

@@ -2,11 +2,11 @@ using System;
 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
 {

MCPForUnity/Editor/Windows/Components/ClientConfig/McpClientConfigSection.cs

@@ -4,13 +4,13 @@ using System.Diagnostics;
 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
 {

MCPForUnity/Editor/Windows/MCPSetupWindow.cs

@@ -1,10 +1,10 @@
 using System;
-using UnityEditor;
-using UnityEngine;
-using UnityEngine.UIElements;
 using MCPForUnity.Editor.Dependencies;
 using MCPForUnity.Editor.Dependencies.Models;
 using MCPForUnity.Editor.Helpers;
+using UnityEditor;
+using UnityEngine;
+using UnityEngine.UIElements;
 
 namespace MCPForUnity.Editor.Windows
 {

Server/src/services/tools/manage_editor.py

@@ -13,8 +13,7 @@ from transport.legacy.unity_connection import async_send_command_with_retry
 )
 async def manage_editor(
     ctx: Context,
-    action: Annotated[Literal["telemetry_status", "telemetry_ping", "play", "pause", "stop", "get_state", "get_project_root", "get_windows",
+    action: Annotated[Literal["telemetry_status", "telemetry_ping", "play", "pause", "stop", "set_active_tool", "add_tag", "remove_tag", "add_layer", "remove_layer"], "Get and update the Unity Editor state."],
-                              "get_active_tool", "get_selection", "get_prefab_stage", "set_active_tool", "add_tag", "remove_tag", "get_tags", "add_layer", "remove_layer", "get_layers"], "Get and update the Unity Editor state."],
     wait_for_completion: Annotated[bool | str,
                                    "Optional. If True, waits for certain actions (accepts true/false or 'true'/'false')"] | None = None,
     tool_name: Annotated[str,

Server/src/services/tools/manage_gameobject.py

@@ -13,7 +13,7 @@ from transport.legacy.unity_connection import async_send_command_with_retry
 )
 async def manage_gameobject(
     ctx: Context,
-    action: Annotated[Literal["create", "modify", "delete", "find", "add_component", "remove_component", "set_component_property", "get_components"], "Perform CRUD operations on GameObjects and components."],
+    action: Annotated[Literal["create", "modify", "delete", "find", "add_component", "remove_component", "set_component_property", "get_components", "get_component"], "Perform CRUD operations on GameObjects and components."],
     target: Annotated[str,
                       "GameObject identifier by name or path for modify/delete/component actions"] | None = None,
     search_method: Annotated[Literal["by_id", "by_name", "by_path", "by_tag", "by_layer", "by_component"],

Server/src/services/tools/manage_prefabs.py

@@ -8,11 +8,11 @@ from transport.legacy.unity_connection import async_send_command_with_retry
 
 
 @mcp_for_unity_tool(
-    description="Performs prefab operations (create, modify, delete, etc.)."
+    description="Performs prefab operations (open_stage, close_stage, save_open_stage, create_from_gameobject)."
 )
 async def manage_prefabs(
     ctx: Context,
-    action: Annotated[Literal["create", "modify", "delete", "get_components"], "Perform prefab operations."],
+    action: Annotated[Literal["open_stage", "close_stage", "save_open_stage", "create_from_gameobject"], "Perform prefab operations."],
     prefab_path: Annotated[str,
                            "Prefab asset path relative to Assets e.g. Assets/Prefabs/favorite.prefab"] | None = None,
     mode: Annotated[str,
@@ -25,8 +25,6 @@ async def manage_prefabs(
                                "Allow replacing an existing prefab at the same path"] | None = None,
     search_inactive: Annotated[bool,
                                "Include inactive objects when resolving the target name"] | None = None,
-    component_properties: Annotated[str,
-                                    "Component properties in JSON format"] | None = None,
 ) -> dict[str, Any]:
     # Get active instance from session state
     # Removed session_state import