relay-runtime 10.1.3 → 11.0.2

package/lib/query/fetchQueryInternal.js

@@ -14,7 +14,7 @@ var Observable = require('../network/RelayObservable');
 
 var RelayReplaySubject = require('../util/RelayReplaySubject');
 
-var invariant = require("fbjs/lib/invariant");
+var invariant = require('invariant');
 
 var WEAKMAP_SUPPORTED = typeof WeakMap === 'function';
 var requestCachesByEnvironment = WEAKMAP_SUPPORTED ? new WeakMap() : new Map();
@@ -71,8 +71,7 @@ var requestCachesByEnvironment = WEAKMAP_SUPPORTED ? new WeakMap() : new Map();
  * Cancelling requests:
  * ====================
  * If the subscription returned by subscribe is called while the
- * request is in-flight, apart from releasing retained data, the request will
- * also be cancelled.
+ * request is in-flight, the request will be cancelled.
  *
  * ```
  * const subscription = fetchQuery(...).subscribe(...);
@@ -193,7 +192,7 @@ function getActiveStatusObservableForCachedRequest(environment, requestCache, ca
 }
 /**
  * If a request is active for the given query, variables and environment,
- * this function will return a Promise that will resolve when that request has
+ * this function will return a Promise that will resolve when that request
  * stops being active (receives a final payload), and the data has been saved
  * to the store.
  * If no request is active, null will be returned

package/lib/query/fetchQuery_DEPRECATED.js

@@ -0,0 +1,39 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * 
+ * @format
+ */
+// flowlint ambiguous-object-type:error
+'use strict';
+
+var _require = require('../store/RelayModernOperationDescriptor'),
+    createOperationDescriptor = _require.createOperationDescriptor;
+
+var _require2 = require('./GraphQLTag'),
+    getRequest = _require2.getRequest;
+
+/**
+ * A helper function to fetch the results of a query. Note that results for
+ * fragment spreads are masked: fields must be explicitly listed in the query in
+ * order to be accessible in the result object.
+ */
+function fetchQuery_DEPRECATED(environment, taggedNode, variables, cacheConfig) {
+  var query = getRequest(taggedNode);
+
+  if (query.params.operationKind !== 'query') {
+    throw new Error('fetchQuery: Expected query operation');
+  }
+
+  var operation = createOperationDescriptor(query, variables, cacheConfig);
+  return environment.execute({
+    operation: operation
+  }).map(function () {
+    return environment.lookup(operation.fragment).data;
+  }).toPromise();
+}
+
+module.exports = fetchQuery_DEPRECATED;

package/lib/store/DataChecker.js

@@ -35,7 +35,7 @@ var cloneRelayScalarHandleSourceField = require('./cloneRelayScalarHandleSourceF
 
 var getOperation = require('../util/getOperation');
 
-var invariant = require("fbjs/lib/invariant");
+var invariant = require('invariant');
 
 var _require = require('./ClientID'),
     isClientID = _require.isClientID;
@@ -48,6 +48,7 @@ var _require3 = require('./TypeID'),
     generateTypeID = _require3.generateTypeID;
 
 var CONDITION = RelayConcreteNode.CONDITION,
+    CLIENT_COMPONENT = RelayConcreteNode.CLIENT_COMPONENT,
     CLIENT_EXTENSION = RelayConcreteNode.CLIENT_EXTENSION,
     DEFER = RelayConcreteNode.DEFER,
     FLIGHT_FIELD = RelayConcreteNode.FLIGHT_FIELD,
@@ -75,11 +76,11 @@ var ROOT_ID = RelayStoreUtils.ROOT_ID,
  * If all records are present, returns `true`, otherwise `false`.
  */
 
-function check(source, target, selector, handlers, operationLoader, getDataID) {
+function check(source, target, selector, handlers, operationLoader, getDataID, shouldProcessClientComponents) {
   var dataID = selector.dataID,
       node = selector.node,
       variables = selector.variables;
-  var checker = new DataChecker(source, target, variables, handlers, operationLoader, getDataID);
+  var checker = new DataChecker(source, target, variables, handlers, operationLoader, getDataID, shouldProcessClientComponents);
   return checker.check(node, dataID);
 }
 /**
@@ -88,7 +89,7 @@ function check(source, target, selector, handlers, operationLoader, getDataID) {
 
 
 var DataChecker = /*#__PURE__*/function () {
-  function DataChecker(source, target, variables, handlers, operationLoader, getDataID) {
+  function DataChecker(source, target, variables, handlers, operationLoader, getDataID, shouldProcessClientComponents) {
     var mutator = new RelayRecordSourceMutator(source, target);
     this._mostRecentlyInvalidatedAt = null;
     this._handlers = handlers;
@@ -98,6 +99,7 @@ var DataChecker = /*#__PURE__*/function () {
     this._recordWasMissing = false;
     this._source = source;
     this._variables = variables;
+    this._shouldProcessClientComponents = shouldProcessClientComponents;
   }
 
   var _proto = DataChecker.prototype;
@@ -115,7 +117,8 @@ var DataChecker = /*#__PURE__*/function () {
   };
 
   _proto._getVariableValue = function _getVariableValue(name) {
-    !this._variables.hasOwnProperty(name) ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayAsyncLoader(): Undefined variable `%s`.', name) : invariant(false) : void 0;
+    !this._variables.hasOwnProperty(name) ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayAsyncLoader(): Undefined variable `%s`.', name) : invariant(false) : void 0; // $FlowFixMe[cannot-write]
+
     return this._variables[name];
   };
 
@@ -365,7 +368,7 @@ var DataChecker = /*#__PURE__*/function () {
         // $FlowFixMe[incompatible-type]
 
         case FRAGMENT_SPREAD:
-          !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayAsyncLoader(): Unexpected ast kind `%s`.', selection.kind) : invariant(false) : void 0; // $FlowExpectedError[unreachable-code] - we need the break; for OSS linter
+          _this2._traverseSelections(selection.fragment.selections, dataID);
 
           break;
 
@@ -408,6 +411,15 @@ var DataChecker = /*#__PURE__*/function () {
 
           break;
 
+        case CLIENT_COMPONENT:
+          if (_this2._shouldProcessClientComponents === false) {
+            break;
+          }
+
+          _this2._traverseSelections(selection.fragment.selections, dataID);
+
+          break;
+
         default:
           selection;
           !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayAsyncLoader(): Unexpected ast kind `%s`.', selection.kind) : invariant(false) : void 0;
@@ -520,28 +532,28 @@ var DataChecker = /*#__PURE__*/function () {
 
     var tree = this._mutator.getValue(linkedID, RelayStoreReactFlightUtils.REACT_FLIGHT_TREE_STORAGE_KEY);
 
-    var reachableQueries = this._mutator.getValue(linkedID, RelayStoreReactFlightUtils.REACT_FLIGHT_QUERIES_STORAGE_KEY);
+    var reachableExecutableDefinitions = this._mutator.getValue(linkedID, RelayStoreReactFlightUtils.REACT_FLIGHT_EXECUTABLE_DEFINITIONS_STORAGE_KEY);
 
-    if (tree == null || !Array.isArray(reachableQueries)) {
+    if (tree == null || !Array.isArray(reachableExecutableDefinitions)) {
       this._handleMissing();
 
       return;
     }
 
     var operationLoader = this._operationLoader;
-    !(operationLoader !== null) ? process.env.NODE_ENV !== "production" ? invariant(false, 'DataChecker: Expected an operationLoader to be configured when using ' + 'React Flight.') : invariant(false) : void 0; // In Flight, the variables that are in scope for reachable queries aren't
-    // the same as what's in scope for the outer query.
+    !(operationLoader !== null) ? process.env.NODE_ENV !== "production" ? invariant(false, 'DataChecker: Expected an operationLoader to be configured when using ' + 'React Flight.') : invariant(false) : void 0; // In Flight, the variables that are in scope for reachable executable
+    // definitions aren't the same as what's in scope for the outer query.
 
     var prevVariables = this._variables; // $FlowFixMe[incompatible-cast]
 
-    var _iterator4 = (0, _createForOfIteratorHelper2["default"])(reachableQueries),
+    var _iterator4 = (0, _createForOfIteratorHelper2["default"])(reachableExecutableDefinitions),
         _step4;
 
     try {
       for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
-        var query = _step4.value;
-        this._variables = query.variables;
-        var normalizationRootNode = operationLoader.get(query.module);
+        var definition = _step4.value;
+        this._variables = definition.variables;
+        var normalizationRootNode = operationLoader.get(definition.module);
 
         if (normalizationRootNode != null) {
           var operation = getOperation(normalizationRootNode);

package/lib/store/{RelayModernQueryExecutor.js → OperationExecutor.js}

@@ -21,6 +21,8 @@ var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers
 
 var RelayError = require('../util/RelayError');
 
+var RelayFeatureFlags = require('../util/RelayFeatureFlags');
+
 var RelayModernRecord = require('./RelayModernRecord');
 
 var RelayObservable = require('../network/RelayObservable');
@@ -31,17 +33,19 @@ var RelayResponseNormalizer = require('./RelayResponseNormalizer');
 
 var getOperation = require('../util/getOperation');
 
-var invariant = require("fbjs/lib/invariant");
+var invariant = require('invariant');
 
 var stableCopy = require('../util/stableCopy');
 
 var warning = require("fbjs/lib/warning");
 
 var _require = require('./ClientID'),
-    generateClientID = _require.generateClientID;
+    generateClientID = _require.generateClientID,
+    generateUniqueClientID = _require.generateUniqueClientID;
 
 var _require2 = require('./RelayModernSelector'),
-    createNormalizationSelector = _require2.createNormalizationSelector;
+    createNormalizationSelector = _require2.createNormalizationSelector,
+    createReaderSelector = _require2.createReaderSelector;
 
 var _require3 = require('./RelayStoreUtils'),
     ROOT_TYPE = _require3.ROOT_TYPE,
@@ -76,7 +80,9 @@ var Executor = /*#__PURE__*/function () {
         treatMissingFieldsAsNull = _ref.treatMissingFieldsAsNull,
         getDataID = _ref.getDataID,
         isClientPayload = _ref.isClientPayload,
-        reactFlightPayloadDeserializer = _ref.reactFlightPayloadDeserializer;
+        reactFlightPayloadDeserializer = _ref.reactFlightPayloadDeserializer,
+        reactFlightServerErrorHandler = _ref.reactFlightServerErrorHandler,
+        shouldProcessClientComponents = _ref.shouldProcessClientComponents;
     this._getDataID = getDataID;
     this._treatMissingFieldsAsNull = treatMissingFieldsAsNull;
     this._incrementalPayloadsPending = false;
@@ -99,6 +105,9 @@ var Executor = /*#__PURE__*/function () {
     this._updater = updater;
     this._isClientPayload = isClientPayload === true;
     this._reactFlightPayloadDeserializer = reactFlightPayloadDeserializer;
+    this._reactFlightServerErrorHandler = reactFlightServerErrorHandler;
+    this._isSubscriptionOperation = this._operation.request.node.params.operationKind === 'subscription';
+    this._shouldProcessClientComponents = shouldProcessClientComponents;
     var id = this._nextSubscriptionId++;
     source.subscribe({
       complete: function complete() {
@@ -154,7 +163,7 @@ var Executor = /*#__PURE__*/function () {
       this._optimisticUpdates = null;
       optimisticUpdates.forEach(function (update) {
         return _this2._publishQueue.revertUpdate(update);
-      });
+      }); // OK: run revert on cancel
 
       this._publishQueue.run();
     }
@@ -200,7 +209,7 @@ var Executor = /*#__PURE__*/function () {
 
       default:
         this._state;
-        !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayModernQueryExecutor: invalid executor state.') : invariant(false) : void 0;
+        !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'OperationExecutor: invalid executor state.') : invariant(false) : void 0;
     }
 
     this._operationExecutions.set(this._operation.request.identifier, activeState);
@@ -333,7 +342,7 @@ var Executor = /*#__PURE__*/function () {
     var isOptimistic = ((_response$extensions = response.extensions) === null || _response$extensions === void 0 ? void 0 : _response$extensions.isOptimistic) === true;
 
     if (isOptimistic && this._state !== 'started') {
-      !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayModernQueryExecutor: optimistic payload received after server payload.') : invariant(false) : void 0;
+      !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'OperationExecutor: optimistic payload received after server payload.') : invariant(false) : void 0;
     }
 
     if (isOptimistic) {
@@ -387,7 +396,9 @@ var Executor = /*#__PURE__*/function () {
 
     var _partitionGraphQLResp = partitionGraphQLResponses(responsesWithData),
         nonIncrementalResponses = _partitionGraphQLResp[0],
-        incrementalResponses = _partitionGraphQLResp[1]; // In theory this doesn't preserve the ordering of the batch.
+        incrementalResponses = _partitionGraphQLResp[1];
+
+    var hasNonIncrementalResponses = nonIncrementalResponses.length > 0; // In theory this doesn't preserve the ordering of the batch.
     // The idea is that a batch is always:
     //  * at most one non-incremental payload
     //  * followed by zero or more incremental payloads
@@ -395,37 +406,68 @@ var Executor = /*#__PURE__*/function () {
     // with the initial payload followed by some early-to-resolve incremental
     // payloads (although, can that even happen?)
 
+    if (hasNonIncrementalResponses) {
+      var payloadFollowups = this._processResponses(nonIncrementalResponses);
 
-    if (nonIncrementalResponses.length > 0) {
-      var payloadFollowups = this._processResponses(nonIncrementalResponses); // Please note that we're passing `this._operation` to the publish
-      // queue here, which will later passed to the store (via notify)
-      // to indicate that this is an operation that caused the store to update
-
-
-      var updatedOwners = this._publishQueue.run(this._operation);
+      if (!RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+        var updatedOwners = this._publishQueue.run(this._operation);
 
-      this._updateOperationTracker(updatedOwners);
+        this._updateOperationTracker(updatedOwners);
+      }
 
       this._processPayloadFollowups(payloadFollowups);
 
-      if (this._incrementalPayloadsPending && !this._retainDisposable) {
-        this._retainDisposable = this._store.retain(this._operation);
+      if (!RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+        if (this._incrementalPayloadsPending && !this._retainDisposable) {
+          this._retainDisposable = this._store.retain(this._operation);
+        }
       }
     }
 
     if (incrementalResponses.length > 0) {
-      var _payloadFollowups = this._processIncrementalResponses(incrementalResponses); // For the incremental case, we're only handling follow-up responses
-      // for already initiated operation (and we're not passing it to
-      // the run(...) call)
+      var _payloadFollowups = this._processIncrementalResponses(incrementalResponses);
 
+      if (!RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+        // For the incremental case, we're only handling follow-up responses
+        // for already initiated operation (and we're not passing it to
+        // the run(...) call)
+        var _updatedOwners = this._publishQueue.run();
 
-      var _updatedOwners = this._publishQueue.run();
-
-      this._updateOperationTracker(_updatedOwners);
+        this._updateOperationTracker(_updatedOwners);
+      }
 
       this._processPayloadFollowups(_payloadFollowups);
     }
 
+    if (this._isSubscriptionOperation && RelayFeatureFlags.ENABLE_UNIQUE_SUBSCRIPTION_ROOT) {
+      // We attach the id to allow the `requestSubscription` to read from the store using
+      // the current id in its `onNext` callback
+      if (responsesWithData[0].extensions == null) {
+        // $FlowFixMe[cannot-write]
+        responsesWithData[0].extensions = {
+          __relay_subscription_root_id: this._operation.fragment.dataID
+        };
+      } else {
+        responsesWithData[0].extensions.__relay_subscription_root_id = this._operation.fragment.dataID;
+      }
+    }
+
+    if (RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+      // OK: run once after each new payload
+      // If we have non-incremental responses, we passing `this._operation` to
+      // the publish queue here, which will later be passed to the store (via
+      // notify) to indicate that this operation caused the store to update
+      var _updatedOwners2 = this._publishQueue.run(hasNonIncrementalResponses ? this._operation : undefined);
+
+      if (hasNonIncrementalResponses) {
+        if (this._incrementalPayloadsPending && !this._retainDisposable) {
+          this._retainDisposable = this._store.retain(this._operation);
+        }
+      }
+
+      this._updateOperationTracker(_updatedOwners2);
+    }
+
     this._sink.next(response);
   };
 
@@ -445,6 +487,8 @@ var Executor = /*#__PURE__*/function () {
         getDataID: this._getDataID,
         path: [],
         reactFlightPayloadDeserializer: this._reactFlightPayloadDeserializer,
+        reactFlightServerErrorHandler: this._reactFlightServerErrorHandler,
+        shouldProcessClientComponents: this._shouldProcessClientComponents,
         treatMissingFieldsAsNull: treatMissingFieldsAsNull
       });
       validateOptimisticResponsePayload(payload);
@@ -473,7 +517,8 @@ var Executor = /*#__PURE__*/function () {
     this._optimisticUpdates = optimisticUpdates;
     optimisticUpdates.forEach(function (update) {
       return _this6._publishQueue.applyUpdate(update);
-    });
+    }); // OK: only called on construction and when receiving an optimistic payload from network,
+    // which doesn't fall-through to the regular next() handling
 
     this._publishQueue.run();
   };
@@ -516,7 +561,9 @@ var Executor = /*#__PURE__*/function () {
       getDataID: this._getDataID,
       path: moduleImportPayload.path,
       reactFlightPayloadDeserializer: this._reactFlightPayloadDeserializer,
-      treatMissingFieldsAsNull: this._treatMissingFieldsAsNull
+      reactFlightServerErrorHandler: this._reactFlightServerErrorHandler,
+      treatMissingFieldsAsNull: this._treatMissingFieldsAsNull,
+      shouldProcessClientComponents: this._shouldProcessClientComponents
     });
   };
 
@@ -553,11 +600,12 @@ var Executor = /*#__PURE__*/function () {
       });
 
       if (_this7._optimisticUpdates == null) {
-        process.env.NODE_ENV !== "production" ? warning(false, 'RelayModernQueryExecutor: Unexpected ModuleImport optimistic ' + 'update in operation %s.' + _this7._operation.request.node.params.name) : void 0;
+        process.env.NODE_ENV !== "production" ? warning(false, 'OperationExecutor: Unexpected ModuleImport optimistic ' + 'update in operation %s.' + _this7._operation.request.node.params.name) : void 0;
       } else {
         var _this$_optimisticUpda;
 
-        (_this$_optimisticUpda = _this7._optimisticUpdates).push.apply(_this$_optimisticUpda, (0, _toConsumableArray2["default"])(moduleImportOptimisticUpdates));
+        (_this$_optimisticUpda = _this7._optimisticUpdates).push.apply(_this$_optimisticUpda, (0, _toConsumableArray2["default"])(moduleImportOptimisticUpdates)); // OK: always have to run() after an module import resolves async
+
 
         _this7._publishQueue.run();
       }
@@ -586,7 +634,9 @@ var Executor = /*#__PURE__*/function () {
         getDataID: _this8._getDataID,
         path: [],
         reactFlightPayloadDeserializer: _this8._reactFlightPayloadDeserializer,
-        treatMissingFieldsAsNull: _this8._treatMissingFieldsAsNull
+        reactFlightServerErrorHandler: _this8._reactFlightServerErrorHandler,
+        treatMissingFieldsAsNull: _this8._treatMissingFieldsAsNull,
+        shouldProcessClientComponents: _this8._shouldProcessClientComponents
       });
 
       _this8._publishQueue.commitPayload(_this8._operation, relayPayload, _this8._updater);
@@ -652,9 +702,11 @@ var Executor = /*#__PURE__*/function () {
           });
 
           if (relayPayloads.length > 0) {
-            var updatedOwners = _this9._publishQueue.run();
+            if (!RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+              var updatedOwners = _this9._publishQueue.run();
 
-            _this9._updateOperationTracker(updatedOwners);
+              _this9._updateOperationTracker(updatedOwners);
+            }
 
             _this9._processPayloadFollowups(relayPayloads);
           }
@@ -664,15 +716,22 @@ var Executor = /*#__PURE__*/function () {
   };
 
   _proto._maybeCompleteSubscriptionOperationTracking = function _maybeCompleteSubscriptionOperationTracking() {
-    var isSubscriptionOperation = this._operation.request.node.params.operationKind === 'subscription';
-
-    if (!isSubscriptionOperation) {
+    if (!this._isSubscriptionOperation) {
       return;
     }
 
     if (this._pendingModulePayloadsCount === 0 && this._incrementalPayloadsPending === false) {
       this._completeOperationTracker();
     }
+
+    if (RelayFeatureFlags.ENABLE_UNIQUE_SUBSCRIPTION_ROOT) {
+      var nextID = generateUniqueClientID();
+      this._operation = {
+        request: this._operation.request,
+        fragment: createReaderSelector(this._operation.fragment.node, nextID, this._operation.fragment.variables, this._operation.fragment.owner),
+        root: createNormalizationSelector(this._operation.root.node, nextID, this._operation.root.variables)
+      };
+    }
   }
   /**
    * Processes a ModuleImportPayload, asynchronously resolving the normalization
@@ -716,7 +775,12 @@ var Executor = /*#__PURE__*/function () {
       })).map(function (operation) {
         if (operation != null) {
           _this10._schedule(function () {
-            _this10._handleModuleImportPayload(moduleImportPayload, getOperation(operation));
+            _this10._handleModuleImportPayload(moduleImportPayload, getOperation(operation)); // OK: always have to run after an async module import resolves
+
+
+            var updatedOwners = _this10._publishQueue.run();
+
+            _this10._updateOperationTracker(updatedOwners);
           });
         }
       }).subscribe({
@@ -742,9 +806,11 @@ var Executor = /*#__PURE__*/function () {
 
     this._publishQueue.commitPayload(this._operation, relayPayload);
 
-    var updatedOwners = this._publishQueue.run();
+    if (!RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+      var updatedOwners = this._publishQueue.run();
 
-    this._updateOperationTracker(updatedOwners);
+      this._updateOperationTracker(updatedOwners);
+    }
 
     this._processPayloadFollowups([relayPayload]);
   }
@@ -843,9 +909,11 @@ var Executor = /*#__PURE__*/function () {
     if (pendingResponses != null) {
       var payloadFollowups = this._processIncrementalResponses(pendingResponses);
 
-      var updatedOwners = this._publishQueue.run();
+      if (!RelayFeatureFlags.ENABLE_BATCHED_STORE_UPDATES) {
+        var updatedOwners = this._publishQueue.run();
 
-      this._updateOperationTracker(updatedOwners);
+        this._updateOperationTracker(updatedOwners);
+      }
 
       this._processPayloadFollowups(payloadFollowups);
     }
@@ -929,7 +997,9 @@ var Executor = /*#__PURE__*/function () {
       getDataID: this._getDataID,
       path: placeholder.path,
       reactFlightPayloadDeserializer: this._reactFlightPayloadDeserializer,
-      treatMissingFieldsAsNull: this._treatMissingFieldsAsNull
+      reactFlightServerErrorHandler: this._reactFlightServerErrorHandler,
+      treatMissingFieldsAsNull: this._treatMissingFieldsAsNull,
+      shouldProcessClientComponents: this._shouldProcessClientComponents
     });
 
     this._publishQueue.commitPayload(this._operation, relayPayload); // Load the version of the parent record from which this incremental data
@@ -1081,7 +1151,9 @@ var Executor = /*#__PURE__*/function () {
       getDataID: this._getDataID,
       path: [].concat((0, _toConsumableArray2["default"])(normalizationPath), [responseKey, String(itemIndex)]),
       reactFlightPayloadDeserializer: this._reactFlightPayloadDeserializer,
-      treatMissingFieldsAsNull: this._treatMissingFieldsAsNull
+      reactFlightServerErrorHandler: this._reactFlightServerErrorHandler,
+      treatMissingFieldsAsNull: this._treatMissingFieldsAsNull,
+      shouldProcessClientComponents: this._shouldProcessClientComponents
     });
     return {
       fieldPayloads: fieldPayloads,
@@ -1094,15 +1166,13 @@ var Executor = /*#__PURE__*/function () {
   };
 
   _proto._updateOperationTracker = function _updateOperationTracker(updatedOwners) {
-    if (this._operationTracker != null && updatedOwners != null && updatedOwners.length > 0) {
+    if (updatedOwners != null && updatedOwners.length > 0) {
       this._operationTracker.update(this._operation.request, new Set(updatedOwners));
     }
   };
 
   _proto._completeOperationTracker = function _completeOperationTracker() {
-    if (this._operationTracker != null) {
-      this._operationTracker.complete(this._operation.request);
-    }
+    this._operationTracker.complete(this._operation.request);
   };
 
   return Executor;
@@ -1117,7 +1187,7 @@ function partitionGraphQLResponses(responses) {
           path = response.path;
 
       if (label == null || path == null) {
-        !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayModernQueryExecutor: invalid incremental payload, expected ' + '`path` and `label` to either both be null/undefined, or ' + '`path` to be an `Array<string | number>` and `label` to be a ' + '`string`.') : invariant(false) : void 0;
+        !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'OperationExecutor: invalid incremental payload, expected ' + '`path` and `label` to either both be null/undefined, or ' + '`path` to be an `Array<string | number>` and `label` to be a ' + '`string`.') : invariant(false) : void 0;
       }
 
       incrementalResponses.push({
@@ -1157,7 +1227,7 @@ function validateOptimisticResponsePayload(payload) {
   var incrementalPlaceholders = payload.incrementalPlaceholders;
 
   if (incrementalPlaceholders != null && incrementalPlaceholders.length !== 0) {
-    !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayModernQueryExecutor: optimistic responses cannot be returned ' + 'for operations that use incremental data delivery (@defer, ' + '@stream, and @stream_connection).') : invariant(false) : void 0;
+    !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'OperationExecutor: optimistic responses cannot be returned ' + 'for operations that use incremental data delivery (@defer, ' + '@stream, and @stream_connection).') : invariant(false) : void 0;
   }
 }
 

package/lib/store/RelayConcreteVariables.js

@@ -14,7 +14,7 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 
 var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread2"));
 
-var invariant = require("fbjs/lib/invariant");
+var invariant = require('invariant');
 
 /**
  * Determines the variables that are in scope for a fragment given the variables
@@ -28,7 +28,8 @@ function getFragmentVariables(fragment, rootVariables, argumentVariables) {
   fragment.argumentDefinitions.forEach(function (definition) {
     if (argumentVariables.hasOwnProperty(definition.name)) {
       return;
-    }
+    } // $FlowFixMe[cannot-spread-interface]
+
 
     variables = variables || (0, _objectSpread2["default"])({}, argumentVariables);
 
@@ -48,9 +49,12 @@ function getFragmentVariables(fragment, rootVariables, argumentVariables) {
            * RelayStoreUtils.getStableVariableValue() that variable keys are all
            * present.
            */
+          // $FlowFixMe[incompatible-use]
           variables[definition.name] = undefined;
           break;
-        }
+        } // $FlowFixMe[incompatible-use]
+        // $FlowFixMe[cannot-write]
+
 
         variables[definition.name] = rootVariables[definition.name];
         break;
@@ -73,7 +77,7 @@ function getFragmentVariables(fragment, rootVariables, argumentVariables) {
 function getOperationVariables(operation, variables) {
   var operationVariables = {};
   operation.argumentDefinitions.forEach(function (def) {
-    var value = def.defaultValue;
+    var value = def.defaultValue; // $FlowFixMe[cannot-write]
 
     if (variables[def.name] != null) {
       value = variables[def.name];