[react-query] Remove empty variables from useQuery output

[nandorojo]

Imagine I have this query:

query PopularAssets {
  assets {
    bars(timeRange: day) {
      price
    }
  }
}

This generates this React Query hook:

export const usePopularAssetsQuery = <
  TData = PopularAssetsQuery,
  TError = unknown
>(
  variables?: PopularAssetsQueryVariables,
  options?: UseQueryOptions<PopularAssetsQuery, TError, TData>
) => useQuery<PopularAssetsQuery, TError, TData>(...)

However, since there are no possible query variables, the first argument should be options directly. I expect this output:

export const usePopularAssetsQuery = <
  TData = PopularAssetsQuery,
  TError = unknown
>(
  options?: UseQueryOptions<PopularAssetsQuery, TError, TData>
) => useQuery<PopularAssetsQuery, TError, TData>(...)

This is what changed:

export const usePopularAssetsQuery = <
  TData = PopularAssetsQuery,
  TError = unknown
>(
-  variables?: PopularAssetsQueryVariables,
  options?: UseQueryOptions<PopularAssetsQuery, TError, TData>
) => useQuery<PopularAssetsQuery, TError, TData>(...)

Any thoughts on how I could do this? Is there an option for ignoreEmptyVariables or something?

Why

Currently, to consume this hook and use options, I need to do this:

usePopularAssets(undefined, {
  onSuccess() {
    // ...
  }
})

And it's not that enjoyable to pass undefined.