Skip to content

fix(slack): return replyable thread ID from channel posts#720

Open
akkadaska wants to merge 1 commit into
vercel:mainfrom
akkadaska:fix/slack-channel-post-thread-id
Open

fix(slack): return replyable thread ID from channel posts#720
akkadaska wants to merge 1 commit into
vercel:mainfrom
akkadaska:fix/slack-channel-post-thread-id

Conversation

@akkadaska

Copy link
Copy Markdown
Contributor

summary

fixes #719

updates Slack channel posts to return a replyable thread ID rooted at the newly posted top-level message

postChannelMessage still omits thread_ts for the initial channel post, then uses Slack's returned message ts to construct slack:<channel>:<ts> for the SentMessage

this allows chat.thread(sent.threadId).post(...) to reply to a message returned by channel.post(...) instead of creating another top-level channel message

file-only uploads without a Slack message ts keep their existing channel-scoped thread ID

test plan

  • added a Slack adapter regression assertion that postChannelMessage returns slack:C123:<message ts>
  • ran the @chat-adapter/slack test suite
  • ran pnpm validate

checklist

  • all commits are signed and verified
  • targeted builds and tests pass
  • changeset added or not applicable
  • documentation updated or not applicable

@akkadaska
akkadaska requested a review from a team as a code owner July 16, 2026 12:22
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@akkadaska is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

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.

Slack channel.post() returns a threadId that cannot be used to reply to the posted message

1 participant