feat: add eslint-plugin-eslint-plugin (#616)

Author: Dimitri POSTOLOV

.eslintignore

@@ -0,0 +1,48 @@
+module.exports = {
+  reportUnusedDisableDirectives: true,
+  ignorePatterns: ['examples'],
+  parser: '@typescript-eslint/parser',
+  extends: [
+    'eslint:recommended',
+    'plugin:@typescript-eslint/recommended',
+    'standard',
+    'plugin:eslint-plugin/recommended',
+    'prettier',
+  ],
+  plugins: ['unicorn'],
+  rules: {
+    'no-empty': 'off',
+    'no-console': 'error',
+    'no-prototype-builtins': 'off',
+    'no-useless-constructor': 'off',
+    'no-unused-vars': 'off', // disable base rule as it can report incorrect errors
+    '@typescript-eslint/no-unused-vars': ['warn', { args: 'none' }],
+    '@typescript-eslint/no-use-before-define': 'off',
+    '@typescript-eslint/no-namespace': 'off',
+    '@typescript-eslint/no-empty-interface': 'off',
+    '@typescript-eslint/no-empty-function': 'off',
+    '@typescript-eslint/no-var-requires': 'off',
+    '@typescript-eslint/no-explicit-any': 'off',
+    '@typescript-eslint/no-non-null-assertion': 'off',
+    '@typescript-eslint/explicit-function-return-type': 'off',
+    '@typescript-eslint/ban-ts-ignore': 'off',
+    '@typescript-eslint/ban-types': 'off',
+    'unicorn/prefer-array-some': 'error',
+    'unicorn/prefer-includes': 'error',
+    'eslint-plugin/test-case-shorthand-strings': 'error',
+  },
+  overrides: [
+    {
+      files: ['*.{spec,test}.{ts,js}'],
+      env: {
+        jest: true,
+      },
+    },
+    {
+      files: ['**/tests/mocks/*.{ts,js}'],
+      rules: {
+        '@typescript-eslint/no-unused-vars': 'off',
+      },
+    },
+  ],
+};

.eslintrc.js

@@ -13,7 +13,7 @@
   "scripts": {
     "generate:docs": "ts-node scripts/generate-docs.ts",
     "postinstall": "patch-package",
-    "lint": "eslint --config .eslintrc.json --ext ts,js .",
+    "lint": "eslint --ignore-path .gitignore --ext ts,js .",
     "prebuild": "rimraf packages/*/dist",
     "transpile-ts": "tsc --project tsconfig.json",
     "build": "yarn transpile-ts && bob build",
@@ -39,13 +39,14 @@
     "eslint": "7.32.0",
     "eslint-config-prettier": "8.3.0",
     "eslint-config-standard": "16.0.3",
+    "eslint-plugin-eslint-plugin": "^3.5.3",
     "eslint-plugin-import": "2.24.2",
     "eslint-plugin-node": "11.1.0",
     "eslint-plugin-promise": "5.1.0",
     "eslint-plugin-standard": "5.0.0",
+    "eslint-plugin-unicorn": "35.0.0",
     "husky": "7.0.2",
     "jest": "27.2.0",
-    "eslint-plugin-unicorn": "35.0.0",
     "json-schema-to-markdown": "1.1.1",
     "lint-staged": "11.1.2",
     "patch-package": "6.4.7",

.eslintrc.json

@@ -37,10 +37,10 @@
   "devDependencies": {
     "@types/eslint": "7.28.0",
     "@types/graphql-depth-limit": "1.1.3",
+    "@types/lodash.lowercase": "^4.3.6",
     "bob-the-bundler": "1.5.1",
     "graphql": "15.5.3",
-    "typescript": "4.4.3",
-    "@types/lodash.camelcase": "4.3.6"
+    "typescript": "4.4.3"
   },
   "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.json

@@ -1,12 +1,13 @@
 import { statSync } from 'fs';
 import { dirname } from 'path';
-import { Lexer, GraphQLSchema, Token, DocumentNode, Source } from 'graphql';
+import { Lexer, GraphQLSchema, Source, ObjectTypeDefinitionNode, ObjectTypeExtensionNode, Kind } from 'graphql';
 import { AST } from 'eslint';
 import { asArray, Source as LoaderSource } from '@graphql-tools/utils';
 import lowerCase from 'lodash.lowercase';
 import { GraphQLESLintRuleContext } from './types';
 import { SiblingOperations } from './sibling-operations';
 import { UsedFields, ReachableTypes } from './graphql-ast';
+import { GraphQLESTreeNode } from './estree-parser';
 
 export function requireSiblingsOperations(
   ruleName: string,
@@ -144,10 +145,14 @@ export const loaderCache: Record<string, LoaderSource[]> = new Proxy(Object.crea
   },
 });
 
-const isObjectType = (node): boolean => ['ObjectTypeDefinition', 'ObjectTypeExtension'].includes(node.type);
-export const isQueryType = (node): boolean => isObjectType(node) && node.name.value === 'Query';
-export const isMutationType = (node): boolean => isObjectType(node) && node.name.value === 'Mutation';
-export const isSubscriptionType = (node): boolean => isObjectType(node) && node.name.value === 'Subscription';
+type ObjectTypeNode = GraphQLESTreeNode<ObjectTypeDefinitionNode | ObjectTypeExtensionNode>;
+
+const isObjectType = (node: ObjectTypeNode): boolean =>
+  [Kind.OBJECT_TYPE_DEFINITION, Kind.OBJECT_TYPE_EXTENSION].includes(node.type);
+export const isQueryType = (node: ObjectTypeNode): boolean => isObjectType(node) && node.name.value === 'Query';
+export const isMutationType = (node: ObjectTypeNode): boolean => isObjectType(node) && node.name.value === 'Mutation';
+export const isSubscriptionType = (node: ObjectTypeNode): boolean =>
+  isObjectType(node) && node.name.value === 'Subscription';
 
 export enum CaseStyle {
   camelCase = 'camelCase',

packages/plugin/package.json

@@ -5,28 +5,22 @@ const ruleTester = new GraphQLRuleTester();
 
 ruleTester.runGraphQLTests('avoid-typename-prefix', rule, {
   valid: [
-    {
-      code: /* GraphQL */ `
-        type User {
-          id: ID!
-        }
-      `,
-    },
-    {
-      code: /* GraphQL */ `
-        interface Node {
-          id: ID!
-        }
-      `,
-    },
-    {
-      code: /* GraphQL */ `
-        type User {
-          # eslint-disable-next-line
-          userId: ID!
-        }
-      `,
-    },
+    /* GraphQL */ `
+      type User {
+        id: ID!
+      }
+    `,
+    /* GraphQL */ `
+      interface Node {
+        id: ID!
+      }
+    `,
+    /* GraphQL */ `
+      type User {
+        # eslint-disable-next-line
+        userId: ID!
+      }
+    `,
   ],
   invalid: [
     {

packages/plugin/src/utils.ts

@@ -37,9 +37,7 @@ ruleTester.runGraphQLTests('description-style', rule, {
       code: BLOCK_SDL,
       options: [{ style: 'block' }],
     },
-    {
-      code: INLINE_SDL,
-    },
+    INLINE_SDL,
   ],
   invalid: [
     {

packages/plugin/tests/avoid-typename-prefix.spec.ts

@@ -6,44 +6,34 @@ const ruleTester = new GraphQLRuleTester();
 
 ruleTester.runGraphQLTests('test-directives', rule, {
   valid: [
-    {
-      code: /* GraphQL */ `
-        # eslint-disable-next-line
-        query {
-          a
-        }
-      `,
-    },
-    {
-      code: /* GraphQL */ `
-        # eslint-disable-next-line test-directives
-        query {
-          a
-        }
-      `,
-    },
-    {
-      code: `
-        query { # eslint-disable-line test-directives
-          a
-        }
-      `,
-    },
-    {
-      code: `
-        query { # eslint-disable-line
-          a
-        }
-      `,
-    },
-    {
-      code: /* GraphQL */ `
-        # eslint-disable
-        query {
-          a
-        }
-      `,
-    },
+    /* GraphQL */ `
+      # eslint-disable-next-line
+      query {
+        a
+      }
+    `,
+    /* GraphQL */ `
+      # eslint-disable-next-line test-directives
+      query {
+        a
+      }
+    `,
+    `
+      query { # eslint-disable-line test-directives
+        a
+      }
+    `,
+    `
+      query { # eslint-disable-line
+        a
+      }
+    `,
+    /* GraphQL */ `
+      # eslint-disable
+      query {
+        a
+      }
+    `,
     {
       filename: join(__dirname, 'mocks/test-directives-with-import.graphql'),
       code: ruleTester.fromMockFile('test-directives-with-import.graphql'),

packages/plugin/tests/description-style.spec.ts

@@ -10,12 +10,7 @@ ruleTester.runGraphQLTests('input-name', rule, {
       options: [{ checkInputType: true }],
     },
     {
-      code: 'type Mutation { SetMessage(input: SetMessageInput): String }',
-      options: [{ checkInputType: true }],
-    },
-    {
-      code:
-        'type Mutation { CreateMessage(input: CreateMessageInput): String DeleteMessage(input: DeleteMessageInput): Boolean }',
+      code: 'type Mutation { CreateMessage(input: CreateMessageInput): String DeleteMessage(input: DeleteMessageInput): Boolean }',
       options: [{ checkInputType: true }],
     },
     {