Skip to content

fix(gateway): use OGG for Telegram auto-TTS voice replies#41644

Merged
teknium1 merged 2 commits into
mainfrom
hermes/hermes-d15a8774
Jun 8, 2026
Merged

fix(gateway): use OGG for Telegram auto-TTS voice replies#41644
teknium1 merged 2 commits into
mainfrom
hermes/hermes-d15a8774

Conversation

@teknium1

@teknium1 teknium1 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Telegram auto-TTS voice replies now send as native voice bubbles instead of plain file attachments.

Telegram's send_voice adapter path only renders a native voice bubble for OGG/Opus; .mp3 silently falls through to send_audio (a file attachment). The gateway auto-TTS path in _send_voice_reply() hardcoded a .mp3 output path, bypassing Telegram-aware format selection.

Changes

  • gateway/run.py: request .ogg output when the platform is Telegram, keep .mp3 for all other platforms.
  • tests/gateway/test_auto_voice_reply_format.py: new regression tests (Telegram → ogg, Slack → mp3).
  • scripts/release.py: add gilad@smiti.ai → giladbau to AUTHOR_MAP (CI author gate).

Validation

Before After
Telegram auto-TTS .mp3send_audio (file attachment) .oggsend_voice (voice bubble)
Non-Telegram auto-TTS .mp3 .mp3 (unchanged)

tests/gateway/test_auto_voice_reply_format.py — 2 passed.

Salvaged from #20182 by @giladbau (Gilad Bauman); authorship preserved via cherry-pick.

Infographic

ogg-for-telegram-voice

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

🔎 Lint report: hermes/hermes-d15a8774 vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 10033 on HEAD, 10032 on base (🆕 +1)

🆕 New issues (1):

Rule Count
unresolved-import 1
First entries
tests/gateway/test_auto_voice_reply_format.py:7: [unresolved-import] unresolved-import: Cannot resolve imported module `pytest`

✅ Fixed issues: none

Unchanged: 5198 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.

@teknium1 teknium1 merged commit 9c5d1af into main Jun 8, 2026
23 checks passed
@teknium1 teknium1 deleted the hermes/hermes-d15a8774 branch June 8, 2026 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant