ha-mcp-dev 7.14.2.dev2083__tar.gz → 7.14.2.dev2086__tar.gz

{ha_mcp_dev-7.14.2.dev2083/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.14.2.dev2086}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.14.2.dev2083"
+version = "7.14.2.dev2086"
 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.14.2.dev2083 → ha_mcp_dev-7.14.2.dev2086}/src/ha_mcp/dashboard_screenshot/capture.py

@@ -855,11 +855,15 @@ async def capture_dashboard_images(
     preset's native orientation. ``full_page`` is a compatibility alias for
     requesting the engine's native ``WIDTHxauto`` viewport.
 
-    The batch is bracketed by a :class:`ThemeGuard` that restores the engine
-    user's saved frontend theme if rendering changed it (issue #1909).
-    ``client`` supplies the guard's non-add-on credential fallback and
-    ``capture_warnings`` collects the guard's non-fatal warnings; both are
-    optional and never affect the captures themselves.
+    The batch used to be bracketed by a :class:`ThemeGuard` that restored the
+    engine user's saved frontend theme when a cold render changed it (issue
+    #1909). That bracket is currently disabled (#1991): upstream Puppet no
+    longer dispatches ``settheme`` on cold renders, so there is nothing to undo.
+    The guard construction and the ``client`` / ``capture_warnings`` plumbing
+    are retained so the bracket can be re-enabled by uncommenting the
+    snapshot/restore calls if a future engine regression reintroduces the write;
+    while disabled ``capture_warnings`` simply stays empty and never affects the
+    captures themselves.
     """
     path = _validate_dashboard_path(dashboard_path)
     options = validate_capture_parameters(
@@ -884,13 +888,16 @@ async def capture_dashboard_images(
     mime_type = _MIME_TYPES[options.image_format]
     captures: list[DashboardImageCapture] = []
 
-    # The engine dispatches a theme write into the authenticated frontend on
-    # cold renders, which Home Assistant persists to the engine user's real
-    # profile (issue #1909). Snapshot before, restore after — including when
-    # the batch fails, since the engine may already have rendered (and
-    # written) before the failure.
+    # ThemeGuard bracket — currently DISABLED (#1991). Stock Puppet used to
+    # dispatch a theme write into the authenticated frontend on cold renders,
+    # which Home Assistant persisted to the engine user's real profile (#1909);
+    # ha-mcp snapshotted before and restored after to undo it. Upstream Puppet
+    # has since fixed the cold-render settheme dispatch, so the bracket is no
+    # longer needed. The guard is still constructed (and the snapshot/restore
+    # calls kept below, commented out) so it can be re-enabled by uncommenting
+    # if a future engine regression reintroduces the write.
     guard = ThemeGuard.for_capture(engine_target.addon_credential, client)
-    await guard.take_snapshot()
+    # await guard.take_snapshot()
     batch_error: ToolError | None = None
     try:
         async with httpx.AsyncClient(
@@ -970,7 +977,7 @@ async def capture_dashboard_images(
         # and its outcome can be attached to the error payload below.
         batch_error = exc
     finally:
-        await guard.restore()
+        # await guard.restore()  # ThemeGuard bracket disabled (#1991) — see above.
         if capture_warnings is not None:
             capture_warnings.extend(guard.warnings)
 

{ha_mcp_dev-7.14.2.dev2083 → ha_mcp_dev-7.14.2.dev2086}/src/ha_mcp/dashboard_screenshot/theme_guard.py

@@ -1,5 +1,11 @@
 """Snapshot and restore the engine user's saved frontend theme (issue #1909).
 
+NOTE: This guard is currently disabled at its call site (``capture.py``)
+because upstream Puppet fixed the cold-render ``settheme`` dispatch that made
+the bracket necessary (#1991). The code here is retained unchanged so the
+bracket can be re-enabled by uncommenting the snapshot/restore calls in
+``capture.py`` if a future engine regression reintroduces the write.
+
 Stock Puppet dispatches Home Assistant's ``settheme`` event on every
 cold-browser render — its ``dark`` query flag is presence-based, so "not
 requested" reaches the frontend as an explicit "light". Home Assistant

{ha_mcp_dev-7.14.2.dev2083 → ha_mcp_dev-7.14.2.dev2086}/src/ha_mcp/read_only.py

@@ -143,14 +143,6 @@ def _updates_write(args: dict[str, Any]) -> str | None:
     return f"action={action!r}"
 
 
-def _dashboard_get_write(args: dict[str, Any]) -> str | None:
-    """Allow config/list/search reads; fail closed on a screenshot render."""
-    include_screenshot = args.get("include_screenshot")
-    if include_screenshot is None or include_screenshot is False:
-        return None
-    return "dashboard screenshot render (Puppet persists theme/dark preferences)"
-
-
 def _radio_write(args: dict[str, Any]) -> str | None:
     action = args.get("action")
     # Reads (allowed): per-node diagnostics, the integration/network summary,
@@ -191,10 +183,6 @@ def _radio_write(args: dict[str, Any]) -> str | None:
 # against the real registered catalog at PR time, so the two sets
 # cannot drift apart silently.
 READ_ONLY_EXEMPT_TOOLS: dict[str, ReadOnlyExemption] = {
-    "ha_config_get_dashboard": ReadOnlyExemption(
-        _dashboard_get_write,
-        "dashboard config, list, and search reads with include_screenshot=false",
-    ),
     "ha_manage_backup": ReadOnlyExemption(
         _backup_write,
         "listing, viewing, and diffing per-edit backups (scope='edits', "

{ha_mcp_dev-7.14.2.dev2083 → ha_mcp_dev-7.14.2.dev2086}/src/ha_mcp/tools/tools_config_dashboards.py

@@ -1749,7 +1749,7 @@ class DashboardConfigTools:
         tags={"Dashboards"},
         annotations={
             "openWorldHint": False,
-            "destructiveHint": False,
+            "readOnlyHint": True,
             "idempotentHint": True,
             "title": "Get Dashboard",
         },

{ha_mcp_dev-7.14.2.dev2083 → ha_mcp_dev-7.14.2.dev2086}/src/ha_mcp/tools/tools_dashboard_screenshot.py

@@ -109,7 +109,7 @@ class DashboardScreenshotTools:
         tags={"Dashboard", "beta"},
         annotations={
             "openWorldHint": False,
-            "destructiveHint": False,
+            "readOnlyHint": True,
             "title": "Get Dashboard Screenshot",
         },
     )

{ha_mcp_dev-7.14.2.dev2083 → ha_mcp_dev-7.14.2.dev2086/src/ha_mcp_dev.egg-info}/PKG-INFO

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