feat(ios): add Talk voice directive hint toggle#18250
Conversation
|
The formal models extracted constants ( This check is informational (not blocking merges yet). If this change is intentional, follow up by updating the formal models repo or regenerating the extracted artifacts there. |
|
The formal models extracted constants ( This check is informational (not blocking merges yet). If this change is intentional, follow up by updating the formal models repo or regenerating the extracted artifacts there. |
|
The formal models extracted constants ( This check is informational (not blocking merges yet). If this change is intentional, follow up by updating the formal models repo or regenerating the extracted artifacts there. |
* feat(ios): add Talk voice directive hint toggle * docs(changelog): credit voice directive hint slice --------- Co-authored-by: Mariano Belinky <[email protected]>
Intent
Ship Slice 1 only: a low-risk toggle to control whether Talk Mode includes the ElevenLabs voice directive hint in the prompt.
Changes
includeVoiceDirectiveHintparameter toTalkPromptBuilder.build(...)(default:true)talk.voiceDirectiveHint.enabledtrueVoice Directive HintScope Guardrails
Validation
swift test --package-path apps/shared/OpenClawKitGreptile Summary
Added an iOS-only feature toggle (
talk.voiceDirectiveHint.enabled) to control whether the ElevenLabs voice directive hint is included in Talk Mode prompts. The implementation adds a new parameter toTalkPromptBuilder.build(...)with a safe default (true) that preserves existing behavior, wires it to iOS Settings UI, and includes comprehensive unit tests.Key changes:
includeVoiceDirectiveHintparameter toTalkPromptBuilder.build(...)(defaults totrue)TalkModeManagerreadstalk.voiceDirectiveHint.enabledpreference (defaults totrueif missing)Observations:
TalkModeRuntime.swift:397still callsTalkPromptBuilder.build(...)without the new parameter (relies on default). Consider whether macOS should also have this preference or if iOS-only is intentional per the PR scope.Confidence Score: 5/5
true, ensuring existing behavior is maintained. No logical errors, security issues, or breaking changes detected.Last reviewed commit: 93bf0cf
Attribution