@mikey-pro/eslint-config 8.0.15 → 8.0.16

package/index.js

@@ -14,6 +14,7 @@ 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 { baseOverrides } from './overrides.js';
 import { baseRules } from './rules.js';
@@ -81,6 +82,7 @@ const config = [
       perfectionist: perfectionistPlugin,
       'no-secrets': noSecretsPlugin,
       'no-only-tests': noOnlyTestsPlugin,
+      // Remove jest plugin from here since it's in overrides
     },
     rules: {
       ...baseRules,

package/overrides.js

@@ -4,6 +4,7 @@ import typescriptParser from '@typescript-eslint/parser';
 import typeScriptPlugin from '@typescript-eslint/eslint-plugin';
 import importPlugin from 'eslint-plugin-import';
 import markdownPlugin from 'eslint-plugin-markdownlint';
+import jestPlugin from 'eslint-plugin-jest';
 
 const currentDir = path.dirname(fileURLToPath(import.meta.url));
 
@@ -357,13 +358,17 @@ const json5 = {
 };
 
 const jestJs = {
-  files: ['*.test.js'],
-  plugins: ['jest'],
+  files: ['*.test.js', '**/__tests__/**/*.js'],
+  plugins: {
+    jest: jestPlugin,
+  },
   extends: ['plugin:jest/all'],
   env: {
     jest: true,
   },
   rules: {
+    'jest/prefer-spy-on': 'warn',
+    'jest/require-top-level-describe': 'error',
     'unicorn/no-array-callback-reference': 'off',
     'jest/unbound-method': 'off',
     'unicorn/prevent-abbreviations': [
@@ -376,14 +381,19 @@ const jestJs = {
 };
 
 const jestTs = {
-  files: ['*.test.ts'],
+  files: ['*.test.ts', '**/__tests__/**/*.ts'],
   parser: '@typescript-eslint/parser',
-  plugins: ['@typescript-eslint', 'jest'],
+  plugins: {
+    '@typescript-eslint': typeScriptPlugin,
+    jest: jestPlugin,
+  },
   extends: ['plugin:jest/all'],
   env: {
     jest: true,
   },
   rules: {
+    'jest/prefer-spy-on': 'warn',
+    'jest/require-top-level-describe': 'error',
     'unicorn/no-array-callback-reference': 'off',
     'jest/unbound-method': 'off',
     'unicorn/prevent-abbreviations': [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikey-pro/eslint-config",
-  "version": "8.0.15",
+  "version": "8.0.16",
   "description": "Mikey Pro ESLint configuration",
   "main": "index.js",
   "types": "index.d.ts",

package/rules.js

@@ -202,38 +202,20 @@ export const baseRules = {
 
   // Security
   'security/detect-non-literal-fs-filename': 'error',
-  'security/detect-unsafe-regex': ['error', {
-    allowDollarMatchAll: false,
-    maxLength: 50
-  }],
+  'security/detect-unsafe-regex': 'error',
   'security/detect-buffer-noassert': 'error',
   'security/detect-child-process': 'warn',
   'security/detect-disable-mustache-escape': 'error',
-  'security/detect-possible-timing-attacks': ['error', {
-    threshold: 8,
-    catchAliases: true
-  }],
-  'security/detect-non-literal-regexp': ['error', {
-    report: 'error',
-    warnOnDynamicRegexp: true
-  }],
+  'security/detect-possible-timing-attacks': 'error',
+  'security/detect-non-literal-regexp': 'error',
   'security/detect-non-literal-require': 'error',
 
-  // Better testing
-  'jest/prefer-spy-on': 'warn',
-  'jest/require-top-level-describe': 'error',
-
   // Code Quality
   'sonarjs/cognitive-complexity': ['error', 15],
   'sonarjs/no-duplicate-string': ['error', 5],
   'sonarjs/no-redundant-boolean': 'error',
   'sonarjs/prefer-immediate-return': 'error',
 
-  // RegExp
-  'regexp/no-missing-g-flag': 'error',
-  'regexp/no-useless-flag': 'error',
-  'regexp/prefer-quantifier': 'error',
-
   // Enhanced TypeScript
   'etc/no-commented-out-code': 'warn',
   'etc/no-implicit-any-catch': 'error',