@codfish/eslint-config 0.0.0-PR-155--2a9a213 → 0.0.0-semantically-released

package/README.md

@@ -1,7 +1,7 @@
 # @codfish/eslint-config
 
 > Modern ESLint configuration with TypeScript, React/Next.js, YAML, Testing Library, and testing framework support using
-> ESLint v10+ flat config format.
+> ESLint v9+ flat config format.
 
 [![version](https://img.shields.io/npm/v/@codfish/eslint-config.svg)](http://npm.im/@codfish/eslint-config)
 [![downloads](https://img.shields.io/npm/dm/@codfish/eslint-config.svg)](http://npm-stat.com/charts.html?package=@codfish/eslint-config&from=2015-08-01)
@@ -24,10 +24,6 @@
   - [Use in combination with prettier-plugin-tailwindcss](#use-in-combination-with-prettier-plugin-tailwindcss)
 - [Example scripts](#example-scripts)
 - [Commitlint Configuration](#commitlint-configuration)
-- [Upgrading to ESLint 10](#upgrading-to-eslint-10)
-  - [Breaking Changes in ESLint 10](#breaking-changes-in-eslint-10)
-  - [Migration Steps](#migration-steps)
-  - [What Changed](#what-changed)
 - [Migration from Legacy Config](#migration-from-legacy-config)
 
 <!-- END doctoc generated TOC please keep comment here to allow auto update -->
@@ -35,7 +31,7 @@
 
 ## Features
 
-- **Modern ESLint v10+ flat config**: Uses the new flat configuration format
+- **Modern ESLint v9+ flat config**: Uses the new flat configuration format
 - **Dynamic feature detection**: Automatically configures based on your project's dependencies
 - **TypeScript support**: Full TypeScript linting with modern typescript-eslint parser and rules
 - **React ecosystem**: React, React Hooks, and JSX accessibility rules when React is detected
@@ -53,7 +49,7 @@
 Install the package and required peer dependencies:
 
 ```sh
-npm i -D eslint@10 @codfish/eslint-config
+npm i -D eslint@9 @codfish/eslint-config
 
 # Optionally, you can uninstall plugins or presets you don't need to manage anymore,
 # @codfish/eslint-config includes them all.
@@ -383,7 +379,7 @@ export default {
 ### Use in combination with prettier-plugin-tailwindcss
 
 ```sh
-npm i -D eslint@10 @codfish/eslint-config prettier-plugin-tailwindcss
+npm i -D eslint@9 @codfish/eslint-config prettier-plugin-tailwindcss
 ```
 
 ```js
@@ -505,72 +501,11 @@ jobs:
           --verbose
 ```
 
-## Upgrading to ESLint 10
-
-This package now requires **ESLint 10.0.0 or higher** and **Node.js v20.19.0 or higher**.
-
-### Breaking Changes in ESLint 10
-
-- **ESLint 10 Required**: Minimum ESLint version is now 10.0.0
-- **Node.js v20.19.0+**: Minimum Node.js version increased from v20.0.0 to v20.19.0
-- **Legacy eslintrc removed**: ESLint 10 completely removes the deprecated eslintrc config system (this package already
-  uses flat config)
-- **Improved JSX tracking**: ESLint 10 properly tracks JSX references in scope analysis, which may surface previously
-  hidden unused import warnings in React files
-- **New recommended rules**: Three new rules enabled in `eslint:recommended`:
-  - `no-unassigned-vars` - Disallow variables that are assigned but never used
-  - `no-useless-assignment` - Disallow assignments that don't change the value
-  - `preserve-caught-error` - Enforce that caught errors are not reassigned
-
-### Migration Steps
-
-1. **Update Node.js** (if needed):
-
-   ```sh
-   node --version  # Ensure you're on v20.19.0+ or v22.13.0+
-   ```
-
-2. **Update ESLint and this config**:
-
-   ```sh
-   npm install --save-dev eslint@10 @codfish/eslint-config@latest
-   ```
-
-3. **Install dependencies with legacy peer deps** (required until all plugins update):
-
-   ```sh
-   npm install --legacy-peer-deps
-   ```
-
-   This is necessary because some ESLint plugins haven't updated their `peerDependencies` to include ESLint 10 yet. The
-   plugins still work correctly with ESLint 10.
-
-4. **Run linting** and check for new violations:
-
-   ```sh
-   npm run lint
-   ```
-
-5. **Review React files** for newly reported unused imports. ESLint 10's improved JSX tracking may flag imports that
-   were previously ignored but are actually used in JSX.
-
-### What Changed
-
-Since this package already uses ESLint's flat config format (the biggest change in ESLint 9), the upgrade to ESLint 10
-is relatively smooth. The main changes are:
-
-- ✅ Flat config format (already implemented)
-- ✅ Plugin configurations updated to ESLint 10-compatible versions
-- ✅ All tests passing with ESLint 10
-- ⚠️ Some plugins show peer dependency warnings but work correctly (ecosystem is catching up)
-
-For more details, see the [ESLint 10 Migration Guide](https://eslint.org/docs/latest/use/migrate-to-10.0.0).
-
 ## Migration from Legacy Config
 
 If you're upgrading from an older version that used Airbnb presets:
 
-1. **Update to ESLint v10+**: `npm install --save-dev eslint@10`
+1. **Update to ESLint v9+**: `npm install --save-dev eslint@9`
 2. **Switch to flat config**: Replace `.eslintrc.js` with `eslint.config.js`
 3. **Use import syntax**: Change from `require()` to `import` statements
 4. **Remove explicit React config**: React support is now automatically detected

package/dist/utils.d.ts

@@ -1,8 +1,3 @@
-/**
- * Get the version of a dependency from the consumer's package.json.
- * Returns the major version as a string (e.g. '18', '19'), or null if not found.
- */
-export function getDepVersion(dep: any): string | null;
 export function hasLocalConfig(moduleName: any, searchOptions?: {}): boolean;
 export function hasDep(props: any): boolean;
 export function hasDevDep(props: any): boolean;

package/dist/utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../utils.js"],"names":[],"mappings":"AAyBA;;;GAGG;AACH,uDAKC;AAED,6EAKC;AA1BgC,4CAA+D;AAA/D,+CAA+D;AAA/D,gDAA+D;AAMzF,8CAA8E;AAE9E,yDAAmE"}
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../utils.js"],"names":[],"mappings":"AAyBA,6EAKC;AAfgC,4CAA+D;AAA/D,+CAA+D;AAA/D,gDAA+D;AAMzF,8CAA8E;AAE9E,yDAAmE"}

package/index.js

@@ -8,13 +8,14 @@ import prettierConfig from 'eslint-plugin-prettier/recommended';
 import simpleImportSortPlugin from 'eslint-plugin-simple-import-sort';
 import ymlPlugin from 'eslint-plugin-yml';
 import globals from 'globals';
+import tseslint from 'typescript-eslint';
 
 import prettierBuiltInConfig from './prettier.js';
 import configFilesConfig from './rules/config-files.js';
 import reactConfig from './rules/react.js';
 import testConfig from './rules/tests.js';
-import tseslintConfig, { rules as tsRules } from './rules/typescript.js';
 import { hasLocalConfig, ifAnyDep } from './utils.js';
+
 const useBuiltinPrettierConfig = !hasLocalConfig('prettier');
 
 /**
@@ -33,7 +34,7 @@ export default defineConfig([
 
     files: ['**/*.{js,mjs,cjs,jsx,ts,mts,cts,tsx}'],
 
-    extends: [js.configs.recommended, ...tseslintConfig],
+    extends: [js.configs.recommended, tseslint.configs.recommended],
 
     plugins: {
       'simple-import-sort': simpleImportSortPlugin,
@@ -42,9 +43,6 @@ export default defineConfig([
     languageOptions: {
       ecmaVersion: 'latest',
       sourceType: 'module',
-      parserOptions: {
-        ecmaFeatures: { jsx: true },
-      },
       globals: {
         ...globals.browser,
         ...globals.node,
@@ -56,10 +54,8 @@ export default defineConfig([
       // Error handling rules to enforce using the Error object
       'no-throw-literal': 'error',
       'prefer-promise-reject-errors': 'error',
-      'no-unused-vars': ['error', { ignoreRestSiblings: true }],
-      'no-empty-function': 'off',
 
-      // Asynchronous functions that don't use await might not need to be asynchronous functions
+      // Asynchronous functions that don’t use await might not need to be asynchronous functions
       // Usually result of refactoring, leads to misunderstanding/misusage
       'require-await': 'error',
 
@@ -89,6 +85,44 @@ export default defineConfig([
       'simple-import-sort/exports': 'error',
       'sort-imports': 'off',
 
+      // Allows destructuring of rest properties even if they are unused
+      '@typescript-eslint/no-unused-vars': ['error', { ignoreRestSiblings: true }],
+      '@typescript-eslint/no-empty-function': 'off',
+      '@typescript-eslint/naming-convention': [
+        'error',
+        {
+          selector: ['interface', 'typeAlias'],
+          format: ['PascalCase'],
+          custom: {
+            regex: '^I[A-Z]', // prevent prefixing interfaces and type alias declarations with "I"
+            match: false,
+          },
+        },
+      ],
+      '@typescript-eslint/ban-ts-comment': [
+        'error',
+        {
+          // If you need to use a ts comment, make sure you have a description.
+          'ts-ignore': 'allow-with-description',
+          'ts-expect-error': 'allow-with-description',
+          'ts-nocheck': 'allow-with-description',
+        },
+      ],
+
+      '@typescript-eslint/no-restricted-types': [
+        'error',
+        {
+          types: {
+            'React.FC': {
+              message: 'Useless and has some drawbacks, see https://github.com/facebook/create-react-app/pull/8177',
+            },
+            'React.FunctionComponent': {
+              message: 'Useless and has some drawbacks, see https://github.com/facebook/create-react-app/pull/8177',
+            },
+          },
+        },
+      ],
+
       // 1. Encouraging `lodash-es` imports per file
       // lodash imports should use `lodash-es` package and should be imported per file.
       //    E.G: `import get from "lodash-es/get"`
@@ -136,9 +170,6 @@ export default defineConfig([
           ],
         },
       ],
-
-      // TypeScript rules (empty object when typescript is not installed)
-      ...tsRules,
     },
   },
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codfish/eslint-config",
-  "version": "0.0.0-PR-155--2a9a213",
+  "version": "0.0.0-semantically-released",
   "description": "Modern ESLint configuration with TypeScript, React, and testing framework support.",
   "type": "module",
   "repository": {
@@ -54,45 +54,44 @@
     "prepare": "husky"
   },
   "dependencies": {
-    "@commitlint/cli": "^20.4.2",
-    "@commitlint/config-conventional": "^20.4.2",
-    "@eslint/js": "^10.0.0",
-    "@eslint/json": "^1.0.1",
-    "@eslint/markdown": "^8.0.0",
-    "@html-eslint/eslint-plugin": "^0.58.0",
-    "@html-eslint/parser": "^0.58.0",
+    "@commitlint/cli": "^20.4.0",
+    "@commitlint/config-conventional": "^20.4.0",
+    "@eslint/js": "^9.35.0",
+    "@eslint/json": "^1.0.0",
+    "@eslint/markdown": "^7.5.1",
+    "@html-eslint/eslint-plugin": "^0.54.2",
+    "@html-eslint/parser": "^0.54.0",
     "@next/eslint-plugin-next": "^16.1.6",
     "@tanstack/eslint-plugin-query": "^5.91.4",
-    "@vitest/eslint-plugin": "^1.6.9",
+    "@vitest/eslint-plugin": "^1.3.10",
     "cosmiconfig": "^9.0.0",
     "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-jest": "^29.15.0",
+    "eslint-plugin-jest": "^29.0.1",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-prettier": "^5.5.5",
+    "eslint-plugin-prettier": "^5.5.4",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.0.1",
-    "eslint-plugin-react-refresh": "^0.5.2",
+    "eslint-plugin-react-refresh": "^0.5.0",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "eslint-plugin-testing-library": "^7.16.0",
-    "eslint-plugin-yml": "^3.3.0",
+    "eslint-plugin-testing-library": "^7.1.0",
+    "eslint-plugin-yml": "^3.0.0",
     "globals": "^17.3.0",
     "lodash.has": "^4.5.2",
     "prettier": ">= 3",
     "read-package-up": "^12.0.0",
-    "typescript-eslint": "^8.56.1"
+    "typescript-eslint": "^8.44.0"
   },
   "devDependencies": {
-    "@types/node": "^25.3.3",
     "@vitest/coverage-v8": "^4.0.18",
-    "doctoc": "^2.3.0",
-    "eslint": "^10.0.2",
+    "doctoc": "^2.2.1",
+    "eslint": "^9.35.0",
     "husky": "^9.1.7",
-    "lint-staged": "^17.0.0",
-    "typescript": "^6.0.0",
+    "lint-staged": "^16.1.6",
+    "typescript": "^5.9.2",
     "vitest": "^4.0.18"
   },
   "engines": {
-    "node": ">=20.19.0"
+    "node": ">=20.0.0"
   },
   "files": [
     "dist",
@@ -105,7 +104,7 @@
     "commitlint.js"
   ],
   "peerDependencies": {
-    "eslint": ">= 10",
+    "eslint": ">= 9",
     "prettier": ">= 3"
   },
   "peerDependenciesMeta": {
@@ -128,6 +127,6 @@
     ]
   },
   "volta": {
-    "node": "24.14.1"
+    "node": "24.8.0"
   }
 }

package/rules/react.js

@@ -7,7 +7,7 @@ import reactHooks from 'eslint-plugin-react-hooks';
 import reactRefresh from 'eslint-plugin-react-refresh';
 import globals from 'globals';
 
-import { getDepVersion, ifAnyDep } from '../utils.js';
+import { ifAnyDep } from '../utils.js';
 
 /**
  * React ESLint configuration. Includes React, React Hooks, and JSX accessibility rules.
@@ -41,9 +41,7 @@ export default defineConfig([
 
     settings: {
       react: {
-        // Avoid 'detect' — it uses context.getFilename() which was removed in ESLint 10.
-        // Read the major version from the consumer's package.json instead.
-        version: getDepVersion('react') || 'detect',
+        version: 'detect',
       },
     },
 

package/utils.js

@@ -23,17 +23,6 @@ export const hasAnyDep = args => [hasDep, hasDevDep, hasPeerDep].some(fn => fn(a
 
 export const ifAnyDep = (deps, t, f) => (hasAnyDep([deps].flat()) ? t : f);
 
-/**
- * Get the version of a dependency from the consumer's package.json.
- * Returns the major version as a string (e.g. '18', '19'), or null if not found.
- */
-export function getDepVersion(dep) {
-  const spec = pkg?.dependencies?.[dep] || pkg?.devDependencies?.[dep] || pkg?.peerDependencies?.[dep];
-  if (!spec) return null;
-  const match = spec.match(/\d+/);
-  return match ? match[0] : null;
-}
-
 export function hasLocalConfig(moduleName, searchOptions = {}) {
   const explorerSync = cosmiconfigSync(moduleName, searchOptions);
   const result = explorerSync.search(pkgPath || './');

package/dist/rules/typescript.d.ts

@@ -1,38 +0,0 @@
-export default tseslintConfig;
-export const rules: {
-    '@typescript-eslint/no-unused-vars': (string | {
-        ignoreRestSiblings: boolean;
-    })[];
-    '@typescript-eslint/no-empty-function': string;
-    '@typescript-eslint/naming-convention': (string | {
-        selector: string[];
-        format: string[];
-        custom: {
-            regex: string;
-            match: boolean;
-        };
-    })[];
-    '@typescript-eslint/ban-ts-comment': (string | {
-        'ts-ignore': string;
-        'ts-expect-error': string;
-        'ts-nocheck': string;
-    })[];
-    '@typescript-eslint/no-restricted-types': (string | {
-        types: {
-            'React.FC': {
-                message: string;
-            };
-            'React.FunctionComponent': {
-                message: string;
-            };
-        };
-    })[];
-} | {
-    '@typescript-eslint/no-unused-vars'?: never;
-    '@typescript-eslint/no-empty-function'?: never;
-    '@typescript-eslint/naming-convention'?: never;
-    '@typescript-eslint/ban-ts-comment'?: never;
-    '@typescript-eslint/no-restricted-types'?: never;
-};
-declare const tseslintConfig: import("eslint").Linter.Config[];
-//# sourceMappingURL=typescript.d.ts.map

package/dist/rules/typescript.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"typescript.d.ts","sourceRoot":"","sources":["../../rules/typescript.js"],"names":[],"mappings":";AAeA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAwCO;AA9CP,8BACa,OAAO,QAAQ,EAAE,MAAM,CAAC,MAAM,EAAE,CACsC"}

package/dist/tests/integration/react-integration.spec.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=react-integration.spec.d.ts.map

package/dist/tests/integration/react-integration.spec.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"react-integration.spec.d.ts","sourceRoot":"","sources":["../../../tests/integration/react-integration.spec.js"],"names":[],"mappings":""}

package/rules/typescript.js

@@ -1,56 +0,0 @@
-import { hasAnyDep } from '../utils.js';
-
-/**
- * TypeScript ESLint configuration. Dynamically loads typescript-eslint
- * only when typescript is installed in the consuming project.
- *
- * - https://typescript-eslint.io/
- */
-const hasTypeScript = hasAnyDep('typescript');
-const tseslintConfig =
-  /** @type {import('eslint').Linter.Config[]} */
-  (hasTypeScript ? [(await import('typescript-eslint')).configs.recommended] : []);
-
-export default tseslintConfig;
-
-export const rules = hasTypeScript
-  ? {
-      // Allows destructuring of rest properties even if they are unused
-      '@typescript-eslint/no-unused-vars': ['error', { ignoreRestSiblings: true }],
-      '@typescript-eslint/no-empty-function': 'off',
-      '@typescript-eslint/naming-convention': [
-        'error',
-        {
-          selector: ['interface', 'typeAlias'],
-          format: ['PascalCase'],
-          custom: {
-            regex: '^I[A-Z]', // prevent prefixing interfaces and type alias declarations with "I"
-            match: false,
-          },
-        },
-      ],
-      '@typescript-eslint/ban-ts-comment': [
-        'error',
-        {
-          // If you need to use a ts comment, make sure you have a description.
-          'ts-ignore': 'allow-with-description',
-          'ts-expect-error': 'allow-with-description',
-          'ts-nocheck': 'allow-with-description',
-        },
-      ],
-
-      '@typescript-eslint/no-restricted-types': [
-        'error',
-        {
-          types: {
-            'React.FC': {
-              message: 'Useless and has some drawbacks, see https://github.com/facebook/create-react-app/pull/8177',
-            },
-            'React.FunctionComponent': {
-              message: 'Useless and has some drawbacks, see https://github.com/facebook/create-react-app/pull/8177',
-            },
-          },
-        },
-      ],
-    }
-  : {};