ha-mcp-dev 6.7.2.dev248__tar.gz → 6.7.2.dev250__tar.gz

{ha_mcp_dev-6.7.2.dev248/src/ha_mcp_dev.egg-info → ha_mcp_dev-6.7.2.dev250}/PKG-INFO

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

{ha_mcp_dev-6.7.2.dev248 → ha_mcp_dev-6.7.2.dev250}/pyproject.toml

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

{ha_mcp_dev-6.7.2.dev248 → ha_mcp_dev-6.7.2.dev250}/src/ha_mcp/client/rest_client.py

@@ -831,18 +831,58 @@ class HomeAssistantClient:
                 "template": message.get("template"),
             }
 
+    async def _resolve_script_id(self, identifier: str) -> str:
+        """
+        Resolve a script identifier to its storage key via the entity registry.
+
+        Scripts may be renamed in the HA UI, changing the entity_id but keeping
+        the original storage key. This method looks up the entity registry via
+        WebSocket to find the actual storage key (unique_id).
+
+        Unlike automations (which expose their storage key in state attributes),
+        scripts require a WebSocket entity registry lookup.
+
+        Args:
+            identifier: Script ID (with or without 'script.' prefix)
+
+        Returns:
+            The storage key for the configuration API
+        """
+        bare_id = identifier.removeprefix("script.")
+        entity_id = f"script.{bare_id}"
+        try:
+            result = await self.send_websocket_message(
+                {"type": "config/entity_registry/get", "entity_id": entity_id}
+            )
+            if result.get("success") is not False:
+                unique_id = result.get("result", {}).get("unique_id")
+                if unique_id:
+                    if unique_id != bare_id:
+                        logger.debug(
+                            f"Resolved script entity_id {entity_id} to storage key {unique_id}"
+                        )
+                    return str(unique_id)
+        except Exception:
+            logger.debug(
+                f"Entity registry lookup failed for {entity_id}, using bare id: {bare_id}",
+                exc_info=True # Log full traceback for better debugging
+            )
+        return bare_id
+
     async def get_script_config(self, script_id: str) -> dict[str, Any]:
         """Get Home Assistant script configuration by script_id."""
+        resolved_id = await self._resolve_script_id(script_id)
         try:
-            endpoint = f"config/script/config/{script_id}"
+            endpoint = f"config/script/config/{resolved_id}"
             response = await self._request("GET", endpoint)
 
-            return {"success": True, "script_id": script_id, "config": response}
+            return {"success": True, "script_id": resolved_id, "config": response}
         except HomeAssistantAPIError as e:
             if e.status_code == 404:
-                raise HomeAssistantAPIError(
-                    f"Script not found: {script_id}", status_code=404
-                ) from e
+                msg = f"Script not found: {script_id}"
+                if resolved_id != script_id:
+                    msg += f" (resolved storage key: {resolved_id})"
+                raise HomeAssistantAPIError(msg, status_code=404) from e
             raise
         except Exception as e:
             logger.error(f"Failed to get script config for {script_id}: {e}")
@@ -852,8 +892,9 @@ class HomeAssistantClient:
         self, config: dict[str, Any], script_id: str
     ) -> dict[str, Any]:
         """Create or update Home Assistant script configuration."""
+        resolved_id = await self._resolve_script_id(script_id)
         try:
-            endpoint = f"config/script/config/{script_id}"
+            endpoint = f"config/script/config/{resolved_id}"
 
             # Validate required fields
             if "alias" not in config:
@@ -870,7 +911,7 @@ class HomeAssistantClient:
 
             return {
                 "success": True,
-                "script_id": script_id,
+                "script_id": resolved_id,
                 "result": response.get("result", "ok"),
                 "operation": "created" if response.get("result") == "ok" else "updated",
             }
@@ -880,21 +921,23 @@ class HomeAssistantClient:
 
     async def delete_script_config(self, script_id: str) -> dict[str, Any]:
         """Delete Home Assistant script configuration."""
+        resolved_id = await self._resolve_script_id(script_id)
         try:
-            endpoint = f"config/script/config/{script_id}"
+            endpoint = f"config/script/config/{resolved_id}"
             response = await self._request("DELETE", endpoint)
 
             return {
                 "success": True,
-                "script_id": script_id,
+                "script_id": resolved_id,
                 "result": response.get("result", "ok"),
                 "operation": "deleted",
             }
         except HomeAssistantAPIError as e:
             if e.status_code == 404:
-                raise HomeAssistantAPIError(
-                    f"Script not found: {script_id}", status_code=404
-                ) from e
+                msg = f"Script not found: {script_id}"
+                if resolved_id != script_id:
+                    msg += f" (resolved storage key: {resolved_id})"
+                raise HomeAssistantAPIError(msg, status_code=404) from e
             elif e.status_code == 405:
                 raise HomeAssistantAPIError(
                     f"Cannot delete script '{script_id}': The HTTP DELETE method is blocked. "

{ha_mcp_dev-6.7.2.dev248 → ha_mcp_dev-6.7.2.dev250}/src/ha_mcp/tools/tools_search.py

@@ -492,11 +492,19 @@ def register_search_tools(mcp: Any, client: Any, **kwargs: Any) -> None:
                 description="Include entity_id field for entities (None = auto based on level). Full defaults to True.",
             ),
         ] = None,
+        include_notifications: Annotated[
+            bool | str | None,
+            Field(
+                default=True,
+                description="Include active persistent notifications (default: True). Set False to skip.",
+            ),
+        ] = True,
     ) -> dict[str, Any]:
         """Get AI-friendly system overview with intelligent categorization.
 
         Returns comprehensive system information at the requested detail level,
-        including Home Assistant base_url, version, location, timezone, and entity overview.
+        including Home Assistant base_url, version, location, timezone, entity overview,
+        and active persistent notifications (if any).
         Use 'standard' (default) for most queries. Optionally customize entity fields and limits.
         """
         # Coerce boolean parameters that may come as strings from XML-style calls
@@ -506,6 +514,9 @@ def register_search_tools(mcp: Any, client: Any, **kwargs: Any) -> None:
         include_entity_id_bool = coerce_bool_param(
             include_entity_id, "include_entity_id", default=None
         )
+        include_notifications_bool = coerce_bool_param(
+            include_notifications, "include_notifications", default=True
+        )
 
         result = await smart_tools.get_system_overview(
             detail_level,
@@ -544,6 +555,29 @@ def register_search_tools(mcp: Any, client: Any, **kwargs: Any) -> None:
         except Exception as e:
             logger.warning(f"Failed to fetch system info for overview: {e}")
 
+        # Include active persistent notifications
+        if include_notifications_bool:
+            result["notification_count"] = 0
+            try:
+                ws_result = await client.send_websocket_message(
+                    {"type": "persistent_notification/get"}
+                )
+                if ws_result.get("success"):
+                    notifications = ws_result.get("result", [])
+                    result["notification_count"] = len(notifications)
+                    if notifications:
+                        result["notifications"] = [
+                            {
+                                "notification_id": n.get("notification_id"),
+                                "title": n.get("title"),
+                                "message": n.get("message"),
+                                "created_at": n.get("created_at"),
+                            }
+                            for n in notifications
+                        ]
+            except Exception as e:
+                logger.debug(f"Failed to fetch notifications for overview: {e}")
+
         return result
 
     @mcp.tool(

{ha_mcp_dev-6.7.2.dev248 → ha_mcp_dev-6.7.2.dev250/src/ha_mcp_dev.egg-info}/PKG-INFO

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