ha-mcp-dev 7.13.0.dev2031__tar.gz → 7.13.0.dev2035__tar.gz

{ha_mcp_dev-7.13.0.dev2031/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.13.0.dev2035}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.13.0.dev2031"
+version = "7.13.0.dev2035"
 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.dev2031 → ha_mcp_dev-7.13.0.dev2035}/src/ha_mcp/read_only.py

@@ -66,9 +66,11 @@ class ReadOnlyExemption(NamedTuple):
 def _backup_write(args: dict[str, Any]) -> str | None:
     scope = args.get("scope")
     action = args.get("action")
-    # Read-only-safe: per-edit backup listing/viewing, and snapshot listing
+    # Read-only-safe: per-edit backup listing/viewing/diffing, and snapshot listing
     # (issue #1586 — pure ``backup/info`` read, no tarball mutation).
-    if scope == "edits" and action in ("list", "view"):
+    # ``diff`` (added in #1632) only reads the stored snapshot and fetches
+    # the live config to compare; it never mutates (issue #1944).
+    if scope == "edits" and action in ("list", "view", "diff"):
         return None
     if scope == "snapshot" and action == "list":
         return None
@@ -195,8 +197,9 @@ READ_ONLY_EXEMPT_TOOLS: dict[str, ReadOnlyExemption] = {
     ),
     "ha_manage_backup": ReadOnlyExemption(
         _backup_write,
-        "listing and viewing per-edit backups (scope='edits', action='list' or "
-        "'view') and listing snapshots (scope='snapshot', action='list')",
+        "listing, viewing, and diffing per-edit backups (scope='edits', "
+        "action='list', 'view', or 'diff') and listing snapshots "
+        "(scope='snapshot', action='list')",
     ),
     "ha_manage_addon": ReadOnlyExemption(
         _addon_write,

{ha_mcp_dev-7.13.0.dev2031 → ha_mcp_dev-7.13.0.dev2035}/src/ha_mcp/tools/tools_yaml_read.py

@@ -82,6 +82,28 @@ def _failure_text(payload: dict[str, Any]) -> str:
     return str(error or "read failed")
 
 
+def _parse_error_result(
+    target: str, yaml_path: str, parse_error: str, *, is_glob: bool
+) -> tuple[None, str]:
+    """Decide a file the component could not parse: raise for a single target.
+
+    A single explicit target has no siblings to salvage, so a parse failure
+    raises rather than soft-degrading to a warning that reads as "key absent".
+    Under a glob it stays a per-file warning so one broken file does not sink
+    the whole search.
+    """
+    if not is_glob:
+        raise_tool_error(
+            create_error_response(
+                ErrorCode.CONFIG_INVALID,
+                f"{target} could not be parsed as YAML: {parse_error}",
+                suggestions=[f"Fix the YAML syntax error in {target} and retry."],
+                context={"file": target, "yaml_path": yaml_path},
+            )
+        )
+    return None, f"{target} was not searched: {parse_error}."
+
+
 def _evaluate_read(
     response: Any,
     target: str,
@@ -125,7 +147,7 @@ def _evaluate_read(
 
     parse_error = unwrapped.get("parse_error")
     if parse_error:
-        return None, f"{target} was not searched: {parse_error}."
+        return _parse_error_result(target, yaml_path, str(parse_error), is_glob=is_glob)
 
     if unwrapped.get("subtree") is None:
         return None, None

{ha_mcp_dev-7.13.0.dev2031 → ha_mcp_dev-7.13.0.dev2035/src/ha_mcp_dev.egg-info}/PKG-INFO

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