feat: add suggest to 'react-x/no-missing-button-type'

.github/workflows/check.yml

@@ -14,10 +14,10 @@ jobs:
     runs-on: ubuntu-24.04-arm
     steps:
       - uses: actions/checkout@v4
-      - name: Setup node@23
+      - name: Setup node@24
         uses: actions/setup-node@v4
         with:
-          node-version: 23
+          node-version: 24
       - name: Enable Corepack
         run: corepack enable
       - name: Install front-end dependencies

.github/workflows/codeql.yml

@@ -27,7 +27,7 @@ jobs:
     #   - https://gh.io/supported-runners-and-hardware-resources
     #   - https://gh.io/using-larger-runners (GitHub.com only)
     # Consider using larger runners or machines with greater resources for possible analysis time improvements.
-    runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }}
+    runs-on: ${{ (matrix.language == 'swift' && 'ubuntu-24.04-arm') || 'ubuntu-latest' }}
     timeout-minutes: ${{ (matrix.language == 'swift' && 120) || 360 }}
     permissions:
       # required for all workflows

.github/workflows/publish.yml

@@ -22,10 +22,10 @@ jobs:
       id-token: write
     steps:
       - uses: actions/checkout@v4
-      - name: Setup node@23
+      - name: Setup node@24
         uses: actions/setup-node@v4
         with:
-          node-version: 23
+          node-version: 24
           registry-url: "https://registry.npmjs.org"
       - name: Enable Corepack
         run: corepack enable

.github/workflows/test.yml

@@ -15,23 +15,23 @@ jobs:
     runs-on: ubuntu-24.04-arm
     steps:
       - uses: actions/checkout@v4
-      - name: Setup node@23
+      - name: Setup node@24
         uses: actions/setup-node@v4
         with:
-          node-version: 23
+          node-version: 24
       - name: Enable Corepack
         run: corepack enable
       - name: Install front-end dependencies
         run: pnpm install
       - name: Build front-end assets
         run: pnpm run build
-      - name: Test on node@23
+      - name: Test on node@24
         run: pnpm run test
-      - name: Setup node@18
+      - name: Setup node@20
         uses: actions/setup-node@v4
         with:
-          node-version: 18
+          node-version: 20
       - name: Enable Corepack
         run: corepack enable
-      - name: Test on node@18
+      - name: Test on node@20
         run: pnpm run test

.pkgs/configs/eslint.js

@@ -13,6 +13,7 @@ const templateIndentTags = [
     "ts",
     "tsx",
     "html",
+    "glsl",
     "dedent",
     "outdent",
 ];

.pkgs/configs/eslint.ts

@@ -17,6 +17,7 @@ const templateIndentTags = [
   "ts",
   "tsx",
   "html",
+  "glsl",
   "dedent",
   "outdent",
 ];

.pkgs/eslint-plugin-local/package.json

@@ -47,11 +47,11 @@
     "tsup": "^8.5.0"
   },
   "peerDependencies": {
-    "eslint": "^8.57.0 || ^9.0.0",
-    "typescript": "^4.9.5 || ^5.3.3"
+    "eslint": "^9.29.0",
+    "typescript": "^4.9.5 || ^5.4.5"
   },
   "engines": {
     "bun": ">=1.0.15",
-    "node": ">=18.18.0"
+    "node": ">=20.19.0"
   }
 }

.pkgs/eslint-plugin-local/tsup.config.ts

@@ -11,6 +11,6 @@ export default {
   platform: "node",
   sourcemap: false,
   splitting: false,
-  target: "node18",
+  target: "node20",
   treeshake: true,
 } satisfies Options;

README.md

@@ -41,7 +41,6 @@
 - [`eslint-plugin-react-x`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-x) - X Rules (renderer-agnostic, compatible with x-platform).
 - [`eslint-plugin-react-dom`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-dom) - DOM specific rules for React DOM.
 - [`eslint-plugin-react-web-api`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-web-api) - Rules for interacting with Web APIs.
-- [`eslint-plugin-react-hooks-extra`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-hooks-extra) - Extra React Hooks rules.
 - [`eslint-plugin-react-naming-convention`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-naming-convention) - Naming convention rules.
 
 ### Unified
@@ -53,8 +52,8 @@
 > [!NOTE]\
 > ESLint React requires the following minimum versions:
 >
-> - Node.js: 18.18.0
-> - ESLint: 8.57.0
+> - Node.js: 20.19.0
+> - ESLint: 9.24.0
 > - TypeScript: 4.9.5
 
 ### Install

VERSION

@@ -1 +1 @@
-1.52.3-next.6
+2.0.0-next.47

apps/website/content/docs/contributing.mdx

@@ -38,7 +38,6 @@ flowchart TB
                 ReactX["eslint-plugin-react-x"]:::plugins
                 ReactDOM["eslint-plugin-react-dom"]:::plugins
                 WebAPI["eslint-plugin-react-web-api"]:::plugins
-                HooksExtra["eslint-plugin-react-hooks-extra"]:::plugins
                 NamingConvention["eslint-plugin-react-naming-convention"]:::plugins
                 Aggregated["Aggregated Plugin (@eslint-react/eslint-plugin)"]:::plugins
             end
@@ -56,13 +55,11 @@ flowchart TB
     Core -->|"dependency"| ReactX
     Core -->|"dependency"| ReactDOM
     Core -->|"dependency"| WebAPI
-    Core -->|"dependency"| HooksExtra
     Core -->|"dependency"| NamingConvention
 
     Shared -->|"dependency"| ReactX
     Shared -->|"dependency"| ReactDOM
     Shared -->|"dependency"| WebAPI
-    Shared -->|"dependency"| HooksExtra
     Shared -->|"dependency"| NamingConvention
 
     %% Utilities used by Core and Plugins
@@ -80,7 +77,6 @@ flowchart TB
     ReactX -->|"aggregated"| Aggregated
     ReactDOM -->|"aggregated"| Aggregated
     WebAPI -->|"aggregated"| Aggregated
-    HooksExtra -->|"aggregated"| Aggregated
     NamingConvention -->|"aggregated"| Aggregated
 
     %% Website Documentation uses Core and Shared docs
@@ -97,7 +93,6 @@ flowchart TB
     Scripts -->|"CI/CD"| ReactX
     Scripts -->|"CI/CD"| ReactDOM
     Scripts -->|"CI/CD"| WebAPI
-    Scripts -->|"CI/CD"| HooksExtra
     Scripts -->|"CI/CD"| NamingConvention
     Scripts -->|"CI/CD"| Aggregated
     Scripts -->|"CI/CD"| Tests
@@ -114,7 +109,6 @@ flowchart TB
     click ReactX "https://github.com/rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-x"
     click ReactDOM "https://github.com/rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-dom"
     click WebAPI "https://github.com/rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-web-api"
-    click HooksExtra "https://github.com/rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-hooks-extra"
     click NamingConvention "https://github.com/rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-naming-convention"
     click Aggregated "https://github.com/rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin"
     click Scripts "https://github.com/rel1cx/eslint-react/tree/main/scripts"
@@ -154,7 +148,6 @@ This section provides a summary of the packages in the monorepo.
   - `packages/plugins/eslint-plugin-react-x`: Core React rules
   - `packages/plugins/eslint-plugin-react-dom`: React DOM rules
   - `packages/plugins/eslint-plugin-react-web-api`: Web API interaction rules
-  - `packages/plugins/eslint-plugin-react-hooks-extra`: Extra React Hooks rules
   - `packages/plugins/eslint-plugin-react-naming-convention`: Naming convention rules
   - `packages/plugins/eslint-plugin-react-debug`: Debugging rules for inspecting React patterns in code
   - `packages/plugins/eslint-plugin`: A unified plugin that combines all individual plugins into one

apps/website/content/docs/faq.mdx

@@ -39,7 +39,6 @@ Currently, it includes the following:
   - [`eslint-plugin-react-x`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-x) - Core React rules.
   - [`eslint-plugin-react-dom`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-dom) - React DOM rules.
   - [`eslint-plugin-react-web-api`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-web-api) - Web API interaction rules.
-  - [`eslint-plugin-react-hooks-extra`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-hooks-extra) - Extra React Hooks rules.
   - [`eslint-plugin-react-naming-convention`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-naming-convention) - Naming convention rules.
   - [`eslint-plugin-react-debug`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin-react-debug) - Debugging rules for inspecting React patterns in code.
   - [`@eslint-react/eslint-plugin`](https://github.com/Rel1cx/eslint-react/tree/main/packages/plugins/eslint-plugin) - A unified plugin that combines all individual plugins into one.

apps/website/content/docs/getting-started/javascript.mdx

@@ -7,8 +7,8 @@ import { Step, Steps } from "fumadocs-ui/components/steps";
 
 <Callout type="info" title="This instruction requires the following minimum versions:">
 
-- Node.js: 18.18.0
-- ESLint: 8.57.0
+- Node.js: 20.19.0
+- ESLint: 9.24.0
 - TypeScript: 4.9.5
 
 </Callout>

apps/website/content/docs/getting-started/typescript.mdx

@@ -7,8 +7,8 @@ import { Step, Steps } from "fumadocs-ui/components/steps";
 
 <Callout type="info" title="This instruction requires the following minimum versions:">
 
-- Node.js: 18.18.0
-- ESLint: 8.57.0
+- Node.js: 20.19.0
+- ESLint: 9.24.0
 - TypeScript: 4.9.5
 
 </Callout>

apps/website/content/docs/rules/meta.json

@@ -3,9 +3,12 @@
     "overview",
     "---X Rules---",
     "jsx-key-before-spread",
+    "jsx-no-comment-textnodes",
     "jsx-no-duplicate-props",
     "jsx-no-iife",
     "jsx-no-undef",
+    "jsx-shorthand-boolean",
+    "jsx-shorthand-fragment",
     "jsx-uses-react",
     "jsx-uses-vars",
     "no-access-state-in-setstate",
@@ -18,15 +21,15 @@
     "no-children-to-array",
     "no-class-component",
     "no-clone-element",
-    "no-comment-textnodes",
-    "no-complex-conditional-rendering",
     "no-component-will-mount",
     "no-component-will-receive-props",
     "no-component-will-update",
     "no-context-provider",
     "no-create-ref",
     "no-default-props",
     "no-direct-mutation-state",
+    "no-direct-set-state-in-use-effect",
+    "no-direct-set-state-in-use-layout-effect",
     "no-duplicate-key",
     "no-forward-ref",
     "no-implicit-key",
@@ -43,6 +46,9 @@
     "no-set-state-in-component-did-update",
     "no-set-state-in-component-will-update",
     "no-string-refs",
+    "no-unnecessary-use-callback",
+    "no-unnecessary-use-memo",
+    "no-unnecessary-use-prefix",
     "no-unsafe-component-will-mount",
     "no-unsafe-component-will-receive-props",
     "no-unsafe-component-will-update",
@@ -52,14 +58,10 @@
     "no-unused-state",
     "no-use-context",
     "no-useless-forward-ref",
-    "no-useless-fragment",
     "prefer-destructuring-assignment",
-    "prefer-react-namespace-import",
+    "prefer-namespace-import",
     "prefer-read-only-props",
-    "prefer-shorthand-boolean",
-    "prefer-shorthand-fragment",
-    "avoid-shorthand-boolean",
-    "avoid-shorthand-fragment",
+    "prefer-use-state-lazy-initialization",
     "---DOM Rules---",
     "dom-no-dangerously-set-innerhtml",
     "dom-no-dangerously-set-innerhtml-with-children",
@@ -82,13 +84,6 @@
     "web-api-no-leaked-interval",
     "web-api-no-leaked-resize-observer",
     "web-api-no-leaked-timeout",
-    "---Hooks Extra Rules---",
-    "hooks-extra-no-direct-set-state-in-use-effect",
-    "hooks-extra-no-direct-set-state-in-use-layout-effect",
-    "hooks-extra-no-unnecessary-use-callback",
-    "hooks-extra-no-unnecessary-use-memo",
-    "hooks-extra-no-unnecessary-use-prefix",
-    "hooks-extra-prefer-use-state-lazy-initialization",
     "---Naming Convention Rules---",
     "naming-convention-component-name",
     "naming-convention-context-name",

apps/website/content/docs/rules/overview.mdx

@@ -35,6 +35,8 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
 | [`jsx-no-duplicate-props`](./jsx-no-duplicate-props)                                 | 1️⃣  |           | Disallow duplicate props in JSX elements                                                            |          |
 | [`jsx-no-iife`](./jsx-no-iife)                                                       | 0️⃣  |   `🧪`    | Disallows `IIFE` in JSX elements                                                                     |          |
 | [`jsx-no-undef`](./jsx-no-undef)                                                     | 0️⃣  |           | Disallow undefined variables in JSX elements                                                        |          |
+| [`jsx-shorthand-boolean`](./jsx-shorthand-boolean)                                   | 0️⃣  | `🔧` `⚙️` | Enforces the use of shorthand syntax for boolean attributes                                          |          |
+| [`jsx-shorthand-fragment`](./jsx-shorthand-fragment)                                 | 0️⃣  | `🔧` `⚙️` | Enforces the use of shorthand syntax for fragments                                                  |          |
 | [`jsx-uses-react`](./jsx-uses-react)                                                 | 1️⃣  |           | Marks React variables as used when JSX is used                                                      |          |
 | [`jsx-uses-vars`](./jsx-uses-vars)                                                   | 1️⃣  |           | Marks variables used in JSX elements as used                                                        |          |
 | [`no-access-state-in-setstate`](./no-access-state-in-setstate)                       | 2️⃣  |           | Disallow accessing `this.state` inside `setState` calls                                             |          |
@@ -72,6 +74,9 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
 | [`no-set-state-in-component-did-update`](./no-set-state-in-component-did-update)     | 1️⃣  |           | Disallow calling `this.setState` in `componentDidUpdate` outside of functions, such as callbacks    |          |
 | [`no-set-state-in-component-will-update`](./no-set-state-in-component-will-update)   | 1️⃣  |           | Disallow calling `this.setState` in `componentWillUpdate` outside of functions, such as callbacks   |          |
 | [`no-string-refs`](./no-string-refs)                                                 | 2️⃣  |   `🔄`    | Replaces string refs with callback refs                                                             | >=16.3.0 |
+| [`no-unnecessary-use-callback`](./no-unnecessary-use-callback)                       | 0️⃣  |   `🧪`     | Disallow unnecessary usage of `useCallback`                                                         |          |
+| [`no-unnecessary-use-memo`](./no-unnecessary-use-memo)                               | 0️⃣  |   `🧪`     | Disallow unnecessary usage of `useMemo`                                                             |          |
+| [`no-unnecessary-use-prefix`](./no-unnecessary-use-prefix)                           | 0️⃣  |            | Enforces that a function with the `use` prefix should use at least one Hook inside of it            |          |
 | [`no-unsafe-component-will-mount`](./no-unsafe-component-will-mount)                 | 1️⃣  |           | Warns the usage of `UNSAFE_componentWillMount` in class components                                  |          |
 | [`no-unsafe-component-will-receive-props`](./no-unsafe-component-will-receive-props) | 1️⃣  |           | Warns the usage of `UNSAFE_componentWillReceiveProps` in class components                           |          |
 | [`no-unsafe-component-will-update`](./no-unsafe-component-will-update)               | 1️⃣  |           | Warns the usage of `UNSAFE_componentWillUpdate` in class components                                 |          |
@@ -85,10 +90,7 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
 | [`prefer-destructuring-assignment`](./prefer-destructuring-assignment)               | 0️⃣  |           | Enforces destructuring assignment for component props and context                                   |          |
 | [`prefer-react-namespace-import`](./prefer-react-namespace-import)                   | 0️⃣  |   `🔧`    | Enforces React is imported via a namespace import                                                   |          |
 | [`prefer-read-only-props`](./prefer-read-only-props)                                 | 0️⃣  |   `💭`    | Enforces read-only props in components                                                              |          |
-| [`prefer-shorthand-boolean`](./prefer-shorthand-boolean)                             | 0️⃣  |   `🔧`    | Enforces shorthand syntax for boolean attributes                                                    |          |
-| [`prefer-shorthand-fragment`](./prefer-shorthand-fragment)                           | 0️⃣  |   `🔧`    | Enforces shorthand syntax for fragments                                                             |          |
-| [`avoid-shorthand-boolean`](./avoid-shorthand-boolean)                               | 0️⃣  |   `🔧`    | Enforces explicit boolean values for boolean attributes                                             |          |
-| [`avoid-shorthand-fragment`](./avoid-shorthand-fragment)                             | 0️⃣  |           | Enforces explicit `<Fragment>` components instead of the shorthand `<>` or `</>` syntax             |          |
+| [`prefer-use-state-lazy-initialization`](./prefer-use-state-lazy-initialization)     | 1️⃣  |           | Enforces function calls made inside `useState` to be wrapped in an `initializer function`           |          |
 
 ## DOM Rules
 
@@ -132,10 +134,6 @@ This section contains rules that are not part of the official `eslint-plugin-rea
 | :--------------------------------------------------------------------------------------------------- | :-- | :--: | :---------------------------------------------------------------------------------------- |
 | [`no-direct-set-state-in-use-effect`](./hooks-extra-no-direct-set-state-in-use-effect)               | 1️⃣  |     | Disallow direct calls to the `set` function of `useState` in `useEffect`                  |
 | [`no-direct-set-state-in-use-layout-effect`](./hooks-extra-no-direct-set-state-in-use-layout-effect) | 0️⃣  | `🧪` | Disallow direct calls to the `set` function of `useState` in `useLayoutEffect`            |
-| [`no-unnecessary-use-callback`](./hooks-extra-no-unnecessary-use-callback)                           | 0️⃣  | `🧪` | Disallow unnecessary usage of `useCallback`                                               |
-| [`no-unnecessary-use-memo`](./hooks-extra-no-unnecessary-use-memo)                                   | 0️⃣  | `🧪` | Disallow unnecessary usage of `useMemo`                                                   |
-| [`no-unnecessary-use-prefix`](./hooks-extra-no-unnecessary-use-prefix)                               | 1️⃣  |      | Enforces that a function with the `use` prefix should use at least one hook inside of it  |
-| [`prefer-use-state-lazy-initialization`](./hooks-extra-prefer-use-state-lazy-initialization)         | 1️⃣  |      | Enforces function calls made inside `useState` to be wrapped in an `initializer function` |
 
 ## Naming Convention Rules