Skip to content

fix(messaging): attach Slack handoffs in source threads - #999

Merged
huntharo merged 1 commit into
mainfrom
fix/slack-handoff-threads
Jul 21, 2026
Merged

fix(messaging): attach Slack handoffs in source threads#999
huntharo merged 1 commit into
mainfrom
fix/slack-handoff-threads

Conversation

@huntharo

Copy link
Copy Markdown
Contributor

Summary

  • I fixed Slack Agent handoffs so a request from a channel message attaches to a native Slack thread under that source message instead of reporting that child conversations are unsupported.
  • I reuse Slack's preserved source timestamp as thread_ts; the first status delivery materializes the thread, while nested threads and sources without a message timestamp remain unsupported.
  • I added regression coverage from an inbound app mention through managed-conversation discovery, binding creation, and threaded delivery.

Verification

  • pnpm test — 4,743 passed, 3 skipped
  • pnpm typecheck
  • pnpm lint:eslint — 0 errors; existing warning baseline only
  • pnpm lint:boundaries

Notes

  • This uses the Slack adapter's existing chat.postMessage path and does not require an additional OAuth scope.

@huntharo
huntharo merged commit e809f3b into main Jul 21, 2026
8 checks passed
@huntharo
huntharo deleted the fix/slack-handoff-threads branch July 21, 2026 06:26
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.

1 participant