Skip to content

fix(slack): skip urls during mention resolution#619

Merged
dancer merged 4 commits into
mainfrom
josh/sss
Jun 22, 2026
Merged

fix(slack): skip urls during mention resolution#619
dancer merged 4 commits into
mainfrom
josh/sss

Conversation

@dancer

@dancer dancer commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

summary

prevents cached Slack display names inside urls from being resolved as user mentions before payload formatting

shares url-aware mention handling between cached user lookup and Slack formatting while preserving real mentions

follow-up to #618, which only protected the final formatting pass

@dancer dancer requested a review from a team as a code owner June 16, 2026 11:38
@vercel

vercel Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
chat Ready Ready Preview, Comment, Open in v0 Jun 22, 2026 10:07am
chat-sdk-nextjs-chat Ready Ready Preview, Comment, Open in v0 Jun 22, 2026 10:07am

Comment thread packages/adapter-slack/src/markdown.ts Fixed
@dancer dancer merged commit a8c4af7 into main Jun 22, 2026
17 checks passed
@dancer dancer deleted the josh/sss branch June 22, 2026 10:24
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.

3 participants