@mikey-pro/eslint-config 10.3.4 → 10.3.8

package/base-config.js

@@ -63,7 +63,7 @@ export const globalPlugins = {
 };
 
 export const baseConfig = {
-  files: ['**/*.{js,jsx,ts,tsx,mjs,cjs}'],
+  files: ['**/*.{js,jsx,ts,tsx,mjs,cjs,mts,cts}'],
   languageOptions: {
     ecmaVersion: 'latest',
     globals: {
@@ -117,9 +117,19 @@ export const baseConfig = {
       'error',
       {
         devDependencies: [
-          '**/*.test.{js,ts}',
-          '**/*.spec.{js,ts}',
+          '**/*.{test,spec}.{js,jsx,mjs,cjs,ts,tsx,mts,cts}',
+          '**/__tests__/**',
+          '**/__mocks__/**',
           '**/test/**',
+          '**/tests/**',
+          '**/{test,vitest,jest}.setup.*',
+          '**/setupTests.*',
+          '**/testSetup.*',
+          '**/vitest.config.*',
+          '**/vite.config.*',
+          '**/jest.config.*',
+          '**/playwright.config.*',
+          '**/cypress.config.*',
         ],
       },
     ],
@@ -140,10 +150,20 @@ export const baseConfig = {
     ],
     'import-x/no-useless-path-segments': 'error',
     'import-x/no-webpack-loader-syntax': 'error',
+    // Must agree with simple-import-sort/imports (node: builtins, packages,
+    // then aliases like @/, then relative with parent/sibling/index merged).
+    // simple-import-sort owns ordering within a group (no alphabetize here:
+    // it would put '.' before '../x', the opposite of simple-import-sort), and
+    // scoped packages (@scope/pkg) sort among externals, so no '@*' pathGroup.
     'import-x/order': [
       'error',
       {
-        alphabetize: { order: 'asc' },
+        groups: [
+          'builtin',
+          'external',
+          'internal',
+          ['parent', 'sibling', 'index'],
+        ],
         'newlines-between': 'always',
         pathGroups: [
           {
@@ -151,15 +171,9 @@ export const baseConfig = {
             pattern: 'node:*',
             position: 'before',
           },
-          {
-            group: 'external',
-            pattern: '@*',
-            position: 'after',
-          },
           {
             group: 'internal',
-            pattern: '@/**',
-            position: 'after',
+            pattern: '{@,~}/**',
           },
         ],
         pathGroupsExcludedImportTypes: ['builtin'],
@@ -381,7 +395,8 @@ export const baseConfig = {
     'sort-imports': 'off',
 
     'sort-vars': 'warn',
-    'spaced-comment': ['warn', 'always'],
+    // markers: ['/'] exempts TypeScript triple-slash directives (`/// <reference />`)
+    'spaced-comment': ['warn', 'always', { markers: ['/'] }],
     strict: ['error', 'never'],
     'symbol-description': 'error',
 

package/overrides.js

@@ -17,7 +17,7 @@ import * as yamlParser from 'yaml-eslint-parser';
 
 // TypeScript configuration
 export const ts = {
-  files: ['**/*.ts', '**/*.tsx'],
+  files: ['**/*.ts', '**/*.tsx', '**/*.mts', '**/*.cts'],
   languageOptions: {
     ecmaVersion: 'latest',
     parser: tsParser,
@@ -195,10 +195,20 @@ export const ts = {
     '@typescript-eslint/switch-exhaustiveness-check': 'error',
     '@typescript-eslint/unbound-method': 'off',
     '@typescript-eslint/use-unknown-in-catch-callback-variable': 'off',
+    // Override base config's babel parser for TypeScript files
+    'prettier/prettier': [
+      'warn',
+      {
+        endOfLine: 'lf',
+        parser: 'typescript',
+        singleQuote: true,
+        trailingComma: 'all',
+      },
+    ],
   },
   settings: {
     'import-x/parsers': {
-      '@typescript-eslint/parser': ['.ts', '.tsx', '.d.ts'],
+      '@typescript-eslint/parser': ['.ts', '.tsx', '.mts', '.cts', '.d.ts'],
     },
     'import-x/resolver': {
       typescript: { alwaysTryTypes: true },
@@ -389,7 +399,11 @@ export const json5 = {
 
 // Jest JavaScript files
 export const jestJs = {
-  files: ['**/*.test.js', '**/__tests__/**/*.js'],
+  files: [
+    '**/*.{test,spec}.{js,jsx,mjs,cjs}',
+    '**/__tests__/**/*.{js,jsx,mjs,cjs}',
+    '**/__mocks__/**/*.{js,jsx,mjs,cjs}',
+  ],
   languageOptions: {
     globals: { jest: true },
   },
@@ -432,14 +446,36 @@ export const jestJs = {
     'unicorn/no-array-callback-reference': 'off',
     'unicorn/prevent-abbreviations': ['warn', { ignore: [/e2e/] }],
   },
+  settings: {
+    jest: {
+      // Recognize Vitest's import { describe, it, expect } from 'vitest' as test globals
+      globalPackage: 'vitest',
+    },
+  },
 };
 
 // Jest TypeScript files
 export const jestTs = {
-  files: ['**/*.test.ts', '**/__tests__/**/*.ts'],
+  files: [
+    '**/*.{test,spec}.{ts,tsx,mts,cts}',
+    '**/__tests__/**/*.{ts,tsx,mts,cts}',
+    '**/__mocks__/**/*.{ts,tsx,mts,cts}',
+  ],
   languageOptions: {
     globals: { jest: true },
     parser: tsParser,
+    // Preserve the `ts` override's JSX + type-aware parserOptions (overrides.js:20-29).
+    // Flat config merges languageOptions per matching block in array order; this
+    // block matches after `ts`, and without an explicit parserOptions here a naive
+    // merge could drop JSX support for `.test.tsx`/`.spec.tsx` files. ESLint 10's
+    // languageOptions merge actually preserves keys the later block omits, but this
+    // is pinned explicitly so the behavior doesn't silently depend on merge internals.
+    parserOptions: {
+      ecmaFeatures: { jsx: true },
+      extraFileExtensions: ['.vue', '.svelte'],
+      project: true,
+      tsconfigRootDir: process.cwd(),
+    },
   },
   rules: {
     ...jest.configs['flat/all'].rules,
@@ -476,6 +512,12 @@ export const jestTs = {
     'unicorn/no-array-callback-reference': 'off',
     'unicorn/prevent-abbreviations': ['warn', { ignore: [/e2e/] }],
   },
+  settings: {
+    jest: {
+      // Recognize Vitest's import { describe, it, expect } from 'vitest' as test globals
+      globalPackage: 'vitest',
+    },
+  },
 };
 
 // Cypress files

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikey-pro/eslint-config",
-  "version": "10.3.4",
+  "version": "10.3.8",
   "description": "Mikey Pro ESLint configuration - Ultimate coding style guide for excellence",
   "main": "index.js",
   "types": "index.d.ts",