ha-mcp-dev 7.5.0.dev510__tar.gz → 7.5.0.dev512__tar.gz

{ha_mcp_dev-7.5.0.dev510/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.5.0.dev512}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 7.5.0.dev510
+Version: 7.5.0.dev512
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Author-email: Julien <github@qc-h.net>
 License: MIT

{ha_mcp_dev-7.5.0.dev510 → ha_mcp_dev-7.5.0.dev512}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.5.0.dev510"
+version = "7.5.0.dev512"
 description = "Home Assistant MCP Server - Complete control of Home Assistant through MCP"
 readme = "README.md"
 requires-python = ">=3.13,<3.14"

{ha_mcp_dev-7.5.0.dev510 → ha_mcp_dev-7.5.0.dev512}/src/ha_mcp/tools/tools_config_automations.py

@@ -16,7 +16,6 @@ from ..errors import (
     ErrorCode,
     create_config_error,
     create_error_response,
-    create_resource_not_found_error,
     create_validation_error,
 )
 from ..utils.config_hash import compute_config_hash
@@ -304,26 +303,9 @@ class AutomationConfigTools:
                 "config": normalized_config,
                 "config_hash": config_hash,
             }
+        except ToolError:
+            raise
         except Exception as e:
-            # Handle 404 errors gracefully (often used to verify deletion)
-            error_str = str(e)
-            if (
-                "404" in error_str
-                or "not found" in error_str.lower()
-                or "entity not found" in error_str.lower()
-            ):
-                logger.debug(
-                    f"Automation {identifier} not found (expected for deletion verification)"
-                )
-                error_response = create_resource_not_found_error(
-                    "Automation",
-                    identifier,
-                    details=f"Automation '{identifier}' does not exist in Home Assistant",
-                )
-                error_response["action"] = "get"
-                error_response["reason"] = "not_found"
-                raise_tool_error(error_response)
-
             logger.error(f"Error getting automation: {e}")
             exception_to_structured_error(
                 e,
@@ -1003,30 +985,19 @@ class AutomationConfigTools:
                     result["warning"] = f"Deletion confirmed but removal verification failed: {e}"
 
             return {"success": True, "action": "delete", **result}
+        except ToolError:
+            raise
         except Exception as e:
             logger.error(f"Error deleting automation: {e}")
-            error_str = str(e).lower()
-            if "404" in error_str or "not found" in error_str:
-                error_response = create_resource_not_found_error(
-                    "Automation",
-                    identifier,
-                    details=f"Automation '{identifier}' does not exist",
-                )
-            else:
-                error_response = exception_to_structured_error(
-                    e,
-                    context={"identifier": identifier},
-                    raise_error=False,
-                )
-            error_response["action"] = "delete"
-            # Add automation-specific suggestions
-            if "error" in error_response and isinstance(error_response["error"], dict):
-                error_response["error"]["suggestions"] = [
+            exception_to_structured_error(
+                e,
+                context={"identifier": identifier, "action": "delete"},
+                suggestions=[
                     "Verify automation exists using ha_search_entities(domain_filter='automation')",
                     "Use entity_id format: automation.morning_routine or unique_id",
                     "Check Home Assistant connection",
-                ]
-            raise_tool_error(error_response)
+                ],
+            )
 
 
 def register_config_automation_tools(mcp: Any, client: Any, **kwargs: Any) -> None:

{ha_mcp_dev-7.5.0.dev510 → ha_mcp_dev-7.5.0.dev512}/src/ha_mcp/tools/tools_config_dashboards.py

@@ -16,6 +16,7 @@ from ..errors import ErrorCode, create_error_response, create_resource_not_found
 from ..utils.config_hash import compute_config_hash
 from ..utils.python_sandbox import (
     PythonSandboxError,
+    PythonSandboxExecutionError,
     format_sandbox_error,
     get_security_documentation,
     safe_execute,
@@ -1096,6 +1097,19 @@ def register_config_dashboard_tools(mcp: Any, client: Any, **kwargs: Any) -> Non
                     transformed_config = safe_execute(python_transform, current_config)
                 except PythonSandboxError as e:
                     message, suggestions = format_sandbox_error(e, python_transform)
+                    # A path-shape mismatch (IndexError/KeyError) is almost always
+                    # a hallucinated path; steer the retry toward search mode so
+                    # the next transform is built from a verified jq_path.
+                    if isinstance(e, PythonSandboxExecutionError) and isinstance(
+                        e.__cause__, (IndexError, KeyError)
+                    ):
+                        suggestions = [
+                            "Call ha_config_get_dashboard with card_type=..., "
+                            "entity_id=..., or heading=... to get the verified "
+                            "jq_path for the target card, then build "
+                            "python_transform from that path",
+                            *suggestions,
+                        ]
                     raise_tool_error(
                         create_error_response(
                             ErrorCode.VALIDATION_FAILED,

{ha_mcp_dev-7.5.0.dev510 → ha_mcp_dev-7.5.0.dev512}/src/ha_mcp/utils/python_sandbox.py

@@ -192,6 +192,7 @@ SAFE_METHODS = {
     "strip",
     "split",
     "join",
+    "replace",
 }
 
 # Blocked function names
@@ -514,7 +515,7 @@ PYTHON TRANSFORM SECURITY:
 - Dict unpacking (**) in calls and dict literals: {**d, 'k': v}
 - Keyword arguments: func(key=value)
 - Lambdas (e.g. for `key=`): sorted(items, key=lambda x: x['score'])
-- String methods: startswith, endswith, lower, upper, split, join
+- String methods: startswith, endswith, lower, upper, strip, split, join, replace
 - Safe builtins: isinstance, len, range, enumerate, zip, sorted, reversed,
   min, max, sum, abs, any, all, round, str, int, float, bool, list, dict,
   tuple, set

{ha_mcp_dev-7.5.0.dev510 → ha_mcp_dev-7.5.0.dev512/src/ha_mcp_dev.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 7.5.0.dev510
+Version: 7.5.0.dev512
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Author-email: Julien <github@qc-h.net>
 License: MIT