Skip to content

Fix Telegram rich progress detail updates#93698

Merged
obviyus merged 2 commits into
mainfrom
codex/fix-telegram-rich-progress-details
Jun 16, 2026
Merged

Fix Telegram rich progress detail updates#93698
obviyus merged 2 commits into
mainfrom
codex/fix-telegram-rich-progress-details

Conversation

@obviyus

@obviyus obviyus commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Preserve command details when a terminal command-output update replaces an earlier progress line, so Telegram rich progress no longer collapses to Bash completed.
  • Keep rich Telegram draft previews on the rich-message path for sizing/dedupe instead of projecting rich HTML through legacy parse_mode: HTML text.
  • Flip/add regression tests for both bugs.

Verification

  • node scripts/run-vitest.mjs src/plugin-sdk/channel-streaming.test.ts src/channels/streaming.test.ts extensions/telegram/src/draft-stream.test.ts
  • node scripts/run-tsgo.mjs -p test/tsconfig/tsconfig.extensions.test.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/extensions-test.tsbuildinfo
  • pnpm exec oxfmt --check --threads=1 src/channels/streaming.ts src/plugin-sdk/channel-streaming.test.ts extensions/telegram/src/draft-stream.ts extensions/telegram/src/draft-stream.test.ts extensions/telegram/src/bot-message-dispatch.test.ts
  • git diff --check
  • .agents/skills/autoreview/scripts/autoreview --mode branch --base origin/main -> clean, no accepted/actionable findings
  • Telegram E2E basic: driver message 34408, SUT reply 34409, OPENCLAW_E2E_OK
  • Telegram E2E with richMessages: true and rich progress config: driver message 34413, SUT reply 34414, OPENCLAW_E2E_OK

Known unrelated main drift: node scripts/run-vitest.mjs extensions/telegram/src/bot-message-dispatch.test.ts currently fails on origin/main with the same two existing assertions (keeps same-message block chunks..., suppresses non-terminal final error warnings...).

@openclaw-barnacle openclaw-barnacle Bot added channel: telegram Channel integration: telegram size: S maintainer Maintainer-authored PR labels Jun 16, 2026
@obviyus
obviyus force-pushed the codex/fix-telegram-rich-progress-details branch from 78a2715 to a0629c2 Compare June 16, 2026 16:26
@clawsweeper

clawsweeper Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: Fix Telegram rich progress detail updates 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.

@obviyus
obviyus force-pushed the codex/fix-telegram-rich-progress-details branch 3 times, most recently from f1b8168 to cff6b27 Compare June 16, 2026 17:15
@obviyus
obviyus force-pushed the codex/fix-telegram-rich-progress-details branch from cff6b27 to 77da499 Compare June 16, 2026 17:17
@obviyus
obviyus merged commit 42dcf70 into main Jun 16, 2026
27 checks passed
@obviyus
obviyus deleted the codex/fix-telegram-rich-progress-details branch June 16, 2026 17:18
@obviyus

obviyus commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Merged via squash.

Thanks @obviyus!

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jun 17, 2026
Merged via squash.

Prepared head SHA: 77da499
Co-authored-by: obviyus <[email protected]>
Co-authored-by: obviyus <[email protected]>
Reviewed-by: @obviyus
crh-code pushed a commit to crh-code/openclaw that referenced this pull request Jun 18, 2026
Merged via squash.

Prepared head SHA: 77da499
Co-authored-by: obviyus <[email protected]>
Co-authored-by: obviyus <[email protected]>
Reviewed-by: @obviyus
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: telegram Channel integration: telegram maintainer Maintainer-authored PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant