Skip to content

Add CODEOWNERS and user search suggestions in popup - #16

Merged
ezyasin merged 2 commits into
mainfrom
feat/issues-1-2
Nov 6, 2026
Merged

ezyasin merged 2 commits into
mainfrom
feat/issues-1-2

Conversation

@ezyasin

@ezyasin ezyasin commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Closes #1
Closes #2

#1 — CODEOWNERS

The "Vouch - Sync CODEOWNERS" workflow failed because the repo had no CODEOWNERS file. Adds .github/CODEOWNERS with * @omarkurt @dogancanbakir @recepgunes1 @ArjinAlbay @baranylcn. Once merged, the sync workflow will add them to VOUCHED.td.

#2 — matching users below the search bar

  • core: new client.searchUsers(query, limit) backed by GET /search/users (in:login). Strips a leading @, rejects input that can't be a GitHub username without an API call, clamps the limit to 1–20, caches results for 10 min. Exports UserSearchResult and USER_SEARCH_CACHE_TTL.
  • extension: new SEARCH_USERS message and background handler.
  • popup: suggestion dropdown under the input with avatar, username, and an org/bot tag. Starts after 2 characters with a 250ms debounce; ignores stale responses. Arrow keys / Enter / Escape navigation; clicking a suggestion runs the score lookup.

Testing

  • 5 new tests in packages/core/__tests__/github/client.test.ts; all 220 tests pass.
  • bun run build, bun run typecheck, bun run lint pass (no new warnings).
  • Not yet manually tested in a browser.

- add .github/CODEOWNERS so the vouch sync-codeowners workflow has a source (closes #1)
- add searchUsers() to core client backed by GitHub user search, with input
  validation, limit clamping and caching
- show matching users below the popup search bar with keyboard navigation (closes #2)
@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f5c23714-de4e-4835-8bdf-0d611676ca72
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@ezyasin
ezyasin requested a review from dogancanbakir November 5, 2026 18:22
@ezyasin
ezyasin merged commit 173f240 into main Nov 6, 2026
3 checks passed
@ezyasin
ezyasin deleted the feat/issues-1-2 branch November 6, 2026 16:01
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.

When searching within the extension, relevant users can be seen below the search bar. add codeowners file

1 participant