feat: add explicit name parameters to all MCP tool decorators

Author: msanatan

UnityMcpBridge/UnityMcpServer~/src/tools/manage_asset.py

@@ -13,7 +13,7 @@
 def register_manage_asset_tools(mcp: FastMCP):
     """Registers the manage_asset tool with the MCP server."""
 
-    @mcp.tool(description="Performs asset operations (import, create, modify, delete, etc.) in Unity.")
+    @mcp.tool(name="manage_asset", description="Performs asset operations (import, create, modify, delete, etc.) in Unity.")
     @telemetry_tool("manage_asset")
     async def manage_asset(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_editor.py

@@ -10,7 +10,7 @@
 def register_manage_editor_tools(mcp: FastMCP):
     """Register all editor management tools with the MCP server."""
 
-    @mcp.tool(description="Controls and queries the Unity editor's state and settings")
+    @mcp.tool(name="manage_editor", description="Controls and queries the Unity editor's state and settings")
     @telemetry_tool("manage_editor")
     def manage_editor(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_gameobject.py

@@ -9,7 +9,7 @@
 def register_manage_gameobject_tools(mcp: FastMCP):
     """Register all GameObject management tools with the MCP server."""
 
-    @mcp.tool(description="Manage GameObjects. Note: for 'get_components', the `data` field contains a dictionary of component names and their serialized properties.")
+    @mcp.tool(name="manage_gameobject", description="Manage GameObjects. Note: for 'get_components', the `data` field contains a dictionary of component names and their serialized properties.")
     @telemetry_tool("manage_gameobject")
     def manage_gameobject(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_menu_item.py

@@ -13,7 +13,7 @@
 def register_manage_menu_item_tools(mcp: FastMCP):
     """Registers the manage_menu_item tool with the MCP server."""
 
-    @mcp.tool(description="Manage Unity menu items (execute/list/exists). If you're not sure what menu item to use, use the 'list' action to find it before using 'execute'.")
+    @mcp.tool(name="manage_menu_item", description="Manage Unity menu items (execute/list/exists). If you're not sure what menu item to use, use the 'list' action to find it before using 'execute'.")
     @telemetry_tool("manage_menu_item")
     async def manage_menu_item(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_prefabs.py

@@ -9,7 +9,7 @@
 def register_manage_prefabs_tools(mcp: FastMCP) -> None:
     """Register prefab management tools with the MCP server."""
 
-    @mcp.tool(description="Bridge for prefab management commands (stage control and creation).")
+    @mcp.tool(name="manage_prefabs", description="Bridge for prefab management commands (stage control and creation).")
     @telemetry_tool("manage_prefabs")
     def manage_prefabs(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_scene.py

@@ -9,7 +9,7 @@
 def register_manage_scene_tools(mcp: FastMCP):
     """Register all scene management tools with the MCP server."""
 
-    @mcp.tool(description="Manage Unity scenes")
+    @mcp.tool(name="manage_scene", description="Manage Unity scenes")
     @telemetry_tool("manage_scene")
     def manage_scene(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_script.py

@@ -73,7 +73,7 @@ def _split_uri(uri: str) -> tuple[str, str]:
         directory = os.path.dirname(effective_path)
         return name, directory
 
-    @mcp.tool(description=(
+    @mcp.tool(name="apply_text_edits", description=(
         "Apply small text edits to a C# script identified by URI.\n\n"
         "IMPORTANT: This tool replaces EXACT character positions. Always verify content at target lines/columns BEFORE editing!\n"
         "RECOMMENDED WORKFLOW:\n"
@@ -364,7 +364,7 @@ def _flip_async():
             return resp
         return {"success": False, "message": str(resp)}
 
-    @mcp.tool(description=("Create a new C# script at the given project path."))
+    @mcp.tool(name="create_script", description=("Create a new C# script at the given project path."))
     @telemetry_tool("create_script")
     def create_script(
         ctx: Context,
@@ -401,7 +401,7 @@ def create_script(
         resp = send_command_with_retry("manage_script", params)
         return resp if isinstance(resp, dict) else {"success": False, "message": str(resp)}
 
-    @mcp.tool(description=("Delete a C# script by URI or Assets-relative path."))
+    @mcp.tool(name="delete_script", description=("Delete a C# script by URI or Assets-relative path."))
     @telemetry_tool("delete_script")
     def delete_script(
         ctx: Context,
@@ -415,7 +415,7 @@ def delete_script(
         resp = send_command_with_retry("manage_script", params)
         return resp if isinstance(resp, dict) else {"success": False, "message": str(resp)}
 
-    @mcp.tool(description=("Validate a C# script and return diagnostics."))
+    @mcp.tool(name="validate_script", description=("Validate a C# script and return diagnostics."))
     @telemetry_tool("validate_script")
     def validate_script(
         ctx: Context,
@@ -448,7 +448,7 @@ def validate_script(
             return {"success": True, "data": {"warnings": warnings, "errors": errors}}
         return resp if isinstance(resp, dict) else {"success": False, "message": str(resp)}
 
-    @mcp.tool(description=("Compatibility router for legacy script operations. Prefer apply_text_edits (ranges) or script_apply_edits (structured) for edits."))
+    @mcp.tool(name="manage_script", description=("Compatibility router for legacy script operations. Prefer apply_text_edits (ranges) or script_apply_edits (structured) for edits."))
     @telemetry_tool("manage_script")
     def manage_script(
         ctx: Context,
@@ -569,7 +569,7 @@ def manage_script(
                 "message": f"Python error managing script: {str(e)}",
             }
 
-    @mcp.tool(description=(
+    @mcp.tool(name="manage_script_capabilities", description=(
         "Get manage_script capabilities (supported ops, limits, and guards).\n\n"
         "Returns:\n- ops: list of supported structured ops\n- text_ops: list of supported text ops\n- max_edit_payload_bytes: server edit payload cap\n- guards: header/using guard enabled flag\n"
     ))
@@ -596,7 +596,7 @@ def manage_script_capabilities(ctx: Context) -> dict[str, Any]:
         except Exception as e:
             return {"success": False, "error": f"capabilities error: {e}"}
 
-    @mcp.tool(description="Get SHA256 and basic metadata for a Unity C# script without returning file contents")
+    @mcp.tool(name="get_sha", description="Get SHA256 and basic metadata for a Unity C# script without returning file contents")
     @telemetry_tool("get_sha")
     def get_sha(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/manage_script_edits.py

@@ -307,7 +307,7 @@ def _err(code: str, message: str, *, expected: Dict[str, Any] | None = None, rew
 
 
 def register_manage_script_edits_tools(mcp: FastMCP):
-    @mcp.tool(description=(
+    @mcp.tool(name="script_apply_edits", description=(
         "Structured C# edits (methods/classes) with safer boundaries — prefer this over raw text.\n\n"
         "Best practices:\n"
         "- Prefer anchor_* ops for pattern-based insert/replace near stable markers\n"

UnityMcpBridge/UnityMcpServer~/src/tools/manage_shader.py

@@ -10,7 +10,7 @@
 def register_manage_shader_tools(mcp: FastMCP):
     """Register all shader script management tools with the MCP server."""
 
-    @mcp.tool(description="Manages shader scripts in Unity (create, read, update, delete).")
+    @mcp.tool(name="manage_shader", description="Manages shader scripts in Unity (create, read, update, delete).")
     @telemetry_tool("manage_shader")
     def manage_shader(
         ctx: Context,

UnityMcpBridge/UnityMcpServer~/src/tools/read_console.py

@@ -12,7 +12,7 @@
 def register_read_console_tools(mcp: FastMCP):
     """Registers the read_console tool with the MCP server."""
 
-    @mcp.tool(description="Gets messages from or clears the Unity Editor console.")
+    @mcp.tool(name="read_console", description="Gets messages from or clears the Unity Editor console.")
     @telemetry_tool("read_console")
     def read_console(
         ctx: Context,