@mikey-pro/eslint-config 8.0.16 → 8.0.18

package/index.js

@@ -1,4 +1,3 @@
-import babelParser from '@babel/eslint-parser';
 import cypressJson from '@cypress/eslint-plugin-json';
 import eslintJs from '@eslint/js';
 import compatPlugin from 'eslint-plugin-compat';
@@ -14,7 +13,16 @@ import importSortPlugin from 'eslint-plugin-simple-import-sort';
 import perfectionistPlugin from 'eslint-plugin-perfectionist';
 import noSecretsPlugin from 'eslint-plugin-no-secrets';
 import noOnlyTestsPlugin from 'eslint-plugin-no-only-tests';
-// Remove jest plugin import since it's now in overrides.js
+import sonarjsPlugin from 'eslint-plugin-sonarjs';
+import regexpPlugin from 'eslint-plugin-regexp';
+import etcPlugin from 'eslint-plugin-etc';
+import typescriptSortKeysPlugin from 'eslint-plugin-typescript-sort-keys';
+import sortDestructureKeysPlugin from 'eslint-plugin-sort-destructure-keys';
+import writeGoodCommentsPlugin from 'eslint-plugin-write-good-comments';
+import boundariesPlugin from 'eslint-plugin-boundaries';
+import nPlugin from 'eslint-plugin-n';
+import radarPlugin from 'eslint-plugin-radar';
+import optimizeRegexPlugin from 'eslint-plugin-optimize-regex';
 
 import { baseOverrides } from './overrides.js';
 import { baseRules } from './rules.js';
@@ -42,25 +50,10 @@ const config = [
   },
   eslintJs.configs.recommended,
   {
+    files: ['**/*.js', '**/*.jsx', '**/*.mjs', '**/*.cjs'],  // Add files pattern
     languageOptions: {
       ecmaVersion: 'latest',
       sourceType: 'module',
-      parser: babelParser,
-      parserOptions: {
-        babelOptions: {
-          presets: [
-            [
-              '@babel/preset-env',
-              {
-                targets: {
-                  node: 'current',
-                },
-              },
-            ],
-          ],
-        },
-        requireConfigFile: false,
-      },
       globals: {
         ...globals.browser,
         ...globals.commonjs,
@@ -69,6 +62,10 @@ const config = [
         ...globals.node
       },
     },
+    linterOptions: {
+      noInlineConfig: true,
+      reportUnusedDisableDirectives: true,
+    },
     plugins: {
       prettier,
       unicorn,
@@ -82,7 +79,17 @@ const config = [
       perfectionist: perfectionistPlugin,
       'no-secrets': noSecretsPlugin,
       'no-only-tests': noOnlyTestsPlugin,
-      // Remove jest plugin from here since it's in overrides
+      sonarjs: sonarjsPlugin,
+      regexp: regexpPlugin,
+      etc: etcPlugin,
+      'typescript-sort-keys': typescriptSortKeysPlugin,
+      'sort-destructure-keys': sortDestructureKeysPlugin,
+      'write-good-comments': writeGoodCommentsPlugin,
+      boundaries: boundariesPlugin,
+      n: nPlugin,
+      radar: radarPlugin,
+      'optimize-regex': optimizeRegexPlugin,
+      compat: compatPlugin,
     },
     rules: {
       ...baseRules,
@@ -90,6 +97,7 @@ const config = [
       ...compatPlugin.configs.recommended.rules,
       ...cssModules.configs.recommended.rules,
       ...importPlugin.configs.recommended.rules,
+      ...sonarjsPlugin.configs.recommended.rules,
       'prettier/prettier': ['warn', {
         parser: 'babel',
         endOfLine: 'lf',
@@ -127,6 +135,15 @@ const config = [
       'import/no-cycle': 'warn',
       'import/no-useless-path-segments': 'warn',
       'import/no-anonymous-default-export': 'warn',
+
+      // SonarJS rules
+      'sonarjs/cognitive-complexity': ['error', 15],
+      'sonarjs/no-duplicate-string': ['error', { threshold: 5 }],
+      'sonarjs/no-redundant-boolean': 'error',
+      'sonarjs/prefer-immediate-return': 'error',
+      'sonarjs/no-small-switch': 'warn',
+      'sonarjs/no-duplicated-branches': 'error',
+      'sonarjs/max-switch-cases': ['warn', 10],
     },
     settings: {
       'json/json-with-comments-files': [],

package/overrides.js

@@ -11,17 +11,14 @@ const currentDir = path.dirname(fileURLToPath(import.meta.url));
 const ts = {
   files: ['*.ts', '*.tsx'],
   languageOptions: {
-    parser: typescriptParser,
-    parserOptions: {
-      ecmaFeatures: {
-        jsx: true,
-      },
-      ecmaVersion: 'latest',
-      extraFileExtensions: ['.vue', '.svelte'],
-      sourceType: 'module',
-      project: 'tsconfig.json',
-      tsconfigRootDir: path.join(currentDir, '../../..'),
+    ecmaFeatures: {
+      jsx: true,
     },
+    ecmaVersion: 'latest',
+    extraFileExtensions: ['.vue', '.svelte'],
+    sourceType: 'module',
+    project: 'tsconfig.json',
+    tsconfigRootDir: path.join(currentDir, '../../..'),
   },
   plugins: {
     '@typescript-eslint': typeScriptPlugin,
@@ -157,7 +154,6 @@ const ts = {
       'error',
       { ignoreArrowShorthand: true }
     ],
-    '@typescript-eslint/prefer-return-this-type': 'error',
     '@typescript-eslint/prefer-string-starts-ends-with': 'error',
     '@typescript-eslint/promise-function-async': 'error',
     '@typescript-eslint/require-array-sort-compare': [
@@ -261,9 +257,6 @@ const toml = {
 
 const md = {
   files: ['*.md'],
-  languageOptions: {
-    parser: 'eslint-plugin-markdownlint/parser',
-  },
   plugins: {
     markdownlint: markdownPlugin,
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikey-pro/eslint-config",
-  "version": "8.0.16",
+  "version": "8.0.18",
   "description": "Mikey Pro ESLint configuration",
   "main": "index.js",
   "types": "index.d.ts",
@@ -15,9 +15,9 @@
     "@html-eslint/parser": "^0.33.0",
     "@typescript-eslint/eslint-plugin": "^8.22.0",
     "@typescript-eslint/parser": "^8.22.0",
-    "eslint-config-prettier": "^10.0",
+    "eslint-config-prettier": "^10.0.1",
     "eslint-import-resolver-typescript": "^3.7.0",
-    "eslint-plugin-compat": "^6.0",
+    "eslint-plugin-compat": "^6.0.2",
     "eslint-plugin-css-modules": "^2.12",
     "eslint-plugin-cypress": "^4.1.0",
     "eslint-plugin-import": "^2.31.0",
@@ -29,7 +29,7 @@
     "eslint-plugin-perfectionist": "^4.7.0",
     "eslint-plugin-no-secrets": "^2.1.1",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-prettier": "^5.2",
+    "eslint-plugin-prettier": "^5.2.3",
     "eslint-plugin-promise": "^7.2.1",
     "eslint-plugin-security": "^3.0.1",
     "eslint-plugin-simple-import-sort": "^12.1.1",

package/rules.js

@@ -40,7 +40,7 @@ export const baseRules = {
   'no-delete-var': 'warn',
   'no-dupe-class-members': 'warn',
   'no-dupe-keys': 'warn',
-  'no-duplicate-imports': 'warn',
+  'no-duplicate-imports': 'error',
   'no-else-return': 'warn',
   'no-empty': 'off',
   'no-empty-pattern': 'off',
@@ -210,11 +210,12 @@ export const baseRules = {
   'security/detect-non-literal-regexp': 'error',
   'security/detect-non-literal-require': 'error',
 
-  // Code Quality
-  'sonarjs/cognitive-complexity': ['error', 15],
-  'sonarjs/no-duplicate-string': ['error', 5],
-  'sonarjs/no-redundant-boolean': 'error',
-  'sonarjs/prefer-immediate-return': 'error',
+  'no-unreachable': 'error',
+  'no-constant-condition': 'error',
+  'complexity': ['error', { max: 15 }],
+
+  // RegExp
+  'regexp/no-missing-g-flag': 'error',
 
   // Enhanced TypeScript
   'etc/no-commented-out-code': 'warn',
@@ -240,9 +241,6 @@ export const baseRules = {
 
   // Code Quality
   'write-good-comments/write-good-comments': 'warn',
-  'sonarjs/no-small-switch': 'warn',
-  'sonarjs/no-duplicated-branches': 'error',
-  'sonarjs/max-switch-cases': ['warn', 10],
 
   // Import Safety
   'import/no-cycle': ['error', {
@@ -252,7 +250,6 @@ export const baseRules = {
   'import/no-relative-packages': 'error',
   'import/no-self-import': 'error',
   'import/no-namespace': 'error',
-  'import/no-empty-named-blocks': 'error',
   'import/no-duplicates': ['error', { 'prefer-inline': true }],
   'import/no-import-module-exports': 'error',