@graphql-eslint/eslint-plugin 2.3.0-alpha-4c161e5.0 → 2.3.1

package/index.mjs

@@ -286,6 +286,7 @@ const validationToRule = (name, ruleName, docs, getDocumentNode) => {
             meta: {
                 docs: {
                     ...docs,
+                    graphQLJSRuleName: ruleName,
                     category: 'Validation',
                     recommended: true,
                     requiresSchema,
@@ -314,6 +315,14 @@ const validationToRule = (name, ruleName, docs, getDocumentNode) => {
         },
     };
 };
+const importFiles = (context) => {
+    const code = context.getSourceCode().text;
+    if (!isGraphQLImportFile(code)) {
+        return null;
+    }
+    // Import documents because file contains '#import' comments
+    return processImport(context.getFilename());
+};
 const GRAPHQL_JS_VALIDATIONS = Object.assign({}, validationToRule('executable-definitions', 'ExecutableDefinitions', {
     description: `A GraphQL document is only valid for execution if all definitions are either operation or fragment definitions.`,
 }), validationToRule('fields-on-correct-type', 'FieldsOnCorrectType', {
@@ -390,14 +399,7 @@ const GRAPHQL_JS_VALIDATIONS = Object.assign({}, validationToRule('executable-de
             \``,
         },
     ],
-}, context => {
-    const code = context.getSourceCode().text;
-    if (!isGraphQLImportFile(code)) {
-        return null;
-    }
-    // Import documents because file contains '#import' comments
-    return processImport(context.getFilename());
-}), validationToRule('known-type-names', 'KnownTypeNames', {
+}, importFiles), validationToRule('known-type-names', 'KnownTypeNames', {
     description: `A GraphQL document is only valid if referenced types (specifically variable definitions and fragment conditions) are defined by the type schema.`,
 }), validationToRule('lone-anonymous-operation', 'LoneAnonymousOperation', {
     description: `A GraphQL document is only valid if when it contains an anonymous operation (the query short-hand) that it contains only that one operation definition.`,
@@ -408,7 +410,7 @@ const GRAPHQL_JS_VALIDATIONS = Object.assign({}, validationToRule('executable-de
     description: `A GraphQL fragment is only valid when it does not have cycles in fragments usage.`,
 }), validationToRule('no-undefined-variables', 'NoUndefinedVariables', {
     description: `A GraphQL operation is only valid if all variables encountered, both directly and via fragment spreads, are defined by that operation.`,
-}), validationToRule('no-unused-fragments', 'NoUnusedFragments', {
+}, importFiles), validationToRule('no-unused-fragments', 'NoUnusedFragments', {
     description: `A GraphQL document is only valid if all fragment definitions are spread within operations, or spread within other fragments spread within operations.`,
     requiresSiblings: true,
 }, context => {
@@ -439,7 +441,7 @@ const GRAPHQL_JS_VALIDATIONS = Object.assign({}, validationToRule('executable-de
     return getParentNode(context.getFilename());
 }), validationToRule('no-unused-variables', 'NoUnusedVariables', {
     description: `A GraphQL operation is only valid if all variables defined by an operation are used, either directly or within a spread fragment.`,
-}), validationToRule('overlapping-fields-can-be-merged', 'OverlappingFieldsCanBeMerged', {
+}, importFiles), validationToRule('overlapping-fields-can-be-merged', 'OverlappingFieldsCanBeMerged', {
     description: `A selection set is only valid if all fields (including spreading any fragments) either correspond to distinct response names or can be merged without ambiguity.`,
 }), validationToRule('possible-fragment-spread', 'PossibleFragmentSpreads', {
     description: `A fragment spread is only valid if the type condition could ever possibly be true: if there is a non-empty intersection of the possible parent types, and possible types which pass the type condition.`,
@@ -649,10 +651,7 @@ const rule = {
                                 line: start.line,
                                 column: start.column - (isVariableNode ? 2 : 1),
                             },
-                            end: {
-                                line: end.line,
-                                column: end.column,
-                            },
+                            end,
                         },
                         messageId: ALPHABETIZE,
                         data: isVariableNode
@@ -783,6 +782,7 @@ const rule$1 = {
         messages: {
             [AVOID_DUPLICATE_FIELDS]: `{{ type }} "{{ fieldName }}" defined multiple times.`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -902,15 +902,16 @@ const rule$2 = {
                         const testKeyword = caseSensitive ? keyword : keyword.toLowerCase();
                         const testName = caseSensitive ? node.name.value : node.name.value.toLowerCase();
                         if (testName.startsWith(testKeyword)) {
+                            const { start } = node.name.loc;
                             context.report({
                                 loc: {
                                     start: {
-                                        line: node.name.loc.start.line,
-                                        column: node.name.loc.start.column - 1,
+                                        line: start.line,
+                                        column: start.column - 1,
                                     },
                                     end: {
-                                        line: node.name.loc.start.line,
-                                        column: node.name.loc.start.column + testKeyword.length - 1,
+                                        line: start.line,
+                                        column: start.column - 1 + testKeyword.length,
                                     },
                                 },
                                 data: {
@@ -953,6 +954,7 @@ const rule$3 = {
                 },
             ],
         },
+        schema: [],
     },
     create(context) {
         const schema = requireGraphQLSchemaFromContext('avoid-scalar-result-type-on-mutation', context);
@@ -1008,22 +1010,33 @@ const rule$4 = {
         messages: {
             [AVOID_TYPENAME_PREFIX]: `Field "{{ fieldName }}" starts with the name of the parent type "{{ typeName }}"`,
         },
+        schema: [],
     },
     create(context) {
         return {
             'ObjectTypeDefinition, ObjectTypeExtension, InterfaceTypeDefinition, InterfaceTypeExtension'(node) {
                 const typeName = node.name.value;
-                const lowerTypeName = (typeName || '').toLowerCase();
+                const lowerTypeName = typeName.toLowerCase();
                 for (const field of node.fields) {
-                    const fieldName = field.name.value || '';
-                    if (fieldName && lowerTypeName && fieldName.toLowerCase().startsWith(lowerTypeName)) {
+                    const fieldName = field.name.value;
+                    if (fieldName.toLowerCase().startsWith(lowerTypeName)) {
+                        const { start } = field.loc;
                         context.report({
-                            node: field.name,
                             data: {
                                 fieldName,
                                 typeName,
                             },
                             messageId: AVOID_TYPENAME_PREFIX,
+                            loc: {
+                                start: {
+                                    line: start.line,
+                                    column: start.column - 1,
+                                },
+                                end: {
+                                    line: start.line,
+                                    column: start.column - 1 + lowerTypeName.length,
+                                },
+                            },
                         });
                     }
                 }
@@ -1721,20 +1734,24 @@ const rule$9 = {
         messages: {
             [NO_ANONYMOUS_OPERATIONS]: `Anonymous GraphQL operations are forbidden. Please make sure to name your {{ operation }}!`,
         },
+        schema: [],
     },
     create(context) {
         return {
             OperationDefinition(node) {
-                if (node && (!node.name || node.name.value === '')) {
+                var _a;
+                const isAnonymous = (((_a = node.name) === null || _a === void 0 ? void 0 : _a.value) || '').length === 0;
+                if (isAnonymous) {
+                    const { start } = node.loc;
                     context.report({
                         loc: {
                             start: {
-                                column: node.loc.start.column - 1,
-                                line: node.loc.start.line,
+                                column: start.column - 1,
+                                line: start.line,
                             },
                             end: {
-                                column: node.loc.start.column + node.operation.length,
-                                line: node.loc.start.line,
+                                column: start.column - 1 + node.operation.length,
+                                line: start.line,
                             },
                         },
                         data: {
@@ -1784,6 +1801,7 @@ const rule$a = {
         messages: {
             [ERROR_MESSAGE_ID]: `Case-insensitive enum values duplicates are not allowed! Found: "{{ found }}"`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1878,6 +1896,7 @@ const rule$b = {
         messages: {
             [NO_DEPRECATED]: `This {{ type }} is marked as deprecated in your GraphQL schema {{ reason }}`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1964,6 +1983,7 @@ const rule$c = {
             ],
         },
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         return {
@@ -2024,15 +2044,28 @@ const rule$d = {
         messages: {
             [NO_OPERATION_NAME_SUFFIX]: `Unnecessary "{{ invalidSuffix }}" suffix in your operation name!`,
         },
+        schema: [],
     },
     create(context) {
         return {
             'OperationDefinition, FragmentDefinition'(node) {
-                if (node && node.name && node.name.value !== '') {
-                    const invalidSuffix = (node.type === 'OperationDefinition' ? node.operation : 'fragment').toLowerCase();
-                    if (node.name.value.toLowerCase().endsWith(invalidSuffix)) {
+                var _a;
+                const name = ((_a = node.name) === null || _a === void 0 ? void 0 : _a.value) || '';
+                if (name.length > 0) {
+                    const invalidSuffix = 'operation' in node ? node.operation : 'fragment';
+                    if (name.toLowerCase().endsWith(invalidSuffix)) {
+                        const { start, end } = node.name.loc;
                         context.report({
-                            node: node.name,
+                            loc: {
+                                start: {
+                                    column: start.column - 1 + name.length - invalidSuffix.length,
+                                    line: start.line,
+                                },
+                                end: {
+                                    column: end.column - 1 + name.length,
+                                    line: end.line,
+                                },
+                            },
                             data: {
                                 invalidSuffix,
                             },
@@ -2089,6 +2122,7 @@ const rule$e = {
         },
         fixable: 'code',
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         const reachableTypes = requireReachableTypesFromContext(RULE_NAME, context);
@@ -2180,6 +2214,7 @@ const rule$f = {
         },
         fixable: 'code',
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         const usedFields = requireUsedFieldsFromContext(RULE_NAME$1, context);
@@ -2445,6 +2480,7 @@ const rule$h = {
             ],
         },
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         return {
@@ -2481,15 +2517,18 @@ const DESCRIBABLE_NODES = [
 function verifyRule(context, node) {
     if (node) {
         if (!node.description || !node.description.value || node.description.value.trim().length === 0) {
+            const { start, end } = ('name' in node ? node.name : node).loc;
             context.report({
                 loc: {
                     start: {
-                        line: node.loc.start.line,
-                        column: node.loc.start.column - 1,
+                        line: start.line,
+                        column: start.column - 1,
                     },
                     end: {
-                        line: node.loc.end.line,
-                        column: node.loc.end.column,
+                        line: end.line,
+                        column: 
+                        // node.name don't exist on SchemaDefinition
+                        'name' in node ? end.column - 1 + node.name.value.length : end.column,
                     },
                 },
                 messageId: REQUIRE_DESCRIPTION_ERROR,
@@ -2603,6 +2642,7 @@ const rule$j = {
                 },
             ],
         },
+        schema: [],
     },
     create(context) {
         const schema = requireGraphQLSchemaFromContext(RULE_NAME$2, context);
@@ -3222,6 +3262,7 @@ const rule$n = {
         messages: {
             [UNIQUE_FRAGMENT_NAME]: 'Fragment named "{{ documentName }}" already defined in:\n{{ summary }}',
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -3284,6 +3325,7 @@ const rule$o = {
         messages: {
             [UNIQUE_OPERATION_NAME]: 'Operation named "{{ documentName }}" already defined in:\n{{ summary }}',
         },
+        schema: [],
     },
     create(context) {
         return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@graphql-eslint/eslint-plugin",
-  "version": "2.3.0-alpha-4c161e5.0",
+  "version": "2.3.1",
   "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"

package/types.d.ts

@@ -56,9 +56,11 @@ export declare type RuleDocsInfo<T> = Rule.RuleMetaData & {
             usage?: T;
         }[];
         optionsForConfig?: T;
+        graphQLJSRuleName?: string;
     };
 };
 export declare type GraphQLESLintRule<Options = any[], WithTypeInfo extends boolean = false> = {
     create(context: GraphQLESLintRuleContext<Options>): GraphQLESLintRuleListener<WithTypeInfo>;
     meta: Rule.RuleMetaData & RuleDocsInfo<Options>;
 };
+export declare type ValueOf<T> = T[keyof T];