Skip to content

Prioritize channel members in mention autocomplete#1431

Merged
wesbillman merged 1 commit into
mainfrom
pinky/mention-autocomplete-order
Jul 1, 2026
Merged

Prioritize channel members in mention autocomplete#1431
wesbillman merged 1 commit into
mainfrom
pinky/mention-autocomplete-order

Conversation

@wesbillman

Copy link
Copy Markdown
Collaborator

Summary

  • rank mention autocomplete so channel members appear first, then active/runnable persona-backed suggestions, then other people, then other agents
  • preserve ownership metadata for local managed agents so autocomplete can show owned by you
  • extract mention ranking into a tested helper

Test plan

  • pnpm --dir desktop test -- src/features/messages/lib/mentionRanking.test.mjs
  • pnpm --dir desktop typecheck
  • pnpm --dir desktop check
  • git diff --check

Co-authored-by: Pinky <44b8e82baa6e0e254e0208d68f335c283c94e7b78dd1fa10d5a49d3f13dd0435@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
@wesbillman wesbillman force-pushed the pinky/mention-autocomplete-order branch from ea82095 to bd968dc Compare July 1, 2026 16:58
@wesbillman wesbillman merged commit 8fb33bd into main Jul 1, 2026
25 checks passed
@wesbillman wesbillman deleted the pinky/mention-autocomplete-order branch July 1, 2026 17:54
tlongwell-block pushed a commit that referenced this pull request Jul 2, 2026
Brings the branch current with main (~20 commits, incl. relay perf #1453/#1454
and mention ranking #1431). One conflict resolved in useMentions.ts: kept
main's rankMentionCandidates pipeline, re-applied this branch's suggestion
slice change (Math.max(MENTION_SUGGESTION_LIMIT, mentionCandidates.length)).

Verified post-merge: tsc --noEmit clean, biome check clean, desktop unit
tests 1475/1475, cargo check --workspace clean.

Co-authored-by: Tyler Longwell <tlongwell@block.xyz>
Signed-off-by: Tyler Longwell <tlongwell@block.xyz>
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