relay-compiler 10.0.1 → 10.1.0

package/lib/transforms/DeclarativeConnectionMutationTransform.js

@@ -16,9 +16,9 @@ var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/de
 
 var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
 
-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; }
 
@@ -35,10 +35,15 @@ var _require2 = require('relay-runtime'),
     ConnectionInterface = _require2.ConnectionInterface;
 
 var DELETE_RECORD = 'deleteRecord';
+var DELETE_EDGE = 'deleteEdge';
 var APPEND_EDGE = 'appendEdge';
 var PREPEND_EDGE = 'prependEdge';
-var LINKED_FIELD_DIRECTIVES = [APPEND_EDGE, PREPEND_EDGE];
-var SCHEMA_EXTENSION = "\n  directive @".concat(DELETE_RECORD, " on FIELD\n  directive @").concat(APPEND_EDGE, "(\n    connections: [String!]!\n  ) on FIELD\n  directive @").concat(PREPEND_EDGE, "(\n    connections: [String!]!\n  ) on FIELD\n");
+var APPEND_NODE = 'appendNode';
+var PREPEND_NODE = 'prependNode';
+var EDGE_LINKED_FIELD_DIRECTIVES = [APPEND_EDGE, PREPEND_EDGE];
+var NODE_LINKED_FIELD_DIRECTIVES = [APPEND_NODE, PREPEND_NODE];
+var LINKED_FIELD_DIRECTIVES = [].concat(EDGE_LINKED_FIELD_DIRECTIVES, NODE_LINKED_FIELD_DIRECTIVES);
+var SCHEMA_EXTENSION = "\n  directive @".concat(DELETE_RECORD, " on FIELD\n  directive @").concat(DELETE_EDGE, "(\n    connections: [ID!]!\n  ) on FIELD\n  directive @").concat(APPEND_EDGE, "(\n    connections: [ID!]!\n  ) on FIELD\n  directive @").concat(PREPEND_EDGE, "(\n    connections: [ID!]!\n  ) on FIELD\n  directive @").concat(APPEND_NODE, "(\n    connections: [ID!]!\n    edgeTypeName: String!\n  ) on FIELD\n  directive @").concat(PREPEND_NODE, "(\n    connections: [ID!]!\n    edgeTypeName: String!\n  ) on FIELD\n");
 
 function transform(context) {
   return IRTransformer.transform(context, {
@@ -62,29 +67,42 @@ function visitScalarField(field) {
     throw createUserError("Invalid use of @".concat(linkedFieldDirective.name, " on scalar field '").concat(field.name, "'"), [linkedFieldDirective.loc]);
   }
 
-  var deleteDirective = field.directives.find(function (directive) {
+  var deleteNodeDirective = field.directives.find(function (directive) {
     return directive.name === DELETE_RECORD;
   });
+  var deleteEdgeDirective = field.directives.find(function (directive) {
+    return directive.name === DELETE_EDGE;
+  });
+
+  if (deleteNodeDirective != null && deleteEdgeDirective != null) {
+    throw createUserError("Both @deleteNode and @deleteEdge are used on field '".concat(field.name, "'. Only one directive is supported for now."), [deleteNodeDirective.loc, deleteEdgeDirective.loc]);
+  }
+
+  var targetDirective = deleteNodeDirective !== null && deleteNodeDirective !== void 0 ? deleteNodeDirective : deleteEdgeDirective;
 
-  if (deleteDirective == null) {
+  if (targetDirective == null) {
     return field;
   }
 
   var schema = this.getContext().getSchema();
 
-  if (!schema.isId(field.type)) {
-    throw createUserError("Invalid use of @".concat(DELETE_RECORD, " on field '").concat(field.name, "'. Expected field type ID, got ").concat(schema.getTypeString(field.type), "."), [deleteDirective.loc]);
+  if (!schema.isId(schema.getRawType(field.type))) {
+    throw createUserError("Invalid use of @".concat(targetDirective.name, " on field '").concat(field.name, "'. Expected field to return an ID or list of ID values, got ").concat(schema.getTypeString(field.type), "."), [targetDirective.loc]);
   }
 
+  var connectionsArg = targetDirective.args.find(function (arg) {
+    return arg.name === 'connections';
+  });
   var handle = {
-    name: DELETE_RECORD,
+    name: targetDirective.name,
     key: '',
     dynamicKey: null,
-    filters: null
+    filters: null,
+    handleArgs: connectionsArg ? [connectionsArg] : undefined
   };
-  return _objectSpread({}, field, {
+  return _objectSpread(_objectSpread({}, field), {}, {
     directives: field.directives.filter(function (directive) {
-      return directive !== deleteDirective;
+      return directive !== targetDirective;
     }),
     handles: field.handles ? [].concat((0, _toConsumableArray2["default"])(field.handles), [handle]) : [handle]
   });
@@ -101,64 +119,104 @@ function visitLinkedField(field) {
   }
 
   var edgeDirective = transformedField.directives.find(function (directive) {
-    return LINKED_FIELD_DIRECTIVES.indexOf(directive.name) > -1;
+    return EDGE_LINKED_FIELD_DIRECTIVES.indexOf(directive.name) > -1;
+  });
+  var nodeDirective = transformedField.directives.find(function (directive) {
+    return NODE_LINKED_FIELD_DIRECTIVES.indexOf(directive.name) > -1;
   });
 
-  if (edgeDirective == null) {
+  if (edgeDirective == null && nodeDirective == null) {
     return transformedField;
   }
 
-  var connectionsArg = edgeDirective.args.find(function (arg) {
+  if (edgeDirective != null && nodeDirective != null) {
+    throw createUserError("Invalid use of @".concat(edgeDirective.name, " and @").concat(nodeDirective.name, " on field '").concat(transformedField.name, "' - these directives cannot be used together."), [edgeDirective.loc]);
+  }
+
+  var targetDirective = edgeDirective !== null && edgeDirective !== void 0 ? edgeDirective : nodeDirective;
+  var connectionsArg = targetDirective.args.find(function (arg) {
     return arg.name === 'connections';
   });
 
   if (connectionsArg == null) {
-    throw createUserError("Expected the 'connections' argument to be defined on @".concat(edgeDirective.name, "."), [edgeDirective.loc]);
+    throw createUserError("Expected the 'connections' argument to be defined on @".concat(targetDirective.name, "."), [targetDirective.loc]);
   }
 
   var schema = this.getContext().getSchema();
-  var fields = schema.getFields(transformedField.type);
-  var cursorFieldID;
-  var nodeFieldID;
-
-  var _iterator = _createForOfIteratorHelper(fields),
-      _step;
-
-  try {
-    for (_iterator.s(); !(_step = _iterator.n()).done;) {
-      var fieldID = _step.value;
-      var fieldName = schema.getFieldName(fieldID);
-
-      if (fieldName === ConnectionInterface.get().CURSOR) {
-        cursorFieldID = fieldID;
-      } else if (fieldName === ConnectionInterface.get().NODE) {
-        nodeFieldID = fieldID;
-      }
-    } // Edge
-
-  } catch (err) {
-    _iterator.e(err);
-  } finally {
-    _iterator.f();
-  }
 
-  if (cursorFieldID != null && nodeFieldID != null) {
-    var handle = {
-      name: edgeDirective.name,
-      key: '',
-      dynamicKey: null,
-      filters: null,
-      handleArgs: [connectionsArg]
-    };
-    return _objectSpread({}, transformedField, {
-      directives: transformedField.directives.filter(function (directive) {
-        return directive !== edgeDirective;
-      }),
-      handles: transformedField.handles ? [].concat((0, _toConsumableArray2["default"])(transformedField.handles), [handle]) : [handle]
+  if (edgeDirective) {
+    var fields = schema.getFields(transformedField.type);
+    var cursorFieldID;
+    var nodeFieldID;
+
+    var _iterator = _createForOfIteratorHelper(fields),
+        _step;
+
+    try {
+      for (_iterator.s(); !(_step = _iterator.n()).done;) {
+        var fieldID = _step.value;
+        var fieldName = schema.getFieldName(fieldID);
+
+        if (fieldName === ConnectionInterface.get().CURSOR) {
+          cursorFieldID = fieldID;
+        } else if (fieldName === ConnectionInterface.get().NODE) {
+          nodeFieldID = fieldID;
+        }
+      } // Edge
+
+    } catch (err) {
+      _iterator.e(err);
+    } finally {
+      _iterator.f();
+    }
+
+    if (cursorFieldID != null && nodeFieldID != null) {
+      var handle = {
+        name: edgeDirective.name,
+        key: '',
+        dynamicKey: null,
+        filters: null,
+        handleArgs: [connectionsArg]
+      };
+      return _objectSpread(_objectSpread({}, transformedField), {}, {
+        directives: transformedField.directives.filter(function (directive) {
+          return directive !== edgeDirective;
+        }),
+        handles: transformedField.handles ? [].concat((0, _toConsumableArray2["default"])(transformedField.handles), [handle]) : [handle]
+      });
+    }
+
+    throw createUserError("Unsupported use of @".concat(edgeDirective.name, " on field '").concat(transformedField.name, "', expected an edge field (a field with 'cursor' and 'node' selection)."), [targetDirective.loc]);
+  } else {
+    // Node
+    var edgeTypeNameArg = nodeDirective.args.find(function (arg) {
+      return arg.name === 'edgeTypeName';
     });
-  }
 
-  throw createUserError("Unsupported use of @".concat(edgeDirective.name, " on field '").concat(transformedField.name, "', expected an edge field (a field with 'cursor' and 'node' selection)."), [edgeDirective.loc]);
+    if (!edgeTypeNameArg) {
+      throw createUserError("Unsupported use of @".concat(nodeDirective.name, " on field '").concat(transformedField.name, "', 'edgeTypeName' argument must be provided."), [targetDirective.loc]);
+    }
+
+    var rawType = schema.getRawType(transformedField.type);
+
+    if (schema.canHaveSelections(rawType)) {
+      var _handle = {
+        name: nodeDirective.name,
+        key: '',
+        dynamicKey: null,
+        filters: null,
+        handleArgs: [connectionsArg, edgeTypeNameArg]
+      };
+      return _objectSpread(_objectSpread({}, transformedField), {}, {
+        directives: transformedField.directives.filter(function (directive) {
+          return directive !== nodeDirective;
+        }),
+        handles: transformedField.handles ? [].concat((0, _toConsumableArray2["default"])(transformedField.handles), [_handle]) : [_handle]
+      });
+    }
+
+    throw createUserError("Unsupported use of @".concat(nodeDirective.name, " on field '").concat(transformedField.name, "'. Expected an object, union or interface, but got '").concat(schema.getTypeString(transformedField.type), "'."), [nodeDirective.loc]);
+  }
 }
 
 module.exports = {

package/lib/transforms/DeferStreamTransform.js

@@ -95,7 +95,7 @@ function visitLinkedField(field, state) {
     throw createUserError("Invalid use of @stream on non-plural field '".concat(field.name, "'"), [streamDirective.loc]);
   }
 
-  transformedField = _objectSpread({}, transformedField, {
+  transformedField = _objectSpread(_objectSpread({}, transformedField), {}, {
     directives: transformedField.directives.filter(function (directive) {
       return directive.name !== 'stream';
     })
@@ -173,7 +173,7 @@ function visitFragmentSpread(spread, state) {
     return transformedSpread;
   }
 
-  transformedSpread = _objectSpread({}, transformedSpread, {
+  transformedSpread = _objectSpread(_objectSpread({}, transformedSpread), {}, {
     directives: transformedSpread.directives.filter(function (directive) {
       return directive.name !== 'defer';
     })

package/lib/transforms/DisallowTypenameOnRoot.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/transforms/FieldHandleTransform.js

@@ -58,7 +58,7 @@ function visitField(field) {
   var filters = handle.filters;
   var args = filters ? nextField.args.filter(function (arg) {
     return filters.indexOf(arg.name) !== -1;
-  }) : []; // T45504512: new connection model
+  }) : [];
 
   if (handle.dynamicKey != null) {
     args.push({
@@ -70,7 +70,7 @@ function visitField(field) {
     });
   }
 
-  return _objectSpread({}, nextField, {
+  return _objectSpread(_objectSpread({}, nextField), {}, {
     args: args,
     alias: alias,
     name: name,

package/lib/transforms/FilterCompilerDirectivesTransform.js

@@ -0,0 +1,29 @@
+/**
+ * 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
+ */
+'use strict';
+
+var IRTransformer = require('../core/IRTransformer');
+
+var COMPILE_TIME_DIRECTIVES = new Set(['required']);
+/**
+ * A transform that removes any directives that are only interpreted by the Relay compiler.
+ */
+
+function filterDirectivesTransform(context) {
+  return IRTransformer.transform(context, {
+    Directive: function Directive(directive) {
+      return COMPILE_TIME_DIRECTIVES.has(directive.name) ? null : directive;
+    }
+  });
+}
+
+module.exports = {
+  transform: filterDirectivesTransform
+};

package/lib/transforms/FlattenTransform.js

@@ -45,9 +45,10 @@ function flattenTransformImpl(context, options) {
     Condition: visitorFn,
     Defer: visitorFn,
     Fragment: visitorFn,
-    InlineFragment: visitorFn,
     InlineDataFragmentSpread: visitorFn,
+    InlineFragment: visitorFn,
     LinkedField: visitorFn,
+    ModuleImport: visitorFn,
     Root: visitorFn,
     SplitOperation: visitorFn
   }, function () {
@@ -84,7 +85,7 @@ function memoizedFlattenSelection(cache) {
 
     var nextSelections = new Map();
     var hasFlattened = flattenSelectionsInto(context.getSchema(), nextSelections, node, state, type);
-    var flattenedNode = hasFlattened ? _objectSpread({}, node, {
+    var flattenedNode = hasFlattened ? _objectSpread(_objectSpread({}, node), {}, {
       selections: Array.from(nextSelections.values())
     }) : node;
     state.parentType = type;
@@ -124,7 +125,7 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
         throw createCompilerError("FlattenTransform: Expected an InlineFragment, got a '".concat(selection.kind, "'"), [selection.loc]);
       }
 
-      flattenedSelections.set(nodeIdentifier, _objectSpread({}, flattenedSelection, {
+      flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({}, flattenedSelection), {}, {
         selections: mergeSelections(schema, flattenedSelection, selection, state, selection.typeCondition)
       }));
     } else if (flattenedSelection.kind === 'Condition') {
@@ -132,7 +133,7 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
         throw createCompilerError("FlattenTransform: Expected a Condition, got a '".concat(selection.kind, "'"), [selection.loc]);
       }
 
-      flattenedSelections.set(nodeIdentifier, _objectSpread({}, flattenedSelection, {
+      flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({}, flattenedSelection), {}, {
         selections: mergeSelections(schema, flattenedSelection, selection, state, type)
       }));
     } else if (flattenedSelection.kind === 'ClientExtension') {
@@ -140,7 +141,7 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
         throw createCompilerError("FlattenTransform: Expected a ClientExtension, got a '".concat(selection.kind, "'"), [selection.loc]);
       }
 
-      flattenedSelections.set(nodeIdentifier, _objectSpread({}, flattenedSelection, {
+      flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({}, flattenedSelection), {}, {
         selections: mergeSelections(schema, flattenedSelection, selection, state, type)
       }));
     } else if (flattenedSelection.kind === 'FragmentSpread') {// Ignore duplicate fragment spreads.
@@ -153,7 +154,7 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
         throw createUserError('Found conflicting @module selections: use a unique alias on the ' + 'parent fields.', [selection.loc, flattenedSelection.loc]);
       }
 
-      flattenedSelections.set(nodeIdentifier, _objectSpread({}, flattenedSelection, {
+      flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({}, flattenedSelection), {}, {
         selections: mergeSelections(schema, flattenedSelection, selection, state, type)
       }));
     } else if (flattenedSelection.kind === 'Defer') {
@@ -161,9 +162,9 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
         throw createCompilerError("FlattenTransform: Expected a Defer, got a '".concat(selection.kind, "'"), [selection.loc]);
       }
 
-      flattenedSelections.set(nodeIdentifier, _objectSpread({
+      flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({
         kind: 'Defer'
-      }, flattenedSelection, {
+      }, flattenedSelection), {}, {
         selections: mergeSelections(schema, flattenedSelection, selection, state, type)
       }));
     } else if (flattenedSelection.kind === 'Stream') {
@@ -171,9 +172,9 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
         throw createCompilerError("FlattenTransform: Expected a Stream, got a '".concat(selection.kind, "'"), [selection.loc]);
       }
 
-      flattenedSelections.set(nodeIdentifier, _objectSpread({
+      flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({
         kind: 'Stream'
-      }, flattenedSelection, {
+      }, flattenedSelection), {}, {
         selections: mergeSelections(schema, flattenedSelection, selection, state, type)
       }));
     } else if (flattenedSelection.kind === 'LinkedField') {
@@ -204,9 +205,9 @@ function flattenSelectionsInto(schema, flattenedSelections, node, state, type) {
       assertUniqueArgsForAlias(selection, flattenedSelection);
 
       if (selection.handles && selection.handles.length > 0) {
-        flattenedSelections.set(nodeIdentifier, _objectSpread({
+        flattenedSelections.set(nodeIdentifier, _objectSpread(_objectSpread({
           kind: 'ScalarField'
-        }, flattenedSelection, {
+        }, flattenedSelection), {}, {
           handles: mergeHandles(selection, flattenedSelection)
         }));
       }

package/lib/transforms/GenerateIDFieldTransform.js

@@ -95,7 +95,7 @@ function visitLinkedField(field, state) {
   var unmodifiedType = schema.assertCompositeType(schema.getRawType(field.type)); // If the field type has an `id` subfield add an `id` selection
 
   if (schema.canHaveSelections(unmodifiedType) && schema.hasId(unmodifiedType)) {
-    return _objectSpread({}, transformedNode, {
+    return _objectSpread(_objectSpread({}, transformedNode), {}, {
       selections: [].concat((0, _toConsumableArray2["default"])(transformedNode.selections), [state.idFieldForType(unmodifiedType)])
     });
   } // If the field type is abstract, then generate a `... on Node { id }`
@@ -126,7 +126,7 @@ function visitLinkedField(field, state) {
     }).forEach(function (concreteType) {
       selections.push(state.idFragmentForType(concreteType));
     });
-    return _objectSpread({}, transformedNode, {
+    return _objectSpread(_objectSpread({}, transformedNode), {}, {
       selections: selections
     });
   }

package/lib/transforms/GenerateTypeNameTransform.js

@@ -69,7 +69,7 @@ function visitFragment(fragment, state) {
 
   if (!isClientType && schema.isAbstractType(rawType)) {
     var abstractKey = generateAbstractTypeRefinementKey(schema, rawType);
-    transformedNode = _objectSpread({}, transformedNode, {
+    transformedNode = _objectSpread(_objectSpread({}, transformedNode), {}, {
       selections: [{
         kind: 'ScalarField',
         alias: abstractKey,
@@ -105,7 +105,7 @@ function visitInlineFragment(fragment, state) {
 
   if (!isClientType && schema.isAbstractType(rawType)) {
     var abstractKey = generateAbstractTypeRefinementKey(schema, rawType);
-    transformedNode = _objectSpread({}, transformedNode, {
+    transformedNode = _objectSpread(_objectSpread({}, transformedNode), {}, {
       selections: [{
         kind: 'ScalarField',
         alias: abstractKey,
@@ -139,7 +139,7 @@ function visitLinkedField(field, state) {
   transformedNode = this.traverse(field, state);
 
   if (schema.isAbstractType(schema.getRawType(transformedNode.type)) && !hasUnaliasedSelection(transformedNode, TYPENAME_KEY)) {
-    transformedNode = _objectSpread({}, transformedNode, {
+    transformedNode = _objectSpread(_objectSpread({}, transformedNode), {}, {
       selections: [state.typenameField].concat((0, _toConsumableArray2["default"])(transformedNode.selections))
     });
   }

package/lib/transforms/InlineDataFragmentTransform.js

@@ -49,11 +49,11 @@ function visitFragment(fragment) {
     return transformedFragment;
   }
 
-  return _objectSpread({}, transformedFragment, {
+  return _objectSpread(_objectSpread({}, transformedFragment), {}, {
     directives: transformedFragment.directives.filter(function (directive) {
       return directive !== inlineDirective;
     }),
-    metadata: _objectSpread({}, transformedFragment.metadata || {}, {
+    metadata: _objectSpread(_objectSpread({}, transformedFragment.metadata || {}), {}, {
       inlineData: true
     })
   });

package/lib/transforms/MaskTransform.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; }
 
@@ -56,7 +56,7 @@ function visitFragment(fragment, state) {
   }
 
   var joinedArgumentDefinitions = joinArgumentDefinitions(this.getContext().getSchema(), fragment, state.reachableArguments, '@relay(unmask: true)');
-  return _objectSpread({}, result, {
+  return _objectSpread(_objectSpread({}, result), {}, {
     argumentDefinitions: joinedArgumentDefinitions
   });
 }

package/lib/transforms/MatchTransform.js

@@ -64,7 +64,7 @@ function matchTransform(context) {
 }
 
 function visitInlineFragment(node, state) {
-  return this.traverse(node, _objectSpread({}, state, {
+  return this.traverse(node, _objectSpread(_objectSpread({}, state), {}, {
     parentType: node.typeCondition
   }));
 }
@@ -111,7 +111,7 @@ function visitLinkedField(node, state) {
   }
 
   state.path.push(node);
-  var transformedNode = this.traverse(node, _objectSpread({}, state, {
+  var transformedNode = this.traverse(node, _objectSpread(_objectSpread({}, state), {}, {
     moduleKey: moduleKey,
     parentType: node.type
   }));
@@ -135,6 +135,10 @@ function visitLinkedField(node, state) {
   });
 
   if (supportedArgumentDefinition == null) {
+    if (moduleKey == null) {
+      throw createUserError('@match on a field without the `supported` argument is a no-op, please remove the `@match`.', [node.loc]);
+    }
+
     return transformedNode;
   }
 
@@ -418,7 +422,7 @@ function visitFragmentSpread(spread, _ref2) {
       module: moduleName,
       sourceDocument: documentName,
       name: spread.name,
-      selections: [_objectSpread({}, spread, {
+      selections: [_objectSpread(_objectSpread({}, spread), {}, {
         directives: spread.directives.filter(function (directive) {
           return directive !== moduleDirective;
         })