feat!: remove 'jsx-shorthand-*' rules (#1091)

Author: Rel1cx

apps/website/content/docs/roadmap.md

@@ -38,9 +38,7 @@ Minimum supported versions:
 
 - [ ] `function-component-definition`
 - [x] `no-useless-fragment`
-- [x] `prefer-react-namespace-import`
-- [x] `prefer-shorthand-fragment`
-- [x] `prefer-shorthand-boolean`
+- [x] `prefer-namespace-import`
 
 ### Add suggestion-fix feature to rules that can be fixed interactively
 
@@ -56,8 +54,6 @@ Minimum supported versions:
 ### New Rules
 
 - [x] `jsx-no-comment-textnodes` - Disallow text nodes with comments in JSX (Replaces `no-comment-textnodes`)
-- [x] `jsx-shorthand-boolean` - Enforces shorthand boolean attributes in JSX (Replaces `prefer-shorthand-boolean`)
-- [x] `jsx-shorthand-fragment` - Enforces shorthand fragment syntax in JSX (Replaces `prefer-shorthand-fragment`)
 - [x] `no-context-provider` - Replaces usages of `<Context.Provider>` with `<Context>` (React 19)
 - [x] `no-forward-ref` - Replaces usages of `forwardRef` with passing `ref` as a prop (React 19)
 - [x] `no-use-context` - Replaces usages of `useContext` with `use` (React 19)
@@ -74,10 +70,12 @@ Minimum supported versions:
 
 ### Removed Rules
 
+- [x] `avoid-shorthand-boolean`
+- [x] `avoid-shorthand-fragment`
 - [x] `no-comment-textnodes` - Replaced by `jsx-no-comment-textnodes`
 - [x] `prefer-react-namespace-import` - Replaced by `prefer-namespace-import`
 - [x] `prefer-shorthand-boolean` - Replaced by `jsx-shorthand-boolean`
-- [x] `prefer-shorthand-fragment` - Replaced by `jsx-shorthand-fragment`
+- [x] `prefer-shorthand-fragment`
 
 ### Versioning Policy
 

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

@@ -5,8 +5,6 @@
     "jsx-no-comment-textnodes",
     "jsx-no-duplicate-props",
     "jsx-no-undef",
-    "jsx-shorthand-boolean",
-    "jsx-shorthand-fragment",
     "jsx-uses-react",
     "jsx-uses-vars",
     "no-access-state-in-setstate",

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

@@ -34,8 +34,6 @@ The `jsx-*` rules check for issues exclusive to JSX syntax, which are absent fro
 | [`jsx-no-comment-textnodes`](./jsx-no-comment-textnodes)                             | 1️⃣  |           | Prevents comments from being inserted as text nodes                                                 |          |
 | [`jsx-no-duplicate-props`](./jsx-no-duplicate-props)                                 | 1️⃣  |           | Disallow duplicate props 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 or explicit boolean attributes                                        |          |
-| [`jsx-shorthand-fragment`](./jsx-shorthand-fragment)                                 | 0️⃣  |           | Enforces the use of shorthand `<>` or `</>` syntax or `<React.Fragment>` element                    |          |
 | [`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                                             |          |

packages/plugins/eslint-plugin-react-x/src/configs/recommended.ts

@@ -7,8 +7,6 @@ export const rules = {
   "react-x/jsx-no-comment-textnodes": "warn",
   "react-x/jsx-no-duplicate-props": "warn",
   // "react-x/jsx-no-undef": "error",
-  // "react-x/jsx-shorthand-boolean": "warn",
-  // "react-x/jsx-shorthand-fragment": "warn",
   "react-x/jsx-uses-react": "warn",
   "react-x/jsx-uses-vars": "warn",
   "react-x/no-access-state-in-setstate": "error",

packages/plugins/eslint-plugin-react-x/src/plugin.ts

@@ -2,8 +2,6 @@ import { name, version } from "../package.json";
 import jsxNoCommentTextnodes from "./rules/jsx-no-comment-textnodes";
 import jsxNoDuplicateProps from "./rules/jsx-no-duplicate-props";
 import jsxNoUndef from "./rules/jsx-no-undef";
-import jsxShorthandBoolean from "./rules/jsx-shorthand-boolean";
-import jsxShorthandFragment from "./rules/jsx-shorthand-fragment";
 import jsxUsesReact from "./rules/jsx-uses-react";
 import jsxUsesVars from "./rules/jsx-uses-vars";
 import noAccessStateInSetstate from "./rules/no-access-state-in-setstate";
@@ -62,8 +60,6 @@ export const plugin = {
     "jsx-no-comment-textnodes": jsxNoCommentTextnodes,
     "jsx-no-duplicate-props": jsxNoDuplicateProps,
     "jsx-no-undef": jsxNoUndef,
-    "jsx-shorthand-boolean": jsxShorthandBoolean,
-    "jsx-shorthand-fragment": jsxShorthandFragment,
     "jsx-uses-react": jsxUsesReact,
     "jsx-uses-vars": jsxUsesVars,
     "no-access-state-in-setstate": noAccessStateInSetstate,