@coderwyd/eslint-config 2.4.1 → 2.4.2

package/dist/cli.cjs

@@ -46,18 +46,19 @@ var import_parse_gitignore = __toESM(require("parse-gitignore"), 1);
 var import_picocolors = __toESM(require("picocolors"), 1);
 
 // package.json
-var version = "2.4.1";
+var version = "2.4.2";
 var devDependencies = {
   "@antfu/ni": "^0.21.12",
+  "@eslint-react/eslint-plugin": "^1.5.12",
   "@eslint/config-inspector": "^0.4.8",
   "@types/eslint": "^8.56.10",
   "@types/fs-extra": "^11.0.4",
-  "@types/node": "^20.12.11",
+  "@types/node": "^20.12.12",
   "@types/prompts": "^2.4.9",
   "@types/yargs": "^17.0.32",
   "@unocss/eslint-plugin": "^0.60.2",
   bumpp: "^9.4.1",
-  eslint: "9.2.0",
+  eslint: "^9.3.0",
   "eslint-plugin-react": "^7.34.1",
   "eslint-plugin-react-hooks": "^4.6.2",
   "eslint-plugin-react-refresh": "^0.4.7",
@@ -71,7 +72,7 @@ var devDependencies = {
   svelte: "^4.2.17",
   "svelte-eslint-parser": "^0.36.0",
   tsup: "^8.0.2",
-  tsx: "^4.10.2",
+  tsx: "^4.10.5",
   typescript: "^5.4.5"
 };
 

package/dist/cli.js

@@ -17,18 +17,19 @@ import parse from "parse-gitignore";
 import c from "picocolors";
 
 // package.json
-var version = "2.4.1";
+var version = "2.4.2";
 var devDependencies = {
   "@antfu/ni": "^0.21.12",
+  "@eslint-react/eslint-plugin": "^1.5.12",
   "@eslint/config-inspector": "^0.4.8",
   "@types/eslint": "^8.56.10",
   "@types/fs-extra": "^11.0.4",
-  "@types/node": "^20.12.11",
+  "@types/node": "^20.12.12",
   "@types/prompts": "^2.4.9",
   "@types/yargs": "^17.0.32",
   "@unocss/eslint-plugin": "^0.60.2",
   bumpp: "^9.4.1",
-  eslint: "9.2.0",
+  eslint: "^9.3.0",
   "eslint-plugin-react": "^7.34.1",
   "eslint-plugin-react-hooks": "^4.6.2",
   "eslint-plugin-react-refresh": "^0.4.7",
@@ -42,7 +43,7 @@ var devDependencies = {
   svelte: "^4.2.17",
   "svelte-eslint-parser": "^0.36.0",
   tsup: "^8.0.2",
-  tsx: "^4.10.2",
+  tsx: "^4.10.5",
   typescript: "^5.4.5"
 };
 

package/dist/index.cjs

@@ -87,7 +87,6 @@ async function comments() {
 // src/constants/glob.ts
 var GLOB_SRC_EXT = "?([cm])[jt]s?(x)";
 var GLOB_SRC = "**/*.?([cm])[jt]s?(x)";
-var GLOB_JSX = "**/*.?([cm])jsx";
 var GLOB_TS = "**/*.?([cm])ts";
 var GLOB_TSX = "**/*.?([cm])tsx";
 var GLOB_VUE = "**/*.vue";
@@ -1438,86 +1437,132 @@ async function perfectionist() {
 // src/configs/react.ts
 var import_local_pkg2 = require("local-pkg");
 var ReactRefreshAllowConstantExportPackages = ["vite"];
+var RemixPackages = [
+  "@remix-run/node",
+  "@remix-run/react",
+  "@remix-run/serve",
+  "@remix-run/dev"
+];
+var NextJsPackages = ["next"];
 async function react(options = {}) {
-  const {
-    files = [GLOB_JSX, GLOB_TSX],
-    overrides = {},
-    typescript: typescript2 = true
-  } = options;
+  const { files = [GLOB_TS, GLOB_TSX], overrides = {} } = options;
   await ensurePackages([
-    "eslint-plugin-react",
+    "@eslint-react/eslint-plugin",
     "eslint-plugin-react-hooks",
     "eslint-plugin-react-refresh"
   ]);
-  const [pluginReact, pluginReactHooks, pluginReactRefresh] = await Promise.all(
-    [
-      interopDefault(import("eslint-plugin-react")),
-      interopDefault(import("eslint-plugin-react-hooks")),
-      interopDefault(import("eslint-plugin-react-refresh"))
-    ]
-  );
+  const tsconfigPath = options?.tsconfigPath ? toArray(options.tsconfigPath) : void 0;
+  const isTypeAware = !!tsconfigPath;
+  const [pluginReact, pluginReactHooks, pluginReactRefresh, parserTs] = await Promise.all([
+    interopDefault(import("@eslint-react/eslint-plugin")),
+    interopDefault(import("eslint-plugin-react-hooks")),
+    interopDefault(import("eslint-plugin-react-refresh")),
+    interopDefault(import("@typescript-eslint/parser"))
+  ]);
   const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some(
     (i) => (0, import_local_pkg2.isPackageExists)(i)
   );
+  const isUsingRemix = RemixPackages.some((i) => (0, import_local_pkg2.isPackageExists)(i));
+  const isUsingNext = NextJsPackages.some((i) => (0, import_local_pkg2.isPackageExists)(i));
+  const plugins = pluginReact.configs.all.plugins;
   return [
     {
       name: "coderwyd/react/setup",
       plugins: {
-        react: pluginReact,
+        react: plugins["@eslint-react"],
+        "react-dom": plugins["@eslint-react/dom"],
         "react-hooks": pluginReactHooks,
+        "react-hooks-extra": plugins["@eslint-react/hooks-extra"],
+        "react-naming-convention": plugins["@eslint-react/naming-convention"],
         "react-refresh": pluginReactRefresh
-      },
-      settings: {
-        react: {
-          version: "detect"
-        }
       }
     },
     {
       files,
       languageOptions: {
+        parser: parserTs,
         parserOptions: {
           ecmaFeatures: {
             jsx: true
-          }
-        }
+          },
+          ...isTypeAware ? { project: tsconfigPath } : {}
+        },
+        sourceType: "module"
       },
       name: "coderwyd/react/rules",
       rules: {
+        // recommended rules from @eslint-react/dom
+        "react-dom/no-children-in-void-dom-elements": "warn",
+        "react-dom/no-dangerously-set-innerhtml": "warn",
+        "react-dom/no-dangerously-set-innerhtml-with-children": "error",
+        "react-dom/no-find-dom-node": "error",
+        "react-dom/no-missing-button-type": "warn",
+        "react-dom/no-missing-iframe-sandbox": "warn",
+        "react-dom/no-namespace": "error",
+        "react-dom/no-render-return-value": "error",
+        "react-dom/no-script-url": "warn",
+        "react-dom/no-unsafe-iframe-sandbox": "warn",
+        "react-dom/no-unsafe-target-blank": "warn",
         // recommended rules react-hooks
         "react-hooks/exhaustive-deps": "warn",
         "react-hooks/rules-of-hooks": "error",
         // react refresh
         "react-refresh/only-export-components": [
           "warn",
-          { allowConstantExport: isAllowConstantExport }
+          {
+            allowConstantExport: isAllowConstantExport,
+            allowExportNames: [
+              ...isUsingNext ? [
+                "config",
+                "generateStaticParams",
+                "metadata",
+                "generateMetadata",
+                "viewport",
+                "generateViewport"
+              ] : [],
+              ...isUsingRemix ? ["meta", "links", "headers", "loader", "action"] : []
+            ]
+          }
         ],
-        // recommended rules react
-        "react/display-name": "error",
-        "react/jsx-key": "error",
-        "react/jsx-no-comment-textnodes": "error",
-        "react/jsx-no-duplicate-props": "error",
-        "react/jsx-no-target-blank": "error",
-        "react/jsx-no-undef": "error",
-        "react/jsx-uses-react": "error",
-        "react/jsx-uses-vars": "error",
-        "react/no-children-prop": "error",
-        "react/no-danger-with-children": "error",
-        "react/no-deprecated": "error",
+        // recommended rules from @eslint-react
+        "react/ensure-forward-ref-using-ref": "warn",
+        "react/no-access-state-in-setstate": "error",
+        "react/no-array-index-key": "warn",
+        "react/no-children-count": "warn",
+        "react/no-children-for-each": "warn",
+        "react/no-children-map": "warn",
+        "react/no-children-only": "warn",
+        "react/no-children-prop": "warn",
+        "react/no-children-to-array": "warn",
+        "react/no-clone-element": "warn",
+        "react/no-comment-textnodes": "warn",
+        "react/no-component-will-mount": "error",
+        "react/no-component-will-receive-props": "error",
+        "react/no-component-will-update": "error",
+        "react/no-create-ref": "error",
         "react/no-direct-mutation-state": "error",
-        "react/no-find-dom-node": "error",
-        "react/no-is-mounted": "error",
-        "react/no-render-return-value": "error",
+        "react/no-duplicate-key": "error",
+        "react/no-implicit-key": "error",
+        "react/no-missing-key": "error",
+        "react/no-nested-components": "warn",
+        "react/no-redundant-should-component-update": "error",
+        "react/no-set-state-in-component-did-mount": "warn",
+        "react/no-set-state-in-component-did-update": "warn",
+        "react/no-set-state-in-component-will-update": "warn",
         "react/no-string-refs": "error",
-        "react/no-unescaped-entities": "error",
-        "react/no-unknown-property": "error",
-        "react/no-unsafe": "off",
-        "react/prop-types": "error",
-        "react/react-in-jsx-scope": "off",
-        "react/require-render-return": "error",
-        ...typescript2 ? {
-          "react/jsx-no-undef": "off",
-          "react/prop-type": "off"
+        "react/no-unsafe-component-will-mount": "warn",
+        "react/no-unsafe-component-will-receive-props": "warn",
+        "react/no-unsafe-component-will-update": "warn",
+        "react/no-unstable-context-value": "error",
+        "react/no-unstable-default-props": "error",
+        "react/no-unused-class-component-members": "warn",
+        "react/no-unused-state": "warn",
+        "react/no-useless-fragment": "warn",
+        "react/prefer-destructuring-assignment": "warn",
+        "react/prefer-shorthand-boolean": "warn",
+        "react/prefer-shorthand-fragment": "warn",
+        ...isTypeAware ? {
+          "react/no-leaked-conditional-rendering": "warn"
         } : {},
         // overrides
         ...overrides
@@ -1904,7 +1949,7 @@ async function defineConfig(options = {}, ...userConfigs) {
     configs2.push(
       react({
         overrides: getOverrides(options, "react"),
-        typescript: !!enableTypeScript
+        tsconfigPath: getOverrides(options, "typescript").tsconfigPath
       })
     );
   }