ha-mcp-dev 7.9.0.dev757__tar.gz → 7.9.0.dev759__tar.gz

{ha_mcp_dev-7.9.0.dev757/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.9.0.dev759}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 7.9.0.dev757
+Version: 7.9.0.dev759
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Author-email: Julien <github@qc-h.net>
 License: MIT
@@ -462,7 +462,14 @@ This project is licensed under the MIT License - see the [LICENSE](LICENSE) file
 - **[@L1AD](https://github.com/L1AD)** — Filed #966 proposing tool security policies; pointed to PolicyLayer's MCP-security work as prior art that inspired the predicate DSL shape.
 - **[@nightcityblade](https://github.com/nightcityblade)** — Updated stale Home Assistant Advanced Mode references after HA 2026.6 made formerly advanced options available by default (#1533).
 - **[@emmelutzer](https://github.com/emmelutzer)** — Financial support via [GitHub Sponsors](https://github.com/sponsors/julienld). Thank you! ☕
-
+- **[@pkkr](https://github.com/pkkr)** — `ha_knx_get_project` tool exposing KNX group addresses from an uploaded ETS project file.
+- **[@cbowns](https://github.com/cbowns)** — Fixed inconsistent hyphen in setup.astro Codex CLI docs.
+- **[@Shaan-alpha](https://github.com/Shaan-alpha)** — Extended `ha_restart` known-good error patterns to cover 502/503 responses from reverse proxies.
+- **[@rebelancap](https://github.com/rebelancap)** — Fixed UTC-to-local timezone conversion in `add_timezone_metadata`.
+- **[@saevras](https://github.com/saevras)** — Fixed blueprint import E2E test to use local URL instead of host-to-container networking.
+- **[@jasonjhofmann](https://github.com/jasonjhofmann)** — Recurring calendar events via `rrule` support in `ha_config_set_calendar_event`.
+- **[@vpciii](https://github.com/vpciii)** — Coerce JSON-encoded strings on dict/list tool params.
+- **[@pburtchaell](https://github.com/pburtchaell)** — Financial support via [GitHub Sponsors](https://github.com/sponsors/julienld). Thank you! ☕
 ---
 
 ## 💬 Community

{ha_mcp_dev-7.9.0.dev757 → ha_mcp_dev-7.9.0.dev759}/README.md

@@ -430,7 +430,14 @@ This project is licensed under the MIT License - see the [LICENSE](LICENSE) file
 - **[@L1AD](https://github.com/L1AD)** — Filed #966 proposing tool security policies; pointed to PolicyLayer's MCP-security work as prior art that inspired the predicate DSL shape.
 - **[@nightcityblade](https://github.com/nightcityblade)** — Updated stale Home Assistant Advanced Mode references after HA 2026.6 made formerly advanced options available by default (#1533).
 - **[@emmelutzer](https://github.com/emmelutzer)** — Financial support via [GitHub Sponsors](https://github.com/sponsors/julienld). Thank you! ☕
-
+- **[@pkkr](https://github.com/pkkr)** — `ha_knx_get_project` tool exposing KNX group addresses from an uploaded ETS project file.
+- **[@cbowns](https://github.com/cbowns)** — Fixed inconsistent hyphen in setup.astro Codex CLI docs.
+- **[@Shaan-alpha](https://github.com/Shaan-alpha)** — Extended `ha_restart` known-good error patterns to cover 502/503 responses from reverse proxies.
+- **[@rebelancap](https://github.com/rebelancap)** — Fixed UTC-to-local timezone conversion in `add_timezone_metadata`.
+- **[@saevras](https://github.com/saevras)** — Fixed blueprint import E2E test to use local URL instead of host-to-container networking.
+- **[@jasonjhofmann](https://github.com/jasonjhofmann)** — Recurring calendar events via `rrule` support in `ha_config_set_calendar_event`.
+- **[@vpciii](https://github.com/vpciii)** — Coerce JSON-encoded strings on dict/list tool params.
+- **[@pburtchaell](https://github.com/pburtchaell)** — Financial support via [GitHub Sponsors](https://github.com/sponsors/julienld). Thank you! ☕
 ---
 
 ## 💬 Community

{ha_mcp_dev-7.9.0.dev757 → ha_mcp_dev-7.9.0.dev759}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.9.0.dev757"
+version = "7.9.0.dev759"
 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.dev757 → ha_mcp_dev-7.9.0.dev759}/src/ha_mcp/client/rest_client.py

@@ -606,7 +606,7 @@ class HomeAssistantClient:
         self._supervised_detected = is_supervised
         return is_supervised
 
-    async def get_addon_logs(self, slug: str) -> str:
+    async def get_addon_logs(self, slug: str, lines: int | None = None) -> str:
         """Fetch an add-on's container logs.
 
         Branch on ``is_running_in_addon()`` (which keys off ``SUPERVISOR_TOKEN``
@@ -621,6 +621,15 @@ class HomeAssistantClient:
         HA URL), falls back to the HA Core proxy path. That path requires an
         admin LLA but works fine when not invoked from the add-on container.
 
+        ``lines`` sets the journald window Supervisor serves via its
+        ``?lines=`` query param (supervisor/api/host.py). Without it,
+        Supervisor returns only its default window (``DEFAULT_LINES = 100``
+        in supervisor/api/const.py) — which silently capped every larger
+        caller-side limit before this param existed. Both branches carry
+        it: the direct endpoint parses the query natively, and HA Core's
+        hassio proxy forwards ``request.query`` upstream
+        (homeassistant/components/hassio/http.py).
+
         Both branches return ``text/plain`` log content.
 
         Raises:
@@ -633,17 +642,18 @@ class HomeAssistantClient:
             HomeAssistantConnectionError: Network, timeout, or transport error.
         """
         if is_running_in_addon():
-            return await self._get_addon_logs_via_supervisor(slug)
+            return await self._get_addon_logs_via_supervisor(slug, lines=lines)
 
         logger.debug(f"Fetching addon logs for slug={slug} via HA Core proxy")
         response = await self._raw_request(
             "GET",
             f"/hassio/addons/{slug}/logs",
             headers={"Accept": "text/plain"},
+            params={"lines": lines} if lines is not None else None,
         )
         return response.text
 
-    async def _supervisor_logs_get(self, path: str) -> str:
+    async def _supervisor_logs_get(self, path: str, lines: int | None = None) -> str:
         """Fetch ``text/plain`` logs from a Supervisor REST endpoint.
 
         ``path`` is everything between ``http://supervisor/`` and ``/logs``:
@@ -652,6 +662,9 @@ class HomeAssistantClient:
         - ``"<service>"`` (where service ∈ {supervisor, host, core, dns, audio,
           cli, multicast, observer}) for system-service logs
 
+        ``lines`` maps to the endpoint's ``?lines=`` journald-window query
+        param; omitted → Supervisor's 100-line default window.
+
         Bypasses ``HomeAssistantClient.httpx_client`` because the Supervisor
         endpoint uses a different base URL (``http://supervisor``) and a
         different token (``SUPERVISOR_TOKEN``) than HA Core REST. Both
@@ -681,9 +694,10 @@ class HomeAssistantClient:
 
         relative_path = f"/{path}/logs"
         logger.debug(
-            "Fetching %s%s via Supervisor direct",
+            "Fetching %s%s via Supervisor direct (lines=%s)",
             get_supervisor_base_url(),
             relative_path,
+            lines,
         )
 
         try:
@@ -694,6 +708,7 @@ class HomeAssistantClient:
                 response = await client.get(
                     relative_path,
                     headers={"Accept": "text/plain"},
+                    params={"lines": lines} if lines is not None else None,
                 )
         except httpx.TimeoutException as e:
             raise HomeAssistantConnectionError(
@@ -753,7 +768,9 @@ class HomeAssistantClient:
             )
         return response.text
 
-    async def _get_addon_logs_via_supervisor(self, slug: str) -> str:
+    async def _get_addon_logs_via_supervisor(
+        self, slug: str, lines: int | None = None
+    ) -> str:
         """Fetch add-on container logs directly from Supervisor's REST API.
 
         Distinct from ``tools_bug_report._fetch_addon_logs``: that helper is
@@ -766,9 +783,11 @@ class HomeAssistantClient:
         Delegates to ``_supervisor_logs_get`` so error handling stays in
         lockstep with ``_get_system_service_logs``.
         """
-        return await self._supervisor_logs_get(f"addons/{slug}")
+        return await self._supervisor_logs_get(f"addons/{slug}", lines=lines)
 
-    async def _get_system_service_logs(self, service: str) -> str:
+    async def _get_system_service_logs(
+        self, service: str, lines: int | None = None
+    ) -> str:
         """Fetch HA system-service logs.
 
         ``service`` must be one of the eight Supervisor-managed services:
@@ -796,13 +815,14 @@ class HomeAssistantClient:
         logs) call kept working through its own Core-proxy fallback.
         """
         if is_running_in_addon():
-            return await self._supervisor_logs_get(service)
+            return await self._supervisor_logs_get(service, lines=lines)
 
         logger.debug(f"Fetching {service} logs via HA Core proxy")
         response = await self._raw_request(
             "GET",
             f"/hassio/{service}/logs",
             headers={"Accept": "text/plain"},
+            params={"lines": lines} if lines is not None else None,
         )
         return response.text
 

{ha_mcp_dev-7.9.0.dev757 → ha_mcp_dev-7.9.0.dev759}/src/ha_mcp/tools/backup.py

@@ -329,11 +329,19 @@ async def _poll_backup_completion(
         )
 
         if event_state == "failed":
+            # Surface the failure event verbatim — HA's failed event
+            # carries the cause (e.g. another backup already in
+            # progress), and dropping it left CI failures reading
+            # "Backup creation failed: Backup creation failed" with
+            # nothing to diagnose.
             raise_tool_error(
                 create_error_response(
                     ErrorCode.SERVICE_CALL_FAILED,
                     "Backup creation failed",
-                    context={"backup_job_id": backup_job_id},
+                    context={
+                        "backup_job_id": backup_job_id,
+                        "last_action_event": last_event,
+                    },
                 )
             )
 
@@ -410,7 +418,11 @@ async def _poll_backup_completion(
                 create_error_response(
                     ErrorCode.SERVICE_CALL_FAILED,
                     "Backup creation failed (observed at post-timeout lookup)",
-                    context={"backup_job_id": backup_job_id, "name": name},
+                    context={
+                        "backup_job_id": backup_job_id,
+                        "name": name,
+                        "last_action_event": last_event,
+                    },
                 )
             )
         if state != "idle":

{ha_mcp_dev-7.9.0.dev757 → ha_mcp_dev-7.9.0.dev759}/src/ha_mcp/tools/tools_utility.py

@@ -53,6 +53,11 @@ SYSTEM_SERVICE_SLUGS = frozenset(
 
 DEFAULT_LIMIT = 50
 DEFAULT_LOG_LIMIT = 100
+# Journald window to request from Supervisor when a search filter is
+# active: matches are found within the fetched window only, so a
+# search over the caller's (often small) limit needs more history
+# behind it than the limit itself.
+SUPERVISOR_SEARCH_WINDOW_LINES = 2000
 MAX_LIMIT = 500
 
 # Regex to match log level at the start of a log line
@@ -752,8 +757,18 @@ class UtilityTools:
             limit, default=DEFAULT_LOG_LIMIT, suggestion_example="100"
         )
 
+        # Request a journald window sized to the caller's limit —
+        # Supervisor's /logs endpoints default to their last-100-lines
+        # window, which silently capped any larger limit before ?lines=
+        # was plumbed through (found via #1721's e2e).
+        fetch_lines = (
+            max(effective_limit, SUPERVISOR_SEARCH_WINDOW_LINES)
+            if search
+            else effective_limit
+        )
+
         try:
-            log_text = await self._client.get_addon_logs(slug)
+            log_text = await self._client.get_addon_logs(slug, lines=fetch_lines)
 
             lines = log_text.splitlines() if log_text else []
 
@@ -892,8 +907,16 @@ class UtilityTools:
             limit, default=DEFAULT_LOG_LIMIT, suggestion_example="100"
         )
 
+        fetch_lines = (
+            max(effective_limit, SUPERVISOR_SEARCH_WINDOW_LINES)
+            if search
+            else effective_limit
+        )
+
         try:
-            log_text = await self._client._get_system_service_logs(service)
+            log_text = await self._client._get_system_service_logs(
+                service, lines=fetch_lines
+            )
 
             lines = log_text.splitlines() if log_text else []
 

{ha_mcp_dev-7.9.0.dev757 → ha_mcp_dev-7.9.0.dev759}/src/ha_mcp/utils/kill_signal_diagnostics.py

@@ -1,6 +1,6 @@
 """Kill-signal diagnostics for the HA MCP add-on.
 
-Opt-in (gated by the "Advanced debug logging" addon toggle) signal handler
+Opt-in (armed when the addon's effective log level is DEBUG) signal handler
 that, on SIGTERM/SIGINT/SIGHUP, captures and logs:
 
 - Signal name + ``si_code`` (USER/KERNEL/QUEUE/TKILL/...).
@@ -244,7 +244,7 @@ def format_diagnostic_block(
 
     lines = [
         "=" * 80,
-        "ADVANCED DEBUG LOGGING — kill-signal diagnostics",
+        "KILL-SIGNAL DIAGNOSTICS",
         "=" * 80,
         f"Signal:         {sig_name} ({signum})",
         f"si_code:        {code_name}",
@@ -356,7 +356,7 @@ def _make_handler() -> Any:
             try:
                 os.write(
                     2,
-                    f"advanced_debug_logging handler failed for signal {signum}: {exc!r}\n".encode(
+                    f"kill-signal diagnostics handler failed for signal {signum}: {exc!r}\n".encode(
                         "utf-8", errors="replace"
                     ),
                 )
@@ -386,14 +386,14 @@ def install_kill_signal_diagnostics() -> bool:
 
     if sys.platform != "linux":
         logger.warning(
-            "advanced_debug_logging is Linux-only; skipping signal handler install on %s",
+            "kill-signal diagnostics are Linux-only; skipping signal handler install on %s",
             sys.platform,
         )
         return False
 
     if _handler_refs:
         logger.warning(
-            "advanced_debug_logging: install_kill_signal_diagnostics already called; skipping"
+            "kill-signal diagnostics: install_kill_signal_diagnostics already called; skipping"
         )
         return True
 
@@ -401,7 +401,7 @@ def install_kill_signal_diagnostics() -> bool:
         libc_path = ctypes.util.find_library("c")
         if libc_path is None:
             logger.warning(
-                "advanced_debug_logging: libc not found; skipping signal handler install"
+                "kill-signal diagnostics: libc not found; skipping signal handler install"
             )
             return False
 
@@ -441,7 +441,7 @@ def install_kill_signal_diagnostics() -> bool:
             if rc != 0:
                 err = ctypes.get_errno()
                 logger.warning(
-                    "advanced_debug_logging: sigaction(%s) failed: errno=%d",
+                    "kill-signal diagnostics: sigaction(%s) failed: errno=%d",
                     sig.name,
                     err,
                 )
@@ -449,7 +449,7 @@ def install_kill_signal_diagnostics() -> bool:
             installed_for.append(sig.name)
     except Exception as exc:
         logger.warning(
-            "advanced_debug_logging: install failed (%r); continuing without diagnostics",
+            "kill-signal diagnostics: install failed (%r); continuing without diagnostics",
             exc,
         )
         _handler_refs.clear()
@@ -460,7 +460,7 @@ def install_kill_signal_diagnostics() -> bool:
     if installed_for:
         chained_signals = sorted(signal.Signals(s).name for s in _chained_handlers)
         logger.info(
-            "advanced_debug_logging enabled — kill-signal diagnostics installed for: %s "
+            "kill-signal diagnostics installed for: %s "
             "(chains to existing handlers for: %s)",
             ", ".join(installed_for),
             ", ".join(chained_signals) or "<none>",
@@ -502,13 +502,13 @@ def schedule_install_after_uvicorn(
             current = signal.getsignal(signal.SIGTERM)
             if callable(current) and current not in (signal.SIG_DFL, signal.SIG_IGN):
                 logger.debug(
-                    "advanced_debug_logging: detected uvicorn signal handler; installing on top"
+                    "kill-signal diagnostics: detected uvicorn signal handler; installing on top"
                 )
                 install()
                 return
             time.sleep(poll_interval_secs)
         logger.info(
-            "advanced_debug_logging: uvicorn handler not detected within %.1fs; installing anyway",
+            "kill-signal diagnostics: uvicorn handler not detected within %.1fs; installing anyway",
             timeout_secs,
         )
         install()

{ha_mcp_dev-7.9.0.dev757 → ha_mcp_dev-7.9.0.dev759/src/ha_mcp_dev.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 7.9.0.dev757
+Version: 7.9.0.dev759
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Author-email: Julien <github@qc-h.net>
 License: MIT
@@ -462,7 +462,14 @@ This project is licensed under the MIT License - see the [LICENSE](LICENSE) file
 - **[@L1AD](https://github.com/L1AD)** — Filed #966 proposing tool security policies; pointed to PolicyLayer's MCP-security work as prior art that inspired the predicate DSL shape.
 - **[@nightcityblade](https://github.com/nightcityblade)** — Updated stale Home Assistant Advanced Mode references after HA 2026.6 made formerly advanced options available by default (#1533).
 - **[@emmelutzer](https://github.com/emmelutzer)** — Financial support via [GitHub Sponsors](https://github.com/sponsors/julienld). Thank you! ☕
-
+- **[@pkkr](https://github.com/pkkr)** — `ha_knx_get_project` tool exposing KNX group addresses from an uploaded ETS project file.
+- **[@cbowns](https://github.com/cbowns)** — Fixed inconsistent hyphen in setup.astro Codex CLI docs.
+- **[@Shaan-alpha](https://github.com/Shaan-alpha)** — Extended `ha_restart` known-good error patterns to cover 502/503 responses from reverse proxies.
+- **[@rebelancap](https://github.com/rebelancap)** — Fixed UTC-to-local timezone conversion in `add_timezone_metadata`.
+- **[@saevras](https://github.com/saevras)** — Fixed blueprint import E2E test to use local URL instead of host-to-container networking.
+- **[@jasonjhofmann](https://github.com/jasonjhofmann)** — Recurring calendar events via `rrule` support in `ha_config_set_calendar_event`.
+- **[@vpciii](https://github.com/vpciii)** — Coerce JSON-encoded strings on dict/list tool params.
+- **[@pburtchaell](https://github.com/pburtchaell)** — Financial support via [GitHub Sponsors](https://github.com/sponsors/julienld). Thank you! ☕
 ---
 
 ## 💬 Community