eslint-config-agent 1.9.3 → 2.0.0

package/index.js

@@ -1,4 +1,5 @@
 import js from '@eslint/js'
+import earlyReturn from 'eslint-plugin-early-return'
 import reactHooks from 'eslint-plugin-react-hooks'
 import { defineConfig } from 'eslint/config'
 import tseslint from 'typescript-eslint'
@@ -104,23 +105,10 @@ const config = defineConfig([
   {
     plugins,
   },
-  // Use recommended-latest if available (v5+), otherwise create flat config equivalent of legacy recommended
-  ...(reactHooks.configs['recommended-latest']
-    ? [reactHooks.configs['recommended-latest']]
-    : [
-        {
-          name: 'react-hooks/recommended-flat',
-          plugins: {
-            'react-hooks': reactHooks,
-          },
-          rules: {
-            'react-hooks/rules-of-hooks': 'error',
-            'react-hooks/exhaustive-deps': 'warn',
-          },
-        },
-      ]),
+  reactHooks.configs['recommended-latest'],
   js.configs.recommended,
   ...tseslint.configs.recommended,
+  earlyReturn.configs.recommended,
 
   // Base plugin strict configs (error, default, guard-clauses)
   ...basePluginsConfig,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-agent",
-  "version": "1.9.3",
+  "version": "2.0.0",
   "description": "ESLint configuration package with TypeScript support",
   "main": "index.js",
   "type": "module",
@@ -61,37 +61,17 @@
   "author": "tupe12334",
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/tupe12334/eslint-config.git"
+    "url": "git+https://github.com/tupe12334/eslint-config-agent.git"
   },
   "bugs": {
-    "url": "https://github.com/tupe12334/eslint-config/issues"
+    "url": "https://github.com/tupe12334/eslint-config-agent/issues"
   },
-  "homepage": "https://github.com/tupe12334/eslint-config#readme",
+  "homepage": "https://github.com/tupe12334/eslint-config-agent#readme",
   "license": "MIT",
   "publishConfig": {
     "access": "public"
   },
   "packageManager": "pnpm@10.18.2",
-  "peerDependencies": {
-    "@eslint/eslintrc": "^3.3.1",
-    "@eslint/js": "^9.34.0",
-    "@typescript-eslint/eslint-plugin": "^8.40.0",
-    "@typescript-eslint/parser": "^8.40.0",
-    "eslint": "^9.34.0",
-    "eslint-plugin-class-export": "^1.0.1",
-    "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-n": "^17.21.3",
-    "eslint-plugin-react": "^7.37.5",
-    "eslint-plugin-react-hooks": "^5.2.0",
-    "eslint-plugin-security": "^3.0.1",
-    "eslint-plugin-storybook": "^9.1.5",
-    "typescript-eslint": "^8.40.0"
-  },
-  "peerDependenciesMeta": {
-    "eslint-plugin-preact": {
-      "optional": true
-    }
-  },
   "devDependencies": {
     "@commitlint/cli": "^20.1.0",
     "@commitlint/config-conventional": "^20.0.0",
@@ -119,12 +99,27 @@
     "typescript-eslint": "^8.46.1"
   },
   "dependencies": {
+    "@eslint/eslintrc": "^3.3.1",
+    "@eslint/js": "^9.34.0",
+    "@typescript-eslint/eslint-plugin": "^8.40.0",
+    "@typescript-eslint/parser": "^8.40.0",
+    "eslint": "^9.34.0",
+    "eslint-plugin-class-export": "^1.0.1",
     "eslint-plugin-ddd": "^0.5.2",
     "eslint-plugin-default": "^1.1.0",
+    "eslint-plugin-early-return": "^1.0.0",
     "eslint-plugin-error": "^1.3.0",
+    "eslint-plugin-import": "^2.32.0",
+    "eslint-plugin-n": "^17.21.3",
     "eslint-plugin-no-optional-chaining": "^1.0.0",
+    "eslint-plugin-preact": "^0.1.0",
+    "eslint-plugin-react": "^7.37.5",
+    "eslint-plugin-react-hooks": "^5.2.0",
     "eslint-plugin-required-exports": "^0.2.0",
+    "eslint-plugin-security": "^3.0.1",
     "eslint-plugin-single-export": "^1.1.2",
-    "globals": "^16.4.0"
+    "eslint-plugin-storybook": "^9.1.5",
+    "globals": "^16.4.0",
+    "typescript-eslint": "^8.40.0"
   }
 }

package/plugins/index.js

@@ -17,17 +17,10 @@ import errorPlugin from 'eslint-plugin-error'
 import defaultPlugin from 'eslint-plugin-default'
 import noOptionalChainingPlugin from 'eslint-plugin-no-optional-chaining'
 import dddPlugin from 'eslint-plugin-ddd'
+import preactPlugin from 'eslint-plugin-preact'
 import allRules from '../rules/index.js'
 import { noDefaultClassExportRule } from '../rules/no-default-class-export/index.js'
 
-// Conditionally import preact plugin if available
-let preactPlugin = null
-try {
-  preactPlugin = (await import('eslint-plugin-preact')).default
-} catch {
-  // eslint-plugin-preact is not available
-}
-
 // Centralized plugin configuration
 export const plugins = {
   react: reactPlugin,
@@ -42,7 +35,7 @@ export const plugins = {
   default: defaultPlugin,
   'no-optional-chaining': noOptionalChainingPlugin,
   ddd: dddPlugin,
-  ...(preactPlugin && { preact: preactPlugin }),
+  preact: preactPlugin,
   custom: {
     rules: {
       'no-default-class-export': noDefaultClassExportRule,

package/rules/error-only-exports/index.js

@@ -119,26 +119,28 @@ export const errorOnlyExportsRule = {
         // Only check on program exit to ensure we've seen all exports
       },
       'Program:exit'(node) {
-        if (isErrorOnlyFile(context)) {
-          // Check if the file already has the eslint-disable comment
-          const sourceCode = context.sourceCode || context.getSourceCode()
-          const comments = sourceCode.getAllComments()
-
-          const hasDisableComment = comments.some(
-            comment =>
-              comment.value.includes('eslint-disable ddd/require-spec-file') ||
-              comment.value.includes(
-                'eslint-disable-next-line ddd/require-spec-file'
-              )
-          )
-
-          // Only report if the disable comment is not present
-          if (!hasDisableComment) {
-            context.report({
-              node,
-              messageId: 'errorOnlyFile',
-            })
-          }
+        if (!isErrorOnlyFile(context)) {
+          return
+        }
+
+        // Check if the file already has the eslint-disable comment
+        const sourceCode = context.sourceCode || context.getSourceCode()
+        const comments = sourceCode.getAllComments()
+
+        const hasDisableComment = comments.some(
+          comment =>
+            comment.value.includes('eslint-disable ddd/require-spec-file') ||
+            comment.value.includes(
+              'eslint-disable-next-line ddd/require-spec-file'
+            )
+        )
+
+        // Only report if the disable comment is not present
+        if (!hasDisableComment) {
+          context.report({
+            node,
+            messageId: 'errorOnlyFile',
+          })
         }
       },
     }