fix(discord): react 🎤 on voice messages when STT is disabled#252
Merged
thepagent merged 1 commit intoopenabdev:mainfrom Apr 15, 2026
Merged
Conversation
When STT is not configured, voice messages were silently dropped — no reaction, no reply. Users thought the bot was broken. Changes: - Track audio_skipped flag during attachment processing - Upgrade debug!() → warn!() for operator log visibility - React with 🎤 emoji so users know their voice message was noticed - Early return for voice-only messages to avoid sending empty prompts to the downstream agent For mixed messages (text + voice), the text is still processed normally and the 🎤 reaction signals that the audio portion was ignored.
shaun-agent
force-pushed
the
fix/voice-msg-feedback-when-stt-disabled
branch
from
ab03835 to
d4c934b
Compare
github-actions
bot
added
the
closing-soon
|
All PRs must reference a prior Discord discussion to ensure community alignment before implementation. Please edit the PR description to include a link like: This PR will be automatically closed in 3 days if the link is not added. |
thepagent
approved these changes
Apr 15, 2026
dogzzdogzz
added a commit
to dogzzdogzz/openab
that referenced
this pull request
Apr 15, 2026
Resolve discord.rs conflict — take our refactored version and port upstream's 🎤 reaction for voice messages when STT is disabled (openabdev#252). 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
4 participants
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 STT is not configured, voice messages were silently dropped — no reaction, no reply. Users thought the bot was broken.
Closes #251
Changes
src/discord.rswarnimport, trackaudio_skipped, react 🎤, early returnWhat it does
audio_skippedbool set when an audio attachment is encountered with STT disableddebug!()→warn!()so operators see it in production logsprompt.is_empty()and only the sender_context block exists incontent_blocks, bail out instead of sending an empty prompt to the agentBehavior matrix
Related