Conversation
Greptile SummaryThis PR fixes a duplicate-message flash in Telegram DM preview streaming by switching the answer preview lane from native draft transport ( Key changes:
Confidence Score: 5/5
Last reviewed commit: b65aa78 |
|
Root cause note: This PR exists because the remaining Telegram DM duplicate is not a normal double-send bug. It is the What we verified:
So this change takes the pragmatic path: for Telegram DMs, keep preview streaming on a real message ( Manual repro after this patch: no duplicate flash across plain text and tool-use DM streaming cases. |
|
I like this change just because the amount of issues we get about the "duplicate" send. I vote we revisit sendMessageDraft if/when telegram makes it less visually jarring. |
b65aa78 to
3456d64
Compare
|
Landed via temp rebase onto main. |
|
This PR trades native The original Would it be worth exploring a fix that keeps draft transport but addresses the materialize hop specifically? For example, clearing the draft before materializing, or using a short delay to let the client reconcile. Losing the typewriter effect in DMs feels like a regression for users who valued the smooth streaming experience. |
|
@yusnake if you can make a PR for it, I'll be happy to review it. |
|
Drop a note in clawtribotors or telegram channel in discord if you can when the pr is compete. @yusnake |
Summary
sendMessage+editMessageTextfor Telegram DM preview lanesFixes #36697
Related #33453
Testing
pnpm test -- src/telegram/draft-stream.test.ts src/telegram/lane-delivery.test.ts src/telegram/bot-message-dispatch.test.ts2026.3.8with text-only and tool-use prompts; no duplicate flash