@bfra.me/eslint-config 0.50.2 → 0.51.0

package/lib/index.js

@@ -1,7 +1,7 @@
 import "./chunk-EGRHWZRV.js";
 
 // src/define-config.ts
-import { isPackageExists as isPackageExists4 } from "local-pkg";
+import { isPackageExists as isPackageExists3 } from "local-pkg";
 
 // ../es/src/env/ci.ts
 import isCI from "is-in-ci";
@@ -147,7 +147,7 @@ var GLOB_EXCLUDE = [
 import { fileURLToPath } from "url";
 
 // package.json
-var version = "0.50.2";
+var version = "0.51.0";
 
 // src/parsers/any-parser.ts
 var lineBreakPattern = /\r\n|[\n\r\u2028\u2029]/u;
@@ -1287,7 +1287,6 @@ async function pnpm() {
 
 // src/configs/prettier.ts
 import process4 from "process";
-import { isPackageExists as isPackageExists2 } from "local-pkg";
 function getConfigRules(configs) {
   if (Array.isArray(configs)) {
     const configWithRules = [...configs].reverse().find(
@@ -1309,8 +1308,8 @@ async function prettier(options = {}) {
       const [configPrettier, pluginPrettier, pluginJsonc, pluginYaml] = await Promise.all([
         interopDefault(import("eslint-config-prettier")),
         interopDefault(import("eslint-plugin-prettier")),
-        isPackageExists2("eslint-plugin-jsonc") ? interopDefault(import("eslint-plugin-jsonc")) : Promise.resolve(void 0),
-        isPackageExists2("eslint-plugin-yml") ? interopDefault(import("eslint-plugin-yml")) : Promise.resolve(void 0)
+        interopDefault(import("eslint-plugin-jsonc")).catch(() => void 0),
+        interopDefault(import("eslint-plugin-yml")).catch(() => void 0)
       ]);
       const jsoncPrettierRules = getConfigRules(pluginJsonc?.configs.prettier);
       const yamlPrettierRules = getConfigRules(pluginYaml?.configs.prettier);
@@ -1371,7 +1370,7 @@ async function prettier(options = {}) {
 }
 
 // src/configs/react.ts
-import { isPackageExists as isPackageExists3 } from "local-pkg";
+import { isPackageExists as isPackageExists2 } from "local-pkg";
 var ReactRefreshAllowConstantExportPackages = ["vite"];
 var RemixPackages = ["@remix-run/node", "@remix-run/react", "@remix-run/serve", "@remix-run/dev"];
 var ReactRouterPackages = [
@@ -1382,8 +1381,8 @@ var ReactRouterPackages = [
 ];
 var NextJsPackages = ["next"];
 var ReactTypeAwareRules = {
-  "react/no-implicit-key": "warn",
-  "react/no-leaked-conditional-rendering": "warn"
+  "@eslint-react/no-implicit-key": "warn",
+  "@eslint-react/no-leaked-conditional-rendering": "warn"
 };
 async function react(options = {}) {
   const {
@@ -1397,31 +1396,25 @@ async function react(options = {}) {
   } = options;
   const isTypeAware = typeof tsconfigPath === "string" && tsconfigPath.trim().length > 0;
   return requireOf(
-    ["@eslint-react/eslint-plugin", "eslint-plugin-react-hooks", "eslint-plugin-react-refresh"],
+    ["@eslint-react/eslint-plugin", "eslint-plugin-react-refresh"],
     async () => {
-      const [pluginReact, pluginReactHooks, pluginReactRefresh] = await Promise.all([
+      const [pluginReact, pluginReactRefresh] = await Promise.all([
         interopDefault(import("@eslint-react/eslint-plugin")),
-        interopDefault(import("eslint-plugin-react-hooks")),
         import("eslint-plugin-react-refresh").then((m) => m.reactRefresh)
       ]);
       const plugins = pluginReact.configs.all.plugins;
       const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some(
-        (i) => isPackageExists3(i)
+        (i) => isPackageExists2(i)
       );
-      const isUsingRemix = RemixPackages.some((i) => isPackageExists3(i));
-      const isUsingReactRouter = ReactRouterPackages.some((i) => isPackageExists3(i));
-      const isUsingNext = NextJsPackages.some((i) => isPackageExists3(i));
+      const isUsingRemix = RemixPackages.some((i) => isPackageExists2(i));
+      const isUsingReactRouter = ReactRouterPackages.some((i) => isPackageExists2(i));
+      const isUsingNext = NextJsPackages.some((i) => isPackageExists2(i));
       return [
         {
           name: "@bfra.me/react/setup",
           plugins: {
-            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.plugin,
-            "react-web-api": plugins["@eslint-react/web-api"]
+            ...plugins,
+            "react-refresh": pluginReactRefresh.plugin
           }
         },
         {
@@ -1436,26 +1429,7 @@ async function react(options = {}) {
             sourceType: "module"
           },
           rules: {
-            // recommended rules from eslint-plugin-react-dom https://eslint-react.xyz/docs/rules/overview#dom-rules
-            "react-dom/no-namespace": "error",
-            "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-flush-sync": "error",
-            "react-dom/no-hydrate": "error",
-            "react-dom/no-missing-button-type": "warn",
-            "react-dom/no-missing-iframe-sandbox": "warn",
-            "react-dom/no-render": "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",
-            "react-dom/no-use-form-state": "error",
-            "react-dom/no-void-elements-with-children": "error",
-            // recommended rules from eslint-plugin-react-hooks-extra https://eslint-react.xyz/docs/rules/overview#hooks-extra-rules
-            "react-hooks-extra/no-direct-set-state-in-use-effect": "warn",
-            // recommended rules eslint-plugin-react-hooks https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks/src/rules
-            ...pluginReactHooks.configs.recommended.rules,
+            ...pluginReact.configs.recommended.rules,
             // preconfigured rules from eslint-plugin-react-refresh https://github.com/ArnaudBarre/eslint-plugin-react-refresh/tree/main/src
             "react-refresh/only-export-components": [
               "warn",
@@ -1491,51 +1465,6 @@ async function react(options = {}) {
                 ]
               }
             ],
-            // recommended rules from eslint-plugin-react-web-api https://eslint-react.xyz/docs/rules/overview#web-api-rules
-            "react-web-api/no-leaked-event-listener": "warn",
-            "react-web-api/no-leaked-interval": "warn",
-            "react-web-api/no-leaked-resize-observer": "warn",
-            "react-web-api/no-leaked-timeout": "warn",
-            // recommended rules from eslint-plugin-react-x https://eslint-react.xyz/docs/rules/overview#core-rules
-            "react/jsx-no-comment-textnodes": "warn",
-            "react/jsx-no-duplicate-props": "warn",
-            "react/jsx-uses-vars": "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-to-array": "warn",
-            "react/no-clone-element": "warn",
-            "react/no-component-will-mount": "error",
-            "react/no-component-will-receive-props": "error",
-            "react/no-component-will-update": "error",
-            "react/no-context-provider": "warn",
-            "react/no-create-ref": "error",
-            "react/no-default-props": "error",
-            "react/no-direct-mutation-state": "error",
-            "react/no-duplicate-key": "warn",
-            "react/no-forward-ref": "warn",
-            "react/no-missing-key": "error",
-            "react/no-nested-component-definitions": "error",
-            "react/no-prop-types": "error",
-            "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-unnecessary-use-prefix": "warn",
-            "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": "warn",
-            "react/no-unstable-default-props": "warn",
-            "react/no-unused-class-component-members": "warn",
-            "react/no-unused-state": "warn",
-            "react/no-use-context": "warn",
-            "react/no-useless-forward-ref": "warn",
-            "react/prefer-use-state-lazy-initialization": "warn",
             ...overrides
           }
         },
@@ -2306,10 +2235,10 @@ async function defineConfig(options = {}, ...userConfigs) {
     packageJson: enablePackageJson = false,
     perfectionist: enablePerfectionist = true,
     pnpm: enableCatalogs = false,
-    prettier: enablePrettier = isPackageExists4("prettier"),
+    prettier: enablePrettier = isPackageExists3("prettier"),
     react: enableReact = false,
     regexp: enableRegexp = true,
-    typescript: enableTypeScript = isPackageExists4("typescript"),
+    typescript: enableTypeScript = isPackageExists3("typescript"),
     unicorn: enableUnicorn = true
   } = options;
   let isInEditor = options.isInEditor;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bfra.me/eslint-config",
-  "version": "0.50.2",
+  "version": "0.51.0",
   "description": "Shared ESLint configuration for bfra.me",
   "keywords": [
     "bfra.me",
@@ -38,7 +38,7 @@
   ],
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "4.7.1",
-    "@eslint/markdown": "7.5.1",
+    "@eslint/markdown": "8.0.1",
     "@stylistic/eslint-plugin": "5.10.0",
     "eslint-config-flat-gitignore": "2.3.0",
     "eslint-flat-config-utils": "3.1.0",
@@ -52,30 +52,30 @@
     "eslint-plugin-perfectionist": "5.8.0",
     "eslint-plugin-regexp": "3.1.0",
     "eslint-plugin-toml": "1.3.1",
-    "eslint-plugin-unicorn": "63.0.0",
+    "eslint-plugin-unicorn": "64.0.0",
     "eslint-plugin-unused-imports": "4.4.1",
     "eslint-plugin-yml": "3.3.1",
-    "globals": "17.4.0",
+    "globals": "17.5.0",
     "is-in-ci": "2.0.0",
     "local-pkg": "1.1.2",
     "package-manager-detector": "1.6.0",
     "sort-package-json": "3.6.1",
-    "typescript-eslint": "8.58.0",
+    "typescript-eslint": "8.58.1",
     "@bfra.me/es": "0.1.0"
   },
   "devDependencies": {
-    "@eslint-react/eslint-plugin": "2.13.0",
+    "@eslint-react/eslint-plugin": "4.2.3",
     "@eslint/config-inspector": "1.5.0",
-    "@eslint/core": "1.1.1",
-    "@next/eslint-plugin-next": "16.2.2",
+    "@eslint/core": "1.2.1",
+    "@next/eslint-plugin-next": "16.2.3",
     "@types/eslint-config-prettier": "6.11.3",
     "@types/eslint-plugin-jsx-a11y": "6.10.1",
-    "@typescript-eslint/types": "8.58.0",
-    "@vitest/eslint-plugin": "1.6.14",
+    "@typescript-eslint/types": "8.58.1",
+    "@vitest/eslint-plugin": "1.6.15",
     "astro-eslint-parser": "1.4.0",
-    "eslint": "10.1.0",
+    "eslint": "10.2.0",
     "eslint-config-prettier": "10.1.8",
-    "eslint-plugin-astro": "1.6.0",
+    "eslint-plugin-astro": "1.7.0",
     "eslint-plugin-erasable-syntax-only": "0.4.0",
     "eslint-plugin-jsx-a11y": "6.10.2",
     "eslint-plugin-node-dependencies": "2.2.0",
@@ -84,12 +84,12 @@
     "eslint-plugin-react-hooks": "7.0.1",
     "eslint-plugin-react-refresh": "0.5.2",
     "eslint-typegen": "2.3.1",
-    "@bfra.me/prettier-config": "0.16.7",
-    "@bfra.me/tsconfig": "0.12.2",
+    "@bfra.me/tsconfig": "0.13.0",
+    "@bfra.me/prettier-config": "0.16.8",
     "@bfra.me/works": "0.0.0-development"
   },
   "peerDependencies": {
-    "@eslint-react/eslint-plugin": "^2.2.3",
+    "@eslint-react/eslint-plugin": "^4.2.3",
     "@next/eslint-plugin-next": ">=15.5.3",
     "@vitest/eslint-plugin": "^1.1.21",
     "astro-eslint-parser": "^1.2.2",

package/src/configs/prettier.ts

@@ -1,7 +1,6 @@
 import type {Config} from '../config'
 import type {Flatten, OptionsIsInEditor, OptionsOverrides} from '../options'
 import process from 'node:process'
-import {isPackageExists} from 'local-pkg'
 import {GLOB_EXT_IN_MARKDOWN_FILES, GLOB_MARKDOWN_FILES, GLOB_TOML_FILES} from '../globs'
 import {requireOf} from '../require-of'
 import {interopDefault} from '../utils'
@@ -46,12 +45,8 @@ export async function prettier(options: PrettierOptions = {}): Promise<Config[]>
       const [configPrettier, pluginPrettier, pluginJsonc, pluginYaml] = await Promise.all([
         interopDefault(import('eslint-config-prettier')),
         interopDefault(import('eslint-plugin-prettier')),
-        isPackageExists('eslint-plugin-jsonc')
-          ? interopDefault(import('eslint-plugin-jsonc'))
-          : Promise.resolve(undefined),
-        isPackageExists('eslint-plugin-yml')
-          ? interopDefault(import('eslint-plugin-yml'))
-          : Promise.resolve(undefined),
+        interopDefault(import('eslint-plugin-jsonc')).catch(() => undefined),
+        interopDefault(import('eslint-plugin-yml')).catch(() => undefined),
       ])
       const jsoncPrettierRules = getConfigRules(pluginJsonc?.configs.prettier)
       const yamlPrettierRules = getConfigRules(pluginYaml?.configs.prettier)

package/src/configs/react.ts

@@ -1,4 +1,3 @@
-import type {Plugin} from '@eslint/core'
 import type {Config} from '../config'
 import type {
   Flatten,
@@ -25,8 +24,8 @@ const ReactRouterPackages = [
 const NextJsPackages = ['next']
 
 const ReactTypeAwareRules: Config['rules'] = {
-  'react/no-implicit-key': 'warn',
-  'react/no-leaked-conditional-rendering': 'warn',
+  '@eslint-react/no-implicit-key': 'warn',
+  '@eslint-react/no-leaked-conditional-rendering': 'warn',
 }
 
 /**
@@ -77,15 +76,14 @@ export async function react(options: ReactOptions = {}): Promise<Config[]> {
   const isTypeAware = typeof tsconfigPath === 'string' && tsconfigPath.trim().length > 0
 
   return requireOf(
-    ['@eslint-react/eslint-plugin', 'eslint-plugin-react-hooks', 'eslint-plugin-react-refresh'],
+    ['@eslint-react/eslint-plugin', 'eslint-plugin-react-refresh'],
     async () => {
-      const [pluginReact, pluginReactHooks, pluginReactRefresh] = await Promise.all([
+      const [pluginReact, pluginReactRefresh] = await Promise.all([
         interopDefault(import('@eslint-react/eslint-plugin')),
-        interopDefault(import('eslint-plugin-react-hooks')),
         import('eslint-plugin-react-refresh').then(m => m.reactRefresh),
       ] as const)
 
-      const plugins = (pluginReact.configs.all as {plugins: Record<string, Plugin>}).plugins
+      const plugins = pluginReact.configs.all.plugins as NonNullable<Config['plugins']>
       const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some(i =>
         isPackageExists(i),
       )
@@ -97,13 +95,8 @@ export async function react(options: ReactOptions = {}): Promise<Config[]> {
         {
           name: '@bfra.me/react/setup',
           plugins: {
-            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'],
+            ...plugins,
             'react-refresh': pluginReactRefresh.plugin,
-            'react-web-api': plugins['@eslint-react/web-api'],
           } as Config['plugins'],
         },
         {
@@ -118,28 +111,7 @@ export async function react(options: ReactOptions = {}): Promise<Config[]> {
             sourceType: 'module',
           },
           rules: {
-            // recommended rules from eslint-plugin-react-dom https://eslint-react.xyz/docs/rules/overview#dom-rules
-            'react-dom/no-namespace': 'error',
-            '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-flush-sync': 'error',
-            'react-dom/no-hydrate': 'error',
-            'react-dom/no-missing-button-type': 'warn',
-            'react-dom/no-missing-iframe-sandbox': 'warn',
-            'react-dom/no-render': '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',
-            'react-dom/no-use-form-state': 'error',
-            'react-dom/no-void-elements-with-children': 'error',
-
-            // recommended rules from eslint-plugin-react-hooks-extra https://eslint-react.xyz/docs/rules/overview#hooks-extra-rules
-            'react-hooks-extra/no-direct-set-state-in-use-effect': 'warn',
-
-            // recommended rules eslint-plugin-react-hooks https://github.com/facebook/react/tree/main/packages/eslint-plugin-react-hooks/src/rules
-            ...pluginReactHooks.configs.recommended.rules,
+            ...pluginReact.configs.recommended.rules,
 
             // preconfigured rules from eslint-plugin-react-refresh https://github.com/ArnaudBarre/eslint-plugin-react-refresh/tree/main/src
             'react-refresh/only-export-components': [
@@ -181,53 +153,6 @@ export async function react(options: ReactOptions = {}): Promise<Config[]> {
               },
             ],
 
-            // recommended rules from eslint-plugin-react-web-api https://eslint-react.xyz/docs/rules/overview#web-api-rules
-            'react-web-api/no-leaked-event-listener': 'warn',
-            'react-web-api/no-leaked-interval': 'warn',
-            'react-web-api/no-leaked-resize-observer': 'warn',
-            'react-web-api/no-leaked-timeout': 'warn',
-
-            // recommended rules from eslint-plugin-react-x https://eslint-react.xyz/docs/rules/overview#core-rules
-            'react/jsx-no-comment-textnodes': 'warn',
-            'react/jsx-no-duplicate-props': 'warn',
-            'react/jsx-uses-vars': '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-to-array': 'warn',
-            'react/no-clone-element': 'warn',
-            'react/no-component-will-mount': 'error',
-            'react/no-component-will-receive-props': 'error',
-            'react/no-component-will-update': 'error',
-            'react/no-context-provider': 'warn',
-            'react/no-create-ref': 'error',
-            'react/no-default-props': 'error',
-            'react/no-direct-mutation-state': 'error',
-            'react/no-duplicate-key': 'warn',
-            'react/no-forward-ref': 'warn',
-            'react/no-missing-key': 'error',
-            'react/no-nested-component-definitions': 'error',
-            'react/no-prop-types': 'error',
-            '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-unnecessary-use-prefix': 'warn',
-            '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': 'warn',
-            'react/no-unstable-default-props': 'warn',
-            'react/no-unused-class-component-members': 'warn',
-            'react/no-unused-state': 'warn',
-            'react/no-use-context': 'warn',
-            'react/no-useless-forward-ref': 'warn',
-            'react/prefer-use-state-lazy-initialization': 'warn',
-
             ...overrides,
           },
         },