mcpforunityserver 9.3.0b20260128055651__py3-none-any.whl → 9.3.0b20260129121506__py3-none-any.whl

cli/commands/vfx.py

@@ -7,7 +7,9 @@ from typing import Optional, Tuple, Any
 
 from cli.utils.config import get_config
 from cli.utils.output import format_output, print_error, print_success
-from cli.utils.connection import run_command, UnityConnectionError
+from cli.utils.connection import run_command, handle_unity_errors
+from cli.utils.parsers import parse_json_list_or_exit, parse_json_dict_or_exit
+from cli.utils.constants import SEARCH_METHOD_CHOICE_TAGGED
 
 
 _VFX_TOP_LEVEL_KEYS = {"action", "target", "searchMethod", "properties"}
@@ -49,7 +51,8 @@ def particle():
 
 @particle.command("info")
 @click.argument("target")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def particle_info(target: str, search_method: Optional[str]):
     """Get particle system info.
 
@@ -63,19 +66,16 @@ def particle_info(target: str, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @particle.command("play")
 @click.argument("target")
 @click.option("--with-children", is_flag=True, help="Also play child particle systems.")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def particle_play(target: str, with_children: bool, search_method: Optional[str]):
     """Play a particle system.
 
@@ -91,21 +91,18 @@ def particle_play(target: str, with_children: bool, search_method: Optional[str]
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-        if result.get("success"):
-            print_success(f"Playing particle system: {target}")
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
+    if result.get("success"):
+        print_success(f"Playing particle system: {target}")
 
 
 @particle.command("stop")
 @click.argument("target")
 @click.option("--with-children", is_flag=True, help="Also stop child particle systems.")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def particle_stop(target: str, with_children: bool, search_method: Optional[str]):
     """Stop a particle system."""
     config = get_config()
@@ -115,20 +112,17 @@ def particle_stop(target: str, with_children: bool, search_method: Optional[str]
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-        if result.get("success"):
-            print_success(f"Stopped particle system: {target}")
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
+    if result.get("success"):
+        print_success(f"Stopped particle system: {target}")
 
 
 @particle.command("pause")
 @click.argument("target")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def particle_pause(target: str, search_method: Optional[str]):
     """Pause a particle system."""
     config = get_config()
@@ -136,19 +130,16 @@ def particle_pause(target: str, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @particle.command("restart")
 @click.argument("target")
 @click.option("--with-children", is_flag=True)
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def particle_restart(target: str, with_children: bool, search_method: Optional[str]):
     """Restart a particle system."""
     config = get_config()
@@ -158,19 +149,16 @@ def particle_restart(target: str, with_children: bool, search_method: Optional[s
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @particle.command("clear")
 @click.argument("target")
 @click.option("--with-children", is_flag=True)
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def particle_clear(target: str, with_children: bool, search_method: Optional[str]):
     """Clear all particles from a particle system."""
     config = get_config()
@@ -180,13 +168,9 @@ def particle_clear(target: str, with_children: bool, search_method: Optional[str
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 # =============================================================================
@@ -201,7 +185,8 @@ def line():
 
 @line.command("info")
 @click.argument("target")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def line_info(target: str, search_method: Optional[str]):
     """Get line renderer info.
 
@@ -214,19 +199,16 @@ def line_info(target: str, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @line.command("set-positions")
 @click.argument("target")
 @click.option("--positions", "-p", required=True, help='Positions as JSON array: [[0,0,0], [1,1,1], [2,0,0]]')
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def line_set_positions(target: str, positions: str, search_method: Optional[str]):
     """Set all positions on a line renderer.
 
@@ -236,11 +218,7 @@ def line_set_positions(target: str, positions: str, search_method: Optional[str]
     """
     config = get_config()
 
-    try:
-        positions_list = json.loads(positions)
-    except json.JSONDecodeError as e:
-        print_error(f"Invalid JSON for positions: {e}")
-        sys.exit(1)
+    positions_list = parse_json_list_or_exit(positions, "positions")
 
     params: dict[str, Any] = {
         "action": "line_set_positions",
@@ -250,20 +228,17 @@ def line_set_positions(target: str, positions: str, search_method: Optional[str]
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @line.command("create-line")
 @click.argument("target")
 @click.option("--start", nargs=3, type=float, required=True, help="Start point X Y Z")
 @click.option("--end", nargs=3, type=float, required=True, help="End point X Y Z")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def line_create_line(target: str, start: Tuple[float, float, float], end: Tuple[float, float, float], search_method: Optional[str]):
     """Create a simple line between two points.
 
@@ -281,13 +256,9 @@ def line_create_line(target: str, start: Tuple[float, float, float], end: Tuple[
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @line.command("create-circle")
@@ -295,7 +266,8 @@ def line_create_line(target: str, start: Tuple[float, float, float], end: Tuple[
 @click.option("--center", nargs=3, type=float, default=(0, 0, 0), help="Center point X Y Z")
 @click.option("--radius", type=float, required=True, help="Circle radius")
 @click.option("--segments", type=int, default=32, help="Number of segments")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def line_create_circle(target: str, center: Tuple[float, float, float], radius: float, segments: int, search_method: Optional[str]):
     """Create a circle shape.
 
@@ -315,18 +287,15 @@ def line_create_circle(target: str, center: Tuple[float, float, float], radius:
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @line.command("clear")
 @click.argument("target")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def line_clear(target: str, search_method: Optional[str]):
     """Clear all positions from a line renderer."""
     config = get_config()
@@ -334,13 +303,9 @@ def line_clear(target: str, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 # =============================================================================
@@ -355,7 +320,8 @@ def trail():
 
 @trail.command("info")
 @click.argument("target")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def trail_info(target: str, search_method: Optional[str]):
     """Get trail renderer info."""
     config = get_config()
@@ -363,19 +329,16 @@ def trail_info(target: str, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @trail.command("set-time")
 @click.argument("target")
 @click.argument("duration", type=float)
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def trail_set_time(target: str, duration: float, search_method: Optional[str]):
     """Set trail duration.
 
@@ -392,18 +355,15 @@ def trail_set_time(target: str, duration: float, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 @trail.command("clear")
 @click.argument("target")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def trail_clear(target: str, search_method: Optional[str]):
     """Clear a trail renderer."""
     config = get_config()
@@ -411,13 +371,9 @@ def trail_clear(target: str, search_method: Optional[str]):
     if search_method:
         params["searchMethod"] = search_method
 
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(params), config)
+    click.echo(format_output(result, config.format))
 
 
 # =============================================================================
@@ -428,7 +384,8 @@ def trail_clear(target: str, search_method: Optional[str]):
 @click.argument("action")
 @click.argument("target", required=False)
 @click.option("--params", "-p", default="{}", help="Additional parameters as JSON.")
-@click.option("--search-method", type=click.Choice(["by_name", "by_path", "by_id", "by_tag"]), default=None)
+@click.option("--search-method", type=SEARCH_METHOD_CHOICE_TAGGED, default=None)
+@handle_unity_errors
 def vfx_raw(action: str, target: Optional[str], params: str, search_method: Optional[str]):
     """Execute any VFX action directly.
 
@@ -449,14 +406,7 @@ def vfx_raw(action: str, target: Optional[str], params: str, search_method: Opti
     """
     config = get_config()
 
-    try:
-        extra_params = json.loads(params)
-    except json.JSONDecodeError as e:
-        print_error(f"Invalid JSON for params: {e}")
-        sys.exit(1)
-    if not isinstance(extra_params, dict):
-        print_error("Invalid JSON for params: expected an object")
-        sys.exit(1)
+    extra_params = parse_json_dict_or_exit(params, "params")
 
     request_params: dict[str, Any] = {"action": action}
     if target:
@@ -466,10 +416,6 @@ def vfx_raw(action: str, target: Optional[str], params: str, search_method: Opti
 
     # Merge extra params
     request_params.update(extra_params)
-    try:
-        result = run_command(
-            "manage_vfx", _normalize_vfx_params(request_params), config)
-        click.echo(format_output(result, config.format))
-    except UnityConnectionError as e:
-        print_error(str(e))
-        sys.exit(1)
+    result = run_command(
+        "manage_vfx", _normalize_vfx_params(request_params), config)
+    click.echo(format_output(result, config.format))

cli/utils/confirmation.py

@@ -0,0 +1,37 @@
+"""Confirmation dialog utilities for CLI commands."""
+
+import click
+
+
+def confirm_destructive_action(
+    action: str,
+    item_type: str,
+    item_name: str,
+    force: bool,
+    extra_context: str = ""
+) -> None:
+    """Prompt user to confirm destructive action unless --force flag is set.
+
+    Args:
+        action: The action being performed (e.g., "Delete", "Remove")
+        item_type: The type of item (e.g., "script", "GameObject", "asset")
+        item_name: The name/path of the item
+        force: If True, skip confirmation prompt
+        extra_context: Optional additional context (e.g., "from 'Player'")
+
+    Raises:
+        click.Abort: If user declines confirmation
+
+    Examples:
+        confirm_destructive_action("Delete", "script", "MyScript.cs", force=False)
+        # Prompts: "Delete script 'MyScript.cs'?"
+
+        confirm_destructive_action("Remove", "Rigidbody", "Player", force=False, extra_context="from")
+        # Prompts: "Remove Rigidbody from 'Player'?"
+    """
+    if not force:
+        if extra_context:
+            message = f"{action} {item_type} {extra_context} '{item_name}'?"
+        else:
+            message = f"{action} {item_type} '{item_name}'?"
+        click.confirm(message, abort=True)

cli/utils/connection.py

@@ -1,9 +1,9 @@
 """Connection utilities for CLI to communicate with Unity via MCP server."""
 
 import asyncio
-import json
+import functools
 import sys
-from typing import Any, Dict, Optional
+from typing import Any, Callable, Dict, Optional, TypeVar
 
 import httpx
 
@@ -15,6 +15,36 @@ class UnityConnectionError(Exception):
     pass
 
 
+F = TypeVar("F", bound=Callable[..., Any])
+
+
+def handle_unity_errors(func: F) -> F:
+    """Decorator that handles UnityConnectionError consistently.
+
+    Wraps a CLI command function and catches UnityConnectionError,
+    printing a formatted error message and exiting with code 1.
+
+    Usage:
+        @scene.command("active")
+        @handle_unity_errors
+        def active():
+            config = get_config()
+            result = run_command("manage_scene", {"action": "get_active"}, config)
+            click.echo(format_output(result, config.format))
+    """
+    from cli.utils.output import print_error
+
+    @functools.wraps(func)
+    def wrapper(*args: Any, **kwargs: Any) -> Any:
+        try:
+            return func(*args, **kwargs)
+        except UnityConnectionError as e:
+            print_error(str(e))
+            sys.exit(1)
+
+    return wrapper  # type: ignore[return-value]
+
+
 def warn_if_remote_host(config: CLIConfig) -> None:
     """Warn user if connecting to a non-localhost server.
 

cli/utils/constants.py

@@ -0,0 +1,23 @@
+"""Common constants for CLI commands."""
+import click
+
+# Search method constants used across various CLI commands
+# These define how GameObjects and other Unity objects can be located
+
+# Full set of search methods (used by gameobject commands)
+SEARCH_METHODS_FULL = ["by_name", "by_path", "by_id", "by_tag", "by_layer", "by_component"]
+
+# Basic search methods (used by component, animation, audio commands)
+SEARCH_METHODS_BASIC = ["by_id", "by_name", "by_path"]
+
+# Extended search methods for renderer-based commands (material commands)
+SEARCH_METHODS_RENDERER = ["by_id", "by_name", "by_path", "by_tag", "by_layer", "by_component"]
+
+# Tagged search methods (used by VFX commands)
+SEARCH_METHODS_TAGGED = ["by_name", "by_path", "by_id", "by_tag"]
+
+# Click choice options for each set
+SEARCH_METHOD_CHOICE_FULL = click.Choice(SEARCH_METHODS_FULL)
+SEARCH_METHOD_CHOICE_BASIC = click.Choice(SEARCH_METHODS_BASIC)
+SEARCH_METHOD_CHOICE_RENDERER = click.Choice(SEARCH_METHODS_RENDERER)
+SEARCH_METHOD_CHOICE_TAGGED = click.Choice(SEARCH_METHODS_TAGGED)

cli/utils/parsers.py

@@ -0,0 +1,112 @@
+"""JSON and value parsing utilities for CLI commands."""
+import json
+import sys
+from typing import Any
+
+from cli.utils.output import print_error, print_info
+
+
+def parse_value_safe(value: str) -> Any:
+    """Parse a value, trying JSON → float → string fallback.
+
+    This is used for property values that could be JSON objects/arrays,
+    numbers, or strings. Never raises an exception.
+
+    Args:
+        value: The string value to parse
+
+    Returns:
+        Parsed JSON object/array, float, or original string
+
+    Examples:
+        >>> parse_value_safe('{"x": 1}')
+        {'x': 1}
+        >>> parse_value_safe('3.14')
+        3.14
+        >>> parse_value_safe('hello')
+        'hello'
+    """
+    try:
+        return json.loads(value)
+    except json.JSONDecodeError:
+        # Try to parse as number
+        try:
+            return float(value)
+        except ValueError:
+            # Keep as string
+            return value
+
+
+def parse_json_or_exit(value: str, context: str = "parameter") -> Any:
+    """Parse JSON string, trying to fix common issues, or exit with error.
+
+    Attempts to parse JSON with automatic fixes for:
+    - Single quotes instead of double quotes
+    - Python-style True/False instead of true/false
+
+    Args:
+        value: The JSON string to parse
+        context: Description of what's being parsed (for error messages)
+
+    Returns:
+        Parsed JSON object
+
+    Exits:
+        Calls sys.exit(1) if JSON is invalid after attempting fixes
+    """
+    try:
+        return json.loads(value)
+    except json.JSONDecodeError:
+        # Try to fix common shell quoting issues (single quotes, Python bools)
+        try:
+            fixed = value.replace("'", '"').replace("True", "true").replace("False", "false")
+            return json.loads(fixed)
+        except json.JSONDecodeError as e:
+            print_error(f"Invalid JSON for {context}: {e}")
+            print_info("Example: --params '{\"key\":\"value\"}'")
+            print_info("Tip: wrap JSON in single quotes to avoid shell escaping issues.")
+            sys.exit(1)
+
+
+def parse_json_dict_or_exit(value: str, context: str = "parameter") -> dict[str, Any]:
+    """Parse JSON object (dict), or exit with error.
+
+    Like parse_json_or_exit, but ensures result is a dictionary.
+
+    Args:
+        value: The JSON string to parse
+        context: Description of what's being parsed (for error messages)
+
+    Returns:
+        Parsed JSON object as dictionary
+
+    Exits:
+        Calls sys.exit(1) if JSON is invalid or not an object
+    """
+    result = parse_json_or_exit(value, context)
+    if not isinstance(result, dict):
+        print_error(f"Invalid JSON for {context}: expected an object, got {type(result).__name__}")
+        sys.exit(1)
+    return result
+
+
+def parse_json_list_or_exit(value: str, context: str = "parameter") -> list[Any]:
+    """Parse JSON array (list), or exit with error.
+
+    Like parse_json_or_exit, but ensures result is a list.
+
+    Args:
+        value: The JSON string to parse
+        context: Description of what's being parsed (for error messages)
+
+    Returns:
+        Parsed JSON array as list
+
+    Exits:
+        Calls sys.exit(1) if JSON is invalid or not an array
+    """
+    result = parse_json_or_exit(value, context)
+    if not isinstance(result, list):
+        print_error(f"Invalid JSON for {context}: expected an array, got {type(result).__name__}")
+        sys.exit(1)
+    return result

core/config.py

@@ -47,10 +47,6 @@ class ServerConfig:
     # Align with telemetry.py default Cloud Run endpoint
     telemetry_endpoint: str = "https://api-prod.coplay.dev/telemetry/events"
 
-    def configure_logging(self) -> None:
-        level = getattr(logging, self.log_level, logging.INFO)
-        logging.basicConfig(level=level, format=self.log_format)
-
 
 # Create a global config instance
 config = ServerConfig()