fix(telegram): enable voice-note transcription in DMs#61008
Merged
obviyus merged 2 commits intoApr 5, 2026
Conversation
openclaw-barnacle
Bot
added
channel: telegram
Contributor
Greptile SummaryThis PR fixes DM voice-note transcription that was accidentally dropped during the Confidence Score: 5/5Safe to merge — precise two-line logic change with a matching test, no regressions in existing group cases. The fix is minimal and well-targeted: it restores a documented regression, the condition restructuring is logically equivalent for groups and correct for DMs, and the new test exercises the exact broken path. All remaining findings are P2 or lower. No files require special attention. Reviews (1): Last reviewed commit: "fix(telegram): enable voice-note transcr..." | Re-trigger Greptile |
manueltarouca
force-pushed
the
fix/telegram-dm-voice-transcription
branch
from
April 4, 2026 18:54
0147b8d to
59aa0a7
Compare
Contributor
Author
|
CI failures are unrelated to this change -- all stem from The |
The preflight transcription condition only triggered for group chats (isGroup && requireMention), so voice notes sent in direct messages were never transcribed -- they arrived as raw <media:audio> placeholders. This regression was introduced when the Telegram channel was moved from src/telegram/ to extensions/telegram/, losing the fix from c15385f. Widen the condition to fire whenever there is audio and no accompanying text, regardless of chat type. Group-specific guards (requireMention, disableAudioPreflight, senderAllowedForAudioPreflight) still apply only in group contexts.
obviyus
force-pushed
the
fix/telegram-dm-voice-transcription
branch
from
April 5, 2026 04:18
990cc24 to
b83de68
Compare
obviyus
approved these changes
Apr 5, 2026
obviyus
left a comment
obviyus
left a comment
Contributor
There was a problem hiding this comment.
Reviewed latest changes; landing now.
Contributor
|
Landed on main. Thanks @manueltarouca. |
3 tasks
28 tasks
This was referenced Apr 25, 2026
lovewanwan
pushed a commit
to lovewanwan/openclaw
that referenced
this pull request
Apr 28, 2026
…hanks @manueltarouca) * fix(telegram): enable voice-note transcription in DMs The preflight transcription condition only triggered for group chats (isGroup && requireMention), so voice notes sent in direct messages were never transcribed -- they arrived as raw <media:audio> placeholders. This regression was introduced when the Telegram channel was moved from src/telegram/ to extensions/telegram/, losing the fix from eb56ced. Widen the condition to fire whenever there is audio and no accompanying text, regardless of chat type. Group-specific guards (requireMention, disableAudioPreflight, senderAllowedForAudioPreflight) still apply only in group contexts. * fix: restore Telegram DM voice-note transcription (openclaw#61008) (thanks @manueltarouca) --------- Co-authored-by: Ayaan Zaidi <[email protected]>
ogt-redknie
pushed a commit
to ogt-redknie/OPENX
that referenced
this pull request
May 2, 2026
…hanks @manueltarouca) * fix(telegram): enable voice-note transcription in DMs The preflight transcription condition only triggered for group chats (isGroup && requireMention), so voice notes sent in direct messages were never transcribed -- they arrived as raw <media:audio> placeholders. This regression was introduced when the Telegram channel was moved from src/telegram/ to extensions/telegram/, losing the fix from a9137f9. Widen the condition to fire whenever there is audio and no accompanying text, regardless of chat type. Group-specific guards (requireMention, disableAudioPreflight, senderAllowedForAudioPreflight) still apply only in group contexts. * fix: restore Telegram DM voice-note transcription (openclaw#61008) (thanks @manueltarouca) --------- Co-authored-by: Ayaan Zaidi <[email protected]>
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
May 9, 2026
…hanks @manueltarouca) * fix(telegram): enable voice-note transcription in DMs The preflight transcription condition only triggered for group chats (isGroup && requireMention), so voice notes sent in direct messages were never transcribed -- they arrived as raw <media:audio> placeholders. This regression was introduced when the Telegram channel was moved from src/telegram/ to extensions/telegram/, losing the fix from cbe04c0. Widen the condition to fire whenever there is audio and no accompanying text, regardless of chat type. Group-specific guards (requireMention, disableAudioPreflight, senderAllowedForAudioPreflight) still apply only in group contexts. * fix: restore Telegram DM voice-note transcription (openclaw#61008) (thanks @manueltarouca) --------- Co-authored-by: Ayaan Zaidi <[email protected]>
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
May 24, 2026
…hanks @manueltarouca) * fix(telegram): enable voice-note transcription in DMs The preflight transcription condition only triggered for group chats (isGroup && requireMention), so voice notes sent in direct messages were never transcribed -- they arrived as raw <media:audio> placeholders. This regression was introduced when the Telegram channel was moved from src/telegram/ to extensions/telegram/, losing the fix from cbe04c0. Widen the condition to fire whenever there is audio and no accompanying text, regardless of chat type. Group-specific guards (requireMention, disableAudioPreflight, senderAllowedForAudioPreflight) still apply only in group contexts. * fix: restore Telegram DM voice-note transcription (openclaw#61008) (thanks @manueltarouca) --------- Co-authored-by: Ayaan Zaidi <[email protected]>
Nachx639
pushed a commit
to Nachx639/clawdbot
that referenced
this pull request
Jun 17, 2026
…hanks @manueltarouca) * fix(telegram): enable voice-note transcription in DMs The preflight transcription condition only triggered for group chats (isGroup && requireMention), so voice notes sent in direct messages were never transcribed -- they arrived as raw <media:audio> placeholders. This regression was introduced when the Telegram channel was moved from src/telegram/ to extensions/telegram/, losing the fix from 38d8c93. Widen the condition to fire whenever there is audio and no accompanying text, regardless of chat type. Group-specific guards (requireMention, disableAudioPreflight, senderAllowedForAudioPreflight) still apply only in group contexts. * fix: restore Telegram DM voice-note transcription (openclaw#61008) (thanks @manueltarouca) --------- Co-authored-by: Ayaan Zaidi <[email protected]>
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
3 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
src/telegram/toextensions/telegram/migrationisGroup && requireMention, so voice notes in direct messages were never transcribed and arrived as raw<media:audio>placeholdersrequireMention,disableAudioPreflight,senderAllowedForAudioPreflight)Original fix: c15385f, regression introduced during extension refactor.
Test plan
pnpm tsgopasses<media:audio>