Skip to content

fix(slack): include recipient user in reply streams#21

Merged
parkedwards merged 1 commit into
mainfrom
ep/fix-slack-client-missing-userid
Mar 11, 2026
Merged

fix(slack): include recipient user in reply streams#21
parkedwards merged 1 commit into
mainfrom
ep/fix-slack-client-missing-userid

Conversation

@parkedwards

Copy link
Copy Markdown
Collaborator

Summary

  • include recipient_user_id for streamed Slack replies whenever the incoming message has a user id
  • persist that user id on the stream handle so chat.stopStream uses the same recipient context
  • cover the channel-thread and missing-user-id cases in docketeer-slack tests

Validation

  • cd docketeer-slack && pytest

@parkedwards
parkedwards merged commit 2da99a9 into main Mar 11, 2026
3 checks passed
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