Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sentence Trigger only works without automation conditions #132072

Closed
andreasbrett opened this issue Dec 2, 2024 · 1 comment
Closed

Sentence Trigger only works without automation conditions #132072

andreasbrett opened this issue Dec 2, 2024 · 1 comment

Comments

@andreasbrett
Copy link
Contributor

The problem

When a sentence triggered automation that has conditions is triggered but does not fire because of the conditions, there is neither LLM fallback (new 2024.12 feature) nor is the sentence processed by internal intents. Instead an empty Assist response is sent, which then results in it being overwritten with "Done" (see code).

What version of Home Assistant Core has the issue?

core-2024.12.0b2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

conversation

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

triggers:
  - trigger: conversation
    command:
      - "{sentence}"
conditions:
  - condition: and
    conditions:
      - condition: template
        value_template: "{{ floor_name(trigger.device_id) is not none }}"
      - condition: or
        conditions:
          - condition: state
            entity_id: input_boolean.guest_mode
            state: "on"
          - condition: state
            entity_id: alarm_control_panel.house
            state: armed_vacation
          - condition: state
            entity_id: alarm_control_panel.house
            state: armed_away
          - condition: state
            entity_id: alarm_control_panel.house
            state: triggered
    enabled: true
actions:
  - set_conversation_response: I'm not allowed to speak to you
mode: single

Anything in the logs that might be useful for us?

No response

Additional information

Let me explain why I want to use a sentence trigger with conditions:
I want to "disable" Assist for all inhouse satellites when we are away and the sitter is here. I still want to be able to use Assist on my mobile phone during these times though. Based on an approach by @jlpouffier I created a sentence trigger for "{sentence}" matching everything and respond with "I'm not allowed to speak to you". To only let this affect requests from the inhouse satellites, I added conditions regarding the triggering device's area and guest/alarm mode.

@andreasbrett andreasbrett changed the title Sentence Trigger only work without automation conditions Sentence Trigger only works without automation conditions Dec 2, 2024
@andreasbrett
Copy link
Contributor Author

bot did not add the integration:conversation label, so I re-added the issue here

@andreasbrett andreasbrett closed this as not planned Won't fix, can't repro, duplicate, stale Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant