Skip to content

Fix Telegram progress draft cleanup before tool output#92998

Closed
zhangguiping-xydt wants to merge 2 commits into
openclaw:mainfrom
zhangguiping-xydt:fix/issue-90753-dispatch-lifecycle
Closed

Fix Telegram progress draft cleanup before tool output#92998
zhangguiping-xydt wants to merge 2 commits into
openclaw:mainfrom
zhangguiping-xydt:fix/issue-90753-dispatch-lifecycle

Conversation

@zhangguiping-xydt

Copy link
Copy Markdown
Contributor

Summary

  • Problem: Telegram progress-mode drafts could remain visible when verbose tool output or visible tool artifacts were delivered as standalone messages.
  • Solution: Rotate and clear tool-progress-only answer drafts before sending visible tool output payloads.
  • What changed: Updated Telegram dispatch ordering and added regression coverage for verbose tool text and visible tool artifact paths.
  • What did NOT change: Telegram transport, rich message rendering, provider behavior, tool execution, auth, secrets, network, or progress draft implementation internals.

Fixes #90753

Regression Test Plan

  • Coverage level: Unit test
  • Target test file: extensions/telegram/src/bot-message-dispatch.test.ts
  • Scenario locked in: progress drafts are cleared before durable verbose tool output and visible tool artifacts are delivered.
  • Why this is the smallest reliable guardrail: the regression is dispatch ordering, so the test asserts draft clearing happens before standalone Telegram deliveries.

Verification run:

pnpm test extensions/telegram/src/bot-message-dispatch.test.ts -- -t "clears progress drafts before"

Test Files  1 passed (1)
Tests       2 passed | 118 skipped (120)

Root Cause

  • Root cause: progress-mode tool status drafts were not rotated before all visible tool-output delivery paths.
  • Missing detection / guardrail: existing tests covered final-answer cleanup but not durable verbose tool output or visible tool artifact delivery.

@openclaw-barnacle openclaw-barnacle Bot added channel: telegram Channel integration: telegram size: S r: too-many-prs Auto-close: author has more than twenty active PRs. labels Jun 14, 2026
@openclaw-barnacle openclaw-barnacle Bot added the triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup. label Jun 14, 2026
@openclaw-barnacle

Copy link
Copy Markdown

Closing this PR because the author has more than 20 active PRs in this repo. Please reduce the active PR queue and reopen or resubmit once it is back under the limit. You can close your own PRs to get back under the limit.

@openclaw-barnacle

Copy link
Copy Markdown

Closing this PR because the author has more than 20 active PRs in this repo. Please reduce the active PR queue and reopen or resubmit once it is back under the limit. You can close your own PRs to get back under the limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: telegram Channel integration: telegram r: too-many-prs Auto-close: author has more than twenty active PRs. size: S triage: needs-real-behavior-proof Candidate: external PR needs after-fix proof from a real setup.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Telegram progress bubble remains above subsequent assistant messages in progress streaming mode

1 participant