Avoid destructively modifying result.data in QueryInfo#markResult.

Should help with #9293.

src/core/QueryInfo.ts

@@ -365,7 +365,7 @@ export class QueryInfo {
       | "fetchPolicy"
       | "errorPolicy">,
     cacheWriteBehavior: CacheWriteBehavior,
-  ) {
+  ): typeof result {
     const merger = new DeepMerger();
     const graphQLErrors = isNonEmptyArray(result.errors)
       ? result.errors.slice(0)
@@ -413,7 +413,10 @@ export class QueryInfo {
             });
 
             this.lastWrite = {
-              result,
+              // Make a shallow defensive copy of the result object, in case we
+              // later later modify result.data in place, since we don't want
+              // that mutation affecting the saved lastWrite.result.data.
+              result: { ...result },
               variables: options.variables,
               dmCount: destructiveMethodCounts.get(this.cache),
             };
@@ -454,7 +457,10 @@ export class QueryInfo {
                 this.lastDiff.diff.complete) {
               // Reuse data from the last good (complete) diff that we
               // received, when possible.
-              result.data = this.lastDiff.diff.result;
+              result = {
+                ...result,
+                data: this.lastDiff.diff.result,
+              };
               return;
             }
             // If the previous this.diff was incomplete, fall through to
@@ -479,14 +485,18 @@ export class QueryInfo {
           // Set without setDiff to avoid triggering a notify call, since
           // we have other ways of notifying for this result.
           this.updateLastDiff(diff, diffOptions);
-          if (diff.complete) {
-            result.data = diff.result;
-          }
+          result = {
+            ...result,
+            // TODO Improve types so we don't need this cast.
+            data: diff.result as any,
+          };
         });
       } else {
         this.lastWrite = void 0;
       }
     }
+
+    return result;
   }
 
   public markReady() {

src/core/QueryManager.ts

@@ -1148,7 +1148,12 @@ export class QueryManager<TStore> {
           // Use linkDocument rather than queryInfo.document so the
           // operation/fragments used to write the result are the same as the
           // ones used to obtain it from the link.
-          queryInfo.markResult(result, linkDocument, options, cacheWriteBehavior);
+          result = queryInfo.markResult(
+            result,
+            linkDocument,
+            options,
+            cacheWriteBehavior,
+          );
           queryInfo.markReady();
         }