graphql 16.0.1 → 16.3.0

package/utilities/buildClientSchema.js

@@ -5,25 +5,25 @@ Object.defineProperty(exports, '__esModule', {
 });
 exports.buildClientSchema = buildClientSchema;
 
-var _inspect = require('../jsutils/inspect.js');
-
 var _devAssert = require('../jsutils/devAssert.js');
 
-var _keyValMap = require('../jsutils/keyValMap.js');
+var _inspect = require('../jsutils/inspect.js');
 
 var _isObjectLike = require('../jsutils/isObjectLike.js');
 
+var _keyValMap = require('../jsutils/keyValMap.js');
+
 var _parser = require('../language/parser.js');
 
-var _schema = require('../type/schema.js');
+var _definition = require('../type/definition.js');
 
 var _directives = require('../type/directives.js');
 
-var _scalars = require('../type/scalars.js');
-
 var _introspection = require('../type/introspection.js');
 
-var _definition = require('../type/definition.js');
+var _scalars = require('../type/scalars.js');
+
+var _schema = require('../type/schema.js');
 
 var _valueFromAST = require('./valueFromAST.js');
 
@@ -151,6 +151,8 @@ function buildClientSchema(introspection, options) {
   function buildType(type) {
     // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
     if (type != null && type.name != null && type.kind != null) {
+      // FIXME: Properly type IntrospectionType, it's a breaking change so fix in v17
+      // eslint-disable-next-line @typescript-eslint/switch-exhaustiveness-check
       switch (type.kind) {
         case _introspection.TypeKind.SCALAR:
           return buildScalarDef(type);

package/utilities/buildClientSchema.mjs

@@ -1,27 +1,27 @@
-import { inspect } from '../jsutils/inspect.mjs';
 import { devAssert } from '../jsutils/devAssert.mjs';
-import { keyValMap } from '../jsutils/keyValMap.mjs';
+import { inspect } from '../jsutils/inspect.mjs';
 import { isObjectLike } from '../jsutils/isObjectLike.mjs';
+import { keyValMap } from '../jsutils/keyValMap.mjs';
 import { parseValue } from '../language/parser.mjs';
-import { GraphQLSchema } from '../type/schema.mjs';
-import { GraphQLDirective } from '../type/directives.mjs';
-import { specifiedScalarTypes } from '../type/scalars.mjs';
-import { introspectionTypes, TypeKind } from '../type/introspection.mjs';
 import {
-  isInputType,
-  isOutputType,
+  assertInterfaceType,
+  assertNullableType,
+  assertObjectType,
+  GraphQLEnumType,
+  GraphQLInputObjectType,
+  GraphQLInterfaceType,
   GraphQLList,
   GraphQLNonNull,
-  GraphQLScalarType,
   GraphQLObjectType,
-  GraphQLInterfaceType,
+  GraphQLScalarType,
   GraphQLUnionType,
-  GraphQLEnumType,
-  GraphQLInputObjectType,
-  assertNullableType,
-  assertObjectType,
-  assertInterfaceType,
+  isInputType,
+  isOutputType,
 } from '../type/definition.mjs';
+import { GraphQLDirective } from '../type/directives.mjs';
+import { introspectionTypes, TypeKind } from '../type/introspection.mjs';
+import { specifiedScalarTypes } from '../type/scalars.mjs';
+import { GraphQLSchema } from '../type/schema.mjs';
 import { valueFromAST } from './valueFromAST.mjs';
 /**
  * Build a GraphQLSchema for use by client tools.
@@ -141,6 +141,8 @@ export function buildClientSchema(introspection, options) {
   function buildType(type) {
     // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
     if (type != null && type.name != null && type.kind != null) {
+      // FIXME: Properly type IntrospectionType, it's a breaking change so fix in v17
+      // eslint-disable-next-line @typescript-eslint/switch-exhaustiveness-check
       switch (type.kind) {
         case TypeKind.SCALAR:
           return buildScalarDef(type);

package/utilities/coerceInputValue.js

@@ -5,21 +5,21 @@ Object.defineProperty(exports, '__esModule', {
 });
 exports.coerceInputValue = coerceInputValue;
 
+var _didYouMean = require('../jsutils/didYouMean.js');
+
 var _inspect = require('../jsutils/inspect.js');
 
 var _invariant = require('../jsutils/invariant.js');
 
-var _didYouMean = require('../jsutils/didYouMean.js');
+var _isIterableObject = require('../jsutils/isIterableObject.js');
 
 var _isObjectLike = require('../jsutils/isObjectLike.js');
 
-var _suggestionList = require('../jsutils/suggestionList.js');
+var _Path = require('../jsutils/Path.js');
 
 var _printPathArray = require('../jsutils/printPathArray.js');
 
-var _Path = require('../jsutils/Path.js');
-
-var _isIterableObject = require('../jsutils/isIterableObject.js');
+var _suggestionList = require('../jsutils/suggestionList.js');
 
 var _GraphQLError = require('../error/GraphQLError.js');
 
@@ -140,7 +140,7 @@ function coerceInputValueImpl(inputValue, type, onError, path) {
     }
 
     return coercedValue;
-  } // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
+  }
 
   if ((0, _definition.isLeafType)(type)) {
     let parseResult; // Scalars and Enums determine if a input value is valid via parseValue(),
@@ -179,7 +179,9 @@ function coerceInputValueImpl(inputValue, type, onError, path) {
     }
 
     return parseResult;
-  } // istanbul ignore next (Not reachable. All possible input types have been considered)
+  }
+  /* c8 ignore next 3 */
+  // Not reachable, all possible types have been considered.
 
   false ||
     (0, _invariant.invariant)(

package/utilities/coerceInputValue.mjs

@@ -1,15 +1,15 @@
+import { didYouMean } from '../jsutils/didYouMean.mjs';
 import { inspect } from '../jsutils/inspect.mjs';
 import { invariant } from '../jsutils/invariant.mjs';
-import { didYouMean } from '../jsutils/didYouMean.mjs';
+import { isIterableObject } from '../jsutils/isIterableObject.mjs';
 import { isObjectLike } from '../jsutils/isObjectLike.mjs';
-import { suggestionList } from '../jsutils/suggestionList.mjs';
-import { printPathArray } from '../jsutils/printPathArray.mjs';
 import { addPath, pathToArray } from '../jsutils/Path.mjs';
-import { isIterableObject } from '../jsutils/isIterableObject.mjs';
+import { printPathArray } from '../jsutils/printPathArray.mjs';
+import { suggestionList } from '../jsutils/suggestionList.mjs';
 import { GraphQLError } from '../error/GraphQLError.mjs';
 import {
-  isLeafType,
   isInputObjectType,
+  isLeafType,
   isListType,
   isNonNullType,
 } from '../type/definition.mjs';
@@ -125,7 +125,7 @@ function coerceInputValueImpl(inputValue, type, onError, path) {
     }
 
     return coercedValue;
-  } // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
+  }
 
   if (isLeafType(type)) {
     let parseResult; // Scalars and Enums determine if a input value is valid via parseValue(),
@@ -164,7 +164,9 @@ function coerceInputValueImpl(inputValue, type, onError, path) {
     }
 
     return parseResult;
-  } // istanbul ignore next (Not reachable. All possible input types have been considered)
+  }
+  /* c8 ignore next 3 */
+  // Not reachable, all possible types have been considered.
 
   false || invariant(false, 'Unexpected input type: ' + inspect(type));
 }

package/utilities/extendSchema.d.ts

@@ -1,7 +1,7 @@
 import type { DocumentNode } from '../language/ast';
 import type {
-  GraphQLSchemaValidationOptions,
   GraphQLSchemaNormalizedConfig,
+  GraphQLSchemaValidationOptions,
 } from '../type/schema';
 import { GraphQLSchema } from '../type/schema';
 interface Options extends GraphQLSchemaValidationOptions {

package/utilities/extendSchema.js

@@ -6,33 +6,33 @@ Object.defineProperty(exports, '__esModule', {
 exports.extendSchema = extendSchema;
 exports.extendSchemaImpl = extendSchemaImpl;
 
-var _keyMap = require('../jsutils/keyMap.js');
+var _devAssert = require('../jsutils/devAssert.js');
 
 var _inspect = require('../jsutils/inspect.js');
 
-var _mapValue = require('../jsutils/mapValue.js');
-
 var _invariant = require('../jsutils/invariant.js');
 
-var _devAssert = require('../jsutils/devAssert.js');
+var _keyMap = require('../jsutils/keyMap.js');
+
+var _mapValue = require('../jsutils/mapValue.js');
 
 var _kinds = require('../language/kinds.js');
 
 var _predicates = require('../language/predicates.js');
 
-var _validate = require('../validation/validate.js');
+var _definition = require('../type/definition.js');
 
-var _values = require('../execution/values.js');
+var _directives = require('../type/directives.js');
 
-var _schema = require('../type/schema.js');
+var _introspection = require('../type/introspection.js');
 
 var _scalars = require('../type/scalars.js');
 
-var _introspection = require('../type/introspection.js');
+var _schema = require('../type/schema.js');
 
-var _directives = require('../type/directives.js');
+var _validate = require('../validation/validate.js');
 
-var _definition = require('../type/definition.js');
+var _values = require('../execution/values.js');
 
 var _valueFromAST = require('./valueFromAST.js');
 
@@ -229,11 +229,13 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
 
     if ((0, _definition.isEnumType)(type)) {
       return extendEnumType(type);
-    } // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
+    }
 
     if ((0, _definition.isInputObjectType)(type)) {
       return extendInputObjectType(type);
-    } // istanbul ignore next (Not reachable. All possible types have been considered)
+    }
+    /* c8 ignore next 3 */
+    // Not reachable, all possible type definition nodes have been considered.
 
     false ||
       (0, _invariant.invariant)(
@@ -391,8 +393,9 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$operationTypes;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const operationTypesNodes =
+        /* c8 ignore next */
         (_node$operationTypes = node.operationTypes) !== null &&
         _node$operationTypes !== void 0
           ? _node$operationTypes
@@ -463,8 +466,9 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$fields;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const nodeFields =
+        /* c8 ignore next */
         (_node$fields = node.fields) !== null && _node$fields !== void 0
           ? _node$fields
           : [];
@@ -493,8 +497,10 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
   }
 
   function buildArgumentMap(args) {
-    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
-    const argsNodes = args !== null && args !== void 0 ? args : [];
+    // FIXME: https://github.com/graphql/graphql-js/issues/2203
+    const argsNodes =
+      /* c8 ignore next */
+      args !== null && args !== void 0 ? args : [];
     const argConfigMap = Object.create(null);
 
     for (const arg of argsNodes) {
@@ -526,8 +532,9 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$fields2;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const fieldsNodes =
+        /* c8 ignore next */
         (_node$fields2 = node.fields) !== null && _node$fields2 !== void 0
           ? _node$fields2
           : [];
@@ -565,8 +572,9 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$values;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const valuesNodes =
+        /* c8 ignore next */
         (_node$values = node.values) !== null && _node$values !== void 0
           ? _node$values
           : [];
@@ -595,18 +603,21 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
     // validation with validateSchema() will produce more actionable results.
     // @ts-expect-error
     return nodes.flatMap(
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       (node) => {
         var _node$interfaces$map, _node$interfaces;
 
-        return (_node$interfaces$map =
-          (_node$interfaces = node.interfaces) === null ||
-          _node$interfaces === void 0
-            ? void 0
-            : _node$interfaces.map(getNamedType)) !== null &&
-          _node$interfaces$map !== void 0
-          ? _node$interfaces$map
-          : [];
+        return (
+          /* c8 ignore next */
+          (_node$interfaces$map =
+            (_node$interfaces = node.interfaces) === null ||
+            _node$interfaces === void 0
+              ? void 0
+              : _node$interfaces.map(getNamedType)) !== null &&
+            _node$interfaces$map !== void 0
+            ? _node$interfaces$map
+            : []
+        );
       },
     );
   }
@@ -617,17 +628,20 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
     // validation with validateSchema() will produce more actionable results.
     // @ts-expect-error
     return nodes.flatMap(
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       (node) => {
         var _node$types$map, _node$types;
 
-        return (_node$types$map =
-          (_node$types = node.types) === null || _node$types === void 0
-            ? void 0
-            : _node$types.map(getNamedType)) !== null &&
-          _node$types$map !== void 0
-          ? _node$types$map
-          : [];
+        return (
+          /* c8 ignore next */
+          (_node$types$map =
+            (_node$types = node.types) === null || _node$types === void 0
+              ? void 0
+              : _node$types.map(getNamedType)) !== null &&
+            _node$types$map !== void 0
+            ? _node$types$map
+            : []
+        );
       },
     );
   }
@@ -745,13 +759,7 @@ function extendSchemaImpl(schemaConfig, documentAST, options) {
           extensionASTNodes,
         });
       }
-    } // istanbul ignore next (Not reachable. All possible type definition nodes have been considered)
-
-    false ||
-      (0, _invariant.invariant)(
-        false,
-        'Unexpected type definition node: ' + (0, _inspect.inspect)(astNode),
-      );
+    }
   }
 }
 

package/utilities/extendSchema.mjs

@@ -1,47 +1,47 @@
-import { keyMap } from '../jsutils/keyMap.mjs';
+import { devAssert } from '../jsutils/devAssert.mjs';
 import { inspect } from '../jsutils/inspect.mjs';
-import { mapValue } from '../jsutils/mapValue.mjs';
 import { invariant } from '../jsutils/invariant.mjs';
-import { devAssert } from '../jsutils/devAssert.mjs';
+import { keyMap } from '../jsutils/keyMap.mjs';
+import { mapValue } from '../jsutils/mapValue.mjs';
 import { Kind } from '../language/kinds.mjs';
 import {
   isTypeDefinitionNode,
   isTypeExtensionNode,
 } from '../language/predicates.mjs';
-import { assertValidSDLExtension } from '../validation/validate.mjs';
-import { getDirectiveValues } from '../execution/values.mjs';
-import { assertSchema, GraphQLSchema } from '../type/schema.mjs';
-import {
-  specifiedScalarTypes,
-  isSpecifiedScalarType,
-} from '../type/scalars.mjs';
-import {
-  introspectionTypes,
-  isIntrospectionType,
-} from '../type/introspection.mjs';
-import {
-  GraphQLDirective,
-  GraphQLDeprecatedDirective,
-  GraphQLSpecifiedByDirective,
-} from '../type/directives.mjs';
 import {
-  isScalarType,
-  isObjectType,
-  isInterfaceType,
-  isUnionType,
-  isListType,
-  isNonNullType,
-  isEnumType,
-  isInputObjectType,
+  GraphQLEnumType,
+  GraphQLInputObjectType,
+  GraphQLInterfaceType,
   GraphQLList,
   GraphQLNonNull,
-  GraphQLScalarType,
   GraphQLObjectType,
-  GraphQLInterfaceType,
+  GraphQLScalarType,
   GraphQLUnionType,
-  GraphQLEnumType,
-  GraphQLInputObjectType,
+  isEnumType,
+  isInputObjectType,
+  isInterfaceType,
+  isListType,
+  isNonNullType,
+  isObjectType,
+  isScalarType,
+  isUnionType,
 } from '../type/definition.mjs';
+import {
+  GraphQLDeprecatedDirective,
+  GraphQLDirective,
+  GraphQLSpecifiedByDirective,
+} from '../type/directives.mjs';
+import {
+  introspectionTypes,
+  isIntrospectionType,
+} from '../type/introspection.mjs';
+import {
+  isSpecifiedScalarType,
+  specifiedScalarTypes,
+} from '../type/scalars.mjs';
+import { assertSchema, GraphQLSchema } from '../type/schema.mjs';
+import { assertValidSDLExtension } from '../validation/validate.mjs';
+import { getDirectiveValues } from '../execution/values.mjs';
 import { valueFromAST } from './valueFromAST.mjs';
 
 /**
@@ -234,11 +234,13 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
 
     if (isEnumType(type)) {
       return extendEnumType(type);
-    } // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
+    }
 
     if (isInputObjectType(type)) {
       return extendInputObjectType(type);
-    } // istanbul ignore next (Not reachable. All possible types have been considered)
+    }
+    /* c8 ignore next 3 */
+    // Not reachable, all possible type definition nodes have been considered.
 
     false || invariant(false, 'Unexpected type: ' + inspect(type));
   }
@@ -392,8 +394,9 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$operationTypes;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const operationTypesNodes =
+        /* c8 ignore next */
         (_node$operationTypes = node.operationTypes) !== null &&
         _node$operationTypes !== void 0
           ? _node$operationTypes
@@ -464,8 +467,9 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$fields;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const nodeFields =
+        /* c8 ignore next */
         (_node$fields = node.fields) !== null && _node$fields !== void 0
           ? _node$fields
           : [];
@@ -494,8 +498,10 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
   }
 
   function buildArgumentMap(args) {
-    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
-    const argsNodes = args !== null && args !== void 0 ? args : [];
+    // FIXME: https://github.com/graphql/graphql-js/issues/2203
+    const argsNodes =
+      /* c8 ignore next */
+      args !== null && args !== void 0 ? args : [];
     const argConfigMap = Object.create(null);
 
     for (const arg of argsNodes) {
@@ -527,8 +533,9 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$fields2;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const fieldsNodes =
+        /* c8 ignore next */
         (_node$fields2 = node.fields) !== null && _node$fields2 !== void 0
           ? _node$fields2
           : [];
@@ -563,8 +570,9 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
     for (const node of nodes) {
       var _node$values;
 
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       const valuesNodes =
+        /* c8 ignore next */
         (_node$values = node.values) !== null && _node$values !== void 0
           ? _node$values
           : [];
@@ -593,18 +601,21 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
     // validation with validateSchema() will produce more actionable results.
     // @ts-expect-error
     return nodes.flatMap(
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       (node) => {
         var _node$interfaces$map, _node$interfaces;
 
-        return (_node$interfaces$map =
-          (_node$interfaces = node.interfaces) === null ||
-          _node$interfaces === void 0
-            ? void 0
-            : _node$interfaces.map(getNamedType)) !== null &&
-          _node$interfaces$map !== void 0
-          ? _node$interfaces$map
-          : [];
+        return (
+          /* c8 ignore next */
+          (_node$interfaces$map =
+            (_node$interfaces = node.interfaces) === null ||
+            _node$interfaces === void 0
+              ? void 0
+              : _node$interfaces.map(getNamedType)) !== null &&
+            _node$interfaces$map !== void 0
+            ? _node$interfaces$map
+            : []
+        );
       },
     );
   }
@@ -615,17 +626,20 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
     // validation with validateSchema() will produce more actionable results.
     // @ts-expect-error
     return nodes.flatMap(
-      // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
+      // FIXME: https://github.com/graphql/graphql-js/issues/2203
       (node) => {
         var _node$types$map, _node$types;
 
-        return (_node$types$map =
-          (_node$types = node.types) === null || _node$types === void 0
-            ? void 0
-            : _node$types.map(getNamedType)) !== null &&
-          _node$types$map !== void 0
-          ? _node$types$map
-          : [];
+        return (
+          /* c8 ignore next */
+          (_node$types$map =
+            (_node$types = node.types) === null || _node$types === void 0
+              ? void 0
+              : _node$types.map(getNamedType)) !== null &&
+            _node$types$map !== void 0
+            ? _node$types$map
+            : []
+        );
       },
     );
   }
@@ -743,10 +757,7 @@ export function extendSchemaImpl(schemaConfig, documentAST, options) {
           extensionASTNodes,
         });
       }
-    } // istanbul ignore next (Not reachable. All possible type definition nodes have been considered)
-
-    false ||
-      invariant(false, 'Unexpected type definition node: ' + inspect(astNode));
+    }
   }
 }
 const stdTypeMap = keyMap(

package/utilities/findBreakingChanges.js

@@ -7,24 +7,22 @@ exports.DangerousChangeType = exports.BreakingChangeType = void 0;
 exports.findBreakingChanges = findBreakingChanges;
 exports.findDangerousChanges = findDangerousChanges;
 
-var _keyMap = require('../jsutils/keyMap.js');
-
 var _inspect = require('../jsutils/inspect.js');
 
 var _invariant = require('../jsutils/invariant.js');
 
-var _naturalCompare = require('../jsutils/naturalCompare.js');
+var _keyMap = require('../jsutils/keyMap.js');
 
 var _printer = require('../language/printer.js');
 
-var _visitor = require('../language/visitor.js');
+var _definition = require('../type/definition.js');
 
 var _scalars = require('../type/scalars.js');
 
-var _definition = require('../type/definition.js');
-
 var _astFromValue = require('./astFromValue.js');
 
+var _sortValueNode = require('./sortValueNode.js');
+
 let BreakingChangeType;
 exports.BreakingChangeType = BreakingChangeType;
 
@@ -497,11 +495,13 @@ function typeKindName(type) {
 
   if ((0, _definition.isEnumType)(type)) {
     return 'an Enum type';
-  } // istanbul ignore else (See: 'https://github.com/graphql/graphql-js/issues/2618')
+  }
 
   if ((0, _definition.isInputObjectType)(type)) {
     return 'an Input type';
-  } // istanbul ignore next (Not reachable. All possible named types have been considered)
+  }
+  /* c8 ignore next 3 */
+  // Not reachable, all possible types have been considered.
 
   false ||
     (0, _invariant.invariant)(
@@ -513,20 +513,7 @@ function typeKindName(type) {
 function stringifyValue(value, type) {
   const ast = (0, _astFromValue.astFromValue)(value, type);
   ast != null || (0, _invariant.invariant)(false);
-  const sortedAST = (0, _visitor.visit)(ast, {
-    ObjectValue(objectNode) {
-      // Make a copy since sort mutates array
-      const fields = [...objectNode.fields];
-      fields.sort((fieldA, fieldB) =>
-        (0, _naturalCompare.naturalCompare)(
-          fieldA.name.value,
-          fieldB.name.value,
-        ),
-      );
-      return { ...objectNode, fields };
-    },
-  });
-  return (0, _printer.print)(sortedAST);
+  return (0, _printer.print)((0, _sortValueNode.sortValueNode)(ast));
 }
 
 function diff(oldArray, newArray) {