Solid Suspense is triggered on data access after checking isSuccess when loaded by ensureQueryData

[Syfaro]

Describe the bug

Hi there, I'm running into a weird issue where a suspense is being triggered even after checking that isSuccess is true, but only if the data was loaded from an ensureQueryData call (I've only tested this from a createFileRoute loader, maybe that's somehow related?).

It seems like somehow the Solid resource ends up behind the query. I'm not very familiar with the TanStack or Solid code, but in digging through the debugger in Chrome I found this suspect situation. The getter on the query identifies that the data exists (and while I'm here I can see the isSuccess set to true) and tries to load the latest data from the Solid resource:

But the Solid resource doesn't believe the fetcher was resolved yet, leading to it triggering a read which adds to the suspense context instead of just returning the value:

Your minimal, reproducible example

https://stackblitz.com/edit/tanstack-router-lwma9xac?file=src%2Froutes%2Fposts.tsx

Steps to reproduce

1. Visit the posts page (/posts) in the sandbox, and observe that the wasMounted signal remains false.
2. Uncomment the loader section (in src/routes/posts.tsx), and observe that the Suspense fallback is mounted.

Expected behavior

I expect that the query data getter would behave the same no matter how the data was initially loaded.

How often does this bug happen?

Every time

Screenshots or Videos

Platform

• OS: macOS
• Browser: Safari 26.1, Chrome 142.0

Tanstack Query adapter

solid-query

TanStack Query version

5.90.12

TypeScript version

5.9.3

Additional context

I realize this seems like a tiny bug, and in almost any case I don't think it'd even be perceptible, but unfortunately the suspense is a significant problem for me because it breaks the iframe that Square's web payments SDK generates.

[coderabbitai]

📝 CodeRabbit Plan Mode

Generate an implementation plan and agent prompts for this issue.

• Create Plan

Examples

• Example 1
• Example 2

🔗 Similar & Duplicate Issues

Related Issues

• TanStack/query#9883
• TanStack/query#9851
• TanStack/query#9440
• TanStack/router#5065

🔗 Related PRs

#9813 - docs(solid-query): fix incorrect Vue.js code example in Suspense guide [merged] | docs/framework/solid/guides/suspense.md:
"""

AI-generated summary of changes

• Replaces a Vue-based Suspense example (defineComponent with setup and suspense) with a SolidJS functional component.
• Changes code block language from Vue single-file component pattern to TSX; removes defineComponent, setup, and suspense awaiting.
• Reframes data access from returning a data object after awaiting suspense to a functional component that calls useQuery with a queryKey and queryFn and renders query.data directly inside the component.
• Eliminates the explicit suspense() control flow and relies on Suspense boundary behavior to suspend until data is ready, with a note in comments about accessing query.data within the Suspense boundary.
• Overall rendering pattern shifts from a Vue lifecycle suspend-and-return approach to a SolidJS reactive component that returns JSX and relies on Suspense for suspension.

Alterations to the declarations of exported or public entities

• No changes to exported/public entity signatures detected.

Manifest File Analyzer

"""
TanStack/router#5638 - test(solid-start): manual suspense boundaries transition on navigation [merged] | e2e/react-start/basic-react-query/src/routeTree.gen.ts:
"""

AI-generated summary of changes

The change introduces a new route called /suspense-transition by importing SuspenseTransitionRoute from ./routes/suspense-transition. It adds this route consistently across multiple route data structures: it updates individual route definitions (SuspenseTransitionRoute) by setting the id, path, and getParentRoute properties; it expands the FileRoutesByFullPath, FileRoutesByTo, FileRoutesById, fileRoutesByFullPath, fileRoutesByTo, and fileRoutesById interfaces to include the new route; it adds an entry for /suspense-transition into the FileRouteTypes's to, fullPaths, and id union types; it updates the RootRouteChildren to include the new route; it modifies the module declaration for '@tanstack/react-router' to include the route in FileRoutesByPath and RootRouteChildren; and it updates the rootRouteChildren and the overall routeTree to embed the new route within the route hierarchy. These changes collectively make the /suspense-transition route recognized throughout the routing configuration, allowing it to be used in path-based, to-based, and ID-based routing lookups, and ensuring it is integrated into the route tree structure.

Alterations to the declarations of exported or public entities

• Added SuspenseTransitionRoute as a new route constant with its id, path, and getParentRoute properties.
• Expanded FileRoutesByFullPath interface to include the key '/suspense-transition'.
• Expanded FileRoutesByTo interface to include the key '/suspense-transition'.
• Expanded FileRoutesById interface to include the key '/suspense-transition'.
• Extended FileRouteTypes union types (fullPaths, to, and id) to include '/suspense-transition'.
• Updated RootRouteChildren interface and instance to include SuspenseTransitionRoute.
• Modified the module '@tanstack/react-router''s FileRoutesByPath and RootRouteChildren interfaces to include /suspense-transition.

Estimated code review effort: High
"""

---

e2e/react-start/basic-react-query/src/routes/suspense-transition.tsx:
"""

AI-generated summary of changes

Adds a new React route component file that demonstrates a suspense-enabled data flow with React Query. Introduces a route at /suspense-transition with SSR disabled. Defines a doubleQueryOptions helper to simulate an async query that resolves after a 500ms delay. The route component renders an Increase link that increments a query parameter n, and a Result section that executes a useQuery using doubleQueryOptions(search.n). The Result is wrapped in a Suspense boundary with a fallback, and displays the current n and the computed double value. Public export includes Route, created via createFileRoute with search validation and SSR setting.

• Method added: Route exported from file src/routes/suspense-transition.tsx via createFileRoute('/suspense-transition')({ ... })

Manifest File Analyzer

Estimated code review effort: Medium
"""

---

e2e/react-start/basic-react-query/tests/app.spec.ts:
"""

AI-generated summary of changes

Adds a new end-to-end test named "Manual Suspense boundaries should transition on navigation" to the Playwright test suite. The test navigates to /suspense-transition, asserts initial content values, triggers a navigation-change by clicking an increase button, verifies that during the transition the old content remains visible and the suspense fallback is not shown, and finally asserts that the new content values (n-value becomes 2 and double-value becomes 4) are rendered after the transition.

Alterations to the declarations of exported or public entities

• Test added: test('Manual Suspense boundaries should transition on navigation', async ({ page }) => { ... }) in file e2e/react-start/basic-react-query/tests/app.spec.ts

Estimated code review effort: Medium
"""

---

e2e/solid-start/basic-solid-query/src/routeTree.gen.ts:
"""

AI-generated summary of changes

• Introduces a new SuspenseTransitionRoute by importing its route and creating SuspenseTransitionRoute via update with id "/suspense-transition", path "/suspense-transition", and parent rootRouteImport.
• Adds the new route to the top-level route map and all associated type declarations:
  • FileRoutesByFullPath now includes '/suspense-transition': typeof SuspenseTransitionRoute
  • FileRoutesByTo now includes '/suspense-transition': typeof SuspenseTransitionRoute
  • FileRoutesById now includes '/suspense-transition': typeof SuspenseTransitionRoute
  • FileRouteTypes includes the suspense-transition segment in fullPaths, to, and id, and adds the route to fileRoutesById
  • RootRouteChildren now includes SuspenseTransitionRoute: typeof SuspenseTransitionRoute
  • TanStack Solid Router module augmentation now adds a '/suspense-transition' entry with id, path, fullPath, preLoaderRoute, and parentRoute
  • routeTree construction now registers SuspenseTransitionRoute among rootRouteChildren
• Keeps existing routes and structure intact; only augments with a new route and all its public typings and mappings.

Alterations to the declarations of exported or public entities

• Added: Constant SuspenseTransitionRoute
  • SuspenseTransitionRoute = SuspenseTransitionRouteImport.update({ id: '/suspense-transition', path: '/suspense-transition', getParentRoute: () => rootRouteImport } as any)
• Updated: FileRoutesByFullPath
  • Added: '/suspense-transition': typeof SuspenseTransitionRoute
• Updated: FileRoutesByTo
  • Added: '/suspense-transition': typeof SuspenseTransitionRoute
• Updated: FileRoutesById
  • Added: '/suspense-transition': typeof SuspenseTransitionRoute
• Updated: FileRouteTypes
  • Added to fileRoutesByFullPath: '/suspense-transition': typeof SuspenseTransitionRoute
  • Added to fileRoutesByTo: '/suspense-transition': typeof SuspenseTransitionRoute
  • Added to id union: '/suspense-transition'
  • Added to fullPaths: '/suspense-transition'
• Updated: RootRouteChildren
  • Added: SuspenseTransitionRoute: typeof SuspenseTransitionRoute
• Updated: declare module '@tanstack/solid-router'
  • In FileRoutesByPath: added entry for '/suspense-transition' with id '/suspense-transition', path '/suspense-transition', fullPath '/suspense-transition', preLoaderRoute typeof SuspenseTransitionRouteImport, parentRoute typeof rootRouteImport
• Updated: routeTree root
  • Added: SuspenseTransitionRoute to rootRouteChildren

</diff_to_summarize>

Estimated code review effort: Medium
"""

---

e2e/solid-start/basic-solid-query/src/routes/suspense-transition.tsx:
"""

AI-generated summary of changes

Adds a new route at /suspense-transition using Solid Router. The route is defined with SSR disabled and a validateSearch that defaults n to 1. The component renders a UI with an Increase link to mutate the search param n, and a Result subcomponent that subscribes to the route search, triggers a double query via tanstack/solid-query with an artificial 500ms delay, and displays n and the computed double value inside a Suspense boundary that shows a loading fallback until data is ready. The doubleQuery uses placeholderData to mirror previous data and ensures the transition is observable. The UI includes test-data attributes for targeted testing.

Alterations to the declarations of exported or public entities

• Exported signature added: Route in file src/routes/suspense-transition.tsx → export const Route = createFileRoute('/suspense-transition')({ validateSearch: (search: { n?: number }) => ({ n: search.n ?? 1 }), component: SuspenseTransitionComponent, ssr: false, })

Manifest File Analyzer

Estimated code review effort: High
"""

---

e2e/solid-start/basic-solid-query/tests/app.spec.ts:
"""

AI-generated summary of changes

• Added three new tests to e2e/solid-start/basic-solid-query/tests/app.spec.ts:

  • Two tests are skipped: "Navigating nested layouts" and "Navigating to a not-found route".
  • One active test: "Manual Suspense boundaries should transition on navigation" that navigates to /suspense-transition, asserts initial values, triggers navigation via a button, verifies absence of suspense fallback during transition, verifies content visibility during transition, and final content after transition.

• No changes to existing exported/public declarations.

Alterations to the declarations of exported or public entities

• None

<manifest_file>
package.json
</manifest_file

Estimated code review effort: Medium
"""
TanStack/router#5647 - test(solid-start): fix dev server for suspense transition [merged] | e2e/solid-start/basic-solid-query/src/routes/posts.$postId.tsx:
"""

AI-generated summary of changes

• Added a new exported PostErrorComponent at the top of the file, returning an ErrorComponent with the provided error.
• Removed the previously duplicated PostErrorComponent definition that appeared after Route and before PostComponent, eliminating the second export of the same component.
• No changes to the Route configuration semantics or the PostComponent implementation; the errorComponent reference now points to the top-defined PostErrorComponent.

Alterations to the declarations of exported or public entities

• Added: export function PostErrorComponent({ error }: { error: any }) in file e2e/solid-start/basic-solid-query/src/routes/posts.$postId.tsx
• Removed: export function PostErrorComponent({ error }: { error: any }) in file e2e/solid-start/basic-solid-query/src/routes/posts.$postId.tsx (the later duplicate)

Estimated code review effort: Medium
"""
TanStack/router#5802 - test(router, start): query navigation transitions [merged] | e2e/react-router/basic-react-query-file-based/src/routes/transition/count/query.tsx:
"""

AI-generated summary of changes

• Change in data shape returned by the queryFn: from returning a number (n * 2) to returning an object { n, double: n * 2 }.

• Update in UI rendering: instead of displaying doubleQuery.data directly as a primitive, display its properties via optional chaining (doubleQuery.data?.n and doubleQuery.data?.double).

• Adjusted imports: removed useMemo from React imports; Suspense now imported alone.

• No change to route loading logic besides consuming the updated data shape; loader still uses doubleQueryOptions(n) and ensures query data, now aligning with the new object structure.

• Public/exported declarations: None modified (no public API changes; Route signature remains the same).

Estimated code review effort: Medium
"""

---

e2e/react-router/basic-react-query-file-based/tests/transition.spec.ts:
"""

AI-generated summary of changes

• Test name changed from 'react-query transitions keep previous data during navigation' to 'transitions/count/query should keep old values visible during navigation'.
• Introduced bodyTexts array to replace bodySnapshots for collecting body text during polling.
• Replaced interval variable name: interval -> pollInterval, and corresponding logic to push to bodyTexts instead of bodySnapshots.
• After first click on increase-button:
  • Added immediate assertions that n-value and double-value reflect expected values with short timeouts.
• Removed old interval clearing logic and replaced with awaited assertions verifying updated values after navigation completes, including subsequent timeouts.
• Added explicit assertions for subsequent click counts (2 clicks) and for additional navigation steps:
  • Validate n-value and double-value after each stage (n: 2/double: 4; n: 4/double: 8; etc.).
• Inserted new wait periods (200 ms) between actions to align with transition timing.
• Replaced a previous final snapshot check with a transition-safety check:
  • Collects body texts during polling, checks for any remaining "Loading..." text, and throws an explicit error if found.
• Removed earlier loading-snapshot-based assertion (sawLoading) and replaced with hasLoadingText check.
• Added final clearInterval call for pollInterval.
• Overall, logic now verifies transition visibility and absence of loading text during navigation, with explicit, granular assertions after each interaction.

Alterations to the declarations of exported or public entities

• None

Estimated code review effort: Medium
"""

---

e2e/solid-router/basic-solid-query-file-based/src/routes/transition/count/query.tsx:
"""

AI-generated summary of changes

• The query function now returns an object { n, double } instead of a plain number (previously n * 2).
• Rendering logic is updated to read data?.n and data?.double from the query result, replacing data being a raw number with separate fields.

Estimated code review effort: Medium
"""

---

e2e/solid-router/basic-solid-query-file-based/tests/transition.spec.ts:
"""

AI-generated summary of changes

• Renames and reframes the test: the test description changes from "solid-query transitions keep previous data during navigation" to "transitions/count/query should keep old values visible during navigation".
• Reworks data collection during the test: local collection of body text changes from bodySnapshots (string array) to bodyTexts, and the polling interval variable is renamed from interval to pollInterval.
• Replaces the original polling and cleanup: the interval cleanup is adjusted (clearInterval removed earlier, and pollInterval is cleared later); the polling logic now uses bodyTexts for assertions.
• Adds stepwise interaction and assertions: introduces multiple sequential clicks on the increase button with corresponding assertions of n-value and double-value after each step (1, 2, and 3 clicks), including updated timeouts and additional wait periods.
• Augments navigation validation: replaces a simple loading check with a more explicit assertion that "Loading..." does not appear during navigation; includes a throw on failure.
• Maintains existing initial checks but tightens timing: retains initial checks for n-value and double-value, with updated timeout formatting; expands verification to ensure values progress correctly through multiple transitions.
• Minor formatting consistency: aligns timeout literals from 2_000 to 2000 in several places.

Alterations to the declarations of exported or public entities

• Method signature updated: test in module containing e2e/solid-router/basic-solid-query-file-based/tests/transition.spec.ts from
  test('solid-query transitions keep previous data during navigation', async ({ page }) => { ... })
  to
  test('transitions/count/query should keep old values visible during navigation', async ({ page }) => { ... })

Estimated code review effort: High
"""

---

e2e/react-start/basic-react-query/package.json:
"""

AI-generated summary of changes

Adds a new public dependency "zod" with version "^4.1.12" to the dependencies section of e2e/react-start/basic-react-query/package.json; maintains existing "vite" entry, with formatting adjusted to accommodate the new dependency.

Alterations to the declarations of exported or public entities

• Dependency added: zod with version ^4.1.12 in e2e/react-start/basic-react-query/package.json under dependencies

Manifest File Analyzer

Estimated code review effort: Medium
"""

---

e2e/react-start/basic-react-query/src/routeTree.gen.ts:
"""

AI-generated summary of changes

Adds a new route: /transition/count/query. Introduces TransitionCountQueryRoute as a public route, wired to rootRouteImport, and registers it across all route maps and type declarations:

• Import: new TransitionCountQueryRouteImport.
• Route definition: TransitionCountQueryRoute with id and path '/transition/count/query' and parent route root.
• Public maps updated: FileRoutesByFullPath, FileRoutesByTo, FileRoutesById, and FileRouteTypes (including fullPath, to, id, and fileRoutesByFullPath mappings).
• Exposed in RootRouteChildren.
• React Router module augmentation updated to include '/transition/count/query' with corresponding preLoaderRoute and parentRoute.

• Alterations to the declarations of exported or public entities

• New public/exported entity:

  • TransitionCountQueryRoute added: initialization via TransitionCountQueryRouteImport.update({ id: '/transition/count/query', path: '/transition/count/query', getParentRoute: () => rootRouteImport }).

• Updated public route maps to include the new route:

  • FileRoutesByFullPath: add '/transition/count/query': typeof TransitionCountQueryRoute
  • FileRoutesByTo: add '/transition/count/query': typeof TransitionCountQueryRoute
  • FileRoutesById: add '/transition/count/query': typeof TransitionCountQueryRoute
  • FileRouteTypes:
    • fileRoutesByFullPath includes '/transition/count/query'
    • fullPaths includes '/transition/count/query'
    • fileRoutesByTo includes '/transition/count/query'
    • to includes '/transition/count/query'
    • fileRoutesById includes '/transition/count/query'
    • id includes '/transition/count/query'
  • RootRouteChildren: add TransitionCountQueryRoute: typeof TransitionCountQueryRoute

• React Router module augmentation updates:

  • FileRoutesByPath gains entry for '/transition/count/query' with id, path, fullPath, preLoaderRoute, parentRoute.

• Root route tree construction:

  • rootRouteChildren includes TransitionCountQueryRoute: TransitionCountQueryRoute

Manifest File Analyzer

Estimated code review effort: Medium
"""

---

e2e/react-start/basic-react-query/src/routes/transition/count/query.tsx:
"""

AI-generated summary of changes

Introduces a new React route file at src/routes/transition/count/query.tsx that wires up a Suspense-enabled page using TanStack React Router and React Query. It defines a search schema (n with default 1) and a reusable query option creator doubleQueryOptions(n) that fetches a simple payload after a 1-second delay and returns { n, double: n*2 }. The route's loader ensures data for the current n via queryClient.ensureQueryData(doubleQueryOptions(n)). The page component uses Route.useSearch to read n, executes useQuery with the same options, and renders a link to increment n (preserving the search shape) plus displays the current n and its double. All rendering is wrapped in a Suspense with a "Loading..." fallback.

Alterations to the declarations of exported or public entities

• Exported constant added: Route in src/routes/transition/count/query.tsx

Manifest File Analyzer

Estimated code review effort: Medium
"""

---

e2e/react-start/basic-react-query/tests/transition.spec.ts:
"""

AI-generated summary of changes

Adds a Playwright end-to-end test at tests/transition.spec.ts to verify that during navigation transitions of a count/double value UI, previous values remain visible until new ones load, and that no "Loading..." text appears. The test navigates to /transition/count/query, asserts initial values, then performs a series of increments (1, 2, 3 clicks) with intermediate explicit timeouts and final checks of updated values. It also streams periodic body text content to a buffer and asserts absence of "Loading..." in that captured text during navigation. If "Loading..." is detected, the test fails with a descriptive error. The test relies on test IDs: n-value, double-value, increase-button, and uses a pollInterval to capture transient DOM content to verify transitional behavior.

Alterations to the declarations of exported or public entities

• None

Manifest File Analyzer

Estimated code review effort: High
"""

---

e2e/solid-start/basic-solid-query/package.json:
"""

AI-generated summary of changes

• Added a new dependency "zod": "^4.1.12" to the dependencies in e2e/solid-start/basic-solid-query/package.json.
• Adjusted formatting by adding a trailing comma after the existing "vite": "^7.1.7" entry and placing "zod" on a new line.

Alterations to the declarations of exported or public entities

• (none)

Estimated code review effort: Medium
"""

---

e2e/solid-start/basic-solid-query/src/routeTree.gen.ts:
"""

AI-generated summary of changes

• Adds a new route: transition/count/query by importing TransitionCountQueryRoute and creating a TransitionCountQueryRoute with id, path, and parentRoot. Updates the route tree to include this new route in all route maps and type declarations.
• Extends public route typings to expose the new route in:
  • FileRoutesByFullPath, FileRoutesByTo, FileRoutesById
  • FileRouteTypes (fullPath unions, to unions, id unions)
  • RootRouteChildren
  • TanStack Solid Router module declarations (path-based routing map)
• Registers the new route under rootRouteChildren and expands the root route configuration to recognize the new child with correct preLoaderRoute and parentRoute.

Alterations to the declarations of exported or public entities

• Added public route: TransitionCountQueryRoute (new route constant) in e2e/solid-start/basic-solid-query/src/routeTree.gen.ts
• Updated interface FileRoutesByFullPath:
  • Added entry: '/transition/count/query': typeof TransitionCountQueryRoute
• Updated interface FileRoutesByTo:
  • Added entry: '/transition/count/query': typeof TransitionCountQueryRoute
• Updated interface FileRoutesById:
  • Added entry: '/transition/count/query': typeof TransitionCountQueryRoute
• Updated interface FileRouteTypes:
  • fileRoutesByFullPath: include '/transition/count/query'
  • fileRoutesByTo: include '/transition/count/query'
  • fileRoutesById: include '/transition/count/query'
  • fullPaths, to, and id unions: include '/transition/count/query'
• Updated interface RootRouteChildren:
  • Added TransitionCountQueryRoute: typeof TransitionCountQueryRoute
• Updated TanStack router module augmentation:
  • Added path-based route entry for '/transition/count/query' with id, path, fullPath, preLoaderRoute, parentRoute
• Updated rootRouteChildren object:
  • Added TransitionCountQueryRoute: TransitionCountQueryRoute

Manifest file to determine:

Estimated code review effort: Medium
"""

---

e2e/solid-start/basic-solid-query/src/routes/transition/count/query.tsx:
"""

AI-generated summary of changes

Adds a new SolidJS route file that defines a /transition/count/query page. It introduces:

• A search schema validating a numeric query parameter n with default 1.
• A data-fetching option generator doubleQueryOptions that fetches { n, double: n*2 } after a 1s delay, with placeholderData wired to existing data.
• A file-route export Route that validates location.search, preloads data via loader by ensuring query data for doubleQueryOptions(n), and renders TransitionPage.
• A TransitionPage component that reads n from the route search, uses useQuery with doubleQueryOptions(search().n), and renders a Suspense boundary with an Increase link that updates the n parameter by incrementing it in the URL. It then displays n and double values from the query result.

Alterations to the declarations of exported or public entities

• Exported entity added: Route constant in e2e/solid-start/basic-solid-query/src/routes/transition/count/query.tsx
  Signature (approximate):
  export const Route = createFileRoute('/transition/count/query')({ validateSearch: searchSchema, loader: ({ context: { queryClient }, location }) => { const { n } = searchSchema.parse(location.search); return queryClient.ensureQueryData(doubleQueryOptions(n)) }, component: TransitionPage })

Manifest File Analyzer

Estimated code review effort: Medium
"""

---

e2e/solid-start/basic-solid-query/tests/transition.spec.ts:
"""

AI-generated summary of changes

Adds a new end-to-end Playwright test file at e2e/solid-start/basic-solid-query/tests/transition.spec.ts that verifies transition behavior during navigation. The test navigates to a transition page, asserts initial numeric values (n-value, double-value), then repeatedly clicks an increase button to increment state across multiple steps (1, 2, 4, 7), asserting expected text updates with precise timeouts. It also polls the page body content periodically to capture live text, and finally checks that no "Loading..." indicators appeared during transitions, ensuring old values remain visible while new ones load.

Alterations to the declarations of exported or public entities

• None

Estimated code review effort: Medium
"""
#9886 - fix: Prevent infinite render loops when useSuspenseQueries has duplicate queryKeys [merged] | .changeset/nine-shoes-tap.md:
"""

AI-generated summary of changes

• Adds a changeset entry bumping two TanStack packages to patch releases: @tanstack/react-query and @tanstack/query-core. Includes a fix description: prevents infinite render loops when useSuspenseQueries has duplicate queryKeys.

Alterations to the declarations of exported or public entities

• None

Estimated code review effort: Medium
"""

---

packages/query-core/src/queriesObserver.ts:
"""

AI-generated summary of changes

• prevObserversMap now maps queryHash to an array of observers instead of a single observer, built by grouping existing observers.
• For each query, the code computes defaultedOptions and then selects an observer by shifting from the grouped array; if none exist, it creates a new QueryObserver.
• Each query in the observers list is now paired with a defaultedQueryOptions and an observer (reused or newly created), removing prior branching based on a single matching observer.
• The initial construction of prevObserversMap switches from a direct map of observer.options.queryHash to an array-backed grouping approach.
• Minor structural changes accompany the new grouping logic, with unchanged external behavior aside from observer reuse/creation paths.

Alterations to the declarations of exported or public entities

• No exported or public API declarations were modified.

Manifest File Analyzer

Estimated code review effort: Medium

Manifest file usage

"""

---

packages/react-query/src/tests/useSuspenseQueries.test.tsx:
"""

AI-generated summary of changes

• Adds a new test "should handle duplicate query keys without infinite loops" to verify that using duplicate query keys in useSuspenseQueries with different selectors does not cause an infinite render loop. The test defines a shared key with two queries (name and age), renders via Suspense, and asserts:
  • initial suspend and resolution occur once
  • subsequent timer advancement resolves with expected data
  • renderCount remains below 10, indicating no runaway renders
• The test is inserted in two places within the same file, duplicating the scenario.

Alterations to the declarations of exported or public entities

• None

Manifest File Analyzer

"""

👤 Suggested Assignees

• birkskyum
• teidesu
• MarkSFrancis
• longzheng
• ldhwaddell

---

🧪 Issue enrichment is currently in early access.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false