fix(chat): preserve Telegram sender labels in dashboard history#39414
Merged
fix(chat): preserve Telegram sender labels in dashboard history#39414
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Contributor
Greptile SummaryThis PR fixes the root cause of missing per-sender attribution in the dashboard chat history for Telegram group messages. It threads a Key changes:
Confidence Score: 5/5
Last reviewed commit: 864145c |
Contributor
Author
864145c to
f8c0717
Compare
Contributor
Author
|
Landed via temp rebase onto
|
mrosmarin
added a commit
to mrosmarin/openclaw
that referenced
this pull request
Mar 8, 2026
* main: (290 commits) test: stabilize exec resolver timeout fixture chore: add changelog and format fix for openclaw#39414 fix(chat): preserve sender labels in dashboard history docs: clean up latest changelog sections docs: dedupe changelog contributor attribution fix(ci): resolve current gate regressions refactor(voice-call): share tts deep merge fix: land openclaw#39337 by @goodspeed-apps for acpx MCP bootstrap fix(ci): resolve type regressions on main fix: document discord agentComponents schema parity (openclaw#39378) (thanks @gambletan) (openclaw#39378) fix(discord): validate agentComponents config test: cover daemon probe auth seam refactor: preserve explicit mock voice-call values refactor: register gateway service adapters refactor: reuse shared gateway probe auth refactor: split daemon status gathering refactor: centralize strict numeric parsing refactor: normalize voice-call runtime defaults fix(ci): pin multi-arch docker base digests docs: add changelog for Telegram DM draft restore (openclaw#39398) ...
mcaxtr
pushed a commit
to mcaxtr/openclaw
that referenced
this pull request
Mar 8, 2026
ziomancer
pushed a commit
to ziomancer/openclaw
that referenced
this pull request
Mar 8, 2026
Saitop
pushed a commit
to NomiciAI/openclaw
that referenced
this pull request
Mar 8, 2026
GordonSH-oss
pushed a commit
to GordonSH-oss/openclaw
that referenced
this pull request
Mar 9, 2026
jenawant
pushed a commit
to jenawant/openclaw
that referenced
this pull request
Mar 10, 2026
4 tasks
dhoman
pushed a commit
to dhoman/chrono-claw
that referenced
this pull request
Mar 11, 2026
senw-developers
pushed a commit
to senw-developers/va-openclaw
that referenced
this pull request
Mar 17, 2026
V-Gutierrez
pushed a commit
to V-Gutierrez/openclaw-vendor
that referenced
this pull request
Mar 17, 2026
alexey-pelykh
pushed a commit
to remoteclaw/remoteclaw
that referenced
this pull request
Mar 22, 2026
(cherry picked from commit 5214859)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
senderLabelin the shared gateway chat sanitizer before stripping AI-only metadata from user messagesFixes #38989.
Root cause
The dashboard was stripping inbound Telegram metadata before rendering, but sender attribution only existed inside those AI-only metadata blocks. After sanitization, the UI had no per-message sender identity left, so it grouped consecutive group messages only by role and labeled them as "You".
This patch fixes the shared sanitized message shape instead of reparsing stripped prompt metadata in the UI.
Testing