ha-mcp-dev 7.5.0.dev534__tar.gz → 7.5.0.dev536__tar.gz

{ha_mcp_dev-7.5.0.dev534/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.5.0.dev536}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 7.5.0.dev534
+Version: 7.5.0.dev536
 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.dev534 → ha_mcp_dev-7.5.0.dev536}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.5.0.dev534"
+version = "7.5.0.dev536"
 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.dev534 → ha_mcp_dev-7.5.0.dev536}/src/ha_mcp/tools/helpers.py

@@ -396,6 +396,19 @@ def exception_to_structured_error(
 
     error_response = _classify_exception(error, error_str, error_msg, context)
 
+    # Tracebacks are operationally valuable only for genuinely unclassified
+    # exceptions (programmer errors, library bugs) — every other branch in
+    # _classify_exception produces a structured signal that's sufficient on
+    # its own. Logging at exception level here gives operators line numbers
+    # for the bug class where ``str(error)`` is least informative, without
+    # re-introducing the duplicate ERROR-log noise that classified failures
+    # produced.
+    if (
+        isinstance(error_response.get("error"), dict)
+        and error_response["error"].get("code") == ErrorCode.INTERNAL_ERROR
+    ):
+        logger.exception("Unclassified exception: %s", error_msg)
+
     if suggestions and "error" in error_response and isinstance(error_response["error"], dict):
         # Set both `suggestion` (singular, first item) and `suggestions`
         # (plural, full list). create_error_response (errors.py) sets the

{ha_mcp_dev-7.5.0.dev534 → ha_mcp_dev-7.5.0.dev536}/src/ha_mcp/tools/tools_config_automations.py

@@ -317,7 +317,6 @@ class AutomationConfigTools:
             return {
                 "success": True,
                 "action": "get",
-                "identifier": identifier,
                 "automation_id": entity_id or identifier,
                 "config": normalized_config,
                 "config_hash": config_hash,
@@ -325,7 +324,6 @@ class AutomationConfigTools:
         except ToolError:
             raise
         except Exception as e:
-            logger.error(f"Error getting automation: {e}")
             exception_to_structured_error(
                 e,
                 context={"identifier": identifier, "action": "get"},
@@ -761,7 +759,6 @@ class AutomationConfigTools:
         except ToolError:
             raise
         except Exception as e:
-            logger.error(f"Error upserting automation: {e}")
             suggestions = [
                 "Check automation configuration format",
                 "Ensure required fields: alias, trigger, action",
@@ -1033,7 +1030,6 @@ class AutomationConfigTools:
         except ToolError:
             raise
         except Exception as e:
-            logger.error(f"Error deleting automation: {e}")
             exception_to_structured_error(
                 e,
                 context={"identifier": identifier, "action": "delete"},

{ha_mcp_dev-7.5.0.dev534 → ha_mcp_dev-7.5.0.dev536}/src/ha_mcp/tools/tools_config_dashboards.py

@@ -803,12 +803,6 @@ def register_config_dashboard_tools(mcp: Any, client: Any, **kwargs: Any) -> Non
             raise
         except Exception as e:
             if search_mode:
-                logger.error(
-                    f"Error finding card in dashboard: url_path={url_path}, "
-                    f"entity_id={entity_id}, card_type={card_type}, heading={heading}, "
-                    f"error={e}",
-                    exc_info=True,
-                )
                 suggestions = [
                     "Check HA connection",
                     "Verify dashboard with ha_config_get_dashboard(list_only=True)",
@@ -821,7 +815,6 @@ def register_config_dashboard_tools(mcp: Any, client: Any, **kwargs: Any) -> Non
                     "heading": heading,
                 }
             else:
-                logger.error(f"Error getting dashboard: {e}", exc_info=True)
                 suggestions = [
                     "Use ha_config_get_dashboard(list_only=True) to see available dashboards",
                     "Check if you have permission to access this dashboard",
@@ -1482,7 +1475,6 @@ def register_config_dashboard_tools(mcp: Any, client: Any, **kwargs: Any) -> Non
         except ToolError:
             raise
         except Exception as e:
-            logger.error(f"Error setting dashboard: {e}")
             exception_to_structured_error(
                 e,
                 context={"action": "set", "url_path": url_path},
@@ -1564,8 +1556,6 @@ def register_config_dashboard_tools(mcp: Any, client: Any, **kwargs: Any) -> Non
                 else:
                     error_str = str(error_msg)
 
-                logger.error(f"Error deleting dashboard: {error_str}")
-
                 # If the error is "not found" / "doesn't exist", treat as success (idempotent)
                 if (
                     "unable to find" in error_str.lower()
@@ -1606,7 +1596,6 @@ def register_config_dashboard_tools(mcp: Any, client: Any, **kwargs: Any) -> Non
         except ToolError:
             raise
         except Exception as e:
-            logger.error(f"Error deleting dashboard: {e}")
             exception_to_structured_error(
                 e,
                 context={"action": "delete", "url_path": url_path},

{ha_mcp_dev-7.5.0.dev534 → ha_mcp_dev-7.5.0.dev536/src/ha_mcp_dev.egg-info}/PKG-INFO

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