graphql 15.3.0 → 15.5.1

package/type/schema.js

@@ -7,29 +7,29 @@ exports.isSchema = isSchema;
 exports.assertSchema = assertSchema;
 exports.GraphQLSchema = void 0;
 
-var _find = _interopRequireDefault(require("../polyfills/find"));
+var _find = _interopRequireDefault(require("../polyfills/find.js"));
 
-var _arrayFrom3 = _interopRequireDefault(require("../polyfills/arrayFrom"));
+var _arrayFrom3 = _interopRequireDefault(require("../polyfills/arrayFrom.js"));
 
-var _objectValues5 = _interopRequireDefault(require("../polyfills/objectValues"));
+var _objectValues5 = _interopRequireDefault(require("../polyfills/objectValues.js"));
 
-var _symbols = require("../polyfills/symbols");
+var _symbols = require("../polyfills/symbols.js");
 
-var _inspect = _interopRequireDefault(require("../jsutils/inspect"));
+var _inspect = _interopRequireDefault(require("../jsutils/inspect.js"));
 
-var _toObjMap = _interopRequireDefault(require("../jsutils/toObjMap"));
+var _toObjMap = _interopRequireDefault(require("../jsutils/toObjMap.js"));
 
-var _devAssert = _interopRequireDefault(require("../jsutils/devAssert"));
+var _devAssert = _interopRequireDefault(require("../jsutils/devAssert.js"));
 
-var _instanceOf = _interopRequireDefault(require("../jsutils/instanceOf"));
+var _instanceOf = _interopRequireDefault(require("../jsutils/instanceOf.js"));
 
-var _isObjectLike = _interopRequireDefault(require("../jsutils/isObjectLike"));
+var _isObjectLike = _interopRequireDefault(require("../jsutils/isObjectLike.js"));
 
-var _introspection = require("./introspection");
+var _introspection = require("./introspection.js");
 
-var _directives = require("./directives");
+var _directives = require("./directives.js");
 
-var _definition = require("./definition");
+var _definition = require("./definition.js");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
@@ -331,7 +331,7 @@ var GraphQLSchema = /*#__PURE__*/function () {
       extensionASTNodes: (_this$extensionASTNod = this.extensionASTNodes) !== null && _this$extensionASTNod !== void 0 ? _this$extensionASTNod : [],
       assumeValid: this.__validationErrors !== undefined
     };
-  } // $FlowFixMe Flow doesn't support computed properties yet
+  } // $FlowFixMe[unsupported-syntax] Flow doesn't support computed properties yet
   ;
 
   _createClass(GraphQLSchema, [{

package/type/schema.js.flow

@@ -1,5 +1,4 @@
 // @flow strict
-
 import find from '../polyfills/find';
 import arrayFrom from '../polyfills/arrayFrom';
 import objectValues from '../polyfills/objectValues';
@@ -144,7 +143,7 @@ export class GraphQLSchema {
   // Used as a cache for validateSchema().
   __validationErrors: ?$ReadOnlyArray<GraphQLError>;
 
-  constructor(config: $ReadOnly<GraphQLSchemaConfig>): void {
+  constructor(config: $ReadOnly<GraphQLSchemaConfig>) {
     // If this schema was built from a source known to be valid, then it may be
     // marked with assumeValid to avoid an additional type system validation.
     this.__validationErrors = config.assumeValid === true ? [] : undefined;
@@ -357,7 +356,7 @@ export class GraphQLSchema {
     };
   }
 
-  // $FlowFixMe Flow doesn't support computed properties yet
+  // $FlowFixMe[unsupported-syntax] Flow doesn't support computed properties yet
   get [SYMBOL_TO_STRING_TAG]() {
     return 'GraphQLSchema';
   }

package/type/schema.mjs

@@ -310,7 +310,7 @@ export var GraphQLSchema = /*#__PURE__*/function () {
       extensionASTNodes: (_this$extensionASTNod = this.extensionASTNodes) !== null && _this$extensionASTNod !== void 0 ? _this$extensionASTNod : [],
       assumeValid: this.__validationErrors !== undefined
     };
-  } // $FlowFixMe Flow doesn't support computed properties yet
+  } // $FlowFixMe[unsupported-syntax] Flow doesn't support computed properties yet
   ;
 
   _createClass(GraphQLSchema, [{

package/type/validate.js

@@ -6,29 +6,27 @@ Object.defineProperty(exports, "__esModule", {
 exports.validateSchema = validateSchema;
 exports.assertValidSchema = assertValidSchema;
 
-var _find = _interopRequireDefault(require("../polyfills/find"));
+var _find = _interopRequireDefault(require("../polyfills/find.js"));
 
-var _flatMap = _interopRequireDefault(require("../polyfills/flatMap"));
+var _objectValues5 = _interopRequireDefault(require("../polyfills/objectValues.js"));
 
-var _objectValues5 = _interopRequireDefault(require("../polyfills/objectValues"));
+var _inspect = _interopRequireDefault(require("../jsutils/inspect.js"));
 
-var _inspect = _interopRequireDefault(require("../jsutils/inspect"));
+var _GraphQLError = require("../error/GraphQLError.js");
 
-var _GraphQLError = require("../error/GraphQLError");
+var _locatedError = require("../error/locatedError.js");
 
-var _locatedError = require("../error/locatedError");
+var _assertValidName = require("../utilities/assertValidName.js");
 
-var _assertValidName = require("../utilities/assertValidName");
+var _typeComparators = require("../utilities/typeComparators.js");
 
-var _typeComparators = require("../utilities/typeComparators");
+var _schema = require("./schema.js");
 
-var _schema = require("./schema");
+var _introspection = require("./introspection.js");
 
-var _directives = require("./directives");
+var _directives = require("./directives.js");
 
-var _introspection = require("./introspection");
-
-var _definition = require("./definition");
+var _definition = require("./definition.js");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
@@ -166,6 +164,13 @@ function validateDirectives(context) {
       if (!(0, _definition.isInputType)(arg.type)) {
         context.reportError("The type of @".concat(directive.name, "(").concat(arg.name, ":) must be Input Type ") + "but got: ".concat((0, _inspect.default)(arg.type), "."), arg.astNode);
       }
+
+      if ((0, _definition.isRequiredArgument)(arg) && arg.deprecationReason != null) {
+        var _arg$astNode;
+
+        context.reportError("Required argument @".concat(directive.name, "(").concat(arg.name, ":) cannot be deprecated."), [getDeprecatedDirectiveNode(arg.astNode), // istanbul ignore next (TODO need to write coverage tests)
+        (_arg$astNode = arg.astNode) === null || _arg$astNode === void 0 ? void 0 : _arg$astNode.type]);
+      }
     }
   }
 }
@@ -248,9 +253,16 @@ function validateFields(context, type) {
       validateName(context, arg); // Ensure the type is an input type
 
       if (!(0, _definition.isInputType)(arg.type)) {
-        var _arg$astNode;
+        var _arg$astNode2;
 
-        context.reportError("The type of ".concat(type.name, ".").concat(field.name, "(").concat(argName, ":) must be Input ") + "Type but got: ".concat((0, _inspect.default)(arg.type), "."), (_arg$astNode = arg.astNode) === null || _arg$astNode === void 0 ? void 0 : _arg$astNode.type);
+        context.reportError("The type of ".concat(type.name, ".").concat(field.name, "(").concat(argName, ":) must be Input ") + "Type but got: ".concat((0, _inspect.default)(arg.type), "."), (_arg$astNode2 = arg.astNode) === null || _arg$astNode2 === void 0 ? void 0 : _arg$astNode2.type);
+      }
+
+      if ((0, _definition.isRequiredArgument)(arg) && arg.deprecationReason != null) {
+        var _arg$astNode3;
+
+        context.reportError("Required argument ".concat(type.name, ".").concat(field.name, "(").concat(argName, ":) cannot be deprecated."), [getDeprecatedDirectiveNode(arg.astNode), // istanbul ignore next (TODO need to write coverage tests)
+        (_arg$astNode3 = arg.astNode) === null || _arg$astNode3 === void 0 ? void 0 : _arg$astNode3.type]);
       }
     }
   }
@@ -431,6 +443,13 @@ function validateInputFields(context, inputObj) {
 
       context.reportError("The type of ".concat(inputObj.name, ".").concat(field.name, " must be Input Type ") + "but got: ".concat((0, _inspect.default)(field.type), "."), (_field$astNode2 = field.astNode) === null || _field$astNode2 === void 0 ? void 0 : _field$astNode2.type);
     }
+
+    if ((0, _definition.isRequiredInputField)(field) && field.deprecationReason != null) {
+      var _field$astNode3;
+
+      context.reportError("Required input field ".concat(inputObj.name, ".").concat(field.name, " cannot be deprecated."), [getDeprecatedDirectiveNode(field.astNode), // istanbul ignore next (TODO need to write coverage tests)
+      (_field$astNode3 = field.astNode) === null || _field$astNode3 === void 0 ? void 0 : _field$astNode3.type]);
+    }
   }
 }
 
@@ -491,12 +510,17 @@ function getAllNodes(object) {
 }
 
 function getAllSubNodes(object, getter) {
-  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
-  return (0, _flatMap.default)(getAllNodes(object), function (item) {
+  var subNodes = [];
+
+  for (var _i32 = 0, _getAllNodes2 = getAllNodes(object); _i32 < _getAllNodes2.length; _i32++) {
     var _getter;
 
-    return (_getter = getter(item)) !== null && _getter !== void 0 ? _getter : [];
-  });
+    var node = _getAllNodes2[_i32];
+    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+    subNodes = subNodes.concat((_getter = getter(node)) !== null && _getter !== void 0 ? _getter : []);
+  }
+
+  return subNodes;
 }
 
 function getAllImplementsInterfaceNodes(type, iface) {
@@ -514,3 +538,12 @@ function getUnionMemberTypeNodes(union, typeName) {
     return typeNode.name.value === typeName;
   });
 }
+
+function getDeprecatedDirectiveNode(definitionNode) {
+  var _definitionNode$direc;
+
+  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  return definitionNode === null || definitionNode === void 0 ? void 0 : (_definitionNode$direc = definitionNode.directives) === null || _definitionNode$direc === void 0 ? void 0 : _definitionNode$direc.find(function (node) {
+    return node.name.value === _directives.GraphQLDeprecatedDirective.name;
+  });
+}

package/type/validate.js.flow

@@ -1,7 +1,5 @@
 // @flow strict
-
 import find from '../polyfills/find';
-import flatMap from '../polyfills/flatMap';
 import objectValues from '../polyfills/objectValues';
 
 import inspect from '../jsutils/inspect';
@@ -12,6 +10,7 @@ import { locatedError } from '../error/locatedError';
 import type {
   ASTNode,
   NamedTypeNode,
+  DirectiveNode,
   OperationTypeNode,
 } from '../language/ast';
 
@@ -27,8 +26,8 @@ import type {
   GraphQLInputObjectType,
 } from './definition';
 import { assertSchema } from './schema';
-import { isDirective } from './directives';
 import { isIntrospectionType } from './introspection';
+import { isDirective, GraphQLDeprecatedDirective } from './directives';
 import {
   isObjectType,
   isInterfaceType,
@@ -40,6 +39,7 @@ import {
   isInputType,
   isOutputType,
   isRequiredArgument,
+  isRequiredInputField,
 } from './definition';
 
 /**
@@ -110,7 +110,7 @@ class SchemaValidationContext {
   }
 }
 
-function validateRootTypes(context) {
+function validateRootTypes(context: SchemaValidationContext): void {
   const schema = context.schema;
   const queryType = schema.getQueryType();
   if (!queryType) {
@@ -185,6 +185,17 @@ function validateDirectives(context: SchemaValidationContext): void {
           arg.astNode,
         );
       }
+
+      if (isRequiredArgument(arg) && arg.deprecationReason != null) {
+        context.reportError(
+          `Required argument @${directive.name}(${arg.name}:) cannot be deprecated.`,
+          [
+            getDeprecatedDirectiveNode(arg.astNode),
+            // istanbul ignore next (TODO need to write coverage tests)
+            arg.astNode?.type,
+          ],
+        );
+      }
     }
   }
 }
@@ -290,6 +301,17 @@ function validateFields(
           arg.astNode?.type,
         );
       }
+
+      if (isRequiredArgument(arg) && arg.deprecationReason != null) {
+        context.reportError(
+          `Required argument ${type.name}.${field.name}(${argName}:) cannot be deprecated.`,
+          [
+            getDeprecatedDirectiveNode(arg.astNode),
+            // istanbul ignore next (TODO need to write coverage tests)
+            arg.astNode?.type,
+          ],
+        );
+      }
     }
   }
 }
@@ -525,12 +547,23 @@ function validateInputFields(
         field.astNode?.type,
       );
     }
+
+    if (isRequiredInputField(field) && field.deprecationReason != null) {
+      context.reportError(
+        `Required input field ${inputObj.name}.${field.name} cannot be deprecated.`,
+        [
+          getDeprecatedDirectiveNode(field.astNode),
+          // istanbul ignore next (TODO need to write coverage tests)
+          field.astNode?.type,
+        ],
+      );
+    }
   }
 }
 
 function createInputObjectCircularRefsValidator(
   context: SchemaValidationContext,
-) {
+): (GraphQLInputObjectType) => void {
   // Modified copy of algorithm from 'src/validation/rules/NoFragmentCycles.js'.
   // Tracks already visited types to maintain O(N) and to ensure that cycles
   // are not redundantly reported.
@@ -547,7 +580,7 @@ function createInputObjectCircularRefsValidator(
   // This does a straight-forward DFS to find cycles.
   // It does not terminate when a cycle was found but continues to explore
   // the graph to find all possible cycles.
-  function detectCycleRecursive(inputObj: GraphQLInputObjectType) {
+  function detectCycleRecursive(inputObj: GraphQLInputObjectType): void {
     if (visitedTypes[inputObj.name]) {
       return;
     }
@@ -601,8 +634,12 @@ function getAllSubNodes<T: ASTNode, K: ASTNode, L: ASTNode>(
   object: SDLDefinedObject<T, K>,
   getter: (T | K) => ?(L | $ReadOnlyArray<L>),
 ): $ReadOnlyArray<L> {
-  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
-  return flatMap(getAllNodes(object), (item) => getter(item) ?? []);
+  let subNodes = [];
+  for (const node of getAllNodes(object)) {
+    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+    subNodes = subNodes.concat(getter(node) ?? []);
+  }
+  return subNodes;
 }
 
 function getAllImplementsInterfaceNodes(
@@ -622,3 +659,12 @@ function getUnionMemberTypeNodes(
     (typeNode) => typeNode.name.value === typeName,
   );
 }
+
+function getDeprecatedDirectiveNode(
+  definitionNode: ?{ +directives?: $ReadOnlyArray<DirectiveNode>, ... },
+): ?DirectiveNode {
+  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  return definitionNode?.directives?.find(
+    (node) => node.name.value === GraphQLDeprecatedDirective.name,
+  );
+}

package/type/validate.mjs

@@ -1,5 +1,4 @@
 import find from "../polyfills/find.mjs";
-import flatMap from "../polyfills/flatMap.mjs";
 import objectValues from "../polyfills/objectValues.mjs";
 import inspect from "../jsutils/inspect.mjs";
 import { GraphQLError } from "../error/GraphQLError.mjs";
@@ -7,9 +6,9 @@ import { locatedError } from "../error/locatedError.mjs";
 import { isValidNameError } from "../utilities/assertValidName.mjs";
 import { isEqualType, isTypeSubTypeOf } from "../utilities/typeComparators.mjs";
 import { assertSchema } from "./schema.mjs";
-import { isDirective } from "./directives.mjs";
 import { isIntrospectionType } from "./introspection.mjs";
-import { isObjectType, isInterfaceType, isUnionType, isEnumType, isInputObjectType, isNamedType, isNonNullType, isInputType, isOutputType, isRequiredArgument } from "./definition.mjs";
+import { isDirective, GraphQLDeprecatedDirective } from "./directives.mjs";
+import { isObjectType, isInterfaceType, isUnionType, isEnumType, isInputObjectType, isNamedType, isNonNullType, isInputType, isOutputType, isRequiredArgument, isRequiredInputField } from "./definition.mjs";
 /**
  * Implements the "Type Validation" sub-sections of the specification's
  * "Type System" section.
@@ -144,6 +143,13 @@ function validateDirectives(context) {
       if (!isInputType(arg.type)) {
         context.reportError("The type of @".concat(directive.name, "(").concat(arg.name, ":) must be Input Type ") + "but got: ".concat(inspect(arg.type), "."), arg.astNode);
       }
+
+      if (isRequiredArgument(arg) && arg.deprecationReason != null) {
+        var _arg$astNode;
+
+        context.reportError("Required argument @".concat(directive.name, "(").concat(arg.name, ":) cannot be deprecated."), [getDeprecatedDirectiveNode(arg.astNode), // istanbul ignore next (TODO need to write coverage tests)
+        (_arg$astNode = arg.astNode) === null || _arg$astNode === void 0 ? void 0 : _arg$astNode.type]);
+      }
     }
   }
 }
@@ -226,9 +232,16 @@ function validateFields(context, type) {
       validateName(context, arg); // Ensure the type is an input type
 
       if (!isInputType(arg.type)) {
-        var _arg$astNode;
+        var _arg$astNode2;
 
-        context.reportError("The type of ".concat(type.name, ".").concat(field.name, "(").concat(argName, ":) must be Input ") + "Type but got: ".concat(inspect(arg.type), "."), (_arg$astNode = arg.astNode) === null || _arg$astNode === void 0 ? void 0 : _arg$astNode.type);
+        context.reportError("The type of ".concat(type.name, ".").concat(field.name, "(").concat(argName, ":) must be Input ") + "Type but got: ".concat(inspect(arg.type), "."), (_arg$astNode2 = arg.astNode) === null || _arg$astNode2 === void 0 ? void 0 : _arg$astNode2.type);
+      }
+
+      if (isRequiredArgument(arg) && arg.deprecationReason != null) {
+        var _arg$astNode3;
+
+        context.reportError("Required argument ".concat(type.name, ".").concat(field.name, "(").concat(argName, ":) cannot be deprecated."), [getDeprecatedDirectiveNode(arg.astNode), // istanbul ignore next (TODO need to write coverage tests)
+        (_arg$astNode3 = arg.astNode) === null || _arg$astNode3 === void 0 ? void 0 : _arg$astNode3.type]);
       }
     }
   }
@@ -409,6 +422,13 @@ function validateInputFields(context, inputObj) {
 
       context.reportError("The type of ".concat(inputObj.name, ".").concat(field.name, " must be Input Type ") + "but got: ".concat(inspect(field.type), "."), (_field$astNode2 = field.astNode) === null || _field$astNode2 === void 0 ? void 0 : _field$astNode2.type);
     }
+
+    if (isRequiredInputField(field) && field.deprecationReason != null) {
+      var _field$astNode3;
+
+      context.reportError("Required input field ".concat(inputObj.name, ".").concat(field.name, " cannot be deprecated."), [getDeprecatedDirectiveNode(field.astNode), // istanbul ignore next (TODO need to write coverage tests)
+      (_field$astNode3 = field.astNode) === null || _field$astNode3 === void 0 ? void 0 : _field$astNode3.type]);
+    }
   }
 }
 
@@ -469,12 +489,17 @@ function getAllNodes(object) {
 }
 
 function getAllSubNodes(object, getter) {
-  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
-  return flatMap(getAllNodes(object), function (item) {
+  var subNodes = [];
+
+  for (var _i32 = 0, _getAllNodes2 = getAllNodes(object); _i32 < _getAllNodes2.length; _i32++) {
     var _getter;
 
-    return (_getter = getter(item)) !== null && _getter !== void 0 ? _getter : [];
-  });
+    var node = _getAllNodes2[_i32];
+    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+    subNodes = subNodes.concat((_getter = getter(node)) !== null && _getter !== void 0 ? _getter : []);
+  }
+
+  return subNodes;
 }
 
 function getAllImplementsInterfaceNodes(type, iface) {
@@ -492,3 +517,12 @@ function getUnionMemberTypeNodes(union, typeName) {
     return typeNode.name.value === typeName;
   });
 }
+
+function getDeprecatedDirectiveNode(definitionNode) {
+  var _definitionNode$direc;
+
+  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  return definitionNode === null || definitionNode === void 0 ? void 0 : (_definitionNode$direc = definitionNode.directives) === null || _definitionNode$direc === void 0 ? void 0 : _definitionNode$direc.find(function (node) {
+    return node.name.value === GraphQLDeprecatedDirective.name;
+  });
+}

package/utilities/TypeInfo.js

@@ -6,19 +6,19 @@ Object.defineProperty(exports, "__esModule", {
 exports.visitWithTypeInfo = visitWithTypeInfo;
 exports.TypeInfo = void 0;
 
-var _find = _interopRequireDefault(require("../polyfills/find"));
+var _find = _interopRequireDefault(require("../polyfills/find.js"));
 
-var _kinds = require("../language/kinds");
+var _kinds = require("../language/kinds.js");
 
-var _ast = require("../language/ast");
+var _ast = require("../language/ast.js");
 
-var _visitor = require("../language/visitor");
+var _visitor = require("../language/visitor.js");
 
-var _definition = require("../type/definition");
+var _definition = require("../type/definition.js");
 
-var _introspection = require("../type/introspection");
+var _introspection = require("../type/introspection.js");
 
-var _typeFromAST = require("./typeFromAST");
+var _typeFromAST = require("./typeFromAST.js");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 

package/utilities/TypeInfo.js.flow

@@ -1,5 +1,4 @@
 // @flow strict
-
 import find from '../polyfills/find';
 
 import type { Visitor } from '../language/visitor';
@@ -66,7 +65,7 @@ export class TypeInfo {
     // Initial type may be provided in rare cases to facilitate traversals
     // beginning somewhere other than documents.
     initialType?: GraphQLType,
-  ): void {
+  ) {
     this._schema = schema;
     this._typeStack = [];
     this._parentTypeStack = [];

package/utilities/assertValidName.js

@@ -6,9 +6,9 @@ Object.defineProperty(exports, "__esModule", {
 exports.assertValidName = assertValidName;
 exports.isValidNameError = isValidNameError;
 
-var _devAssert = _interopRequireDefault(require("../jsutils/devAssert"));
+var _devAssert = _interopRequireDefault(require("../jsutils/devAssert.js"));
 
-var _GraphQLError = require("../error/GraphQLError");
+var _GraphQLError = require("../error/GraphQLError.js");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 

package/utilities/assertValidName.js.flow

@@ -1,5 +1,4 @@
 // @flow strict
-
 import devAssert from '../jsutils/devAssert';
 
 import { GraphQLError } from '../error/GraphQLError';

package/utilities/astFromValue.js

@@ -5,25 +5,23 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.astFromValue = astFromValue;
 
-var _isFinite = _interopRequireDefault(require("../polyfills/isFinite"));
+var _isFinite = _interopRequireDefault(require("../polyfills/isFinite.js"));
 
-var _arrayFrom3 = _interopRequireDefault(require("../polyfills/arrayFrom"));
+var _objectValues3 = _interopRequireDefault(require("../polyfills/objectValues.js"));
 
-var _objectValues3 = _interopRequireDefault(require("../polyfills/objectValues"));
+var _inspect = _interopRequireDefault(require("../jsutils/inspect.js"));
 
-var _inspect = _interopRequireDefault(require("../jsutils/inspect"));
+var _invariant = _interopRequireDefault(require("../jsutils/invariant.js"));
 
-var _invariant = _interopRequireDefault(require("../jsutils/invariant"));
+var _isObjectLike = _interopRequireDefault(require("../jsutils/isObjectLike.js"));
 
-var _isObjectLike = _interopRequireDefault(require("../jsutils/isObjectLike"));
+var _safeArrayFrom = _interopRequireDefault(require("../jsutils/safeArrayFrom.js"));
 
-var _isCollection = _interopRequireDefault(require("../jsutils/isCollection"));
+var _kinds = require("../language/kinds.js");
 
-var _kinds = require("../language/kinds");
+var _scalars = require("../type/scalars.js");
 
-var _scalars = require("../type/scalars");
-
-var _definition = require("../type/definition");
+var _definition = require("../type/definition.js");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
@@ -75,13 +73,13 @@ function astFromValue(value, type) {
 
   if ((0, _definition.isListType)(type)) {
     var itemType = type.ofType;
+    var items = (0, _safeArrayFrom.default)(value);
 
-    if ((0, _isCollection.default)(value)) {
-      var valuesNodes = []; // Since we transpile for-of in loose mode it doesn't support iterators
-      // and it's required to first convert iteratable into array
+    if (items != null) {
+      var valuesNodes = [];
 
-      for (var _i2 = 0, _arrayFrom2 = (0, _arrayFrom3.default)(value); _i2 < _arrayFrom2.length; _i2++) {
-        var item = _arrayFrom2[_i2];
+      for (var _i2 = 0; _i2 < items.length; _i2++) {
+        var item = items[_i2];
         var itemNode = astFromValue(item, itemType);
 
         if (itemNode != null) {

package/utilities/astFromValue.js.flow

@@ -1,13 +1,11 @@
 // @flow strict
-
 import isFinite from '../polyfills/isFinite';
-import arrayFrom from '../polyfills/arrayFrom';
 import objectValues from '../polyfills/objectValues';
 
 import inspect from '../jsutils/inspect';
 import invariant from '../jsutils/invariant';
 import isObjectLike from '../jsutils/isObjectLike';
-import isCollection from '../jsutils/isCollection';
+import safeArrayFrom from '../jsutils/safeArrayFrom';
 
 import type { ValueNode } from '../language/ast';
 import { Kind } from '../language/kinds';
@@ -66,11 +64,11 @@ export function astFromValue(value: mixed, type: GraphQLInputType): ?ValueNode {
   // the value is not an array, convert the value using the list's item type.
   if (isListType(type)) {
     const itemType = type.ofType;
-    if (isCollection(value)) {
+
+    const items = safeArrayFrom(value);
+    if (items != null) {
       const valuesNodes = [];
-      // Since we transpile for-of in loose mode it doesn't support iterators
-      // and it's required to first convert iteratable into array
-      for (const item of arrayFrom(value)) {
+      for (const item of items) {
         const itemNode = astFromValue(item, itemType);
         if (itemNode != null) {
           valuesNodes.push(itemNode);
@@ -78,6 +76,7 @@ export function astFromValue(value: mixed, type: GraphQLInputType): ?ValueNode {
       }
       return { kind: Kind.LIST, values: valuesNodes };
     }
+
     return astFromValue(value, itemType);
   }
 

package/utilities/astFromValue.mjs

@@ -1,10 +1,9 @@
 import isFinite from "../polyfills/isFinite.mjs";
-import arrayFrom from "../polyfills/arrayFrom.mjs";
 import objectValues from "../polyfills/objectValues.mjs";
 import inspect from "../jsutils/inspect.mjs";
 import invariant from "../jsutils/invariant.mjs";
 import isObjectLike from "../jsutils/isObjectLike.mjs";
-import isCollection from "../jsutils/isCollection.mjs";
+import safeArrayFrom from "../jsutils/safeArrayFrom.mjs";
 import { Kind } from "../language/kinds.mjs";
 import { GraphQLID } from "../type/scalars.mjs";
 import { isLeafType, isEnumType, isInputObjectType, isListType, isNonNullType } from "../type/definition.mjs";
@@ -57,13 +56,13 @@ export function astFromValue(value, type) {
 
   if (isListType(type)) {
     var itemType = type.ofType;
+    var items = safeArrayFrom(value);
 
-    if (isCollection(value)) {
-      var valuesNodes = []; // Since we transpile for-of in loose mode it doesn't support iterators
-      // and it's required to first convert iteratable into array
+    if (items != null) {
+      var valuesNodes = [];
 
-      for (var _i2 = 0, _arrayFrom2 = arrayFrom(value); _i2 < _arrayFrom2.length; _i2++) {
-        var item = _arrayFrom2[_i2];
+      for (var _i2 = 0; _i2 < items.length; _i2++) {
+        var item = items[_i2];
         var itemNode = astFromValue(item, itemType);
 
         if (itemNode != null) {