Use createGraphQLError

Author: ardatan

.changeset/stupid-webs-stand.md

@@ -0,0 +1,5 @@
+---
+'@graphql-tools/import': patch
+---
+
+enhance GraphQL schema import error debugging with source locations

packages/import/src/index.ts

@@ -40,7 +40,7 @@ import {
   UnionTypeExtensionNode,
 } from 'graphql';
 import resolveFrom from 'resolve-from';
-import { parseGraphQLSDL } from '@graphql-tools/utils';
+import { createGraphQLError, parseGraphQLSDL } from '@graphql-tools/utils';
 import { extractLinkImplementations } from '@theguild/federation-composition';
 
 const builtinTypes = ['String', 'Float', 'Int', 'Boolean', 'ID', 'Upload'];
@@ -287,9 +287,9 @@ function visitFile(
                   !allImportedDefinitionsMap.has(dependencyName) &&
                   !definitionsByName.has(dependencyName)
                 ) {
-                  throw new GraphQLError(
+                  throw createGraphQLError(
                     `Couldn't find type ${dependencyName} in any of the schemas.`,
-                    Array.from(dependencyNodes),
+                    { nodes: Array.from(dependencyNodes) },
                   );
                 }
                 const dependencyDefinitionsFromImports =

packages/utils/tests/mergeIncrementalResult.spec.ts

@@ -102,43 +102,43 @@ describe('mergeIncrementalResult', () => {
   it('should add errors', () => {
     const executionResult = { data: { user: { name: 'John' } } };
     const incrementalResult = {
-      errors: [new GraphQLError('error 1'), new GraphQLError('error 2')],
+      errors: [createGraphQLError('error 1'), createGraphQLError('error 2')],
     };
 
     mergeIncrementalResult({ incrementalResult, executionResult });
 
     expect(executionResult).toEqual({
       data: { user: { name: 'John' } },
-      errors: [new GraphQLError('error 1'), new GraphQLError('error 2')],
+      errors: [createGraphQLError('error 1'), createGraphQLError('error 2')],
     });
   });
 
   it('should keep errors', () => {
-    const executionResult = { errors: [new GraphQLError('error 1')] };
+    const executionResult = { errors: [createGraphQLError('error 1')] };
     const incrementalResult = { data: { user: { name: 'John' } }, path: [] };
 
     mergeIncrementalResult({ incrementalResult, executionResult });
 
     expect(executionResult).toEqual({
       data: { user: { name: 'John' } },
-      errors: [new GraphQLError('error 1')],
+      errors: [createGraphQLError('error 1')],
     });
   });
 
   it('should merge errors', () => {
-    const executionResult = { errors: [new GraphQLError('error 1')] };
+    const executionResult = { errors: [createGraphQLError('error 1')] };
 
     const incrementalResult = {
-      errors: [new GraphQLError('error 2'), new GraphQLError('error 3')],
+      errors: [createGraphQLError('error 2'), createGraphQLError('error 3')],
     };
 
     mergeIncrementalResult({ incrementalResult, executionResult });
 
     expect(executionResult).toEqual({
       errors: [
-        new GraphQLError('error 1'),
-        new GraphQLError('error 2'),
-        new GraphQLError('error 3'),
+        createGraphQLError('error 1'),
+        createGraphQLError('error 2'),
+        createGraphQLError('error 3'),
       ],
     });
   });