Why throw in onSuccess doesn't trigger onError in useQuery but DOES in useMutation?

[nvmnghia]

Example:

https://codesandbox.io/s/broken-lake-idleic?file=/src/App.js

I have a vague feeling that this is because mutate() in useMutation() is triggered inside a callback, which swallow the error.

[clicktodev]

I thought this might be related to #3862 but the problem still exists after upgrading to v4. Perhaps @TkDodo can chime in.

[TkDodo]

yeah I think this is not implemented in queries. code would be here, I'd accept a PR that fixes this:

query/packages/query-core/src/queryObserver.ts

Lines 677 to 683 in 46ad29a

	if (notifyOptions.onSuccess) {
	this.options.onSuccess?.(this.currentResult.data!)
	this.options.onSettled?.(this.currentResult.data!, null)
	} else if (notifyOptions.onError) {
	this.options.onError?.(this.currentResult.error!)
	this.options.onSettled?.(undefined, this.currentResult.error!)
	}