Tolerate policies.identify failure when dataId already provided.

src/cache/inmemory/writeToStore.ts

@@ -365,22 +365,28 @@ export class StoreWriter {
 
     // Identify the result object, even if dataId was already provided,
     // since we always need keyObject below.
-    const [id, keyObject] = policies.identify(result, {
-      selectionSet,
-      fragmentMap: context.fragmentMap,
-      storeObject: incoming,
-      readField,
-    });
+    try {
+      const [id, keyObject] = policies.identify(result, {
+        typename,
+        selectionSet,
+        fragmentMap: context.fragmentMap,
+        storeObject: incoming,
+        readField,
+      });
 
-    // If dataId was not provided, fall back to the id just generated by
-    // policies.identify.
-    dataId = dataId || id;
+      // If dataId was not provided, fall back to the id just generated by
+      // policies.identify.
+      dataId = dataId || id;
 
-    // Write any key fields that were used during identification, even if
-    // they were not mentioned in the original query.
-    if (keyObject) {
-      // TODO Reverse the order of the arguments?
-      incoming = context.merge(incoming, keyObject);
+      // Write any key fields that were used during identification, even if
+      // they were not mentioned in the original query.
+      if (keyObject) {
+        // TODO Reverse the order of the arguments?
+        incoming = context.merge(incoming, keyObject);
+      }
+    } catch (e) {
+      // If dataId was provided, tolerate failure of policies.identify.
+      if (!dataId) throw e;
     }
 
     if ("string" === typeof dataId) {