@graphql-eslint/eslint-plugin 3.3.0 → 3.4.0-alpha-d977284.0

package/README.md

@@ -42,7 +42,7 @@ npm install --save-dev @graphql-eslint/eslint-plugin
 
 > Make sure you have `graphql` dependency in your project.
 
-### Configuration
+## Configuration
 
 To get started, define an override in your ESLint config to apply this plugin to `.graphql` files. Add the [rules](docs/README.md) you want applied.
 
@@ -65,7 +65,7 @@ To get started, define an override in your ESLint config to apply this plugin to
 
 If your GraphQL definitions are defined only in `.graphql` files, and you're only using rules that apply to individual files, you should be good to go 👍. If you would like use a remote schema or use rules that apply across the entire collection of definitions at once, see [here](#using-a-remote-schema-or-rules-with-constraints-that-span-the-entire-schema).
 
-#### Tell ESLint to apply this plugin to GraphQL definitions defined in code files
+### Apply this plugin to GraphQL definitions defined in code files
 
 If you are defining GraphQL schema or GraphQL operations in code files, you'll want to define an additional override to extend the functionality of this plugin to the schema and operations in those files.
 
@@ -90,7 +90,7 @@ If you are defining GraphQL schema or GraphQL operations in code files, you'll w
 
 Under the hood, specifying the `@graphql-eslint/graphql` processor for code files will cause `graphql-eslint/graphql` to extract the schema and operation definitions from these files into virtual GraphQL documents with `.graphql` extensions. This will allow the overrides you've defined for `.graphql` files, via `"files": ["*.graphql"]`, to get applied to the definitions defined in your code files.
 
-#### Using a remote schema or rules with constraints that span the entire schema
+### Extended linting rules with GraphQL Schema
 
 Some rules require an understanding of the entire schema at once. For example, [no-unreachable-types](https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/no-unreachable-types.md#no-unreachable-types) checks that all types are reachable by root-level fields.
 
@@ -120,7 +120,7 @@ The parser allows you to specify a json file / graphql files(s) / url / raw stri
 
 > Some rules require type information to operate, it's marked in the docs for each rule!
 
-#### Extended linting rules with siblings operations
+### Extended linting rules with siblings operations
 
 While implementing this tool, we had to find solutions for a better integration of the GraphQL ecosystem and ESLint core.
 
@@ -180,7 +180,7 @@ You can find a list of [ESLint directives here](https://eslint.org/docs/2.13.1/u
 
 You can find a complete list of [all available rules here](docs/README.md).
 
-## Deprecated Rules
+### Deprecated Rules
 
 See [docs/deprecated-rules.md](docs/deprecated-rules.md).
 
@@ -201,7 +201,7 @@ See [docs/deprecated-rules.md](docs/deprecated-rules.md).
 
 > If you are in a monorepo project, you probably need both sets of rules.
 
-## Config usage
+### Config usage
 
 For example, to enable the `schema-recommended` config, enable it in your `.eslintrc` file with the `extends` option:
 

package/docs/rules/possible-type-extension.md

@@ -2,7 +2,7 @@
 
 - Category: `Schema`
 - Rule name: `@graphql-eslint/possible-type-extension`
-- Requires GraphQL Schema: `false` [ℹ️](../../README.md#extended-linting-rules-with-graphql-schema)
+- Requires GraphQL Schema: `true` [ℹ️](../../README.md#extended-linting-rules-with-graphql-schema)
 - Requires GraphQL Operations: `false` [ℹ️](../../README.md#extended-linting-rules-with-siblings-operations)
 
 A type extension is only valid if the type is defined and has the same kind.

package/graphql-config.d.ts

@@ -1,3 +1,3 @@
 import { GraphQLConfig } from 'graphql-config';
 import { ParserOptions } from './types';
-export declare function loadGraphqlConfig(options: ParserOptions): GraphQLConfig;
+export declare function loadGraphQLConfig(options: ParserOptions): GraphQLConfig;

package/index.js

@@ -10,9 +10,10 @@ const fs = require('fs');
 const path = require('path');
 const utils = require('@graphql-tools/utils');
 const lowerCase = _interopDefault(require('lodash.lowercase'));
+const chalk = _interopDefault(require('chalk'));
 const depthLimit = _interopDefault(require('graphql-depth-limit'));
 const graphqlTagPluck = require('@graphql-tools/graphql-tag-pluck');
-const graphqlConfig$1 = require('graphql-config');
+const graphqlConfig = require('graphql-config');
 const codeFileLoader = require('@graphql-tools/code-file-loader');
 const eslint = require('eslint');
 const codeFrame = require('@babel/code-frame');
@@ -192,6 +193,12 @@ function requireGraphQLSchemaFromContext(ruleName, context) {
     }
     return context.parserServices.schema;
 }
+const logger = {
+    // eslint-disable-next-line no-console
+    error: (...args) => console.error(chalk.red('error'), '[graphql-eslint]', chalk(...args)),
+    // eslint-disable-next-line no-console
+    warn: (...args) => console.warn(chalk.yellow('warning'), '[graphql-eslint]', chalk(...args)),
+};
 function requireReachableTypesFromContext(ruleName, context) {
     const schema = requireGraphQLSchemaFromContext(ruleName, context);
     return context.parserServices.reachableTypes(schema);
@@ -326,14 +333,14 @@ function getLocation(loc, fieldName = '', offset) {
     };
 }
 
-function validateDocument(sourceNode, context, schema, documentNode, rule) {
+function validateDocument(context, schema = null, documentNode, rule) {
     if (documentNode.definitions.length === 0) {
         return;
     }
     try {
         const validationErrors = schema
             ? graphql.validate(schema, documentNode, [rule])
-            : validate.validateSDL(documentNode, null, [rule]);
+            : validate.validateSDL(documentNode, schema, [rule]);
         for (const error of validationErrors) {
             /*
              * TODO: Fix ESTree-AST converter because currently it's incorrectly convert loc.end
@@ -358,7 +365,8 @@ function validateDocument(sourceNode, context, schema, documentNode, rule) {
     }
     catch (e) {
         context.report({
-            node: sourceNode,
+            // Report on first character
+            loc: { column: 0, line: 1 },
             message: e.message,
         });
     }
@@ -440,18 +448,17 @@ const validationToRule = (ruleId, ruleName, docs, getDocumentNode) => {
                 },
             },
             create(context) {
+                if (!ruleFn) {
+                    logger.warn(`You rule "${ruleId}" depends on a GraphQL validation rule "${ruleName}" but it's not available in the "graphql-js" version you are using. Skipping...`);
+                    return {};
+                }
                 return {
                     Document(node) {
-                        if (!ruleFn) {
-                            // eslint-disable-next-line no-console
-                            console.warn(`You rule "${ruleId}" depends on a GraphQL validation rule "${ruleName}" but it's not available in the "graphql-js" version you are using. Skipping...`);
-                            return;
-                        }
                         const schema = docs.requiresSchema ? requireGraphQLSchemaFromContext(ruleId, context) : null;
                         const documentNode = getDocumentNode
                             ? getDocumentNode({ ruleId, context, schema, node: node.rawNode() })
                             : node.rawNode();
-                        validateDocument(node, context, schema, documentNode, ruleFn);
+                        validateDocument(context, schema, documentNode, ruleFn);
                     },
                 };
             },
@@ -601,7 +608,8 @@ const GRAPHQL_JS_VALIDATIONS = Object.assign({}, validationToRule('executable-de
 }), validationToRule('possible-type-extension', 'PossibleTypeExtensions', {
     category: 'Schema',
     description: `A type extension is only valid if the type is defined and has the same kind.`,
-    recommended: false, // TODO: enable after https://github.com/dotansimha/graphql-eslint/issues/787 will be fixed
+    recommended: false,
+    requiresSchema: true,
 }), validationToRule('provided-required-arguments', 'ProvidedRequiredArguments', {
     category: ['Schema', 'Operations'],
     description: `A field or directive is only valid if all required (non-null without a default value) field arguments have been provided.`,
@@ -3034,12 +3042,13 @@ const rule$j = {
     },
 };
 
+const RULE_ID$2 = 'selection-set-depth';
 const rule$k = {
     meta: {
         docs: {
             category: 'Operations',
             description: `Limit the complexity of the GraphQL operations solely by their depth. Based on [graphql-depth-limit](https://github.com/stems/graphql-depth-limit).`,
-            url: 'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/selection-set-depth.md',
+            url: `https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/${RULE_ID$2}.md`,
             requiresSiblings: true,
             examples: [
                 {
@@ -3113,11 +3122,10 @@ const rule$k = {
     create(context) {
         let siblings = null;
         try {
-            siblings = requireSiblingsOperations('selection-set-depth', context);
+            siblings = requireSiblingsOperations(RULE_ID$2, context);
         }
         catch (e) {
-            // eslint-disable-next-line no-console
-            console.warn(`Rule "selection-set-depth" works best with siblings operations loaded. For more info: http://bit.ly/graphql-eslint-operations`);
+            logger.warn(`Rule "${RULE_ID$2}" works best with siblings operations loaded. For more info: http://bit.ly/graphql-eslint-operations`);
         }
         const { maxDepth } = context.options[0];
         const ignore = context.options[0].ignore || [];
@@ -3142,8 +3150,7 @@ const rule$k = {
                     });
                 }
                 catch (e) {
-                    // eslint-disable-next-line no-console
-                    console.warn(`Rule "selection-set-depth" check failed due to a missing siblings operations. For more info: http://bit.ly/graphql-eslint-operations`, e);
+                    logger.warn(`Rule "${RULE_ID$2}" check failed due to a missing siblings operations. For more info: http://bit.ly/graphql-eslint-operations`, e);
                 }
             },
         };
@@ -3525,8 +3532,7 @@ function createGraphqlProcessor() {
                     }
                 }
                 catch (e) {
-                    // eslint-disable-next-line no-console
-                    console.warn(`[graphql-eslint/processor]: Unable to process file "${filename}": `, e);
+                    logger.warn(`Unable to process file "${filename}"`, e);
                 }
             }
             return [text];
@@ -3566,20 +3572,25 @@ const schemaCache = new Map();
 function getSchema(options = {}, gqlConfig) {
     const realFilepath = options.filePath ? getOnDiskFilepath(options.filePath) : null;
     const projectForFile = realFilepath ? gqlConfig.getProjectForFile(realFilepath) : gqlConfig.getDefault();
-    const schemaKey = utils.asArray(projectForFile.schema)
-        .sort()
-        .join(',');
+    const schemaKey = utils.asArray(projectForFile.schema).sort().join(',');
     if (!schemaKey) {
         return null;
     }
-    let schema = schemaCache.get(schemaKey);
-    if (!schema) {
+    if (schemaCache.has(schemaKey)) {
+        return schemaCache.get(schemaKey);
+    }
+    let schema;
+    try {
         schema = projectForFile.loadSchemaSync(projectForFile.schema, 'GraphQLSchema', {
             cache: loaderCache,
-            ...options.schemaOptions
+            ...options.schemaOptions,
         });
-        schemaCache.set(schemaKey, schema);
     }
+    catch (e) {
+        schema = null;
+        logger.error('Error while loading schema\n', e);
+    }
+    schemaCache.set(schemaKey, schema);
     return schema;
 }
 
@@ -3592,10 +3603,10 @@ const handleVirtualPath = (documents) => {
             return source;
         }
         (_a = filepathMap[location]) !== null && _a !== void 0 ? _a : (filepathMap[location] = -1);
-        const index = filepathMap[location] += 1;
+        const index = (filepathMap[location] += 1);
         return {
             ...source,
-            location: path.resolve(location, `${index}_document.graphql`)
+            location: path.resolve(location, `${index}_document.graphql`),
         };
     });
 };
@@ -3604,9 +3615,7 @@ const siblingOperationsCache = new Map();
 const getSiblings = (filePath, gqlConfig) => {
     const realFilepath = filePath ? getOnDiskFilepath(filePath) : null;
     const projectForFile = realFilepath ? gqlConfig.getProjectForFile(realFilepath) : gqlConfig.getDefault();
-    const documentsKey = utils.asArray(projectForFile.documents)
-        .sort()
-        .join(',');
+    const documentsKey = utils.asArray(projectForFile.documents).sort().join(',');
     if (!documentsKey) {
         return [];
     }
@@ -3614,7 +3623,7 @@ const getSiblings = (filePath, gqlConfig) => {
     if (!siblings) {
         const documents = projectForFile.loadDocumentsSync(projectForFile.documents, {
             skipGraphQLImport: true,
-            cache: loaderCache
+            cache: loaderCache,
         });
         siblings = handleVirtualPath(documents);
         operationsCache.set(documentsKey, siblings);
@@ -3627,8 +3636,7 @@ function getSiblingOperations(options, gqlConfig) {
         let printed = false;
         const noopWarn = () => {
             if (!printed) {
-                // eslint-disable-next-line no-console
-                console.warn(`getSiblingOperations was called without any operations. Make sure to set "parserOptions.operations" to make this feature available!`);
+                logger.warn(`getSiblingOperations was called without any operations. Make sure to set "parserOptions.operations" to make this feature available!`);
                 printed = true;
             }
             return [];
@@ -3692,8 +3700,7 @@ function getSiblingOperations(options, gqlConfig) {
                     const name = spread.name.value;
                     const fragmentInfo = getFragment(name);
                     if (fragmentInfo.length === 0) {
-                        // eslint-disable-next-line no-console
-                        console.warn(`Unable to locate fragment named "${name}", please make sure it's loaded using "parserOptions.operations"`);
+                        logger.warn(`Unable to locate fragment named "${name}", please make sure it's loaded using "parserOptions.operations"`);
                         return;
                     }
                     const fragment = fragmentInfo[0];
@@ -3723,35 +3730,36 @@ function getSiblingOperations(options, gqlConfig) {
     return siblingOperations;
 }
 
-let graphqlConfig;
-function loadGraphqlConfig(options) {
+let graphQLConfig;
+function loadGraphQLConfig(options) {
     // We don't want cache config on test environment
     // Otherwise schema and documents will be same for all tests
-    if (process.env.NODE_ENV !== 'test' && graphqlConfig) {
-        return graphqlConfig;
+    if (process.env.NODE_ENV !== 'test' && graphQLConfig) {
+        return graphQLConfig;
     }
     const onDiskConfig = options.skipGraphQLConfig
         ? null
-        : graphqlConfig$1.loadConfigSync({
+        : graphqlConfig.loadConfigSync({
             throwOnEmpty: false,
             throwOnMissing: false,
             extensions: [addCodeFileLoaderExtension],
         });
-    graphqlConfig =
+    const configOptions = options.projects
+        ? { projects: options.projects }
+        : {
+            schema: (options.schema || ''),
+            documents: options.documents || options.operations,
+            extensions: options.extensions,
+            include: options.include,
+            exclude: options.exclude,
+        };
+    graphQLConfig =
         onDiskConfig ||
-            new graphqlConfig$1.GraphQLConfig({
-                config: options.projects
-                    ? { projects: options.projects }
-                    : {
-                        schema: (options.schema || ''),
-                        documents: options.documents || options.operations,
-                        extensions: options.extensions,
-                        include: options.include,
-                        exclude: options.exclude,
-                    },
+            new graphqlConfig.GraphQLConfig({
+                config: configOptions,
                 filepath: 'virtual-config',
             }, [addCodeFileLoaderExtension]);
-    return graphqlConfig;
+    return graphQLConfig;
 }
 const addCodeFileLoaderExtension = api => {
     api.loaders.schema.register(new codeFileLoader.CodeFileLoader());
@@ -3841,7 +3849,7 @@ function parse(code, options) {
     return parseForESLint(code, options).ast;
 }
 function parseForESLint(code, options = {}) {
-    const gqlConfig = loadGraphqlConfig(options);
+    const gqlConfig = loadGraphQLConfig(options);
     const schema = getSchema(options, gqlConfig);
     const parserServices = {
         hasTypeInfo: schema !== null,
@@ -3873,6 +3881,7 @@ function parseForESLint(code, options = {}) {
         };
     }
     catch (e) {
+        e.message = `[graphql-eslint] ${e.message}`;
         // In case of GraphQL parser error, we report it to ESLint as a parser error that matches the requirements
         // of ESLint. This will make sure to display it correctly in IDEs and lint results.
         if (e instanceof graphql.GraphQLError) {
@@ -3880,11 +3889,10 @@ function parseForESLint(code, options = {}) {
                 index: e.positions[0],
                 lineNumber: e.locations[0].line,
                 column: e.locations[0].column,
-                message: `[graphql-eslint]: ${e.message}`,
+                message: e.message,
             };
             throw eslintError;
         }
-        e.message = `[graphql-eslint]: ${e.message}`;
         throw e;
     }
 }

package/index.mjs

@@ -4,6 +4,7 @@ import { statSync, existsSync, readFileSync } from 'fs';
 import { dirname, extname, basename, relative, resolve } from 'path';
 import { asArray, parseGraphQLSDL } from '@graphql-tools/utils';
 import lowerCase from 'lodash.lowercase';
+import chalk from 'chalk';
 import depthLimit from 'graphql-depth-limit';
 import { parseCode } from '@graphql-tools/graphql-tag-pluck';
 import { loadConfigSync, GraphQLConfig } from 'graphql-config';
@@ -186,6 +187,12 @@ function requireGraphQLSchemaFromContext(ruleName, context) {
     }
     return context.parserServices.schema;
 }
+const logger = {
+    // eslint-disable-next-line no-console
+    error: (...args) => console.error(chalk.red('error'), '[graphql-eslint]', chalk(...args)),
+    // eslint-disable-next-line no-console
+    warn: (...args) => console.warn(chalk.yellow('warning'), '[graphql-eslint]', chalk(...args)),
+};
 function requireReachableTypesFromContext(ruleName, context) {
     const schema = requireGraphQLSchemaFromContext(ruleName, context);
     return context.parserServices.reachableTypes(schema);
@@ -320,14 +327,14 @@ function getLocation(loc, fieldName = '', offset) {
     };
 }
 
-function validateDocument(sourceNode, context, schema, documentNode, rule) {
+function validateDocument(context, schema = null, documentNode, rule) {
     if (documentNode.definitions.length === 0) {
         return;
     }
     try {
         const validationErrors = schema
             ? validate(schema, documentNode, [rule])
-            : validateSDL(documentNode, null, [rule]);
+            : validateSDL(documentNode, schema, [rule]);
         for (const error of validationErrors) {
             /*
              * TODO: Fix ESTree-AST converter because currently it's incorrectly convert loc.end
@@ -352,7 +359,8 @@ function validateDocument(sourceNode, context, schema, documentNode, rule) {
     }
     catch (e) {
         context.report({
-            node: sourceNode,
+            // Report on first character
+            loc: { column: 0, line: 1 },
             message: e.message,
         });
     }
@@ -434,18 +442,17 @@ const validationToRule = (ruleId, ruleName, docs, getDocumentNode) => {
                 },
             },
             create(context) {
+                if (!ruleFn) {
+                    logger.warn(`You rule "${ruleId}" depends on a GraphQL validation rule "${ruleName}" but it's not available in the "graphql-js" version you are using. Skipping...`);
+                    return {};
+                }
                 return {
                     Document(node) {
-                        if (!ruleFn) {
-                            // eslint-disable-next-line no-console
-                            console.warn(`You rule "${ruleId}" depends on a GraphQL validation rule "${ruleName}" but it's not available in the "graphql-js" version you are using. Skipping...`);
-                            return;
-                        }
                         const schema = docs.requiresSchema ? requireGraphQLSchemaFromContext(ruleId, context) : null;
                         const documentNode = getDocumentNode
                             ? getDocumentNode({ ruleId, context, schema, node: node.rawNode() })
                             : node.rawNode();
-                        validateDocument(node, context, schema, documentNode, ruleFn);
+                        validateDocument(context, schema, documentNode, ruleFn);
                     },
                 };
             },
@@ -595,7 +602,8 @@ const GRAPHQL_JS_VALIDATIONS = Object.assign({}, validationToRule('executable-de
 }), validationToRule('possible-type-extension', 'PossibleTypeExtensions', {
     category: 'Schema',
     description: `A type extension is only valid if the type is defined and has the same kind.`,
-    recommended: false, // TODO: enable after https://github.com/dotansimha/graphql-eslint/issues/787 will be fixed
+    recommended: false,
+    requiresSchema: true,
 }), validationToRule('provided-required-arguments', 'ProvidedRequiredArguments', {
     category: ['Schema', 'Operations'],
     description: `A field or directive is only valid if all required (non-null without a default value) field arguments have been provided.`,
@@ -3028,12 +3036,13 @@ const rule$j = {
     },
 };
 
+const RULE_ID$2 = 'selection-set-depth';
 const rule$k = {
     meta: {
         docs: {
             category: 'Operations',
             description: `Limit the complexity of the GraphQL operations solely by their depth. Based on [graphql-depth-limit](https://github.com/stems/graphql-depth-limit).`,
-            url: 'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/selection-set-depth.md',
+            url: `https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/${RULE_ID$2}.md`,
             requiresSiblings: true,
             examples: [
                 {
@@ -3107,11 +3116,10 @@ const rule$k = {
     create(context) {
         let siblings = null;
         try {
-            siblings = requireSiblingsOperations('selection-set-depth', context);
+            siblings = requireSiblingsOperations(RULE_ID$2, context);
         }
         catch (e) {
-            // eslint-disable-next-line no-console
-            console.warn(`Rule "selection-set-depth" works best with siblings operations loaded. For more info: http://bit.ly/graphql-eslint-operations`);
+            logger.warn(`Rule "${RULE_ID$2}" works best with siblings operations loaded. For more info: http://bit.ly/graphql-eslint-operations`);
         }
         const { maxDepth } = context.options[0];
         const ignore = context.options[0].ignore || [];
@@ -3136,8 +3144,7 @@ const rule$k = {
                     });
                 }
                 catch (e) {
-                    // eslint-disable-next-line no-console
-                    console.warn(`Rule "selection-set-depth" check failed due to a missing siblings operations. For more info: http://bit.ly/graphql-eslint-operations`, e);
+                    logger.warn(`Rule "${RULE_ID$2}" check failed due to a missing siblings operations. For more info: http://bit.ly/graphql-eslint-operations`, e);
                 }
             },
         };
@@ -3519,8 +3526,7 @@ function createGraphqlProcessor() {
                     }
                 }
                 catch (e) {
-                    // eslint-disable-next-line no-console
-                    console.warn(`[graphql-eslint/processor]: Unable to process file "${filename}": `, e);
+                    logger.warn(`Unable to process file "${filename}"`, e);
                 }
             }
             return [text];
@@ -3560,20 +3566,25 @@ const schemaCache = new Map();
 function getSchema(options = {}, gqlConfig) {
     const realFilepath = options.filePath ? getOnDiskFilepath(options.filePath) : null;
     const projectForFile = realFilepath ? gqlConfig.getProjectForFile(realFilepath) : gqlConfig.getDefault();
-    const schemaKey = asArray(projectForFile.schema)
-        .sort()
-        .join(',');
+    const schemaKey = asArray(projectForFile.schema).sort().join(',');
     if (!schemaKey) {
         return null;
     }
-    let schema = schemaCache.get(schemaKey);
-    if (!schema) {
+    if (schemaCache.has(schemaKey)) {
+        return schemaCache.get(schemaKey);
+    }
+    let schema;
+    try {
         schema = projectForFile.loadSchemaSync(projectForFile.schema, 'GraphQLSchema', {
             cache: loaderCache,
-            ...options.schemaOptions
+            ...options.schemaOptions,
         });
-        schemaCache.set(schemaKey, schema);
     }
+    catch (e) {
+        schema = null;
+        logger.error('Error while loading schema\n', e);
+    }
+    schemaCache.set(schemaKey, schema);
     return schema;
 }
 
@@ -3586,10 +3597,10 @@ const handleVirtualPath = (documents) => {
             return source;
         }
         (_a = filepathMap[location]) !== null && _a !== void 0 ? _a : (filepathMap[location] = -1);
-        const index = filepathMap[location] += 1;
+        const index = (filepathMap[location] += 1);
         return {
             ...source,
-            location: resolve(location, `${index}_document.graphql`)
+            location: resolve(location, `${index}_document.graphql`),
         };
     });
 };
@@ -3598,9 +3609,7 @@ const siblingOperationsCache = new Map();
 const getSiblings = (filePath, gqlConfig) => {
     const realFilepath = filePath ? getOnDiskFilepath(filePath) : null;
     const projectForFile = realFilepath ? gqlConfig.getProjectForFile(realFilepath) : gqlConfig.getDefault();
-    const documentsKey = asArray(projectForFile.documents)
-        .sort()
-        .join(',');
+    const documentsKey = asArray(projectForFile.documents).sort().join(',');
     if (!documentsKey) {
         return [];
     }
@@ -3608,7 +3617,7 @@ const getSiblings = (filePath, gqlConfig) => {
     if (!siblings) {
         const documents = projectForFile.loadDocumentsSync(projectForFile.documents, {
             skipGraphQLImport: true,
-            cache: loaderCache
+            cache: loaderCache,
         });
         siblings = handleVirtualPath(documents);
         operationsCache.set(documentsKey, siblings);
@@ -3621,8 +3630,7 @@ function getSiblingOperations(options, gqlConfig) {
         let printed = false;
         const noopWarn = () => {
             if (!printed) {
-                // eslint-disable-next-line no-console
-                console.warn(`getSiblingOperations was called without any operations. Make sure to set "parserOptions.operations" to make this feature available!`);
+                logger.warn(`getSiblingOperations was called without any operations. Make sure to set "parserOptions.operations" to make this feature available!`);
                 printed = true;
             }
             return [];
@@ -3686,8 +3694,7 @@ function getSiblingOperations(options, gqlConfig) {
                     const name = spread.name.value;
                     const fragmentInfo = getFragment(name);
                     if (fragmentInfo.length === 0) {
-                        // eslint-disable-next-line no-console
-                        console.warn(`Unable to locate fragment named "${name}", please make sure it's loaded using "parserOptions.operations"`);
+                        logger.warn(`Unable to locate fragment named "${name}", please make sure it's loaded using "parserOptions.operations"`);
                         return;
                     }
                     const fragment = fragmentInfo[0];
@@ -3717,12 +3724,12 @@ function getSiblingOperations(options, gqlConfig) {
     return siblingOperations;
 }
 
-let graphqlConfig;
-function loadGraphqlConfig(options) {
+let graphQLConfig;
+function loadGraphQLConfig(options) {
     // We don't want cache config on test environment
     // Otherwise schema and documents will be same for all tests
-    if (process.env.NODE_ENV !== 'test' && graphqlConfig) {
-        return graphqlConfig;
+    if (process.env.NODE_ENV !== 'test' && graphQLConfig) {
+        return graphQLConfig;
     }
     const onDiskConfig = options.skipGraphQLConfig
         ? null
@@ -3731,21 +3738,22 @@ function loadGraphqlConfig(options) {
             throwOnMissing: false,
             extensions: [addCodeFileLoaderExtension],
         });
-    graphqlConfig =
+    const configOptions = options.projects
+        ? { projects: options.projects }
+        : {
+            schema: (options.schema || ''),
+            documents: options.documents || options.operations,
+            extensions: options.extensions,
+            include: options.include,
+            exclude: options.exclude,
+        };
+    graphQLConfig =
         onDiskConfig ||
             new GraphQLConfig({
-                config: options.projects
-                    ? { projects: options.projects }
-                    : {
-                        schema: (options.schema || ''),
-                        documents: options.documents || options.operations,
-                        extensions: options.extensions,
-                        include: options.include,
-                        exclude: options.exclude,
-                    },
+                config: configOptions,
                 filepath: 'virtual-config',
             }, [addCodeFileLoaderExtension]);
-    return graphqlConfig;
+    return graphQLConfig;
 }
 const addCodeFileLoaderExtension = api => {
     api.loaders.schema.register(new CodeFileLoader());
@@ -3835,7 +3843,7 @@ function parse(code, options) {
     return parseForESLint(code, options).ast;
 }
 function parseForESLint(code, options = {}) {
-    const gqlConfig = loadGraphqlConfig(options);
+    const gqlConfig = loadGraphQLConfig(options);
     const schema = getSchema(options, gqlConfig);
     const parserServices = {
         hasTypeInfo: schema !== null,
@@ -3867,6 +3875,7 @@ function parseForESLint(code, options = {}) {
         };
     }
     catch (e) {
+        e.message = `[graphql-eslint] ${e.message}`;
         // In case of GraphQL parser error, we report it to ESLint as a parser error that matches the requirements
         // of ESLint. This will make sure to display it correctly in IDEs and lint results.
         if (e instanceof GraphQLError) {
@@ -3874,11 +3883,10 @@ function parseForESLint(code, options = {}) {
                 index: e.positions[0],
                 lineNumber: e.locations[0].line,
                 column: e.locations[0].column,
-                message: `[graphql-eslint]: ${e.message}`,
+                message: e.message,
             };
             throw eslintError;
         }
-        e.message = `[graphql-eslint]: ${e.message}`;
         throw e;
     }
 }

package/package.json

@@ -1,16 +1,17 @@
 {
   "name": "@graphql-eslint/eslint-plugin",
-  "version": "3.3.0",
+  "version": "3.4.0-alpha-d977284.0",
   "description": "GraphQL plugin for ESLint",
   "sideEffects": false,
   "peerDependencies": {
     "graphql": "^0.8.0 || ^0.9.0 || ^0.10.0 || ^0.11.0 || ^0.12.0 || ^0.13.0 || ^14.0.0 || ^15.0.0 || ^16.0.0"
   },
   "dependencies": {
-    "@babel/code-frame": "7.16.0",
+    "@babel/code-frame": "7.16.7",
     "@graphql-tools/code-file-loader": "7.2.3",
     "@graphql-tools/graphql-tag-pluck": "7.1.4",
     "@graphql-tools/utils": "8.5.5",
+    "chalk": "4.1.2",
     "graphql-config": "4.1.0",
     "graphql-depth-limit": "1.1.0",
     "lodash.lowercase": "4.3.0"

package/utils.d.ts

@@ -6,6 +6,10 @@ import { SiblingOperations } from './sibling-operations';
 import { UsedFields, ReachableTypes } from './graphql-ast';
 export declare function requireSiblingsOperations(ruleName: string, context: GraphQLESLintRuleContext): SiblingOperations | never;
 export declare function requireGraphQLSchemaFromContext(ruleName: string, context: GraphQLESLintRuleContext): GraphQLSchema | never;
+export declare const logger: {
+    error: (...args: any[]) => void;
+    warn: (...args: any[]) => void;
+};
 export declare function requireReachableTypesFromContext(ruleName: string, context: GraphQLESLintRuleContext): ReachableTypes | never;
 export declare function requireUsedFieldsFromContext(ruleName: string, context: GraphQLESLintRuleContext): UsedFields | never;
 export declare function extractTokens(source: Source): AST.Token[];