@codfish/eslint-config 13.0.0 → 13.1.0

package/dist/rules/typescript.d.ts

@@ -0,0 +1,38 @@
+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

@@ -0,0 +1 @@
+{"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/index.js

@@ -8,14 +8,13 @@ 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');
 
 /**
@@ -34,7 +33,7 @@ export default defineConfig([
 
     files: ['**/*.{js,mjs,cjs,jsx,ts,mts,cts,tsx}'],
 
-    extends: [js.configs.recommended, tseslint.configs.recommended],
+    extends: [js.configs.recommended, ...tseslintConfig],
 
     plugins: {
       'simple-import-sort': simpleImportSortPlugin,
@@ -43,6 +42,9 @@ export default defineConfig([
     languageOptions: {
       ecmaVersion: 'latest',
       sourceType: 'module',
+      parserOptions: {
+        ecmaFeatures: { jsx: true },
+      },
       globals: {
         ...globals.browser,
         ...globals.node,
@@ -54,8 +56,10 @@ 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',
 
@@ -85,44 +89,6 @@ 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"`
@@ -170,6 +136,9 @@ 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": "13.0.0",
+  "version": "13.1.0",
   "description": "Modern ESLint configuration with TypeScript, React, and testing framework support.",
   "type": "module",
   "repository": {
@@ -54,40 +54,40 @@
     "prepare": "husky"
   },
   "dependencies": {
-    "@commitlint/cli": "^20.4.1",
-    "@commitlint/config-conventional": "^20.4.1",
+    "@commitlint/cli": "^20.4.2",
+    "@commitlint/config-conventional": "^20.4.2",
     "@eslint/js": "^10.0.0",
     "@eslint/json": "^1.0.1",
     "@eslint/markdown": "^7.5.1",
-    "@html-eslint/eslint-plugin": "^0.55.0",
-    "@html-eslint/parser": "^0.55.0",
+    "@html-eslint/eslint-plugin": "^0.57.1",
+    "@html-eslint/parser": "^0.57.1",
     "@next/eslint-plugin-next": "^16.1.6",
     "@tanstack/eslint-plugin-query": "^5.91.4",
     "@vitest/eslint-plugin": "^1.6.9",
     "cosmiconfig": "^9.0.0",
     "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-jest": "^29.14.0",
+    "eslint-plugin-jest": "^29.15.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-prettier": "^5.5.5",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.0.1",
-    "eslint-plugin-react-refresh": "^0.5.0",
+    "eslint-plugin-react-refresh": "^0.5.2",
     "eslint-plugin-simple-import-sort": "^12.1.1",
-    "eslint-plugin-testing-library": "^7.15.4",
-    "eslint-plugin-yml": "^3.1.2",
+    "eslint-plugin-testing-library": "^7.16.0",
+    "eslint-plugin-yml": "^3.3.0",
     "globals": "^17.3.0",
     "lodash.has": "^4.5.2",
     "prettier": ">= 3",
     "read-package-up": "^12.0.0",
-    "typescript-eslint": "^8.55.0"
+    "typescript-eslint": "^8.56.1"
   },
   "devDependencies": {
-    "@types/node": "^25.2.3",
+    "@types/node": "^25.3.3",
     "@vitest/coverage-v8": "^4.0.18",
-    "doctoc": "^2.2.1",
-    "eslint": "^10.0.0",
+    "doctoc": "^2.3.0",
+    "eslint": "^10.0.2",
     "husky": "^9.1.7",
-    "lint-staged": "^16.1.6",
+    "lint-staged": "^16.3.0",
     "typescript": "^5.9.2",
     "vitest": "^4.0.18"
   },
@@ -128,6 +128,6 @@
     ]
   },
   "volta": {
-    "node": "24.13.0"
+    "node": "24.14.0"
   }
 }

package/rules/typescript.js

@@ -0,0 +1,56 @@
+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',
+            },
+          },
+        },
+      ],
+    }
+  : {};