feat: add rule react/ensure-forward-ref-using-ref (#240)

Author: Rel1cx

CHANGELOG.md

@@ -1,7 +1,9 @@
-## v0.9.8-beta.0 (Draft)
+## v0.9.8-beta.0 (Sun Dec 17 2023)
 
 ### Release Notes
 
+#### Add rule `react/ensure-forward-ref-using-ref`
+
 #### Reduce false positives in rule `react/no-unstable-nested-components`
 
 #### Reduce false positives in rule `debug/function-component`
@@ -10,13 +12,23 @@
 
 #### 🏠 Internal
 
+- `@eslint-react/eslint-plugin-react`
+  - Add rule `react/ensure-forward-ref-using-ref`.
+  - Reduce false positives in rule `react/no-unstable-nested-components`.
+
+- `@eslint-react/eslint-plugin-debug`
+  - Reduce false positives in rule `debug/function-component`.
+
 - `@eslint-react/ast`
   - Refactor `getFunctionIdentifier` function to follow spec convention for `IsAnonymousFunctionDefinition()` usage in ECMAScript spec.
   - Improve module structure.
 
 - `@eslint-react/core`
   - Move construction detection from `@eslint-react/ast` to `@eslint-react/core`.
 
+- `@eslint-react/monorepo`
+  - Update `rollup` to `4.9.1`.
+
 #### Authors: 1
 
 - Eva1ent ([@Rel1cx](https://github.com/Rel1cx))

README.md

@@ -171,6 +171,7 @@ module.exports = {
 - [x] `react/no-constructed-context-value`
 - [ ] `react/no-unused-class-component-methods`
 - [ ] `react/no-unused-state`
+- [ ] `react/ensure-forward-ref-using-ref`
 - [x] `react/prefer-destructuring-assignment`
 - [ ] `react/prefer-readonly-props`
 - [ ] `react/ban-components`

package.json

@@ -89,7 +89,7 @@
     "markdownlint": "0.32.1",
     "pathe": "1.1.1",
     "publint": "0.2.6",
-    "rollup": "4.9.0",
+    "rollup": "4.9.1",
     "rollup-plugin-dts": "6.1.0",
     "rollup-plugin-swc3": "0.11.0",
     "rollup-plugin-visualizer": "5.11.0",

packages/eslint-plugin-debug/src/rules/class-component.ts

@@ -11,7 +11,7 @@ export type MessageID = "CLASS_COMPONENT";
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       // eslint-disable-next-line eslint-plugin/require-meta-docs-description
       description: "report all class components, including anonymous ones",

packages/eslint-plugin-debug/src/rules/function-component.ts

@@ -12,7 +12,7 @@ export type MessageID = ConstantCase<typeof RULE_NAME>;
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       // eslint-disable-next-line eslint-plugin/require-meta-docs-description
       description: "report all function components, including anonymous ones",

packages/eslint-plugin-debug/src/rules/react-hooks.ts

@@ -10,7 +10,7 @@ export type MessageID = ConstantCase<typeof RULE_NAME>;
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       // eslint-disable-next-line eslint-plugin/require-meta-docs-description
       description: "report all react hooks",

packages/eslint-plugin-jsx/src/rules/no-complicated-conditional-rendering.ts

@@ -13,7 +13,7 @@ export type MessageID = ConstantCase<typeof RULE_NAME>;
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       description: "disallow complicated conditional rendering",
       requiresTypeChecking: true,

packages/eslint-plugin-jsx/src/rules/no-spreading-key.ts

@@ -13,7 +13,7 @@ export type MessageID = ConstantCase<typeof RULE_NAME>;
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       description: "disallow spreading `key` from objects.",
       recommended: "recommended",

packages/eslint-plugin-jsx/src/rules/no-useless-fragment.ts

@@ -67,7 +67,7 @@ function isFragmentWithSingleExpression(node: TSESTree.JSXElement | TSESTree.JSX
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       description: "disallow unnecessary fragments",
       recommended: "recommended",

packages/eslint-plugin-jsx/src/rules/prefer-shorthand-boolean.ts

@@ -12,7 +12,7 @@ export type MessageID = ConstantCase<typeof RULE_NAME>;
 export default createRule<[], MessageID>({
   name: RULE_NAME,
   meta: {
-    type: "suggestion",
+    type: "problem",
     docs: {
       description: "enforce `boolean` attributes notation in JSX",
       recommended: "recommended",