esphome icon indicating copy to clipboard operation
esphome copied to clipboard

Add fields/message for remote pipeline trigger

Open synesthesiam opened this issue 1 year ago • 3 comments

What does this implement/fix?

Adds a new message VoiceAssistantTriggerPipeline and some fields to VoiceAssistantRequest to support remote pipeline triggering from Home Assistant.

This will allow HA to trigger a pipeline run on the ESPHome satellite from start_stage to end_stage with specific inputs:

  • When start/end stages are both TTS, announce_text is the text to speak

Also adds:

  • VoiceAssistantSetConfiguration - set active wake words
  • VoiceAssistantConfiguration - lists available wake words

Types of changes

  • [ ] Bugfix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] Other

Related issue or feature (if applicable): fixes

Pull request in esphome-docs with documentation (if applicable): esphome/esphome-docs#<esphome-docs PR number goes here>

Test Environment

  • [ ] ESP32
  • [ ] ESP32 IDF
  • [ ] ESP8266
  • [ ] RP2040
  • [ ] BK72xx
  • [ ] RTL87xx

Example entry for config.yaml:

# Example config.yaml

Checklist:

  • [ ] The code change is tested and works locally.
  • [ ] Tests have been added to verify that the new code works (under tests/ folder).

If user exposed functionality or configuration variables are added/changed:

synesthesiam avatar Aug 26 '24 15:08 synesthesiam

Hey there @jesserockz, mind taking a look at this pull request as it has been labeled with an integration (voice_assistant) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)

probot-esphome[bot] avatar Aug 26 '24 15:08 probot-esphome[bot]

Hey there @ottowinter, mind taking a look at this pull request as it has been labeled with an integration (api) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)

probot-esphome[bot] avatar Aug 26 '24 15:08 probot-esphome[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.95%. Comparing base (4d8b5ed) to head (85cd506). Report is 1215 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #7355      +/-   ##
==========================================
+ Coverage   53.70%   53.95%   +0.24%     
==========================================
  Files          50       50              
  Lines        9408     9668     +260     
  Branches     1654     1707      +53     
==========================================
+ Hits         5053     5216     +163     
- Misses       4056     4128      +72     
- Partials      299      324      +25     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Aug 26 '24 15:08 codecov-commenter

Closing in favor of a simpler design with just announce

synesthesiam avatar Sep 03 '24 14:09 synesthesiam