-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: Discord partial/block streaming can emit misleading mid-output fragments #78561
Copy link
Copy link
Closed as not planned
Closed as not planned
Copy link
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-infoClawSweeper needs more reporter information before it can verify this issue.ClawSweeper needs more reporter information before it can verify this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦪 silver shellfishThin issue quality; more reproduction proof or environment detail is needed.Thin issue quality; more reproduction proof or environment detail is needed.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:needs-infoClawSweeper needs more reporter information before it can verify this issue.ClawSweeper needs more reporter information before it can verify this issue.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦪 silver shellfishThin issue quality; more reproduction proof or environment detail is needed.Thin issue quality; more reproduction proof or environment detail is needed.
Type
Fields
Priority
None yet
Summary
Discord partial/block streaming can expose awkward or misleading mid-output fragments in-channel during long turns, especially around tool-progress / progress-like output. This is separate from the final-send sanitizer bug fixed in #78559.
Observed while debugging
During a live Discord debugging session with
streaming.mode=offand normal Discord replies enabled:📖 Read:/🛠️ Exec:disappear, while✍️ Write:stayed visible;The sanitizer bug has a concrete fix in #78559. This issue tracks the remaining streaming UX/runtime behavior so it does not get conflated with the sanitizer fix.
Why separate from #78559
#78559 addresses the final Discord send boundary:
This issue is about the earlier live/partial/block streaming path and when/how Discord sends intermediate output while a turn is still running.
Expected behavior
When Discord streaming is disabled or configured to avoid partial output, Discord should not emit misleading mid-output fragments. If progress/status is emitted, it should be explicitly classified and rendered consistently, not accidentally treated as assistant prose or sanitizer leakage.
Notes
Related existing issue found during triage:
This may share code paths with
createReplyDispatcherWithTyping, block replies, and Discord draft/live preview handling, but I have not root-caused it enough to propose a patch yet.