ha-mcp-dev 8.1.1.dev2286__tar.gz → 8.1.1.dev2289__tar.gz

{ha_mcp_dev-8.1.1.dev2286/src/ha_mcp_dev.egg-info → ha_mcp_dev-8.1.1.dev2289}/PKG-INFO

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

{ha_mcp_dev-8.1.1.dev2286 → ha_mcp_dev-8.1.1.dev2289}/pyproject.toml

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

@@ -666,13 +666,6 @@ async def _run_with_shutdown(server_coro: Coroutine[Any, Any, Any]) -> None:
     server_task = asyncio.create_task(server_coro)
     shutdown_task = asyncio.create_task(_shutdown_event.wait())
 
-    # Fire-and-forget: ask HACS to surface a paired component update after a
-    # server update (advisory; see hacs_auto_refresh). Not in the wait set —
-    # its completion must not stop the server.
-    from ha_mcp.hacs_auto_refresh import maybe_refresh_hacs_after_update
-
-    hacs_refresh_task = asyncio.create_task(maybe_refresh_hacs_after_update())
-
     try:
         done, pending = await asyncio.wait(
             [server_task, shutdown_task],
@@ -732,7 +725,7 @@ async def _run_with_shutdown(server_coro: Coroutine[Any, Any, Any]) -> None:
             logger.warning("Resource cleanup timed out")
 
         try:
-            await _cancel_tasks(server_task, shutdown_task, hacs_refresh_task)
+            await _cancel_tasks(server_task, shutdown_task)
         except Exception as e:
             # Teardown must never mask the exception being propagated from the
             # try block (Python drops the original if finally raises).

{ha_mcp_dev-8.1.1.dev2286 → ha_mcp_dev-8.1.1.dev2289}/src/ha_mcp/hacs_auto_refresh.py

@@ -25,6 +25,8 @@ import asyncio
 import hashlib
 import json
 import logging
+from collections.abc import AsyncIterator
+from contextlib import asynccontextmanager, suppress
 from pathlib import Path
 from typing import Any
 
@@ -222,8 +224,28 @@ async def maybe_refresh_hacs_after_update() -> None:
         info = await asyncio.to_thread(get_update_info)
         marker = await asyncio.to_thread(_read_marker, ha_url)
         if not _nudge_due(current, info, marker):
+            # DEBUG, not INFO: the not-due return is the per-conversation
+            # stdio hot path and must stay quiet at default levels. The line
+            # exists so a DEBUG-level launcher (the HAOS add-on lane) can
+            # prove scheduling even when a completed earlier pass makes every
+            # later boot legitimately not due.
+            logger.debug(
+                "HACS auto-refresh: pass not due (marker current for server %s)",
+                current,
+            )
             return
 
+        # The one unconditional line a due pass emits BEFORE any WebSocket
+        # work: it proves the launcher scheduled the nudge even where HACS
+        # is absent and the pass ends silently — the observable the HAOS
+        # add-on lane greps for, and the line whose absence exposed the
+        # launcher gap this module's lifespan wiring closed.
+        logger.info(
+            "HACS auto-refresh: startup pass due (server %s); "
+            "asking HACS for repository state",
+            current,
+        )
+
         result = await _refresh_with_retries()
         if result is None:
             logger.debug(
@@ -250,3 +272,26 @@ async def maybe_refresh_hacs_after_update() -> None:
         raise
     except Exception:
         logger.debug("HACS auto-refresh nudge skipped", exc_info=True)
+
+
+@asynccontextmanager
+async def hacs_refresh_lifespan(_server: Any) -> AsyncIterator[dict[str, Any]]:
+    """Schedule the startup nudge for the lifetime of any server run.
+
+    Attached as the FastMCP ``lifespan`` so it runs on EVERY launcher —
+    stdio, the HTTP CLI entry points, and the add-on's ``start.py``, which
+    calls ``mcp.run()`` directly and never passes through ``__main__``'s
+    ``_run_with_shutdown`` (the wiring this replaces; the add-on gap was
+    found live, not by CI, because the e2e suites launch via the CLI
+    entry points).
+    """
+    task = asyncio.create_task(maybe_refresh_hacs_after_update())
+    try:
+        yield {}
+    finally:
+        # The nudge may be mid-retry-sleep; a cancelled task must not
+        # stall shutdown. maybe_refresh_hacs_after_update re-raises
+        # CancelledError by design, so this await returns promptly.
+        task.cancel()
+        with suppress(asyncio.CancelledError):
+            await task

{ha_mcp_dev-8.1.1.dev2286 → ha_mcp_dev-8.1.1.dev2289}/src/ha_mcp/server.py

@@ -20,6 +20,7 @@ from pydantic import Field
 
 from .config import _PACKAGE_VERSION, get_global_settings
 from .errors import ErrorCode, create_error_response
+from .hacs_auto_refresh import hacs_refresh_lifespan
 from .tools.helpers import raise_tool_error
 from .transforms import DEFAULT_PINNED_TOOLS
 
@@ -142,6 +143,7 @@ class HomeAssistantSmartMCPServer:
             version=server_version,
             icons=SERVER_ICONS,
             instructions=instructions,
+            lifespan=hacs_refresh_lifespan,
         )
 
         # Register all tools and expert prompts