RFC: add "merge" functionality

src/query/core/buildSlice.ts

@@ -35,7 +35,11 @@ import {
   QueryThunkArg,
   ThunkResult,
 } from './buildThunks'
-import { AssertTagTypes, EndpointDefinitions } from '../endpointDefinitions'
+import {
+  AssertTagTypes,
+  EndpointDefinitions,
+  QueryDefinition,
+} from '../endpointDefinitions'
 import { applyPatches, Patch } from 'immer'
 import { onFocus, onFocusLost, onOffline, onOnline } from './setupListeners'
 import {
@@ -130,10 +134,14 @@ export function buildSlice({
             meta.arg.queryCacheKey,
             (substate) => {
               if (substate.requestId !== meta.requestId) return
+              const { merge = (x: any) => x } = definitions[
+                meta.arg.endpointName
+              ] as QueryDefinition<any, any, any, any>
+
               substate.status = QueryStatus.fulfilled
               substate.data = copyWithStructuralSharing(
                 substate.data,
-                payload.result
+                merge(payload.result, substate.data)
               )
               delete substate.error
               substate.fulfilledTimeStamp = payload.fulfilledTimeStamp

src/query/endpointDefinitions.ts

@@ -147,6 +147,17 @@ export interface QueryExtraOptions<
    * Not to be used. A query should not invalidate tags in the cache.
    */
   invalidatesTags?: never
+  /**
+   * Can be provided to merge the current cache value into the new cache value.
+   *
+   * Useful if you don't want a new request to completely override the current cache value,
+   * maybe because you have manually updated it from another source and don't want those
+   * updates to get lost.
+   */
+  merge?(
+    newCacheValue: ResultType,
+    currentCacheValue: ResultType | undefined
+  ): ResultType
   /** @deprecated please use `onQuery` instead */
   onStart?(arg: QueryArg, queryApi: QueryApi<ReducerPath, any>): void
   /** @deprecated please use `onQuery` instead */