@graphql-eslint/eslint-plugin 2.3.0-alpha-f7157af.0 → 2.4.0-alpha-b91589b.0

package/README.md

@@ -46,7 +46,7 @@ npm install --save-dev @graphql-eslint/eslint-plugin
 
 ### Configuration
 
-To get started, create an override configuration for your ESLint, while applying it to to `.graphql` files (do that even if you are declaring your operations in code files):
+To get started, create an override configuration for your ESLint, while applying it to `.graphql` files (do that even if you are declaring your operations in code files):
 
 ```json
 {

package/docs/custom-rules.md

@@ -38,7 +38,7 @@ const rule: GraphQLESLintRule = {
 So what happens here?
 
 1. `@graphql-eslint/eslint-plugin` handles the parsing process for your GraphQL content. It will load the GraphQL files (either from code files or from `.graphql` files with SDL), parse it using GraphQL parser, converts it to ESTree structure and let ESLint do the rest.
-1. You rule is being loaded by ESLint, and executes just like any other ESLint rule.
+1. Your rule is being loaded by ESLint, and executes just like any other ESLint rule.
 1. Our custom rule asks ESLint to run our function for every `OperationDefinition` found.
 1. If the `OperationDefinition` node doesn't have a valid `name` - we report an error to ESLint.
 
@@ -49,7 +49,7 @@ You can scan the `packages/plugin/src/rules` directory in this repo for referenc
 ## Accessing original GraphQL AST nodes
 
 Since our parser converts GraphQL AST to ESTree structure, there are some minor differences in the structure of the objects.
-If you are using TypeScript, and you typed your rule with `GraphQLESLintRule` - you'll see that each `node` is a bit different from from the AST nodes of GraphQL (you can read more about that in [graphql-eslint parser documentation](./parser.md)).
+If you are using TypeScript, and you typed your rule with `GraphQLESLintRule` - you'll see that each `node` is a bit different from the AST nodes of GraphQL (you can read more about that in [graphql-eslint parser documentation](./parser.md)).
 
 If you need access to the original GraphQL AST `node`, you can use `.rawNode()` method on each node you get from the AST structure of ESLint.
 

package/estree-parser/converter.d.ts

@@ -1,7 +1,6 @@
 import { GraphQLESTreeNode } from './estree-ast';
 import { ASTNode, TypeInfo } from 'graphql';
-import { Comment } from 'estree';
 export declare function convertToESTree<T extends ASTNode>(node: T, typeInfo?: TypeInfo): {
-    rootTree: GraphQLESTreeNode<T>;
-    comments: Comment[];
+    rootTree: GraphQLESTreeNode<T, false>;
+    comments: import("estree").Comment[];
 };

package/index.js

@@ -790,6 +790,7 @@ const rule$1 = {
         messages: {
             [AVOID_DUPLICATE_FIELDS]: `{{ type }} "{{ fieldName }}" defined multiple times.`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -960,6 +961,7 @@ const rule$3 = {
                 },
             ],
         },
+        schema: [],
     },
     create(context) {
         const schema = requireGraphQLSchemaFromContext('avoid-scalar-result-type-on-mutation', context);
@@ -1015,6 +1017,7 @@ const rule$4 = {
         messages: {
             [AVOID_TYPENAME_PREFIX]: `Field "{{ fieldName }}" starts with the name of the parent type "{{ typeName }}"`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1728,6 +1731,7 @@ const rule$9 = {
         messages: {
             [NO_ANONYMOUS_OPERATIONS]: `Anonymous GraphQL operations are forbidden. Please make sure to name your {{ operation }}!`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1791,6 +1795,7 @@ const rule$a = {
         messages: {
             [ERROR_MESSAGE_ID]: `Case-insensitive enum values duplicates are not allowed! Found: "{{ found }}"`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1885,6 +1890,7 @@ const rule$b = {
         messages: {
             [NO_DEPRECATED]: `This {{ type }} is marked as deprecated in your GraphQL schema {{ reason }}`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1892,7 +1898,7 @@ const rule$b = {
                 requireGraphQLSchemaFromContext('no-deprecated', context);
                 const typeInfo = node.typeInfo();
                 if (typeInfo && typeInfo.enumValue) {
-                    if (typeInfo.enumValue.isDeprecated) {
+                    if (typeInfo.enumValue.deprecationReason) {
                         context.report({
                             loc: node.loc,
                             messageId: NO_DEPRECATED,
@@ -1908,7 +1914,7 @@ const rule$b = {
                 requireGraphQLSchemaFromContext('no-deprecated', context);
                 const typeInfo = node.typeInfo();
                 if (typeInfo && typeInfo.fieldDef) {
-                    if (typeInfo.fieldDef.isDeprecated) {
+                    if (typeInfo.fieldDef.deprecationReason) {
                         context.report({
                             loc: node.loc,
                             messageId: NO_DEPRECATED,
@@ -1971,6 +1977,7 @@ const rule$c = {
             ],
         },
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         return {
@@ -2031,6 +2038,7 @@ const rule$d = {
         messages: {
             [NO_OPERATION_NAME_SUFFIX]: `Unnecessary "{{ invalidSuffix }}" suffix in your operation name!`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -2096,6 +2104,7 @@ const rule$e = {
         },
         fixable: 'code',
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         const reachableTypes = requireReachableTypesFromContext(RULE_NAME, context);
@@ -2187,6 +2196,7 @@ const rule$f = {
         },
         fixable: 'code',
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         const usedFields = requireUsedFieldsFromContext(RULE_NAME$1, context);
@@ -2452,6 +2462,7 @@ const rule$h = {
             ],
         },
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         return {
@@ -2516,7 +2527,7 @@ const rule$i = {
             examples: [
                 {
                     title: 'Incorrect',
-                    usage: [{ on: ['ObjectTypeDefinition', 'FieldDefinition'] }],
+                    usage: [{ on: [graphql.Kind.OBJECT_TYPE_DEFINITION, graphql.Kind.FIELD_DEFINITION] }],
                     code: /* GraphQL */ `
             type someTypeName {
               name: String
@@ -2525,7 +2536,7 @@ const rule$i = {
                 },
                 {
                     title: 'Correct',
-                    usage: [{ on: ['ObjectTypeDefinition', 'FieldDefinition'] }],
+                    usage: [{ on: [graphql.Kind.OBJECT_TYPE_DEFINITION, graphql.Kind.FIELD_DEFINITION] }],
                     code: /* GraphQL */ `
             """
             Some type description
@@ -2610,6 +2621,7 @@ const rule$j = {
                 },
             ],
         },
+        schema: [],
     },
     create(context) {
         const schema = requireGraphQLSchemaFromContext(RULE_NAME$2, context);
@@ -3229,6 +3241,7 @@ const rule$n = {
         messages: {
             [UNIQUE_FRAGMENT_NAME]: 'Fragment named "{{ documentName }}" already defined in:\n{{ summary }}',
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -3291,6 +3304,7 @@ const rule$o = {
         messages: {
             [UNIQUE_OPERATION_NAME]: 'Operation named "{{ documentName }}" already defined in:\n{{ summary }}',
         },
+        schema: [],
     },
     create(context) {
         return {

package/index.mjs

@@ -784,6 +784,7 @@ const rule$1 = {
         messages: {
             [AVOID_DUPLICATE_FIELDS]: `{{ type }} "{{ fieldName }}" defined multiple times.`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -954,6 +955,7 @@ const rule$3 = {
                 },
             ],
         },
+        schema: [],
     },
     create(context) {
         const schema = requireGraphQLSchemaFromContext('avoid-scalar-result-type-on-mutation', context);
@@ -1009,6 +1011,7 @@ const rule$4 = {
         messages: {
             [AVOID_TYPENAME_PREFIX]: `Field "{{ fieldName }}" starts with the name of the parent type "{{ typeName }}"`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1722,6 +1725,7 @@ const rule$9 = {
         messages: {
             [NO_ANONYMOUS_OPERATIONS]: `Anonymous GraphQL operations are forbidden. Please make sure to name your {{ operation }}!`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1785,6 +1789,7 @@ const rule$a = {
         messages: {
             [ERROR_MESSAGE_ID]: `Case-insensitive enum values duplicates are not allowed! Found: "{{ found }}"`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1879,6 +1884,7 @@ const rule$b = {
         messages: {
             [NO_DEPRECATED]: `This {{ type }} is marked as deprecated in your GraphQL schema {{ reason }}`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -1886,7 +1892,7 @@ const rule$b = {
                 requireGraphQLSchemaFromContext('no-deprecated', context);
                 const typeInfo = node.typeInfo();
                 if (typeInfo && typeInfo.enumValue) {
-                    if (typeInfo.enumValue.isDeprecated) {
+                    if (typeInfo.enumValue.deprecationReason) {
                         context.report({
                             loc: node.loc,
                             messageId: NO_DEPRECATED,
@@ -1902,7 +1908,7 @@ const rule$b = {
                 requireGraphQLSchemaFromContext('no-deprecated', context);
                 const typeInfo = node.typeInfo();
                 if (typeInfo && typeInfo.fieldDef) {
-                    if (typeInfo.fieldDef.isDeprecated) {
+                    if (typeInfo.fieldDef.deprecationReason) {
                         context.report({
                             loc: node.loc,
                             messageId: NO_DEPRECATED,
@@ -1965,6 +1971,7 @@ const rule$c = {
             ],
         },
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         return {
@@ -2025,6 +2032,7 @@ const rule$d = {
         messages: {
             [NO_OPERATION_NAME_SUFFIX]: `Unnecessary "{{ invalidSuffix }}" suffix in your operation name!`,
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -2090,6 +2098,7 @@ const rule$e = {
         },
         fixable: 'code',
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         const reachableTypes = requireReachableTypesFromContext(RULE_NAME, context);
@@ -2181,6 +2190,7 @@ const rule$f = {
         },
         fixable: 'code',
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         const usedFields = requireUsedFieldsFromContext(RULE_NAME$1, context);
@@ -2446,6 +2456,7 @@ const rule$h = {
             ],
         },
         type: 'suggestion',
+        schema: [],
     },
     create(context) {
         return {
@@ -2510,7 +2521,7 @@ const rule$i = {
             examples: [
                 {
                     title: 'Incorrect',
-                    usage: [{ on: ['ObjectTypeDefinition', 'FieldDefinition'] }],
+                    usage: [{ on: [Kind.OBJECT_TYPE_DEFINITION, Kind.FIELD_DEFINITION] }],
                     code: /* GraphQL */ `
             type someTypeName {
               name: String
@@ -2519,7 +2530,7 @@ const rule$i = {
                 },
                 {
                     title: 'Correct',
-                    usage: [{ on: ['ObjectTypeDefinition', 'FieldDefinition'] }],
+                    usage: [{ on: [Kind.OBJECT_TYPE_DEFINITION, Kind.FIELD_DEFINITION] }],
                     code: /* GraphQL */ `
             """
             Some type description
@@ -2604,6 +2615,7 @@ const rule$j = {
                 },
             ],
         },
+        schema: [],
     },
     create(context) {
         const schema = requireGraphQLSchemaFromContext(RULE_NAME$2, context);
@@ -3223,6 +3235,7 @@ const rule$n = {
         messages: {
             [UNIQUE_FRAGMENT_NAME]: 'Fragment named "{{ documentName }}" already defined in:\n{{ summary }}',
         },
+        schema: [],
     },
     create(context) {
         return {
@@ -3285,6 +3298,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-f7157af.0",
+  "version": "2.4.0-alpha-b91589b.0",
   "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"