Angular injectMutation: `onSuccess` loses type when `queryClient` is passed to `injectMutation`

[7schmiede]

Describe the bug

When the queryClient is passed in the injectMutation method, the return type of mutationFn is lost in the onSuccess method.

Your minimal, reproducible example

https://stackblitz.com/edit/stackblitz-starters-3zsk2e?file=test.ts

Steps to reproduce

import { injectMutation } from "@tanstack/angular-query-experimental";

// without queryClient 
export const foo = injectMutation(() => ({
    mutationFn: () => Promise.resolve({ foo: true }),
    onSuccess: (result) => {
        result.foo; // result: { foo: boolean; }
    }
}));

// with queryClient
export const bar = injectMutation((queryClient) => ({
    mutationFn: () => Promise.resolve({ bar: true }),
    onSuccess: (result) => {
        result.bar; // result: unknown
    }
}));

Expected behavior

Even if the queryClient is passed, the return type of the mutationFn should remain in the onSuccess method.

How often does this bug happen?

Every time

Screenshots or Videos

No response

Platform

• OS: Windows 11
• Browser: Edge Chromium
• Version: 5.55.4

Tanstack Query adapter

angular-query

TanStack Query version

v5.55.4

TypeScript version

No response

Additional context

No response

[mevans]

Yeah this is a really frustrating issue. I either have to choose between just injecting the query client at the top level, or manually typing all parameters in the callback functions

[rklfss]

Same here. Figured out, specifying the queryClient type does the trick for the moment. At least better than every method arguments.

injectMutation((client: QueryClient) => ({
}));

[ThiloAschebrock]

One could fix this by wrapping TData with NoInfer<TData> in the MutationOptions type of the core package at https://github.com/TanStack/query/blob/main/packages/query-core/src/types.ts#L1005, but that would be a breaking change for all adapters.