feat(webhook): route voice through the deliver-target platform adapter - #4
Closed
BenSheridanEdwards wants to merge 1 commit into
Closed
BenSheridanEdwards wants to merge 1 commit into
BenSheridanEdwards wants to merge 1 commit into
Conversation
Webhook has no native audio transport; the base fallback degraded failed voice sends into a 'Couldn't deliver the audio attachment' message on the deliver:telegram path. Resolve the deliver target's adapter (including per-profile adapters) and forward send_voice there, honoring deliver_extra chat/thread ids and falling back to the platform home channel. Hot-patched live on the install 2026-08-07 (pre-jarvis-webhook-tts backup); this PR brings it under the fixes-via-PR-to-main rule. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Owner
Author
|
Superseded by #20 (fleet cut to v0.20.5). This is an earlier, smaller version of the same fix: #6 is carried into #20, rebased onto upstream's rewritten voice-dispatch block. Nothing here is lost. |
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.
What
Formalizes the webhook voice hot-patch found running uncommitted on the live install (applied 2026-08-07,
run.py.pre-jarvis-webhook-tts-20260807backup). Webhook sessions that deliver elsewhere (e.g. deliver:telegram) now forwardsend_voiceto the deliver target's adapter instead of degrading into a "Couldn't deliver the audio attachment" text.Why
Fixes-via-PR-to-main rule: this code is live on the fleet but existed only as a dirty working tree on the install. This PR is byte-for-byte the running change.
Risk
None at runtime; already running since 2026-08-07. Merging just makes main match the install.
🤖 Generated with Claude Code