Skip to content

Cherry-pick 9ae94390b: fix(slack): resolve replyToMode per-message using chat type#1420

Merged
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/9ae94390b-slack-replytomode
Mar 15, 2026
Merged

Cherry-pick 9ae94390b: fix(slack): resolve replyToMode per-message using chat type#1420
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/9ae94390b-slack-replytomode

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick of upstream commit 9ae94390b — "fix(slack): resolve replyToMode per-message using chat type (openclaw#24717)"

Conflicts resolved:

  • CHANGELOG.md — removed (deleted in fork)

Part of #677

…#24717)

* fix(slack): resolve replyToMode per-message using chat type

The Slack monitor resolved replyToMode once at startup from the
top-level config, ignoring replyToModeByChatType overrides. This caused
DM replies to be threaded even when replyToModeByChatType.direct was
set to "off".

Now the inbound message handler calls resolveSlackReplyToMode(account,
chatType) per-message — the same function already used by the outbound
dock and tool threading context — so per-chat-type overrides take
effect on the inbound path.

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

* Slack: add changelog for per-message replyToMode resolution

---------

Co-authored-by: Claude Opus 4.6 <[email protected]>
Co-authored-by: Tak Hoffman <[email protected]>
(cherry picked from commit 9ae9439)
@alexey-pelykh alexey-pelykh force-pushed the cherry-pick/9ae94390b-slack-replytomode branch from 80c9696 to 6e6fa94 Compare March 15, 2026 12:25
@alexey-pelykh alexey-pelykh merged commit e2bcd07 into main Mar 15, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/9ae94390b-slack-replytomode branch March 15, 2026 12:34
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