@infernodesign/eslint-config 1.25.0 → 1.26.0

package/README.md

@@ -306,7 +306,6 @@ Since flat config requires us to explicitly provide the plugin names (instead of
 | `node`                    | `n`                               | [eslint-plugin-n](https://github.com/eslint-community/eslint-plugin-n)                                |
 | `react`                   | `@eslint-react`                   | [@eslint-react/eslint-plugin](https://github.com/eslint-react/eslint-plugin)                          |
 | `react-dom`               | `@eslint-react/dom`               | [@eslint-react/eslint-plugin](https://github.com/eslint-react/eslint-plugin)                          |
-| `react-hooks-extra`       | `@eslint-react/hooks-extra`       | [@eslint-react/eslint-plugin](https://github.com/eslint-react/eslint-plugin)                          |
 | `react-naming-convention` | `@eslint-react/naming-convention` | [@eslint-react/eslint-plugin](https://github.com/eslint-react/eslint-plugin)                          |
 | `style`                   | `@stylistic`                      | [@stylistic/eslint-plugin](https://github.com/eslint-stylistic/eslint-stylistic)                      |
 | `tailwindcss/*`           | `better-tailwindcss/*`            | [eslint-plugin-better-tailwindcss](https://github.com/unjs/eslint-plugin-better-tailwindcss)          |
@@ -496,7 +495,7 @@ export default config( {
 Running `npx eslint` should prompt you to install the required dependencies, otherwise, you can install them manually:
 
 ```bash
-npm i -D @eslint-react/eslint-plugin eslint-plugin-react-hooks eslint-plugin-react-refresh
+npm i -D @eslint-react/eslint-plugin eslint-plugin-react-refresh
 ```
 
 #### Next.js

package/dist/cli.js

@@ -8,7 +8,7 @@ import { cac } from "cac";
 import parse from "parse-gitignore";
 import { execSync } from "node:child_process";
 //#region package.json
-var version = "1.25.0";
+var version = "1.26.0";
 //#endregion
 //#region src/cli/constants.ts
 const vscodeSettingsString = `
@@ -122,11 +122,7 @@ const dependenciesMap = {
 	formatterAstro: ["prettier-plugin-astro"],
 	nextjs: ["@next/eslint-plugin-next"],
 	node: [],
-	react: [
-		"@eslint-react/eslint-plugin",
-		"eslint-plugin-react-hooks",
-		"eslint-plugin-react-refresh"
-	],
+	react: ["@eslint-react/eslint-plugin", "eslint-plugin-react-refresh"],
 	slidev: ["prettier-plugin-slidev"],
 	solid: ["eslint-plugin-solid"],
 	storybook: ["eslint-plugin-storybook"],
@@ -198,7 +194,6 @@ const versionsMap = {
 	"eslint-plugin-astro": "^1.6.0",
 	"eslint-plugin-better-tailwindcss": "^4.3.2",
 	"eslint-plugin-format": "^2.0.1",
-	"eslint-plugin-react-hooks": "^7.0.1",
 	"eslint-plugin-react-refresh": "^0.5.2",
 	"eslint-plugin-solid": "^0.14.5",
 	"eslint-plugin-storybook": "^10.3.1",

package/dist/index.d.ts

@@ -3377,124 +3377,6 @@ interface RuleOptions {
    * @see https://eslint-react.xyz/docs/rules/dom-prefer-namespace-import
    */
   'react-dom/prefer-namespace-import'?: Linter.RuleEntry<[]>;
-  /**
-   * Verifies that automatic effect dependencies are compiled if opted-in
-   */
-  'react-hooks/automatic-effect-dependencies'?: Linter.RuleEntry<ReactHooksAutomaticEffectDependencies>;
-  /**
-   * Validates against calling capitalized functions/methods instead of using JSX
-   */
-  'react-hooks/capitalized-calls'?: Linter.RuleEntry<ReactHooksCapitalizedCalls>;
-  /**
-   * Validates against higher order functions defining nested components or hooks. Components and hooks should be defined at the module level
-   */
-  'react-hooks/component-hook-factories'?: Linter.RuleEntry<ReactHooksComponentHookFactories>;
-  /**
-   * Validates the compiler configuration options
-   */
-  'react-hooks/config'?: Linter.RuleEntry<ReactHooksConfig>;
-  /**
-   * Validates usage of error boundaries instead of try/catch for errors in child components
-   */
-  'react-hooks/error-boundaries'?: Linter.RuleEntry<ReactHooksErrorBoundaries>;
-  /**
-   * verifies the list of dependencies for Hooks like useEffect and similar
-   * @see https://github.com/facebook/react/issues/14920
-   */
-  'react-hooks/exhaustive-deps'?: Linter.RuleEntry<ReactHooksExhaustiveDeps>;
-  /**
-   * Validates usage of fbt
-   */
-  'react-hooks/fbt'?: Linter.RuleEntry<ReactHooksFbt>;
-  /**
-   * Validates usage of `fire`
-   */
-  'react-hooks/fire'?: Linter.RuleEntry<ReactHooksFire>;
-  /**
-   * Validates configuration of [gating mode](https://react.dev/reference/react-compiler/gating)
-   */
-  'react-hooks/gating'?: Linter.RuleEntry<ReactHooksGating>;
-  /**
-   * Validates against assignment/mutation of globals during render, part of ensuring that [side effects must render outside of render](https://react.dev/reference/rules/components-and-hooks-must-be-pure#side-effects-must-run-outside-of-render)
-   */
-  'react-hooks/globals'?: Linter.RuleEntry<ReactHooksGlobals>;
-  /**
-   * Validates the rules of hooks
-   */
-  'react-hooks/hooks'?: Linter.RuleEntry<ReactHooksHooks>;
-  /**
-   * Validates against mutating props, state, and other values that [are immutable](https://react.dev/reference/rules/components-and-hooks-must-be-pure#props-and-state-are-immutable)
-   */
-  'react-hooks/immutability'?: Linter.RuleEntry<ReactHooksImmutability>;
-  /**
-   * Validates against usage of libraries which are incompatible with memoization (manual or automatic)
-   */
-  'react-hooks/incompatible-library'?: Linter.RuleEntry<ReactHooksIncompatibleLibrary>;
-  /**
-   * Internal invariants
-   */
-  'react-hooks/invariant'?: Linter.RuleEntry<ReactHooksInvariant>;
-  /**
-   * Validates that effect dependencies are memoized
-   */
-  'react-hooks/memoized-effect-dependencies'?: Linter.RuleEntry<ReactHooksMemoizedEffectDependencies>;
-  /**
-   * Validates against deriving values from state in an effect
-   */
-  'react-hooks/no-deriving-state-in-effects'?: Linter.RuleEntry<ReactHooksNoDerivingStateInEffects>;
-  /**
-   * Validates that existing manual memoized is preserved by the compiler. React Compiler will only compile components and hooks if its inference [matches or exceeds the existing manual memoization](https://react.dev/learn/react-compiler/introduction#what-should-i-do-about-usememo-usecallback-and-reactmemo)
-   */
-  'react-hooks/preserve-manual-memoization'?: Linter.RuleEntry<ReactHooksPreserveManualMemoization>;
-  /**
-   * Validates that [components/hooks are pure](https://react.dev/reference/rules/components-and-hooks-must-be-pure) by checking that they do not call known-impure functions
-   */
-  'react-hooks/purity'?: Linter.RuleEntry<ReactHooksPurity>;
-  /**
-   * Validates correct usage of refs, not reading/writing during render. See the "pitfalls" section in [`useRef()` usage](https://react.dev/reference/react/useRef#usage)
-   */
-  'react-hooks/refs'?: Linter.RuleEntry<ReactHooksRefs>;
-  /**
-   * Validates against suppression of other rules
-   */
-  'react-hooks/rule-suppression'?: Linter.RuleEntry<ReactHooksRuleSuppression>;
-  /**
-   * enforces the Rules of Hooks
-   * @see https://react.dev/reference/rules/rules-of-hooks
-   */
-  'react-hooks/rules-of-hooks'?: Linter.RuleEntry<ReactHooksRulesOfHooks>;
-  /**
-   * Validates against calling setState synchronously in an effect, which can lead to re-renders that degrade performance
-   */
-  'react-hooks/set-state-in-effect'?: Linter.RuleEntry<ReactHooksSetStateInEffect>;
-  /**
-   * Validates against setting state during render, which can trigger additional renders and potential infinite render loops
-   */
-  'react-hooks/set-state-in-render'?: Linter.RuleEntry<ReactHooksSetStateInRender>;
-  /**
-   * Validates that components are static, not recreated every render. Components that are recreated dynamically can reset state and trigger excessive re-rendering
-   */
-  'react-hooks/static-components'?: Linter.RuleEntry<ReactHooksStaticComponents>;
-  /**
-   * Validates against invalid syntax
-   */
-  'react-hooks/syntax'?: Linter.RuleEntry<ReactHooksSyntax>;
-  /**
-   * Unimplemented features
-   */
-  'react-hooks/todo'?: Linter.RuleEntry<ReactHooksTodo>;
-  /**
-   * Validates against syntax that we do not plan to support in React Compiler
-   */
-  'react-hooks/unsupported-syntax'?: Linter.RuleEntry<ReactHooksUnsupportedSyntax>;
-  /**
-   * Validates usage of the useMemo() hook against common mistakes. See [`useMemo()` docs](https://react.dev/reference/react/useMemo) for more information.
-   */
-  'react-hooks/use-memo'?: Linter.RuleEntry<ReactHooksUseMemo>;
-  /**
-   * Validates that useMemos always return a value and that the result of the useMemo is used by the component/hook. See [`useMemo()` docs](https://react.dev/reference/react/useMemo) for more information.
-   */
-  'react-hooks/void-use-memo'?: Linter.RuleEntry<ReactHooksVoidUseMemo>;
   /**
    * Enforces the context name to be a valid component name with the suffix 'Context'.
    * @see https://eslint-react.xyz/docs/rules/naming-convention-context-name
@@ -14134,96 +14016,6 @@ type Radix = [] | [("always" | "as-needed")]; // ----- react-dom/no-unknown-prop
 type ReactDomNoUnknownProperty = [] | [{
   ignore?: string[];
   requireDataLowercase?: boolean;
-}]; // ----- react-hooks/automatic-effect-dependencies -----
-type ReactHooksAutomaticEffectDependencies = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/capitalized-calls -----
-type ReactHooksCapitalizedCalls = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/component-hook-factories -----
-type ReactHooksComponentHookFactories = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/config -----
-type ReactHooksConfig = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/error-boundaries -----
-type ReactHooksErrorBoundaries = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/exhaustive-deps -----
-type ReactHooksExhaustiveDeps = [] | [{
-  additionalHooks?: string;
-  enableDangerousAutofixThisMayCauseInfiniteLoops?: boolean;
-  experimental_autoDependenciesHooks?: string[];
-  requireExplicitEffectDeps?: boolean;
-}]; // ----- react-hooks/fbt -----
-type ReactHooksFbt = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/fire -----
-type ReactHooksFire = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/gating -----
-type ReactHooksGating = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/globals -----
-type ReactHooksGlobals = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/hooks -----
-type ReactHooksHooks = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/immutability -----
-type ReactHooksImmutability = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/incompatible-library -----
-type ReactHooksIncompatibleLibrary = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/invariant -----
-type ReactHooksInvariant = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/memoized-effect-dependencies -----
-type ReactHooksMemoizedEffectDependencies = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/no-deriving-state-in-effects -----
-type ReactHooksNoDerivingStateInEffects = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/preserve-manual-memoization -----
-type ReactHooksPreserveManualMemoization = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/purity -----
-type ReactHooksPurity = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/refs -----
-type ReactHooksRefs = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/rule-suppression -----
-type ReactHooksRuleSuppression = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/rules-of-hooks -----
-type ReactHooksRulesOfHooks = [] | [{
-  additionalHooks?: string;
-}]; // ----- react-hooks/set-state-in-effect -----
-type ReactHooksSetStateInEffect = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/set-state-in-render -----
-type ReactHooksSetStateInRender = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/static-components -----
-type ReactHooksStaticComponents = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/syntax -----
-type ReactHooksSyntax = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/todo -----
-type ReactHooksTodo = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/unsupported-syntax -----
-type ReactHooksUnsupportedSyntax = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/use-memo -----
-type ReactHooksUseMemo = [] | [{
-  [k: string]: unknown | undefined;
-}]; // ----- react-hooks/void-use-memo -----
-type ReactHooksVoidUseMemo = [] | [{
-  [k: string]: unknown | undefined;
 }]; // ----- react-refresh/only-export-components -----
 type ReactRefreshOnlyExportComponents = [] | [{
   extraHOCs?: string[];
@@ -18722,7 +18514,6 @@ type TypedFlatConfigItem = Omit<ConfigWithExtends, 'plugins' | 'rules'> & {
 declare const defaultPluginRenaming: {
   '@eslint-react': string;
   '@eslint-react/dom': string;
-  '@eslint-react/hooks-extra': string;
   '@eslint-react/naming-convention': string;
   '@next/next': string;
   '@stylistic': string;

package/dist/index.js

@@ -1753,35 +1753,24 @@ const NextJsPackages = ["next"];
 const ReactCompilerPackages = ["babel-plugin-react-compiler"];
 async function react(options = {}) {
 	const { files = [GLOB_SRC], filesTypeAware = [GLOB_TS, GLOB_TSX], ignoresTypeAware = [`${GLOB_MARKDOWN}/**`, GLOB_ASTRO_TS], overrides = {}, reactCompiler = ReactCompilerPackages.some((i) => isPackageExists(i)), tsconfigPath } = options;
-	await ensurePackages([
-		"@eslint-react/eslint-plugin",
-		"eslint-plugin-react-hooks",
-		"eslint-plugin-react-refresh"
-	]);
+	await ensurePackages(["@eslint-react/eslint-plugin", "eslint-plugin-react-refresh"]);
 	const isTypeAware = !!tsconfigPath;
 	const typeAwareRules = {
 		"react/no-implicit-key": "error",
 		"react/no-leaked-conditional-rendering": "warn"
 	};
-	const [pluginReact, pluginReactHooks, pluginReactRefresh] = await Promise.all([
-		interopDefault(import("@eslint-react/eslint-plugin")),
-		interopDefault(import("eslint-plugin-react-hooks")),
-		interopDefault(import("eslint-plugin-react-refresh"))
-	]);
+	const [pluginReact, pluginReactRefresh] = await Promise.all([interopDefault(import("@eslint-react/eslint-plugin")), interopDefault(import("eslint-plugin-react-refresh"))]);
 	const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some((i) => isPackageExists(i));
 	const isUsingRemix = RemixPackages.some((i) => isPackageExists(i));
 	const isUsingReactRouter = ReactRouterPackages.some((i) => isPackageExists(i));
 	const isUsingNext = NextJsPackages.some((i) => isPackageExists(i));
 	const plugins = pluginReact.configs.all.plugins;
-	const pluginReactHooksExtra = plugins["@eslint-react/hooks-extra"];
 	return [
 		{
 			name: "config/react/setup",
 			plugins: {
 				"react": plugins["@eslint-react"],
 				"react-dom": plugins["@eslint-react/dom"],
-				"react-hooks": pluginReactHooks,
-				...pluginReactHooksExtra ? { "react-hooks-extra": pluginReactHooksExtra } : {},
 				"react-naming-convention": plugins["@eslint-react/naming-convention"],
 				"react-refresh": pluginReactRefresh,
 				"react-rsc": plugins["@eslint-react/rsc"],
@@ -1798,9 +1787,6 @@ async function react(options = {}) {
 			rules: {
 				"react/jsx-key-before-spread": "warn",
 				"react/jsx-no-comment-textnodes": "warn",
-				"react/jsx-no-duplicate-props": "warn",
-				"react/jsx-uses-react": "warn",
-				"react/jsx-uses-vars": "warn",
 				"react/no-access-state-in-setstate": "error",
 				"react/no-array-index-key": "warn",
 				"react/no-children-count": "warn",
@@ -1814,26 +1800,22 @@ async function react(options = {}) {
 				"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-forward-ref": "warn",
 				"react/no-missing-key": "error",
 				"react/no-nested-component-definitions": "error",
 				"react/no-nested-lazy-component-declarations": "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-unused-class-component-members": "warn",
 				"react/no-use-context": "warn",
-				"react/no-useless-forward-ref": "warn",
-				"react/prefer-use-state-lazy-initialization": "warn",
+				"react/set-state-in-effect": "warn",
 				"react/prefer-namespace-import": "error",
 				"react-rsc/function-definition": "error",
 				"react-dom/no-dangerously-set-innerhtml": "warn",
@@ -1848,32 +1830,24 @@ async function react(options = {}) {
 				"react-dom/no-unsafe-iframe-sandbox": "warn",
 				"react-dom/no-use-form-state": "error",
 				"react-dom/no-void-elements-with-children": "error",
-				...pluginReactHooksExtra ? { "react-hooks-extra/no-direct-set-state-in-use-effect": "warn" } : {},
 				"react-naming-convention/context-name": "warn",
 				"react-naming-convention/ref-name": "warn",
-				"react-naming-convention/use-state": "warn",
 				"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",
-				"react-hooks/exhaustive-deps": "warn",
-				"react-hooks/rules-of-hooks": "error",
+				"react/exhaustive-deps": "warn",
+				"react/rules-of-hooks": "error",
 				...reactCompiler ? {
-					"react-hooks/component-hook-factories": "error",
-					"react-hooks/config": "error",
-					"react-hooks/error-boundaries": "error",
-					"react-hooks/gating": "error",
-					"react-hooks/globals": "error",
-					"react-hooks/immutability": "error",
-					"react-hooks/incompatible-library": "warn",
-					"react-hooks/preserve-manual-memoization": "error",
-					"react-hooks/purity": "error",
-					"react-hooks/refs": "error",
-					"react-hooks/set-state-in-effect": "error",
-					"react-hooks/set-state-in-render": "error",
-					"react-hooks/static-components": "error",
-					"react-hooks/unsupported-syntax": "warn",
-					"react-hooks/use-memo": "error"
+					"react/component-hook-factories": "error",
+					"react/error-boundaries": "error",
+					"react/immutability": "error",
+					"react/purity": "error",
+					"react/refs": "error",
+					"react/set-state-in-effect": "error",
+					"react/set-state-in-render": "error",
+					"react/unsupported-syntax": "warn",
+					"react/use-memo": "error"
 				} : {},
 				"react-refresh/only-export-components": ["error", {
 					allowConstantExport: isAllowConstantExport,
@@ -1912,11 +1886,7 @@ async function react(options = {}) {
 			name: "config/react/typescript",
 			rules: {
 				"react-dom/no-string-style-prop": "off",
-				"react-dom/no-unknown-property": "off",
-				"react/jsx-no-duplicate-props": "off",
-				"react/jsx-no-undef": "off",
-				"react/jsx-uses-react": "off",
-				"react/jsx-uses-vars": "off"
+				"react-dom/no-unknown-property": "off"
 			}
 		},
 		...isTypeAware ? [{
@@ -3109,7 +3079,6 @@ const VuePackages = [
 const defaultPluginRenaming = {
 	"@eslint-react": "react",
 	"@eslint-react/dom": "react-dom",
-	"@eslint-react/hooks-extra": "react-hooks-extra",
 	"@eslint-react/naming-convention": "react-naming-convention",
 	"@next/next": "next",
 	"@stylistic": "style",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@infernodesign/eslint-config",
   "description": "ESLint config for Inferno Design.",
-  "version": "1.25.0",
+  "version": "1.26.0",
   "type": "module",
   "author": "Inferno Design <support@infernodesign.com>",
   "license": "MIT",
@@ -104,7 +104,6 @@
     "eslint-plugin-erasable-syntax-only": "0.4.0",
     "eslint-plugin-format": "2.0.1",
     "eslint-plugin-jsx-a11y": "6.10.2",
-    "eslint-plugin-react-hooks": "7.0.1",
     "eslint-plugin-react-refresh": "0.5.2",
     "eslint-plugin-solid": "0.14.5",
     "eslint-plugin-storybook": "10.3.1",
@@ -137,7 +136,6 @@
     "eslint-plugin-erasable-syntax-only": "^0.4.0",
     "eslint-plugin-format": "^2.0.1",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-react-refresh": "^0.5.2",
     "eslint-plugin-solid": "^0.14.5",
     "eslint-plugin-storybook": "^10.3.1",
@@ -178,9 +176,6 @@
     "eslint-plugin-jsx-a11y": {
       "optional": true
     },
-    "eslint-plugin-react-hooks": {
-      "optional": true
-    },
     "eslint-plugin-react-refresh": {
       "optional": true
     },