eslint-config-angular-strict 2.3.48 → 2.3.50

package/index.js

@@ -1,13 +1,14 @@
+import { configs, rules } from 'eslint-config-airbnb-extended';
 import angularEslintPlugin from '@angular-eslint/eslint-plugin';
 import angularTemplateParser from '@angular-eslint/template-parser';
 import angularTemplatePlugin from '@angular-eslint/eslint-plugin-template';
 import importXPlugin from 'eslint-plugin-import-x';
 import ngModuleSortPlugin from 'eslint-plugin-ng-module-sort';
+import perfectionistPlugin from 'eslint-plugin-perfectionist';
 import stylistic from '@stylistic/eslint-plugin';
 import tsEslintParser from '@typescript-eslint/parser';
 import tsEslintPlugin from '@typescript-eslint/eslint-plugin';
 import unicornPlugin from 'eslint-plugin-unicorn';
-import { configs, rules } from 'eslint-config-airbnb-extended';
 
 export default [
   {
@@ -22,9 +23,10 @@ export default [
   },
 
   ...configs.base.all,
-  unicornPlugin.configs.all,
+  perfectionistPlugin.configs['recommended-natural'],
   rules.base.importsStrict,
   rules.typescript.typescriptEslintStrict,
+  unicornPlugin.configs.all,
 
   // TypeScript files configuration
   {
@@ -51,7 +53,6 @@ export default [
       'no-return-assign': 'off',
       'no-underscore-dangle': ['error', { allowAfterThis: true }],
       'radix': ['error', 'as-needed'],
-      'sort-keys': ['error', 'asc', { allowLineSeparatedGroups: true, natural: true }],
 
       // Import rules
       'import-x/no-anonymous-default-export': 'error',
@@ -59,22 +60,7 @@ export default [
       'import-x/no-cycle': ['error', { maxDepth: 2 }],
       'import-x/no-deprecated': 'error',
       'import-x/no-import-module-exports': 'off',
-      'import-x/order': [
-        'error',
-        {
-          alphabetize: { order: 'asc', caseInsensitive: true },
-          named: true,
-          'newlines-between': 'always',
-          pathGroups: [
-            { pattern: '@angular/**', group: 'external', position: 'before' },
-            { pattern: '@nestjs/**', group: 'external', position: 'before' },
-            { pattern: 'firebase*/**', group: 'external', position: 'before', patternOptions: { partial: true } },
-            { pattern: 'ng-zorro-antd/**', group: 'external', position: 'before' },
-          ],
-          pathGroupsExcludedImportTypes: [],
-          warnOnUnassignedImports: true,
-        },
-      ],
+      'import-x/order': 'off',
       'import-x/prefer-default-export': 'off',
 
       // NgModule sort rules
@@ -111,7 +97,6 @@ export default [
           overrides: { accessors: 'explicit', constructors: 'no-public', methods: 'explicit', properties: 'explicit', parameterProperties: 'explicit' },
         },
       ],
-      '@typescript-eslint/member-ordering': ['error', { default: { order: 'natural-case-insensitive' } }],
       '@typescript-eslint/naming-convention': [
         'error',
         { selector: 'function', format: ['camelCase'] },
@@ -136,16 +121,41 @@ export default [
       '@typescript-eslint/no-useless-default-assignment': 'error',
       '@typescript-eslint/prefer-readonly': 'error',
       '@typescript-eslint/promise-function-async': ['error', { checkArrowFunctions: false }],
-      '@typescript-eslint/sort-type-constituents': 'error',
       '@typescript-eslint/unbound-method': ['error', { ignoreStatic: true }],
 
+      // Perfectionist rules
+      'perfectionist/sort-imports': [
+        'error',
+        {
+          customGroups: [
+            { elementNamePattern: '^@angular/', groupName: 'angular' },
+            { elementNamePattern: '^@nestjs/', groupName: 'nestjs' },
+            { elementNamePattern: '^firebase', groupName: 'firebase' },
+            { elementNamePattern: '^ng-zorro-antd/', groupName: 'ng-zorro' },
+          ],
+          groups: [
+            'angular',
+            'nestjs',
+            'firebase',
+            'ng-zorro',
+            'type-import',
+            ['value-builtin', 'value-external'],
+            'type-internal',
+            'value-internal',
+            ['type-parent', 'type-sibling', 'type-index'],
+            ['value-parent', 'value-sibling', 'value-index'],
+            'ts-equals-import',
+            'unknown',
+          ],
+        },
+      ],
+
       // Unicorn rules
       'unicorn/consistent-function-scoping': ['error', { checkArrowFunctions: false }],
       'unicorn/new-for-builtins': 'off',
       'unicorn/no-abusive-eslint-disable': 'off',
       'unicorn/no-array-for-each': 'off',
       'unicorn/no-null': 'off',
-      'unicorn/no-useless-promise-resolve-reject': 'off',
       'unicorn/prefer-top-level-await': 'off',
       'unicorn/switch-case-braces': ['error', 'avoid'],
     },
@@ -168,6 +178,7 @@ export default [
       '@angular-eslint/template/label-has-associated-control': 'off',
       '@angular-eslint/template/no-call-expression':'off',
       '@angular-eslint/template/no-inline-styles': 'off',
+      '@angular-eslint/template/prefer-ngsrc': 'off',
 
       // Unicorn rules
       'unicorn/prefer-string-raw': 'off',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-angular-strict",
-  "version": "2.3.48",
+  "version": "2.3.50",
   "description": "Modern ESLint configuration with strict rules for Angular development.",
   "keywords": [
     "angular",
@@ -48,6 +48,7 @@
     "eslint-config-airbnb-extended": "3.0.1",
     "eslint-plugin-import-x": "4.16.2",
     "eslint-plugin-ng-module-sort": "1.4.0",
+    "eslint-plugin-perfectionist": "5.7.0",
     "eslint-plugin-unicorn": "64.0.0"
   },
   "devDependencies": {