ha-mcp-dev 8.4.3.dev2625__tar.gz → 8.4.3.dev2629__tar.gz

{ha_mcp_dev-8.4.3.dev2625/src/ha_mcp_dev.egg-info → ha_mcp_dev-8.4.3.dev2629}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "8.4.3.dev2625"
+version = "8.4.3.dev2629"
 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
@@ -94,6 +94,14 @@ mypy_path = "src"
 # /tmp keeps the cache on Linux tmpfs — avoids the slow Windows FS in WSL2.
 # Windows-native contributors can remove this line or override with MYPY_CACHE_DIR.
 cache_dir = "/tmp/.mypy_cache"
+# Vendored skills submodule: upstream's code, not ours to type-check. This is
+# the exclusion that was actually failing a local run, because `mypy src/`
+# recurses into the submodule when it is initialized while CI's lint job checks
+# out without `submodules: true` and never sees it.
+# A regex, NOT a [[tool.mypy.overrides]] module entry like the ha_mcp._vendor.*
+# one below: `skills-vendor` contains a hyphen, so it has no importable module
+# path for a module pattern to match. Do not "simplify" it into that block.
+exclude = ['^src/ha_mcp/resources/skills-vendor/']
 warn_return_any = true
 warn_unused_configs = true
 disallow_untyped_defs = true
@@ -150,6 +158,14 @@ extend-exclude = [
     # Vendored websockets — upstream's code style, not ours. Kept in sync
     # with PATHS_IGNORE in scripts/codeql_quality_gate.py.
     "src/ha_mcp/_vendor/websockets",
+    # Vendored skills submodule (src/ha_mcp/resources/skills-vendor) — upstream's
+    # code, shipped as package data. This covers the directory-scoped commands,
+    # `ruff check src/ ...` in CI and the ones in CONTRIBUTING.md. It does NOT
+    # affect lefthook, which passes {staged_files} as explicit paths that ruff
+    # lints regardless of extend-exclude (force-exclude is not set), and a
+    # submodule's contents are never staged in the superproject anyway. Kept in
+    # sync with PATHS_IGNORE in scripts/codeql_quality_gate.py.
+    "src/ha_mcp/resources/skills-vendor",
     # HA integration manifests must be strict JSON (no trailing commas);
     # ruff 0.15+ formats .json files as JSONC and inserts trailing commas
     # which makes Python's json.load() reject the file at HA startup.

{ha_mcp_dev-8.4.3.dev2625 → ha_mcp_dev-8.4.3.dev2629}/src/ha_mcp/__main__.py

@@ -616,7 +616,7 @@ async def _run_with_shutdown(server_coro: Coroutine[Any, Any, Any]) -> None:
     shutdown_task = asyncio.create_task(_shutdown_event.wait())
 
     try:
-        done, pending = await asyncio.wait(
+        done, _ = await asyncio.wait(
             [server_task, shutdown_task],
             return_when=asyncio.FIRST_COMPLETED,
         )

{ha_mcp_dev-8.4.3.dev2625 → ha_mcp_dev-8.4.3.dev2629}/src/ha_mcp/http_transport.py

@@ -3,7 +3,10 @@
 from __future__ import annotations
 
 import logging
-from dataclasses import dataclass
+from collections.abc import Iterator
+from contextlib import contextmanager
+from dataclasses import dataclass, field
+from threading import Lock
 from time import monotonic
 from typing import Any
 from uuid import uuid4
@@ -19,6 +22,42 @@ from .config import get_global_settings
 logger = logging.getLogger(__name__)
 
 
+@dataclass
+class _DiagnosticLogLevel:
+    """Shared ownership while HTTP app lifespans overlap during a restart."""
+
+    users: int = 0
+    previous_level: int = logging.NOTSET
+    changed: bool = False
+    lock: Lock = field(default_factory=Lock)
+
+
+@contextmanager
+def _diagnostic_logging() -> Iterator[None]:
+    """Temporarily enable diagnostic records without changing other loggers."""
+    # Logger objects survive embedded-server module purges. Keep ownership on
+    # this logger so a retiring worker cannot reset its replacement's level.
+    state: _DiagnosticLogLevel = vars(logger).setdefault(
+        "_ha_mcp_http_diagnostics_level", _DiagnosticLogLevel()
+    )
+    with state.lock:
+        if state.users == 0:
+            state.previous_level = logger.level
+            state.changed = logger.getEffectiveLevel() > logging.INFO
+            if state.changed:
+                # Use the normal setter: HA's explicit per-logger overrides
+                # remain authoritative, and existing DEBUG levels stay intact.
+                logger.setLevel(logging.INFO)
+        state.users += 1
+    try:
+        yield
+    finally:
+        with state.lock:
+            state.users -= 1
+            if state.users == 0 and state.changed and logger.level == logging.INFO:
+                logger.setLevel(state.previous_level)
+
+
 class HttpTransportFastMCP(FastMCP):
     """Apply instance HTTP experiments at the shared app-construction boundary.
 
@@ -117,6 +156,10 @@ class TransportDiagnostics:
         self.path = path.rstrip("/")
 
     async def __call__(self, scope: Scope, receive: Receive, send: Send) -> None:
+        if scope["type"] == "lifespan":
+            with _diagnostic_logging():
+                await self.app(scope, receive, send)
+            return
         if scope["type"] != "http" or scope["path"].rstrip("/") != self.path:
             await self.app(scope, receive, send)
             return

{ha_mcp_dev-8.4.3.dev2625 → ha_mcp_dev-8.4.3.dev2629}/src/ha_mcp/settings_ui/__init__.py

@@ -55,7 +55,6 @@ from ._persistence import (
 )
 from ._supervisor import (
     _BACKGROUND_RESTART_TASKS,
-    _SUPERVISOR_SELF_RESTART_FLUSH_DELAY_S,
     _schedule_supervisor_self_restart,
     _supervisor_fetch_current_options,
     _supervisor_merge_and_post_options,

{ha_mcp_dev-8.4.3.dev2625 → ha_mcp_dev-8.4.3.dev2629}/src/ha_mcp/settings_ui/locales/en.json

@@ -449,7 +449,7 @@
     "advanced.environment.label": "Environment",
     "advanced.environment.help": "development or production. Affects logging verbosity. Restart required.",
     "advanced.http_transport_diagnostics.label": "HTTP transport diagnostics",
-    "advanced.http_transport_diagnostics.help": "Experimental. Log MCP HTTP byte counts, timing and completion at INFO level, without payloads or credentials. Does not confirm client receipt. HTTP only; restart required.",
+    "advanced.http_transport_diagnostics.help": "Experimental. Automatically enable INFO logging for HTTP diagnostics while the server runs, without payloads or credentials. Other loggers are unchanged; explicit Home Assistant logger overrides and filters still apply. Does not confirm client receipt. HTTP only; restart required.",
     "advanced.http_json_response.label": "JSON responses instead of streaming",
     "advanced.http_json_response.help": "Experimental. Return a single JSON response instead of an SSE stream. Affects all HTTP clients on this server and disables streamed progress for these responses. HTTP only; restart required.",
     "advanced.log_level.label": "Log level",

{ha_mcp_dev-8.4.3.dev2625 → ha_mcp_dev-8.4.3.dev2629}/src/ha_mcp/settings_ui/settings.js

@@ -3981,7 +3981,7 @@ const ADVANCED_FIELD_META = {
   environment:         { label: "Environment",                 help: "'development' or 'production'. Affects logging verbosity. Restart required." },
   log_level:           { label: "Log level",                   help: "DEBUG/INFO/WARNING/ERROR/CRITICAL. Set once at startup; restart required." },
   debug:               { label: "Debug mode",                  help: "Verbose request logging. Logs sensitive data; do not enable in production. Restart required." },
-  http_transport_diagnostics: { label: "HTTP transport diagnostics", help: "Experimental. Log MCP HTTP byte counts, timing and completion at INFO level, without payloads or credentials. Does not confirm client receipt. HTTP only; restart required." },
+  http_transport_diagnostics: { label: "HTTP transport diagnostics", help: "Experimental. Automatically enable INFO logging for HTTP diagnostics while the server runs, without payloads or credentials. Other loggers are unchanged; explicit Home Assistant logger overrides and filters still apply. Does not confirm client receipt. HTTP only; restart required." },
   http_json_response: { label: "JSON responses instead of streaming", help: "Experimental. Return a single JSON response instead of an SSE stream. Affects all HTTP clients on this server and disables streamed progress for these responses. HTTP only; restart required." },
   code_mode_max_duration:    { label: "Code-mode max duration (s)",   help: "Wall-clock budget per sandbox run. Range 1–300. Restart required." },
   code_mode_max_memory:      { label: "Code-mode max memory (bytes)", help: "RSS cap per sandbox run. Range 1 MB–256 MB. Restart required." },

{ha_mcp_dev-8.4.3.dev2625 → ha_mcp_dev-8.4.3.dev2629/src/ha_mcp_dev.egg-info}/PKG-INFO

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