fix(telegram): handle channel post updates#25327
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds Telegram regression coverage and adapter changes to correctly process channel broadcast updates (Update.channel_post) by using effective_message and preserving a usable identity when from_user is missing.
Changes:
- Added regression tests for Telegram
channel_posthandling and channel identity mapping. - Updated Telegram handlers to consume
effective_message(so channel posts are not dropped). - Normalized Telegram chat type parsing and mapped channel posts without
from_userto use channel identity.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| tests/gateway/test_telegram_channel_posts.py | New regression tests for channel posts and identity behavior. |
| gateway/platforms/telegram.py | Use effective_message in handlers; normalize chat type; set channel identity fallback when from_user is absent. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thanks for flagging. I looked at #9500 and agree they overlap on #3634, but this PR is intentionally a bit broader:
I also checked current python-telegram-bot filter behavior locally: |
Summary
Test Plan