fix(query): prevent onQueryStarted from triggering at end-of-list

[joseph0926]

Fixes #5167

Summary

When fetchNextPage() or fetchPreviousPage() is called at end-of-list (when getNextPageParam/getPreviousPageParam returns undefined), the thunk was still executing and triggering onQueryStarted with stale cached data.

To solve the problem added an early bail-out in the thunk's condition function -> to check if the next/previous page actually exists before processing any action.

Changes

• Added direction check in buildThunks.ts condition function
• Added test case for the fix
• Updated existing test expectation (removed isFetchingPreviousPage: true when no previous page exists)

Test

• yarn test
• yarn type-tests
• yarn eslint

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 475a8e3:

Sandbox	Source
@examples-query-react/basic	Configuration
@examples-query-react/advanced	Configuration
@examples-action-listener/counter	Configuration
rtk-esm-cra	Configuration

[netlify]

✅ Deploy Preview for redux-starter-kit-docs ready!

Name	Link
🔨 Latest commit	475a8e3
🔍 Latest deploy log	https://app.netlify.com/projects/redux-starter-kit-docs/deploys/696f6186922ed80008fea6b0
😎 Deploy Preview	https://deploy-preview-5182--redux-starter-kit-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[joseph0926]

475a8e3

The tests added in the previous commit seemed slightly insufficient, so I added additional test code for possible edge cases.

For example, while only forward testing existed before, this commit adds verification for backward cases, as well as handling for skip operations and null values....

[themavik]

Gating initiate when getNextPageParam/getPreviousPageParam returns null stops the extra onQueryStarted at the tail—matches the new tests. nit: the guard only runs when cachedData.pages.length is truthy; if an infinite query ever has a zero-page cached shape you might still dispatch—probably unreachable, but worth a glance.