@tinkoff/eslint-config 5.2.1 → 5.2.2

package/CHANGELOG.md

@@ -3,6 +3,15 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+### [5.2.2](https://github.com/tramvaijs/linters/compare/v5.2.1...v5.2.2) (2026-05-12)
+
+
+### Reverts
+
+* Revert "migrate to eslint v9" ([634f3b5](https://github.com/tramvaijs/linters/commit/634f3b5f8e88d7581efb11c700c9097846aaaa0c))
+
+
+
 ### [5.2.1](https://github.com/tramvaijs/linters/compare/v5.2.0...v5.2.1) (2026-04-23)
 
 **Note:** Version bump only for package @tinkoff/eslint-config

package/README.md

@@ -1,10 +1,6 @@
 # @tinkoff/eslint-config
 
-ESLint config includes rules for JS and TS codebase. Better to use with framework-oriented packages `@tinkoff/eslint-config-react` or `@tinkoff/eslint-config-angular`.
-
-## Requirements
-
-ESLint >= 9 is required.
+ESlint plugin includes rules for JS and TS codebase. Better to use with framework-oriented packages `@tinkoff/eslint-config-react` or `@tinkoff/eslint-config-angular`
 
 ## Usage
 
@@ -14,60 +10,62 @@ Install from npm
 npm i --save-dev @tinkoff/eslint-config
 ```
 
-Create `eslint.config.js` at the project root. Package includes eslint and prettier as dependencies, so you should remove them from your project `package.json`.
+Then, need to include necessary configurations sets to `.eslintrc`. Wee need to choose base configuration, and any
+necessary additional configs. Package include eslint and prettier dependencies, so you should remove eslint, prettier from your project package.json because those packages are transitive dependencies.
 
 ### Base configurations
 
 #### For application
 
-```js
-import appConfig from '@tinkoff/eslint-config/app';
-
-export default [...appConfig];
+```bash
+{
+  "extends": ["@tinkoff/eslint-config/app"]
+}
 ```
 
 #### For library
 
-```js
-import libConfig from '@tinkoff/eslint-config/lib';
-
-export default [...libConfig];
+```bash
+{
+  "extends": ["@tinkoff/eslint-config/lib"]
+}
 ```
 
 ### Additional configurations
 
-#### With Jest
+#### If we use Jest
 
-```js
-import appConfig from '@tinkoff/eslint-config/app';
-import jestConfig from '@tinkoff/eslint-config/jest';
-
-export default [...appConfig, ...jestConfig];
+```bash
+{
+  "extends": ["@tinkoff/eslint-config/app", "@tinkoff/eslint-config/jest"]
+}
 ```
 
 ## Configurations overview
 
-Main configurations include common rules:
+Main configurations sets contains common rules
 
-- `@tinkoff/eslint-config/app` — common rules and specific rules for applications
-- `@tinkoff/eslint-config/lib` — common rules and specific rules for libraries
+- `@tinkoff/eslint-config/app` - common rules and specific rules for applications
+- `@tinkoff/eslint-config/lib` - common rules and specific rules for libraries
 
-Additional configurations. These configs **do not** include common eslint rules and must be included alongside main configurations:
+Additional configurations sets. This configs **not** contain common eslint rules, and must be included with main
+configurations
 
-- `@tinkoff/eslint-config/jest` — rules for linting Jest test suites
+- `@tinkoff/eslint-config/jest` - rules for lint Jest test suits
 
 ## Internal used plugins
 
 `@tinkoff/eslint-config/app` and `@tinkoff/eslint-config/lib` include:
 
-- `eslint-config-airbnb-extended` — updated version of common and popular configuration `eslint-config-airbnb`
-- `@eslint-community/eslint-plugin-eslint-comments` — validate `eslint` comments
-- `eslint-plugin-import` — validate proper imports
-- `eslint-plugin-promise` — enforce best practices for promises
-- `eslint-plugin-jest` — validate jest tests
-- `@typescript-eslint/eslint-plugin` — lint TypeScript files, adopt many eslint rules to TS code, and provide specific TS rules
-- `eslint-plugin-prettier` — disable code formatting using eslint tools and transfer all the logic to prettier,
-  and report differences as eslint issues
+- `eslint-config-airbnb` - common and popular configuration
+- `eslint-plugin-eslint-comments` - validate `eslint` comments
+- `eslint-plugin-import` - validate proper imports
+- `eslint-plugin-promise` - enforce best practices for promises
+- `eslint-plugin-jest` - validate jest tests
+- `@typescript-eslint/eslint-plugin` - lint TypeScript files, adopt many eslint rules to TS code, and provide specific
+  TS rules
+- `eslint-plugin-prettier` - disable code formatting using eslint tools and transfers all the logic to a prettier, and
+  report differences as eslint issues
 
 ## Troubleshooting
 
@@ -82,80 +80,86 @@ then reinstall dependencies.
 
 #### In root config
 
-Create `eslint.config.js` at the workspace root:
+1. Add `"@tinkoff/eslint-config"` to extends section on config.
+2. Set parserOptions.project for typescript files
 
-```js
-import appConfig from '@tinkoff/eslint-config';
+```json5
+{
 
-export default [
-  ...appConfig,
-  {
-    ignores: ['**/*'],
-  },
-];
+    "root": true,
+    "ignorePatterns": ["**/*"],
+    "extends": ["@tinkoff/eslint-config"]
+    // ...optionally some other configs
+    "overrides": [
+        {
+            "files": ["*.ts", "*.tsx"],
+            "parserOptions": {
+                "project": ["./tsconfig.base.json"]
+            }
+        }
+    ]
+}
 ```
 
 #### In apps and non-buildable/non-publishable libs
 
-Create `eslint.config.js` in the project directory:
+Add `"@tinkoff/eslint-config/app"` in extends section after root config
 
-```js
-import rootConfig from '../../eslint.config';
-import appConfig from '@tinkoff/eslint-config/app';
-
-export default [
-  ...rootConfig,
-  ...appConfig,
-  {
-    files: ['**/*.ts', '**/*.tsx'],
-    languageOptions: {
-      parserOptions: {
-        project: ['./tsconfig.json'],
-      },
-    },
-  },
-];
+```json
+{
+    "extends": ["../../.eslintrc.json", "@tinkoff/eslint-config/app"],
+    "ignorePatterns": ["!**/*"]
+    ...optionally some other configs
+}
 ```
 
 #### In buildable/publishable libs
 
-```js
-import rootConfig from '../../eslint.config';
-import libConfig from '@tinkoff/eslint-config/lib';
-
-export default [
-  ...rootConfig,
-  ...libConfig,
-  {
-    files: ['**/*.ts', '**/*.tsx'],
-    languageOptions: {
-      parserOptions: {
-        project: ['./tsconfig.json'],
-      },
-    },
-  },
-];
+Add `"@tinkoff/eslint-config/lib"` in extends section after root config
+
+```json
+{
+    "extends": ["../../.eslintrc.json", "@tinkoff/eslint-config/lib"],
+    "ignorePatterns": ["!**/*"]
+    ...optionally some other configs
+}
 ```
 
-### Disable false-positive detections from `import/no-unresolved`
+### Non-JSON configs usage
+
+Nx means usage `.eslintrc.json` for configure ESLint. Just provide it! In `.eslintrc.json` add extension of actual root config file:
+
+```
+{
+  "extends": ["./.eslintrc.js"]
+}
+```
 
-Override the rule in the project's `eslint.config.js`:
+After that:
 
-```js
-import appConfig from '@tinkoff/eslint-config/app';
+- Nx see what they need and don't re-initialize lint infrastructure
+- You can use [better config formats](https://eslint.org/docs/latest/user-guide/configuring/configuration-files#configuration-file-formats) and nx generators
 
-export default [
-  ...appConfig,
-  {
-    files: ['**/*.ts', '**/*.tsx', '**/*.js', '**/*.jsx'],
-    rules: {
-      'import/no-unresolved': [
-        'error',
-        {
-          ignore: ['^@tinkoff/'],
-        },
-      ],
+### Disable false-positive detections from `import/no-unresolved`
+
+override rule in `.eslintrc` of apps/libs that contains problem:
+
+```json5
+{
+  // ...some other configs...
+  // disable detection for packages starts with @tinkoff/
+  overrides: [
+    {
+      files: ['*.ts', '*.tsx', '*.js', '*.jsx'],
+      rules: {
+        'import/no-unresolved': [
+          'error',
+          {
+            ignore: ['^@tinkoff/'],
+          },
+        ],
+      },
     },
-  },
-];
+  ],
+}
 ```

package/app.js

@@ -1,3 +1,7 @@
-import index from './index.js';
+module.exports = {
+  extends: ['./index'],
 
-export default [...index];
+  plugins: [],
+
+  rules: {},
+};

package/errors/index.js

@@ -1,17 +1,14 @@
-import { rules } from 'eslint-config-airbnb-extended/legacy';
-import typescriptParser from '@typescript-eslint/parser';
-
-export default [
-  {
-    files: ['**/*.js', '**/*.ts', '**/*.tsx'],
-    languageOptions: {
-      parser: typescriptParser,
+module.exports = {
+  overrides: [
+    {
+      files: ['*.js', '*.ts', '*.tsx'],
+      parser: '@typescript-eslint/parser',
+      extends: ['eslint-config-airbnb-base/rules/errors'],
+      rules: {
+        // eslint-config-airbnb-base/errors
+        'no-empty': ['error', { allowEmptyCatch: true }],
+        'no-await-in-loop': 'off',
+      },
     },
-    rules: {
-      ...rules.base.errors.rules,
-      // Overrides
-      'no-empty': ['error', { allowEmptyCatch: true }],
-      'no-await-in-loop': 'off',
-    },
-  },
-];
+  ],
+};

package/index.js

@@ -1,26 +1,16 @@
-import globals from 'globals';
-import base from './internal/base.js';
-import sortClassMembers from './internal/sort-class-members.js';
-import importConfig from './internal/import.js';
-import promise from './internal/promise.js';
-import testFiles from './internal/test-files.js';
-import typescript from './internal/typescript.js';
-import prettier from './internal/prettier.js';
+module.exports = {
+  extends: [
+    './internal/base',
+    './internal/sort-class-members',
+    './internal/import',
+    './internal/promise',
+    './internal/test-files',
+    './internal/typescript',
+    './internal/prettier',
+  ],
 
-export default [
-  ...base,
-  ...sortClassMembers,
-  ...importConfig,
-  ...promise,
-  ...testFiles,
-  ...typescript,
-  ...prettier,
-  {
-    languageOptions: {
-      globals: {
-        ...globals.browser,
-        ...globals.node,
-      },
-    },
+  env: {
+    browser: true,
+    node: true,
   },
-];
+};

package/internal/base.js

@@ -1,94 +1,83 @@
-import eslintCommentsPlugin from '@eslint-community/eslint-plugin-eslint-comments';
-import babelParser from '@babel/eslint-parser';
-import { configs } from 'eslint-config-airbnb-extended/legacy';
-import errorsConfig from '../errors/index.js';
+module.exports = {
+  extends: [
+    'eslint-config-airbnb-base',
+    'plugin:eslint-comments/recommended',
+    '../errors/index.js',
+  ],
 
-export default [
-  ...configs.base.recommended,
-  ...errorsConfig,
-  {
-    plugins: {
-      'eslint-comments': eslintCommentsPlugin,
+  parser: '@babel/eslint-parser',
+
+  settings: {
+    'import/parser': '@babel/eslint-parser',
+  },
+
+  plugins: ['@babel'],
+
+  parserOptions: {
+    ecmaFeatures: {
+      legacyDecorators: true,
     },
-    languageOptions: {
-      parser: babelParser,
-      parserOptions: {
-        ecmaFeatures: {
-          legacyDecorators: true,
-        },
-        requireConfigFile: false,
+    requireConfigFile: false,
+  },
+
+  rules: {
+    'no-unused-expressions': [
+      'error',
+      {
+        allowShortCircuit: true,
+        allowTernary: true,
       },
-    },
-    settings: {
-      'import/parser': {
-        '@babel/eslint-parser': ['.js', '.jsx', '.ts', '.tsx'],
+    ],
+    'no-use-before-define': [
+      'error',
+      {
+        functions: false,
+        classes: false,
+        variables: true,
       },
-    },
-    rules: {
-      'eslint-comments/disable-enable-pair': 'error',
-      'eslint-comments/no-aggregating-enable': 'error',
-      'eslint-comments/no-duplicate-disable': 'error',
-      'eslint-comments/no-unlimited-disable': 'error',
-      'eslint-comments/no-unused-enable': 'error',
-
-      'no-unused-expressions': [
-        'error',
-        {
-          allowShortCircuit: true,
-          allowTernary: true,
-        },
-      ],
-      'no-use-before-define': [
-        'error',
-        {
-          functions: false,
-          classes: false,
-          variables: true,
-        },
-      ],
-      'func-name-matching': 'off',
-      'global-require': 'off',
-      'class-methods-use-this': 'off',
-      'no-continue': 'off',
-      'no-restricted-syntax': 'off',
-      'guard-for-in': 'off',
-      'default-case': 'off',
-      'no-plusplus': 'off',
-      'func-names': 'off',
-      'consistent-return': 'warn',
-      'vars-on-top': 'warn',
-      'no-var': 'warn',
-      camelcase: [
-        'warn',
-        {
-          allow: ['^UNSAFE_'],
-          ignoreDestructuring: false,
-          properties: 'never',
-        },
-      ],
-      'func-style': [
-        'error',
-        'declaration',
-        {
-          allowArrowFunctions: true,
-        },
-      ],
-      'max-depth': 'off',
-      'max-params': 'off',
-      'max-classes-per-file': ['error', 4],
-      complexity: ['error', 25],
-      'max-statements': ['error', 25],
-      'no-underscore-dangle': 'off',
-      'no-return-assign': ['error', 'except-parens'],
-      'lines-between-class-members': [
-        'error',
-        'always',
-        { exceptAfterSingleLine: true },
-      ],
-      'spaced-comment': ['error', 'always', { exceptions: ['*'] }],
-      'max-nested-callbacks': ['error', 4],
-      'no-bitwise': 'warn',
-      'no-useless-escape': 'warn',
-    },
+    ],
+    'func-name-matching': 'off',
+    'global-require': 'off',
+    'class-methods-use-this': 'off',
+    'no-continue': 'off',
+    'no-restricted-syntax': 'off',
+    'guard-for-in': 'off',
+    'default-case': 'off',
+    'no-plusplus': 'off',
+    'func-names': 'off',
+    'consistent-return': 'warn',
+    'vars-on-top': 'warn',
+    'no-var': 'warn',
+    camelcase: [
+      'warn',
+      {
+        allow: ['^UNSAFE_'],
+        ignoreDestructuring: false,
+        properties: 'never',
+      },
+    ],
+    'func-style': [
+      'error',
+      'declaration',
+      {
+        allowArrowFunctions: true,
+      },
+    ],
+    'max-depth': 'off',
+    'max-params': 'off',
+    'max-classes-per-file': ['error', 4],
+    complexity: ['error', 25],
+    'max-statements': ['error', 25],
+    'no-underscore-dangle': 'off',
+    'no-return-assign': ['error', 'except-parens'],
+    'lines-between-class-members': [
+      'error',
+      'always',
+      { exceptAfterSingleLine: true },
+    ],
+    'spaced-comment': ['error', 'always', { exceptions: ['*'] }],
+    'max-nested-callbacks': ['error', 4],
+    'no-bitwise': 'warn',
+    'no-useless-escape': 'warn',
   },
-];
+};

package/internal/import.js

@@ -1,45 +1,36 @@
-import importPlugin from 'eslint-plugin-import';
+module.exports = {
+  plugins: ['import'],
 
-export default [
-  {
-    plugins: {
-      import: importPlugin,
-    },
-
-    settings: {
-      'import/resolver': {
-        node: {
-          extensions: ['.js', '.mjs', '.cjs', '.ts'],
-        },
-        typescript: {},
-        webpack: {},
+  settings: {
+    'import/resolver': {
+      node: {
+        extensions: ['.js', '.mjs', '.cjs', '.ts'],
       },
+      typescript: {},
+      webpack: {},
     },
+  },
 
-    rules: {
-      'import/no-commonjs': 'off',
-      'import/unambiguous': 'off',
-      'import/prefer-default-export': 'off', // default imports is evil
-      'import/no-deprecated': 'warn',
-      'import/default': 'error', // enable default import validation
-      'import/extensions': [
-        'error',
-        'always',
-        {
-          js: 'never',
-          mjs: 'never',
-          cjs: 'never',
-          jsx: 'never',
-          ts: 'never',
-          tsx: 'never',
-        },
-      ],
-      'import/order': [
-        'warn',
-        { groups: [['builtin', 'external', 'internal']] },
-      ],
-      'import/no-extraneous-dependencies': 'off', // need fine tuning
-      'import/no-cycle': 'error',
-    },
+  rules: {
+    'import/no-commonjs': 'off',
+    'import/unambiguous': 'off',
+    'import/prefer-default-export': 'off', // default imports is evil
+    'import/no-deprecated': 'warn',
+    'import/default': 'error', // enable default import validation
+    'import/extensions': [
+      'error',
+      'always',
+      {
+        js: 'never',
+        mjs: 'never',
+        cjs: 'never',
+        jsx: 'never',
+        ts: 'never',
+        tsx: 'never',
+      },
+    ],
+    'import/order': ['warn', { groups: [['builtin', 'external', 'internal']] }],
+    'import/no-extraneous-dependencies': 'off', // need fine tuning
+    'import/no-cycle': 'error',
   },
-];
+};

package/internal/prettier.js

@@ -1,16 +1,3 @@
-import prettierPlugin from 'eslint-plugin-prettier';
-import prettierConfig from 'eslint-config-prettier';
-
-export default [
-  {
-    plugins: {
-      prettier: prettierPlugin,
-    },
-    rules: {
-      ...prettierConfig.rules,
-      'prettier/prettier': 'error',
-      'arrow-body-style': 'off',
-      'prefer-arrow-callback': 'off',
-    },
-  },
-];
+module.exports = {
+  extends: ['plugin:prettier/recommended'],
+};

package/internal/promise.js

@@ -1,13 +1,10 @@
-import promisePlugin from 'eslint-plugin-promise';
+module.exports = {
+  extends: ['plugin:promise/recommended'],
 
-export default [
-  promisePlugin.configs['flat/recommended'],
-  {
-    rules: {
-      'promise/always-return': 'off',
-      'promise/no-callback-in-promise': 'off',
-      'promise/catch-or-return': 'warn',
-      'promise/param-names': 'warn',
-    },
+  rules: {
+    'promise/always-return': 'off',
+    'promise/no-callback-in-promise': 'off',
+    'promise/catch-or-return': 'warn',
+    'promise/param-names': 'warn',
   },
-];
+};