fix(graphcache): fix restoring variables on mutation operations (#3046)

urql-graphql · Mar 14, 2023 · d41fac8 · d41fac8
1 parent c8b0623
commit d41fac8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/.changeset/fifty-jeans-invite.md b/.changeset/fifty-jeans-invite.md
@@ -0,0 +1,5 @@
+---
+'@urql/exchange-graphcache': patch
+---
+
+Restore variables correctly on mutations
diff --git a/exchanges/graphcache/src/cacheExchange.ts b/exchanges/graphcache/src/cacheExchange.ts
@@ -153,7 +153,7 @@ export const cacheExchange = <C extends Partial<CacheExchangeOpts>>(
             )
           : operation.variables,
       },
-      operation.context
+      { ...operation.context, originalVariables: operation.variables }
     );
   };
 
@@ -208,6 +208,11 @@ export const cacheExchange = <C extends Partial<CacheExchangeOpts>>(
       : result.operation;
 
     if (operation.kind === 'mutation') {
+      if (result.operation.context.originalVariables) {
+        operation.variables = result.operation.context.originalVariables;
+        delete result.operation.context.originalVariables;
+      }
+
       // Collect previous dependencies that have been written for optimistic updates
       const dependencies = optimisticKeysToDependencies.get(operation.key);
       collectPendingOperations(pendingOperations, dependencies);