@perfective/eslint-config 0.20.0-alpha → 0.20.0-alpha.2

package/index.js

@@ -107,7 +107,7 @@ module.exports = {
                             '@testing-library/jest-dom/extend-expect',
                         ],
                     }],
-                'max-nested-callbacks': ['error', 4],
+                'max-nested-callbacks': ['error', 5],
                 'prefer-arrow/prefer-arrow-functions': 'off',
                 'promise/always-return': 'off',
                 'rxjs/no-topromise': 'off',

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@perfective/eslint-config",
-    "version": "0.20.0-alpha",
+    "version": "0.20.0-alpha.2",
     "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>",
@@ -12,17 +12,17 @@
     "license": "MIT",
     "peerDependencies": {
         "@babel/eslint-parser": "^7.19.1",
-        "@typescript-eslint/eslint-plugin": "^5.47.1",
-        "@typescript-eslint/eslint-plugin-tslint": "^5.47.1",
-        "@typescript-eslint/parser": "^5.47.1",
-        "eslint": "^8.30.0",
-        "eslint-import-resolver-typescript": "^3.5.2",
-        "eslint-plugin-array-func": "^3.1.7",
+        "@typescript-eslint/eslint-plugin": "^5.48.2",
+        "@typescript-eslint/eslint-plugin-tslint": "^5.48.2",
+        "@typescript-eslint/parser": "^5.48.2",
+        "eslint": "^8.32.0",
+        "eslint-import-resolver-typescript": "^3.5.3",
+        "eslint-plugin-array-func": "^3.1.8",
         "eslint-plugin-cypress": "^2.12.1",
         "eslint-plugin-deprecation": "^1.3.3",
         "eslint-plugin-eslint-comments": "^3.2.0",
-        "eslint-plugin-import": "^2.26.0",
-        "eslint-plugin-jest": "^27.1.7",
+        "eslint-plugin-import": "^2.27.5",
+        "eslint-plugin-jest": "^27.2.1",
         "eslint-plugin-jest-dom": "^4.0.3",
         "eslint-plugin-jest-formatting": "^3.1.0",
         "eslint-plugin-jsdoc": "^39.6.4",
@@ -30,8 +30,8 @@
         "eslint-plugin-prefer-arrow": "^1.2.3",
         "eslint-plugin-promise": "^6.1.1",
         "eslint-plugin-rxjs": "^5.0.2",
-        "eslint-plugin-simple-import-sort": "^8.0.0",
-        "eslint-plugin-sonarjs": "^0.17.0",
+        "eslint-plugin-simple-import-sort": "^9.0.0",
+        "eslint-plugin-sonarjs": "^0.18.0",
         "eslint-plugin-testing-library": "^5.9.1",
         "eslint-plugin-unicorn": "^45.0.2",
         "tslint": "^6.1.3"

package/rules/eslint/layout-formatting.js

@@ -65,7 +65,7 @@ module.exports = {
                 allowObjectEnd: false,
                 allowArrayStart: false,
                 allowArrayEnd: false,
-                allowClassStart: false,
+                allowClassStart: true,
                 allowClassEnd: false,
             }],
         'lines-between-class-members': ['warn', 'always', {

package/rules/eslint/suggestions.js

@@ -105,7 +105,7 @@ module.exports = {
             }],
         'no-multi-assign': 'error',
         'no-multi-str': 'error',
-        'no-negated-condition': 'error',
+        'no-negated-condition': 'off',
         'no-nested-ternary': 'error',
         'no-new': 'error',
         'no-new-func': 'error',

package/rules/import/helpful-warnings.js

@@ -3,11 +3,12 @@ const no_extraneous_dependencies_1 = require("./rules/no-extraneous-dependencies
 module.exports = {
     rules: {
         'import/export': 'error',
-        'import/no-named-as-default': 'error',
-        'import/no-named-as-default-member': 'error',
         'import/no-deprecated': 'off',
+        'import/no-empty-named-blocks': 'warn',
         'import/no-extraneous-dependencies': ['error', (0, no_extraneous_dependencies_1.importNoExtraneousDependencies)()],
         'import/no-mutable-exports': 'error',
+        'import/no-named-as-default': 'error',
+        'import/no-named-as-default-member': 'error',
         'import/no-unused-modules': 'error',
     },
 };

package/rules/import/module-systems.js

@@ -1,13 +1,13 @@
 "use strict";
 module.exports = {
     rules: {
-        'import/unambiguous': 'error',
+        'import/no-amd': 'error',
         'import/no-commonjs': ['error', {
                 allowConditionalRequire: true,
                 allowPrimitiveModules: true,
             }],
-        'import/no-amd': 'error',
-        'import/no-nodejs-modules': 'off',
         'import/no-import-module-exports': 'error',
+        'import/no-nodejs-modules': 'off',
+        'import/unambiguous': 'error',
     },
 };

package/rules/import/static-analysis.js

@@ -1,28 +1,29 @@
 "use strict";
 module.exports = {
     rules: {
-        'import/no-unresolved': ['error', {
-                caseSensitive: true,
-                caseSensitiveStrict: true,
-            }],
+        'import/default': 'error',
         'import/named': ['error', {
                 commonjs: false,
             }],
-        'import/default': 'error',
         'import/namespace': 'error',
-        'import/no-restricted-paths': 'off',
         'import/no-absolute-path': 'error',
+        'import/no-cycle': ['error', {
+                ignoreExternal: true,
+                allowUnsafeDynamicCyclicDependency: false,
+            }],
         'import/no-dynamic-require': ['error', {
                 esmodule: true,
             }],
         'import/no-internal-modules': 'off',
-        'import/no-webpack-loader-syntax': 'error',
+        'import/no-relative-packages': 'warn',
+        'import/no-relative-parent-imports': 'off',
+        'import/no-restricted-paths': 'off',
         'import/no-self-import': 'error',
-        'import/no-cycle': ['error', {
-                ignoreExternal: true,
+        'import/no-unresolved': ['error', {
+                caseSensitive: true,
+                caseSensitiveStrict: true,
             }],
         'import/no-useless-path-segments': 'warn',
-        'import/no-relative-parent-imports': 'off',
-        'import/no-relative-packages': 'warn',
+        'import/no-webpack-loader-syntax': 'error',
     },
 };

package/rules/import/style-guide.js

@@ -1,13 +1,26 @@
 "use strict";
 module.exports = {
     rules: {
-        'import/first': 'off',
+        'import/consistent-type-specifier-style': ['warn', 'prefer-inline'],
+        'import/dynamic-import-chunkname': 'off',
         'import/exports-last': 'off',
+        'import/extensions': ['error', 'never'],
+        'import/first': 'off',
+        'import/group-exports': 'off',
+        'import/max-dependencies': 'off',
+        'import/newline-after-import': ['warn', {
+                considerComments: true,
+            }],
+        'import/no-anonymous-default-export': 'error',
+        'import/no-default-export': 'error',
         'import/no-duplicates': ['warn', {
-                considerQueryString: true,
+                'considerQueryString': true,
+                'prefer-inline': true,
             }],
+        'import/no-named-default': 'error',
+        'import/no-named-export': 'off',
         'import/no-namespace': 'off',
-        'import/extensions': ['error', 'never'],
+        'import/no-unassigned-import': 'error',
         'import/order': ['off', {
                 'groups': ['unknown', 'builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
                 'newlines-between': 'always',
@@ -16,15 +29,6 @@ module.exports = {
                     caseInsensitive: false,
                 },
             }],
-        'import/newline-after-import': 'warn',
         'import/prefer-default-export': 'off',
-        'import/max-dependencies': 'off',
-        'import/no-unassigned-import': 'error',
-        'import/no-named-default': 'error',
-        'import/no-default-export': 'error',
-        'import/no-named-export': 'off',
-        'import/no-anonymous-default-export': 'error',
-        'import/group-exports': 'off',
-        'import/dynamic-import-chunkname': 'off',
     },
 };

package/rules/jest/index.js

@@ -49,6 +49,7 @@ module.exports = {
         'jest/no-standalone-expect': 'error',
         'jest/no-test-prefixes': 'warn',
         'jest/no-test-return-statement': 'error',
+        'jest/no-untyped-mock-factory': 'warn',
         'jest/prefer-called-with': 'error',
         'jest/prefer-comparison-matcher': 'warn',
         'jest/prefer-each': 'error',

package/rules/simple-import-sort/rules/imports.d.ts

@@ -1,5 +1,5 @@
-export declare type Match = string;
-export declare type Group = Match[];
+export type Match = string;
+export type Group = Match[];
 export interface SimpleImportSortImports {
     groups: Group[];
 }

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

@@ -1,8 +1,8 @@
-export declare type TypescriptEslintNamingConventionFormat = 'camelCase' | 'strictCamelCase' | 'PascalCase' | 'StrictPascalCase' | 'snake_case' | 'UPPER_CASE';
-export declare type TypescriptEslintNamingConventionIndividualSelector = 'variable' | 'function' | 'parameter' | 'classProperty' | 'objectLiteralProperty' | 'typeProperty' | 'parameterProperty' | 'classMethod' | 'objectLiteralMethod' | 'typeMethod' | 'accessor' | 'enumMember' | 'class' | 'interface' | 'typeAlias' | 'enum' | 'typeParameter';
-export declare type TypescriptEslintNamingConventionGroupSelector = 'default' | 'variableLike' | 'memberLike' | 'typeLike' | 'property' | 'method';
-export declare type TypescriptEslintNamingConventionSelector = TypescriptEslintNamingConventionIndividualSelector | TypescriptEslintNamingConventionGroupSelector;
-export declare type TypescriptEslintNamingConventionUnderscore = 'forbid' | 'require' | 'requireDouble' | 'allow' | 'allowDouble' | 'allowSingleOrDouble';
+export type TypescriptEslintNamingConventionFormat = 'camelCase' | 'strictCamelCase' | 'PascalCase' | 'StrictPascalCase' | 'snake_case' | 'UPPER_CASE';
+export type TypescriptEslintNamingConventionIndividualSelector = 'variable' | 'function' | 'parameter' | 'classProperty' | 'objectLiteralProperty' | 'typeProperty' | 'parameterProperty' | 'classMethod' | 'objectLiteralMethod' | 'typeMethod' | 'accessor' | 'enumMember' | 'class' | 'interface' | 'typeAlias' | 'enum' | 'typeParameter';
+export type TypescriptEslintNamingConventionGroupSelector = 'default' | 'variableLike' | 'memberLike' | 'typeLike' | 'property' | 'method';
+export type TypescriptEslintNamingConventionSelector = TypescriptEslintNamingConventionIndividualSelector | TypescriptEslintNamingConventionGroupSelector;
+export type TypescriptEslintNamingConventionUnderscore = 'forbid' | 'require' | 'requireDouble' | 'allow' | 'allowDouble' | 'allowSingleOrDouble';
 export interface TypescriptEslintNamingConvention {
     selector: TypescriptEslintNamingConventionSelector | TypescriptEslintNamingConventionSelector[];
     format: TypescriptEslintNamingConventionFormat[] | null;

package/rules/typescript-eslint/rules/typescript-eslint-tslint-config.d.ts

@@ -1,5 +1,5 @@
-export declare type TsLintRule = boolean | [boolean, unknown];
-export declare type TsLintRules = Record<string, TsLintRule>;
+export type TsLintRule = boolean | [boolean, unknown];
+export type TsLintRules = Record<string, TsLintRule>;
 export interface TypescriptEslintTslintConfig {
     rules: TsLintRules;
     rulesDirectory?: string[];

package/rules/unicorn/rules/prevent-abbreviations.d.ts

@@ -1,4 +1,4 @@
-export declare type UnicornPreventAbbreviationReplacements = Record<string, boolean | Record<string, boolean>>;
+export type UnicornPreventAbbreviationReplacements = Record<string, boolean | Record<string, boolean>>;
 export interface UnicornPreventAbbreviations {
     replacements: UnicornPreventAbbreviationReplacements;
     extendDefaultReplacements: boolean;