ha-mcp-dev 7.5.0.dev508__tar.gz → 7.5.0.dev509__tar.gz

{ha_mcp_dev-7.5.0.dev508/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.5.0.dev509}/PKG-INFO

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

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

@@ -1135,6 +1135,17 @@ def _apply_array_ops(
                         parameter=f"array_patch.operations[{index}].patches",
                     )
                 )
+            if not patches:
+                # `target.update({})` is a no-op; reject so the caller learns
+                # their op was meaningless rather than seeing a silent
+                # `summary["patched"]` entry with `"fields": []`.
+                raise_tool_error(
+                    create_validation_error(
+                        f"array_patch patch op #{index} 'patches' cannot be empty "
+                        "(no fields to update)",
+                        parameter=f"array_patch.operations[{index}].patches",
+                    )
+                )
             target = next(
                 (
                     it
@@ -1197,6 +1208,17 @@ def _apply_array_ops(
                     )
                 )
             new_id = new_item[id_field]
+            if new_id is None or new_id == "":
+                # Items missing the id field have `dict.get(id_field) == None`
+                # by default, so allowing None/"" ids would let later patch /
+                # delete ops match unrelated items.
+                raise_tool_error(
+                    create_validation_error(
+                        f"array_patch add op #{index} item {id_field!r} cannot be "
+                        "None or an empty string",
+                        parameter=f"array_patch.operations[{index}].item.{id_field}",
+                    )
+                )
             if any(
                 isinstance(it, dict) and it.get(id_field) == new_id for it in working
             ):
@@ -1235,9 +1257,23 @@ def _apply_array_ops(
                 if not (isinstance(it, dict) and it.get(field) == value)
             ]
             removed = before - len(working)
-            summary["deleted_where"].append(
-                {"field": field, "value": value, "count": removed}
-            )
+            entry: dict[str, Any] = {"field": field, "value": value, "count": removed}
+            # Distinguish "value not present" from "field name unknown to any
+            # item" — the latter is almost always a typo and would otherwise
+            # be a silent count=0. Only warn when there are dict items to
+            # inspect; an empty array or all-non-dict array would `not any(...)`
+            # trivially and produce a misleading typo suggestion.
+            inspectable = [it for it in working if isinstance(it, dict)]
+            if (
+                removed == 0
+                and inspectable
+                and not any(field in it for it in inspectable)
+            ):
+                entry["warning"] = (
+                    f"field {field!r} is not present on any item — "
+                    "check for a typo in the field name"
+                )
+            summary["deleted_where"].append(entry)
 
     return working, summary
 
@@ -1830,9 +1866,9 @@ def register_addon_tools(mcp: Any, client: HomeAssistantClient, **kwargs: Any) -
                     "Proxy/array-patch mode: extra HTTP headers to send to the addon API. "
                     "Useful for addon-specific requirements such as Node-RED's "
                     "`Node-RED-Deployment-Type: full`. The proxy's internal framing "
-                    "(`X-Ingress-Path`, `X-Hass-Source`, `Content-Type`) is layered on "
-                    "top, so caller-supplied values for those keys are overridden. "
-                    "Not valid in config mode."
+                    "(`X-Ingress-Path`, `X-Hass-Source`, `Cookie`, `Content-Type`) is "
+                    "layered on top, so caller-supplied values for those keys are "
+                    "overridden. Not valid in config or websocket mode."
                 ),
                 default=None,
             ),
@@ -2049,6 +2085,19 @@ def register_addon_tools(mcp: Any, client: HomeAssistantClient, **kwargs: Any) -
                     )
                 )
 
+        # request_headers applies only to HTTP / array_patch mode.
+        # _call_addon_ws does not accept caller headers — reject the combo
+        # rather than silently dropping them (matches the fail-loud-on-misroute
+        # pattern used for message_limit / message_offset / summarize on HTTP).
+        if request_headers is not None and websocket:
+            raise_tool_error(
+                create_validation_error(
+                    "request_headers applies only to HTTP and array_patch modes; "
+                    "remove it or set websocket=False",
+                    parameter="request_headers",
+                )
+            )
+
         # Config mode: update Supervisor settings
         if config_data:
             ignored_fields: list[str] = []  # populated only when options are provided

{ha_mcp_dev-7.5.0.dev508 → ha_mcp_dev-7.5.0.dev509/src/ha_mcp_dev.egg-info}/PKG-INFO

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