fix: CI ruff lint and E2E voice assistant proxy error propagation

- Let ToolError propagate through proxy instead of wrapping it, so
  FastMCP handles isError flag correctly (fixes voice assistant test)
- Remove trailing whitespace on blank lines in test_traces.py (W293)
- Fix Yoda condition in test_tool_proxy.py (SIM300)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: kingpanther13

src/ha_mcp/tools/tool_proxy.py

@@ -20,6 +20,7 @@
 import typing
 from typing import Annotated, Any
 
+from fastmcp.exceptions import ToolError
 from pydantic import Field
 
 from ..errors import (
@@ -573,6 +574,8 @@ async def ha_execute_tool(
             result = await implementation(**parsed_args)
             return result
 
+        except ToolError:
+            raise  # Let ToolErrors propagate — FastMCP handles isError flag
         except TypeError as e:
             return create_validation_error(
                 message=f"Parameter error: {e}",

tests/src/e2e/workflows/automation/test_traces.py

@@ -170,18 +170,18 @@ async def check_automation_traces():
 
             assert detailed_data.get("success") is True
             assert detailed_data.get("run_id") == run_id
-            
+
             # Verify detailed content structure (Deep verification)
             # This ensures we correctly parsed the flat structure (trigger/0, action/0)
             assert "trigger" in detailed_data, "Detailed trace should contain trigger info"
             assert "action_trace" in detailed_data, "Detailed trace should contain action_trace"
             assert isinstance(detailed_data["action_trace"], list), "action_trace should be a list"
             assert len(detailed_data["action_trace"]) > 0, "action_trace should not be empty"
-            
+
             # Check for path property to ensure flat structure parsing worked
             first_action = detailed_data["action_trace"][0]
             assert "path" in first_action, "Action trace element should contain 'path'"
-            
+
             logger.info(f"Detailed trace verified: Found {len(detailed_data['action_trace'])} actions")
 
     async def test_empty_traces_with_diagnostics(

tests/src/unit/test_tool_proxy.py

@@ -219,5 +219,5 @@ class TestProxyModulesConfig:
     def test_expected_modules(self):
         expected = {"tools_zones", "tools_labels", "tools_addons",
                     "tools_voice_assistant", "tools_traces"}
-        assert PROXY_MODULES == expected
+        assert expected == PROXY_MODULES
         assert all(m.startswith("tools_") for m in PROXY_MODULES)