relay-runtime 0.0.0-main-9262f2ac → 0.0.0-main-668c9f2e

package/relay-runtime.min.js

@@ -1,5 +1,5 @@
 /**
- * Relay v0.0.0-main-9262f2ac
+ * Relay v0.0.0-main-668c9f2e
  *
  * Copyright (c) Facebook, Inc. and its affiliates.
  *

package/store/DataChecker.js.flow

@@ -73,12 +73,8 @@ const {
   STREAM,
   TYPE_DISCRIMINATOR,
 } = RelayConcreteNode;
-const {
-  ROOT_ID,
-  getModuleOperationKey,
-  getStorageKey,
-  getArgumentValues,
-} = RelayStoreUtils;
+const {ROOT_ID, getModuleOperationKey, getStorageKey, getArgumentValues} =
+  RelayStoreUtils;
 
 /**
  * Synchronously check whether the records required to fulfill the given
@@ -592,10 +588,8 @@ class DataChecker {
       const prevMutator = this._mutator;
       const prevRecordSourceProxy = this._recordSourceProxy;
 
-      const [
-        mutator,
-        recordSourceProxy,
-      ] = this._getMutatorAndRecordProxyForActor(actorIdentifier);
+      const [mutator, recordSourceProxy] =
+        this._getMutatorAndRecordProxyForActor(actorIdentifier);
 
       this._source = this._getSourceForActor(actorIdentifier);
       this._mutator = mutator;

package/store/OperationExecutor.js.flow

@@ -448,7 +448,8 @@ class Executor<TMutation: MutationParameters> {
         error.stack;
         throw error;
       } else {
-        const responseWithData: GraphQLResponseWithData = (response: $FlowFixMe);
+        const responseWithData: GraphQLResponseWithData =
+          (response: $FlowFixMe);
         results.push(responseWithData);
       }
     });
@@ -523,10 +524,8 @@ class Executor<TMutation: MutationParameters> {
       return;
     }
 
-    const [
-      nonIncrementalResponses,
-      incrementalResponses,
-    ] = partitionGraphQLResponses(responsesWithData);
+    const [nonIncrementalResponses, incrementalResponses] =
+      partitionGraphQLResponses(responsesWithData);
     const hasNonIncrementalResponses = nonIncrementalResponses.length > 0;
 
     // In theory this doesn't preserve the ordering of the batch.
@@ -563,9 +562,8 @@ class Executor<TMutation: MutationParameters> {
     }
 
     if (incrementalResponses.length > 0) {
-      const payloadFollowups = this._processIncrementalResponses(
-        incrementalResponses,
-      );
+      const payloadFollowups =
+        this._processIncrementalResponses(incrementalResponses);
 
       this._processPayloadFollowups(payloadFollowups);
     }
@@ -578,7 +576,8 @@ class Executor<TMutation: MutationParameters> {
           __relay_subscription_root_id: this._operation.fragment.dataID,
         };
       } else {
-        responsesWithData[0].extensions.__relay_subscription_root_id = this._operation.fragment.dataID;
+        responsesWithData[0].extensions.__relay_subscription_root_id =
+          this._operation.fragment.dataID;
       }
     }
 
@@ -677,10 +676,8 @@ class Executor<TMutation: MutationParameters> {
             if (operation == null) {
               this._processAsyncOptimisticModuleImport(followupPayload);
             } else {
-              const moduleImportOptimisticUpdates = this._processOptimisticModuleImport(
-                operation,
-                followupPayload,
-              );
+              const moduleImportOptimisticUpdates =
+                this._processOptimisticModuleImport(operation, followupPayload);
               optimisticUpdates.push(...moduleImportOptimisticUpdates);
             }
             break;
@@ -775,10 +772,8 @@ class Executor<TMutation: MutationParameters> {
         if (operation == null || this._state !== 'started') {
           return;
         }
-        const moduleImportOptimisticUpdates = this._processOptimisticModuleImport(
-          operation,
-          moduleImportPayload,
-        );
+        const moduleImportOptimisticUpdates =
+          this._processOptimisticModuleImport(operation, moduleImportPayload);
         moduleImportOptimisticUpdates.forEach(update =>
           this._getPublishQueueAndSaveActor().applyUpdate(update),
         );
@@ -1180,9 +1175,8 @@ class Executor<TMutation: MutationParameters> {
     // If there were any queued responses, process them now that placeholders
     // are in place
     if (pendingResponses != null) {
-      const payloadFollowups = this._processIncrementalResponses(
-        pendingResponses,
-      );
+      const payloadFollowups =
+        this._processIncrementalResponses(pendingResponses);
       this._processPayloadFollowups(payloadFollowups);
     }
   }
@@ -1232,10 +1226,7 @@ class Executor<TMutation: MutationParameters> {
         // but Relay records paths relative to the parent of the stream node:
         // therefore we strip the last two elements just to lookup the path
         // (the item index is used later to insert the element in the list)
-        const pathKey = path
-          .slice(0, -2)
-          .map(String)
-          .join('.');
+        const pathKey = path.slice(0, -2).map(String).join('.');
         let resultForPath = resultForLabel.get(pathKey);
         if (resultForPath == null) {
           resultForPath = {kind: 'response', responses: [incrementalResponse]};

package/store/RelayModernFragmentSpecResolver.js.flow

@@ -45,7 +45,7 @@ const warning = require('warning');
 type Props = {[key: string]: mixed, ...};
 type Resolvers = {
   [key: string]: ?(SelectorListResolver | SelectorResolver),
-  ...,
+  ...
 };
 
 /**

package/store/RelayOperationTracker.js.flow

@@ -49,9 +49,8 @@ class RelayOperationTracker {
     const newlyAffectedOwnersIdentifier = new Set();
     for (const owner of affectedOwners) {
       const ownerIdentifier = owner.identifier;
-      const pendingOperationsAffectingOwner = this._ownersToPendingOperations.get(
-        ownerIdentifier,
-      );
+      const pendingOperationsAffectingOwner =
+        this._ownersToPendingOperations.get(ownerIdentifier);
       if (pendingOperationsAffectingOwner != null) {
         // In this case the `ownerIdentifier` already affected by some operations
         // We just need to detect, is it the same operation that we already
@@ -113,9 +112,8 @@ class RelayOperationTracker {
     // and some other operations
     const updatedOwnersIdentifier = new Set();
     for (const ownerIdentifier of affectedOwnersIdentifier) {
-      const pendingOperationsAffectingOwner = this._ownersToPendingOperations.get(
-        ownerIdentifier,
-      );
+      const pendingOperationsAffectingOwner =
+        this._ownersToPendingOperations.get(ownerIdentifier);
       if (!pendingOperationsAffectingOwner) {
         continue;
       }
@@ -151,16 +149,13 @@ class RelayOperationTracker {
     this._ownersToPendingPromise.delete(ownerIdentifier);
   }
 
-  getPendingOperationsAffectingOwner(
-    owner: RequestDescriptor,
-  ): {|
+  getPendingOperationsAffectingOwner(owner: RequestDescriptor): {|
     promise: Promise<void>,
     pendingOperations: $ReadOnlyArray<RequestDescriptor>,
   |} | null {
     const ownerIdentifier = owner.identifier;
-    const pendingOperationsForOwner = this._ownersToPendingOperations.get(
-      ownerIdentifier,
-    );
+    const pendingOperationsForOwner =
+      this._ownersToPendingOperations.get(ownerIdentifier);
     if (
       pendingOperationsForOwner == null ||
       pendingOperationsForOwner.size === 0
@@ -168,9 +163,8 @@ class RelayOperationTracker {
       return null;
     }
 
-    const cachedPromiseEntry = this._ownersToPendingPromise.get(
-      ownerIdentifier,
-    );
+    const cachedPromiseEntry =
+      this._ownersToPendingPromise.get(ownerIdentifier);
     if (cachedPromiseEntry != null) {
       return {
         promise: cachedPromiseEntry.promise,

package/store/RelayPublishQueue.js.flow

@@ -307,7 +307,8 @@ class RelayPublishQueue implements PublishQueue {
       const selectorData = lookupSelector(source, selector);
       updater(recordSourceSelectorProxy, selectorData);
     }
-    const idsMarkedForInvalidation = recordSourceProxy.getIDsMarkedForInvalidation();
+    const idsMarkedForInvalidation =
+      recordSourceProxy.getIDsMarkedForInvalidation();
     this._store.publish(source, idsMarkedForInvalidation);
     return recordSourceProxy.isStoreMarkedForInvalidation();
   }
@@ -348,7 +349,8 @@ class RelayPublishQueue implements PublishQueue {
         );
         invalidatedStore =
           invalidatedStore || recordSourceProxy.isStoreMarkedForInvalidation();
-        const idsMarkedForInvalidation = recordSourceProxy.getIDsMarkedForInvalidation();
+        const idsMarkedForInvalidation =
+          recordSourceProxy.getIDsMarkedForInvalidation();
 
         this._store.publish(sink, idsMarkedForInvalidation);
       }

package/store/RelayReader.js.flow

@@ -322,8 +322,8 @@ class RelayReader {
 
             // store flags to reset after reading
             const parentIsMissingData = this._isMissingData;
-            const parentIsWithinUnmatchedTypeRefinement = this
-              ._isWithinUnmatchedTypeRefinement;
+            const parentIsWithinUnmatchedTypeRefinement =
+              this._isWithinUnmatchedTypeRefinement;
 
             const typeName = RelayModernRecord.getType(record);
             const typeID = generateTypeID(typeName);
@@ -336,7 +336,8 @@ class RelayReader {
               parentIsWithinUnmatchedTypeRefinement ||
               implementsInterface === false;
             this._traverseSelections(selection.selections, record, data);
-            this._isWithinUnmatchedTypeRefinement = parentIsWithinUnmatchedTypeRefinement;
+            this._isWithinUnmatchedTypeRefinement =
+              parentIsWithinUnmatchedTypeRefinement;
 
             if (implementsInterface === false) {
               // Type known to not implement the interface, no data expected
@@ -533,10 +534,8 @@ class RelayReader {
   ): ?mixed {
     const applicationName = field.alias ?? field.name;
     const storageKey = getStorageKey(field, this._variables);
-    const reactFlightClientResponseRecordID = RelayModernRecord.getLinkedRecordID(
-      record,
-      storageKey,
-    );
+    const reactFlightClientResponseRecordID =
+      RelayModernRecord.getLinkedRecordID(record, storageKey);
     if (reactFlightClientResponseRecordID == null) {
       data[applicationName] = reactFlightClientResponseRecordID;
       if (reactFlightClientResponseRecordID === undefined) {
@@ -757,9 +756,8 @@ class RelayReader {
       ? getArgumentValues(fragmentSpread.args, this._variables)
       : {};
     data[FRAGMENT_OWNER_KEY] = this._owner;
-    data[
-      IS_WITHIN_UNMATCHED_TYPE_REFINEMENT
-    ] = this._isWithinUnmatchedTypeRefinement;
+    data[IS_WITHIN_UNMATCHED_TYPE_REFINEMENT] =
+      this._isWithinUnmatchedTypeRefinement;
   }
 
   _createInlineDataOrResolverFragmentPointer(

package/store/RelayResponseNormalizer.js.flow

@@ -505,7 +505,11 @@ class RelayResponseNormalizer {
           this._validateConflictingFieldsWithIdenticalId(
             record,
             storageKey,
-            fieldValue,
+            // When using `treatMissingFieldsAsNull` the conflicting validation raises a false positive
+            // because the value is set using `null` but validated using `fieldValue` which at this point
+            // will be `undefined`.
+            // Setting this to `null` matches the value that we actually set to the `fieldValue`.
+            null,
           );
         }
       }
@@ -762,7 +766,8 @@ class RelayResponseNormalizer {
       reactFlightClientResponse,
     );
 
-    const reachableExecutableDefinitions: Array<ReactFlightReachableExecutableDefinitions> = [];
+    const reachableExecutableDefinitions: Array<ReactFlightReachableExecutableDefinitions> =
+      [];
     for (const query of reactFlightPayload.queries) {
       if (query.response.data != null) {
         this._followupPayloads.push({

package/subscription/requestSubscription.js.flow

@@ -69,14 +69,8 @@ function requestSubscription<TSubscriptionPayload>(
   if (subscription.params.operationKind !== 'subscription') {
     throw new Error('requestSubscription: Must use Subscription operation');
   }
-  const {
-    configs,
-    onCompleted,
-    onError,
-    onNext,
-    variables,
-    cacheConfig,
-  } = config;
+  const {configs, onCompleted, onError, onNext, variables, cacheConfig} =
+    config;
   const operation = createOperationDescriptor(
     subscription,
     variables,

package/util/getPaginationMetadata.js.flow

@@ -32,10 +32,8 @@ function getPaginationMetadata(
   paginationMetadata: ReaderPaginationMetadata,
   stream: boolean,
 |} {
-  const {
-    refetchableRequest: paginationRequest,
-    refetchMetadata,
-  } = getRefetchMetadata(fragmentNode, componentDisplayName);
+  const {refetchableRequest: paginationRequest, refetchMetadata} =
+    getRefetchMetadata(fragmentNode, componentDisplayName);
 
   const paginationMetadata = refetchMetadata.connection;
   invariant(

package/util/getPaginationVariables.js.flow

@@ -29,10 +29,8 @@ function getPaginationVariables(
   extraVariables: Variables,
   paginationMetadata: ReaderPaginationMetadata,
 ): {[string]: mixed, ...} {
-  const {
-    backward: backwardMetadata,
-    forward: forwardMetadata,
-  } = paginationMetadata;
+  const {backward: backwardMetadata, forward: forwardMetadata} =
+    paginationMetadata;
 
   if (direction === 'backward') {
     invariant(

package/util/getRefetchMetadata.js.flow

@@ -51,11 +51,10 @@ function getRefetchMetadata(
   );
 
   // handle both commonjs and es modules
-  const refetchableRequest:
-    | ConcreteRequest
-    | string = (refetchMetadata: $FlowFixMe).operation.default
-    ? (refetchMetadata: $FlowFixMe).operation.default
-    : refetchMetadata.operation;
+  const refetchableRequest: ConcreteRequest | string =
+    (refetchMetadata: $FlowFixMe).operation.default
+      ? (refetchMetadata: $FlowFixMe).operation.default
+      : refetchMetadata.operation;
   const fragmentRefPathInResponse = refetchMetadata.fragmentPathInResult;
   invariant(
     typeof refetchableRequest !== 'string',