ha-mcp-dev 7.9.0.dev737__tar.gz → 7.9.0.dev738__tar.gz

{ha_mcp_dev-7.9.0.dev737/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.9.0.dev738}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.9.0.dev737"
+version = "7.9.0.dev738"
 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.9.0.dev737 → ha_mcp_dev-7.9.0.dev738}/src/ha_mcp/tools/tools_entities.py

@@ -745,30 +745,45 @@ class EntityTools:
 
         exposure_result: dict[str, bool] | None = succeeded if succeeded else None
 
-        # If no prior phase populated entity_entry, fetch current entity state
-        if not entity_entry:
-            get_msg: dict[str, Any] = {
-                "type": "config/entity_registry/get",
-                "entity_id": entity_id,
-            }
-            get_result = await self._client.send_websocket_message(get_msg)
-            if get_result.get("success"):
-                entity_entry = get_result.get("result") or {}
-            else:
-                raise_tool_error(
-                    create_error_response(
-                        ErrorCode.ENTITY_NOT_FOUND,
-                        f"Entity '{entity_id}' not found in registry after applying exposure changes",
-                        context={
-                            "entity_id": entity_id,
-                            "exposure_succeeded": exposure_result,
-                        },
-                        suggestions=[
-                            "Verify the entity_id exists using ha_search()",
-                            "The entity's exposure settings were likely changed, but its current state could not be confirmed.",
-                        ],
-                    )
+        # Exposure mutates the registry entry's options, so refetch to return the
+        # post-exposure state. This is unconditional: the method returns early on
+        # an empty expose_to (line 695) and any exposure failure already raised
+        # above, so reaching here always means at least one exposure was applied.
+        get_msg: dict[str, Any] = {
+            "type": "config/entity_registry/get",
+            "entity_id": entity_id,
+        }
+        get_result = await self._client.send_websocket_message(get_msg)
+        if get_result.get("success"):
+            entity_entry = get_result.get("result") or {}
+        elif entity_entry:
+            # The exposure already committed; only the cosmetic post-exposure
+            # refresh failed. Keep the pre-exposure snapshot and warn rather
+            # than reporting the whole (successful) operation as a failure.
+            logger.warning(
+                f"Exposure applied to {entity_id} but its registry state could "
+                f"not be refreshed: {_extract_ws_error(get_result)}"
+            )
+        else:
+            # No prior-phase snapshot to fall back on -- surface the read
+            # failure with the actual WebSocket error.
+            raise_tool_error(
+                create_error_response(
+                    ErrorCode.ENTITY_NOT_FOUND,
+                    f"Entity '{entity_id}' could not be read after applying "
+                    f"exposure changes: {_extract_ws_error(get_result)}",
+                    context={
+                        "entity_id": entity_id,
+                        "exposure_succeeded": exposure_result,
+                        "has_registry_updates": has_registry_updates,
+                        "options_succeeded": options_succeeded,
+                    },
+                    suggestions=[
+                        "Verify the entity_id exists using ha_search()",
+                        "The entity's exposure settings were likely changed, but its current state could not be confirmed.",
+                    ],
                 )
+            )
 
         return exposure_result, entity_entry
 

{ha_mcp_dev-7.9.0.dev737 → ha_mcp_dev-7.9.0.dev738/src/ha_mcp_dev.egg-info}/PKG-INFO

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