relay-runtime 8.0.0 → 10.0.1

package/lib/store/RelayOperationTracker.js

@@ -10,11 +10,15 @@
 // flowlint ambiguous-object-type:error
 'use strict';
 
+function _createForOfIteratorHelper(o) { if (typeof Symbol === "undefined" || o[Symbol.iterator] == null) { if (Array.isArray(o) || (o = _unsupportedIterableToArray(o))) { var i = 0; var F = function F() {}; return { s: F, n: function n() { if (i >= o.length) return { done: true }; return { done: false, value: o[i++] }; }, e: function e(_e) { throw _e; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var it, normalCompletion = true, didErr = false, err; return { s: function s() { it = o[Symbol.iterator](); }, n: function n() { var step = it.next(); normalCompletion = step.done; return step; }, e: function e(_e2) { didErr = true; err = _e2; }, f: function f() { try { if (!normalCompletion && it["return"] != null) it["return"](); } finally { if (didErr) throw err; } } }; }
+
+function _unsupportedIterableToArray(o, minLen) { if (!o) return; if (typeof o === "string") return _arrayLikeToArray(o, minLen); var n = Object.prototype.toString.call(o).slice(8, -1); if (n === "Object" && o.constructor) n = o.constructor.name; if (n === "Map" || n === "Set") return Array.from(n); if (n === "Arguments" || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(n)) return _arrayLikeToArray(o, minLen); }
+
+function _arrayLikeToArray(arr, len) { if (len == null || len > arr.length) len = arr.length; for (var i = 0, arr2 = new Array(len); i < len; i++) { arr2[i] = arr[i]; } return arr2; }
+
 var invariant = require("fbjs/lib/invariant");
 
-var RelayOperationTracker =
-/*#__PURE__*/
-function () {
+var RelayOperationTracker = /*#__PURE__*/function () {
   function RelayOperationTracker() {
     this._ownersToPendingOperations = new Map();
     this._pendingOperationsToOwners = new Map();
@@ -34,12 +38,12 @@ function () {
     }
 
     var newlyAffectedOwners = new Set();
-    var _iteratorNormalCompletion = true;
-    var _didIteratorError = false;
-    var _iteratorError = undefined;
+
+    var _iterator = _createForOfIteratorHelper(affectedOwners),
+        _step;
 
     try {
-      for (var _iterator = affectedOwners[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
+      for (_iterator.s(); !(_step = _iterator.n()).done;) {
         var owner = _step.value;
 
         var pendingOperationsAffectingOwner = this._ownersToPendingOperations.get(owner);
@@ -61,18 +65,9 @@ function () {
       } // No new owners were affected by this operation, we may stop here
 
     } catch (err) {
-      _didIteratorError = true;
-      _iteratorError = err;
+      _iterator.e(err);
     } finally {
-      try {
-        if (!_iteratorNormalCompletion && _iterator["return"] != null) {
-          _iterator["return"]();
-        }
-      } finally {
-        if (_didIteratorError) {
-          throw _iteratorError;
-        }
-      }
+      _iterator.f();
     }
 
     if (newlyAffectedOwners.size === 0) {
@@ -82,12 +77,12 @@ function () {
 
 
     var ownersAffectedByOperation = this._pendingOperationsToOwners.get(pendingOperation) || new Set();
-    var _iteratorNormalCompletion2 = true;
-    var _didIteratorError2 = false;
-    var _iteratorError2 = undefined;
+
+    var _iterator2 = _createForOfIteratorHelper(newlyAffectedOwners),
+        _step2;
 
     try {
-      for (var _iterator2 = newlyAffectedOwners[Symbol.iterator](), _step2; !(_iteratorNormalCompletion2 = (_step2 = _iterator2.next()).done); _iteratorNormalCompletion2 = true) {
+      for (_iterator2.s(); !(_step2 = _iterator2.n()).done;) {
         var _owner = _step2.value;
 
         this._resolveOwnerResolvers(_owner);
@@ -95,18 +90,9 @@ function () {
         ownersAffectedByOperation.add(_owner);
       }
     } catch (err) {
-      _didIteratorError2 = true;
-      _iteratorError2 = err;
+      _iterator2.e(err);
     } finally {
-      try {
-        if (!_iteratorNormalCompletion2 && _iterator2["return"] != null) {
-          _iterator2["return"]();
-        }
-      } finally {
-        if (_didIteratorError2) {
-          throw _iteratorError2;
-        }
-      }
+      _iterator2.f();
     }
 
     this._pendingOperationsToOwners.set(pendingOperation, ownersAffectedByOperation);
@@ -129,12 +115,12 @@ function () {
     // and some other operations
 
     var updatedOwners = new Set();
-    var _iteratorNormalCompletion3 = true;
-    var _didIteratorError3 = false;
-    var _iteratorError3 = undefined;
+
+    var _iterator3 = _createForOfIteratorHelper(affectedOwners),
+        _step3;
 
     try {
-      for (var _iterator3 = affectedOwners[Symbol.iterator](), _step3; !(_iteratorNormalCompletion3 = (_step3 = _iterator3.next()).done); _iteratorNormalCompletion3 = true) {
+      for (_iterator3.s(); !(_step3 = _iterator3.n()).done;) {
         var owner = _step3.value;
 
         var pendingOperationsAffectingOwner = this._ownersToPendingOperations.get(owner);
@@ -153,26 +139,16 @@ function () {
       } // Complete subscriptions for all owners, affected by `pendingOperation`
 
     } catch (err) {
-      _didIteratorError3 = true;
-      _iteratorError3 = err;
+      _iterator3.e(err);
     } finally {
-      try {
-        if (!_iteratorNormalCompletion3 && _iterator3["return"] != null) {
-          _iterator3["return"]();
-        }
-      } finally {
-        if (_didIteratorError3) {
-          throw _iteratorError3;
-        }
-      }
+      _iterator3.f();
     }
 
-    var _iteratorNormalCompletion4 = true;
-    var _didIteratorError4 = false;
-    var _iteratorError4 = undefined;
+    var _iterator4 = _createForOfIteratorHelper(completedOwners),
+        _step4;
 
     try {
-      for (var _iterator4 = completedOwners[Symbol.iterator](), _step4; !(_iteratorNormalCompletion4 = (_step4 = _iterator4.next()).done); _iteratorNormalCompletion4 = true) {
+      for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
         var _owner2 = _step4.value;
 
         this._resolveOwnerResolvers(_owner2);
@@ -182,44 +158,25 @@ function () {
       // are affected by other operations
 
     } catch (err) {
-      _didIteratorError4 = true;
-      _iteratorError4 = err;
+      _iterator4.e(err);
     } finally {
-      try {
-        if (!_iteratorNormalCompletion4 && _iterator4["return"] != null) {
-          _iterator4["return"]();
-        }
-      } finally {
-        if (_didIteratorError4) {
-          throw _iteratorError4;
-        }
-      }
+      _iterator4.f();
     }
 
-    var _iteratorNormalCompletion5 = true;
-    var _didIteratorError5 = false;
-    var _iteratorError5 = undefined;
+    var _iterator5 = _createForOfIteratorHelper(updatedOwners),
+        _step5;
 
     try {
-      for (var _iterator5 = updatedOwners[Symbol.iterator](), _step5; !(_iteratorNormalCompletion5 = (_step5 = _iterator5.next()).done); _iteratorNormalCompletion5 = true) {
+      for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
         var _owner3 = _step5.value;
 
         this._resolveOwnerResolvers(_owner3);
       } // Finally, remove pending operation
 
     } catch (err) {
-      _didIteratorError5 = true;
-      _iteratorError5 = err;
+      _iterator5.e(err);
     } finally {
-      try {
-        if (!_iteratorNormalCompletion5 && _iterator5["return"] != null) {
-          _iterator5["return"]();
-        }
-      } finally {
-        if (_didIteratorError5) {
-          throw _iteratorError5;
-        }
-      }
+      _iterator5.f();
     }
 
     this._pendingOperationsToOwners["delete"](pendingOperation);

package/lib/store/RelayOptimisticRecordSource.js

@@ -12,7 +12,11 @@
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 
-var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/objectSpread"));
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+
+function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); if (enumerableOnly) symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; }); keys.push.apply(keys, symbols); } return keys; }
+
+function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i] != null ? arguments[i] : {}; if (i % 2) { ownKeys(Object(source), true).forEach(function (key) { (0, _defineProperty2["default"])(target, key, source[key]); }); } else if (Object.getOwnPropertyDescriptors) { Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)); } else { ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } } return target; }
 
 var RelayRecordSource = require('./RelayRecordSource');
 
@@ -25,9 +29,7 @@ var UNPUBLISH_RECORD_SENTINEL = Object.freeze({
  * shadow the base version of the record rather than updating/replacing them.
  */
 
-var RelayOptimisticRecordSource =
-/*#__PURE__*/
-function () {
+var RelayOptimisticRecordSource = /*#__PURE__*/function () {
   function RelayOptimisticRecordSource(base) {
     this._base = base;
     this._sink = RelayRecordSource.create();
@@ -100,7 +102,7 @@ function () {
   _proto.toJSON = function toJSON() {
     var _this = this;
 
-    var merged = (0, _objectSpread2["default"])({}, this._base.toJSON());
+    var merged = _objectSpread({}, this._base.toJSON());
 
     this._sink.getRecordIDs().forEach(function (dataID) {
       var record = _this.get(dataID);

package/lib/store/RelayPublishQueue.js

@@ -10,10 +10,6 @@
 // flowlint ambiguous-object-type:error
 'use strict';
 
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-
-var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
-
 var ErrorUtils = require("fbjs/lib/ErrorUtils");
 
 var RelayReader = require('./RelayReader');
@@ -41,9 +37,7 @@ var warning = require("fbjs/lib/warning");
  *   - Executes handlers for "handle" fields.
  *   - Reverts and reapplies pending optimistic updates.
  */
-var RelayPublishQueue =
-/*#__PURE__*/
-function () {
+var RelayPublishQueue = /*#__PURE__*/function () {
   // True if the next `run()` should apply the backup and rerun all optimistic
   // updates performing a rebase.
   // Payloads to apply or Sources to publish to the store with the next `run()`.
@@ -208,11 +202,9 @@ function () {
     var payload = pendingPayload.payload,
         operation = pendingPayload.operation,
         updater = pendingPayload.updater;
-    var connectionEvents = payload.connectionEvents,
-        source = payload.source,
+    var source = payload.source,
         fieldPayloads = payload.fieldPayloads;
-    var combinedConnectionEvents = connectionEvents ? connectionEvents.slice() : [];
-    var mutator = new RelayRecordSourceMutator(this._store.getSource(), source, combinedConnectionEvents);
+    var mutator = new RelayRecordSourceMutator(this._store.getSource(), source);
     var recordSourceProxy = new RelayRecordSourceProxy(mutator, this._getDataID);
 
     if (fieldPayloads && fieldPayloads.length) {
@@ -236,10 +228,6 @@ function () {
 
     this._store.publish(source, idsMarkedForInvalidation);
 
-    if (combinedConnectionEvents.length !== 0) {
-      this._store.publishConnectionEvents_UNSTABLE(combinedConnectionEvents, true);
-    }
-
     return recordSourceProxy.isStoreMarkedForInvalidation();
   }
   /**
@@ -269,18 +257,13 @@ function () {
       } else {
         var updater = data.updater;
         var sink = RelayRecordSource.create();
-        var connectionEvents = [];
-        var mutator = new RelayRecordSourceMutator(_this2._store.getSource(), sink, connectionEvents);
+        var mutator = new RelayRecordSourceMutator(_this2._store.getSource(), sink);
         var recordSourceProxy = new RelayRecordSourceProxy(mutator, _this2._getDataID);
         ErrorUtils.applyWithGuard(updater, null, [recordSourceProxy], null, 'RelayPublishQueue:commitData');
         invalidatedStore = invalidatedStore || recordSourceProxy.isStoreMarkedForInvalidation();
         var idsMarkedForInvalidation = recordSourceProxy.getIDsMarkedForInvalidation();
 
         _this2._store.publish(sink, idsMarkedForInvalidation);
-
-        if (connectionEvents.length !== 0) {
-          _this2._store.publishConnectionEvents_UNSTABLE(connectionEvents, true);
-        }
       }
     });
 
@@ -299,8 +282,7 @@ function () {
     var _this3 = this;
 
     var sink = RelayRecordSource.create();
-    var combinedConnectionEvents = [];
-    var mutator = new RelayRecordSourceMutator(this._store.getSource(), sink, combinedConnectionEvents);
+    var mutator = new RelayRecordSourceMutator(this._store.getSource(), sink);
     var recordSourceProxy = new RelayRecordSourceProxy(mutator, this._getDataID, this._handlerProvider);
 
     var processUpdate = function processUpdate(optimisticUpdate) {
@@ -311,8 +293,7 @@ function () {
         var operation = optimisticUpdate.operation,
             payload = optimisticUpdate.payload,
             updater = optimisticUpdate.updater;
-        var connectionEvents = payload.connectionEvents,
-            source = payload.source,
+        var source = payload.source,
             fieldPayloads = payload.fieldPayloads;
         var recordSourceSelectorProxy = new RelayRecordSourceSelectorProxy(mutator, recordSourceProxy, operation.fragment);
         var selectorData;
@@ -322,10 +303,6 @@ function () {
           selectorData = lookupSelector(source, operation.fragment);
         }
 
-        if (connectionEvents) {
-          combinedConnectionEvents.push.apply(combinedConnectionEvents, (0, _toConsumableArray2["default"])(connectionEvents));
-        }
-
         if (updater) {
           ErrorUtils.applyWithGuard(updater, null, [recordSourceSelectorProxy, selectorData], null, 'RelayPublishQueue:applyUpdates');
         }
@@ -349,10 +326,6 @@ function () {
     }
 
     this._store.publish(sink);
-
-    if (combinedConnectionEvents.length !== 0) {
-      this._store.publishConnectionEvents_UNSTABLE(combinedConnectionEvents, false);
-    }
   };
 
   return RelayPublishQueue;

package/lib/store/RelayReader.js

@@ -10,7 +10,7 @@
 // flowlint ambiguous-object-type:error
 'use strict';
 
-var RelayConnection = require('./RelayConnection');
+var RelayFeatureFlags = require('../util/RelayFeatureFlags');
 
 var RelayModernRecord = require('./RelayModernRecord');
 
@@ -18,7 +18,6 @@ var invariant = require("fbjs/lib/invariant");
 
 var _require = require('../util/RelayConcreteNode'),
     CLIENT_EXTENSION = _require.CLIENT_EXTENSION,
-    CONNECTION = _require.CONNECTION,
     CONDITION = _require.CONDITION,
     DEFER = _require.DEFER,
     FRAGMENT_SPREAD = _require.FRAGMENT_SPREAD,
@@ -34,11 +33,16 @@ var _require2 = require('./RelayStoreUtils'),
     FRAGMENT_OWNER_KEY = _require2.FRAGMENT_OWNER_KEY,
     FRAGMENT_PROP_NAME_KEY = _require2.FRAGMENT_PROP_NAME_KEY,
     ID_KEY = _require2.ID_KEY,
+    IS_WITHIN_UNMATCHED_TYPE_REFINEMENT = _require2.IS_WITHIN_UNMATCHED_TYPE_REFINEMENT,
     MODULE_COMPONENT_KEY = _require2.MODULE_COMPONENT_KEY,
+    ROOT_ID = _require2.ROOT_ID,
     getArgumentValues = _require2.getArgumentValues,
     getStorageKey = _require2.getStorageKey,
     getModuleComponentKey = _require2.getModuleComponentKey;
 
+var _require3 = require('./TypeID'),
+    generateTypeID = _require3.generateTypeID;
+
 function read(recordSource, selector) {
   var reader = new RelayReader(recordSource, selector);
   return reader.read();
@@ -48,11 +52,10 @@ function read(recordSource, selector) {
  */
 
 
-var RelayReader =
-/*#__PURE__*/
-function () {
+var RelayReader = /*#__PURE__*/function () {
   function RelayReader(recordSource, selector) {
     this._isMissingData = false;
+    this._isWithinUnmatchedTypeRefinement = false;
     this._owner = selector.owner;
     this._recordSource = recordSource;
     this._seenRecords = {};
@@ -65,13 +68,61 @@ function () {
   _proto.read = function read() {
     var _this$_selector = this._selector,
         node = _this$_selector.node,
-        dataID = _this$_selector.dataID;
+        dataID = _this$_selector.dataID,
+        isWithinUnmatchedTypeRefinement = _this$_selector.isWithinUnmatchedTypeRefinement;
+    var abstractKey = node.abstractKey;
+
+    var record = this._recordSource.get(dataID); // Relay historically allowed child fragments to be read even if the root object
+    // did not match the type of the fragment: either the root object has a different
+    // concrete type than the fragment (for concrete fragments) or the root object does
+    // not conform to the interface/union for abstract fragments.
+    // For suspense purposes, however, we want to accurately compute whether any data
+    // is missing: but if the fragment type doesn't match (or a parent type didn't
+    // match), then no data is expected to be present.
+    // By default data is expected to be present unless this selector was read out
+    // from within a non-matching type refinement in a parent fragment:
+
+
+    var isDataExpectedToBePresent = !isWithinUnmatchedTypeRefinement; // If this is a concrete fragment and the concrete type of the record does not
+    // match, then no data is expected to be present.
+
+    if (isDataExpectedToBePresent && abstractKey == null && record != null) {
+      var recordType = RelayModernRecord.getType(record);
+
+      if (recordType !== node.type && dataID !== ROOT_ID) {
+        isDataExpectedToBePresent = false;
+      }
+    } // If this is an abstract fragment (and the precise refinement GK is enabled)
+    // then data is only expected to be present if the record type is known to
+    // implement the interface. If we aren't sure whether the record implements
+    // the interface, that itself constitutes "expected" data being missing.
+
+
+    if (isDataExpectedToBePresent && abstractKey != null && record != null && RelayFeatureFlags.ENABLE_PRECISE_TYPE_REFINEMENT) {
+      var _recordType = RelayModernRecord.getType(record);
+
+      var typeID = generateTypeID(_recordType);
+
+      var typeRecord = this._recordSource.get(typeID);
+
+      var implementsInterface = typeRecord != null ? RelayModernRecord.getValue(typeRecord, abstractKey) : null;
+
+      if (implementsInterface === false) {
+        // Type known to not implement the interface
+        isDataExpectedToBePresent = false;
+      } else if (implementsInterface == null) {
+        // Don't know if the type implements the interface or not
+        this._isMissingData = true;
+      }
+    }
+
+    this._isWithinUnmatchedTypeRefinement = !isDataExpectedToBePresent;
 
     var data = this._traverse(node, dataID, null);
 
     return {
       data: data,
-      isMissingData: this._isMissingData,
+      isMissingData: this._isMissingData && isDataExpectedToBePresent,
       seenRecords: this._seenRecords,
       selector: this._selector
     };
@@ -131,14 +182,53 @@ function () {
           break;
 
         case INLINE_FRAGMENT:
-          var typeName = RelayModernRecord.getType(record);
-
-          if (typeName != null && typeName === selection.type) {
-            this._traverseSelections(selection.selections, record, data);
+          {
+            var abstractKey = selection.abstractKey;
+
+            if (abstractKey == null) {
+              // concrete type refinement: only read data if the type exactly matches
+              var typeName = RelayModernRecord.getType(record);
+
+              if (typeName != null && typeName === selection.type) {
+                this._traverseSelections(selection.selections, record, data);
+              }
+            } else if (RelayFeatureFlags.ENABLE_PRECISE_TYPE_REFINEMENT) {
+              // Similar to the logic in read(): data is only expected to be present
+              // if the record is known to conform to the interface. If we don't know
+              // whether the type conforms or not, that constitutes missing data.
+              // store flags to reset after reading
+              var parentIsMissingData = this._isMissingData;
+              var parentIsWithinUnmatchedTypeRefinement = this._isWithinUnmatchedTypeRefinement;
+
+              var _typeName = RelayModernRecord.getType(record);
+
+              var typeID = generateTypeID(_typeName);
+
+              var typeRecord = this._recordSource.get(typeID);
+
+              var implementsInterface = typeRecord != null ? RelayModernRecord.getValue(typeRecord, abstractKey) : null;
+              this._isWithinUnmatchedTypeRefinement = parentIsWithinUnmatchedTypeRefinement || implementsInterface === false;
+
+              this._traverseSelections(selection.selections, record, data);
+
+              this._isWithinUnmatchedTypeRefinement = parentIsWithinUnmatchedTypeRefinement;
+
+              if (implementsInterface === false) {
+                // Type known to not implement the interface, no data expected
+                this._isMissingData = parentIsMissingData;
+              } else if (implementsInterface == null) {
+                // Don't know if the type implements the interface or not
+                this._isMissingData = true;
+              }
+            } else {
+              // legacy behavior for abstract refinements: always read even
+              // if the type doesn't conform and don't reset isMissingData
+              this._traverseSelections(selection.selections, record, data);
+            }
+
+            break;
           }
 
-          break;
-
         case FRAGMENT_SPREAD:
           this._createFragmentPointer(selection, record, data);
 
@@ -163,11 +253,6 @@ function () {
           this._isMissingData = isMissingData;
           break;
 
-        case CONNECTION:
-          this._readConnection(selection, record, data);
-
-          break;
-
         case STREAM:
           this._traverseSelections(selection.selections, record, data);
 
@@ -180,19 +265,6 @@ function () {
     }
   };
 
-  _proto._readConnection = function _readConnection(field, record, data) {
-    var parentID = RelayModernRecord.getDataID(record);
-    var connectionID = RelayConnection.createConnectionID(parentID, field.label);
-    var edgesField = field.edges;
-    var reference = {
-      variables: this._variables,
-      edgesField: edgesField,
-      id: connectionID,
-      label: field.label
-    };
-    data[RelayConnection.CONNECTION_KEY] = reference;
-  };
-
   _proto._readScalar = function _readScalar(field, record, data) {
     var _field$alias;
 
@@ -225,10 +297,7 @@ function () {
     }
 
     var prevData = data[applicationName];
-    !(prevData == null || typeof prevData === 'object') ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Expected data for field `%s` on record `%s` ' + 'to be an object, got `%s`.', applicationName, RelayModernRecord.getDataID(record), prevData) : invariant(false) : void 0;
-    /* $FlowFixMe(>=0.98.0 site=www,mobile,react_native_fb,oss) This comment
-     * suppresses an error found when Flow v0.98 was deployed. To see the error
-     * delete this comment and run Flow. */
+    !(prevData == null || typeof prevData === 'object') ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Expected data for field `%s` on record `%s` ' + 'to be an object, got `%s`.', applicationName, RelayModernRecord.getDataID(record), prevData) : invariant(false) : void 0; // $FlowFixMe[incompatible-variance]
 
     data[applicationName] = this._traverse(field, linkedID, prevData);
   };
@@ -259,10 +328,7 @@ function () {
       if (linkedID == null) {
         if (linkedID === undefined) {
           _this._isMissingData = true;
-        }
-        /* $FlowFixMe(>=0.98.0 site=www,mobile,react_native_fb,oss) This comment
-         * suppresses an error found when Flow v0.98 was deployed. To see the
-         * error delete this comment and run Flow. */
+        } // $FlowFixMe[cannot-write]
 
 
         linkedArray[nextIndex] = linkedID;
@@ -270,10 +336,8 @@ function () {
       }
 
       var prevItem = linkedArray[nextIndex];
-      !(prevItem == null || typeof prevItem === 'object') ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Expected data for field `%s` on record `%s` ' + 'to be an object, got `%s`.', applicationName, RelayModernRecord.getDataID(record), prevItem) : invariant(false) : void 0;
-      /* $FlowFixMe(>=0.98.0 site=www,mobile,react_native_fb,oss) This comment
-       * suppresses an error found when Flow v0.98 was deployed. To see the
-       * error delete this comment and run Flow. */
+      !(prevItem == null || typeof prevItem === 'object') ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Expected data for field `%s` on record `%s` ' + 'to be an object, got `%s`.', applicationName, RelayModernRecord.getDataID(record), prevItem) : invariant(false) : void 0; // $FlowFixMe[cannot-write]
+      // $FlowFixMe[incompatible-variance]
 
       linkedArray[nextIndex] = _this._traverse(field, linkedID, prevItem);
     });
@@ -325,11 +389,15 @@ function () {
 
     if (data[ID_KEY] == null) {
       data[ID_KEY] = RelayModernRecord.getDataID(record);
-    } // $FlowFixMe - writing into read-only field
+    } // $FlowFixMe[cannot-write] - writing into read-only field
 
 
     fragmentPointers[fragmentSpread.name] = fragmentSpread.args ? getArgumentValues(fragmentSpread.args, this._variables) : {};
     data[FRAGMENT_OWNER_KEY] = this._owner;
+
+    if (RelayFeatureFlags.ENABLE_PRECISE_TYPE_REFINEMENT) {
+      data[IS_WITHIN_UNMATCHED_TYPE_REFINEMENT] = this._isWithinUnmatchedTypeRefinement;
+    }
   };
 
   _proto._createInlineDataFragmentPointer = function _createInlineDataFragmentPointer(inlineDataFragmentSpread, record, data) {
@@ -347,7 +415,7 @@ function () {
 
     var inlineData = {};
 
-    this._traverseSelections(inlineDataFragmentSpread.selections, record, inlineData); // $FlowFixMe - writing into read-only field
+    this._traverseSelections(inlineDataFragmentSpread.selections, record, inlineData); // $FlowFixMe[cannot-write] - writing into read-only field
 
 
     fragmentPointers[inlineDataFragmentSpread.name] = inlineData;

package/lib/store/RelayRecordSource.js

@@ -10,22 +10,15 @@
 // flowlint ambiguous-object-type:error
 'use strict';
 
-var RelayFeatureFlags = require('../util/RelayFeatureFlags');
-
 var RelayRecordSourceMapImpl = require('./RelayRecordSourceMapImpl');
 
-var RelayRecordSourceObjectImpl = require('./RelayRecordSourceObjectImpl');
-
-var RelayRecordSource =
-/*#__PURE__*/
-function () {
+var RelayRecordSource = /*#__PURE__*/function () {
   function RelayRecordSource(records) {
     return RelayRecordSource.create(records);
   }
 
   RelayRecordSource.create = function create(records) {
-    var RecordSourceImpl = RelayFeatureFlags.USE_RECORD_SOURCE_MAP_IMPL ? RelayRecordSourceMapImpl : RelayRecordSourceObjectImpl;
-    return new RecordSourceImpl(records);
+    return new RelayRecordSourceMapImpl(records);
   };
 
   return RelayRecordSource;