graphql 16.0.0 → 16.2.0

package/README.md

@@ -71,9 +71,9 @@ or an array of promises. A more complex example is included in the top-level [te
 Then, serve the result of a query against that type schema.
 
 ```js
-var query = '{ hello }';
+var source = '{ hello }';
 
-graphql(schema, query).then((result) => {
+graphql({ schema, source }).then((result) => {
   // Prints
   // {
   //   data: { hello: "world" }
@@ -87,9 +87,9 @@ first ensure the query is syntactically and semantically valid before executing
 it, reporting errors otherwise.
 
 ```js
-var query = '{ BoyHowdy }';
+var source = '{ BoyHowdy }';
 
-graphql(schema, query).then((result) => {
+graphql({ schema, source }).then((result) => {
   // Prints
   // {
   //   errors: [

package/error/GraphQLError.js

@@ -151,7 +151,9 @@ class GraphQLError extends Error {
         enumerable: false,
       },
     }); // Include (non-enumerable) stack trace.
-    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2317')
+
+    /* c8 ignore start */
+    // FIXME: https://github.com/graphql/graphql-js/issues/2317
 
     if (
       originalError !== null &&
@@ -172,6 +174,7 @@ class GraphQLError extends Error {
         configurable: true,
       });
     }
+    /* c8 ignore stop */
   }
 
   get [Symbol.toStringTag]() {

package/error/GraphQLError.mjs

@@ -150,7 +150,9 @@ export class GraphQLError extends Error {
         enumerable: false,
       },
     }); // Include (non-enumerable) stack trace.
-    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2317')
+
+    /* c8 ignore start */
+    // FIXME: https://github.com/graphql/graphql-js/issues/2317
 
     if (
       originalError !== null &&
@@ -171,6 +173,7 @@ export class GraphQLError extends Error {
         configurable: true,
       });
     }
+    /* c8 ignore stop */
   }
 
   get [Symbol.toStringTag]() {

package/error/locatedError.js

@@ -5,7 +5,7 @@ Object.defineProperty(exports, '__esModule', {
 });
 exports.locatedError = locatedError;
 
-var _inspect = require('../jsutils/inspect.js');
+var _toError = require('../jsutils/toError.js');
 
 var _GraphQLError = require('./GraphQLError.js');
 
@@ -17,13 +17,7 @@ var _GraphQLError = require('./GraphQLError.js');
 function locatedError(rawOriginalError, nodes, path) {
   var _nodes;
 
-  // Sometimes a non-error is thrown, wrap it as an Error instance to ensure a consistent Error interface.
-  const originalError =
-    rawOriginalError instanceof Error
-      ? rawOriginalError
-      : new Error(
-          'Unexpected error value: ' + (0, _inspect.inspect)(rawOriginalError),
-        ); // Note: this uses a brand-check to support GraphQL errors originating from other contexts.
+  const originalError = (0, _toError.toError)(rawOriginalError); // Note: this uses a brand-check to support GraphQL errors originating from other contexts.
 
   if (isLocatedGraphQLError(originalError)) {
     return originalError;

package/error/locatedError.mjs

@@ -1,4 +1,4 @@
-import { inspect } from '../jsutils/inspect.mjs';
+import { toError } from '../jsutils/toError.mjs';
 import { GraphQLError } from './GraphQLError.mjs';
 /**
  * Given an arbitrary value, presumably thrown while attempting to execute a
@@ -9,11 +9,7 @@ import { GraphQLError } from './GraphQLError.mjs';
 export function locatedError(rawOriginalError, nodes, path) {
   var _nodes;
 
-  // Sometimes a non-error is thrown, wrap it as an Error instance to ensure a consistent Error interface.
-  const originalError =
-    rawOriginalError instanceof Error
-      ? rawOriginalError
-      : new Error('Unexpected error value: ' + inspect(rawOriginalError)); // Note: this uses a brand-check to support GraphQL errors originating from other contexts.
+  const originalError = toError(rawOriginalError); // Note: this uses a brand-check to support GraphQL errors originating from other contexts.
 
   if (isLocatedGraphQLError(originalError)) {
     return originalError;

package/execution/collectFields.d.ts

@@ -7,11 +7,11 @@ import type {
 import type { GraphQLSchema } from '../type/schema';
 import type { GraphQLObjectType } from '../type/definition';
 /**
- * Given a selectionSet, collect all of the fields and returns it at the end.
+ * Given a selectionSet, collects all of the fields and returns them.
  *
- * CollectFields requires the "runtime type" of an object. For a field which
+ * CollectFields requires the "runtime type" of an object. For a field that
  * returns an Interface or Union type, the "runtime type" will be the actual
- * Object type returned by that field.
+ * object type returned by that field.
  *
  * @internal
  */
@@ -26,11 +26,11 @@ export declare function collectFields(
 ): Map<string, ReadonlyArray<FieldNode>>;
 /**
  * Given an array of field nodes, collects all of the subfields of the passed
- * in fields, and returns it at the end.
+ * in fields, and returns them at the end.
  *
- * CollectFields requires the "return type" of an object. For a field which
+ * CollectSubFields requires the "return type" of an object. For a field that
  * returns an Interface or Union type, the "return type" will be the actual
- * Object type returned by that field.
+ * object type returned by that field.
  *
  * @internal
  */

package/execution/collectFields.js

@@ -17,11 +17,11 @@ var _typeFromAST = require('../utilities/typeFromAST.js');
 var _values = require('./values.js');
 
 /**
- * Given a selectionSet, collect all of the fields and returns it at the end.
+ * Given a selectionSet, collects all of the fields and returns them.
  *
- * CollectFields requires the "runtime type" of an object. For a field which
+ * CollectFields requires the "runtime type" of an object. For a field that
  * returns an Interface or Union type, the "runtime type" will be the actual
- * Object type returned by that field.
+ * object type returned by that field.
  *
  * @internal
  */
@@ -46,11 +46,11 @@ function collectFields(
 }
 /**
  * Given an array of field nodes, collects all of the subfields of the passed
- * in fields, and returns it at the end.
+ * in fields, and returns them at the end.
  *
- * CollectFields requires the "return type" of an object. For a field which
+ * CollectSubFields requires the "return type" of an object. For a field that
  * returns an Interface or Union type, the "return type" will be the actual
- * Object type returned by that field.
+ * object type returned by that field.
  *
  * @internal
  */

package/execution/collectFields.mjs

@@ -7,11 +7,11 @@ import { isAbstractType } from '../type/definition.mjs';
 import { typeFromAST } from '../utilities/typeFromAST.mjs';
 import { getDirectiveValues } from './values.mjs';
 /**
- * Given a selectionSet, collect all of the fields and returns it at the end.
+ * Given a selectionSet, collects all of the fields and returns them.
  *
- * CollectFields requires the "runtime type" of an object. For a field which
+ * CollectFields requires the "runtime type" of an object. For a field that
  * returns an Interface or Union type, the "runtime type" will be the actual
- * Object type returned by that field.
+ * object type returned by that field.
  *
  * @internal
  */
@@ -37,11 +37,11 @@ export function collectFields(
 }
 /**
  * Given an array of field nodes, collects all of the subfields of the passed
- * in fields, and returns it at the end.
+ * in fields, and returns them at the end.
  *
- * CollectFields requires the "return type" of an object. For a field which
+ * CollectSubFields requires the "return type" of an object. For a field that
  * returns an Interface or Union type, the "return type" will be the actual
- * Object type returned by that field.
+ * object type returned by that field.
  *
  * @internal
  */

package/execution/execute.js

@@ -102,6 +102,12 @@ const collectSubfields = (0, _memoize.memoize3)(
  * a GraphQLError will be thrown immediately explaining the invalid input.
  */
 function execute(args) {
+  // Temporary for v15 to v16 migration. Remove in v17
+  arguments.length < 2 ||
+    (0, _devAssert.devAssert)(
+      false,
+      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+    );
   const { schema, document, variableValues, rootValue } = args; // If arguments are missing or incorrect, throw an error.
 
   assertValidExecutionArguments(schema, document, variableValues); // If a valid execution context cannot be created due to incorrect arguments,
@@ -247,6 +253,8 @@ function buildExecutionContext(args) {
       case _kinds.Kind.FRAGMENT_DEFINITION:
         fragments[definition.name.value] = definition;
         break;
+
+      default: // ignore non-executable definitions
     }
   }
 
@@ -260,7 +268,9 @@ function buildExecutionContext(args) {
     }
 
     return [new _GraphQLError.GraphQLError('Must provide an operation.')];
-  } // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  } // FIXME: https://github.com/graphql/graphql-js/issues/2203
+
+  /* c8 ignore next */
 
   const variableDefinitions =
     (_operation$variableDe = operation.variableDefinitions) !== null &&
@@ -424,7 +434,7 @@ function executeFields(exeContext, parentType, sourceValue, path, fields) {
   return (0, _promiseForObject.promiseForObject)(results);
 }
 /**
- * Implements the "Executing field" section of the spec
+ * Implements the "Executing fields" section of the spec
  * In particular, this function figures out the value that the field returns by
  * calling its resolve function, then calls completeValue to complete promises,
  * serialize scalars, or execute the sub-selection-set for objects.
@@ -541,7 +551,7 @@ function handleFieldError(error, returnType, exeContext) {
 }
 /**
  * Implements the instructions for completeValue as defined in the
- * "Field entries" section of the spec.
+ * "Value Completion" section of the spec.
  *
  * If the field type is Non-Null, then this recursively completes the value
  * for the inner type. It throws a field error if that completion returns null,
@@ -618,7 +628,6 @@ function completeValue(exeContext, returnType, fieldNodes, info, path, result) {
       result,
     );
   } // If field type is Object, execute and complete all sub-selections.
-  // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
 
   if ((0, _definition.isObjectType)(returnType)) {
     return completeObjectValue(
@@ -629,7 +638,9 @@ function completeValue(exeContext, returnType, fieldNodes, info, path, result) {
       path,
       result,
     );
-  } // istanbul ignore next (Not reachable. All possible output types have been considered)
+  }
+  /* c8 ignore next 6 */
+  // Not reachable, all possible output types have been considered.
 
   false ||
     (0, _invariant.invariant)(

package/execution/execute.mjs

@@ -83,6 +83,12 @@ const collectSubfields = memoize3((exeContext, returnType, fieldNodes) =>
  * a GraphQLError will be thrown immediately explaining the invalid input.
  */
 export function execute(args) {
+  // Temporary for v15 to v16 migration. Remove in v17
+  arguments.length < 2 ||
+    devAssert(
+      false,
+      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+    );
   const { schema, document, variableValues, rootValue } = args; // If arguments are missing or incorrect, throw an error.
 
   assertValidExecutionArguments(schema, document, variableValues); // If a valid execution context cannot be created due to incorrect arguments,
@@ -232,6 +238,8 @@ export function buildExecutionContext(args) {
       case Kind.FRAGMENT_DEFINITION:
         fragments[definition.name.value] = definition;
         break;
+
+      default: // ignore non-executable definitions
     }
   }
 
@@ -241,7 +249,9 @@ export function buildExecutionContext(args) {
     }
 
     return [new GraphQLError('Must provide an operation.')];
-  } // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  } // FIXME: https://github.com/graphql/graphql-js/issues/2203
+
+  /* c8 ignore next */
 
   const variableDefinitions =
     (_operation$variableDe = operation.variableDefinitions) !== null &&
@@ -405,7 +415,7 @@ function executeFields(exeContext, parentType, sourceValue, path, fields) {
   return promiseForObject(results);
 }
 /**
- * Implements the "Executing field" section of the spec
+ * Implements the "Executing fields" section of the spec
  * In particular, this function figures out the value that the field returns by
  * calling its resolve function, then calls completeValue to complete promises,
  * serialize scalars, or execute the sub-selection-set for objects.
@@ -520,7 +530,7 @@ function handleFieldError(error, returnType, exeContext) {
 }
 /**
  * Implements the instructions for completeValue as defined in the
- * "Field entries" section of the spec.
+ * "Value Completion" section of the spec.
  *
  * If the field type is Non-Null, then this recursively completes the value
  * for the inner type. It throws a field error if that completion returns null,
@@ -597,7 +607,6 @@ function completeValue(exeContext, returnType, fieldNodes, info, path, result) {
       result,
     );
   } // If field type is Object, execute and complete all sub-selections.
-  // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
 
   if (isObjectType(returnType)) {
     return completeObjectValue(
@@ -608,7 +617,9 @@ function completeValue(exeContext, returnType, fieldNodes, info, path, result) {
       path,
       result,
     );
-  } // istanbul ignore next (Not reachable. All possible output types have been considered)
+  }
+  /* c8 ignore next 6 */
+  // Not reachable, all possible output types have been considered.
 
   false ||
     invariant(

package/execution/mapAsyncIterator.js

@@ -23,7 +23,8 @@ function mapAsyncIterator(iterable, callback) {
         done: false,
       };
     } catch (error) {
-      // istanbul ignore else (FIXME: add test case)
+      /* c8 ignore start */
+      // FIXME: add test case
       if (typeof iterator.return === 'function') {
         try {
           await iterator.return();
@@ -33,6 +34,7 @@ function mapAsyncIterator(iterable, callback) {
       }
 
       throw error;
+      /* c8 ignore stop */
     }
   }
 

package/execution/mapAsyncIterator.mjs

@@ -16,7 +16,8 @@ export function mapAsyncIterator(iterable, callback) {
         done: false,
       };
     } catch (error) {
-      // istanbul ignore else (FIXME: add test case)
+      /* c8 ignore start */
+      // FIXME: add test case
       if (typeof iterator.return === 'function') {
         try {
           await iterator.return();
@@ -26,6 +27,7 @@ export function mapAsyncIterator(iterable, callback) {
       }
 
       throw error;
+      /* c8 ignore stop */
     }
   }
 

package/execution/subscribe.js

@@ -8,6 +8,8 @@ exports.subscribe = subscribe;
 
 var _inspect = require('../jsutils/inspect.js');
 
+var _devAssert = require('../jsutils/devAssert.js');
+
 var _isAsyncIterable = require('../jsutils/isAsyncIterable.js');
 
 var _Path = require('../jsutils/Path.js');
@@ -46,6 +48,12 @@ var _mapAsyncIterator = require('./mapAsyncIterator.js');
  * Accepts either an object with named arguments, or individual arguments.
  */
 async function subscribe(args) {
+  // Temporary for v15 to v16 migration. Remove in v17
+  arguments.length < 2 ||
+    (0, _devAssert.devAssert)(
+      false,
+      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+    );
   const {
     schema,
     document,

package/execution/subscribe.mjs

@@ -1,4 +1,5 @@
 import { inspect } from '../jsutils/inspect.mjs';
+import { devAssert } from '../jsutils/devAssert.mjs';
 import { isAsyncIterable } from '../jsutils/isAsyncIterable.mjs';
 import { addPath, pathToArray } from '../jsutils/Path.mjs';
 import { GraphQLError } from '../error/GraphQLError.mjs';
@@ -36,6 +37,12 @@ import { mapAsyncIterator } from './mapAsyncIterator.mjs';
  */
 
 export async function subscribe(args) {
+  // Temporary for v15 to v16 migration. Remove in v17
+  arguments.length < 2 ||
+    devAssert(
+      false,
+      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+    );
   const {
     schema,
     document,

package/execution/values.js

@@ -169,7 +169,9 @@ function coerceVariableValues(schema, varDefNodes, inputs, onError) {
 function getArgumentValues(def, node, variableValues) {
   var _node$arguments;
 
-  const coercedValues = {}; // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  const coercedValues = {}; // FIXME: https://github.com/graphql/graphql-js/issues/2203
+
+  /* c8 ignore next */
 
   const argumentNodes =
     (_node$arguments = node.arguments) !== null && _node$arguments !== void 0
@@ -275,7 +277,6 @@ function getArgumentValues(def, node, variableValues) {
 function getDirectiveValues(directiveDef, node, variableValues) {
   var _node$directives;
 
-  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
   const directiveNode =
     (_node$directives = node.directives) === null || _node$directives === void 0
       ? void 0

package/execution/values.mjs

@@ -145,7 +145,9 @@ function coerceVariableValues(schema, varDefNodes, inputs, onError) {
 export function getArgumentValues(def, node, variableValues) {
   var _node$arguments;
 
-  const coercedValues = {}; // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+  const coercedValues = {}; // FIXME: https://github.com/graphql/graphql-js/issues/2203
+
+  /* c8 ignore next */
 
   const argumentNodes =
     (_node$arguments = node.arguments) !== null && _node$arguments !== void 0
@@ -238,7 +240,6 @@ export function getArgumentValues(def, node, variableValues) {
 export function getDirectiveValues(directiveDef, node, variableValues) {
   var _node$directives;
 
-  // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
   const directiveNode =
     (_node$directives = node.directives) === null || _node$directives === void 0
       ? void 0

package/graphql.js

@@ -6,6 +6,8 @@ Object.defineProperty(exports, '__esModule', {
 exports.graphql = graphql;
 exports.graphqlSync = graphqlSync;
 
+var _devAssert = require('./jsutils/devAssert.js');
+
 var _isPromise = require('./jsutils/isPromise.js');
 
 var _parser = require('./language/parser.js');
@@ -38,6 +40,12 @@ function graphqlSync(args) {
 }
 
 function graphqlImpl(args) {
+  // Temporary for v15 to v16 migration. Remove in v17
+  arguments.length < 2 ||
+    (0, _devAssert.devAssert)(
+      false,
+      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+    );
   const {
     schema,
     source,

package/graphql.mjs

@@ -1,3 +1,4 @@
+import { devAssert } from './jsutils/devAssert.mjs';
 import { isPromise } from './jsutils/isPromise.mjs';
 import { parse } from './language/parser.mjs';
 import { validate } from './validation/validate.mjs';
@@ -65,6 +66,12 @@ export function graphqlSync(args) {
 }
 
 function graphqlImpl(args) {
+  // Temporary for v15 to v16 migration. Remove in v17
+  arguments.length < 2 ||
+    devAssert(
+      false,
+      'graphql@16 dropped long-deprecated support for positional arguments, please pass an object instead.',
+    );
   const {
     schema,
     source,

package/index.d.ts

@@ -30,6 +30,8 @@ export type { GraphQLArgs } from './graphql';
 export { graphql, graphqlSync } from './graphql';
 /** Create and operate on GraphQL type definitions and schema. */
 export {
+  resolveObjMapThunk,
+  resolveReadonlyArrayThunk,
   /** Definitions */
   GraphQLSchema,
   GraphQLDirective,
@@ -48,6 +50,9 @@ export {
   GraphQLString,
   GraphQLBoolean,
   GraphQLID,
+  /** Int boundaries constants */
+  GRAPHQL_MAX_INT,
+  GRAPHQL_MIN_INT,
   /** Built-in Directives defined by the Spec */
   specifiedDirectives,
   GraphQLIncludeDirective,

package/index.js

@@ -51,6 +51,18 @@ Object.defineProperty(exports, 'FragmentsOnCompositeTypesRule', {
     return _index4.FragmentsOnCompositeTypesRule;
   },
 });
+Object.defineProperty(exports, 'GRAPHQL_MAX_INT', {
+  enumerable: true,
+  get: function () {
+    return _index.GRAPHQL_MAX_INT;
+  },
+});
+Object.defineProperty(exports, 'GRAPHQL_MIN_INT', {
+  enumerable: true,
+  get: function () {
+    return _index.GRAPHQL_MIN_INT;
+  },
+});
 Object.defineProperty(exports, 'GraphQLBoolean', {
   enumerable: true,
   get: function () {
@@ -1107,6 +1119,18 @@ Object.defineProperty(exports, 'printType', {
     return _index6.printType;
   },
 });
+Object.defineProperty(exports, 'resolveObjMapThunk', {
+  enumerable: true,
+  get: function () {
+    return _index.resolveObjMapThunk;
+  },
+});
+Object.defineProperty(exports, 'resolveReadonlyArrayThunk', {
+  enumerable: true,
+  get: function () {
+    return _index.resolveReadonlyArrayThunk;
+  },
+});
 Object.defineProperty(exports, 'responsePathAsArray', {
   enumerable: true,
   get: function () {

package/index.mjs

@@ -32,6 +32,8 @@ export { graphql, graphqlSync } from './graphql.mjs';
 /** Create and operate on GraphQL type definitions and schema. */
 
 export {
+  resolveObjMapThunk,
+  resolveReadonlyArrayThunk,
   /** Definitions */
   GraphQLSchema,
   GraphQLDirective,
@@ -50,6 +52,9 @@ export {
   GraphQLString,
   GraphQLBoolean,
   GraphQLID,
+  /** Int boundaries constants */
+  GRAPHQL_MAX_INT,
+  GRAPHQL_MIN_INT,
   /** Built-in Directives defined by the Spec */
   specifiedDirectives,
   GraphQLIncludeDirective,

package/jsutils/devAssert.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, '__esModule', {
 exports.devAssert = devAssert;
 
 function devAssert(condition, message) {
-  const booleanCondition = Boolean(condition); // istanbul ignore else (See transformation done in './resources/inlineInvariant.js')
+  const booleanCondition = Boolean(condition);
 
   if (!booleanCondition) {
     throw new Error(message);

package/jsutils/devAssert.mjs

@@ -1,5 +1,5 @@
 export function devAssert(condition, message) {
-  const booleanCondition = Boolean(condition); // istanbul ignore else (See transformation done in './resources/inlineInvariant.js')
+  const booleanCondition = Boolean(condition);
 
   if (!booleanCondition) {
     throw new Error(message);

package/jsutils/instanceOf.js

@@ -14,7 +14,9 @@ var _inspect = require('./inspect.js');
  * See: https://webpack.js.org/guides/production/
  */
 const instanceOf =
-  process.env.NODE_ENV === 'production' // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2317')
+  /* c8 ignore next 5 */
+  // FIXME: https://github.com/graphql/graphql-js/issues/2317
+  process.env.NODE_ENV === 'production'
     ? function instanceOf(value, constructor) {
         return value instanceof constructor;
       }

package/jsutils/instanceOf.mjs

@@ -7,7 +7,9 @@ import { inspect } from './inspect.mjs';
  */
 
 export const instanceOf =
-  process.env.NODE_ENV === 'production' // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2317')
+  /* c8 ignore next 5 */
+  // FIXME: https://github.com/graphql/graphql-js/issues/2317
+  process.env.NODE_ENV === 'production'
     ? function instanceOf(value, constructor) {
         return value instanceof constructor;
       }

package/jsutils/invariant.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, '__esModule', {
 exports.invariant = invariant;
 
 function invariant(condition, message) {
-  const booleanCondition = Boolean(condition); // istanbul ignore else (See transformation done in './resources/inlineInvariant.js')
+  const booleanCondition = Boolean(condition);
 
   if (!booleanCondition) {
     throw new Error(

package/jsutils/invariant.mjs

@@ -1,5 +1,5 @@
 export function invariant(condition, message) {
-  const booleanCondition = Boolean(condition); // istanbul ignore else (See transformation done in './resources/inlineInvariant.js')
+  const booleanCondition = Boolean(condition);
 
   if (!booleanCondition) {
     throw new Error(

package/jsutils/mapValue.js

@@ -12,8 +12,8 @@ exports.mapValue = mapValue;
 function mapValue(map, fn) {
   const result = Object.create(null);
 
-  for (const [key, value] of Object.entries(map)) {
-    result[key] = fn(value, key);
+  for (const key of Object.keys(map)) {
+    result[key] = fn(map[key], key);
   }
 
   return result;

package/jsutils/mapValue.mjs

@@ -5,8 +5,8 @@
 export function mapValue(map, fn) {
   const result = Object.create(null);
 
-  for (const [key, value] of Object.entries(map)) {
-    result[key] = fn(value, key);
+  for (const key of Object.keys(map)) {
+    result[key] = fn(map[key], key);
   }
 
   return result;