Skip to content

ref(cmdk): Remove projects array from identifier lookup query key#114512

Merged
JonasBa merged 1 commit intomasterfrom
jonasba/ref/cmdk-remove-projects-from-querykey
Apr 30, 2026
Merged

ref(cmdk): Remove projects array from identifier lookup query key#114512
JonasBa merged 1 commit intomasterfrom
jonasba/ref/cmdk-remove-projects-from-querykey

Conversation

@JonasBa
Copy link
Copy Markdown
Member

@JonasBa JonasBa commented Apr 30, 2026

Remove projects from CMDK to mitigate the performance cost of serializing large project sets in order to perform cache lookup

Remove the full projects array from the useQuery queryKey in
ResolvedIdentifierCommandPaletteAction. TanStack Query serializes
keys for cache identity, and including the entire projects array
is expensive for orgs with thousands of projects — especially since
it runs on every keystroke even when the query is disabled.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 30, 2026
@JonasBa JonasBa marked this pull request as ready for review April 30, 2026 22:50
@JonasBa JonasBa requested a review from a team as a code owner April 30, 2026 22:50
@JonasBa JonasBa enabled auto-merge (squash) April 30, 2026 22:56
@JonasBa JonasBa merged commit 98b85ea into master Apr 30, 2026
73 checks passed
@JonasBa JonasBa deleted the jonasba/ref/cmdk-remove-projects-from-querykey branch April 30, 2026 22:57
cleptric pushed a commit that referenced this pull request May 5, 2026
…14512)

Remove projects from CMDK to mitigate the performance cost of
serializing large project sets in order to perform cache lookup
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