Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions packages/query-core/src/infiniteQueryBehavior.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import type {
InfiniteData,
InfiniteQueryOptions,
QueryFunctionContext,
QueryKey,
} from './types'

export function infiniteQueryBehavior<
Expand Down Expand Up @@ -74,15 +75,20 @@ export function infiniteQueryBehavior<
return Promise.resolve(pages)
}

const queryFnContext: Omit<QueryFunctionContext, 'signal'> = {
const queryFnContext: Omit<
QueryFunctionContext<QueryKey, unknown>,
'signal'
> = {
queryKey: context.queryKey,
pageParam: param,
meta: context.options.meta,
}

addSignalProperty(queryFnContext)

const queryFnResult = queryFn(queryFnContext as QueryFunctionContext)
const queryFnResult = queryFn(
queryFnContext as QueryFunctionContext<QueryKey, unknown>,
)

const promise = Promise.resolve(queryFnResult).then((page) =>
buildNewPages(pages, param, page, previous),
Expand Down
1 change: 0 additions & 1 deletion packages/query-core/src/query.ts
Original file line number Diff line number Diff line change
Expand Up @@ -362,7 +362,6 @@ export class Query<
// Create query function context
const queryFnContext: Omit<QueryFunctionContext<TQueryKey>, 'signal'> = {
queryKey: this.queryKey,
pageParam: undefined,
meta: this.meta,
}

Expand Down
37 changes: 33 additions & 4 deletions packages/query-core/src/queryClient.ts
Original file line number Diff line number Diff line change
Expand Up @@ -275,8 +275,15 @@ export class QueryClient {
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = never,
>(
options: FetchQueryOptions<TQueryFnData, TError, TData, TQueryKey>,
options: FetchQueryOptions<
TQueryFnData,
TError,
TData,
TQueryKey,
TPageParam
>,
): Promise<TData> {
const defaultedOptions = this.defaultQueryOptions(options)

Expand Down Expand Up @@ -308,8 +315,15 @@ export class QueryClient {
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = unknown,
>(
options: FetchInfiniteQueryOptions<TQueryFnData, TError, TData, TQueryKey>,
options: FetchInfiniteQueryOptions<
TQueryFnData,
TError,
TData,
TQueryKey,
TPageParam
>,
): Promise<InfiniteData<TData>> {
options.behavior = infiniteQueryBehavior<TQueryFnData, TError, TData>()
return this.fetchQuery(options)
Expand All @@ -320,8 +334,15 @@ export class QueryClient {
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = unknown,
>(
options: FetchInfiniteQueryOptions<TQueryFnData, TError, TData, TQueryKey>,
options: FetchInfiniteQueryOptions<
TQueryFnData,
TError,
TData,
TQueryKey,
TPageParam
>,
): Promise<void> {
return this.fetchInfiniteQuery(options).then(noop).catch(noop)
}
Expand Down Expand Up @@ -405,9 +426,17 @@ export class QueryClient {
TData = TQueryFnData,
TQueryData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = never,
>(
options?:
| QueryObserverOptions<TQueryFnData, TError, TData, TQueryData, TQueryKey>
| QueryObserverOptions<
TQueryFnData,
TError,
TData,
TQueryData,
TQueryKey,
TPageParam
>
| DefaultedQueryObserverOptions<
TQueryFnData,
TError,
Expand Down
1 change: 1 addition & 0 deletions packages/query-core/src/tests/query.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,7 @@ describe('query', () => {
expect(queryFn).toHaveBeenCalledTimes(1)
const args = queryFn.mock.calls[0]![0]
expect(args).toBeDefined()
// @ts-expect-error page param should be undefined
expect(args.pageParam).toBeUndefined()
expect(args.queryKey).toEqual(key)
expect(args.signal).toBeInstanceOf(AbortSignal)
Expand Down
10 changes: 6 additions & 4 deletions packages/query-core/src/tests/queryClient.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -565,15 +565,16 @@ describe('queryClient', () => {
pageParams: [0],
} as const

const fetchFn: QueryFunction<StrictData, StrictQueryKey> = () =>
const fetchFn: QueryFunction<StrictData, StrictQueryKey, number> = () =>
Promise.resolve(data.pages[0])

await expect(
queryClient.fetchInfiniteQuery<
StrictData,
any,
StrictData,
StrictQueryKey
StrictQueryKey,
number
>({ queryKey: key, queryFn: fetchFn, defaultPageParam: 0 }),
).resolves.toEqual(data)
})
Expand Down Expand Up @@ -603,14 +604,15 @@ describe('queryClient', () => {
type StrictQueryKey = ['strict', ...ReturnType<typeof queryKey>]
const key: StrictQueryKey = ['strict', ...queryKey()]

const fetchFn: QueryFunction<StrictData, StrictQueryKey> = () =>
const fetchFn: QueryFunction<StrictData, StrictQueryKey, number> = () =>
Promise.resolve('data')

await queryClient.prefetchInfiniteQuery<
StrictData,
any,
StrictData,
StrictQueryKey
StrictQueryKey,
number
>({ queryKey: key, queryFn: fetchFn, defaultPageParam: 0 })

const result = queryClient.getQueryData(key)
Expand Down
74 changes: 48 additions & 26 deletions packages/query-core/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,17 +22,24 @@ export type QueryKey = readonly unknown[]
export type QueryFunction<
T = unknown,
TQueryKey extends QueryKey = QueryKey,
> = (context: QueryFunctionContext<TQueryKey>) => T | Promise<T>
TPageParam = never,
> = (context: QueryFunctionContext<TQueryKey, TPageParam>) => T | Promise<T>

export interface QueryFunctionContext<
export type QueryFunctionContext<
TQueryKey extends QueryKey = QueryKey,
TPageParam = any,
> {
queryKey: TQueryKey
signal: AbortSignal
pageParam?: TPageParam
meta: QueryMeta | undefined
}
TPageParam = never,
> = [TPageParam] extends [never]
? {
queryKey: TQueryKey
signal: AbortSignal
meta: QueryMeta | undefined
}
: {
queryKey: TQueryKey
signal: AbortSignal
pageParam: TPageParam
meta: QueryMeta | undefined
}

export type InitialDataFunction<T> = () => T | undefined

Expand All @@ -50,15 +57,15 @@ export type QueryKeyHashFunction<TQueryKey extends QueryKey> = (
queryKey: TQueryKey,
) => string

export type GetPreviousPageParamFunction<TQueryFnData = unknown> = (
export type GetPreviousPageParamFunction<TPageParam, TQueryFnData = unknown> = (
firstPage: TQueryFnData,
allPages: TQueryFnData[],
) => unknown
) => TPageParam | undefined

export type GetNextPageParamFunction<TQueryFnData = unknown> = (
export type GetNextPageParamFunction<TPageParam, TQueryFnData = unknown> = (
lastPage: TQueryFnData,
allPages: TQueryFnData[],
) => unknown
) => TPageParam | undefined

export interface InfiniteData<TData> {
pages: TData[]
Expand All @@ -76,6 +83,7 @@ export interface QueryOptions<
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = never,
> {
/**
* If `false`, failed queries will not retry by default.
Expand All @@ -87,7 +95,7 @@ export interface QueryOptions<
retryDelay?: RetryDelayValue<TError>
networkMode?: NetworkMode
gcTime?: number
queryFn?: QueryFunction<TQueryFnData, TQueryKey>
queryFn?: QueryFunction<TQueryFnData, TQueryKey, TPageParam>
queryHash?: string
queryKey?: TQueryKey
queryKeyHashFn?: QueryKeyHashFunction<TQueryKey>
Expand All @@ -114,22 +122,24 @@ export interface QueryOptions<
maxPages?: number
}

export interface DefaultPageParam {
defaultPageParam: unknown
export interface DefaultPageParam<TPageParam = unknown> {
defaultPageParam: TPageParam
}

export interface InfiniteQueryOptions<TQueryFnData = unknown>
extends DefaultPageParam {
export interface InfiniteQueryOptions<
TQueryFnData = unknown,
TPageParam = unknown,
> extends DefaultPageParam<TPageParam> {
/**
* This function can be set to automatically get the previous cursor for infinite queries.
* The result will also be used to determine the value of `hasPreviousPage`.
*/
getPreviousPageParam?: GetPreviousPageParamFunction<TQueryFnData>
getPreviousPageParam?: GetPreviousPageParamFunction<TPageParam, TQueryFnData>
/**
* This function can be set to automatically get the next cursor for infinite queries.
* The result will also be used to determine the value of `hasNextPage`.
*/
getNextPageParam: GetNextPageParamFunction<TQueryFnData>
getNextPageParam: GetNextPageParamFunction<TPageParam, TQueryFnData>
}

export type ThrowErrors<
Expand All @@ -150,7 +160,14 @@ export interface QueryObserverOptions<
TData = TQueryFnData,
TQueryData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
> extends QueryOptions<TQueryFnData, TError, TQueryData, TQueryKey> {
TPageParam = never,
> extends QueryOptions<
TQueryFnData,
TError,
TQueryData,
TQueryKey,
TPageParam
> {
/**
* Set this to `false` to disable automatic refetching when the query mounts or changes query keys.
* To refetch the query, use the `refetch` method returned from the `useQuery` instance.
Expand Down Expand Up @@ -289,14 +306,16 @@ export interface InfiniteQueryObserverOptions<
TData = TQueryFnData,
TQueryData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = unknown,
> extends QueryObserverOptions<
TQueryFnData,
TError,
InfiniteData<TData>,
InfiniteData<TQueryData>,
TQueryKey
TQueryKey,
TPageParam
>,
InfiniteQueryOptions<TQueryFnData> {}
InfiniteQueryOptions<TQueryFnData, TPageParam> {}

export type DefaultedInfiniteQueryObserverOptions<
TQueryFnData = unknown,
Expand All @@ -320,8 +339,9 @@ export interface FetchQueryOptions<
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = never,
> extends WithRequired<
QueryOptions<TQueryFnData, TError, TData, TQueryKey>,
QueryOptions<TQueryFnData, TError, TData, TQueryKey, TPageParam>,
'queryKey'
> {
/**
Expand All @@ -336,13 +356,15 @@ export interface FetchInfiniteQueryOptions<
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = unknown,
> extends FetchQueryOptions<
TQueryFnData,
TError,
InfiniteData<TData>,
TQueryKey
TQueryKey,
TPageParam
>,
DefaultPageParam {}
DefaultPageParam<TPageParam> {}

export interface ResultOptions {
throwOnError?: boolean
Expand Down
4 changes: 3 additions & 1 deletion packages/react-query/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,13 +41,15 @@ export interface UseInfiniteQueryOptions<
TData = TQueryFnData,
TQueryData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = unknown,
> extends WithRequired<
InfiniteQueryObserverOptions<
TQueryFnData,
TError,
TData,
TQueryData,
TQueryKey
TQueryKey,
TPageParam
>,
'queryKey'
> {}
Expand Down
4 changes: 3 additions & 1 deletion packages/react-query/src/useInfiniteQuery.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,15 @@ export function useInfiniteQuery<
TError = RegisteredError,
TData = TQueryFnData,
TQueryKey extends QueryKey = QueryKey,
TPageParam = unknown,
>(
options: UseInfiniteQueryOptions<
TQueryFnData,
TError,
TData,
TQueryFnData,
TQueryKey
TQueryKey,
TPageParam
>,
queryClient?: QueryClient,
): UseInfiniteQueryResult<TData, TError> {
Expand Down
Loading