ref(ui): do not disable eslint-plugin-query for debouncing

Author: TkDodo

static/app/components/searchQueryBuilder/tokens/filter/valueCombobox.tsx

@@ -365,18 +365,16 @@ function useFilterSuggestions({
   );
 
   const baseQueryKey = useMemo(
-    () => ['search-query-builder-tag-values', queryParams],
+    () => ['search-query-builder-tag-values', queryParams] as const,
     [queryParams]
   );
   const queryKey = useDebouncedValue(baseQueryKey);
   const isDebouncing = baseQueryKey !== queryKey;
 
   // TODO(malwilley): Display error states
-  const {data, isFetching} = useQuery<string[]>({
-    // disable exhaustive deps because we want to debounce the query key above
-    // eslint-disable-next-line @tanstack/query/exhaustive-deps
+  const {data, isFetching} = useQuery({
     queryKey,
-    queryFn: () => getTagValues(...queryParams),
+    queryFn: ctx => getTagValues(...ctx.queryKey[1]),
     placeholderData: keepPreviousData,
     enabled: shouldFetchValues,
   });

static/app/views/dashboards/globalFilter/filterSelector.tsx

@@ -83,16 +83,13 @@ function FilterSelector({
     ? !fullTag.predefined && predefinedValues === null
     : true;
 
-  const baseQueryKey = useMemo(
-    () => ['global-dashboard-filters-tag-values', tag, selection, searchQuery],
-    [tag, selection, searchQuery]
-  );
-  const queryKey = useDebouncedValue(baseQueryKey);
-
-  const queryResult = useQuery<string[]>({
-    // Disable exhaustive deps because we want to debounce the query key above
-    // eslint-disable-next-line @tanstack/query/exhaustive-deps
-    queryKey,
+  const queryResult = useQuery({
+    queryKey: useDebouncedValue(
+      useMemo(
+        () => ['global-dashboard-filters-tag-values', tag, selection, searchQuery],
+        [tag, selection, searchQuery]
+      )
+    ),
     queryFn: async () => {
       const result = await searchBarData.getTagValues(tag, searchQuery);
       return result ?? [];