@perfective/eslint-config 0.17.0-alpha.2 → 0.18.0-alpha

package/LICENSE

@@ -1,6 +1,6 @@
 The MIT License
 
-Copyright (c) 2021 Andrey Mikheychik (https://github.com/amikheychik)
+Copyright (c) 2020-2022 Andrey Mikheychik (https://github.com/amikheychik)
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@perfective/eslint-config",
-    "version": "0.17.0-alpha.2",
+    "version": "0.18.0-alpha",
     "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.16.5",
-        "@typescript-eslint/eslint-plugin": "^5.10.1",
-        "@typescript-eslint/eslint-plugin-tslint": "^5.10.1",
-        "@typescript-eslint/parser": "^5.10.1",
-        "eslint": "^8.8.0",
-        "eslint-import-resolver-typescript": "^2.5.0",
+        "@babel/eslint-parser": "^7.17.0",
+        "@typescript-eslint/eslint-plugin": "^5.17.0",
+        "@typescript-eslint/eslint-plugin-tslint": "^5.17.0",
+        "@typescript-eslint/parser": "^5.17.0",
+        "eslint": "^8.12.0",
+        "eslint-import-resolver-typescript": "^2.7.1",
         "eslint-plugin-array-func": "^3.1.7",
         "eslint-plugin-cypress": "^2.12.1",
         "eslint-plugin-deprecation": "^1.3.2",
         "eslint-plugin-eslint-comments": "^3.2.0",
         "eslint-plugin-import": "^2.25.4",
-        "eslint-plugin-jest": "^26.0.0",
+        "eslint-plugin-jest": "^26.1.3",
         "eslint-plugin-jest-dom": "^4.0.1",
         "eslint-plugin-jest-formatting": "^3.1.0",
-        "eslint-plugin-jsdoc": "^37.7.0",
+        "eslint-plugin-jsdoc": "^38.1.6",
         "eslint-plugin-node": "^11.1.0",
         "eslint-plugin-prefer-arrow": "^1.2.3",
         "eslint-plugin-promise": "^6.0.0",
         "eslint-plugin-rxjs": "^5.0.2",
         "eslint-plugin-simple-import-sort": "^7.0.0",
-        "eslint-plugin-sonarjs": "^0.11.0",
-        "eslint-plugin-testing-library": "^5.0.4",
-        "eslint-plugin-unicorn": "^40.1.0",
+        "eslint-plugin-sonarjs": "^0.13.0",
+        "eslint-plugin-testing-library": "^5.2.0",
+        "eslint-plugin-unicorn": "^42.0.0",
         "tslint": "^6.1.3"
     },
     "main": "./index.js",

package/rules/eslint/possible-problems.js

@@ -58,6 +58,10 @@ module.exports = {
         'no-unused-vars': ['error', {
                 args: 'after-used',
                 argsIgnorePattern: '^_',
+                caughtErrors: 'all',
+                destructuredArrayIgnorePattern: '^_',
+                ignoreRestSiblings: false,
+                vars: 'all',
             }],
         'no-use-before-define': ['error', {
                 functions: false,
@@ -68,6 +72,8 @@ module.exports = {
                 allowProperties: false,
             }],
         'use-isnan': 'error',
-        'valid-typeof': 'error',
+        'valid-typeof': ['error', {
+                requireStringLiterals: true,
+            }],
     },
 };

package/rules/eslint/suggestions.js

@@ -54,6 +54,7 @@ module.exports = {
         'no-case-declarations': 'error',
         'no-confusing-arrow': ['warn', {
                 allowParens: true,
+                onlyOneSimpleParam: false,
             }],
         'no-console': 'error',
         'no-continue': 'error',

package/rules/import/static-analysis.js

@@ -6,7 +6,7 @@ module.exports = {
                 caseSensitiveStrict: true,
             }],
         'import/named': ['error', {
-                commonjs: true,
+                commonjs: false,
             }],
         'import/default': 'error',
         'import/namespace': 'error',

package/rules/jest/index.js

@@ -21,6 +21,7 @@ module.exports = {
         'jest/no-alias-methods': 'warn',
         'jest/no-commented-out-tests': 'error',
         'jest/no-conditional-expect': 'error',
+        'jest/no-conditional-in-test': 'error',
         'jest/no-deprecated-functions': 'warn',
         'jest/no-disabled-tests': 'error',
         'jest/no-done-callback': 'error',
@@ -29,7 +30,7 @@ module.exports = {
         'jest/no-focused-tests': 'warn',
         'jest/no-hooks': 'off',
         'jest/no-identical-title': 'error',
-        'jest/no-if': 'error',
+        'jest/no-if': 'off',
         'jest/no-interpolation-in-snapshots': 'error',
         'jest/no-jasmine-globals': 'warn',
         'jest/no-jest-import': 'error',
@@ -56,6 +57,7 @@ module.exports = {
                 ignore: [],
                 ignoreTopLevelDescribe: true,
             }],
+        'jest/prefer-snapshot-hint': ['error', 'always'],
         'jest/prefer-spy-on': 'warn',
         'jest/prefer-strict-equal': 'warn',
         'jest/prefer-to-be': 'warn',

package/rules/jsdoc/index.js

@@ -103,8 +103,10 @@ module.exports = {
         'jsdoc/require-throws': 'error',
         'jsdoc/require-yields': 'error',
         'jsdoc/require-yields-check': 'error',
+        'jsdoc/sort-tags': 'warn',
         'jsdoc/tag-lines': ['off', 'never', {
                 count: 1,
+                dropEndLines: false,
                 noEndLines: true,
                 tags: {},
             }],

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

@@ -9,7 +9,7 @@ module.exports = {
         'sonarjs/no-duplicate-string': 'off',
         'sonarjs/no-duplicated-branches': 'error',
         'sonarjs/no-gratuitous-expressions': 'error',
-        'sonarjs/no-identical-functions': 'error',
+        'sonarjs/no-identical-functions': ['error', 3],
         'sonarjs/no-inverted-boolean-check': 'warn',
         'sonarjs/no-nested-switch': 'error',
         'sonarjs/no-nested-template-literals': 'error',

package/rules/testing-library/index.js

@@ -16,6 +16,7 @@ module.exports = {
         'testing-library/no-container': 'error',
         'testing-library/no-debugging-utils': 'error',
         'testing-library/no-dom-import': 'warn',
+        'testing-library/no-global-regexp-flag-in-query': 'warn',
         'testing-library/no-manual-cleanup': 'error',
         'testing-library/no-node-access': 'error',
         'testing-library/no-promise-in-fire-event': 'error',
@@ -27,7 +28,10 @@ module.exports = {
         'testing-library/no-wait-for-snapshot': 'error',
         'testing-library/prefer-explicit-assert': 'error',
         'testing-library/prefer-find-by': 'warn',
-        'testing-library/prefer-presence-queries': 'error',
+        'testing-library/prefer-presence-queries': ['error', {
+                presence: true,
+                absence: true,
+            }],
         'testing-library/prefer-query-by-disappearance': 'error',
         'testing-library/prefer-user-event': 'error',
         'testing-library/prefer-screen-queries': 'error',

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

@@ -99,6 +99,10 @@ module.exports = {
         '@typescript-eslint/no-unused-vars': ['error', {
                 args: 'after-used',
                 argsIgnorePattern: '^_',
+                caughtErrors: 'all',
+                destructuredArrayIgnorePattern: '^_',
+                ignoreRestSiblings: false,
+                vars: 'all',
             }],
         'no-use-before-define': 'off',
         '@typescript-eslint/no-use-before-define': ['error', {
@@ -126,6 +130,8 @@ module.exports = {
         '@typescript-eslint/return-await': ['warn', 'never'],
         'semi': 'off',
         '@typescript-eslint/semi': ['warn', 'always'],
+        'space-before-blocks': 'off',
+        '@typescript-eslint/space-before-blocks': ['warn', 'always'],
         'space-before-function-paren': 'off',
         '@typescript-eslint/space-before-function-paren': ['warn', {
                 anonymous: 'always',

package/rules/typescript-eslint/rules/typescript-eslint-naming-convention.d.ts

@@ -5,7 +5,7 @@ export declare type TypescriptEslintNamingConventionSelector = TypescriptEslintN
 export declare type TypescriptEslintNamingConventionUnderscore = 'forbid' | 'require' | 'requireDouble' | 'allow' | 'allowDouble' | 'allowSingleOrDouble';
 export interface TypescriptEslintNamingConvention {
     selector: TypescriptEslintNamingConventionSelector | TypescriptEslintNamingConventionSelector[];
-    format?: TypescriptEslintNamingConventionFormat[];
+    format: TypescriptEslintNamingConventionFormat[] | null;
     modifiers?: string[];
     leadingUnderscore?: TypescriptEslintNamingConventionUnderscore;
     trailingUnderscore?: TypescriptEslintNamingConventionUnderscore;

package/rules/typescript-eslint/rules/typescript-eslint-naming-convention.js

@@ -14,6 +14,10 @@ function typescriptEslintNamingConventionDefaults() {
         }, {
             selector: ['memberLike', 'variableLike'],
             format: ['strictCamelCase'],
+        }, {
+            selector: ['objectLiteralProperty'],
+            modifiers: ['requiresQuotes'],
+            format: null,
         }, {
             selector: ['parameter'],
             modifiers: ['unused'],

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

@@ -145,12 +145,16 @@ module.exports = {
                 checkNever: true,
             }],
         '@typescript-eslint/no-misused-new': 'error',
-        '@typescript-eslint/no-misused-promises': 'error',
+        '@typescript-eslint/no-misused-promises': ['error', {
+                checksConditionals: true,
+                checksVoidReturn: true,
+            }],
         '@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',
+        '@typescript-eslint/no-redundant-type-constituents': 'error',
         '@typescript-eslint/no-require-imports': 'error',
         '@typescript-eslint/no-this-alias': 'error',
         '@typescript-eslint/no-type-alias': 'off',
@@ -253,5 +257,6 @@ module.exports = {
                 ignoreStatic: true,
             }],
         '@typescript-eslint/unified-signatures': 'error',
+        '@typescript-eslint/no-useless-empty-export': 'warn',
     },
 };

package/rules/unicorn/index.js

@@ -55,12 +55,14 @@ module.exports = {
         'unicorn/no-thenable': 'error',
         'unicorn/no-this-assignment': 'error',
         'unicorn/no-unreadable-array-destructuring': 'error',
+        'unicorn/no-unreadable-iife': '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-promise-resolve-reject': 'warn',
         'unicorn/no-useless-spread': 'warn',
+        'unicorn/no-useless-switch-case': 'error',
         'unicorn/no-useless-undefined': 'off',
         'unicorn/no-zero-fractions': 'off',
         'unicorn/number-literal-case': 'warn',
@@ -102,7 +104,9 @@ module.exports = {
         'unicorn/prefer-keyboard-event-key': 'warn',
         'unicorn/prefer-math-trunc': 'warn',
         'unicorn/prefer-modern-dom-apis': 'warn',
+        'unicorn/prefer-modern-math-apis': 'warn',
         'unicorn/prefer-module': 'off',
+        'unicorn/prefer-native-coercion-functions': 'warn',
         'unicorn/prefer-negative-index': 'warn',
         'unicorn/prefer-node-protocol': ['warn', {
                 checkRequire: false,
@@ -139,6 +143,7 @@ module.exports = {
         'unicorn/require-post-message-target-origin': 'error',
         'unicorn/string-content': 'off',
         'unicorn/template-indent': 'warn',
+        'unicorn/text-encoding-identifier-case': 'error',
         'unicorn/throw-new-error': 'warn',
     },
 };