@petbee/eslint-config 3.0.4 → 3.0.5

package/index.mjs

@@ -10,12 +10,14 @@ import importsConfig from './rules/imports.js'
 import nestjsConfig from './rules/nestjs.js'
 import nodeConfig from './rules/node.js'
 import prettierConfig from './rules/prettier.js'
+import reactConfig from './rules/react.js'
 import styleConfig from './rules/style.js'
 import testsConfig from './rules/tests.js'
 import typescriptConfig from './rules/typescript.js'
 import variablesConfig from './rules/variables.js'
 
 const hasTypescript = hasPackage('typescript')
+const hasReact = hasPackage('react')
 
 const ignoreConfig = {
   ignores: ['coverage', 'dist', '**/dist/', 'node_modules', '**/node_modules'],
@@ -33,6 +35,7 @@ const eslintFlatConfig = [
   ...getFlat(bestPracticesConfig),
   ...getFlat(importsConfig),
   ...getFlat(testsConfig),
+  ...(hasReact ? getFlat(reactConfig) : []),
 ]
 
 const nestjsEslintFlatConfig = nestjsConfig.flat

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@petbee/eslint-config",
-  "version": "3.0.4",
+  "version": "3.0.5",
   "description": "Petbee's eslint config",
   "keywords": [
     "eslint",
@@ -81,5 +81,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "13a8a59332c6aa1b447f5d830041e7ed00d3e7a2"
+  "gitHead": "033cab1e3f57a1b92490ef1326415d33a21b0ec9"
 }

package/rules/react.js

@@ -1,23 +1,52 @@
+const reactPlugin = require('eslint-plugin-react')
+const reactHooksPlugin = require('eslint-plugin-react-hooks')
+
+const reactRules = {
+  // Allow non-camelCase naming in React projects (e.g., components, hooks, CSS modules)
+  '@typescript-eslint/naming-convention': 'off',
+  // Enforce consistent usage of function component definition
+  // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md
+  'react/function-component-definition': [
+    'error',
+    {
+      namedComponents: 'arrow-function',
+      unnamedComponents: 'arrow-function',
+    },
+  ],
+  // Prevent missing React when using JSX (React 17+ JSX transform)
+  'react/react-in-jsx-scope': 'off',
+  // Enforce rules of hooks
+  'react-hooks/rules-of-hooks': 'error',
+  // Verify the list of the dependencies for Hooks like useEffect and similar
+  'react-hooks/exhaustive-deps': 'warn',
+}
+
+// Legacy config for ESLint < 9
 module.exports = {
   plugins: ['react'],
   extends: ['plugin:react/recommended', 'plugin:react-hooks/recommended'],
-  rules: {
-    // Allow non-camelCase naming in React projects (e.g., components, hooks, CSS modules)
-    '@typescript-eslint/naming-convention': 'off',
-    // Enforce consistent usage of function component definition
-    // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md
-    'react/function-component-definition': [
-      'error',
-      {
-        namedComponents: 'arrow-function',
-        unnamedComponents: 'arrow-function',
+  rules: reactRules,
+}
+
+// Flat config for ESLint v9+
+module.exports.flat = [
+  {
+    files: ['**/*.{jsx,tsx}'],
+    plugins: {
+      react: reactPlugin,
+      'react-hooks': reactHooksPlugin,
+    },
+    languageOptions: {
+      parserOptions: {
+        ecmaFeatures: {
+          jsx: true,
+        },
       },
-    ],
-    // Prevent missing React when using JSX (React 17+ JSX transform)
-    'react/react-in-jsx-scope': 'off',
-    // Enforce rules of hooks
-    'react-hooks/rules-of-hooks': 'error',
-    // Verify the list of the dependencies for Hooks like useEffect and similar
-    'react-hooks/exhaustive-deps': 'warn',
+    },
+    rules: {
+      ...reactPlugin.configs.recommended.rules,
+      ...reactHooksPlugin.configs.recommended.rules,
+      ...reactRules,
+    },
   },
-}
+]

package/rules/typescript.js

@@ -2,15 +2,15 @@ const { hasPackage } = require('../lib/utils')
 const { hasTypescript, tsParserOptions, tsCommonRules } = require('./shared-config.js')
 
 const hasNestJs = hasPackage('@nestjs/core')
-const hasReact = hasPackage('react')
 const hasNext = hasPackage('next') || hasPackage('nextjs')
 
 // Load framework-specific rules
-const reactRules = hasReact ? require('./react.js').rules : {}
 const nextjsRules = hasNext ? require('./nextjs.js').rules : {}
 // Note: Don't require nestjs.js here to avoid circular dependency.
 // NestJS handles its own flat config completely.
+// React rules are now handled separately in index.mjs
 const nestjsRules = {}
+const reactRules = {}
 
 const tsConfigOptions = [
   {