@mikey-pro/eslint-config 4.7.0 → 4.8.2

package/index.js

@@ -1,4 +1,11 @@
 module.exports = {
+  env: {
+    browser: true,
+    commonjs: true,
+    es2021: true,
+    es6: true,
+    node: true,
+  },
   extends: [
     'eslint:recommended',
     'react-app',
@@ -9,33 +16,44 @@ module.exports = {
     'plugin:css-modules/recommended',
     'plugin:prettier/recommended',
   ],
+  ignorePatterns: [
+    '!.*',
+    'node_modules',
+    'vendor',
+    'dist',
+    'package-lock.json',
+    'LICENSE',
+    '.vscode',
+    '.github',
+    'tsconfig.json',
+  ],
   overrides: [
     {
+      extends: [
+        'plugin:@typescript-eslint/recommended',
+        'plugin:@typescript-eslint/recommended-requiring-type-checking',
+      ],
       files: ['*.ts', '*.tsx'],
       parser: '@typescript-eslint/parser',
       parserOptions: {
-        tsconfigRootDir: __dirname,
-        project: ['../../../tsconfig.json'],
-        extraFileExtensions: ['.vue', '.svelte'],
-        sourceType: 'module',
-        ecmaVersion: 'latest',
         ecmaFeatures: {
           jsx: true,
         },
+        ecmaVersion: 'latest',
+        extraFileExtensions: ['.vue', '.svelte'],
+        project: ['../../../tsconfig.json'],
+        sourceType: 'module',
+        tsconfigRootDir: __dirname,
       },
       plugins: ['@typescript-eslint'],
-      extends: [
-        'plugin:@typescript-eslint/recommended',
-        'plugin:@typescript-eslint/recommended-requiring-type-checking',
-      ],
       rules: {
-        'import/named': 0,
-        'import/namespace': 0,
-        'import/default': 0,
-        'import/no-named-as-default-member': 0,
-        'import/no-unresolved': 0,
-        '@typescript-eslint/indent': 0,
-        '@typescript-eslint/naming-convention': 0,
+        '@typescript-eslint/indent': 1,
+        '@typescript-eslint/naming-convention': 1,
+        'import/default': 1,
+        'import/named': 1,
+        'import/namespace': 1,
+        'import/no-named-as-default-member': 1,
+        'import/no-unresolved': 1,
         'prettier/prettier': [1, { parser: 'typescript' }],
       },
     },
@@ -58,25 +76,25 @@ module.exports = {
       },
     },
     {
+      extends: ['plugin:yaml/recommended'],
       files: ['*.yaml', '*.yml'],
       plugins: ['yaml'],
-      extends: ['plugin:yaml/recommended'],
       rules: {
         'prettier/prettier': [1, { parser: 'yaml' }],
       },
     },
     {
+      extends: ['plugin:toml/standard'],
       files: ['*.toml'],
       parser: 'toml-eslint-parser',
-      extends: ['plugin:toml/standard'],
       rules: {
-        'prettier/prettier': 0,
+        'prettier/prettier': 1,
       },
     },
     {
+      extends: ['plugin:md/recommended'],
       files: ['*.md'],
       parser: 'markdown-eslint-parser',
-      extends: ['plugin:md/recommended'],
       rules: {
         'md/remark': [
           1,
@@ -97,41 +115,28 @@ module.exports = {
       },
     },
     {
+      extends: ['plugin:@html-eslint/recommended'],
       files: ['*.html'],
       parser: '@html-eslint/parser',
-      extends: ['plugin:@html-eslint/recommended'],
       plugins: ['@html-eslint'],
       rules: {
-        'spaced-comment': 0,
-        '@html-eslint/indent': 0,
-        '@html-eslint/no-extra-spacing-attrs': 0,
-        '@html-eslint/require-closing-tags': 0,
+        '@html-eslint/indent': 1,
+        '@html-eslint/no-extra-spacing-attrs': 1,
+        '@html-eslint/require-closing-tags': 1,
         'disable-autofix/@html-eslint/require-closing-tags': [
           1,
           { selfClosing: 'always' },
         ],
         'prettier/prettier': [1, { parser: 'html' }],
+        'spaced-comment': 1,
       },
     },
     {
+      extends: ['plugin:vue/vue3-recommended'],
       files: ['*.vue'],
       parser: 'vue-eslint-parser',
-      extends: ['plugin:vue/vue3-recommended'],
       parserOptions: {
-        requireConfigFile: false,
-        ecmaVersion: 'latest',
-        sourceType: 'module',
         babelOptions: {
-          presets: [
-            [
-              '@babel/preset-env',
-              {
-                targets: {
-                  node: 'current',
-                },
-              },
-            ],
-          ],
           plugins: [
             'eslint-plugin-vue',
             [
@@ -142,45 +147,58 @@ module.exports = {
               },
             ],
           ],
+          presets: [
+            [
+              '@babel/preset-env',
+              {
+                targets: {
+                  node: 'current',
+                },
+              },
+            ],
+          ],
         },
+        ecmaVersion: 'latest',
+        requireConfigFile: false,
+        sourceType: 'module',
       },
       rules: {
+        'prettier/prettier': 1,
+        'vue/component-tags-order': [
+          1,
+          {
+            order: [['script', 'template'], 'style'],
+          },
+        ],
         'vue/html-self-closing': [
           1,
           {
             html: {
-              void: 'always',
-              normal: 'always',
               component: 'always',
+              normal: 'always',
+              void: 'always',
             },
-            svg: 'always',
             math: 'always',
+            svg: 'always',
           },
         ],
-        'vue/component-tags-order': [
-          1,
-          {
-            order: [['script', 'template'], 'style'],
-          },
-        ],
-        'prettier/prettier': 0,
       },
     },
     {
       files: ['*.svelte'],
-      processor: 'svelte3/svelte3',
       plugins: ['svelte3'],
+      processor: 'svelte3/svelte3',
       rules: {
-        'import/first': 0,
-        'import/no-duplicates': 0,
-        'import/no-mutable-exports': 0,
-        'import/no-unresolved': 0,
-        'prettier/prettier': 0,
+        'import/first': 1,
+        'import/no-duplicates': 1,
+        'import/no-mutable-exports': 1,
+        'import/no-unresolved': 1,
+        'prettier/prettier': 1,
       },
     },
     {
-      files: ['*.json', '*.jsonc', '*rc'],
       extends: ['plugin:jsonc/recommended-with-json'],
+      files: ['*.json', '*.jsonc', '*rc'],
       parser: 'jsonc-eslint-parser',
       plugins: ['json-format'],
       rules: {
@@ -188,49 +206,56 @@ module.exports = {
       },
     },
     {
-      files: ['*.json5'],
       extends: ['plugin:jsonc/recommended-with-json5'],
+      files: ['*.json5'],
       parser: 'jsonc-eslint-parser',
       rules: {
         'prettier/prettier': [1, { parser: 'json5' }],
       },
     },
   ],
-  env: {
-    browser: true,
-    node: true,
-    es6: true,
-    commonjs: true,
-    es2021: true,
+  parser: '@babel/eslint-parser',
+  parserOptions: {
+    babelOptions: {
+      presets: [
+        [
+          '@babel/preset-env',
+          {
+            targets: {
+              node: 'current',
+            },
+          },
+        ],
+        '@babel/preset-react',
+      ],
+    },
+    ecmaVersion: 'latest',
+    requireConfigFile: false,
+    sourceType: 'module',
   },
+  plugins: [
+    'prettier',
+    'css-modules',
+    'disable-autofix',
+    '@babel',
+    'unicorn',
+    'sonarjs',
+    'only-warn',
+    'sort-keys-fix',
+  ],
+  root: true,
   rules: {
-    'no-restricted-syntax': [
-      1,
-      {
-        selector: 'ForInStatement',
-        message:
-          'for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array.',
-      },
-      {
-        selector: 'LabeledStatement',
-        message:
-          'Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand.',
-      },
-      {
-        selector: 'WithStatement',
-        message:
-          '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
-      },
-    ],
-    camelcase: 0,
-    'no-console': 0,
-    'func-names': 0,
-    'no-process-exit': 0,
-    'class-methods-use-this': 0,
-    'no-underscore-dangle': 0,
+    camelcase: 1,
+    'class-methods-use-this': 1,
+    'constructor-super': 1,
+    'dot-notation': 1,
+    'filenames/match-regex': 1,
+    'func-names': 1,
+    'import/extensions': [1, 'never', { ignorePackages: true }],
+    'import/no-commonjs': 1,
     'import/no-unresolved': [
       1,
-      { commonjs: true, amd: true, ignore: ['^node:'] },
+      { amd: true, commonjs: true, ignore: ['^node:'] },
     ],
     'import/order': [
       1,
@@ -241,41 +266,55 @@ module.exports = {
         'newlines-between': 'always',
       },
     ],
-    'import/extensions': [1, 'never', { ignorePackages: true }],
-    'prefer-destructuring': [
-      1,
-      {
-        object: true,
-        array: false,
-      },
-    ],
+    'jsx-quotes': 1,
     'keyword-spacing': 1,
-    'require-atomic-updates': 0,
-    'import/no-commonjs': 0,
-    'sort-imports': 0,
-    'constructor-super': 1,
     'no-caller': 1,
+    'no-confusing-arrow': 1,
+    'no-console': 1,
     'no-const-assign': 1,
     'no-delete-var': 1,
     'no-dupe-class-members': 1,
     'no-dupe-keys': 1,
     'no-duplicate-imports': 1,
     'no-else-return': 1,
-    'no-empty-pattern': 0,
-    'no-empty': 0,
-    'no-extra-parens': 0,
+    'no-empty': 1,
+    'no-empty-pattern': 1,
+    'no-extra-bind': 1,
+    'no-extra-parens': 1,
+    'no-extra-semi': 1,
+    'no-floating-decimal': 1,
     'no-iterator': 1,
     'no-lonely-if': 1,
     'no-mixed-spaces-and-tabs': [1, 'smart-tabs'],
     'no-multi-str': 1,
     'no-new-wrappers': 1,
+    'no-process-exit': 1,
     'no-proto': 1,
     'no-redeclare': 1,
+    'no-restricted-syntax': [
+      1,
+      {
+        message:
+          'for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array.',
+        selector: 'ForInStatement',
+      },
+      {
+        message:
+          'Labels are a form of GOTO; using them makes code confusing and hard to maintain and understand.',
+        selector: 'LabeledStatement',
+      },
+      {
+        message:
+          '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
+        selector: 'WithStatement',
+      },
+    ],
+    'no-shadow': 1,
     'no-shadow-restricted-names': 1,
-    'no-shadow': 0,
     'no-spaced-func': 1,
     'no-this-before-super': 1,
     'no-undef-init': 1,
+    'no-underscore-dangle': 1,
     'no-unneeded-ternary': 1,
     'no-unused-vars': [
       1,
@@ -290,91 +329,65 @@ module.exports = {
     'no-useless-constructor': 1,
     'no-useless-escape': 1,
     'no-useless-rename': 1,
+    'no-useless-return': 1,
     'no-var': 1,
     'no-with': 1,
-    semi: 0,
-    strict: [1, 'never'],
-    'object-curly-spacing': [0, 'always'],
-    'rest-spread-spacing': 0,
-    'space-before-function-paren': [0, 'always'],
-    'space-in-parens': [0, 'never'],
+    'object-curly-spacing': [1, 'always'],
     'object-shorthand': 1,
     'prefer-arrow-callback': 1,
+    'prefer-destructuring': [
+      1,
+      {
+        array: false,
+        object: true,
+      },
+    ],
     'prefer-rest-params': 1,
     'prefer-spread': 1,
     'prefer-template': 1,
+    'prettier/prettier': [1, { parser: 'babel' }],
+    'quote-props': [1, 'as-needed'],
     quotes: [
-      0,
+      1,
       'single',
       {
-        avoidEscape: true,
         allowTemplateLiterals: true,
+        avoidEscape: true,
       },
     ],
-    'quote-props': [1, 'as-needed'],
     radix: 1,
-    'unicode-bom': 1,
-    'valid-jsdoc': 0,
-    'spaced-comment': [
-      1,
-      'always',
-      {
-        line: {
-          markers: ['/'],
-          exceptions: ['-', '+'],
-        },
-        block: {
-          markers: ['!'],
-          exceptions: ['*'],
-          balanced: true,
-        },
-      },
-    ],
-    'unicorn/import-index': [1, { ignoreImports: true }],
-    'filenames/match-regex': 0,
-    'unicorn/filename-case': [
-      1,
-      {
-        cases: {
-          camelCase: true,
-          pascalCase: true,
-        },
-        ignore: ['README.md'],
-      },
-    ],
-    'react/state-in-constructor': 0,
-    'react/no-deprecated': 1,
-    'react/react-in-jsx-scope': 0,
+    'react-hooks/exhaustive-deps': 1,
+    'react-hooks/rules-of-hooks': 1,
     'react/display-name': [1, { ignoreTranspilerName: false }],
+    'react/function-component-definition': 1,
+    'react/jsx-curly-spacing': 1,
+    'react/jsx-key': [1, { checkFragmentShorthand: true }],
     'react/jsx-no-bind': [
       1,
       {
-        ignoreRefs: true,
-        allowFunctions: true,
         allowArrowFunctions: true,
+        allowFunctions: true,
+        ignoreRefs: true,
       },
     ],
-    'react/no-string-refs': 1,
-    'react/no-find-dom-node': 1,
     'react/jsx-no-comment-textnodes': 1,
-    'react/jsx-curly-spacing': 1,
-    'react/jsx-no-undef': 1,
-    'react/jsx-uses-react': 1,
-    'react/jsx-uses-vars': 1,
-    'react/function-component-definition': 0,
     'react/jsx-no-duplicate-props': 1,
     'react/jsx-no-target-blank': 1,
+    'react/jsx-no-undef': 1,
     'react/jsx-tag-spacing': [1, { beforeSelfClosing: 'always' }],
-    'react/jsx-key': [1, { checkFragmentShorthand: true }],
-    'react/prefer-es6-class': 1,
-    'react/prefer-stateless-function': 1,
-    'react/require-render-return': 1,
+    'react/jsx-uses-react': 1,
+    'react/jsx-uses-vars': 1,
     'react/no-danger': 1,
+    'react/no-deprecated': 1,
     'react/no-did-mount-set-state': 1,
     'react/no-did-update-set-state': 1,
+    'react/no-find-dom-node': 1,
     'react/no-is-mounted': 1,
-    'react-hooks/rules-of-hooks': 1,
-    'react-hooks/exhaustive-deps': 1,
+    'react/no-string-refs': 1,
+    'react/prefer-es6-class': 1,
+    'react/prefer-stateless-function': 1,
+    'react/react-in-jsx-scope': 1,
+    'react/require-render-return': 1,
     'react/self-closing-comp': [
       'error',
       {
@@ -382,54 +395,51 @@ module.exports = {
         html: false,
       },
     ],
-    'prettier/prettier': [1, { parser: 'babel' }],
+    'react/state-in-constructor': 1,
+    'require-atomic-updates': 1,
+    'rest-spread-spacing': 1,
+    semi: 1,
+    'sort-imports': 1,
+    'sort-keys-fix/sort-keys-fix': 1,
+    'sort-vars': 1,
+    'space-before-function-paren': [1, 'always'],
+    'space-in-parens': [1, 'never'],
+    'spaced-comment': [
+      1,
+      'always',
+      {
+        block: {
+          balanced: true,
+          exceptions: ['*'],
+          markers: ['!'],
+        },
+        line: {
+          exceptions: ['-', '+'],
+          markers: ['/'],
+        },
+      },
+    ],
+    strict: [1, 'never'],
+    'unicode-bom': 1,
+    'unicorn/filename-case': [
+      1,
+      {
+        cases: {
+          camelCase: true,
+          pascalCase: true,
+        },
+        ignore: ['README.md'],
+      },
+    ],
+    'unicorn/import-index': [1, { ignoreImports: true }],
+    'valid-jsdoc': 1,
   },
-  plugins: [
-    'prettier',
-    'css-modules',
-    'disable-autofix',
-    '@babel',
-    'unicorn',
-    'sonarjs',
-    'only-warn',
-  ],
-  ignorePatterns: [
-    '!.*',
-    'node_modules',
-    'vendor',
-    'dist',
-    'package-lock.json',
-    'LICENSE',
-    '.vscode',
-    '.github',
-    'tsconfig.json',
-  ],
-  root: true,
   settings: {
     'json/sort-package-json': 'pro',
     polyfills: ['Promise'],
     react: {
       pragma: 'h',
-      version: 'preact',
-    },
-  },
-  parser: '@babel/eslint-parser',
-  parserOptions: {
-    requireConfigFile: false,
-    ecmaVersion: 'latest',
-    sourceType: 'module',
-    babelOptions: {
-      presets: [
-        [
-          '@babel/preset-env',
-          {
-            targets: {
-              node: 'current',
-            },
-          },
-        ],
-        '@babel/preset-react',
-      ],
+      version: 'detect',
     },
   },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikey-pro/eslint-config",
-  "version": "4.7.0",
+  "version": "4.8.2",
   "description": "Mikey Pro ESLint configuration",
   "main": "index.js",
   "dependencies": {
@@ -11,10 +11,10 @@
     "@babel/preset-env": "^7.16",
     "@html-eslint/eslint-plugin": "^0.13.1",
     "@html-eslint/parser": "^0.13.1",
-    "@typescript-eslint/eslint-plugin": "^5.12.0",
-    "@typescript-eslint/parser": "^5.12.0",
+    "@typescript-eslint/eslint-plugin": "^5.13.0",
+    "@typescript-eslint/parser": "^5.13.0",
     "@vue/babel-plugin-jsx": "^1.1.1",
-    "eslint-config-prettier": "^8.3",
+    "eslint-config-prettier": "^8.4",
     "eslint-config-react-app": "^7.0.0",
     "eslint-import-resolver-typescript": "^2.5.0",
     "eslint-plugin-compat": "^4.0",
@@ -27,13 +27,14 @@
     "eslint-plugin-only-warn": "^1.0.3",
     "eslint-plugin-prettier": "^4.0",
     "eslint-plugin-promise": "^6.0.0",
-    "eslint-plugin-sonarjs": "^0.11.0",
-    "eslint-plugin-svelte3": "^3.4.0",
+    "eslint-plugin-sonarjs": "^0.12.0",
+    "eslint-plugin-sort-keys-fix": "^1.1.2",
+    "eslint-plugin-svelte3": "^3.4.1",
     "eslint-plugin-toml": "^0.3.0",
     "eslint-plugin-unicorn": "^41.0.0",
-    "eslint-plugin-vue": "^8.4",
+    "eslint-plugin-vue": "^8.5",
     "eslint-plugin-yaml": "^0.5.0",
-    "vue-eslint-parser": "^8.2"
+    "vue-eslint-parser": "^8.3"
   },
   "peerDependencies": {
     "eslint": ">=8.3.0"