@agilebot/eslint-config 0.4.1 → 0.4.3

package/dist/cli.js

@@ -1,5 +1,5 @@
 /**
- * @license @agilebot/eslint-config v0.4.1
+ * @license @agilebot/eslint-config v0.4.3
  *
  * Copyright (c) Agilebot, Inc. and its affiliates.
  *

package/dist/index.js

@@ -1,5 +1,5 @@
 /**
- * @license @agilebot/eslint-config v0.4.1
+ * @license @agilebot/eslint-config v0.4.3
  *
  * Copyright (c) Agilebot, Inc. and its affiliates.
  *
@@ -335,7 +335,7 @@ function imports(opts) {
           noUselessIndex: true
         }
       ],
-      "import-x/no-cycle": "error",
+      "import-x/no-cycle": (0, import_eslint_utils2.isInEditor)() ? "off" : "error",
       "import-x/newline-after-import": "error",
       "import-x/first": "error",
       "import-x/no-import-module-exports": "error",
@@ -455,20 +455,13 @@ function react(opts) {
       "@eslint-react",
       "react-hooks",
       "jsx-a11y",
-      "@stylistic"
+      "@stylistic",
+      "@agilebot"
     ],
     extends: ["plugin:jsx-a11y/recommended"],
     rules: {
       // recommended rules from eslint-plugin-react
-      "react/jsx-no-duplicate-props": "error",
-      "react/jsx-no-undef": "error",
-      "react/jsx-uses-react": "error",
-      "react/jsx-uses-vars": "error",
       "react/no-is-mounted": "error",
-      "react/no-unescaped-entities": "error",
-      "react/no-unknown-property": "error",
-      "react/react-in-jsx-scope": "error",
-      "react/require-render-return": "error",
       // recommended rules from @eslint-react
       "@eslint-react/no-direct-mutation-state": "error",
       // Children API
@@ -504,6 +497,8 @@ function react(opts) {
       "@eslint-react/no-access-state-in-setstate": "error",
       "@eslint-react/prefer-shorthand-fragment": "error",
       "@eslint-react/no-missing-component-display-name": "error",
+      "@eslint-react/no-prop-types": "error",
+      "@eslint-react/no-useless-fragment": "error",
       // recommended rules from @eslint-react/dom
       "@eslint-react/dom/no-children-in-void-dom-elements": "error",
       "@eslint-react/dom/no-unsafe-target-blank": "error",
@@ -514,7 +509,7 @@ function react(opts) {
       // recommended rules from @eslint-react/hooks-extra
       "@eslint-react/hooks-extra/prefer-use-state-lazy-initialization": "warn",
       // recommended rules react-hooks
-      "react-hooks/rules-of-hooks": "error",
+      "@agilebot/react-rules-of-hooks": "error",
       // recommended rules from @eslint-react/naming-convention
       "@eslint-react/naming-convention/component-name": "error",
       "@eslint-react/naming-convention/use-state": "error",
@@ -531,8 +526,11 @@ function react(opts) {
           checkLocalVariables: true
         }
       ],
+      "react/no-unused-prop-types": "error",
+      // Stylistic rules are not supported by @eslint-react. Use dprint or @stylistic instead.
+      "@stylistic/jsx-self-closing-comp": "error",
       // <App prop={'Hello'} />没必要，使用<App prop="Hello" />
-      "react/jsx-curly-brace-presence": [
+      "@stylistic/jsx-curly-brace-presence": [
         "error",
         {
           props: "never",
@@ -540,10 +538,6 @@ function react(opts) {
           propElementValues: "always"
         }
       ],
-      // 禁止声明未使用的props
-      "react/no-unused-prop-types": "error",
-      "@eslint-react/no-useless-fragment": "error",
-      "@stylistic/jsx-self-closing-comp": "error",
       "jsx-a11y/click-events-have-key-events": "off",
       "jsx-a11y/no-static-element-interactions": "off",
       "jsx-a11y/alt-text": "off"
@@ -555,6 +549,21 @@ function react(opts) {
     }
   };
 }
+function reactJsOnly() {
+  return {
+    rules: {
+      // The following can be enforced by TypeScript, no need to implement them
+      "react/jsx-no-duplicate-props": "error",
+      "react/jsx-no-undef": "error",
+      "react/jsx-uses-react": "error",
+      "react/jsx-uses-vars": "error",
+      "react/no-unescaped-entities": "error",
+      "react/no-unknown-property": "error",
+      "react/react-in-jsx-scope": "error",
+      "react/require-render-return": "error"
+    }
+  };
+}
 
 // src/configs/vue.ts
 function vue(opts) {
@@ -843,7 +852,8 @@ function factory(root, options) {
           jsOnly(),
           (options == null ? void 0 : options.jsdoc) ? jsdoc({
             ts: false
-          }) : {}
+          }) : {},
+          (options == null ? void 0 : options.react) ? reactJsOnly() : {}
         )),
         __spreadValues({
           files: TS_EXTS.map((ext) => `*${ext}`)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agilebot/eslint-config",
-  "version": "0.4.1",
+  "version": "0.4.3",
   "description": "Agilebot's ESLint config",
   "bin": {
     "eslint-agilebot": "bin/eslint-agilebot"
@@ -38,20 +38,19 @@
     "eslint-plugin-prettier": "^5.2.1",
     "eslint-plugin-promise": "^7.1.0",
     "eslint-plugin-react": "^7.35.0",
-    "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-unicorn": "^55.0.0",
     "eslint-plugin-unused-imports": "^4.1.3",
     "eslint-plugin-vue": "^9.27.0",
     "eslint-plugin-you-dont-need-lodash-underscore": "^6.13.0",
     "vue-eslint-parser": "^9.4.3",
-    "@agilebot/eslint-utils": "0.4.1"
+    "@agilebot/eslint-utils": "0.4.3"
   },
   "devDependencies": {
     "eslint-config-love": "^63.0.0"
   },
   "peerDependencies": {
     "eslint": "^7.0.0 || ^8.0.0 || ^9.0.0",
-    "@agilebot/eslint-plugin": "0.4.1"
+    "@agilebot/eslint-plugin": "0.4.3"
   },
   "files": [
     "bin",