Skip to content

fix: Eliminate N+1 API calls in ProjectSwitcher and on app launch by deriving projects from @me response#955

Merged
charlesvien merged 2 commits intomainfrom
02-19-eliminate_n_1_api_calls_in_projectswitcher_by_deriving_projects_from_me_response
Feb 20, 2026
Merged

fix: Eliminate N+1 API calls in ProjectSwitcher and on app launch by deriving projects from @me response#955
charlesvien merged 2 commits intomainfrom
02-19-eliminate_n_1_api_calls_in_projectswitcher_by_deriving_projects_from_me_response

Conversation

@charlesvien
Copy link
Member

@charlesvien charlesvien commented Feb 19, 2026

  • useProjects now derives the project list from user.organization.teams (already in the @me response) instead of making N individual GET /api/projects/{id}/ calls
  • Pre-seed ["currentUser"] query cache from auth init to eliminate the duplicate getCurrentUser() call

@charlesvien charlesvien changed the title Eliminate N+1 API calls in ProjectSwitcher by deriving projects from @me response fix: Eliminate N+1 API calls in ProjectSwitcher by deriving projects from @me response Feb 19, 2026
@charlesvien charlesvien changed the title fix: Eliminate N+1 API calls in ProjectSwitcher by deriving projects from @me response fix: Eliminate N+1 API calls in ProjectSwitcher on app launch by deriving projects from @me response Feb 19, 2026
@charlesvien charlesvien changed the title fix: Eliminate N+1 API calls in ProjectSwitcher on app launch by deriving projects from @me response fix: Eliminate N+1 API calls in ProjectSwitcher and on app launch by deriving projects from @me response Feb 19, 2026
@charlesvien charlesvien marked this pull request as ready for review February 19, 2026 23:22
@charlesvien charlesvien force-pushed the 02-19-eliminate_n_1_api_calls_in_projectswitcher_by_deriving_projects_from_me_response branch from d70f08a to 6a6b7c7 Compare February 19, 2026 23:30
@charlesvien charlesvien force-pushed the 02-19-eliminate_n_1_api_calls_in_projectswitcher_by_deriving_projects_from_me_response branch from a75062f to b05c977 Compare February 20, 2026 01:18
Copy link
Contributor

@k11kirky k11kirky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@charlesvien charlesvien force-pushed the 02-19-eliminate_n_1_api_calls_in_projectswitcher_by_deriving_projects_from_me_response branch from b05c977 to 81dfb83 Compare February 20, 2026 02:35
Copy link
Member Author

charlesvien commented Feb 20, 2026

Merge activity

  • Feb 20, 2:57 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Feb 20, 2:57 AM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien merged commit 383d7d9 into main Feb 20, 2026
13 checks passed
@charlesvien charlesvien deleted the 02-19-eliminate_n_1_api_calls_in_projectswitcher_by_deriving_projects_from_me_response branch February 20, 2026 02:57
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.

2 participants