@agilebot/eslint-config 0.4.0 → 0.4.2

package/dist/cli.js

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

package/dist/index.js

@@ -1,5 +1,5 @@
 /**
- * @license @agilebot/eslint-config v0.4.0
+ * @license @agilebot/eslint-config v0.4.2
  *
  * 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",
@@ -460,15 +460,7 @@ function react(opts) {
     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 +496,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",
@@ -519,9 +513,9 @@ function react(opts) {
       "@eslint-react/naming-convention/component-name": "error",
       "@eslint-react/naming-convention/use-state": "error",
       // recommended rules from @eslint-react/web-api
-      "@eslint-react/web-api/no-leaked-interval": "error",
-      "@eslint-react/web-api/no-leaked-timeout": "error",
-      "@eslint-react/web-api/no-leaked-event-listener": "error",
+      "@eslint-react/web-api/no-leaked-interval": "warn",
+      "@eslint-react/web-api/no-leaked-timeout": "warn",
+      "@eslint-react/web-api/no-leaked-event-listener": "warn",
       // 以下是自定义规则
       "react/jsx-handler-names": [
         "error",
@@ -531,8 +525,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 +537,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 +548,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 +851,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.0",
+  "version": "0.4.2",
   "description": "Agilebot's ESLint config",
   "bin": {
     "eslint-agilebot": "bin/eslint-agilebot"
@@ -44,14 +44,14 @@
     "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.0"
+    "@agilebot/eslint-utils": "0.4.2"
   },
   "devDependencies": {
     "eslint-config-love": "^63.0.0"
   },
   "peerDependencies": {
     "eslint": "^7.0.0 || ^8.0.0 || ^9.0.0",
-    "@agilebot/eslint-plugin": "0.4.0"
+    "@agilebot/eslint-plugin": "0.4.2"
   },
   "files": [
     "bin",