Allow the LLMs to read menu items, not just execute them (#263)

* Move the current test to a Tools folder

* feat: add env object and disabled flag handling for MCP client configuration

* Format manual config specially for Windsurf and Kiro

* refactor: extract config JSON building logic into dedicated ConfigJsonBuilder class

* refactor: extract unity node population logic into centralized helper method

* refactor: only add env property to config for Windsurf and Kiro clients

If it ain't broke with the other clients, don't fix...

* fix: write UTF-8 without BOM encoding for config files to avoid Windows compatibility issues

* fix: enforce UTF-8 encoding without BOM when writing files to disk

* refactor: replace execute_menu_item with enhanced manage_menu_item tool supporting list/exists/refresh

* Update meta files for older Unity versions

* test: add unit tests for menu item management and execution

* feat: add tips for paths, script compilation, and menu item usage in asset creation strategy

* Use McpLog functionality instead of Unity's Debug

* Add telemetry

* Annotate parameters

More info to LLMs + better validation

* Remove the refresh command

It's only ever useful in the context of listing menu items

* Updated meta files since running in Unity 2021

* Slightly better README

* fix: rename server-version.txt to server_version.txt and update menu item description

Author: msanatan

README.md

@@ -42,7 +42,7 @@ MCP for Unity acts as a bridge, allowing AI assistants (like Claude, Cursor) to
   *   `manage_asset`: Performs asset operations (import, create, modify, delete, etc.).
   *   `manage_shader`: Performs shader CRUD operations (create, read, modify, delete).
   *   `manage_gameobject`: Manages GameObjects: create, modify, delete, find, and component operations.
-  *   `execute_menu_item`: Executes a menu item via its path (e.g., "File/Save Project").
+  *   `manage_menu_item`: List Unity Editor menu items; and check for their existence or execute them (e.g., execute "File/Save Project").
   *   `apply_text_edits`: Precise text edits with precondition hashes and atomic multi-edit batches.
   *   `script_apply_edits`: Structured C# method/class edits (insert/replace/delete) with safer boundaries.
   *   `validate_script`: Fast validation (basic/standard) to catch syntax/structure issues before/after writes.

TestProjects/UnityMCPTests/Assets/Scripts/LongUnityScriptClaudeTest.cs.meta

@@ -0,0 +1,47 @@
+using NUnit.Framework;
+using Newtonsoft.Json.Linq;
+using MCPForUnity.Editor.Tools.MenuItems;
+
+namespace MCPForUnityTests.Editor.Tools.MenuItems
+{
+    public class ManageMenuItemTests
+    {
+        private static JObject ToJO(object o) => JObject.FromObject(o);
+
+        [Test]
+        public void HandleCommand_UnknownAction_ReturnsError()
+        {
+            var res = ManageMenuItem.HandleCommand(new JObject { ["action"] = "unknown_action" });
+            var jo = ToJO(res);
+            Assert.IsFalse((bool)jo["success"], "Expected success false for unknown action");
+            StringAssert.Contains("Unknown action", (string)jo["error"]);
+        }
+
+        [Test]
+        public void HandleCommand_List_RoutesAndReturnsArray()
+        {
+            var res = ManageMenuItem.HandleCommand(new JObject { ["action"] = "list" });
+            var jo = ToJO(res);
+            Assert.IsTrue((bool)jo["success"], "Expected success true");
+            Assert.AreEqual(JTokenType.Array, jo["data"].Type, "Expected data to be an array");
+        }
+
+        [Test]
+        public void HandleCommand_Execute_Blacklisted_RoutesAndErrors()
+        {
+            var res = ManageMenuItem.HandleCommand(new JObject { ["action"] = "execute", ["menuPath"] = "File/Quit" });
+            var jo = ToJO(res);
+            Assert.IsFalse((bool)jo["success"], "Expected success false");
+            StringAssert.Contains("blocked for safety", (string)jo["error"], "Expected blacklist message");
+        }
+
+        [Test]
+        public void HandleCommand_Exists_MissingParam_ReturnsError()
+        {
+            var res = ManageMenuItem.HandleCommand(new JObject { ["action"] = "exists" });
+            var jo = ToJO(res);
+            Assert.IsFalse((bool)jo["success"], "Expected success false when missing menuPath");
+            StringAssert.Contains("Required parameter", (string)jo["error"]);
+        }
+    }
+}

TestProjects/UnityMCPTests/Assets/Scripts/TestAsmdef.meta

@@ -0,0 +1,39 @@
+using NUnit.Framework;
+using Newtonsoft.Json.Linq;
+using MCPForUnity.Editor.Tools.MenuItems;
+
+namespace MCPForUnityTests.Editor.Tools.MenuItems
+{
+    public class MenuItemExecutorTests
+    {
+        private static JObject ToJO(object o) => JObject.FromObject(o);
+
+        [Test]
+        public void Execute_MissingParam_ReturnsError()
+        {
+            var res = MenuItemExecutor.Execute(new JObject());
+            var jo = ToJO(res);
+            Assert.IsFalse((bool)jo["success"], "Expected success false");
+            StringAssert.Contains("Required parameter", (string)jo["error"]);
+        }
+
+        [Test]
+        public void Execute_Blacklisted_ReturnsError()
+        {
+            var res = MenuItemExecutor.Execute(new JObject { ["menuPath"] = "File/Quit" });
+            var jo = ToJO(res);
+            Assert.IsFalse((bool)jo["success"], "Expected success false for blacklisted menu");
+            StringAssert.Contains("blocked for safety", (string)jo["error"], "Expected blacklist message");
+        }
+
+        [Test]
+        public void Execute_NonBlacklisted_ReturnsImmediateSuccess()
+        {
+            // We don't rely on the menu actually existing; execution is delayed and we only check the immediate response shape
+            var res = MenuItemExecutor.Execute(new JObject { ["menuPath"] = "File/Save Project" });
+            var jo = ToJO(res);
+            Assert.IsTrue((bool)jo["success"], "Expected immediate success response");
+            StringAssert.Contains("Attempted to execute menu item", (string)jo["message"], "Expected attempt message");
+        }
+    }
+}