feat: Remove pagination from GetHierarchy method and simplify prefab retrieval

Author: whatevertogo

MCPForUnity/Editor/Tools/Prefabs/ManagePrefabs.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Collections.Generic;
-using System.Linq;
 using System.IO;
 using MCPForUnity.Editor.Helpers;
 using Newtonsoft.Json.Linq;
@@ -26,10 +25,6 @@ public static class ManagePrefabs
         private const string ACTION_GET_HIERARCHY = "get_hierarchy";
         private const string SupportedActions = ACTION_OPEN_STAGE + ", " + ACTION_CLOSE_STAGE + ", " + ACTION_SAVE_OPEN_STAGE + ", " + ACTION_CREATE_FROM_GAMEOBJECT + ", " + ACTION_GET_INFO + ", " + ACTION_GET_HIERARCHY;
 
-        // Pagination constants
-        private const int DefaultPageSize = 50;
-        private const int MaxPageSize = 500;
-
         public static object HandleCommand(JObject @params)
         {
             if (@params == null)
@@ -556,6 +551,7 @@ private static object GetInfo(JObject @params)
 
         /// <summary>
         /// Gets the hierarchical structure of a prefab asset.
+        /// Returns all objects in the prefab for full client-side filtering and search.
         /// </summary>
         private static object GetHierarchy(JObject @params)
         {
@@ -571,11 +567,6 @@ private static object GetHierarchy(JObject @params)
                 return new ErrorResponse($"Invalid prefab path '{prefabPath}'. Path traversal sequences are not allowed.");
             }
 
-            // Parse pagination parameters
-            var pagination = PaginationRequest.FromParams(@params, defaultPageSize: DefaultPageSize);
-            int pageSize = Mathf.Clamp(pagination.PageSize, 1, MaxPageSize);
-            int cursor = pagination.Cursor;
-
             // Load prefab contents in background (without opening stage UI)
             GameObject prefabContents = PrefabUtility.LoadPrefabContents(sanitizedPath);
             if (prefabContents == null)
@@ -585,29 +576,16 @@ private static object GetHierarchy(JObject @params)
 
             try
             {
-                // Build hierarchy items
+                // Build complete hierarchy items (no pagination)
                 var allItems = BuildHierarchyItems(prefabContents.transform);
-                int totalCount = allItems.Count;
-
-                // Apply pagination
-                int startIndex = Mathf.Min(cursor, totalCount);
-                int endIndex = Mathf.Min(startIndex + pageSize, totalCount);
-                var paginatedItems = allItems.Skip(startIndex).Take(endIndex - startIndex).ToList();
-
-                bool truncated = endIndex < totalCount;
-                string nextCursor = truncated ? endIndex.ToString() : null;
 
                 return new SuccessResponse(
-                    $"Successfully retrieved prefab hierarchy. Found {totalCount} objects.",
+                    $"Successfully retrieved prefab hierarchy. Found {allItems.Count} objects.",
                     new
                     {
                         prefabPath = sanitizedPath,
-                        cursor = cursor.ToString(),
-                        pageSize = pageSize,
-                        nextCursor = nextCursor,
-                        truncated = truncated,
-                        total = totalCount,
-                        items = paginatedItems
+                        total = allItems.Count,
+                        items = allItems
                     }
                 );
             }

Server/src/services/tools/manage_prefabs.py

@@ -5,7 +5,7 @@
 
 from services.registry import mcp_for_unity_tool
 from services.tools import get_unity_instance_from_context
-from services.tools.utils import coerce_bool, coerce_int
+from services.tools.utils import coerce_bool
 from transport.unity_transport import send_with_unity_instance
 from transport.legacy.unity_connection import async_send_command_with_retry
 from services.tools.preflight import preflight
@@ -53,8 +53,6 @@ async def manage_prefabs(
     allow_overwrite: Annotated[bool, "Allow replacing existing prefab."] | None = None,
     search_inactive: Annotated[bool, "Include inactive GameObjects in search."] | None = None,
     unlink_if_instance: Annotated[bool, "Unlink from existing prefab before creating new one."] | None = None,
-    page_size: Annotated[int | str, "Items per page for get_hierarchy (default: 50, max: 500)."] | None = None,
-    cursor: Annotated[int | str, "Pagination cursor for get_hierarchy."] | None = None,
 ) -> dict[str, Any]:
     # Validate required parameters
     required = REQUIRED_PARAMS.get(action, [])
@@ -81,10 +79,6 @@ async def manage_prefabs(
         }
 
     try:
-        # Coerce pagination parameters
-        coerced_page_size = coerce_int(page_size, default=None)
-        coerced_cursor = coerce_int(cursor, default=None)
-
         # Build parameters dictionary
         params: dict[str, Any] = {"action": action}
 
@@ -116,13 +110,6 @@ async def manage_prefabs(
         if unlink_if_instance_val is not None:
             params["unlinkIfInstance"] = unlink_if_instance_val
 
-        # Handle pagination parameters
-        if coerced_page_size is not None:
-            params["pageSize"] = coerced_page_size
-
-        if coerced_cursor is not None:
-            params["cursor"] = coerced_cursor
-
         # Send command to Unity
         response = await send_with_unity_instance(
             async_send_command_with_retry, unity_instance, "manage_prefabs", params