-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
Threaded Slack messaging-tool sends can suppress later automatic replies #90044
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.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.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.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.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Type
Fields
Priority
None yet
Summary
When a Slack conversation is already bound to a thread, a successful messaging-tool send inside that thread can be recorded too loosely for later reply dedupe. That can make a later visible reply look like a duplicate of the earlier tool send even when the follow-up reply is part of the same thread-bound conversation lifecycle.
Impact
This shows up as inconsistent Slack thread continuity:
visibleReplies: automaticcan look inconsistent when a prior messaging-tool send happened in the same turnExpected
A messaging-tool send into Slack thread
T1should only count as dedupe evidence for the same routed threadT1.Actual
The current dedupe path can collapse a threaded send down to the channel target, so later replies can be suppressed as if the whole channel already got the response.
Likely cause
There appear to be two contributing problems:
messagetool sends do not always preserve the active session thread id in send telemetryRegression coverage to add or keep
messagesends