arui-presets-lint 9.0.1 → 9.1.0

package/README.md

@@ -34,7 +34,7 @@
 {
     "prettier": "arui-presets-lint/prettier",
     "stylelint": { "extends": "arui-presets-lint/stylelint" },
-    "commitlint": { "extends": "./node_modules/arui-presets-lint/commitlint" }
+    "commitlint": { "extends": "arui-presets-lint/commitlint" }
 }
 ```
 
@@ -70,6 +70,45 @@ export default defineConfig(eslintConfig, [
 ]);
 ```
 
+Если в проекте есть файлы typescript, которые не добавлены в `tsconfig.json`, или исключены из него принудительно, то eslint выдаст ошибку `<filename> was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject`. Есть несколько способов решения этой проблемы:
+
+- (Рекомендуется) Добавить нужные файлы через опцию allowDefaultProject:
+
+```typescript
+import { eslintConfig } from 'arui-presets-lint/eslint';
+import { defineConfig } from 'arui-presets-lint/eslint/config';
+
+export default defineConfig(eslintConfig, [
+    {
+        languageOptions: {
+            parserOptions: {
+                projectService: {
+                    // Это позволит eslint линтить файлы, даже если они не указаны в tsconfig.json
+                    // Обратите внимание, что включить '**' тут нельзя, влияет на производительность!
+                    // https://typescript-eslint.io/packages/parser/#allowdefaultproject
+                    // Конретно тут - разрешаем линтить все файлы с расширениями .ts, .mts и .cts в корневой директории проекта
+                    allowDefaultProject: ['*.ts', '*.mts', '*.cts'],
+                },
+            },
+        },
+        files: ['**/*.{ts,tsx,mts,cts,mtsx,ctsx}'],
+    },
+]);
+```
+
+- (НЕ рекомендуется) Добавить файлы в globalIgnores. В этом случае eslint не будет его проверять:
+
+```typescript
+import { eslintConfig } from 'arui-presets-lint/eslint';
+import { defineConfig } from 'arui-presets-lint/eslint/config';
+
+export default defineConfig(eslintConfig, [
+    {
+        globalIgnores(['eslint.config.mts', 'arui-scripts.config.ts', 'playwright.config.ts']),
+    },
+]);
+```
+
 ## Конфигурация скриптов для запуска в `package.json`:
 
 ```json
@@ -87,7 +126,12 @@ export default defineConfig(eslintConfig, [
 
 Чтобы eslint / stylelint / prettier не проверял конкретные файлы и папки, можно исключить их с помощью файлов .stylelintignore / .prettierignore / .eslintignore Прописывать там файлы, которые уже есть в .gitignore не требуется!
 
-> Вместо файла .eslintignore рекомендуется использовать eslintConfig.ignores, либо globalIgnores в конфиге eslint ([подробнее](https://eslint.org/docs/latest/use/configure/ignore))
+> Вместо файла .eslintignore рекомендуется использовать globalIgnores в конфиге eslint ([подробнее](https://eslint.org/docs/latest/use/configure/ignore)). Импортируем функцию globalIgnores из `arui-presets-lint` вот так:
+
+```typescript
+import { defineConfig, globalIgnores } from 'arui-presets-lint/eslint/config';
+...
+```
 
 > Для запуска eslint/stylelint рекомендуется использовать флаг [--max-warnings](https://eslint.org/docs/latest/user-guide/command-line-interface#--max-warnings), который позволяет ограничить количество возникающих предупреждений.
 

package/V9_MIGRATION_GUIDE.md

@@ -55,16 +55,21 @@ export default defineConfig(eslintConfig, [
         languageOptions: {
             parserOptions: {
                 projectService: {
-                    // Это позволит eslint линтить все файлы в корневой директории проекта, даже если они не указаны в tsconfig.json
-                    // Эти параметры уже включены по умолчанию
-                    allowDefaultProject: ['*.js', '*.ts', '*.mjs', '*.mts', '*.cts', '*.cjs'],
+                    // Это позволит eslint линтить файлы, даже если они не указаны в tsconfig.json
+                    // Обратите внимание, что включить '**' тут нельзя, влияет на производительность!
+                    // https://typescript-eslint.io/packages/parser/#allowdefaultproject
+                    // Конретно тут - разрешаем линтить все файлы с расширениями .ts, .mts и .cts в корневой директории проекта
+                    allowDefaultProject: ['*.ts', '*.mts', '*.cts'],
                 },
             },
         },
+        files: ['**/*.{ts,tsx,mts,cts,mtsx,ctsx}'],
     },
 ]);
 ```
 
+Так же можно исключить этот файл через globalIgnores (не рекомендуется), подробнее в README.md
+
 3. import >> import-x
 
     Нужно пробежаться по проекту и заменить подобные комментарии:

package/eslint/index.ts

@@ -3,15 +3,17 @@ import { type Linter } from 'eslint';
 import gitignore from 'eslint-config-flat-gitignore';
 import globals from 'globals';
 
-import { bestPracticesConfig } from './rules/best-practices';
-import { importsConfig } from './rules/imports';
-import { nodeRulesConfig } from './rules/node';
-import { reactConfig } from './rules/react';
-import { reactA11yConfig } from './rules/react-a11y';
-import { testsConfig } from './rules/tests';
-import { typescriptConfig } from './rules/typescript';
-import { variablesConfig } from './rules/variables';
 import { globalIgnores } from './config';
+import {
+    bestPracticesConfig,
+    importsConfig,
+    nodeRulesConfig,
+    reactA11yConfig,
+    reactConfig,
+    testsConfig,
+    typescriptConfig,
+    variablesConfig,
+} from './rules';
 
 export const eslintConfig = [
     gitignore({
@@ -38,12 +40,6 @@ export const eslintConfig = [
             sourceType: 'module',
             parserOptions: {
                 ecmaFeatures: { jsx: true },
-                projectService: {
-                    // Разрешаем линтить файлы в корне проекта, даже если они не включены в tsconfig.json
-                    // ⛔️ Внимание, включить '**' тут нельзя, влияет на производительность!
-                    // https://typescript-eslint.io/packages/parser/#allowdefaultproject
-                    allowDefaultProject: ['*.js', '*.ts', '*.mjs', '*.mts', '*.cts', '*.cjs'],
-                },
             },
             globals: {
                 ...globals.es2022,

package/eslint/rules/best-practices.ts

@@ -93,10 +93,6 @@ export const bestPracticesConfig: TSESLint.FlatConfig.Config = {
         // https://eslint.org/docs/rules/no-bitwise
         'no-bitwise': 'error',
 
-        // Запрещает оператор continue
-        // https://eslint.org/docs/rules/no-continue
-        'no-continue': 'error',
-
         // Запрещает if как единственный оператор в блоке else
         // https://eslint.org/docs/rules/no-lonely-if
         'no-lonely-if': 'error',

package/eslint/rules/index.ts

@@ -0,0 +1,8 @@
+export { bestPracticesConfig } from './best-practices';
+export { importsConfig } from './imports';
+export { nodeRulesConfig } from './node';
+export { reactA11yConfig } from './react-a11y';
+export { reactConfig } from './react';
+export { testsConfig } from './tests';
+export { typescriptConfig } from './typescript';
+export { variablesConfig } from './variables';

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "arui-presets-lint",
-    "version": "9.0.1",
+    "version": "9.1.0",
     "description": "Config files for arui-apps",
     "author": "core-ds contributors",
     "license": "MIT",
@@ -16,7 +16,7 @@
         "@commitlint/config-conventional": "20.4.1",
         "@eslint/js": "9.39.2",
         "@types/eslint-plugin-jsx-a11y": "6.10.1",
-        "@typescript-eslint/utils": "8.54.0",
+        "@typescript-eslint/utils": "8.55.0",
         "eslint": "9.39.2",
         "eslint-config-flat-gitignore": "2.1.0",
         "eslint-import-resolver-typescript": "4.4.4",
@@ -27,14 +27,14 @@
         "eslint-plugin-react": "7.37.5",
         "eslint-plugin-react-hooks": "7.0.1",
         "eslint-plugin-simple-import-sort": "12.1.1",
-        "eslint-plugin-unicorn": "62.0.0",
+        "eslint-plugin-unicorn": "63.0.0",
         "execa": "^9.6.1",
         "globals": "^17.0.0",
         "jiti": "^2.6.1",
-        "lefthook": "2.1.0",
+        "lefthook": "2.1.1",
         "prettier": "3.8.1",
         "stylelint": "16.26.1",
-        "typescript-eslint": "8.54.0"
+        "typescript-eslint": "8.55.0"
     },
     "engines": {
         "node": ">=18.18.0"
@@ -71,14 +71,7 @@
     "exports": {
         "./eslint": "./eslint/index.ts",
         "./eslint/config": "./eslint/config.ts",
-        "./eslint/best-practices": "./eslint/best-practices.ts",
-        "./eslint/imports": "./eslint/imports.ts",
-        "./eslint/node": "./eslint/node.ts",
-        "./eslint/react-a11y": "./eslint/react-a11y.ts",
-        "./eslint/react": "./eslint/react.ts",
-        "./eslint/tests": "./eslint/tests.ts",
-        "./eslint/typescript": "./eslint/typescript.ts",
-        "./eslint/variables": "./eslint/variables.ts",
+        "./eslint/rules": "./eslint/rules/index.ts",
         "./prettier": "./prettier/index.js",
         "./commitlint": "./commitlint/index.js",
         "./stylelint": "./stylelint/index.js"

package/stylelint/index.js

@@ -1,8 +1,19 @@
 export default {
     rules: {
+        // Запрещает пустые блоки
+        // https://stylelint.io/user-guide/rules/block-no-empty
         'block-no-empty': true,
+
+        // Требует короткую форму hex-цветов
+        // https://stylelint.io/user-guide/rules/color-hex-length
         'color-hex-length': 'short',
+
+        // Запрещает невалидные hex-цвета
+        // https://stylelint.io/user-guide/rules/color-no-invalid-hex
         'color-no-invalid-hex': true,
+
+        // Требует пустую строку перед комментариями
+        // https://stylelint.io/user-guide/rules/comment-empty-line-before
         'comment-empty-line-before': [
             'always',
             {
@@ -10,8 +21,17 @@ export default {
                 ignore: ['stylelint-commands'],
             },
         ],
+
+        // Запрещает пустые комментарии
+        // https://stylelint.io/user-guide/rules/comment-no-empty
         'comment-no-empty': true,
+
+        // Требует пробелы внутри комментариев
+        // https://stylelint.io/user-guide/rules/comment-whitespace-inside
         'comment-whitespace-inside': 'always',
+
+        // Требует пустую строку перед custom properties
+        // https://stylelint.io/user-guide/rules/custom-property-empty-line-before
         'custom-property-empty-line-before': [
             'always',
             {
@@ -19,29 +39,86 @@ export default {
                 ignore: ['after-comment', 'inside-single-line-block'],
             },
         ],
+
+        // Запрещает переопределение shorthand-свойств длинными формами
+        // https://stylelint.io/user-guide/rules/declaration-block-no-shorthand-property-overrides
         'declaration-block-no-shorthand-property-overrides': true,
+
+        // Ограничивает количество деклараций в однострочном блоке
+        // https://stylelint.io/user-guide/rules/declaration-block-single-line-max-declarations
         'declaration-block-single-line-max-declarations': 2,
+
+        // Требует пробелы вокруг операторов в calc()
+        // https://stylelint.io/user-guide/rules/function-calc-no-unspaced-operator
         'function-calc-no-unspaced-operator': true,
+
+        // Запрещает нестандартные направления в linear-gradient()
+        // https://stylelint.io/user-guide/rules/function-linear-gradient-no-nonstandard-direction
         'function-linear-gradient-no-nonstandard-direction': true,
+
+        // Требует нижний регистр для имeн функций
+        // https://stylelint.io/user-guide/rules/function-name-case
         'function-name-case': 'lower',
+
+        // Запрещает !important в keyframe-декларациях
+        // https://stylelint.io/user-guide/rules/keyframe-declaration-no-important
         'keyframe-declaration-no-important': true,
+
+        // Запрещает единицы измерения для нулевых значений длины
+        // https://stylelint.io/user-guide/rules/length-zero-no-unit
         'length-zero-no-unit': true,
+
+        // Запрещает пустые исходники
+        // https://stylelint.io/user-guide/rules/no-empty-source
         'no-empty-source': true,
+
+        // Запрещает невалидные двойные слэш-комментарии
+        // https://stylelint.io/user-guide/rules/no-invalid-double-slash-comments
         'no-invalid-double-slash-comments': true,
+
+        // Запрещает неизвестные псевдоклассы
+        // https://stylelint.io/user-guide/rules/selector-pseudo-class-no-unknown
         'selector-pseudo-class-no-unknown': [
             true,
             {
                 ignorePseudoClasses: ['global'],
             },
         ],
+
+        // Требует одинарную нотацию для псевдоэлементов (:)
+        // https://stylelint.io/user-guide/rules/selector-pseudo-element-colon-notation
         'selector-pseudo-element-colon-notation': 'single',
+
+        // Запрещает неизвестные псевдоэлементы
+        // https://stylelint.io/user-guide/rules/selector-pseudo-element-no-unknown
         'selector-pseudo-element-no-unknown': true,
+
+        // Требует нижний регистр для селекторов типов
+        // https://stylelint.io/user-guide/rules/selector-type-case
         'selector-type-case': 'lower',
+
+        // Запрещает неизвестные селекторы типов
+        // https://stylelint.io/user-guide/rules/selector-type-no-unknown
         'selector-type-no-unknown': true,
+
+        // Запрещает избыточные значения в shorthand-свойствах
+        // https://stylelint.io/user-guide/rules/shorthand-property-no-redundant-values
         'shorthand-property-no-redundant-values': true,
+
+        // Запрещает переносы строк в строках
+        // https://stylelint.io/user-guide/rules/string-no-newline
         'string-no-newline': true,
+
+        // Запрещает неизвестные единицы измерения
+        // https://stylelint.io/user-guide/rules/unit-no-unknown
         'unit-no-unknown': true,
+
+        // Запрещает дублирующиеся селекторы
+        // https://stylelint.io/user-guide/rules/no-duplicate-selectors
         'no-duplicate-selectors': true,
+
+        // Запрещает дублирующиеся свойства в блоке
+        // https://stylelint.io/user-guide/rules/declaration-block-no-duplicate-properties
         'declaration-block-no-duplicate-properties': [
             true,
             {
@@ -49,19 +126,44 @@ export default {
             },
         ],
 
+        // Требует использовать CSS-переменные дизайн-системы вместо хардкода
+        // https://github.com/core-ds/stylelint-core-vars
         'stylelint-core-vars/use-vars': true,
+
+        // Требует использовать миксины дизайн-системы
+        // https://github.com/core-ds/stylelint-core-vars
         'stylelint-core-vars/use-mixins': true,
+
+        // Рекомендует использовать один из доступных vars при выборе значения
+        // https://github.com/core-ds/stylelint-core-vars
         'stylelint-core-vars/use-one-of-vars': [true, { severity: 'warning' }],
+
+        // Рекомендует использовать один из доступных mixins при выборе
+        // https://github.com/core-ds/stylelint-core-vars
         'stylelint-core-vars/use-one-of-mixins': [true, { severity: 'warning' }],
+
+        // Предупреждает об использовании тeмных цветов напрямую
+        // https://github.com/core-ds/stylelint-core-vars
         'stylelint-core-vars/do-not-use-dark-colors': [true, { severity: 'warning' }],
+
+        // Требует пустую строку перед правилами
+        // https://stylelint.io/user-guide/rules/rule-empty-line-before
+        'rule-empty-line-before': [
+            'always',
+            {
+                except: ['first-nested'],
+                ignore: ['after-comment'],
+            },
+        ],
     },
     plugins: ['@alfalab/stylelint-core-vars'],
     overrides: [
         {
             files: ['*.module.css'],
             rules: {
+                // Запрещает дефис в классах (требует camelCase или snake_case)
+                // https://stylelint.io/user-guide/rules/selector-class-pattern
                 'selector-class-pattern': [
-                    // Запрещаем использовать дефис '-'
                     '^[^-]+$',
                     {
                         resolveNestedSelectors: true,