ha-mcp-dev 8.6.0.dev2881__tar.gz → 8.6.0.dev2883__tar.gz

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 8.6.0.dev2881
+Version: 8.6.0.dev2883
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Keywords: mcp,home-assistant,ai,automation,smart-home
 Author: Julien

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "uv_build"
 
 [project]
 name = "ha-mcp-dev"
-version = "8.6.0.dev2881"
+version = "8.6.0.dev2883"
 description = "Home Assistant MCP Server - Complete control of Home Assistant through MCP"
 readme = "README.md"
 requires-python = ">=3.13,<3.15"

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/pyproject.toml.orig

@@ -4,7 +4,7 @@ build-backend = "uv_build"
 
 [project]
 name = "ha-mcp-dev"
-version = "8.6.0.dev2881"
+version = "8.6.0.dev2883"
 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.6.0.dev2883/src/ha_mcp/app_discovery.py

@@ -0,0 +1,90 @@
+"""Offer the app's server to Home Assistant's Model Context Protocol integration (#2307).
+
+Runs once per start from ``server_lifespan``, inside the app only. The
+Supervisor keys the message by (app, service), so announcing on every start
+replaces the URL in place. It is never withdrawn on stop: Core removes the
+discovered entry when the message disappears. Failures never stop the server.
+"""
+
+from __future__ import annotations
+
+import asyncio
+import logging
+from pathlib import Path
+from typing import Any
+
+import httpx
+
+from ._version import is_running_in_addon
+from .client.supervisor_client import make_supervisor_httpx_client
+
+logger = logging.getLogger(__name__)
+
+# First Core release whose Model Context Protocol integration handles app
+# discovery (home-assistant/core#180378). Older cores turn the message into a
+# discovery card that drops the URL and asks for one by hand.
+MCP_DISCOVERY_MIN_CORE = (2026, 10)
+# The app's start.py writes the secret path here before it starts the server,
+# and serves on this port.
+SECRET_PATH_FILE = Path("/data/secret_path.txt")
+APP_PORT = 9583
+
+
+def core_supports_mcp_discovery(version: str) -> bool:
+    """Whether a Core version string (``2026.10.0``, ``2026.10.0b1``) is new enough."""
+    try:
+        year, month = (int(part) for part in version.split(".")[:2])
+    except ValueError:
+        return False
+    return (year, month) >= MCP_DISCOVERY_MIN_CORE
+
+
+async def _supervisor_data(client: httpx.AsyncClient, path: str) -> dict[str, Any]:
+    """GET a Supervisor endpoint and return its ``data`` object."""
+    response = await client.get(path)
+    response.raise_for_status()
+    data = response.json().get("data")
+    return data if isinstance(data, dict) else {}
+
+
+async def announce_mcp_discovery() -> None:
+    """Announce ``http://<app hostname>:9583<secret path>`` to the Supervisor."""
+    if not is_running_in_addon():
+        return
+    try:
+        secret = await asyncio.to_thread(SECRET_PATH_FILE.read_text, encoding="utf-8")
+        secret_path = secret.strip()
+        if not secret_path:
+            logger.warning("%s is empty; MCP discovery skipped.", SECRET_PATH_FILE)
+            return
+        async with make_supervisor_httpx_client(timeout=10.0, verify=True) as client:
+            version = str((await _supervisor_data(client, "/core/info")).get("version"))
+            if not core_supports_mcp_discovery(version):
+                logger.info(
+                    "Home Assistant %s cannot discover MCP apps (needs 2026.10+); "
+                    "restart this app after updating Home Assistant.",
+                    version,
+                )
+                return
+            info = await _supervisor_data(client, "/addons/self/info")
+            hostname = info.get("hostname")
+            if not hostname:
+                logger.warning(
+                    "Supervisor reported no app hostname; MCP discovery skipped."
+                )
+                return
+            # Exactly the route start.py serves: Starlette redirects any path
+            # that differs only by a trailing slash, and Core's client does
+            # not follow redirects.
+            url = f"http://{hostname}:{APP_PORT}{secret_path}"
+            response = await client.post(
+                "/discovery", json={"service": "mcp", "config": {"url": url}}
+            )
+            response.raise_for_status()
+    except (httpx.HTTPError, OSError, ValueError) as err:
+        logger.warning("Could not announce this server to Home Assistant: %r", err)
+        return
+    logger.info(
+        "Announced to Home Assistant: Settings > Devices & services offers to "
+        "add this server to the Model Context Protocol integration."
+    )

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/server.py

@@ -33,6 +33,7 @@ from .server_tool_text import (
     TOOL_DISCOVERY_INSTRUCTIONS,
 )
 from .tools.helpers import raise_tool_error
+from .tools.tool_hints import read_only_hints
 from .transforms import DEFAULT_PINNED_TOOLS
 from .utils.skill_loader import (
     BEST_PRACTICES_SKILL_NAME,
@@ -1201,12 +1202,9 @@ class HomeAssistantSmartMCPServer:
         self.mcp.tool(
             name=SKILL_TOOL_NAME,
             description=tool_description,
-            annotations={
-                "openWorldHint": False,
-                "readOnlyHint": True,
-                "idempotentHint": True,
-                "title": "Get Home Assistant Best Practices Skill Guide",
-            },
+            annotations=read_only_hints(
+                "Get Home Assistant Best Practices Skill Guide", open_world=False
+            ),
             tags={"System"},
         )(ha_get_skill_guide)
         logger.info(

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/server_lifespan.py

@@ -7,6 +7,7 @@ from contextlib import asynccontextmanager, suppress
 from typing import Any
 
 from ._version import is_embedded
+from .app_discovery import announce_mcp_discovery
 from .client.rest_client import HomeAssistantClient
 from .config import OAUTH_MODE_TOKEN, get_global_settings
 from .hacs_auto_refresh import hacs_refresh_lifespan
@@ -37,16 +38,20 @@ async def warn_if_non_admin_token() -> None:
 
 @asynccontextmanager
 async def server_lifespan(server: Any) -> AsyncIterator[dict[str, Any]]:
-    """Run the HACS startup nudge and the admin-token check for the server's lifetime.
+    """Run the HACS nudge, admin-token check and app discovery for the server's lifetime.
 
     Attached as the FastMCP ``lifespan`` so it runs on every launcher, including
     the app's ``start.py``, which calls ``mcp.run()`` directly.
     """
     async with hacs_refresh_lifespan(server) as state:
-        task = asyncio.create_task(warn_if_non_admin_token())
+        tasks = [
+            asyncio.create_task(warn_if_non_admin_token()),
+            asyncio.create_task(announce_mcp_discovery()),
+        ]
         try:
             yield state
         finally:
-            task.cancel()
-            with suppress(asyncio.CancelledError):
-                await task
+            for task in tasks:
+                task.cancel()
+                with suppress(asyncio.CancelledError):
+                    await task

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/backup.py

@@ -56,6 +56,7 @@ from .helpers import (
     raise_tool_error,
     safe_progress,
 )
+from .tool_hints import write_hints
 
 if TYPE_CHECKING:
     from ha_mcp._vendor.fastmcp import FastMCP
@@ -1679,11 +1680,9 @@ survives an agent's own mistakes.
     @mcp.tool(
         description=manage_backup_description,
         tags={"System"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "title": "Manage Backups",
-        },
+        annotations=write_hints(
+            "Manage Backups", destructive=True, idempotent=False, open_world=False
+        ),
     )
     @log_tool_usage
     async def ha_manage_backup(

ha_mcp_dev-8.6.0.dev2883/src/ha_mcp/tools/tool_hints.py

@@ -0,0 +1,71 @@
+"""MCP safety annotations for tool decorators.
+
+Every tool declares all four hints. Home Assistant 2026.10+ copies them into
+its LLM tool metadata and fills an omitted hint with its least-safe default, so
+a read-only tool that leaves one out reaches Core as destructive.
+
+``scripts/extract_tools.py`` and ``tests/src/unit/test_tool_annotations.py``
+load this file by path and evaluate its calls on literal arguments, so it must
+import nothing outside the standard library.
+"""
+
+import ast
+from collections.abc import Callable
+from typing import Any
+
+
+def read_only_hints(title: str, *, open_world: bool) -> dict[str, Any]:
+    """Annotations for a tool that changes nothing."""
+    return {
+        "title": title,
+        "readOnlyHint": True,
+        "destructiveHint": False,
+        "idempotentHint": True,
+        "openWorldHint": open_world,
+    }
+
+
+def write_hints(
+    title: str, *, destructive: bool, idempotent: bool, open_world: bool
+) -> dict[str, Any]:
+    """Annotations for a tool that changes state."""
+    return {
+        "title": title,
+        "readOnlyHint": False,
+        "destructiveHint": destructive,
+        "idempotentHint": idempotent,
+        "openWorldHint": open_world,
+    }
+
+
+_BUILDERS: dict[str, Callable[..., dict[str, Any]]] = {
+    "read_only_hints": read_only_hints,
+    "write_hints": write_hints,
+}
+
+
+def annotations_from_ast(node: ast.expr) -> dict[str, Any]:
+    """What an ``annotations=`` value evaluates to, read from source.
+
+    Handles a dict literal and a call to one of the builders above with literal
+    arguments; anything else yields ``{}``.
+    """
+    if isinstance(node, ast.Dict):
+        found: dict[str, Any] = {}
+        for key, value in zip(node.keys, node.values, strict=True):
+            if (
+                isinstance(key, ast.Constant)
+                and isinstance(key.value, str)
+                and isinstance(value, ast.Constant)
+            ):
+                found[key.value] = value.value
+        return found
+    if isinstance(node, ast.Call) and isinstance(node.func, ast.Name):
+        builder = _BUILDERS.get(node.func.id)
+        if builder is not None:
+            kwargs: dict[str, Any] = {}
+            for kw in node.keywords:
+                if kw.arg is not None:
+                    kwargs[kw.arg] = ast.literal_eval(kw.value)
+            return builder(*(ast.literal_eval(arg) for arg in node.args), **kwargs)
+    return {}

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_addons.py

@@ -60,6 +60,7 @@ from .helpers import (
     raise_tool_error,
     validate_identifier_not_empty,
 )
+from .tool_hints import read_only_hints, write_hints
 
 logger = logging.getLogger(__name__)
 
@@ -3748,12 +3749,7 @@ def register_addon_tools(mcp: Any, client: HomeAssistantClient, **kwargs: Any) -
 
     @mcp.tool(
         tags={"Apps (add-ons)"},
-        annotations={
-            "openWorldHint": True,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "Get Apps (add-ons)",
-        },
+        annotations=read_only_hints("Get Apps (add-ons)", open_world=True),
     )
     @log_tool_usage
     async def ha_get_app(
@@ -3818,13 +3814,9 @@ def register_addon_tools(mcp: Any, client: HomeAssistantClient, **kwargs: Any) -
 
     @mcp.tool(
         tags={"Apps (add-ons)"},
-        annotations={
-            "openWorldHint": True,
-            "destructiveHint": True,
-            "idempotentHint": False,
-            "readOnlyHint": False,
-            "title": "Manage App (add-on)",
-        },
+        annotations=write_hints(
+            "Manage App (add-on)", destructive=True, idempotent=False, open_world=True
+        ),
     )
     @log_tool_usage
     async def ha_manage_app(

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_areas.py

@@ -36,6 +36,7 @@ from .helpers import (
     validate_identifier_not_empty,
 )
 from .response_helpers import project_fields, project_records, result_fields_warning
+from .tool_hints import read_only_hints, write_hints
 
 logger = logging.getLogger(__name__)
 
@@ -134,12 +135,7 @@ class AreaTools:
     @tool(
         name="ha_list_floors_areas",
         tags={"Areas & Floors"},
-        annotations={
-            "openWorldHint": False,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "List Floors and Areas",
-        },
+        annotations=read_only_hints("List Floors and Areas", open_world=False),
     )
     @log_tool_usage
     async def ha_list_floors_areas(
@@ -590,11 +586,12 @@ class AreaTools:
     @tool(
         name="ha_set_area_or_floor",
         tags={"Areas & Floors"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "title": "Create or Update Area or Floor",
-        },
+        annotations=write_hints(
+            "Create or Update Area or Floor",
+            destructive=True,
+            idempotent=False,
+            open_world=False,
+        ),
     )
     @with_auto_backup(
         domain="area_or_floor",
@@ -853,12 +850,9 @@ class AreaTools:
     @tool(
         name="ha_remove_area_or_floor",
         tags={"Areas & Floors"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "idempotentHint": True,
-            "title": "Remove Area or Floor",
-        },
+        annotations=write_hints(
+            "Remove Area or Floor", destructive=True, idempotent=True, open_world=False
+        ),
     )
     @with_auto_backup(
         domain="area_or_floor",

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_assist_pipeline.py

@@ -22,6 +22,7 @@ from .helpers import (
     raise_tool_error,
     register_tool_methods,
 )
+from .tool_hints import write_hints
 from .util_helpers import websocket_error_message
 
 logger = logging.getLogger(__name__)
@@ -526,16 +527,15 @@ class AssistPipelineTools:
     @tool(
         name="ha_manage_pipeline",
         tags={"Assist"},
-        annotations={
-            # action='process' answers with whatever the conversation agent
-            # says, and that agent can be a cloud LLM, so the tool carries
-            # externally-authored content back to the client.
-            "openWorldHint": True,
-            "destructiveHint": True,
-            "idempotentHint": False,
-            "readOnlyHint": False,
-            "title": "Manage Assist Pipeline",
-        },
+        # action='process' answers with whatever the conversation agent
+        # says, and that agent can be a cloud LLM, so the tool carries
+        # externally-authored content back to the client.
+        annotations=write_hints(
+            "Manage Assist Pipeline",
+            destructive=True,
+            idempotent=False,
+            open_world=True,
+        ),
     )
     @log_tool_usage
     async def ha_manage_pipeline(

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_blueprints.py

@@ -33,6 +33,7 @@ from .helpers import (
     raise_tool_error,
     register_tool_methods,
 )
+from .tool_hints import write_hints
 
 logger = logging.getLogger(__name__)
 
@@ -129,15 +130,11 @@ class BlueprintTools:
     @tool(
         name="ha_manage_blueprints",
         tags={"Blueprints"},
-        annotations={
-            "readOnlyHint": False,
-            "destructiveHint": True,
-            "idempotentHint": False,
-            # import fetches arbitrary URLs; list/get return externally
-            # authored blueprint content from an otherwise local read.
-            "openWorldHint": True,
-            "title": "Manage Blueprints",
-        },
+        # import fetches arbitrary URLs; list/get return externally
+        # authored blueprint content from an otherwise local read.
+        annotations=write_hints(
+            "Manage Blueprints", destructive=True, idempotent=False, open_world=True
+        ),
     )
     # ``delete`` and ``save`` are the two actions that can destroy an installed
     # blueprint's contents, so both capture first. The snapshot is keyed on the

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_bug_report.py

@@ -61,6 +61,7 @@ from .helpers import (
     register_tool_methods,
 )
 from .response_helpers import project_fields
+from .tool_hints import read_only_hints
 
 logger = logging.getLogger(__name__)
 
@@ -881,12 +882,7 @@ class BugReportTools:
     @tool(
         name="ha_report_issue",
         tags={"Utilities"},
-        annotations={
-            "openWorldHint": False,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "Report Issue or Feedback",
-        },
+        annotations=read_only_hints("Report Issue or Feedback", open_world=False),
     )
     @log_tool_usage
     async def ha_report_issue(

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_calendar.py

@@ -30,6 +30,7 @@ from .helpers import (
     register_tool_methods,
     validate_identifier_not_empty,
 )
+from .tool_hints import read_only_hints, write_hints
 from .util_helpers import is_connection_error_message
 
 logger = logging.getLogger(__name__)
@@ -160,12 +161,7 @@ class CalendarTools:
     @tool(
         name="ha_config_get_calendar_events",
         tags={"Calendar"},
-        annotations={
-            "openWorldHint": False,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "Get Calendar Events",
-        },
+        annotations=read_only_hints("Get Calendar Events", open_world=False),
     )
     @log_tool_usage
     async def ha_config_get_calendar_events(
@@ -515,11 +511,12 @@ class CalendarTools:
     @tool(
         name="ha_config_set_calendar_event",
         tags={"Calendar"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "title": "Create or Update Calendar Event",
-        },
+        annotations=write_hints(
+            "Create or Update Calendar Event",
+            destructive=True,
+            idempotent=False,
+            open_world=False,
+        ),
     )
     @with_auto_backup(
         domain="calendar_event",
@@ -770,12 +767,9 @@ class CalendarTools:
     @tool(
         name="ha_config_remove_calendar_event",
         tags={"Calendar"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "idempotentHint": True,
-            "title": "Remove Calendar Event",
-        },
+        annotations=write_hints(
+            "Remove Calendar Event", destructive=True, idempotent=True, open_world=False
+        ),
     )
     @with_auto_backup(
         domain="calendar_event",

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_camera.py

@@ -14,6 +14,7 @@ from ha_mcp._vendor.fastmcp.tools import tool
 from ha_mcp._vendor.fastmcp.utilities.types import Image
 
 from .helpers import log_tool_usage, register_tool_methods
+from .tool_hints import read_only_hints
 
 logger = logging.getLogger(__name__)
 
@@ -63,12 +64,7 @@ class CameraTools:
     @tool(
         name="ha_get_camera_image",
         tags={"Camera"},
-        annotations={
-            "openWorldHint": False,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "Get Camera Image",
-        },
+        annotations=read_only_hints("Get Camera Image", open_world=False),
     )
     @log_tool_usage
     async def ha_get_camera_image(

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_categories.py

@@ -25,6 +25,7 @@ from .helpers import (
     register_tool_methods,
     validate_identifier_not_empty,
 )
+from .tool_hints import read_only_hints, write_hints
 
 logger = logging.getLogger(__name__)
 
@@ -38,12 +39,7 @@ class CategoryTools:
     @tool(
         name="ha_config_get_category",
         tags={"Labels & Categories"},
-        annotations={
-            "openWorldHint": False,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "Get Category",
-        },
+        annotations=read_only_hints("Get Category", open_world=False),
     )
     @log_tool_usage
     async def ha_config_get_category(
@@ -165,11 +161,12 @@ class CategoryTools:
     @tool(
         name="ha_config_set_category",
         tags={"Labels & Categories"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "title": "Create or Update Category",
-        },
+        annotations=write_hints(
+            "Create or Update Category",
+            destructive=True,
+            idempotent=False,
+            open_world=False,
+        ),
     )
     @with_auto_backup(
         domain="category",
@@ -311,12 +308,9 @@ class CategoryTools:
     @tool(
         name="ha_config_remove_category",
         tags={"Labels & Categories"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "idempotentHint": True,
-            "title": "Remove Category",
-        },
+        annotations=write_hints(
+            "Remove Category", destructive=True, idempotent=True, open_world=False
+        ),
     )
     @with_auto_backup(
         domain="category",

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_code.py

@@ -42,6 +42,7 @@ from ..client.rest_client import (
 from ..config import get_global_settings
 from ..errors import ErrorCode, create_error_response
 from .helpers import log_tool_usage, raise_tool_error
+from .tool_hints import write_hints
 from .util_helpers import BLOCKED_WS_WRITE_COMMANDS
 
 logger = logging.getLogger(__name__)
@@ -1308,13 +1309,9 @@ def register_code_tools(mcp: Any, client: Any, **kwargs: Any) -> None:
 
     @mcp.tool(
         tags={"System", "beta"},
-        annotations={
-            "openWorldHint": False,
-            "title": "Custom Tool",
-            "destructiveHint": True,
-            "idempotentHint": False,
-            "readOnlyHint": False,
-        },
+        annotations=write_hints(
+            "Custom Tool", destructive=True, idempotent=False, open_world=False
+        ),
     )
     @log_tool_usage
     async def ha_manage_custom_tool(

{ha_mcp_dev-8.6.0.dev2881 → ha_mcp_dev-8.6.0.dev2883}/src/ha_mcp/tools/tools_config_automations.py

@@ -66,6 +66,7 @@ from .helpers import (
     validate_identifier_not_empty,
 )
 from .reference_validator import validate_config_references
+from .tool_hints import read_only_hints, write_hints
 from .ws_waiters import (
     wait_for_automation_entity_by_unique_id,
     wait_for_entity_registered,
@@ -542,12 +543,7 @@ class AutomationConfigTools:
     @tool(
         name="ha_config_get_automation",
         tags={"Automations"},
-        annotations={
-            "openWorldHint": False,
-            "idempotentHint": True,
-            "readOnlyHint": True,
-            "title": "Get Automation Config",
-        },
+        annotations=read_only_hints("Get Automation Config", open_world=False),
     )
     @log_tool_usage
     async def ha_config_get_automation(
@@ -659,11 +655,12 @@ class AutomationConfigTools:
     @tool(
         name="ha_config_set_automation",
         tags={"Automations"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "title": "Create or Update Automation",
-        },
+        annotations=write_hints(
+            "Create or Update Automation",
+            destructive=True,
+            idempotent=False,
+            open_world=False,
+        ),
     )
     @with_auto_backup(
         domain="automation",
@@ -1920,12 +1917,9 @@ class AutomationConfigTools:
     @tool(
         name="ha_config_remove_automation",
         tags={"Automations"},
-        annotations={
-            "openWorldHint": False,
-            "destructiveHint": True,
-            "idempotentHint": True,
-            "title": "Remove Automation",
-        },
+        annotations=write_hints(
+            "Remove Automation", destructive=True, idempotent=True, open_world=False
+        ),
     )
     @with_auto_backup(domain="automation", id_param="identifier")
     @log_tool_usage