Skip to content

feat(telegram): unify progress fallback with assistant preview lane#95573

Closed
snowzlmbot wants to merge 6 commits into
openclaw:mainfrom
snowzlmbot:fix/telegram-unify-progress-placeholder-assistant-preview-clean
Closed

feat(telegram): unify progress fallback with assistant preview lane#95573
snowzlmbot wants to merge 6 commits into
openclaw:mainfrom
snowzlmbot:fix/telegram-unify-progress-placeholder-assistant-preview-clean

Conversation

@snowzlmbot

@snowzlmbot snowzlmbot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

What Problem This Solves

Supersedes #95522 with a clean branch rebuilt on top of the latest main.

#95522 conflicted after #95183 landed on main. Latest main already 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:

  • adds the Telegram progress assistantPreview lane
  • scopes assistantPreview to streaming.mode = "progress"
  • bounds the initial preview delay
  • updates Telegram config hints and generated config metadata
  • updates plugin SDK streaming surface coverage
  • adds regression coverage for progress, partial, block, and off streaming modes

Evidence

Clean Branch / Conflict Resolution

This branch was rebuilt from the current origin/main and replays only the commits that are still unique to the assistant-preview feature. It avoids reintroducing the #95183 placeholder changes that are already merged into main.

Current head: 732094f75f900bdf914d290ed41336e7a46ecdf2

Commits over latest main:

  • feat(telegram): add progress assistant preview lane
  • fix(telegram): scope assistant preview config
  • test(telegram): align assistant preview with rich progress drafts
  • test(plugin-sdk): account for assistant preview public surface
  • fix(telegram): bound assistant preview initial delay
  • fix(plugin-sdk): gate assistant preview to progress mode

Real Behavior Proof

Dedicated Telegram test bot proof from the corrected late segment of the original recording:

The clip shows:

  • progress placeholder appears first
  • short assistant preview appears through the assistant-preview lane
  • final answer replaces the preview path
  • progress draft cleanup still happens correctly

Local Targeted Validation

git diff --check origin/main...HEAD
node scripts/plugin-sdk-surface-report.mjs --check
pnpm config:channels:check
pnpm config:schema:check
node scripts/run-vitest.mjs \
  test/scripts/plugin-sdk-surface-report.test.ts \
  src/plugin-sdk/channel-streaming.test.ts \
  src/agents/model-runtime-policy.test.ts \
  extensions/telegram/src/bot-message-dispatch.test.ts \
  extensions/telegram/src/draft-stream.test.ts \
  extensions/telegram/src/lane-delivery.test.ts \
  src/channels/progress-draft-compositor.test.ts

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 assistantPreview only affects streaming.mode = "progress". Regression tests cover the non-progress modes to prevent preview leakage into partial, block, or off.

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation channel: telegram Channel integration: telegram scripts Repository scripts agents Agent runtime and tooling size: M labels Jun 21, 2026
@snowzlmbot

Copy link
Copy Markdown
Contributor Author

Clean replacement for #95522 is ready from a fresh branch on latest main.

Key cleanup:

@clawsweeper review-pr

@clawsweeper

clawsweeper Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

@openclaw-barnacle openclaw-barnacle Bot added the triage: needs-pr-context Candidate: external PR body lacks required problem context or evidence. label Jun 21, 2026
@clawsweeper

clawsweeper Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

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.

@openclaw-barnacle openclaw-barnacle Bot removed the triage: needs-pr-context Candidate: external PR body lacks required problem context or evidence. label Jun 21, 2026
@snowzlmbot

Copy link
Copy Markdown
Contributor Author

Close this PR, feel free to reopen it if needed!

@snowzlmbot snowzlmbot closed this Jun 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling channel: telegram Channel integration: telegram docs Improvements or additions to documentation scripts Repository scripts size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants