graphql 0.13.1 → 0.13.2

package/error/locatedError.js

@@ -15,6 +15,7 @@ var _GraphQLError = require('./GraphQLError');
 function locatedError(originalError, nodes, path) {
   // Note: this uses a brand-check to support GraphQL errors originating from
   // other contexts.
+  // $FlowFixMe(>=0.68.0)
   if (originalError && Array.isArray(originalError.path)) {
     return originalError;
   }

package/error/locatedError.js.flow

@@ -22,6 +22,7 @@ export function locatedError(
 ): GraphQLError {
   // Note: this uses a brand-check to support GraphQL errors originating from
   // other contexts.
+  // $FlowFixMe(>=0.68.0)
   if (originalError && Array.isArray(originalError.path)) {
     return (originalError: any);
   }

package/error/locatedError.mjs

@@ -18,6 +18,7 @@ import { GraphQLError } from './GraphQLError';
 export function locatedError(originalError, nodes, path) {
   // Note: this uses a brand-check to support GraphQL errors originating from
   // other contexts.
+  // $FlowFixMe(>=0.68.0)
   if (originalError && Array.isArray(originalError.path)) {
     return originalError;
   }

package/execution/execute.js

@@ -29,10 +29,6 @@ var _iterall = require('iterall');
 
 var _error = require('../error');
 
-var _getPromise = require('../jsutils/getPromise');
-
-var _getPromise2 = _interopRequireDefault(_getPromise);
-
 var _invariant = require('../jsutils/invariant');
 
 var _invariant2 = _interopRequireDefault(_invariant);
@@ -45,6 +41,10 @@ var _isNullish = require('../jsutils/isNullish');
 
 var _isNullish2 = _interopRequireDefault(_isNullish);
 
+var _isPromise = require('../jsutils/isPromise');
+
+var _isPromise2 = _interopRequireDefault(_isPromise);
+
 var _memoize = require('../jsutils/memoize3');
 
 var _memoize2 = _interopRequireDefault(_memoize);
@@ -160,9 +160,8 @@ function executeImpl(schema, document, rootValue, contextValue, variableValues,
  * response defined by the "Response" section of the GraphQL specification.
  */
 function buildResponse(context, data) {
-  var promise = (0, _getPromise2.default)(data);
-  if (promise) {
-    return promise.then(function (resolved) {
+  if ((0, _isPromise2.default)(data)) {
+    return data.then(function (resolved) {
       return buildResponse(context, resolved);
     });
   }
@@ -288,9 +287,8 @@ function executeOperation(exeContext, operation, rootValue) {
   // Similar to completeValueCatchingError.
   try {
     var result = operation.operation === 'mutation' ? executeFieldsSerially(exeContext, type, rootValue, path, fields) : executeFields(exeContext, type, rootValue, path, fields);
-    var promise = (0, _getPromise2.default)(result);
-    if (promise) {
-      return promise.then(undefined, function (error) {
+    if ((0, _isPromise2.default)(result)) {
+      return result.then(undefined, function (error) {
         exeContext.errors.push(error);
         return Promise.resolve(null);
       });
@@ -342,9 +340,8 @@ function executeFieldsSerially(exeContext, parentType, sourceValue, path, fields
     if (result === undefined) {
       return results;
     }
-    var promise = (0, _getPromise2.default)(result);
-    if (promise) {
-      return promise.then(function (resolvedResult) {
+    if ((0, _isPromise2.default)(result)) {
+      return result.then(function (resolvedResult) {
         results[responseName] = resolvedResult;
         return results;
       });
@@ -369,7 +366,7 @@ function executeFields(exeContext, parentType, sourceValue, path, fields) {
       return results;
     }
     results[responseName] = result;
-    if ((0, _getPromise2.default)(result)) {
+    if (!containsPromise && (0, _isPromise2.default)(result)) {
       containsPromise = true;
     }
     return results;
@@ -532,8 +529,7 @@ function resolveFieldValueOrError(exeContext, fieldDef, fieldNodes, resolveFn, s
     var context = exeContext.contextValue;
 
     var result = resolveFn(source, args, context, info);
-    var promise = (0, _getPromise2.default)(result);
-    return promise ? promise.then(undefined, asErrorInstance) : result;
+    return (0, _isPromise2.default)(result) ? result.then(undefined, asErrorInstance) : result;
   } catch (error) {
     return asErrorInstance(error);
   }
@@ -558,13 +554,12 @@ function completeValueCatchingError(exeContext, returnType, fieldNodes, info, pa
   // a null value for this field if one is encountered.
   try {
     var completed = completeValueWithLocatedError(exeContext, returnType, fieldNodes, info, path, result);
-    var promise = (0, _getPromise2.default)(completed);
-    if (promise) {
+    if ((0, _isPromise2.default)(completed)) {
       // If `completeValueWithLocatedError` returned a rejected promise, log
       // the rejection error and resolve to null.
       // Note: we don't rely on a `catch` method, but we do expect "thenable"
       // to take a second callback for the error case.
-      return promise.then(undefined, function (error) {
+      return completed.then(undefined, function (error) {
         exeContext.errors.push(error);
         return Promise.resolve(null);
       });
@@ -583,9 +578,8 @@ function completeValueCatchingError(exeContext, returnType, fieldNodes, info, pa
 function completeValueWithLocatedError(exeContext, returnType, fieldNodes, info, path, result) {
   try {
     var completed = completeValue(exeContext, returnType, fieldNodes, info, path, result);
-    var promise = (0, _getPromise2.default)(completed);
-    if (promise) {
-      return promise.then(undefined, function (error) {
+    if ((0, _isPromise2.default)(completed)) {
+      return completed.then(undefined, function (error) {
         return Promise.reject((0, _error.locatedError)(asErrorInstance(error), fieldNodes, responsePathAsArray(path)));
       });
     }
@@ -618,9 +612,8 @@ function completeValueWithLocatedError(exeContext, returnType, fieldNodes, info,
  */
 function completeValue(exeContext, returnType, fieldNodes, info, path, result) {
   // If result is a Promise, apply-lift over completeValue.
-  var promise = (0, _getPromise2.default)(result);
-  if (promise) {
-    return promise.then(function (resolved) {
+  if ((0, _isPromise2.default)(result)) {
+    return result.then(function (resolved) {
       return completeValue(exeContext, returnType, fieldNodes, info, path, resolved);
     });
   }
@@ -690,7 +683,7 @@ function completeListValue(exeContext, returnType, fieldNodes, info, path, resul
     var fieldPath = addPath(path, index);
     var completedItem = completeValueCatchingError(exeContext, itemType, fieldNodes, info, fieldPath, item);
 
-    if (!containsPromise && (0, _getPromise2.default)(completedItem)) {
+    if (!containsPromise && (0, _isPromise2.default)(completedItem)) {
       containsPromise = true;
     }
     completedResults.push(completedItem);
@@ -719,9 +712,8 @@ function completeLeafValue(returnType, result) {
 function completeAbstractValue(exeContext, returnType, fieldNodes, info, path, result) {
   var runtimeType = returnType.resolveType ? returnType.resolveType(result, exeContext.contextValue, info) : defaultResolveTypeFn(result, exeContext.contextValue, info, returnType);
 
-  var promise = (0, _getPromise2.default)(runtimeType);
-  if (promise) {
-    return promise.then(function (resolvedRuntimeType) {
+  if ((0, _isPromise2.default)(runtimeType)) {
+    return runtimeType.then(function (resolvedRuntimeType) {
       return completeObjectValue(exeContext, ensureValidRuntimeType(resolvedRuntimeType, exeContext, returnType, fieldNodes, info, result), fieldNodes, info, path, result);
     });
   }
@@ -753,10 +745,9 @@ function completeObjectValue(exeContext, returnType, fieldNodes, info, path, res
   if (returnType.isTypeOf) {
     var isTypeOf = returnType.isTypeOf(result, exeContext.contextValue, info);
 
-    var promise = (0, _getPromise2.default)(isTypeOf);
-    if (promise) {
-      return promise.then(function (isTypeOfResult) {
-        if (!isTypeOfResult) {
+    if ((0, _isPromise2.default)(isTypeOf)) {
+      return isTypeOf.then(function (resolvedIsTypeOf) {
+        if (!resolvedIsTypeOf) {
           throw invalidReturnTypeError(returnType, result, fieldNodes);
         }
         return collectAndExecuteSubfields(exeContext, returnType, fieldNodes, info, path, result);
@@ -825,9 +816,8 @@ function defaultResolveTypeFn(value, context, info, abstractType) {
     if (type.isTypeOf) {
       var isTypeOfResult = type.isTypeOf(value, context, info);
 
-      var promise = (0, _getPromise2.default)(isTypeOfResult);
-      if (promise) {
-        promisedIsTypeOfResults[i] = promise;
+      if ((0, _isPromise2.default)(isTypeOfResult)) {
+        promisedIsTypeOfResults[i] = isTypeOfResult;
       } else if (isTypeOfResult) {
         return type;
       }

package/execution/execute.js.flow

@@ -9,10 +9,10 @@
 
 import { forEach, isCollection } from 'iterall';
 import { GraphQLError, locatedError } from '../error';
-import getPromise from '../jsutils/getPromise';
 import invariant from '../jsutils/invariant';
 import isInvalid from '../jsutils/isInvalid';
 import isNullish from '../jsutils/isNullish';
+import isPromise from '../jsutils/isPromise';
 import memoize3 from '../jsutils/memoize3';
 import promiseForObject from '../jsutils/promiseForObject';
 import promiseReduce from '../jsutils/promiseReduce';
@@ -33,7 +33,6 @@ import {
   isListType,
   isNonNullType,
 } from '../type/definition';
-import type { GraphQLList } from '../type/wrappers';
 import type {
   GraphQLObjectType,
   GraphQLOutputType,
@@ -43,6 +42,7 @@ import type {
   GraphQLFieldResolver,
   GraphQLResolveInfo,
   ResponsePath,
+  GraphQLList,
 } from '../type/definition';
 import { GraphQLSchema } from '../type/schema';
 import {
@@ -229,9 +229,8 @@ function buildResponse(
   context: ExecutionContext,
   data: MaybePromise<ObjMap<mixed> | null>,
 ) {
-  const promise = getPromise(data);
-  if (promise) {
-    return promise.then(resolved => buildResponse(context, resolved));
+  if (isPromise(data)) {
+    return data.then(resolved => buildResponse(context, resolved));
   }
   return context.errors.length === 0
     ? { data }
@@ -403,9 +402,8 @@ function executeOperation(
       operation.operation === 'mutation'
         ? executeFieldsSerially(exeContext, type, rootValue, path, fields)
         : executeFields(exeContext, type, rootValue, path, fields);
-    const promise = getPromise(result);
-    if (promise) {
-      return promise.then(undefined, error => {
+    if (isPromise(result)) {
+      return result.then(undefined, error => {
         exeContext.errors.push(error);
         return Promise.resolve(null);
       });
@@ -484,9 +482,8 @@ function executeFieldsSerially(
       if (result === undefined) {
         return results;
       }
-      const promise = getPromise(result);
-      if (promise) {
-        return promise.then(resolvedResult => {
+      if (isPromise(result)) {
+        return result.then(resolvedResult => {
           results[responseName] = resolvedResult;
           return results;
         });
@@ -525,7 +522,7 @@ function executeFields(
       return results;
     }
     results[responseName] = result;
-    if (getPromise(result)) {
+    if (!containsPromise && isPromise(result)) {
       containsPromise = true;
     }
     return results;
@@ -684,7 +681,7 @@ function resolveField(
   source: mixed,
   fieldNodes: $ReadOnlyArray<FieldNode>,
   path: ResponsePath,
-): mixed {
+): MaybePromise<mixed> {
   const fieldNode = fieldNodes[0];
   const fieldName = fieldNode.name.value;
 
@@ -773,8 +770,7 @@ export function resolveFieldValueOrError<TSource>(
     const context = exeContext.contextValue;
 
     const result = resolveFn(source, args, context, info);
-    const promise = getPromise(result);
-    return promise ? promise.then(undefined, asErrorInstance) : result;
+    return isPromise(result) ? result.then(undefined, asErrorInstance) : result;
   } catch (error) {
     return asErrorInstance(error);
   }
@@ -795,7 +791,7 @@ function completeValueCatchingError(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<mixed> {
   // If the field type is non-nullable, then it is resolved without any
   // protection from errors, however it still properly locates the error.
   if (isNonNullType(returnType)) {
@@ -820,13 +816,12 @@ function completeValueCatchingError(
       path,
       result,
     );
-    const promise = getPromise(completed);
-    if (promise) {
+    if (isPromise(completed)) {
       // If `completeValueWithLocatedError` returned a rejected promise, log
       // the rejection error and resolve to null.
       // Note: we don't rely on a `catch` method, but we do expect "thenable"
       // to take a second callback for the error case.
-      return promise.then(undefined, error => {
+      return completed.then(undefined, error => {
         exeContext.errors.push(error);
         return Promise.resolve(null);
       });
@@ -849,7 +844,7 @@ function completeValueWithLocatedError(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<mixed> {
   try {
     const completed = completeValue(
       exeContext,
@@ -859,9 +854,8 @@ function completeValueWithLocatedError(
       path,
       result,
     );
-    const promise = getPromise(completed);
-    if (promise) {
-      return promise.then(undefined, error =>
+    if (isPromise(completed)) {
+      return completed.then(undefined, error =>
         Promise.reject(
           locatedError(
             asErrorInstance(error),
@@ -909,11 +903,10 @@ function completeValue(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<mixed> {
   // If result is a Promise, apply-lift over completeValue.
-  const promise = getPromise(result);
-  if (promise) {
-    return promise.then(resolved =>
+  if (isPromise(result)) {
+    return result.then(resolved =>
       completeValue(exeContext, returnType, fieldNodes, info, path, resolved),
     );
   }
@@ -1012,7 +1005,7 @@ function completeListValue(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<$ReadOnlyArray<mixed>> {
   invariant(
     isCollection(result),
     `Expected Iterable, but did not find one for field ${
@@ -1038,7 +1031,7 @@ function completeListValue(
       item,
     );
 
-    if (!containsPromise && getPromise(completedItem)) {
+    if (!containsPromise && isPromise(completedItem)) {
       containsPromise = true;
     }
     completedResults.push(completedItem);
@@ -1074,14 +1067,13 @@ function completeAbstractValue(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<ObjMap<mixed>> {
   const runtimeType = returnType.resolveType
     ? returnType.resolveType(result, exeContext.contextValue, info)
     : defaultResolveTypeFn(result, exeContext.contextValue, info, returnType);
 
-  const promise = getPromise(runtimeType);
-  if (promise) {
-    return promise.then(resolvedRuntimeType =>
+  if (isPromise(runtimeType)) {
+    return runtimeType.then(resolvedRuntimeType =>
       completeObjectValue(
         exeContext,
         ensureValidRuntimeType(
@@ -1103,7 +1095,7 @@ function completeAbstractValue(
   return completeObjectValue(
     exeContext,
     ensureValidRuntimeType(
-      ((runtimeType: any): ?GraphQLObjectType | string),
+      runtimeType,
       exeContext,
       returnType,
       fieldNodes,
@@ -1163,17 +1155,16 @@ function completeObjectValue(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<ObjMap<mixed>> {
   // If there is an isTypeOf predicate function, call it with the
   // current result. If isTypeOf returns false, then raise an error rather
   // than continuing execution.
   if (returnType.isTypeOf) {
     const isTypeOf = returnType.isTypeOf(result, exeContext.contextValue, info);
 
-    const promise = getPromise(isTypeOf);
-    if (promise) {
-      return promise.then(isTypeOfResult => {
-        if (!isTypeOfResult) {
+    if (isPromise(isTypeOf)) {
+      return isTypeOf.then(resolvedIsTypeOf => {
+        if (!resolvedIsTypeOf) {
           throw invalidReturnTypeError(returnType, result, fieldNodes);
         }
         return collectAndExecuteSubfields(
@@ -1220,7 +1211,7 @@ function collectAndExecuteSubfields(
   info: GraphQLResolveInfo,
   path: ResponsePath,
   result: mixed,
-): mixed {
+): MaybePromise<ObjMap<mixed>> {
   // Collect sub-fields to execute to complete this value.
   const subFieldNodes = collectSubfields(exeContext, returnType, fieldNodes);
   return executeFields(exeContext, returnType, result, path, subFieldNodes);
@@ -1289,9 +1280,8 @@ function defaultResolveTypeFn(
     if (type.isTypeOf) {
       const isTypeOfResult = type.isTypeOf(value, context, info);
 
-      const promise = getPromise(isTypeOfResult);
-      if (promise) {
-        promisedIsTypeOfResults[i] = promise;
+      if (isPromise(isTypeOfResult)) {
+        promisedIsTypeOfResults[i] = isTypeOfResult;
       } else if (isTypeOfResult) {
         return type;
       }

package/execution/execute.mjs

@@ -11,10 +11,10 @@ var _typeof = typeof Symbol === "function" && typeof Symbol.iterator === "symbol
 
 import { forEach, isCollection } from 'iterall';
 import { GraphQLError, locatedError } from '../error';
-import getPromise from '../jsutils/getPromise';
 import invariant from '../jsutils/invariant';
 import isInvalid from '../jsutils/isInvalid';
 import isNullish from '../jsutils/isNullish';
+import isPromise from '../jsutils/isPromise';
 import memoize3 from '../jsutils/memoize3';
 import promiseForObject from '../jsutils/promiseForObject';
 import promiseReduce from '../jsutils/promiseReduce';
@@ -116,9 +116,8 @@ function executeImpl(schema, document, rootValue, contextValue, variableValues,
  * response defined by the "Response" section of the GraphQL specification.
  */
 function buildResponse(context, data) {
-  var promise = getPromise(data);
-  if (promise) {
-    return promise.then(function (resolved) {
+  if (isPromise(data)) {
+    return data.then(function (resolved) {
       return buildResponse(context, resolved);
     });
   }
@@ -244,9 +243,8 @@ function executeOperation(exeContext, operation, rootValue) {
   // Similar to completeValueCatchingError.
   try {
     var result = operation.operation === 'mutation' ? executeFieldsSerially(exeContext, type, rootValue, path, fields) : executeFields(exeContext, type, rootValue, path, fields);
-    var promise = getPromise(result);
-    if (promise) {
-      return promise.then(undefined, function (error) {
+    if (isPromise(result)) {
+      return result.then(undefined, function (error) {
         exeContext.errors.push(error);
         return Promise.resolve(null);
       });
@@ -298,9 +296,8 @@ function executeFieldsSerially(exeContext, parentType, sourceValue, path, fields
     if (result === undefined) {
       return results;
     }
-    var promise = getPromise(result);
-    if (promise) {
-      return promise.then(function (resolvedResult) {
+    if (isPromise(result)) {
+      return result.then(function (resolvedResult) {
         results[responseName] = resolvedResult;
         return results;
       });
@@ -325,7 +322,7 @@ function executeFields(exeContext, parentType, sourceValue, path, fields) {
       return results;
     }
     results[responseName] = result;
-    if (getPromise(result)) {
+    if (!containsPromise && isPromise(result)) {
       containsPromise = true;
     }
     return results;
@@ -488,8 +485,7 @@ export function resolveFieldValueOrError(exeContext, fieldDef, fieldNodes, resol
     var context = exeContext.contextValue;
 
     var result = resolveFn(source, args, context, info);
-    var promise = getPromise(result);
-    return promise ? promise.then(undefined, asErrorInstance) : result;
+    return isPromise(result) ? result.then(undefined, asErrorInstance) : result;
   } catch (error) {
     return asErrorInstance(error);
   }
@@ -514,13 +510,12 @@ function completeValueCatchingError(exeContext, returnType, fieldNodes, info, pa
   // a null value for this field if one is encountered.
   try {
     var completed = completeValueWithLocatedError(exeContext, returnType, fieldNodes, info, path, result);
-    var promise = getPromise(completed);
-    if (promise) {
+    if (isPromise(completed)) {
       // If `completeValueWithLocatedError` returned a rejected promise, log
       // the rejection error and resolve to null.
       // Note: we don't rely on a `catch` method, but we do expect "thenable"
       // to take a second callback for the error case.
-      return promise.then(undefined, function (error) {
+      return completed.then(undefined, function (error) {
         exeContext.errors.push(error);
         return Promise.resolve(null);
       });
@@ -539,9 +534,8 @@ function completeValueCatchingError(exeContext, returnType, fieldNodes, info, pa
 function completeValueWithLocatedError(exeContext, returnType, fieldNodes, info, path, result) {
   try {
     var completed = completeValue(exeContext, returnType, fieldNodes, info, path, result);
-    var promise = getPromise(completed);
-    if (promise) {
-      return promise.then(undefined, function (error) {
+    if (isPromise(completed)) {
+      return completed.then(undefined, function (error) {
         return Promise.reject(locatedError(asErrorInstance(error), fieldNodes, responsePathAsArray(path)));
       });
     }
@@ -574,9 +568,8 @@ function completeValueWithLocatedError(exeContext, returnType, fieldNodes, info,
  */
 function completeValue(exeContext, returnType, fieldNodes, info, path, result) {
   // If result is a Promise, apply-lift over completeValue.
-  var promise = getPromise(result);
-  if (promise) {
-    return promise.then(function (resolved) {
+  if (isPromise(result)) {
+    return result.then(function (resolved) {
       return completeValue(exeContext, returnType, fieldNodes, info, path, resolved);
     });
   }
@@ -646,7 +639,7 @@ function completeListValue(exeContext, returnType, fieldNodes, info, path, resul
     var fieldPath = addPath(path, index);
     var completedItem = completeValueCatchingError(exeContext, itemType, fieldNodes, info, fieldPath, item);
 
-    if (!containsPromise && getPromise(completedItem)) {
+    if (!containsPromise && isPromise(completedItem)) {
       containsPromise = true;
     }
     completedResults.push(completedItem);
@@ -675,9 +668,8 @@ function completeLeafValue(returnType, result) {
 function completeAbstractValue(exeContext, returnType, fieldNodes, info, path, result) {
   var runtimeType = returnType.resolveType ? returnType.resolveType(result, exeContext.contextValue, info) : defaultResolveTypeFn(result, exeContext.contextValue, info, returnType);
 
-  var promise = getPromise(runtimeType);
-  if (promise) {
-    return promise.then(function (resolvedRuntimeType) {
+  if (isPromise(runtimeType)) {
+    return runtimeType.then(function (resolvedRuntimeType) {
       return completeObjectValue(exeContext, ensureValidRuntimeType(resolvedRuntimeType, exeContext, returnType, fieldNodes, info, result), fieldNodes, info, path, result);
     });
   }
@@ -709,10 +701,9 @@ function completeObjectValue(exeContext, returnType, fieldNodes, info, path, res
   if (returnType.isTypeOf) {
     var isTypeOf = returnType.isTypeOf(result, exeContext.contextValue, info);
 
-    var promise = getPromise(isTypeOf);
-    if (promise) {
-      return promise.then(function (isTypeOfResult) {
-        if (!isTypeOfResult) {
+    if (isPromise(isTypeOf)) {
+      return isTypeOf.then(function (resolvedIsTypeOf) {
+        if (!resolvedIsTypeOf) {
           throw invalidReturnTypeError(returnType, result, fieldNodes);
         }
         return collectAndExecuteSubfields(exeContext, returnType, fieldNodes, info, path, result);
@@ -781,9 +772,8 @@ function defaultResolveTypeFn(value, context, info, abstractType) {
     if (type.isTypeOf) {
       var isTypeOfResult = type.isTypeOf(value, context, info);
 
-      var promise = getPromise(isTypeOfResult);
-      if (promise) {
-        promisedIsTypeOfResults[i] = promise;
+      if (isPromise(isTypeOfResult)) {
+        promisedIsTypeOfResults[i] = isTypeOfResult;
       } else if (isTypeOfResult) {
         return type;
       }

package/execution/values.js.flow

@@ -24,7 +24,6 @@ import type { GraphQLSchema } from '../type/schema';
 import type {
   FieldNode,
   DirectiveNode,
-  VariableNode,
   VariableDefinitionNode,
 } from '../language/ast';
 
@@ -139,7 +138,7 @@ export function getArgumentValues(
         );
       }
     } else if (argumentNode.value.kind === Kind.VARIABLE) {
-      const variableName = (argumentNode.value: VariableNode).name.value;
+      const variableName = argumentNode.value.name.value;
       if (
         variableValues &&
         Object.prototype.hasOwnProperty.call(variableValues, variableName) &&

package/index.js.flow

@@ -375,6 +375,7 @@ export {
 } from './utilities';
 
 export type {
+  BuildSchemaOptions,
   BreakingChange,
   DangerousChange,
   IntrospectionOptions,

package/jsutils/MaybePromise.js.flow

@@ -7,4 +7,4 @@
  * @flow strict
  */
 
-export type MaybePromise<T> = Promise<T> | T;
+export type MaybePromise<+T> = Promise<T> | T;

package/jsutils/isPromise.js

@@ -0,0 +1,24 @@
+'use strict';
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = isPromise;
+
+
+// eslint-disable-next-line no-redeclare
+function isPromise(value) {
+  return Boolean(value && typeof value.then === 'function');
+} /**
+   * Copyright (c) 2015-present, Facebook, Inc.
+   *
+   * This source code is licensed under the MIT license found in the
+   * LICENSE file in the root directory of this source tree.
+   *
+   *  strict
+   */
+
+/**
+ * Returns true if the value acts like a Promise, i.e. has a "then" function,
+ * otherwise returns false.
+ */

package/jsutils/isPromise.js.flow

@@ -0,0 +1,20 @@
+/**
+ * Copyright (c) 2015-present, Facebook, Inc.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @flow strict
+ */
+
+/**
+ * Returns true if the value acts like a Promise, i.e. has a "then" function,
+ * otherwise returns false.
+ */
+declare function isPromise(value: mixed): boolean %checks(value instanceof
+  Promise);
+
+// eslint-disable-next-line no-redeclare
+export default function isPromise(value) {
+  return Boolean(value && typeof value.then === 'function');
+}