fix(tlon): strip internal tool-trace banners from outbound text#103450
Merged
Conversation
Route assistant-visible Tlon output through the shared sanitizer and prove the rendered Urbit poke payload for DMs, groups, and media captions. Co-authored-by: masatohoshino <[email protected]>
openclaw-barnacle
Bot
added
channel: tlon
Contributor
Author
|
Land-ready proof for exact head
Commands used for focused proof: Known gap: no credentialed hosted Tlon ship acknowledgement was available. The captured authenticated HTTP login and final Urbit poke payload directly prove the changed local transformation; no remote service behavior changed. |
Contributor
Author
|
Merged via squash.
|
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
Jul 10, 2026
…claw#103450) Route assistant-visible Tlon output through the shared sanitizer and prove the rendered Urbit poke payload for DMs, groups, and media captions. Co-authored-by: masatohoshino <[email protected]>
2 tasks
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
1 participant
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.
What Problem This Solves
Tlon outbound delivery did not apply the shared assistant-visible text sanitizer. Internal tool-call banners, raw tool markup, and failed-tool traces could therefore be sent to Tlon users as ordinary message text.
This maintainer replacement supersedes #102211. The contributor branch could not be updated after current-main synchronization because GitHub's signed GraphQL commit path exceeded its 45 MB request ceiling. Part of #90684.
Why This Change Was Made
The Tlon outbound adapter now uses the existing
sanitizeAssistantVisibleTexthook at the shared delivery boundary, matching sibling channel adapters without adding channel-specific parsing.The regression test registers the real Tlon plugin, drives
deliverOutboundPayloads, authenticates against a local HTTP server, and inspects the rendered Urbit poke payload for direct messages, group messages, and media captions. The original contribution is credited to @masatohoshino in the commit trailer.User Impact
Tlon users no longer see internal assistant tool traces in final outbound messages. Ordinary Markdown and fenced examples remain unchanged.
Evidence
search private repos.tbx_01kx57xtadevzxynm9nry0bx67: focused boundary test passed, 1 file / 1 test.pnpm check:changedpassed for theextensionsandextensionTestslanes, including tsgo, lint, storage guards, and import-cycle checks.