Skip to content

fix(telegram): cancel delayed deliveries on disconnect - #17082

Closed
CRWuTJ wants to merge 5 commits into
NousResearch:mainfrom
CRWuTJ:feat/company-os-telegram-stability
Closed

fix(telegram): cancel delayed deliveries on disconnect#17082
CRWuTJ wants to merge 5 commits into
NousResearch:mainfrom
CRWuTJ:feat/company-os-telegram-stability

Conversation

@CRWuTJ

@CRWuTJ CRWuTJ commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • cancel pending Telegram delayed delivery task families during disconnect
  • clear text/photo/media batch queues and polling error task state after cancellation
  • add regression coverage for stale text batches and all pending delivery maps

Tests

  • python -m pytest tests/gateway/test_telegram_text_batching.py -q
  • python -m pytest tests/gateway/test_telegram_text_batching.py tests/gateway/test_telegram_network_reconnect.py tests/gateway/test_telegram_conflict.py -q

@alt-glitch alt-glitch added type/bug Something isn't working P1 High — major feature broken, no workaround platform/telegram Telegram bot adapter comp/gateway Gateway runner, session dispatch, delivery labels Apr 28, 2026
@teknium1 teknium1 added sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jun 29, 2026
@teknium1

teknium1 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Merged via #55971. Your fix was salvaged onto current main (the Telegram adapter moved to plugins/platforms/telegram/adapter.py since you opened this) with your authorship preserved in git log. Thanks for catching the disconnect-race delivery bug!

@teknium1 teknium1 closed this Jul 1, 2026
dtera pushed a commit to dtera/hermes-agent that referenced this pull request Jul 1, 2026
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
Jasper6439 pushed a commit to Jasper6439/hermes-agent that referenced this pull request Jul 5, 2026
habarmc1223-sudo pushed a commit to habarmc1223-sudo/hermes-agent-fluxmem that referenced this pull request Jul 8, 2026
santhreal pushed a commit to santhreal/hermes-agent that referenced this pull request Jul 13, 2026
Gravezzz pushed a commit to Gravezzz/hermes-agent that referenced this pull request Jul 21, 2026
leewenjie pushed a commit to leewenjie/hermes-agent that referenced this pull request Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P1 High — major feature broken, no workaround platform/telegram Telegram bot adapter sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants