fix: harden parameter type handling and resolve PR feedback

Parameter Type Improvements:
- Broaden count in read_console.py to accept int | str
- Broaden build_index in manage_scene.py to accept int | str
- Harden vector parsing in manage_gameobject.py with NaN/Inf checks
- Add whitespace-delimited vector support (e.g., "1 2 3")
- Narrow exception handling from Exception to (ValueError, TypeError)

Test Improvements:
- Harden _load_module in test files with spec/loader validation
- Fix test_manage_gameobject_boolean_and_tag_mapping by mapping tag→search_term

Bug Fixes:
- Fix syntax error in manage_shader.py (remove stray 'x')

Version: Bump to 6.2.3

All tests pass: 41 passed, 5 skipped, 7 xpassed

Author: dsarno

MCPForUnity/UnityMcpServer~/src/server_version.txt

@@ -1 +1 @@
-6.2.2
+6.2.3

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

@@ -83,22 +83,24 @@ def _coerce_bool(value, default=None):
     def _coerce_vec(value, default=None):
         if value is None:
             return default
-        if isinstance(value, list) and len(value) == 3:
+        import math
+        def _to_vec3(parts):
             try:
-                return [float(value[0]), float(value[1]), float(value[2])]
-            except Exception:
+                vec = [float(parts[0]), float(parts[1]), float(parts[2])]
+            except (ValueError, TypeError):
                 return default
+            return vec if all(math.isfinite(n) for n in vec) else default
+        if isinstance(value, list) and len(value) == 3:
+            return _to_vec3(value)
         if isinstance(value, str):
             s = value.strip()
             # minimal tolerant parse for "[x,y,z]" or "x,y,z"
             if s.startswith("[") and s.endswith("]"):
                 s = s[1:-1]
-            parts = [p.strip() for p in s.split(",")]
+            # support "x,y,z" and "x y z"
+            parts = [p.strip() for p in (s.split(",") if "," in s else s.split())]
             if len(parts) == 3:
-                try:
-                    return [float(parts[0]), float(parts[1]), float(parts[2])]
-                except Exception:
-                    return default
+                return _to_vec3(parts)
         return default
 
     position = _coerce_vec(position, default=position)
@@ -112,6 +114,10 @@ def _coerce_vec(value, default=None):
     includeNonPublicSerialized = _coerce_bool(includeNonPublicSerialized)
 
     try:
+        # Map tag to search_term when search_method is by_tag for backward compatibility
+        if action == "find" and search_method == "by_tag" and tag is not None and search_term is None:
+            search_term = tag
+
         # Validate parameter usage to prevent silent failures
         if action == "find":
             if name is not None:

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

@@ -13,8 +13,8 @@ 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[str,
-                           "Build index for load/build settings actions (pass as quoted string, e.g., '0')"] | None = None,
+    build_index: Annotated[int | str,
+                           "Build index for load/build settings actions (accepts int or string, e.g., 0 or '0')"] | None = None,
 ) -> dict[str, Any]:
     ctx.info(f"Processing manage_scene: {action}")
     try:

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

@@ -10,7 +10,7 @@
     description="Manages shader scripts in Unity (create, read, update, delete)."
 )
 def manage_shader(
-    ctx: Context,x
+    ctx: Context,
     action: Annotated[Literal['create', 'read', 'update', 'delete'], "Perform CRUD operations on shader scripts."],
     name: Annotated[str, "Shader name (no .cs extension)"],
     path: Annotated[str, "Asset path (default: \"Assets/\")"],

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

@@ -16,7 +16,7 @@ def read_console(
     action: Annotated[Literal['get', 'clear'], "Get or clear the Unity Editor console."] | None = None,
     types: Annotated[list[Literal['error', 'warning',
                                   'log', 'all']], "Message types to get"] | None = None,
-    count: Annotated[str, "Max messages to return (pass as quoted string, e.g., '5')"] | None = None,
+    count: Annotated[int | str, "Max messages to return (accepts int or string, e.g., 5 or '5')"] | None = None,
     filter_text: Annotated[str, "Text filter for messages"] | None = None,
     since_timestamp: Annotated[str,
                                "Get messages after this timestamp (ISO 8601)"] | None = None,

MCPForUnity/package.json

@@ -1,6 +1,6 @@
 {
   "name": "com.coplaydev.unity-mcp",
-  "version": "6.2.2",
+  "version": "6.2.3",
   "displayName": "MCP for Unity",
   "description": "A bridge that connects AI assistants to Unity via the MCP (Model Context Protocol). Allows AI clients like Claude Code, Cursor, and VSCode to directly control your Unity Editor for enhanced development workflows.\n\nFeatures automated setup wizard, cross-platform support, and seamless integration with popular AI development tools.\n\nJoin Our Discord: https://discord.gg/y4p8KfzrN4",
   "unity": "2021.3",

tests/test_get_sha.py

@@ -10,6 +10,8 @@
 
 def _load_module(path: pathlib.Path, name: str):
     spec = importlib.util.spec_from_file_location(name, path)
+    if spec is None or spec.loader is None:
+        raise ImportError(f"Cannot load module {name} from {path}")
     mod = importlib.util.module_from_spec(spec)
     spec.loader.exec_module(mod)
     return mod

tests/test_manage_asset_param_coercion.py

@@ -12,6 +12,8 @@
 
 def _load_module(path: pathlib.Path, name: str):
     spec = importlib.util.spec_from_file_location(name, path)
+    if spec is None or spec.loader is None:
+        raise ImportError(f"Cannot load module {name} from {path}")
     mod = importlib.util.module_from_spec(spec)
     spec.loader.exec_module(mod)
     return mod

tests/test_manage_gameobject_param_coercion.py

@@ -11,6 +11,8 @@
 
 def _load_module(path: pathlib.Path, name: str):
     spec = importlib.util.spec_from_file_location(name, path)
+    if spec is None or spec.loader is None:
+        raise ImportError(f"Cannot load module {name} from {path}")
     mod = importlib.util.module_from_spec(spec)
     spec.loader.exec_module(mod)
     return mod

tests/test_read_console_truncate.py

@@ -9,6 +9,8 @@
 
 def _load_module(path: pathlib.Path, name: str):
     spec = importlib.util.spec_from_file_location(name, path)
+    if spec is None or spec.loader is None:
+        raise ImportError(f"Cannot load module {name} from {path}")
     mod = importlib.util.module_from_spec(spec)
     spec.loader.exec_module(mod)
     return mod