Skip to content

[pull] main from TanStack:main#462

Merged
pull[bot] merged 3 commits intomanudeli:mainfrom
TanStack:main
May 3, 2026
Merged

[pull] main from TanStack:main#462
pull[bot] merged 3 commits intomanudeli:mainfrom
TanStack:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented May 3, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

neefrehman and others added 3 commits May 3, 2026 14:34
Follow-up to #10510. That PR removed NoInfer from all three persister
generics to fix TQueryFnData inference when the companion queryFn
declares a parameter (#7842). Keeping NoInfer on TQueryKey preserves
that fix while preventing the persister slot from widening TQueryKey
inference.

Without NoInfer on TQueryKey, the persister slot contributes to
TQueryKey inference. When Register.queryKey is augmented to a narrowed
constraint, TQueryKey widens to that constraint instead of the literal
passed to queryKey. Wrappers that brand their return with
DataTag<TQueryKey, ...> then produce a brand on the wider type, which
a plain literal tuple can no longer satisfy in contravariant positions
(vi.mocked(...).mockReturnValue, typed variable assignments, etc.).

TQueryFnData still participates in inference, so #10510's positive and
negative type tests continue to pass.

Co-authored-by: Wonsuk Choi <sukvvon@gmail.com>
fix: remove typo in QueryOptions#retry JSDoc
@pull pull Bot locked and limited conversation to collaborators May 3, 2026
@pull pull Bot added the ⤵️ pull label May 3, 2026
@pull pull Bot merged commit 14926b7 into manudeli:main May 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants