ha-mcp-dev 7.13.0.dev1992__tar.gz → 7.13.0.dev1997__tar.gz

{ha_mcp_dev-7.13.0.dev1992/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.13.0.dev1997}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.13.0.dev1992"
+version = "7.13.0.dev1997"
 description = "Home Assistant MCP Server - Complete control of Home Assistant through MCP"
 readme = "README.md"
 # Upper bound widened to <3.15 so the package installs on Home Assistant's

{ha_mcp_dev-7.13.0.dev1992 → ha_mcp_dev-7.13.0.dev1997}/src/ha_mcp/tools/tools_dev.py

@@ -834,7 +834,9 @@ class DevTools:
             Field(
                 description=(
                     "info: deployment/version report; update_source: point the "
-                    "in-process server at a channel or pip spec and reinstall; "
+                    "ha_mcp_tools component's separate in-process server at a "
+                    "channel or pip spec and reinstall it (never changes the "
+                    "server serving this connection, unless embedded); "
                     "restart: restart this server"
                 )
             ),
@@ -871,14 +873,21 @@ class DevTools:
         as a PR tarball, and restarting the server so config or code
         changes take effect.
 
-        Caveats: restart interrupts this MCP connection in embedded and
-        add-on deployments (the reply arrives just before the server
-        goes down); update_source only self-interrupts in embedded mode
-        — elsewhere it reloads the separate in-process server entry
-        without dropping this connection. Reinstalls can take minutes.
-        update_source requires the ha_mcp_tools component's in-process
-        server entry; restart supports embedded and add-on deployments
-        only (standalone processes must be restarted externally).
+        Caveats: update_source changes ONLY the ha_mcp_tools custom
+        component's separate in-process server entry — it never updates
+        the add-on, Docker, standalone, or PyPI server that may be
+        serving this connection (update those via ha_manage_addon /
+        docker pull / pip). In embedded mode that entry IS this server,
+        so the update self-interrupts; elsewhere this connection is
+        untouched and keeps its current version. Success means the
+        entry's options were applied — the component then reinstalls in
+        the background, which can take minutes and can still fail
+        (check HA logs). update_source requires the component's
+        in-process server entry to exist. restart interrupts this MCP
+        connection in embedded and add-on deployments (the reply
+        arrives just before the server goes down) and supports those
+        two deployments only (standalone processes must be restarted
+        externally).
 
         EXAMPLES:
         ha_dev_manage_server("info")
@@ -936,6 +945,17 @@ class DevTools:
                     "entry_id": entry_id,
                     "channel": options.get(_OPT_CHANNEL),
                     "pip_spec": options.get(_OPT_PIP_SPEC),
+                    "role": (
+                        "this server (embedded)"
+                        if mode == "embedded"
+                        else (
+                            "separate in-process server run by the "
+                            "ha_mcp_tools component; the update_source "
+                            "target. server_version above describes the "
+                            f"{mode} server handling this connection, not "
+                            "this entry."
+                        )
+                    ),
                 }
         except Exception as exc:
             # Best-effort probe: a failure here (including the ToolError the
@@ -1020,6 +1040,9 @@ class DevTools:
                 "data": {
                     "scheduled": True,
                     "entry_id": entry_id,
+                    "target": (
+                        "this server (the embedded ha_mcp_tools in-process entry)"
+                    ),
                     "applying": user_input,
                     "previous": {
                         _OPT_CHANNEL: current.get(_OPT_CHANNEL),
@@ -1047,14 +1070,19 @@ class DevTools:
             "success": True,
             "data": {
                 "entry_id": entry_id,
+                "target": "ha_mcp_tools in-process server entry",
                 "applied": user_input,
                 "previous": {
                     _OPT_CHANNEL: current.get(_OPT_CHANNEL),
                     _OPT_PIP_SPEC: current.get(_OPT_PIP_SPEC),
                 },
                 "note": (
-                    "The component is reloading the in-process server with "
-                    "the new source; installs can take a few minutes."
+                    "Applied to the ha_mcp_tools component's SEPARATE "
+                    "in-process server entry, which is now reinstalling in "
+                    "the background (can take minutes and can still fail — "
+                    "check HA logs). The server handling this connection is "
+                    "NOT affected and keeps its current version; verify the "
+                    "in-process server on its own connect URL, not here."
                 ),
             },
         }

{ha_mcp_dev-7.13.0.dev1992 → ha_mcp_dev-7.13.0.dev1997}/src/ha_mcp/tools/tools_labels.py

@@ -31,6 +31,73 @@ class LabelTools:
     def __init__(self, client: Any) -> None:
         self._client = client
 
+    async def _list_labels(
+        self, context: dict[str, Any] | None = None
+    ) -> list[dict[str, Any]]:
+        """Return all labels from the registry (shared by get/set).
+
+        Raises ToolError (SERVICE_CALL_FAILED) if the list call fails or returns
+        an unexpected non-list envelope — a degraded response must not collapse
+        to an empty registry, or callers would confidently report a real label
+        as missing (mirrors ``backup_manager._require_list``).
+        """
+        result = await self._client.send_websocket_message(
+            {"type": "config/label_registry/list"}
+        )
+        if not result.get("success"):
+            raise_tool_error(
+                create_error_response(
+                    ErrorCode.SERVICE_CALL_FAILED,
+                    result.get("error", "Failed to get labels"),
+                    context=context,
+                )
+            )
+        # No ``[]`` default: a success envelope that omits ``result`` (or sends
+        # a non-list) is a degraded response, not an empty registry — defaulting
+        # would let callers confidently report a real label as missing.
+        labels = result.get("result")
+        if not isinstance(labels, list):
+            raise_tool_error(
+                create_error_response(
+                    ErrorCode.SERVICE_CALL_FAILED,
+                    "Label registry returned a missing or non-list result",
+                    context=context,
+                )
+            )
+        return labels
+
+    async def _require_existing_label(self, label_id: str, name: str) -> None:
+        """Raise RESOURCE_NOT_FOUND if ``label_id`` is not in the registry.
+
+        Enforces the strict update-only contract (issue #1860): update of an
+        unknown id yields an opaque "Unknown error", and create cannot honor a
+        caller-supplied id (HA derives it from the name), so an unknown id is a
+        clear error with a create hint rather than a silently divergent upsert.
+        """
+        existing = await self._list_labels(context={"name": name, "label_id": label_id})
+        if any(lbl.get("label_id") == label_id for lbl in existing):
+            return
+        raise_tool_error(
+            create_error_response(
+                ErrorCode.RESOURCE_NOT_FOUND,
+                f"Label not found: {label_id}",
+                context={
+                    "name": name,
+                    "label_id": label_id,
+                    "available_label_ids": [
+                        lbl.get("label_id") for lbl in existing[:10]
+                    ],
+                },
+                suggestions=[
+                    "To create a new label, omit label_id — Home Assistant "
+                    + "derives the id from the name (e.g. 'vendor:tapo' becomes "
+                    + "'vendor_tapo')",
+                    "To update an existing label, pass its exact current "
+                    + "label_id (list all with ha_config_get_label())",
+                ],
+            )
+        )
+
     @tool(
         name="ha_config_get_label",
         tags={"Labels & Categories"},
@@ -82,22 +149,7 @@ class LabelTools:
                     ],
                     context={"action": "get"},
                 )
-            message: dict[str, Any] = {
-                "type": "config/label_registry/list",
-            }
-
-            result = await self._client.send_websocket_message(message)
-
-            if not result.get("success"):
-                raise_tool_error(
-                    create_error_response(
-                        ErrorCode.SERVICE_CALL_FAILED,
-                        result.get("error", "Failed to get labels"),
-                        context={"label_id": label_id},
-                    )
-                )
-
-            labels = result.get("result", [])
+            labels = await self._list_labels(context={"label_id": label_id})
 
             if label_id is None:
                 return {
@@ -223,6 +275,13 @@ class LabelTools:
                     ],
                     context={"action": "set", "name": name},
                 )
+                # Strict update-only contract (issue #1860): routing to
+                # label_registry/update with an unknown id returns an opaque
+                # "Unknown error", and label_registry/create cannot honor a
+                # caller-supplied id (HA derives it from the name). Verify the
+                # id exists up front and return actionable guidance instead of
+                # dispatching an update that fails cryptically.
+                await self._require_existing_label(label_id, name)
             action = "update" if label_id else "create"
 
             message: dict[str, Any] = {
@@ -254,6 +313,13 @@ class LabelTools:
                     "message": f"Successfully {action_past} label: {name}",
                 }
             else:
+                # The unknown-id case is caught up front by
+                # _require_existing_label. This substring match only catches HA
+                # error texts containing "not found"/"doesn't exist"; HA's label
+                # update does NOT emit those for an unknown/deleted id (it
+                # surfaces "Unknown error"), so it is a best-effort guard for
+                # other/future phrasings only — the check-then-update delete race
+                # is not handled here and still surfaces as SERVICE_CALL_FAILED.
                 error_str = str(result.get("error", "")).lower()
                 if "not found" in error_str or "doesn't exist" in error_str:
                     raise_tool_error(

{ha_mcp_dev-7.13.0.dev1992 → ha_mcp_dev-7.13.0.dev1997/src/ha_mcp_dev.egg-info}/PKG-INFO

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