Skip to content

Support Slack thread targets in send_message - #17741

Closed
carat-jiho wants to merge 1 commit into
NousResearch:mainfrom
carat-jiho:fix-slack-send-message-thread-target
Closed

Support Slack thread targets in send_message#17741
carat-jiho wants to merge 1 commit into
NousResearch:mainfrom
carat-jiho:fix-slack-send-message-thread-target

Conversation

@carat-jiho

Copy link
Copy Markdown

Summary

  • Parse Slack targets of the form slack:<channel_id>:<thread_ts> in send_message
  • Pass parsed thread IDs through to Slack chat.postMessage as thread_ts
  • Add parser coverage for Slack thread targets

Verification

  • python3 -m py_compile tools/send_message_tool.py
  • Direct parser import checks for channel-only, channel+thread_ts, invalid thread, and user ID targets

Note: pytest was not available in the execution environment used to prepare this patch.

@carat-jiho carat-jiho closed this Apr 30, 2026
@carat-jiho
carat-jiho deleted the fix-slack-send-message-thread-target branch April 30, 2026 03:56
@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists platform/slack Slack app adapter comp/tools Tool registry, model_tools, toolsets labels Apr 30, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Likely duplicate of #16992 — same Slack thread_ts parsing fix for send_message. This PR is closed; #16992 remains open.

@alt-glitch

Copy link
Copy Markdown
Collaborator

Likely duplicate of #16992 — same Slack thread_ts parsing fix for send_message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tools Tool registry, model_tools, toolsets P2 Medium — degraded but workaround exists platform/slack Slack app adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants