@tanstack/query-core 4.13.0 → 4.13.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tanstack/query-core",
-  "version": "4.13.0",
+  "version": "4.13.4",
   "description": "The framework agnostic core that powers TanStack Query",
   "author": "tannerlinsley",
   "license": "MIT",

package/src/queryObserver.ts

@@ -517,11 +517,6 @@ export class QueryObserver<
         if (options.select && typeof placeholderData !== 'undefined') {
           try {
             placeholderData = options.select(placeholderData)
-            placeholderData = replaceData(
-              prevResult?.data,
-              placeholderData,
-              options,
-            )
             this.selectError = null
           } catch (selectError) {
             if (process.env.NODE_ENV !== 'production') {
@@ -534,7 +529,7 @@ export class QueryObserver<
 
       if (typeof placeholderData !== 'undefined') {
         status = 'success'
-        data = placeholderData as TData
+        data = replaceData(prevResult?.data, placeholderData, options) as TData
         isPlaceholderData = true
       }
     }

package/src/tests/queryObserver.test.tsx

@@ -505,6 +505,24 @@ describe('queryObserver', () => {
     expect(results[1]).toMatchObject({ status: 'success', data: 'data' })
   })
 
+  test('should structurally share placeholder data', async () => {
+    const key = queryKey()
+    const observer = new QueryObserver(queryClient, {
+      queryKey: key,
+      enabled: false,
+      queryFn: () => 'data',
+      placeholderData: {},
+    })
+
+    const firstData = observer.getCurrentResult().data
+
+    observer.setOptions({ placeholderData: {} })
+
+    const secondData = observer.getCurrentResult().data
+
+    expect(firstData).toBe(secondData)
+  })
+
   test('the retrier should not throw an error when reject if the retrier is already resolved', async () => {
     const key = queryKey()
     let count = 0