@perfective/eslint-config 0.15.1 → 0.16.0-beta

package/README.adoc

@@ -14,7 +14,7 @@ and the `link:https://github.com/typescript-eslint/typescript-eslint/tree/master
 * `link:https://github.com/cypress-io/eslint-plugin-cypress[eslint-plugin-cypress]`;
 * `link:https://github.com/gund/eslint-plugin-deprecation[eslint-plugin-deprecation]`;
 * `link:https://mysticatea.github.io/eslint-plugin-eslint-comments[eslint-plugin-eslint-comments]`;
-* `link:https://github.com/benmosher/eslint-plugin-import[eslint-plugin-import]`;
+* `link:https://github.com/import-js/eslint-plugin-import[eslint-plugin-import]`;
 * `link:https://github.com/jest-community/eslint-plugin-jest[eslint-plugin-jest]`;
 * `link:https://github.com/testing-library/eslint-plugin-jest-dom[eslint-plugin-jest-dom]`;
 * `link:https://github.com/dangreenisrael/eslint-plugin-jest-formatting[eslint-plugin-jest-formatting]`;

package/README.md

@@ -14,7 +14,7 @@ and the [`@typescript-eslint`](https://github.com/typescript-eslint/typescript-e
 -   [`eslint-plugin-cypress`](https://github.com/cypress-io/eslint-plugin-cypress);
 -   [`eslint-plugin-deprecation`](https://github.com/gund/eslint-plugin-deprecation);
 -   [`eslint-plugin-eslint-comments`](https://mysticatea.github.io/eslint-plugin-eslint-comments);
--   [`eslint-plugin-import`](https://github.com/benmosher/eslint-plugin-import);
+-   [`eslint-plugin-import`](https://github.com/import-js/eslint-plugin-import);
 -   [`eslint-plugin-jest`](https://github.com/jest-community/eslint-plugin-jest);
 -   [`eslint-plugin-jest-dom`](https://github.com/testing-library/eslint-plugin-jest-dom);
 -   [`eslint-plugin-jest-formatting`](https://github.com/dangreenisrael/eslint-plugin-jest-formatting);

package/index.js

@@ -39,7 +39,7 @@ module.exports = {
             },
             rules: {
                 'import/no-commonjs': 'off',
-                'import/no-extraneous-dependencies': ['error', no_extraneous_dependencies_1.jsImportNoExtraneousDependencies()],
+                'import/no-extraneous-dependencies': ['error', (0, no_extraneous_dependencies_1.jsImportNoExtraneousDependencies)()],
                 'import/unambiguous': 'off',
                 'jsdoc/no-types': 'off',
                 'jsdoc/require-param': 'error',
@@ -100,7 +100,7 @@ module.exports = {
                     }],
                 '@typescript-eslint/init-declarations': 'off',
                 '@typescript-eslint/unbound-method': 'off',
-                'import/no-extraneous-dependencies': ['error', no_extraneous_dependencies_1.jestImportNoExtraneousDependencies()],
+                'import/no-extraneous-dependencies': ['error', (0, no_extraneous_dependencies_1.jestImportNoExtraneousDependencies)()],
                 'import/no-unassigned-import': ['error', {
                         allow: [
                             '@testing-library/jest-dom',
@@ -124,7 +124,7 @@ module.exports = {
             rules: {
                 'init-declarations': 'off',
                 '@typescript-eslint/init-declarations': 'off',
-                'import/no-extraneous-dependencies': ['error', no_extraneous_dependencies_1.cypressImportNoExtraneousDependencies()],
+                'import/no-extraneous-dependencies': ['error', (0, no_extraneous_dependencies_1.cypressImportNoExtraneousDependencies)()],
                 'max-nested-callbacks': ['error', 4],
                 'new-cap': ['error', {
                         capIsNewExceptions: ['Given', 'When', 'Then', 'And', 'But', 'Before', 'After'],

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@perfective/eslint-config",
-    "version": "0.15.1",
+    "version": "0.16.0-beta",
     "description": "ESLint shareable rules configuration",
     "keywords": ["code quality", "code standard", "code style", "eslint", "eslint-config", "lint", "perfective", "tslint", "tslint-config", "typescript"],
     "author": "Andrey Mikheychik <a.mikheychik@gmail.com>",
@@ -11,29 +11,29 @@
     },
     "license": "MIT",
     "peerDependencies": {
-        "@babel/eslint-parser": "^7.14.7",
-        "@typescript-eslint/eslint-plugin": "^4.28.0",
-        "@typescript-eslint/eslint-plugin-tslint": "^4.28.0",
-        "@typescript-eslint/parser": "^4.28.0",
-        "eslint": "^7.29.0",
-        "eslint-import-resolver-typescript": "^2.4.0",
+        "@babel/eslint-parser": "^7.16.5",
+        "@typescript-eslint/eslint-plugin": "^4.33.0",
+        "@typescript-eslint/eslint-plugin-tslint": "^4.33.0",
+        "@typescript-eslint/parser": "^4.33.0",
+        "eslint": "^7.32.0",
+        "eslint-import-resolver-typescript": "^2.5.0",
         "eslint-plugin-array-func": "^3.1.7",
-        "eslint-plugin-cypress": "^2.11.3",
-        "eslint-plugin-deprecation": "^1.2.1",
+        "eslint-plugin-cypress": "^2.12.1",
+        "eslint-plugin-deprecation": "^1.3.2",
         "eslint-plugin-eslint-comments": "^3.2.0",
-        "eslint-plugin-import": "^2.23.4",
-        "eslint-plugin-jest": "^24.3.6",
-        "eslint-plugin-jest-dom": "^3.9.0",
-        "eslint-plugin-jest-formatting": "^3.0.0",
-        "eslint-plugin-jsdoc": "^35.4.0",
+        "eslint-plugin-import": "^2.25.3",
+        "eslint-plugin-jest": "^24.7.0",
+        "eslint-plugin-jest-dom": "^4.0.0",
+        "eslint-plugin-jest-formatting": "^3.1.0",
+        "eslint-plugin-jsdoc": "^37.4.2",
         "eslint-plugin-node": "^11.1.0",
         "eslint-plugin-prefer-arrow": "^1.2.3",
-        "eslint-plugin-promise": "^5.1.0",
-        "eslint-plugin-rxjs": "^3.3.3",
+        "eslint-plugin-promise": "^6.0.0",
+        "eslint-plugin-rxjs": "^3.3.7",
         "eslint-plugin-simple-import-sort": "^7.0.0",
-        "eslint-plugin-sonarjs": "^0.7.0",
-        "eslint-plugin-testing-library": "^4.6.0",
-        "eslint-plugin-unicorn": "^33.0.1",
+        "eslint-plugin-sonarjs": "^0.11.0",
+        "eslint-plugin-testing-library": "^4.12.4",
+        "eslint-plugin-unicorn": "^36.0.0",
         "tslint": "^6.1.3"
     },
     "main": "./index.js",

package/rules/cypress/index.js

@@ -8,6 +8,7 @@ module.exports = {
         'cypress/no-assigning-return-values': 'error',
         'cypress/no-async-tests': 'error',
         'cypress/no-force': 'error',
+        'cypress/no-pause': 'error',
         'cypress/no-unnecessary-waiting': 'error',
         'cypress/require-data-selectors': 'error',
     },

package/rules/import/helpful-warnings.js

@@ -6,7 +6,7 @@ module.exports = {
         'import/no-named-as-default': 'error',
         'import/no-named-as-default-member': 'error',
         'import/no-deprecated': 'off',
-        'import/no-extraneous-dependencies': ['error', no_extraneous_dependencies_1.importNoExtraneousDependencies()],
+        'import/no-extraneous-dependencies': ['error', (0, no_extraneous_dependencies_1.importNoExtraneousDependencies)()],
         'import/no-mutable-exports': 'error',
         'import/no-unused-modules': 'error',
     },

package/rules/import/static-analysis.js

@@ -1,13 +1,20 @@
 "use strict";
 module.exports = {
     rules: {
-        'import/no-unresolved': 'error',
-        'import/named': 'error',
+        'import/no-unresolved': ['error', {
+                caseSensitive: true,
+                caseSensitiveStrict: true,
+            }],
+        'import/named': ['error', {
+                commonjs: true,
+            }],
         'import/default': 'error',
         'import/namespace': 'error',
         'import/no-restricted-paths': 'off',
         'import/no-absolute-path': 'error',
-        'import/no-dynamic-require': 'error',
+        'import/no-dynamic-require': ['error', {
+                esmodule: true,
+            }],
         'import/no-internal-modules': 'off',
         'import/no-webpack-loader-syntax': 'error',
         'import/no-self-import': 'error',

package/rules/jest/index.js

@@ -17,6 +17,9 @@ module.exports = {
         'jest/lowercase-name': ['error', {
                 ignore: ['describe'],
             }],
+        'jest/max-nested-describe': ['error', {
+                max: 3,
+            }],
         'jest/no-alias-methods': 'warn',
         'jest/no-commented-out-tests': 'error',
         'jest/no-conditional-expect': 'error',
@@ -46,18 +49,21 @@ module.exports = {
         'jest/no-test-return-statement': 'error',
         'jest/prefer-called-with': 'error',
         'jest/prefer-expect-assertions': 'off',
+        'jest/prefer-expect-resolves': 'warn',
         'jest/prefer-hooks-on-top': 'error',
         'jest/prefer-spy-on': 'warn',
         'jest/prefer-strict-equal': 'warn',
+        'jest/prefer-to-be': 'warn',
         'jest/prefer-to-be-null': 'warn',
         'jest/prefer-to-be-undefined': 'warn',
         'jest/prefer-to-contain': 'warn',
         'jest/prefer-to-have-length': 'warn',
         'jest/prefer-todo': 'warn',
+        'jest/require-hook': 'error',
         'jest/require-top-level-describe': 'error',
         'jest/require-to-throw-message': 'error',
         'jest/valid-describe': 'error',
-        'jest/valid-expect-in-promise': 'off',
+        'jest/valid-expect-in-promise': 'error',
         'jest/valid-expect': 'error',
         'jest/valid-title': ['warn', {
                 ignoreTypeOfDescribeName: true,

package/rules/jsdoc/index.js

@@ -17,6 +17,7 @@ module.exports = {
                     replacement: 'emits',
                 },
             },
+            ignoreReplacesDocs: true,
             overrideReplacesDocs: true,
             augmentsExtendsReplacesDocs: false,
             implementsReplacesDoc: false,

package/rules/simple-import-sort/index.js

@@ -6,6 +6,6 @@ module.exports = {
     ],
     rules: {
         'simple-import-sort/exports': 'warn',
-        'simple-import-sort/imports': ['warn', imports_1.simpleImportSortImports()],
+        'simple-import-sort/imports': ['warn', (0, imports_1.simpleImportSortImports)()],
     },
 };

package/rules/sonarjs/bug-detection.js

@@ -3,10 +3,13 @@ module.exports = {
     rules: {
         'sonarjs/no-all-duplicated-branches': 'error',
         'sonarjs/no-element-overwrite': 'error',
+        'sonarjs/no-empty-collection': 'error',
         'sonarjs/no-extra-arguments': 'error',
         'sonarjs/no-identical-conditions': 'error',
         'sonarjs/no-identical-expressions': 'error',
+        'sonarjs/no-ignored-return': 'error',
         'sonarjs/no-one-iteration-loop': 'error',
         'sonarjs/no-use-of-empty-return-value': 'error',
+        'sonarjs/non-existent-operator': 'error',
     },
 };

package/rules/sonarjs/code-smell-detection.js

@@ -2,13 +2,17 @@
 module.exports = {
     rules: {
         'sonarjs/cognitive-complexity': ['off', 5],
+        'sonarjs/elseif-without-else': 'off',
         'sonarjs/max-switch-cases': 'off',
         'sonarjs/no-collapsible-if': 'error',
         'sonarjs/no-collection-size-mischeck': 'error',
         'sonarjs/no-duplicate-string': 'off',
         'sonarjs/no-duplicated-branches': 'error',
+        'sonarjs/no-gratuitous-expressions': 'error',
         'sonarjs/no-identical-functions': 'error',
-        'sonarjs/no-inverted-boolean-check': 'error',
+        'sonarjs/no-inverted-boolean-check': 'warn',
+        'sonarjs/no-nested-switch': 'error',
+        'sonarjs/no-nested-template-literals': 'error',
         'sonarjs/no-redundant-boolean': 'error',
         'sonarjs/no-redundant-jump': 'error',
         'sonarjs/no-same-line-conditional': 'error',

package/rules/testing-library/index.js

@@ -28,6 +28,7 @@ module.exports = {
         'testing-library/prefer-explicit-assert': 'error',
         'testing-library/prefer-find-by': 'warn',
         'testing-library/prefer-presence-queries': 'error',
+        'testing-library/prefer-query-by-disappearance': 'error',
         'testing-library/prefer-user-event': 'error',
         'testing-library/prefer-screen-queries': 'error',
         'testing-library/prefer-wait-for': 'warn',

package/rules/typescript-eslint/extension-rules.js

@@ -82,6 +82,10 @@ module.exports = {
         '@typescript-eslint/no-magic-numbers': 'off',
         'no-redeclare': 'off',
         '@typescript-eslint/no-redeclare': 'error',
+        'no-restricted-imports': 'off',
+        '@typescript-eslint/no-restricted-imports': ['error', {
+                allowTypeImports: false,
+            }],
         'no-shadow': 'off',
         '@typescript-eslint/no-shadow': 'off',
         'no-throw-literal': 'off',
@@ -106,6 +110,8 @@ module.exports = {
                 arraysInObjects: true,
                 objectsInObjects: true,
             }],
+        'padding-line-between-statements': 'off',
+        '@typescript-eslint/padding-line-between-statements': 'off',
         'quotes': 'off',
         '@typescript-eslint/quotes': ['warn', 'single', {
                 avoidEscape: true,

package/rules/typescript-eslint/supported-rules.js

@@ -71,7 +71,7 @@ module.exports = {
                 ],
             }],
         '@typescript-eslint/method-signature-style': ['warn', 'property'],
-        '@typescript-eslint/naming-convention': ['error', ...typescript_eslint_naming_convention_1.typescriptEslintNamingConvention()],
+        '@typescript-eslint/naming-convention': ['error', ...(0, typescript_eslint_naming_convention_1.typescriptEslintNamingConvention)()],
         '@typescript-eslint/no-base-to-string': 'error',
         '@typescript-eslint/no-confusing-non-null-assertion': 'warn',
         '@typescript-eslint/no-confusing-void-expression': ['warn', {
@@ -102,9 +102,13 @@ module.exports = {
         '@typescript-eslint/no-invalid-void-type': ['error', {
                 allowInGenericTypeArguments: true,
             }],
+        '@typescript-eslint/no-meaningless-void-operator': ['warn', {
+                checkNever: true,
+            }],
         '@typescript-eslint/no-misused-new': 'error',
         '@typescript-eslint/no-misused-promises': 'error',
         '@typescript-eslint/no-namespace': 'error',
+        '@typescript-eslint/no-non-null-asserted-nullish-coalescing': 'error',
         '@typescript-eslint/no-non-null-asserted-optional-chain': 'error',
         '@typescript-eslint/no-non-null-assertion': 'error',
         '@typescript-eslint/no-parameter-properties': 'off',
@@ -145,6 +149,7 @@ module.exports = {
         '@typescript-eslint/prefer-readonly-parameter-types': 'off',
         '@typescript-eslint/prefer-reduce-type-parameter': 'warn',
         '@typescript-eslint/prefer-regexp-exec': 'warn',
+        '@typescript-eslint/prefer-return-this-type': 'warn',
         '@typescript-eslint/prefer-string-starts-ends-with': 'warn',
         '@typescript-eslint/prefer-ts-expect-error': 'warn',
         '@typescript-eslint/promise-function-async': ['warn', {
@@ -157,10 +162,11 @@ module.exports = {
                 checkCompoundAssignments: true,
             }],
         '@typescript-eslint/restrict-template-expressions': ['error', {
-                allowNumber: false,
-                allowBoolean: false,
                 allowAny: false,
+                allowBoolean: false,
                 allowNullish: false,
+                allowNumber: false,
+                allowRegExp: false,
             }],
         '@typescript-eslint/sort-type-union-intersection-members': ['off', {
                 checkIntersections: true,

package/rules/typescript-eslint/tslint.js

@@ -5,6 +5,6 @@ module.exports = {
         '@typescript-eslint/tslint',
     ],
     rules: {
-        '@typescript-eslint/tslint/config': ['warn', typescript_eslint_tslint_config_1.typescriptEslintTslintConfig()],
+        '@typescript-eslint/tslint/config': ['warn', (0, typescript_eslint_tslint_config_1.typescriptEslintTslintConfig)()],
     },
 };

package/rules/unicorn/index.js

@@ -26,15 +26,19 @@ module.exports = {
         'unicorn/no-abusive-eslint-disable': 'error',
         'unicorn/no-array-callback-reference': 'off',
         'unicorn/no-array-for-each': 'off',
+        'unicorn/no-array-method-this-argument': 'warn',
         'unicorn/no-array-push-push': ['warn', {
                 ignore: ['readable', 'this', 'this.stream'],
             }],
         'unicorn/no-array-reduce': 'off',
+        'unicorn/no-await-expression-member': 'error',
         'unicorn/no-console-spaces': 'warn',
         'unicorn/no-document-cookie': 'error',
+        'unicorn/no-empty-file': 'error',
         'unicorn/no-for-loop': 'warn',
         'unicorn/no-hex-escape': 'warn',
         'unicorn/no-instanceof-array': 'warn',
+        'unicorn/no-invalid-remove-event-listener': 'error',
         'unicorn/no-keyword-prefix': ['off', {
                 disallowedPrefixes: ['new', 'class'],
                 checkProperties: true,
@@ -52,6 +56,9 @@ module.exports = {
         'unicorn/no-unreadable-array-destructuring': 'error',
         'unicorn/no-unsafe-regex': 'error',
         'unicorn/no-unused-properties': 'off',
+        'unicorn/no-useless-fallback-in-spread': 'warn',
+        'unicorn/no-useless-length-check': 'warn',
+        'unicorn/no-useless-spread': 'warn',
         'unicorn/no-useless-undefined': 'off',
         'unicorn/no-zero-fractions': 'off',
         'unicorn/number-literal-case': 'warn',
@@ -79,23 +86,29 @@ module.exports = {
         'unicorn/prefer-array-flat-map': 'off',
         'unicorn/prefer-array-index-of': 'warn',
         'unicorn/prefer-array-some': 'error',
+        'unicorn/prefer-at': 'off',
+        'unicorn/prefer-code-point': 'error',
         'unicorn/prefer-date-now': 'warn',
         'unicorn/prefer-default-parameters': 'warn',
         'unicorn/prefer-dom-node-append': 'warn',
         'unicorn/prefer-dom-node-dataset': 'warn',
         'unicorn/prefer-dom-node-remove': 'warn',
         'unicorn/prefer-dom-node-text-content': 'warn',
+        'unicorn/prefer-export-from': 'warn',
         'unicorn/prefer-includes': 'warn',
         'unicorn/prefer-keyboard-event-key': 'warn',
         'unicorn/prefer-math-trunc': 'warn',
         'unicorn/prefer-modern-dom-apis': 'warn',
         'unicorn/prefer-module': 'off',
         'unicorn/prefer-negative-index': 'warn',
+        'unicorn/prefer-node-protocol': ['warn', {
+                checkRequire: false,
+            }],
         'unicorn/prefer-number-properties': ['warn', {
                 checkInfinity: true,
             }],
-        'unicorn/prefer-node-protocol': ['warn', {
-                checkRequire: false,
+        'unicorn/prefer-object-from-entries': ['off', {
+                functions: [],
             }],
         'unicorn/prefer-object-has-own': 'off',
         'unicorn/prefer-optional-catch-binding': 'warn',
@@ -114,11 +127,14 @@ module.exports = {
                 emptyDefaultCase: 'do-nothing-comment',
             }],
         'unicorn/prefer-ternary': 'off',
+        'unicorn/prefer-top-level-await': 'off',
         'unicorn/prefer-type-error': 'warn',
-        'unicorn/prevent-abbreviations': ['warn', prevent_abbreviations_1.unicornPreventAbbreviations()],
+        'unicorn/prevent-abbreviations': ['warn', (0, prevent_abbreviations_1.unicornPreventAbbreviations)()],
         'unicorn/require-array-join-separator': 'warn',
         'unicorn/require-number-to-fixed-digits-argument': 'warn',
+        'unicorn/require-post-message-target-origin': 'error',
         'unicorn/string-content': 'off',
+        'unicorn/template-indent': 'warn',
         'unicorn/throw-new-error': 'warn',
     },
 };