Skip to content

feat: resolve Slack usernames in Addie admin thread view#698

Merged
bokelley merged 1 commit into
mainfrom
bokelley/addie-thread-usernames
Jan 10, 2026
Merged

feat: resolve Slack usernames in Addie admin thread view#698
bokelley merged 1 commit into
mainfrom
bokelley/addie-thread-usernames

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

  • Add resolveSlackUserDisplayName and resolveSlackUserDisplayNames shared utilities to slack/client.ts
  • Resolve Slack user IDs to display names in thread titles and message content mentions
  • Persist resolved users to database for future lookups (avoids repeated API calls)
  • Implement concurrency limiting (5 parallel requests) to avoid Slack rate limits

Changes

  • server/src/slack/client.ts: New shared utilities for username resolution with DB caching
  • server/src/addie/member-context.ts: Use shared utility instead of duplicated code
  • server/src/routes/addie-admin.ts: Resolve all user mentions when fetching thread details
  • server/public/admin-addie.html: Replace <@U...> mentions with @DisplayName in rendered messages

Test plan

  • TypeScript type checking passes
  • All tests pass
  • Build succeeds
  • Manual testing: View an Addie thread with user mentions - should show display names instead of raw IDs

🤖 Generated with Claude Code

- Add resolveSlackUserDisplayName and resolveSlackUserDisplayNames
  utilities to slack/client.ts with concurrency limiting (5 at a time)
- Persist resolved users to database for future lookups
- Update member-context.ts to use shared utility
- Update addie-admin.ts thread endpoint to resolve mentions
- Update admin-addie.html to replace <@U...> mentions with @DisplayName

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bokelley bokelley merged commit a7f652f into main Jan 10, 2026
6 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