feat(telegram): enable message_reaction + callback_query in allowedUpdates#2544
Open
smith-vosburg wants to merge 1 commit into
Open
feat(telegram): enable message_reaction + callback_query in allowedUpdates#2544smith-vosburg wants to merge 1 commit into
smith-vosburg wants to merge 1 commit into
Conversation
…dates Without explicitly setting `allowedUpdates`, Telegram excludes update types it considers "advanced" from the polling response by default, including `message_reaction` (added in Bot API 7.0, Feb 2024) and `callback_query`. Once any explicit list is provided, all wanted types must be enumerated. This passes `longPolling.allowedUpdates` with the existing default pair (`message`, `edited_message`) plus `callback_query` and `message_reaction`. The Chat SDK's `processReaction` handler is already wired end-to-end — this commit just asks Telegram to deliver the events. `callback_query` is added pre-emptively so future inline-keyboard adopters don't need another config edit. Behavior for groups that don't consume reactions or callback queries is unchanged — they simply receive update types they ignore (same as today they receive `edited_message` updates that most don't act on). Co-Authored-By: Claude Opus 4.7 <[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
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.
Type of Change
.claude/skills/<name>/, no source changes)Description
Without explicitly setting
allowedUpdates, Telegram excludes update types it considers "advanced" from the polling response by default, includingmessage_reaction(added in Bot API 7.0, Feb 2024) andcallback_query. Once any explicit list is provided, all wanted types must be enumerated.This passes
longPolling.allowedUpdateswith the existing default pair (message,edited_message) pluscallback_queryandmessage_reaction. The Chat SDK'sprocessReactionhandler is already wired end-to-end — this commit just asks Telegram to deliver the events.callback_queryis added pre-emptively so future inline-keyboard adopters don't need another config edit.Behavior for groups that don't consume reactions or callback queries is unchanged — they simply receive update types they ignore (same as today they receive
edited_messageupdates that most don't act on).Diff
Test plan
/add-telegramskill, observe groups created on this branch receive 👍/👎 reaction events via the existingprocessReactionhandleredited_messageandmessagebaseline behavior unchanged🤖 Generated with Claude Code