relay-runtime 10.0.1 → 10.1.0

package/lib/store/RelayModernStore.js

@@ -14,9 +14,9 @@ var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefau
 
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 
-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 _createForOfIteratorHelper(o, allowArrayLike) { var it; if (typeof Symbol === "undefined" || o[Symbol.iterator] == null) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; 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 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 _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(o); 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; }
 
@@ -32,6 +32,8 @@ var RelayReader = require('./RelayReader');
 
 var RelayReferenceMarker = require('./RelayReferenceMarker');
 
+var RelayStoreReactFlightUtils = require('./RelayStoreReactFlightUtils');
+
 var RelayStoreUtils = require('./RelayStoreUtils');
 
 var deepFreeze = require('../util/deepFreeze');
@@ -42,6 +44,8 @@ var hasOverlappingIDs = require('./hasOverlappingIDs');
 
 var invariant = require("fbjs/lib/invariant");
 
+var isEmptyObject = require('../util/isEmptyObject');
+
 var recycleNodesInto = require('../util/recycleNodesInto');
 
 var resolveImmediate = require('../util/resolveImmediate');
@@ -186,7 +190,7 @@ var RelayModernStore = /*#__PURE__*/function () {
         if (rootEntryIsStale) {
           _this2._roots["delete"](id);
 
-          _this2._scheduleGC();
+          _this2.scheduleGC();
         } else {
           _this2._releaseBuffer.push(id); // If the release buffer is now over-full, remove the least-recently
           // added entry and schedule a GC. Note that all items in the release
@@ -198,7 +202,7 @@ var RelayModernStore = /*#__PURE__*/function () {
 
             _this2._roots["delete"](_id);
 
-            _this2._scheduleGC();
+            _this2.scheduleGC();
           }
         }
       }
@@ -266,9 +270,10 @@ var RelayModernStore = /*#__PURE__*/function () {
 
     var source = this.getSource();
     var updatedOwners = [];
+    var hasUpdatedRecords = !isEmptyObject(this._updatedRecordIDs);
 
     this._subscriptions.forEach(function (subscription) {
-      var owner = _this3._updateSubscription(source, subscription);
+      var owner = _this3._updateSubscription(source, subscription, hasUpdatedRecords);
 
       if (owner != null) {
         updatedOwners.push(owner);
@@ -383,7 +388,7 @@ var RelayModernStore = /*#__PURE__*/function () {
         _this5._gcHoldCounter--;
 
         if (_this5._gcHoldCounter === 0 && _this5._shouldScheduleGC) {
-          _this5._scheduleGC();
+          _this5.scheduleGC();
 
           _this5._shouldScheduleGC = false;
         }
@@ -406,12 +411,12 @@ var RelayModernStore = /*#__PURE__*/function () {
   // latest update, or null if it was not affected.
   ;
 
-  _proto._updateSubscription = function _updateSubscription(source, subscription) {
+  _proto._updateSubscription = function _updateSubscription(source, subscription, hasUpdatedRecords) {
     var backup = subscription.backup,
         callback = subscription.callback,
         snapshot = subscription.snapshot,
         stale = subscription.stale;
-    var hasOverlappingUpdates = hasOverlappingIDs(snapshot.seenRecords, this._updatedRecordIDs);
+    var hasOverlappingUpdates = hasUpdatedRecords && hasOverlappingIDs(snapshot.seenRecords, this._updatedRecordIDs);
 
     if (!stale && !hasOverlappingUpdates) {
       return;
@@ -423,7 +428,8 @@ var RelayModernStore = /*#__PURE__*/function () {
       data: nextData,
       isMissingData: nextSnapshot.isMissingData,
       seenRecords: nextSnapshot.seenRecords,
-      selector: nextSnapshot.selector
+      selector: nextSnapshot.selector,
+      missingRequiredFields: nextSnapshot.missingRequiredFields
     };
 
     if (process.env.NODE_ENV !== "production") {
@@ -582,7 +588,7 @@ var RelayModernStore = /*#__PURE__*/function () {
     this._optimisticSource = null;
 
     if (this._shouldScheduleGC) {
-      this._scheduleGC();
+      this.scheduleGC();
     }
 
     this._subscriptions.forEach(function (subscription) {
@@ -598,7 +604,8 @@ var RelayModernStore = /*#__PURE__*/function () {
           data: subscription.snapshot.data,
           isMissingData: backup.isMissingData,
           seenRecords: backup.seenRecords,
-          selector: backup.selector
+          selector: backup.selector,
+          missingRequiredFields: backup.missingRequiredFields
         };
       } else {
         subscription.stale = true;
@@ -606,7 +613,7 @@ var RelayModernStore = /*#__PURE__*/function () {
     });
   };
 
-  _proto._scheduleGC = function _scheduleGC() {
+  _proto.scheduleGC = function scheduleGC() {
     if (this._gcHoldCounter > 0) {
       this._shouldScheduleGC = true;
       return;
@@ -705,7 +712,7 @@ function initializeRecordSource(target) {
 /**
  * Updates the target with information from source, also updating a mapping of
  * which records in the target were changed as a result.
- * Additionally, will marc records as invalidated at the current write epoch
+ * Additionally, will mark records as invalidated at the current write epoch
  * given the set of record ids marked as stale in this update.
  */
 
@@ -767,7 +774,11 @@ function updateTargetFromSource(target, source, currentWriteEpoch, idsMarkedForI
     }
 
     if (sourceRecord && targetRecord) {
-      var nextRecord = RelayModernRecord.update(targetRecord, sourceRecord);
+      // ReactFlightClientResponses are lazy and only materialize when readRoot
+      // is called when we read the field, so if the record is a Flight field
+      // we always use the new record's data regardless of whether
+      // it actually changed. Let React take care of reconciliation instead.
+      var nextRecord = RelayModernRecord.getType(targetRecord) === RelayStoreReactFlightUtils.REACT_FLIGHT_TYPE_NAME ? sourceRecord : RelayModernRecord.update(targetRecord, sourceRecord);
 
       if (nextRecord !== targetRecord) {
         // Prevent mutation of a record from outside the store.

package/lib/store/RelayOperationTracker.js

@@ -10,9 +10,9 @@
 // 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 _createForOfIteratorHelper(o, allowArrayLike) { var it; if (typeof Symbol === "undefined" || o[Symbol.iterator] == null) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; 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 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 _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(o); 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; }
 

package/lib/store/RelayPublishQueue.js

@@ -146,7 +146,7 @@ var RelayPublishQueue = /*#__PURE__*/function () {
 
   _proto.run = function run(sourceOperation) {
     if (process.env.NODE_ENV !== "production") {
-      process.env.NODE_ENV !== "production" ? warning(this._isRunning !== true, 'A store update was detected within another store update. Please ' + 'make sure new store updates aren’t being executed within an ' + 'updater function for a different update.') : void 0;
+      process.env.NODE_ENV !== "production" ? warning(this._isRunning !== true, 'A store update was detected within another store update. Please ' + "make sure new store updates aren't being executed within an " + 'updater function for a different update.') : void 0;
       this._isRunning = true;
     }
 

package/lib/store/RelayReader.js

@@ -20,28 +20,33 @@ var _require = require('../util/RelayConcreteNode'),
     CLIENT_EXTENSION = _require.CLIENT_EXTENSION,
     CONDITION = _require.CONDITION,
     DEFER = _require.DEFER,
+    FLIGHT_FIELD = _require.FLIGHT_FIELD,
     FRAGMENT_SPREAD = _require.FRAGMENT_SPREAD,
     INLINE_DATA_FRAGMENT_SPREAD = _require.INLINE_DATA_FRAGMENT_SPREAD,
     INLINE_FRAGMENT = _require.INLINE_FRAGMENT,
     LINKED_FIELD = _require.LINKED_FIELD,
     MODULE_IMPORT = _require.MODULE_IMPORT,
+    REQUIRED_FIELD = _require.REQUIRED_FIELD,
     SCALAR_FIELD = _require.SCALAR_FIELD,
     STREAM = _require.STREAM;
 
-var _require2 = require('./RelayStoreUtils'),
-    FRAGMENTS_KEY = _require2.FRAGMENTS_KEY,
-    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;
+var _require2 = require('./RelayStoreReactFlightUtils'),
+    getReactFlightClientResponse = _require2.getReactFlightClientResponse;
+
+var _require3 = require('./RelayStoreUtils'),
+    FRAGMENTS_KEY = _require3.FRAGMENTS_KEY,
+    FRAGMENT_OWNER_KEY = _require3.FRAGMENT_OWNER_KEY,
+    FRAGMENT_PROP_NAME_KEY = _require3.FRAGMENT_PROP_NAME_KEY,
+    ID_KEY = _require3.ID_KEY,
+    IS_WITHIN_UNMATCHED_TYPE_REFINEMENT = _require3.IS_WITHIN_UNMATCHED_TYPE_REFINEMENT,
+    MODULE_COMPONENT_KEY = _require3.MODULE_COMPONENT_KEY,
+    ROOT_ID = _require3.ROOT_ID,
+    getArgumentValues = _require3.getArgumentValues,
+    getStorageKey = _require3.getStorageKey,
+    getModuleComponentKey = _require3.getModuleComponentKey;
+
+var _require4 = require('./TypeID'),
+    generateTypeID = _require4.generateTypeID;
 
 function read(recordSource, selector) {
   var reader = new RelayReader(recordSource, selector);
@@ -56,6 +61,7 @@ var RelayReader = /*#__PURE__*/function () {
   function RelayReader(recordSource, selector) {
     this._isMissingData = false;
     this._isWithinUnmatchedTypeRefinement = false;
+    this._missingRequiredFields = null;
     this._owner = selector.owner;
     this._recordSource = recordSource;
     this._seenRecords = {};
@@ -124,7 +130,8 @@ var RelayReader = /*#__PURE__*/function () {
       data: data,
       isMissingData: this._isMissingData && isDataExpectedToBePresent,
       seenRecords: this._seenRecords,
-      selector: this._selector
+      selector: this._selector,
+      missingRequiredFields: this._missingRequiredFields
     };
   };
 
@@ -143,9 +150,9 @@ var RelayReader = /*#__PURE__*/function () {
 
     var data = prevData || {};
 
-    this._traverseSelections(node.selections, record, data);
+    var hadRequiredData = this._traverseSelections(node.selections, record, data);
 
-    return data;
+    return hadRequiredData ? data : null;
   };
 
   _proto._getVariableValue = function _getVariableValue(name) {
@@ -153,11 +160,76 @@ var RelayReader = /*#__PURE__*/function () {
     return this._variables[name];
   };
 
-  _proto._traverseSelections = function _traverseSelections(selections, record, data) {
+  _proto._maybeReportUnexpectedNull = function _maybeReportUnexpectedNull(fieldPath, action, record) {
+    var _this$_missingRequire;
+
+    if (((_this$_missingRequire = this._missingRequiredFields) === null || _this$_missingRequire === void 0 ? void 0 : _this$_missingRequire.action) === 'THROW') {
+      // Chained @required directives may cause a parent `@required(action:
+      // THROW)` field to become null, so the first missing field we
+      // encounter is likely to be the root cause of the error.
+      return;
+    }
+
+    var owner = this._selector.node.name;
+
+    switch (action) {
+      case 'THROW':
+        this._missingRequiredFields = {
+          action: action,
+          field: {
+            path: fieldPath,
+            owner: owner
+          }
+        };
+        return;
+
+      case 'LOG':
+        if (this._missingRequiredFields == null) {
+          this._missingRequiredFields = {
+            action: action,
+            fields: []
+          };
+        }
+
+        this._missingRequiredFields.fields.push({
+          path: fieldPath,
+          owner: owner
+        });
+
+        return;
+
+      default:
+        action;
+    }
+  };
+
+  _proto._traverseSelections = function _traverseSelections(selections, record, data)
+  /* had all expected data */
+  {
     for (var i = 0; i < selections.length; i++) {
       var selection = selections[i];
 
       switch (selection.kind) {
+        case REQUIRED_FIELD:
+          !RelayFeatureFlags.ENABLE_REQUIRED_DIRECTIVES ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Encountered a `@required` directive at path "%s" in `%s` without the `ENABLE_REQUIRED_DIRECTIVES` feature flag enabled.', selection.path, this._selector.node.name) : invariant(false) : void 0;
+
+          var fieldValue = this._readRequiredField(selection, record, data);
+
+          if (fieldValue == null) {
+            var action = selection.action;
+
+            if (action !== 'NONE') {
+              this._maybeReportUnexpectedNull(selection.path, action, record);
+            } // We are going to throw, or our parent is going to get nulled out.
+            // Either way, sibling values are going to be ignored, so we can
+            // bail early here as an optimization.
+
+
+            return false;
+          }
+
+          break;
+
         case SCALAR_FIELD:
           this._readScalar(selection, record, data);
 
@@ -176,7 +248,11 @@ var RelayReader = /*#__PURE__*/function () {
           var conditionValue = this._getVariableValue(selection.condition);
 
           if (conditionValue === selection.passingValue) {
-            this._traverseSelections(selection.selections, record, data);
+            var hasExpectedData = this._traverseSelections(selection.selections, record, data);
+
+            if (!hasExpectedData) {
+              return false;
+            }
           }
 
           break;
@@ -190,7 +266,11 @@ var RelayReader = /*#__PURE__*/function () {
               var typeName = RelayModernRecord.getType(record);
 
               if (typeName != null && typeName === selection.type) {
-                this._traverseSelections(selection.selections, record, data);
+                var _hasExpectedData = this._traverseSelections(selection.selections, record, data);
+
+                if (!_hasExpectedData) {
+                  return false;
+                }
               }
             } else if (RelayFeatureFlags.ENABLE_PRECISE_TYPE_REFINEMENT) {
               // Similar to the logic in read(): data is only expected to be present
@@ -246,15 +326,37 @@ var RelayReader = /*#__PURE__*/function () {
 
         case DEFER:
         case CLIENT_EXTENSION:
-          var isMissingData = this._isMissingData;
+          {
+            var isMissingData = this._isMissingData;
 
-          this._traverseSelections(selection.selections, record, data);
+            var _hasExpectedData2 = this._traverseSelections(selection.selections, record, data);
 
-          this._isMissingData = isMissingData;
-          break;
+            this._isMissingData = isMissingData;
+
+            if (!_hasExpectedData2) {
+              return false;
+            }
+
+            break;
+          }
 
         case STREAM:
-          this._traverseSelections(selection.selections, record, data);
+          {
+            var _hasExpectedData3 = this._traverseSelections(selection.selections, record, data);
+
+            if (!_hasExpectedData3) {
+              return false;
+            }
+
+            break;
+          }
+
+        case FLIGHT_FIELD:
+          if (RelayFeatureFlags.ENABLE_REACT_FLIGHT_COMPONENT_FIELD) {
+            this._readFlightField(selection, record, data);
+          } else {
+            throw new Error('Flight fields are not yet supported.');
+          }
 
           break;
 
@@ -263,13 +365,69 @@ var RelayReader = /*#__PURE__*/function () {
           !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Unexpected ast kind `%s`.', selection.kind) : invariant(false) : void 0;
       }
     }
+
+    return true;
   };
 
-  _proto._readScalar = function _readScalar(field, record, data) {
+  _proto._readRequiredField = function _readRequiredField(selection, record, data) {
+    switch (selection.field.kind) {
+      case SCALAR_FIELD:
+        return this._readScalar(selection.field, record, data);
+
+      case LINKED_FIELD:
+        if (selection.field.plural) {
+          return this._readPluralLink(selection.field, record, data);
+        } else {
+          return this._readLink(selection.field, record, data);
+        }
+
+      default:
+        selection.field.kind;
+        !false ? process.env.NODE_ENV !== "production" ? invariant(false, 'RelayReader(): Unexpected ast kind `%s`.', selection.kind) : invariant(false) : void 0;
+    }
+  };
+
+  _proto._readFlightField = function _readFlightField(field, record, data) {
     var _field$alias;
 
     var applicationName = (_field$alias = field.alias) !== null && _field$alias !== void 0 ? _field$alias : field.name;
     var storageKey = getStorageKey(field, this._variables);
+    var reactFlightClientResponseRecordID = RelayModernRecord.getLinkedRecordID(record, storageKey);
+
+    if (reactFlightClientResponseRecordID == null) {
+      data[applicationName] = reactFlightClientResponseRecordID;
+
+      if (reactFlightClientResponseRecordID === undefined) {
+        this._isMissingData = true;
+      }
+
+      return reactFlightClientResponseRecordID;
+    }
+
+    var reactFlightClientResponseRecord = this._recordSource.get(reactFlightClientResponseRecordID);
+
+    this._seenRecords[reactFlightClientResponseRecordID] = reactFlightClientResponseRecord;
+
+    if (reactFlightClientResponseRecord == null) {
+      data[applicationName] = reactFlightClientResponseRecord;
+
+      if (reactFlightClientResponseRecord === undefined) {
+        this._isMissingData = true;
+      }
+
+      return reactFlightClientResponseRecord;
+    }
+
+    var clientResponse = getReactFlightClientResponse(reactFlightClientResponseRecord);
+    data[applicationName] = clientResponse;
+    return clientResponse;
+  };
+
+  _proto._readScalar = function _readScalar(field, record, data) {
+    var _field$alias2;
+
+    var applicationName = (_field$alias2 = field.alias) !== null && _field$alias2 !== void 0 ? _field$alias2 : field.name;
+    var storageKey = getStorageKey(field, this._variables);
     var value = RelayModernRecord.getValue(record, storageKey);
 
     if (value === undefined) {
@@ -277,12 +435,13 @@ var RelayReader = /*#__PURE__*/function () {
     }
 
     data[applicationName] = value;
+    return value;
   };
 
   _proto._readLink = function _readLink(field, record, data) {
-    var _field$alias2;
+    var _field$alias3;
 
-    var applicationName = (_field$alias2 = field.alias) !== null && _field$alias2 !== void 0 ? _field$alias2 : field.name;
+    var applicationName = (_field$alias3 = field.alias) !== null && _field$alias3 !== void 0 ? _field$alias3 : field.name;
     var storageKey = getStorageKey(field, this._variables);
     var linkedID = RelayModernRecord.getLinkedRecordID(record, storageKey);
 
@@ -293,21 +452,24 @@ var RelayReader = /*#__PURE__*/function () {
         this._isMissingData = true;
       }
 
-      return;
+      return linkedID;
     }
 
     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[incompatible-variance]
 
-    data[applicationName] = this._traverse(field, linkedID, prevData);
+    var value = this._traverse(field, linkedID, prevData);
+
+    data[applicationName] = value;
+    return value;
   };
 
   _proto._readPluralLink = function _readPluralLink(field, record, data) {
     var _this = this;
 
-    var _field$alias3;
+    var _field$alias4;
 
-    var applicationName = (_field$alias3 = field.alias) !== null && _field$alias3 !== void 0 ? _field$alias3 : field.name;
+    var applicationName = (_field$alias4 = field.alias) !== null && _field$alias4 !== void 0 ? _field$alias4 : field.name;
     var storageKey = getStorageKey(field, this._variables);
     var linkedIDs = RelayModernRecord.getLinkedRecordIDs(record, storageKey);
 
@@ -318,7 +480,7 @@ var RelayReader = /*#__PURE__*/function () {
         this._isMissingData = true;
       }
 
-      return;
+      return linkedIDs;
     }
 
     var prevData = data[applicationName];
@@ -342,6 +504,7 @@ var RelayReader = /*#__PURE__*/function () {
       linkedArray[nextIndex] = _this._traverse(field, linkedID, prevItem);
     });
     data[applicationName] = linkedArray;
+    return linkedArray;
   }
   /**
    * Reads a ReaderModuleImport, which was generated from using the @module

package/lib/store/RelayRecordSourceMapImpl.js

@@ -10,9 +10,9 @@
 // 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 _createForOfIteratorHelper(o, allowArrayLike) { var it; if (typeof Symbol === "undefined" || o[Symbol.iterator] == null) { if (Array.isArray(o) || (it = _unsupportedIterableToArray(o)) || allowArrayLike && o && typeof o.length === "number") { if (it) o = it; 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 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 _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(o); 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; }