Instruct LLM to not pass a list to the domain (#118451)

home-assistant · May 30, 2024 · 4834283 · 4834283
1 parent 3e0d951
commit 4834283
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/homeassistant/helpers/llm.py b/homeassistant/helpers/llm.py
@@ -250,9 +250,10 @@ def _async_get_api_prompt(
         prompt = [
             (
                 "Call the intent tools to control Home Assistant. "
+                "Do not pass the domain to the intent tools as a list. "
                 "When controlling a device, prefer passing just its name and its domain "
                 "(what comes before the dot in its entity id). "
-                "When controlling an area, prefer passing just area name and a single domain."
+                "When controlling an area, prefer passing just area name and domain."
             )
         ]
         area: ar.AreaEntry | None = None

diff --git a/tests/helpers/test_llm.py b/tests/helpers/test_llm.py
@@ -423,9 +423,10 @@ def create_entity(device: dr.DeviceEntry, write_state=True) -> None:
     )
     first_part_prompt = (
         "Call the intent tools to control Home Assistant. "
+        "Do not pass the domain to the intent tools as a list. "
         "When controlling a device, prefer passing just its name and its domain "
         "(what comes before the dot in its entity id). "
-        "When controlling an area, prefer passing just area name and a single domain."
+        "When controlling an area, prefer passing just area name and domain."
     )
     no_timer_prompt = "This device does not support timers."