ha-mcp-dev 7.14.2.dev2146__tar.gz → 7.14.2.dev2152__tar.gz

{ha_mcp_dev-7.14.2.dev2146/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.14.2.dev2152}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.14.2.dev2146"
+version = "7.14.2.dev2152"
 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
@@ -173,6 +173,9 @@ testpaths = ["tests"]
 python_files = ["test_*.py", "*_test.py"]
 python_classes = ["Test*"]
 python_functions = ["test_*"]
+# See tests/pytest.ini: an empty parametrize list is a guard that stopped
+# running, and the default mark hides it as a skip the summary ignores.
+empty_parameter_set_mark = "fail_at_collect"
 addopts = [
     "--strict-markers",
     "--strict-config",

{ha_mcp_dev-7.14.2.dev2146 → ha_mcp_dev-7.14.2.dev2152}/src/ha_mcp/server.py

@@ -422,10 +422,10 @@ class HomeAssistantSmartMCPServer(EnhancedToolsMixin):
                 "again \u2014 call it directly.\n\n"
                 f"A few default tools are listed directly "
                 f"({', '.join(DEFAULT_PINNED_TOOLS)}) — these are the "
-                f"starting pins, but users can unpin any of them via the "
-                f"Tools tab in the settings UI, so the actual visible set "
-                f"may be a subset of this list. Everything else must be "
-                f"discovered via search.\n\n"
+                f"starting pins, and users can unpin the non-mandatory "
+                f"ones via the Tools tab in the settings UI, so the "
+                f"actual visible set may be a subset of this list. "
+                f"Everything else must be discovered via search.\n\n"
                 "DO NOT assume a capability is unavailable because you "
                 "don't see a direct tool for it. ALWAYS search first."
             )

{ha_mcp_dev-7.14.2.dev2146 → ha_mcp_dev-7.14.2.dev2152}/src/ha_mcp/settings_ui/_tools_meta.py

@@ -15,6 +15,7 @@ families and ``__init__`` can depend on it without cycles.
 from __future__ import annotations
 
 import logging
+from collections.abc import Iterable
 from typing import TYPE_CHECKING, Any, NamedTuple, NotRequired, TypedDict
 
 from ..transforms import categorize_capability
@@ -50,10 +51,37 @@ class ToolStub(TypedDict):
 
 _VALID_STATES = frozenset({"enabled", "disabled", "pinned"})
 
+# Groups not considered "primary" when choosing a tool's canonical group —
+# these are cross-cutting tags (e.g. Z-Wave, Zigbee) that should not
+# override the tool's real domain group.
+SECONDARY_TAGS = frozenset({"Z-Wave", "Zigbee"})
+
+
+def primary_tag(tags: Iterable[str]) -> str:
+    """Return the group heading the settings UI files a tool under.
+
+    The first tag that is not cross-cutting wins, in sorted order; a tool
+    tagged only with cross-cutting tags falls back to its first tag, and an
+    untagged tool to ``Other``. Exported so the locale check in
+    tests/src/unit/test_locale_parity.py derives the set of reachable group
+    names from this rule itself — a copy of the rule drifts even when the
+    set it reads cannot.
+    """
+    ordered = sorted(tags)
+    primary = [tag for tag in ordered if tag not in SECONDARY_TAGS]
+    if primary:
+        return primary[0]
+    return ordered[0] if ordered else "Other"
+
+
 # Tools that are always enabled regardless of saved config — the server
-# strips them out of any disable list before applying. Four of these
-# overlap with DEFAULT_PINNED_TOOLS in transforms/categorized_search.py
-# (ha_search, ha_get_overview, ha_report_issue, ha_manage_backup);
+# strips them out of any disable list before applying. The settings UI
+# additionally locks the pin toggle for mandatory tools, so the four
+# that overlap with DEFAULT_PINNED_TOOLS in
+# transforms/categorized_search.py
+# (ha_search, ha_get_overview, ha_report_issue, ha_manage_backup)
+# cannot be unpinned either — always pinned, not just always enabled
+# (#2058);
 # ha_get_state is mandatory but not pinned-by-default because it is
 # reachable via the ha_call_read_tool proxy when tool search is on.
 # ha_get_skill_guide is conditionally mandatory via BPS_MANDATORY_TOOLS
@@ -239,16 +267,11 @@ async def _get_tool_metadata(
     list as of v3.2.0.
     """
     tools: list[dict[str, Any]] = []
-    # Groups not considered "primary" when choosing a tool's canonical group —
-    # these are cross-cutting tags (e.g. Z-Wave, Zigbee) that should not
-    # override the tool's real domain group.
-    secondary_tags = {"Z-Wave", "Zigbee"}
 
     registered = await server.mcp.local_provider._list_tools()
     for tool in registered:
         tags = sorted(tool.tags) if tool.tags else []
-        primary_tags = [t for t in tags if t not in secondary_tags]
-        primary = primary_tags[0] if primary_tags else (tags[0] if tags else "Other")
+        primary = primary_tag(tags)
         read_only = bool(
             tool.annotations and getattr(tool.annotations, "readOnlyHint", None)
         )