@graphql-eslint/eslint-plugin 3.0.0-alpha-698204a.0 → 3.0.0-alpha-2918431.0

package/index.js

@@ -39,20 +39,18 @@ const schemaRecommendedConfig = {
             {
                 types: 'PascalCase',
                 fields: 'camelCase',
-                overrides: {
-                    EnumValueDefinition: 'UPPER_CASE',
-                    'FieldDefinition[parent.name.value=Query]': {
-                        forbiddenPrefixes: ['query', 'get'],
-                        forbiddenSuffixes: ['Query'],
-                    },
-                    'FieldDefinition[parent.name.value=Mutation]': {
-                        forbiddenPrefixes: ['mutation'],
-                        forbiddenSuffixes: ['Mutation'],
-                    },
-                    'FieldDefinition[parent.name.value=Subscription]': {
-                        forbiddenPrefixes: ['subscription'],
-                        forbiddenSuffixes: ['Subscription'],
-                    },
+                EnumValueDefinition: 'UPPER_CASE',
+                'FieldDefinition[parent.name.value=Query]': {
+                    forbiddenPrefixes: ['query', 'get'],
+                    forbiddenSuffixes: ['Query'],
+                },
+                'FieldDefinition[parent.name.value=Mutation]': {
+                    forbiddenPrefixes: ['mutation'],
+                    forbiddenSuffixes: ['Mutation'],
+                },
+                'FieldDefinition[parent.name.value=Subscription]': {
+                    forbiddenPrefixes: ['subscription'],
+                    forbiddenSuffixes: ['Subscription'],
                 },
             },
         ],
@@ -92,7 +90,7 @@ const schemaAllConfig = {
         '@graphql-eslint/no-scalar-result-type-on-mutation': 'error',
         '@graphql-eslint/no-unused-fields': 'off',
         '@graphql-eslint/require-deprecation-date': 'error',
-        '@graphql-eslint/require-description': ['error', { types: true, overrides: { DirectiveDefinition: true } }],
+        '@graphql-eslint/require-description': ['error', { types: true, DirectiveDefinition: true }],
         '@graphql-eslint/require-field-of-type-query-in-mutation-result': 'error',
     },
 };
@@ -114,16 +112,14 @@ const operationsRecommendedConfig = {
         '@graphql-eslint/naming-convention': [
             'error',
             {
-                overrides: {
-                    Argument: 'camelCase',
-                    VariableDefinition: 'camelCase',
-                    OperationDefinition: {
-                        style: 'PascalCase',
-                        forbiddenPrefixes: ['Query', 'Mutation', 'Subscription', 'Get'],
-                        forbiddenSuffixes: ['Query', 'Mutation', 'Subscription'],
-                    },
-                    FragmentDefinition: { style: 'PascalCase', forbiddenPrefixes: ['Fragment'], forbiddenSuffixes: ['Fragment'] },
-                },
+                Argument: 'camelCase',
+                VariableDefinition: 'camelCase',
+                OperationDefinition: {
+                    style: 'PascalCase',
+                    forbiddenPrefixes: ['Query', 'Mutation', 'Subscription', 'Get'],
+                    forbiddenSuffixes: ['Query', 'Mutation', 'Subscription'],
+                },
+                FragmentDefinition: { style: 'PascalCase', forbiddenPrefixes: ['Fragment'], forbiddenSuffixes: ['Fragment'] },
             },
         ],
         '@graphql-eslint/no-anonymous-operations': 'error',
@@ -912,23 +908,21 @@ const rule$1 = {
                 properties: {
                     style: {
                         enum: ['block', 'inline'],
-                        default: 'inline',
+                        default: 'block',
                     },
                 },
             },
         ],
     },
     create(context) {
-        const { style } = context.options[0] || { style: 'inline' };
-        const wrongDescriptionType = style === 'block' ? 'inline' : 'block';
+        const { style = 'block' } = context.options[0] || {};
+        const isBlock = style === 'block';
         return {
-            '[description.type="StringValue"]': node => {
-                if (node.description.block !== (style === 'block')) {
-                    context.report({
-                        loc: getLocation(node.description.loc),
-                        message: `Unexpected ${wrongDescriptionType} description`,
-                    });
-                }
+            [`.description[type=StringValue][block!=${isBlock}]`](node) {
+                context.report({
+                    loc: getLocation(node.loc),
+                    message: `Unexpected ${isBlock ? 'inline' : 'block'} description`,
+                });
             },
         };
     },
@@ -1318,38 +1312,34 @@ const rule$4 = {
                     {
                         types: 'PascalCase',
                         fields: 'camelCase',
-                        overrides: {
-                            EnumValueDefinition: 'UPPER_CASE',
-                            'FieldDefinition[parent.name.value=Query]': {
-                                forbiddenPrefixes: ['query', 'get'],
-                                forbiddenSuffixes: ['Query'],
-                            },
-                            'FieldDefinition[parent.name.value=Mutation]': {
-                                forbiddenPrefixes: ['mutation'],
-                                forbiddenSuffixes: ['Mutation'],
-                            },
-                            'FieldDefinition[parent.name.value=Subscription]': {
-                                forbiddenPrefixes: ['subscription'],
-                                forbiddenSuffixes: ['Subscription'],
-                            },
+                        EnumValueDefinition: 'UPPER_CASE',
+                        'FieldDefinition[parent.name.value=Query]': {
+                            forbiddenPrefixes: ['query', 'get'],
+                            forbiddenSuffixes: ['Query'],
+                        },
+                        'FieldDefinition[parent.name.value=Mutation]': {
+                            forbiddenPrefixes: ['mutation'],
+                            forbiddenSuffixes: ['Mutation'],
+                        },
+                        'FieldDefinition[parent.name.value=Subscription]': {
+                            forbiddenPrefixes: ['subscription'],
+                            forbiddenSuffixes: ['Subscription'],
                         },
                     },
                 ],
                 operations: [
                     {
-                        overrides: {
-                            Argument: 'camelCase',
-                            VariableDefinition: 'camelCase',
-                            OperationDefinition: {
-                                style: 'PascalCase',
-                                forbiddenPrefixes: ['Query', 'Mutation', 'Subscription', 'Get'],
-                                forbiddenSuffixes: ['Query', 'Mutation', 'Subscription'],
-                            },
-                            FragmentDefinition: {
-                                style: 'PascalCase',
-                                forbiddenPrefixes: ['Fragment'],
-                                forbiddenSuffixes: ['Fragment'],
-                            },
+                        Argument: 'camelCase',
+                        VariableDefinition: 'camelCase',
+                        OperationDefinition: {
+                            style: 'PascalCase',
+                            forbiddenPrefixes: ['Query', 'Mutation', 'Subscription', 'Get'],
+                            forbiddenSuffixes: ['Query', 'Mutation', 'Subscription'],
+                        },
+                        FragmentDefinition: {
+                            style: 'PascalCase',
+                            forbiddenPrefixes: ['Fragment'],
+                            forbiddenSuffixes: ['Fragment'],
                         },
                     },
                 ],
@@ -1389,14 +1379,6 @@ const rule$4 = {
                 type: 'object',
                 additionalProperties: false,
                 properties: {
-                    types: {
-                        ...schemaOption$1,
-                        description: `Includes:\n\n${TYPES_KINDS.map(kind => `- [${kind}](https://spec.graphql.org/October2021/#${kind})`).join('\n')}`,
-                    },
-                    fields: {
-                        ...schemaOption$1,
-                        description: `Includes:\n\n${FIELDS_KINDS.map(kind => `- [${kind}](https://spec.graphql.org/October2021/#${kind})`).join('\n')}`,
-                    },
                     allowLeadingUnderscore: {
                         type: 'boolean',
                         default: false,
@@ -1405,35 +1387,42 @@ const rule$4 = {
                         type: 'boolean',
                         default: false,
                     },
-                    overrides: {
-                        type: 'object',
-                        additionalProperties: false,
-                        description: [
-                            'May contain the following `ASTNode` names:',
-                            '',
-                            ...ALLOWED_KINDS.map(kind => `- [${kind}](https://spec.graphql.org/October2021/#${kind})`),
-                            '',
-                            "> It's also possible to use a [`selector`](https://eslint.org/docs/developer-guide/selectors) that starts with `ASTNode` name",
-                            '>',
-                            '> Example: pattern property `FieldDefinition[parent.name.value=Query]` will match only fields for type `Query`',
-                        ].join('\n'),
-                        patternProperties: {
-                            [`^(${ALLOWED_KINDS.join('|')})(.+)?$`]: schemaOption$1,
-                        },
+                    types: {
+                        ...schemaOption$1,
+                        description: `Includes:\n\n${TYPES_KINDS.map(kind => `- \`${kind}\``).join('\n')}`,
                     },
-                },
+                    fields: {
+                        ...schemaOption$1,
+                        description: `Includes:\n\n${FIELDS_KINDS.map(kind => `- \`${kind}\``).join('\n')}`,
+                    },
+                    ...Object.fromEntries(ALLOWED_KINDS.map(kind => [
+                        kind,
+                        {
+                            ...schemaOption$1,
+                            description: `Read more about this kind on [spec.graphql.org](https://spec.graphql.org/October2021/#${kind}).`,
+                        },
+                    ])),
+                },
+                patternProperties: {
+                    [`^(${ALLOWED_KINDS.join('|')})(.+)?$`]: schemaOption$1,
+                },
+                description: [
+                    "> It's possible to use a [`selector`](https://eslint.org/docs/developer-guide/selectors) that starts with allowed `ASTNode` names which are described below.",
+                    '>',
+                    '> Paste or drop code into the editor in [ASTExplorer](https://astexplorer.net) and inspect the generated AST to compose your selector.',
+                    '>',
+                    '> Example: pattern property `FieldDefinition[parent.name.value=Query]` will match only fields for type `Query`.',
+                ].join('\n'),
             },
         },
     },
     create(context) {
-        const options = {
-            overrides: {},
-            ...context.options[0],
-        };
+        const options = context.options[0] || {};
+        const { allowLeadingUnderscore, allowTrailingUnderscore, types, fields, ...restOptions } = options;
         function normalisePropertyOption(kind) {
-            let style = options.overrides[kind];
+            let style = options[kind];
             if (!style) {
-                style = TYPES_KINDS.includes(kind) ? options.types : options.fields;
+                style = TYPES_KINDS.includes(kind) ? types : fields;
             }
             return typeof style === 'object' ? style : { style };
         }
@@ -1454,10 +1443,10 @@ const rule$4 = {
             }
             function getErrorMessage() {
                 let name = nodeName;
-                if (options.allowLeadingUnderscore) {
+                if (allowLeadingUnderscore) {
                     name = name.replace(/^_*/, '');
                 }
-                if (options.allowTrailingUnderscore) {
+                if (allowTrailingUnderscore) {
                     name = name.replace(/_*$/, '');
                 }
                 if (prefix && !name.startsWith(prefix)) {
@@ -1491,15 +1480,13 @@ const rule$4 = {
             });
         };
         const listeners = {};
-        if (!options.allowLeadingUnderscore) {
+        if (!allowLeadingUnderscore) {
             listeners['Name[value=/^_/]:matches([parent.kind!=Field], [parent.kind=Field][parent.alias])'] = checkUnderscore;
         }
-        if (!options.allowTrailingUnderscore) {
+        if (!allowTrailingUnderscore) {
             listeners['Name[value=/_$/]:matches([parent.kind!=Field], [parent.kind=Field][parent.alias])'] = checkUnderscore;
         }
-        const selectors = new Set([options.types && TYPES_KINDS, options.fields && FIELDS_KINDS, Object.keys(options.overrides)]
-            .flat()
-            .filter(Boolean));
+        const selectors = new Set([types && TYPES_KINDS, fields && FIELDS_KINDS, Object.keys(restOptions)].flat().filter(Boolean));
         for (const selector of selectors) {
             listeners[selector] = checkNode(selector);
         }
@@ -1904,18 +1891,18 @@ const rule$9 = {
     },
 };
 
-const ROOT_TYPES = ['query', 'mutation', 'subscription'];
+const ROOT_TYPES = ['mutation', 'subscription'];
 const rule$a = {
     meta: {
         type: 'suggestion',
         docs: {
             category: 'Schema',
-            description: 'Disallow using root types for `read-only` or `write-only` schemas.',
+            description: 'Disallow using root types `mutation` and/or `subscription`.',
             url: 'https://github.com/dotansimha/graphql-eslint/blob/master/docs/rules/no-root-type.md',
             requiresSchema: true,
             examples: [
                 {
-                    title: 'Incorrect (`read-only` schema)',
+                    title: 'Incorrect',
                     usage: [{ disallow: ['mutation', 'subscription'] }],
                     code: /* GraphQL */ `
             type Mutation {
@@ -1924,16 +1911,7 @@ const rule$a = {
           `,
                 },
                 {
-                    title: 'Incorrect (`write-only` schema)',
-                    usage: [{ disallow: ['query'] }],
-                    code: /* GraphQL */ `
-            type Query {
-              users: [User!]!
-            }
-          `,
-                },
-                {
-                    title: 'Correct (`read-only` schema)',
+                    title: 'Correct',
                     usage: [{ disallow: ['mutation', 'subscription'] }],
                     code: /* GraphQL */ `
             type Query {
@@ -1968,7 +1946,6 @@ const rule$a = {
         const schema = requireGraphQLSchemaFromContext('no-root-type', context);
         const disallow = new Set(context.options[0].disallow);
         const rootTypeNames = [
-            disallow.has('query') && schema.getQueryType(),
             disallow.has('mutation') && schema.getMutationType(),
             disallow.has('subscription') && schema.getSubscriptionType(),
         ]
@@ -2548,7 +2525,7 @@ const rule$h = {
             examples: [
                 {
                     title: 'Incorrect',
-                    usage: [{ types: true, overrides: { FieldDefinition: true } }],
+                    usage: [{ types: true, FieldDefinition: true }],
                     code: /* GraphQL */ `
             type someTypeName {
               name: String
@@ -2557,7 +2534,7 @@ const rule$h = {
                 },
                 {
                     title: 'Correct',
-                    usage: [{ types: true, overrides: { FieldDefinition: true } }],
+                    usage: [{ types: true, FieldDefinition: true }],
                     code: /* GraphQL */ `
             """
             Some type description
@@ -2574,9 +2551,7 @@ const rule$h = {
             configOptions: [
                 {
                     types: true,
-                    overrides: {
-                        [graphql.Kind.DIRECTIVE_DEFINITION]: true,
-                    },
+                    [graphql.Kind.DIRECTIVE_DEFINITION]: true,
                 },
             ],
         },
@@ -2595,22 +2570,23 @@ const rule$h = {
                 properties: {
                     types: {
                         type: 'boolean',
-                        description: `Includes:\n\n${TYPES_KINDS.map(kind => `- [${kind}](https://spec.graphql.org/October2021/#${kind})`).join('\n')}`,
-                    },
-                    overrides: {
-                        type: 'object',
-                        description: 'Configuration for precise `ASTNode`',
-                        additionalProperties: false,
-                        properties: Object.fromEntries(ALLOWED_KINDS$1.map(kind => [kind, { type: 'boolean' }])),
+                        description: `Includes:\n\n${TYPES_KINDS.map(kind => `- \`${kind}\``).join('\n')}`,
                     },
+                    ...Object.fromEntries([...ALLOWED_KINDS$1].sort().map(kind => [
+                        kind,
+                        {
+                            type: 'boolean',
+                            description: `Read more about this kind on [spec.graphql.org](https://spec.graphql.org/October2021/#${kind}).`,
+                        },
+                    ])),
                 },
             },
         },
     },
     create(context) {
-        const { types, overrides = {} } = context.options[0];
+        const { types, ...restOptions } = context.options[0];
         const kinds = new Set(types ? TYPES_KINDS : []);
-        for (const [kind, isEnabled] of Object.entries(overrides)) {
+        for (const [kind, isEnabled] of Object.entries(restOptions)) {
             if (isEnabled) {
                 kinds.add(kind);
             }