feat(telegram): unify progress fallback with assistant preview lane#95573
feat(telegram): unify progress fallback with assistant preview lane#95573snowzlmbot wants to merge 6 commits into
Conversation
|
Clean replacement for #95522 is ready from a fresh branch on latest Key cleanup:
@clawsweeper review-pr |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: feat(telegram): unify progress fallback with assistant preview lane This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
|
Close this PR, feel free to reopen it if needed! |
What Problem This Solves
Supersedes #95522 with a clean branch rebuilt on top of the latest
main.#95522 conflicted after #95183 landed on
main. Latestmainalready contains the default progress placeholder / delayed-clear fixes from #95183, so this PR intentionally keeps only the remaining assistant-preview/config/SDK/docs changes.This fixes the remaining Telegram progress-streaming gap where a short assistant preview could not share the same progress fallback lane cleanly. The PR:
assistantPreviewlaneassistantPreviewtostreaming.mode = "progress"progress,partial,block, andoffstreaming modesEvidence
Clean Branch / Conflict Resolution
This branch was rebuilt from the current
origin/mainand replays only the commits that are still unique to the assistant-preview feature. It avoids reintroducing the #95183 placeholder changes that are already merged intomain.Current head:
732094f75f900bdf914d290ed41336e7a46ecdf2Commits over latest
main:feat(telegram): add progress assistant preview lanefix(telegram): scope assistant preview configtest(telegram): align assistant preview with rich progress draftstest(plugin-sdk): account for assistant preview public surfacefix(telegram): bound assistant preview initial delayfix(plugin-sdk): gate assistant preview to progress modeReal Behavior Proof
Dedicated Telegram test bot proof from the corrected late segment of the original recording:
The clip shows:
Local Targeted Validation
Result: all listed checks passed; targeted Vitest passed 282 tests total.
Risk
Low-to-medium. The feature is scoped to Telegram progress streaming and explicitly gated so
assistantPreviewonly affectsstreaming.mode = "progress". Regression tests cover the non-progress modes to prevent preview leakage intopartial,block, oroff.