graphql 14.1.1 → 14.3.1

package/utilities/findBreakingChanges.js

@@ -5,26 +5,12 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.findBreakingChanges = findBreakingChanges;
 exports.findDangerousChanges = findDangerousChanges;
-exports.findRemovedTypes = findRemovedTypes;
-exports.findTypesThatChangedKind = findTypesThatChangedKind;
-exports.findArgChanges = findArgChanges;
-exports.findFieldsThatChangedTypeOnObjectOrInterfaceTypes = findFieldsThatChangedTypeOnObjectOrInterfaceTypes;
-exports.findFieldsThatChangedTypeOnInputObjectTypes = findFieldsThatChangedTypeOnInputObjectTypes;
-exports.findTypesRemovedFromUnions = findTypesRemovedFromUnions;
-exports.findTypesAddedToUnions = findTypesAddedToUnions;
-exports.findValuesRemovedFromEnums = findValuesRemovedFromEnums;
-exports.findValuesAddedToEnums = findValuesAddedToEnums;
-exports.findInterfacesRemovedFromObjectTypes = findInterfacesRemovedFromObjectTypes;
-exports.findInterfacesAddedToObjectTypes = findInterfacesAddedToObjectTypes;
-exports.findRemovedDirectives = findRemovedDirectives;
-exports.findRemovedDirectiveArgs = findRemovedDirectiveArgs;
-exports.findAddedNonNullDirectiveArgs = findAddedNonNullDirectiveArgs;
-exports.findRemovedLocationsForDirective = findRemovedLocationsForDirective;
-exports.findRemovedDirectiveLocations = findRemovedDirectiveLocations;
 exports.DangerousChangeType = exports.BreakingChangeType = void 0;
 
 var _find = _interopRequireDefault(require("../polyfills/find"));
 
+var _inspect = _interopRequireDefault(require("../jsutils/inspect"));
+
 var _definition = require("../type/definition");
 
 var _keyMap = _interopRequireDefault(require("../jsutils/keyMap"));
@@ -32,7 +18,7 @@ var _keyMap = _interopRequireDefault(require("../jsutils/keyMap"));
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 /**
- * Copyright (c) 2016-present, Facebook, Inc.
+ * Copyright (c) Facebook, Inc. and its affiliates.
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
@@ -94,10 +80,8 @@ function findRemovedTypes(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var breakingChanges = [];
 
-  var _arr = Object.keys(oldTypeMap);
-
-  for (var _i = 0; _i < _arr.length; _i++) {
-    var typeName = _arr[_i];
+  for (var _i = 0, _Object$keys = Object.keys(oldTypeMap); _i < _Object$keys.length; _i++) {
+    var typeName = _Object$keys[_i];
 
     if (!newTypeMap[typeName]) {
       breakingChanges.push({
@@ -120,10 +104,8 @@ function findTypesThatChangedKind(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var breakingChanges = [];
 
-  var _arr2 = Object.keys(oldTypeMap);
-
-  for (var _i2 = 0; _i2 < _arr2.length; _i2++) {
-    var typeName = _arr2[_i2];
+  for (var _i2 = 0, _Object$keys2 = Object.keys(oldTypeMap); _i2 < _Object$keys2.length; _i2++) {
+    var typeName = _Object$keys2[_i2];
 
     if (!newTypeMap[typeName]) {
       continue;
@@ -156,10 +138,8 @@ function findArgChanges(oldSchema, newSchema) {
   var breakingChanges = [];
   var dangerousChanges = [];
 
-  var _arr3 = Object.keys(oldTypeMap);
-
-  for (var _i3 = 0; _i3 < _arr3.length; _i3++) {
-    var typeName = _arr3[_i3];
+  for (var _i3 = 0, _Object$keys3 = Object.keys(oldTypeMap); _i3 < _Object$keys3.length; _i3++) {
+    var typeName = _Object$keys3[_i3];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -170,10 +150,8 @@ function findArgChanges(oldSchema, newSchema) {
     var oldTypeFields = oldType.getFields();
     var newTypeFields = newType.getFields();
 
-    var _arr4 = Object.keys(oldTypeFields);
-
-    for (var _i4 = 0; _i4 < _arr4.length; _i4++) {
-      var fieldName = _arr4[_i4];
+    for (var _i4 = 0, _Object$keys4 = Object.keys(oldTypeFields); _i4 < _Object$keys4.length; _i4++) {
+      var fieldName = _Object$keys4[_i4];
 
       if (!newTypeFields[fieldName]) {
         continue;
@@ -310,9 +288,12 @@ function typeKindName(type) {
 
   if ((0, _definition.isInputObjectType)(type)) {
     return 'an Input type';
-  }
+  } // Not reachable. All possible named types have been considered.
+
+  /* istanbul ignore next */
 
-  throw new TypeError('Unknown type ' + type.constructor.name);
+
+  throw new TypeError("Unexpected type: ".concat((0, _inspect.default)(type), "."));
 }
 
 function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(oldSchema, newSchema) {
@@ -320,10 +301,8 @@ function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(oldSchema, newSchema)
   var newTypeMap = newSchema.getTypeMap();
   var breakingChanges = [];
 
-  var _arr5 = Object.keys(oldTypeMap);
-
-  for (var _i5 = 0; _i5 < _arr5.length; _i5++) {
-    var typeName = _arr5[_i5];
+  for (var _i5 = 0, _Object$keys5 = Object.keys(oldTypeMap); _i5 < _Object$keys5.length; _i5++) {
+    var typeName = _Object$keys5[_i5];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -334,10 +313,8 @@ function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(oldSchema, newSchema)
     var oldTypeFieldsDef = oldType.getFields();
     var newTypeFieldsDef = newType.getFields();
 
-    var _arr6 = Object.keys(oldTypeFieldsDef);
-
-    for (var _i6 = 0; _i6 < _arr6.length; _i6++) {
-      var fieldName = _arr6[_i6];
+    for (var _i6 = 0, _Object$keys6 = Object.keys(oldTypeFieldsDef); _i6 < _Object$keys6.length; _i6++) {
+      var fieldName = _Object$keys6[_i6];
 
       // Check if the field is missing on the type in the new schema.
       if (!(fieldName in newTypeFieldsDef)) {
@@ -371,10 +348,8 @@ function findFieldsThatChangedTypeOnInputObjectTypes(oldSchema, newSchema) {
   var breakingChanges = [];
   var dangerousChanges = [];
 
-  var _arr7 = Object.keys(oldTypeMap);
-
-  for (var _i7 = 0; _i7 < _arr7.length; _i7++) {
-    var typeName = _arr7[_i7];
+  for (var _i7 = 0, _Object$keys7 = Object.keys(oldTypeMap); _i7 < _Object$keys7.length; _i7++) {
+    var typeName = _Object$keys7[_i7];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -385,10 +360,8 @@ function findFieldsThatChangedTypeOnInputObjectTypes(oldSchema, newSchema) {
     var oldTypeFieldsDef = oldType.getFields();
     var newTypeFieldsDef = newType.getFields();
 
-    var _arr8 = Object.keys(oldTypeFieldsDef);
-
-    for (var _i8 = 0; _i8 < _arr8.length; _i8++) {
-      var fieldName = _arr8[_i8];
+    for (var _i8 = 0, _Object$keys8 = Object.keys(oldTypeFieldsDef); _i8 < _Object$keys8.length; _i8++) {
+      var fieldName = _Object$keys8[_i8];
 
       // Check if the field is missing on the type in the new schema.
       if (!(fieldName in newTypeFieldsDef)) {
@@ -413,10 +386,8 @@ function findFieldsThatChangedTypeOnInputObjectTypes(oldSchema, newSchema) {
     } // Check if a field was added to the input object type
 
 
-    var _arr9 = Object.keys(newTypeFieldsDef);
-
-    for (var _i9 = 0; _i9 < _arr9.length; _i9++) {
-      var _fieldName = _arr9[_i9];
+    for (var _i9 = 0, _Object$keys9 = Object.keys(newTypeFieldsDef); _i9 < _Object$keys9.length; _i9++) {
+      var _fieldName = _Object$keys9[_i9];
 
       if (!(_fieldName in oldTypeFieldsDef)) {
         if ((0, _definition.isRequiredInputField)(newTypeFieldsDef[_fieldName])) {
@@ -441,40 +412,40 @@ function findFieldsThatChangedTypeOnInputObjectTypes(oldSchema, newSchema) {
 }
 
 function isChangeSafeForObjectOrInterfaceField(oldType, newType) {
-  if ((0, _definition.isNamedType)(oldType)) {
-    return (// if they're both named types, see if their names are equivalent
-      (0, _definition.isNamedType)(newType) && oldType.name === newType.name || // moving from nullable to non-null of the same underlying type is safe
-      (0, _definition.isNonNullType)(newType) && isChangeSafeForObjectOrInterfaceField(oldType, newType.ofType)
-    );
-  } else if ((0, _definition.isListType)(oldType)) {
+  if ((0, _definition.isListType)(oldType)) {
     return (// if they're both lists, make sure the underlying types are compatible
       (0, _definition.isListType)(newType) && isChangeSafeForObjectOrInterfaceField(oldType.ofType, newType.ofType) || // moving from nullable to non-null of the same underlying type is safe
       (0, _definition.isNonNullType)(newType) && isChangeSafeForObjectOrInterfaceField(oldType, newType.ofType)
     );
-  } else if ((0, _definition.isNonNullType)(oldType)) {
+  }
+
+  if ((0, _definition.isNonNullType)(oldType)) {
     // if they're both non-null, make sure the underlying types are compatible
     return (0, _definition.isNonNullType)(newType) && isChangeSafeForObjectOrInterfaceField(oldType.ofType, newType.ofType);
   }
 
-  return false;
+  return (// if they're both named types, see if their names are equivalent
+    (0, _definition.isNamedType)(newType) && oldType.name === newType.name || // moving from nullable to non-null of the same underlying type is safe
+    (0, _definition.isNonNullType)(newType) && isChangeSafeForObjectOrInterfaceField(oldType, newType.ofType)
+  );
 }
 
 function isChangeSafeForInputObjectFieldOrFieldArg(oldType, newType) {
-  if ((0, _definition.isNamedType)(oldType)) {
-    // if they're both named types, see if their names are equivalent
-    return (0, _definition.isNamedType)(newType) && oldType.name === newType.name;
-  } else if ((0, _definition.isListType)(oldType)) {
+  if ((0, _definition.isListType)(oldType)) {
     // if they're both lists, make sure the underlying types are compatible
     return (0, _definition.isListType)(newType) && isChangeSafeForInputObjectFieldOrFieldArg(oldType.ofType, newType.ofType);
-  } else if ((0, _definition.isNonNullType)(oldType)) {
+  }
+
+  if ((0, _definition.isNonNullType)(oldType)) {
     return (// if they're both non-null, make sure the underlying types are
       // compatible
       (0, _definition.isNonNullType)(newType) && isChangeSafeForInputObjectFieldOrFieldArg(oldType.ofType, newType.ofType) || // moving from non-null to nullable of the same underlying type is safe
       !(0, _definition.isNonNullType)(newType) && isChangeSafeForInputObjectFieldOrFieldArg(oldType.ofType, newType)
     );
-  }
+  } // if they're both named types, see if their names are equivalent
+
 
-  return false;
+  return (0, _definition.isNamedType)(newType) && oldType.name === newType.name;
 }
 /**
  * Given two schemas, returns an Array containing descriptions of any breaking
@@ -487,10 +458,8 @@ function findTypesRemovedFromUnions(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var typesRemovedFromUnion = [];
 
-  var _arr10 = Object.keys(oldTypeMap);
-
-  for (var _i10 = 0; _i10 < _arr10.length; _i10++) {
-    var typeName = _arr10[_i10];
+  for (var _i10 = 0, _Object$keys10 = Object.keys(oldTypeMap); _i10 < _Object$keys10.length; _i10++) {
+    var typeName = _Object$keys10[_i10];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -567,10 +536,8 @@ function findTypesAddedToUnions(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var typesAddedToUnion = [];
 
-  var _arr11 = Object.keys(newTypeMap);
-
-  for (var _i11 = 0; _i11 < _arr11.length; _i11++) {
-    var typeName = _arr11[_i11];
+  for (var _i11 = 0, _Object$keys11 = Object.keys(newTypeMap); _i11 < _Object$keys11.length; _i11++) {
+    var typeName = _Object$keys11[_i11];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -647,10 +614,8 @@ function findValuesRemovedFromEnums(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var valuesRemovedFromEnums = [];
 
-  var _arr12 = Object.keys(oldTypeMap);
-
-  for (var _i12 = 0; _i12 < _arr12.length; _i12++) {
-    var typeName = _arr12[_i12];
+  for (var _i12 = 0, _Object$keys12 = Object.keys(oldTypeMap); _i12 < _Object$keys12.length; _i12++) {
+    var typeName = _Object$keys12[_i12];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -727,10 +692,8 @@ function findValuesAddedToEnums(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var valuesAddedToEnums = [];
 
-  var _arr13 = Object.keys(oldTypeMap);
-
-  for (var _i13 = 0; _i13 < _arr13.length; _i13++) {
-    var typeName = _arr13[_i13];
+  for (var _i13 = 0, _Object$keys13 = Object.keys(oldTypeMap); _i13 < _Object$keys13.length; _i13++) {
+    var typeName = _Object$keys13[_i13];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -802,10 +765,8 @@ function findInterfacesRemovedFromObjectTypes(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var breakingChanges = [];
 
-  var _arr14 = Object.keys(oldTypeMap);
-
-  for (var _i14 = 0; _i14 < _arr14.length; _i14++) {
-    var typeName = _arr14[_i14];
+  for (var _i14 = 0, _Object$keys14 = Object.keys(oldTypeMap); _i14 < _Object$keys14.length; _i14++) {
+    var typeName = _Object$keys14[_i14];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -860,10 +821,8 @@ function findInterfacesAddedToObjectTypes(oldSchema, newSchema) {
   var newTypeMap = newSchema.getTypeMap();
   var interfacesAddedToObjectTypes = [];
 
-  var _arr15 = Object.keys(newTypeMap);
-
-  for (var _i15 = 0; _i15 < _arr15.length; _i15++) {
-    var typeName = _arr15[_i15];
+  for (var _i15 = 0, _Object$keys15 = Object.keys(newTypeMap); _i15 < _Object$keys15.length; _i15++) {
+    var typeName = _Object$keys15[_i15];
     var oldType = oldTypeMap[typeName];
     var newType = newTypeMap[typeName];
 
@@ -1244,4 +1203,4 @@ function getArgumentMapForDirective(directive) {
   return (0, _keyMap.default)(directive.args, function (arg) {
     return arg.name;
   });
-}
+}

package/utilities/findBreakingChanges.js.flow

@@ -1,5 +1,5 @@
 /**
- * Copyright (c) 2016-present, Facebook, Inc.
+ * Copyright (c) Facebook, Inc. and its affiliates.
  *
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
@@ -8,7 +8,12 @@
  */
 
 import find from '../polyfills/find';
+import inspect from '../jsutils/inspect';
 import {
+  type GraphQLNamedType,
+  type GraphQLFieldMap,
+  type GraphQLType,
+  type GraphQLArgument,
   isScalarType,
   isObjectType,
   isInterfaceType,
@@ -21,20 +26,12 @@ import {
   isRequiredArgument,
   isRequiredInputField,
 } from '../type/definition';
-
-import type {
-  GraphQLNamedType,
-  GraphQLFieldMap,
-  GraphQLType,
-  GraphQLArgument,
-} from '../type/definition';
-
-import type { GraphQLDirective } from '../type/directives';
-import type { GraphQLSchema } from '../type/schema';
+import { type GraphQLDirective } from '../type/directives';
+import { type GraphQLSchema } from '../type/schema';
 import keyMap from '../jsutils/keyMap';
 
-import type { ObjMap } from '../jsutils/ObjMap';
-import type { DirectiveLocationEnum } from '../language/directiveLocation';
+import { type ObjMap } from '../jsutils/ObjMap';
+import { type DirectiveLocationEnum } from '../language/directiveLocation';
 
 export const BreakingChangeType = {
   FIELD_CHANGED_KIND: 'FIELD_CHANGED_KIND',
@@ -120,7 +117,7 @@ export function findDangerousChanges(
  * Given two schemas, returns an Array containing descriptions of any breaking
  * changes in the newSchema related to removing an entire type.
  */
-export function findRemovedTypes(
+function findRemovedTypes(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -143,7 +140,7 @@ export function findRemovedTypes(
  * Given two schemas, returns an Array containing descriptions of any breaking
  * changes in the newSchema related to changing the type of a type.
  */
-export function findTypesThatChangedKind(
+function findTypesThatChangedKind(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -175,7 +172,7 @@ export function findTypesThatChangedKind(
  * (such as removal or change of type of an argument, or a change in an
  * argument's default value).
  */
-export function findArgChanges(
+function findArgChanges(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): {
@@ -296,10 +293,13 @@ function typeKindName(type: GraphQLNamedType): string {
   if (isInputObjectType(type)) {
     return 'an Input type';
   }
-  throw new TypeError('Unknown type ' + type.constructor.name);
+
+  // Not reachable. All possible named types have been considered.
+  /* istanbul ignore next */
+  throw new TypeError(`Unexpected type: ${inspect((type: empty))}.`);
 }
 
-export function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(
+function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -354,7 +354,7 @@ export function findFieldsThatChangedTypeOnObjectOrInterfaceTypes(
   return breakingChanges;
 }
 
-export function findFieldsThatChangedTypeOnInputObjectTypes(
+function findFieldsThatChangedTypeOnInputObjectTypes(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): {
@@ -437,15 +437,7 @@ function isChangeSafeForObjectOrInterfaceField(
   oldType: GraphQLType,
   newType: GraphQLType,
 ): boolean {
-  if (isNamedType(oldType)) {
-    return (
-      // if they're both named types, see if their names are equivalent
-      (isNamedType(newType) && oldType.name === newType.name) ||
-      // moving from nullable to non-null of the same underlying type is safe
-      (isNonNullType(newType) &&
-        isChangeSafeForObjectOrInterfaceField(oldType, newType.ofType))
-    );
-  } else if (isListType(oldType)) {
+  if (isListType(oldType)) {
     return (
       // if they're both lists, make sure the underlying types are compatible
       (isListType(newType) &&
@@ -457,30 +449,38 @@ function isChangeSafeForObjectOrInterfaceField(
       (isNonNullType(newType) &&
         isChangeSafeForObjectOrInterfaceField(oldType, newType.ofType))
     );
-  } else if (isNonNullType(oldType)) {
+  }
+
+  if (isNonNullType(oldType)) {
     // if they're both non-null, make sure the underlying types are compatible
     return (
       isNonNullType(newType) &&
       isChangeSafeForObjectOrInterfaceField(oldType.ofType, newType.ofType)
     );
   }
-  return false;
+
+  return (
+    // if they're both named types, see if their names are equivalent
+    (isNamedType(newType) && oldType.name === newType.name) ||
+    // moving from nullable to non-null of the same underlying type is safe
+    (isNonNullType(newType) &&
+      isChangeSafeForObjectOrInterfaceField(oldType, newType.ofType))
+  );
 }
 
 function isChangeSafeForInputObjectFieldOrFieldArg(
   oldType: GraphQLType,
   newType: GraphQLType,
 ): boolean {
-  if (isNamedType(oldType)) {
-    // if they're both named types, see if their names are equivalent
-    return isNamedType(newType) && oldType.name === newType.name;
-  } else if (isListType(oldType)) {
+  if (isListType(oldType)) {
     // if they're both lists, make sure the underlying types are compatible
     return (
       isListType(newType) &&
       isChangeSafeForInputObjectFieldOrFieldArg(oldType.ofType, newType.ofType)
     );
-  } else if (isNonNullType(oldType)) {
+  }
+
+  if (isNonNullType(oldType)) {
     return (
       // if they're both non-null, make sure the underlying types are
       // compatible
@@ -494,14 +494,16 @@ function isChangeSafeForInputObjectFieldOrFieldArg(
         isChangeSafeForInputObjectFieldOrFieldArg(oldType.ofType, newType))
     );
   }
-  return false;
+
+  // if they're both named types, see if their names are equivalent
+  return isNamedType(newType) && oldType.name === newType.name;
 }
 
 /**
  * Given two schemas, returns an Array containing descriptions of any breaking
  * changes in the newSchema related to removing types from a union type.
  */
-export function findTypesRemovedFromUnions(
+function findTypesRemovedFromUnions(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -535,7 +537,7 @@ export function findTypesRemovedFromUnions(
  * Given two schemas, returns an Array containing descriptions of any dangerous
  * changes in the newSchema related to adding types to a union type.
  */
-export function findTypesAddedToUnions(
+function findTypesAddedToUnions(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<DangerousChange> {
@@ -568,7 +570,7 @@ export function findTypesAddedToUnions(
  * Given two schemas, returns an Array containing descriptions of any breaking
  * changes in the newSchema related to removing values from an enum type.
  */
-export function findValuesRemovedFromEnums(
+function findValuesRemovedFromEnums(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -602,7 +604,7 @@ export function findValuesRemovedFromEnums(
  * Given two schemas, returns an Array containing descriptions of any dangerous
  * changes in the newSchema related to adding values to an enum type.
  */
-export function findValuesAddedToEnums(
+function findValuesAddedToEnums(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<DangerousChange> {
@@ -633,7 +635,7 @@ export function findValuesAddedToEnums(
   return valuesAddedToEnums;
 }
 
-export function findInterfacesRemovedFromObjectTypes(
+function findInterfacesRemovedFromObjectTypes(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -664,7 +666,7 @@ export function findInterfacesRemovedFromObjectTypes(
   return breakingChanges;
 }
 
-export function findInterfacesAddedToObjectTypes(
+function findInterfacesAddedToObjectTypes(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<DangerousChange> {
@@ -695,7 +697,7 @@ export function findInterfacesAddedToObjectTypes(
   return interfacesAddedToObjectTypes;
 }
 
-export function findRemovedDirectives(
+function findRemovedDirectives(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -730,7 +732,7 @@ function findRemovedArgsForDirective(
   return removedArgs;
 }
 
-export function findRemovedDirectiveArgs(
+function findRemovedDirectiveArgs(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -770,7 +772,7 @@ function findAddedArgsForDirective(
   return addedArgs;
 }
 
-export function findAddedNonNullDirectiveArgs(
+function findAddedNonNullDirectiveArgs(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {
@@ -798,7 +800,7 @@ export function findAddedNonNullDirectiveArgs(
   return addedNonNullableArgs;
 }
 
-export function findRemovedLocationsForDirective(
+function findRemovedLocationsForDirective(
   oldDirective: GraphQLDirective,
   newDirective: GraphQLDirective,
 ): Array<DirectiveLocationEnum> {
@@ -814,7 +816,7 @@ export function findRemovedLocationsForDirective(
   return removedLocations;
 }
 
-export function findRemovedDirectiveLocations(
+function findRemovedDirectiveLocations(
   oldSchema: GraphQLSchema,
   newSchema: GraphQLSchema,
 ): Array<BreakingChange> {