refactor: reorganize menu item functionality into separate execute and get commands

An MCP resource for retrieval, and a simple command to execute. Because it's a resource, it's easier for the user to see what's in the menu items

Author: msanatan

MCPForUnity/Editor/MCPForUnityBridge.cs

@@ -14,7 +14,6 @@
 using MCPForUnity.Editor.Helpers;
 using MCPForUnity.Editor.Models;
 using MCPForUnity.Editor.Tools;
-using MCPForUnity.Editor.Tools.MenuItems;
 using MCPForUnity.Editor.Tools.Prefabs;
 
 namespace MCPForUnity.Editor

MCPForUnity/Editor/Tools/MenuItems.meta renamed to MCPForUnity/Editor/Resources/MenuItems.meta

@@ -0,0 +1,71 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using MCPForUnity.Editor.Helpers;
+using Newtonsoft.Json.Linq;
+using UnityEditor;
+
+namespace MCPForUnity.Editor.Resources.MenuItems
+{
+    /// <summary>
+    /// Provides a simple read-only resource that returns Unity menu items.
+    /// </summary>
+    [McpForUnityResource("get_menu_items")]
+    public static class GetMenuItems
+    {
+        private static List<string> _cached;
+
+        [InitializeOnLoadMethod]
+        private static void BuildCache() => Refresh();
+
+        public static object HandleCommand(JObject @params)
+        {
+            bool forceRefresh = @params?["refresh"]?.ToObject<bool>() ?? false;
+            string search = @params?["search"]?.ToString();
+
+            var items = GetMenuItemsInternal(forceRefresh);
+
+            if (!string.IsNullOrEmpty(search))
+            {
+                items = items
+                    .Where(item => item.IndexOf(search, StringComparison.OrdinalIgnoreCase) >= 0)
+                    .ToList();
+            }
+
+            string message = $"Retrieved {items.Count} menu items";
+            return Response.Success(message, items);
+        }
+
+        internal static List<string> GetMenuItemsInternal(bool forceRefresh)
+        {
+            if (forceRefresh || _cached == null)
+            {
+                Refresh();
+            }
+
+            return (_cached ?? new List<string>()).ToList();
+        }
+
+        private static void Refresh()
+        {
+            try
+            {
+                var methods = TypeCache.GetMethodsWithAttribute<MenuItem>();
+                _cached = methods
+                    .SelectMany(m => m
+                        .GetCustomAttributes(typeof(MenuItem), false)
+                        .OfType<MenuItem>()
+                        .Select(attr => attr.menuItem))
+                    .Where(s => !string.IsNullOrEmpty(s))
+                    .Distinct(StringComparer.Ordinal)
+                    .OrderBy(s => s, StringComparer.Ordinal)
+                    .ToList();
+            }
+            catch (Exception ex)
+            {
+                McpLog.Error($"[GetMenuItems] Failed to scan menu items: {ex}");
+                _cached ??= new List<string>();
+            }
+        }
+    }
+}

MCPForUnity/Editor/Resources/MenuItems/GetMenuItems.cs

@@ -1,15 +1,13 @@
 using System;
 using System.Collections.Generic;
+using MCPForUnity.Editor.Helpers;
 using Newtonsoft.Json.Linq;
 using UnityEditor;
-using MCPForUnity.Editor.Helpers;
 
-namespace MCPForUnity.Editor.Tools.MenuItems
+namespace MCPForUnity.Editor.Tools
 {
-    /// <summary>
-    /// Executes Unity Editor menu items by path with safety checks.
-    /// </summary>
-    public static class MenuItemExecutor
+    [McpForUnityTool("execute_menu_item")]
+    public static class ExecuteMenuItem
     {
         // Basic blacklist to prevent execution of disruptive menu items.
         private static readonly HashSet<string> _menuPathBlacklist = new HashSet<string>(
@@ -19,10 +17,11 @@ public static class MenuItemExecutor
         };
 
         /// <summary>
-        /// Execute a specific menu item. Expects 'menu_path' or 'menuPath' in params.
+        /// Routes actions: execute, list, exists, refresh
         /// </summary>
-        public static object Execute(JObject @params)
+        public static object HandleCommand(JObject @params)
         {
+            McpLog.Info("[ExecuteMenuItem] Handling menu item command");
             string menuPath = @params["menu_path"]?.ToString() ?? @params["menuPath"]?.ToString();
             if (string.IsNullOrWhiteSpace(menuPath))
             {