Skip to content

feat(onboarding): Pre-populate repo selector with full repo list#112685

Merged
jaydgoss merged 5 commits intomasterfrom
jaygoss/vdy-46-scm-repo-selector-FE-browse
Apr 13, 2026
Merged

feat(onboarding): Pre-populate repo selector with full repo list#112685
jaydgoss merged 5 commits intomasterfrom
jaygoss/vdy-46-scm-repo-selector-FE-browse

Conversation

@jaydgoss
Copy link
Copy Markdown
Member

@jaydgoss jaydgoss commented Apr 10, 2026

Fetch the complete list of repositories on mount so users can browse and select without typing first. The Select component handles filtering client-side when the user types, so there is no server-side search round-trip.

Previously the dropdown was empty until a search query was entered, requiring users to already know the repo name. Adds the virtualized menu list component to keep the dropdown performant with large repo lists.

Refs VDY-46

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 10, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 10, 2026
@jaydgoss jaydgoss force-pushed the jaygoss/vdy-46-scm-repo-selector-FE-browse branch from 95640d7 to f2bc339 Compare April 10, 2026 17:12
@jaydgoss jaydgoss force-pushed the jaygoss/vdy-46-scm-repo-selector-FE-browse branch from f2bc339 to bc8e69f Compare April 10, 2026 17:14
@jaydgoss jaydgoss force-pushed the jaygoss/vdy-46-scm-repo-selector-FE-browse branch 2 times, most recently from e161280 to cc153f7 Compare April 10, 2026 17:17
Load the complete list of accessible repositories on mount so users
can browse and select without typing first. Search still narrows
results server-side when the user types.

Previously the dropdown was empty until a search query was entered,
requiring users to know what to search for.

Refs VDY-46
Tests expected the old search-driven behavior where data only loaded
after typing. Now that repos fetch on mount, update mocks and
assertions to match.

Add a fallback in ScmVirtualizedMenuList for when the virtualizer
cannot measure the container (jsdom has no layout engine).
Use the same @tanstack/react-virtual mock as scmPlatformFeatures
tests instead of a runtime fallback in the component. Reverts the
jsdom fallback in ScmVirtualizedMenuList.
Update comment to reflect the new fetch-on-mount model instead of
referencing server-side search results. Bump staleTime from 20s to 60s
since the repo list is now fetched once and filtered client-side.
@jaydgoss jaydgoss marked this pull request as ready for review April 10, 2026 17:52
@jaydgoss jaydgoss requested a review from a team as a code owner April 10, 2026 17:52
@jaydgoss jaydgoss requested a review from a team April 13, 2026 17:28
@jaydgoss jaydgoss merged commit 465f1a0 into master Apr 13, 2026
65 checks passed
@jaydgoss jaydgoss deleted the jaygoss/vdy-46-scm-repo-selector-FE-browse branch April 13, 2026 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants