Skip to content

feat(slack): add SLACK_LIST_DMS tool and sender name in DMs#371

Merged
JonasJesus42 merged 2 commits intomainfrom
JonasJesus42/slack-dm-tools
Apr 10, 2026
Merged

feat(slack): add SLACK_LIST_DMS tool and sender name in DMs#371
JonasJesus42 merged 2 commits intomainfrom
JonasJesus42/slack-dm-tools

Conversation

@JonasJesus42
Copy link
Copy Markdown
Contributor

@JonasJesus42 JonasJesus42 commented Apr 10, 2026

Summary

  • Adds SLACK_LIST_DMS tool that lists all bot DM conversations with resolved user names and channel IDs
  • Injects the sender's display name into DM messages sent to the LLM ([Mensagem de {name}]) so the agent knows who it's talking to
  • Adds user? field to SlackChannel type to support IM channel data from Slack API

Test plan

  • Deploy and call SLACK_LIST_DMS — should return DM conversations with user names
  • Use a returned channel_id with SLACK_GET_CHANNEL_HISTORY to read DM messages
  • Send a DM to the bot and verify the LLM receives the sender name in the message

Summary by cubic

Add SLACK_LIST_DMS to list the bot’s DM conversations with resolved user names and channel IDs, and inject the sender’s display name into DM messages so the agent knows who is speaking. Also remove deprecated tsconfig path settings to keep CI green on TypeScript 7.0.

  • New Features

    • SLACK_LIST_DMS returns DMs with { channel_id, user_id, user_name } and optional limit.
    • DM messages include “[Mensagem de {name}]” when the sender is resolved via getUserInfo.
    • SlackChannel adds user? for IM channel data; tool registered in channelTools.
  • Bug Fixes

    • Removed unused baseUrl/paths from tsconfig.json (deprecated in TS 7.0).

Written for commit 989175f. Summary will update on new commits.

Add a dedicated tool to list the bot DM conversations with resolved
user names, and inject the sender display name into DM messages so
the LLM agent knows who it is talking to.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

TypeScript 7.0 deprecated baseUrl. The path aliases were not used
anywhere in the codebase, so removing them fixes the CI check.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@JonasJesus42 JonasJesus42 merged commit 57b5a03 into main Apr 10, 2026
2 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.

1 participant