fix(tts): add matrix to VOICE_BUBBLE_CHANNELS#37080
fix(tts): add matrix to VOICE_BUBBLE_CHANNELS#37080gumadeiras merged 4 commits intoopenclaw:mainfrom
Conversation
Greptile SummaryThis PR adds
Confidence Score: 5/5
Last reviewed commit: c29d57a |
|
Hi @gumadeiras — just checking in on this one. It's a one-line addition (adding Happy to make any adjustments if needed. Thanks! |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f52cd3db6a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
gumadeiras
force-pushed
the
fix/matrix-voice-bubble-tts
branch
from
930f72d to
28ef456
Compare
In `src/tts/tts.ts`, Matrix was missing from the `VOICE_BUBBLE_CHANNELS` set, causing TTS audio replies to be sent as generic file attachments instead of native MSC3245 voice message bubbles. Matrix fully supports voice bubbles via the send pipeline (`extensions/matrix/src/matrix/send.ts`), but the TTS pipeline never flagged the output as voice-compatible. Adding `"matrix"` to the set enables: - Opus format output (matching Telegram/Feishu/WhatsApp) - voiceCompatible: true flag propagation - MSC3245 voice bubble encoding with waveform and duration metadata Fixes openclaw#37061 Co-Authored-By: Claude Haiku 4.5 <[email protected]>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 28ef456a69
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
gumadeiras
force-pushed
the
fix/matrix-voice-bubble-tts
branch
from
28ef456 to
89ca035
Compare
|
Merged via squash.
Thanks @Matthew19990919! |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 89ca0355d4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
* main: (516 commits) fix: use content hash for memory flush dedup instead of compactionCount (openclaw#30115) (openclaw#34222) fix(tts): add matrix to VOICE_BUBBLE_CHANNELS (openclaw#37080) feat(memory): pluggable system prompt section for memory plugins (openclaw#40126) fix: detect nvm services from installed command (openclaw#51146) fix: handle Linux nvm CA env before startup (openclaw#51146) (thanks @GodsBoy) refactor: route Telegram runtime through plugin sdk (openclaw#51772) refactor: route iMessage runtime through plugin sdk (openclaw#51770) refactor: route Slack runtime through plugin sdk (openclaw#51766) refactor(doctor): extract provider and shared config helpers (openclaw#51753) Fix Discord `/codex_resume` picker expiration (openclaw#51260) fix(ci): remove duplicate embedding default export fix(ci): restore embedding defaults and plugin boundaries fix: compaction safeguard summary budget (openclaw#27727) web UI: fix context notice using accumulated inputTokens instead of prompt snapshot (openclaw#51721) fix(status): skip cold-start status probes refactor(doctor): extract telegram provider warnings (openclaw#51704) fix(telegram): default fresh setups to mention-gated groups docs(changelog): note telegram doctor first-run guidance fix(doctor): add telegram first-run guidance fix(doctor): suppress telegram fresh-install group warning ...
Merged via squash. Prepared head SHA: 89ca035 Co-authored-by: Matthew19990919 <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: 89ca035 Co-authored-by: Matthew19990919 <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: 89ca035 Co-authored-by: Matthew19990919 <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: 89ca035 Co-authored-by: Matthew19990919 <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
Merged via squash. Prepared head SHA: 89ca035 Co-authored-by: Matthew19990919 <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras
2 participants
Summary
Impact
Matrix users configured with
messages.tts.auto: "inbound"will now receive TTS replies as native voice message bubbles with:This improves user experience for ~1500+ active Matrix users in the open-source community.
Root Cause
Matrix channel was missing from VOICE_BUBBLE_CHANNELS, causing resolveOutputFormat() to skip voice-compatible settings. Although extensions/matrix/src/matrix/send.ts fully supports audioAsVoice and MSC3245 metadata, the TTS pipeline never set the flag.
Change
Added "matrix" to VOICE_BUBBLE_CHANNELS set:
Test plan
Fixes #37061