feat: enable continue_conversation for Voice PE follow-up dialog#11
Open
L0rz wants to merge 1 commit intotechartdev:mainfrom
Open
feat: enable continue_conversation for Voice PE follow-up dialog#11L0rz wants to merge 1 commit intotechartdev:mainfrom
L0rz wants to merge 1 commit intotechartdev:mainfrom
Conversation
When the assistant's response ends with a question mark or contains common follow-up patterns (EN/DE), set continue_conversation=True on the ConversationResult. This tells Voice PE and other HA voice satellites to automatically re-listen after the response finishes playing, enabling natural back-and-forth dialog without requiring the wake word between turns. Fixes techartdev#7
DarrenBenson
added a commit
to DarrenBenson/OpenClawHomeAssistantIntegration
that referenced
this pull request
Mar 29, 2026
Tier 1 improvements from code review: - Extract _extract_text_recursive and _normalize_optional_text to shared utils.py module (was duplicated in __init__.py and conversation.py) - Map OpenClawConnectionError/AuthError to FAILED_TO_HANDLE instead of UNKNOWN, helping HA's Assist pipeline make smarter fallback decisions - Fix api.py session fallback to log warnings when HA-managed session is unexpectedly closed instead of silently creating orphan sessions - Merge PR techartdev#11 continue_conversation for Voice PE follow-up dialog - Fix regex syntax warning in _should_continue Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
When the assistant's response ends with a question or contains common follow-up patterns, set
continue_conversation=Trueon theConversationResult. This tells Voice PE and other HA voice satellites to automatically re-listen after the response finishes playing, enabling natural back-and-forth dialog without requiring the wake word between each turn.Changes
_should_continue()static method with heuristic detection:continue_conversationtoConversationResultHow it works
Voice PE firmware 25.3.0+ supports continued conversations when the conversation agent signals it. Without this flag, users must say the wake word again after every response — even when the assistant explicitly asks a question.
With this change:
Fixes #7