@dr.pogodin/eslint-configs 0.2.3 → 0.2.4

package/config/javascript.js

@@ -24,8 +24,8 @@ function newConfig({ noPerf } = {}) {
     '@babel/new-cap': 'error',
     '@babel/no-invalid-this': 'error',
 
-    '@babel/no-undef': 'error',
     'no-undef': 'off',
+    '@babel/no-undef': 'error',
 
     '@babel/no-unused-expressions': 'error',
 
@@ -285,6 +285,7 @@ function newConfig({ noPerf } = {}) {
       .sortByNaturalSort()
       .placeAllWithCaseBeforeAllWithOtherCase('uppercase')
       .placeCharacterBefore({ characterAfter: '-', characterBefore: '/' })
+      .placeCharacterAfter({ characterAfter: '@', characterBefore: 'z' })
       .getCharacters();
 
     settings.perfectionist = {
@@ -294,6 +295,24 @@ function newConfig({ noPerf } = {}) {
       newlinesInside: 'ignore',
       partitionByNewLine: true,
     };
+
+    rules['perfectionist/sort-imports'] = ['error', {
+      alphabet,
+      groups: [
+        'builtin',
+        'external',
+        'internal',
+        'parent',
+        'sibling',
+        'index',
+        'style',
+        'unknown',
+      ],
+      newlinesBetween: 1,
+      newlinesInside: 'ignore',
+      partitionByNewLine: false,
+      type: 'custom',
+    }];
   }
 
   return defineConfig([{
@@ -307,9 +326,9 @@ function newConfig({ noPerf } = {}) {
     },
     name: 'dr.pogodin/javascript',
     plugins: {
+      js,
       '@babel': babelPlugin,
       '@stylistic': stylisticPlugin,
-      js,
     },
     rules,
     settings,

package/config/typescript.js

@@ -50,8 +50,8 @@ function newConfig({ noPerf } = {}) {
 
     // NOTE: The core rule variant reports incorrect errors on TypeScript
     // code, thus disabled.
-    '@typescript-eslint/no-shadow': 'error',
     'no-shadow': 'off',
+    '@typescript-eslint/no-shadow': 'error',
 
     '@typescript-eslint/no-unnecessary-boolean-literal-compare': 'error',
     '@typescript-eslint/no-unnecessary-condition': 'error',
@@ -61,8 +61,8 @@ function newConfig({ noPerf } = {}) {
     '@typescript-eslint/no-unnecessary-type-arguments': 'error',
 
     // NOTE: Disables the base rule as it can report incorrect errors.
-    '@typescript-eslint/no-unused-private-class-members': 'error',
     'no-unused-private-class-members': 'off',
+    '@typescript-eslint/no-unused-private-class-members': 'error',
 
     '@typescript-eslint/no-use-before-define': 'error',
     '@typescript-eslint/no-useless-constructor': 'error',
@@ -73,8 +73,8 @@ function newConfig({ noPerf } = {}) {
 
     // NOTE: The core variant of this rule, disabled below, does not work
     // correctly for TypeScript code in some edge cases.
-    '@typescript-eslint/prefer-destructuring': 'error',
     'prefer-destructuring': 'off',
+    '@typescript-eslint/prefer-destructuring': 'error',
 
     '@typescript-eslint/prefer-enum-initializers': 'error',
     '@typescript-eslint/prefer-literal-enum-member': 'error',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dr.pogodin/eslint-configs",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "ESLint configurations for TypeScript and/or React projects",
   "type": "module",
   "main": "./config/index.js",