Skip to content

fix: pass profiles to mention autocomplete in threads and forum composers#541

Merged
wesbillman merged 1 commit into
mainfrom
thread-avatar-fix
May 11, 2026
Merged

fix: pass profiles to mention autocomplete in threads and forum composers#541
wesbillman merged 1 commit into
mainfrom
thread-avatar-fix

Conversation

@wesbillman
Copy link
Copy Markdown
Collaborator

Summary

  • The @ mention autocomplete in thread composers and forum composers showed only initials instead of avatars because the profiles prop (carrying avatar URLs) was not being forwarded to useMentions()
  • Added profiles prop passthrough in MessageThreadPanel, ForumComposer, ForumThreadPanel, ForumView, and PulseView

Changes

File Change
MessageThreadPanel.tsx Added profiles={profiles} to <MessageComposer>
ForumComposer.tsx Added profiles prop type + forwarded to useMentions()
ForumThreadPanel.tsx Added profiles={profiles} to <ForumComposer>
ForumView.tsx Added profiles={profiles} to <ForumComposer>
PulseView.tsx Added profiles={mentionProfiles} to <ForumComposer>

Test plan

  • pnpm exec tsc --noEmit passes clean
  • All pre-commit and pre-push hooks pass (linting, builds, tests)
  • Open a thread in main chat → click @ → verify avatars appear
  • Open a forum post → reply composer → click @ → verify avatars appear
  • Open Pulse → post composer → click @ → verify avatars appear

🤖 Generated with Claude Code

…sers

The `profiles` prop carrying avatar URLs was not being forwarded to
`useMentions()` in thread and forum composers, causing the mention
autocomplete to show initials instead of avatars.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wesbillman wesbillman enabled auto-merge (squash) May 11, 2026 23:11
@wesbillman wesbillman merged commit 19f2906 into main May 11, 2026
15 checks passed
@wesbillman wesbillman deleted the thread-avatar-fix branch May 11, 2026 23:12
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