Skip to content

fix(gateway): prevent Telegram photo burst interrupts#1410

Closed
jplew wants to merge 3 commits into
NousResearch:mainfrom
jplew:fix/telegram-photo-interrupt
Closed

fix(gateway): prevent Telegram photo burst interrupts#1410
jplew wants to merge 3 commits into
NousResearch:mainfrom
jplew:fix/telegram-photo-interrupt

Conversation

@jplew

@jplew jplew commented Mar 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • batch Telegram photo bursts/albums so multi-image sends are processed as one logical turn
  • avoid treating PHOTO follow-ups as priority interrupts while the model is still responding
  • add focused regression coverage for Telegram photo batching behavior

Fixes #1409.

User-visible bug

This addresses the Telegram interruption failure:
Operation interrupted: waiting for model response (0.5s elapsed).

Validation

  • python3 -m pytest tests/gateway/test_telegram_documents.py -q
  • reproduced and manually validated the Telegram image flow locally before/after the fix

Scope

This PR is intentionally scoped to the Telegram photo interrupt bug only. Honcho continuity work remains separate.

Comment thread gateway/run.py Outdated
Comment thread gateway/run.py Outdated
@teknium1

Copy link
Copy Markdown
Contributor

Merged via PR #1422. The core fix was cherry-picked onto current main with authorship preserved, then strengthened with extra regression coverage for the adapter-session interrupt path, non-album photo burst batching, and the gateway priority-interrupt path for photo events.

@teknium1 teknium1 closed this Mar 15, 2026
@jplew

jplew commented Mar 15, 2026

Copy link
Copy Markdown
Contributor Author

Merged via PR #1422. The core fix was cherry-picked onto current main with authorship preserved, then strengthened with extra regression coverage for the adapter-session interrupt path, non-album photo burst batching, and the gateway priority-interrupt path for photo events.

There was a problem with your merged PR, it cherry picked my first commit but omitted the last two. I fixed it up here, please merge this, it removes the extra cruft: #1428

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.

Telegram photo bursts can interrupt active runs with 'Operation interrupted: waiting for model response'

2 participants