refactor: update type hints to use | None syntax for optional parameters

Author: msanatan

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

@@ -15,56 +15,57 @@ def manage_gameobject(
         ctx: Context,
         action: Literal["create", "modify", "delete", "find", "add_component", "remove_component", "set_component_property", "get_components"],
         target: Annotated[str,
-                          "GameObject identifier by name or path for modify/delete/component actions"] = None,
+                          "GameObject identifier by name or path for modify/delete/component actions"] | None = None,
         search_method: Annotated[str,
-                                 "How to find objects ('by_name', 'by_id', 'by_path', etc.). Used with 'find' and some 'target' lookups."] = None,
+                                 "How to find objects ('by_name', 'by_id', 'by_path', etc.). Used with 'find' and some 'target' lookups."] | None = None,
         name: Annotated[str,
-                        "GameObject name - used for both 'create' (initial name) and 'modify' (rename)"] = None,
+                        "GameObject name - used for both 'create' (initial name) and 'modify' (rename)"] | None = None,
         tag: Annotated[str,
-                       "Tag name - used for both 'create' (initial tag) and 'modify' (change tag)"] = None,
+                       "Tag name - used for both 'create' (initial tag) and 'modify' (change tag)"] | None = None,
         parent: Annotated[str,
-                          "Parent GameObject reference - used for both 'create' (initial parent) and 'modify' (change parent)"] = None,
+                          "Parent GameObject reference - used for both 'create' (initial parent) and 'modify' (change parent)"] | None = None,
         position: Annotated[list[float],
-                            "Position - used for both 'create' (initial position) and 'modify' (change position)"] = None,
+                            "Position - used for both 'create' (initial position) and 'modify' (change position)"] | None = None,
         rotation: Annotated[list[float],
-                            "Rotation - used for both 'create' (initial rotation) and 'modify' (change rotation)"] = None,
+                            "Rotation - used for both 'create' (initial rotation) and 'modify' (change rotation)"] | None = None,
         scale: Annotated[list[float],
-                         "Scale - used for both 'create' (initial scale) and 'modify' (change scale)"] = None,
+                         "Scale - used for both 'create' (initial scale) and 'modify' (change scale)"] | None = None,
         components_to_add: Annotated[list[str],
-                                     "List of component names to add"] = None,
+                                     "List of component names to add"] | None = None,
         primitive_type: Annotated[str,
-                                  "Primitive type for 'create' action"] = None,
+                                  "Primitive type for 'create' action"] | None = None,
         save_as_prefab: Annotated[bool,
-                                  "If True, saves the created GameObject as a prefab"] = False,
-        prefab_path: Annotated[str, "Path for prefab creation"] = None,
+                                  "If True, saves the created GameObject as a prefab"] | None = None,
+        prefab_path: Annotated[str, "Path for prefab creation"] | None = None,
         prefab_folder: Annotated[str,
-                                 "Folder for prefab creation"] = "Assets/Prefabs",
+                                 "Folder for prefab creation"] | None = None,
         # --- Parameters for 'modify' ---
         set_active: Annotated[bool,
-                              "If True, sets the GameObject active"] = None,
-        layer: Annotated[str, "Layer name"] = None,
+                              "If True, sets the GameObject active"] | None = None,
+        layer: Annotated[str, "Layer name"] | None = None,
         components_to_remove: Annotated[list[str],
-                                        "List of component names to remove"] = None,
+                                        "List of component names to remove"] | None = None,
         component_properties: Annotated[dict[str, dict[str, Any]],
                                         """Dictionary of component names to their properties to set. For example:
                                         `{"MyScript": {"otherObject": {"find": "Player", "method": "by_name"}}}` assigns GameObject
                                         `{"MyScript": {"playerHealth": {"find": "Player", "component": "HealthComponent"}}}` assigns Component
                                         Example set nested property:
-                                        - Access shared material: `{"MeshRenderer": {"sharedMaterial.color": [1, 0, 0, 1]}}`"""] = None,
+                                        - Access shared material: `{"MeshRenderer": {"sharedMaterial.color": [1, 0, 0, 1]}}`"""] | None = None,
         # --- Parameters for 'find' ---
-        search_term: Annotated[str, "Search term for 'find' action"] = None,
+        search_term: Annotated[str,
+                               "Search term for 'find' action"] | None = None,
         find_all: Annotated[bool,
-                            "If True, finds all GameObjects matching the search term"] = False,
+                            "If True, finds all GameObjects matching the search term"] | None = None,
         search_in_children: Annotated[bool,
-                                      "If True, searches in children of the GameObject"] = False,
+                                      "If True, searches in children of the GameObject"] | None = None,
         search_inactive: Annotated[bool,
-                                   "If True, searches inactive GameObjects"] = False,
+                                   "If True, searches inactive GameObjects"] | None = None,
         # -- Component Management Arguments --
         component_name: Annotated[str,
-                                  "Component name for 'add_component' and 'remove_component' actions"] = None,
+                                  "Component name for 'add_component' and 'remove_component' actions"] | None = None,
         # Controls whether serialization of private [SerializeField] fields is included
         includeNonPublicSerialized: Annotated[bool,
-                                              "Controls whether serialization of private [SerializeField] fields is included"] = None,
+                                              "Controls whether serialization of private [SerializeField] fields is included"] | None = None,
     ) -> dict[str, Any]:
         ctx.info(f"Processing manage_gameobject: {action}")
         try:

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

@@ -18,12 +18,12 @@ def register_manage_menu_item_tools(mcp: FastMCP):
     async def manage_menu_item(
         ctx: Context,
         action: Annotated[Literal["execute", "list", "exists"], "Operations"],
-        menu_path: Annotated[str | None,
-                             "Menu path for 'execute' or 'exists' (e.g., 'File/Save Project')"] = None,
-        search: Annotated[str | None,
-                          "Optional filter string for 'list' (e.g., 'Save')"] = None,
-        refresh: Annotated[bool | None,
-                           "Optional flag to force refresh of the menu cache when listing"] = None,
+        menu_path: Annotated[str,
+                             "Menu path for 'execute' or 'exists' (e.g., 'File/Save Project')"] | None = None,
+        search: Annotated[str,
+                          "Optional filter string for 'list' (e.g., 'Save')"] | None = None,
+        refresh: Annotated[bool,
+                           "Optional flag to force refresh of the menu cache when listing"] | None = None,
     ) -> dict[str, Any]:
         ctx.info(f"Processing manage_menu_item: {action}")
         # Prepare parameters for the C# handler

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

@@ -19,18 +19,18 @@ def manage_prefabs(
             "save_open_stage",
             "create_from_gameobject",
         ], "Operations"],
-        prefab_path: Annotated[str | None,
-                               "Prefab asset path relative to Assets e.g. Assets/Prefabs/favorite.prefab"] = None,
-        mode: Annotated[str | None,
-                        "Optional prefab stage mode (only 'InIsolation' is currently supported)"] = None,
-        save_before_close: Annotated[bool | None,
-                                     "When true, `close_stage` will save the prefab before exiting the stage."] = None,
-        target: Annotated[str | None,
-                          "Scene GameObject name required for create_from_gameobject"] = None,
-        allow_overwrite: Annotated[bool | None,
-                                   "Allow replacing an existing prefab at the same path"] = None,
-        search_inactive: Annotated[bool | None,
-                                   "Include inactive objects when resolving the target name"] = None,
+        prefab_path: Annotated[str,
+                               "Prefab asset path relative to Assets e.g. Assets/Prefabs/favorite.prefab"] | None = None,
+        mode: Annotated[str,
+                        "Optional prefab stage mode (only 'InIsolation' is currently supported)"] | None = None,
+        save_before_close: Annotated[bool,
+                                     "When true, `close_stage` will save the prefab before exiting the stage."] | None = None,
+        target: Annotated[str,
+                          "Scene GameObject name required for create_from_gameobject"] | None = None,
+        allow_overwrite: Annotated[bool,
+                                   "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,
     ) -> dict[str, Any]:
         ctx.info(f"Processing manage_prefabs: {action}")
         try:

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

@@ -18,7 +18,7 @@ def manage_scene(
                         "Scene name. Not required get_active/get_build_settings"] | None = None,
         path: Annotated[str,
                         "Asset path for scene operations (default: 'Assets/')"] | None = None,
-        build_index: Annotated[int | None,
+        build_index: Annotated[int,
                                "Build index for load/build settings actions"] | None = None,
     ) -> dict[str, Any]:
         ctx.info(f"Processing manage_scene: {action}")

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

@@ -92,12 +92,12 @@ def apply_text_edits(
         ctx: Context,
         uri: Annotated[str, "URI of the script to edit under Assets/ directory, unity://path/Assets/... or file://... or Assets/..."],
         edits: Annotated[list[dict[str, Any]], "List of edits to apply to the script, i.e. a list of {startLine,startCol,endLine,endCol,newText} (1-indexed!)"],
-        precondition_sha256: Annotated[str | None,
-                                       "Optional SHA256 of the script to edit, used to prevent concurrent edits"] = None,
-        strict: Annotated[bool | None,
-                          "Optional strict flag, used to enforce strict mode"] = None,
-        options: Annotated[dict[str, Any] | None,
-                           "Optional options, used to pass additional options to the script editor"] = None,
+        precondition_sha256: Annotated[str,
+                                       "Optional SHA256 of the script to edit, used to prevent concurrent edits"] | None = None,
+        strict: Annotated[bool,
+                          "Optional strict flag, used to enforce strict mode"] | None = None,
+        options: Annotated[dict[str, Any],
+                           "Optional options, used to pass additional options to the script editor"] | None = None,
     ) -> dict[str, Any]:
         ctx.info(f"Processing apply_text_edits: {uri}")
         name, directory = _split_uri(uri)
@@ -370,8 +370,8 @@ def create_script(
         ctx: Context,
         path: Annotated[str, "Path under Assets/ to create the script at, e.g., 'Assets/Scripts/My.cs'"],
         contents: Annotated[str, "Contents of the script to create. Note, this is Base64 encoded over transport."],
-        script_type: Annotated[str | None, "Script type (e.g., 'C#')"] = None,
-        namespace: Annotated[str | None, "Namespace for the script"] = None,
+        script_type: Annotated[str, "Script type (e.g., 'C#')"] | None = None,
+        namespace: Annotated[str, "Namespace for the script"] | None = None,
     ) -> dict[str, Any]:
         ctx.info(f"Processing create_script: {path}")
         name = os.path.splitext(os.path.basename(path))[0]
@@ -460,10 +460,9 @@ def manage_script(
         path: Annotated[str, "Asset path (default: 'Assets/')", "Path under Assets/ to create the script at, e.g., 'Assets/Scripts/My.cs'"],
         contents: Annotated[str, "Contents of the script to create",
                             "C# code for 'create'/'update'"] | None = None,
-        script_type: Annotated[str | None,
-                               "Script type (e.g., 'C#')", "Type hint (e.g., 'MonoBehaviour')"] | None = None,
-        namespace: Annotated[str | None, "Namespace for the script",
-                             "Script namespace"] | None = None,
+        script_type: Annotated[str, "Script type (e.g., 'C#')",
+                               "Type hint (e.g., 'MonoBehaviour')"] | None = None,
+        namespace: Annotated[str, "Namespace for the script"] | None = None,
     ) -> dict[str, Any]:
         ctx.info(f"Processing manage_script: {action}")
         try: