Skip to content

Cherry-pick 46da76e26: fix(slack): honor replyToModeByChatType when ThreadLabel exists#1425

Merged
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/46da76e26-slack-reply-threadlabel
Mar 15, 2026
Merged

Cherry-pick 46da76e26: fix(slack): honor replyToModeByChatType when ThreadLabel exists#1425
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/46da76e26-slack-reply-threadlabel

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick of upstream commit 46da76e26 — "fix(slack): honor replyToModeByChatType when ThreadLabel exists (openclaw#26251)"

Conflicts resolved:

  • CHANGELOG.md — removed (deleted in fork)

Part of #677

…claw#26251)

* fix(slack): honor direct replyToMode when thread label exists

ThreadLabel is a session/conversation label, not a reliable indicator
of an actual Slack thread reply. Using it to force replyToMode="all"
overrides replyToModeByChatType.direct="off" in DMs.

Switch to MessageThreadId which indicates a real thread target is
available, preserving expected behavior: thread replies stay threaded,
normal DMs respect the configured mode.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

* Slack: add changelog for threading tool context fix

---------

Co-authored-by: Claude Opus 4.6 <[email protected]>
Co-authored-by: Tak Hoffman <[email protected]>
(cherry picked from commit 46da76e)
@alexey-pelykh alexey-pelykh force-pushed the cherry-pick/46da76e26-slack-reply-threadlabel branch from cc10986 to eff65c7 Compare March 15, 2026 13:36
@alexey-pelykh alexey-pelykh merged commit 69b35f3 into main Mar 15, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/46da76e26-slack-reply-threadlabel branch March 15, 2026 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants