Skip to content

fix: keep telegram block-stream replies across assistant messages#16633

Closed
harshang03 wants to merge 1 commit intoopenclaw:mainfrom
harshang03:fix/issue-16604-telegram-block-stream-break
Closed

fix: keep telegram block-stream replies across assistant messages#16633
harshang03 wants to merge 1 commit intoopenclaw:mainfrom
harshang03:fix/issue-16604-telegram-block-stream-break

Conversation

@harshang03
Copy link
Copy Markdown
Contributor

Describe your changes

  • Add a Telegram-scoped block reply break override so blockStreaming: true uses message_end boundaries.
  • Propagate the break override through reply directives to embedded runs via GetReplyOptions.blockReplyBreak.
  • Keep existing global defaults intact while ensuring Telegram can emit intermediate assistant messages on providers that do not emit text_end events.
  • Add regression coverage for helper resolution, Telegram dispatch reply options, and end-to-end propagation into embedded run params.

Screenshot or video (only for visual changes)

  • N/A

GitHub Issue Link (if applicable)

Testing Plan

  • Explanation of why no additional tests are needed: Added focused automated tests covering option resolution and propagation paths.
  • Unit Tests (JS and/or Python): pnpm test src/telegram/bot/helpers.test.ts src/telegram/bot-message-dispatch.test.ts src/auto-reply/reply.block-streaming.test.ts
  • E2E Tests: Included via the targeted block-streaming test suite above.
  • Any manual testing needed?: Optional: configure Telegram with streamMode: "off" and blockStreaming: true, then verify multi-message turns deliver each assistant message sequentially.

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

Force Telegram block streaming to emit on message_end boundaries when blockStreaming is enabled, so intermediate assistant replies are not dropped on providers that do not emit text_end events.
@obviyus
Copy link
Copy Markdown
Contributor

obviyus commented Feb 22, 2026

Fix is already merged in PR #22613.

@obviyus obviyus closed this Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: telegram Channel integration: telegram size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants