eslint-plugin-react-x 1.42.2-next.5 → 1.43.0-next.0

package/dist/index.js

@@ -141,7 +141,7 @@ var settings3 = {
 
 // package.json
 var name4 = "eslint-plugin-react-x";
-var version = "1.42.2-next.5";
+var version = "1.43.0-next.0";
 var createRule = utils.ESLintUtils.RuleCreator(shared.getDocsUrl("x"));
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -1976,7 +1976,7 @@ function create33(context) {
       const isFunctionComponent = (node2) => {
         return AST8__namespace.isFunction(node2) && functionComponents.some((component) => component.node === node2);
       };
-      const isClassComponent11 = (node2) => {
+      const isClassComponent12 = (node2) => {
         return AST8__namespace.isClass(node2) && classComponents.some((component) => component.node === node2);
       };
       for (const { name: name5, node: component } of functionComponents) {
@@ -2030,7 +2030,7 @@ function create33(context) {
         }
       }
       for (const { name: name5 = "unknown", node: component } of classComponents) {
-        if (AST8__namespace.findParentNode(component, (n) => isClassComponent11(n) || isFunctionComponent(n)) == null) {
+        if (AST8__namespace.findParentNode(component, (n) => isClassComponent12(n) || isFunctionComponent(n)) == null) {
           continue;
         }
         context.report({
@@ -2275,26 +2275,17 @@ function create38(context) {
   };
 }
 var RULE_NAME39 = "no-string-refs";
-var RULE_FEATURES39 = [];
-function containsStringLiteral({ value }) {
-  return value?.type === types.AST_NODE_TYPES.Literal && typeof value.value === "string";
-}
-function containsStringExpressionContainer({ value }) {
-  if (value?.type !== types.AST_NODE_TYPES.JSXExpressionContainer) {
-    return false;
-  }
-  if (value.expression.type === types.AST_NODE_TYPES.Literal) {
-    return typeof value.expression.value === "string";
-  }
-  return value.expression.type === types.AST_NODE_TYPES.TemplateLiteral;
-}
+var RULE_FEATURES39 = [
+  "MOD"
+];
 var no_string_refs_default = createRule({
   meta: {
     type: "problem",
     docs: {
-      description: "Disallow deprecated string `refs`.",
+      description: "Replaces string refs with callback refs.",
       [Symbol.for("rule_features")]: RULE_FEATURES39
     },
+    fixable: "code",
     messages: {
       noStringRefs: "[Deprecated] Use callback refs instead."
     },
@@ -2305,20 +2296,49 @@ var no_string_refs_default = createRule({
   defaultOptions: []
 });
 function create39(context) {
+  const state = {
+    isWithinClassComponent: false
+  };
+  function onClassBodyEnter(node) {
+    if (ER21__namespace.isClassComponent(node.parent)) {
+      state.isWithinClassComponent = true;
+    }
+  }
+  function onClassBodyExit() {
+    state.isWithinClassComponent = false;
+  }
   return {
+    ClassBody: onClassBodyEnter,
+    "ClassBody:exit": onClassBodyExit,
     JSXAttribute(node) {
-      if (node.name.name !== "ref") {
-        return;
-      }
-      if (containsStringLiteral(node) || containsStringExpressionContainer(node)) {
-        context.report({
-          messageId: "noStringRefs",
-          node
-        });
-      }
+      if (node.name.name !== "ref") return;
+      const refNameText = getAttributeValueText(context, node.value);
+      if (refNameText == null) return;
+      context.report({
+        messageId: "noStringRefs",
+        node,
+        fix(fixer) {
+          if (node.value == null) return null;
+          if (!state.isWithinClassComponent) return null;
+          return fixer.replaceText(node.value, `{(ref) => { this.refs[${refNameText}] = ref; }}`);
+        }
+      });
     }
   };
 }
+function getAttributeValueText(context, node) {
+  if (node == null) return null;
+  switch (true) {
+    case (node.type === types.AST_NODE_TYPES.Literal && typeof node.value === "string"):
+      return context.sourceCode.getText(node);
+    case (node.type === types.AST_NODE_TYPES.JSXExpressionContainer && node.expression.type === types.AST_NODE_TYPES.Literal && typeof node.expression.value === "string"):
+      return context.sourceCode.getText(node.expression);
+    case (node.type === types.AST_NODE_TYPES.JSXExpressionContainer && node.expression.type === types.AST_NODE_TYPES.TemplateLiteral):
+      return context.sourceCode.getText(node.expression);
+    default:
+      return null;
+  }
+}
 var RULE_NAME40 = "no-unsafe-component-will-mount";
 var RULE_FEATURES40 = [];
 function isUnsafeComponentWillMount(node) {

package/dist/index.mjs

@@ -113,7 +113,7 @@ var settings3 = {
 
 // package.json
 var name4 = "eslint-plugin-react-x";
-var version = "1.42.2-next.5";
+var version = "1.43.0-next.0";
 var createRule = ESLintUtils.RuleCreator(getDocsUrl("x"));
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -1948,7 +1948,7 @@ function create33(context) {
       const isFunctionComponent = (node2) => {
         return AST8.isFunction(node2) && functionComponents.some((component) => component.node === node2);
       };
-      const isClassComponent11 = (node2) => {
+      const isClassComponent12 = (node2) => {
         return AST8.isClass(node2) && classComponents.some((component) => component.node === node2);
       };
       for (const { name: name5, node: component } of functionComponents) {
@@ -2002,7 +2002,7 @@ function create33(context) {
         }
       }
       for (const { name: name5 = "unknown", node: component } of classComponents) {
-        if (AST8.findParentNode(component, (n) => isClassComponent11(n) || isFunctionComponent(n)) == null) {
+        if (AST8.findParentNode(component, (n) => isClassComponent12(n) || isFunctionComponent(n)) == null) {
           continue;
         }
         context.report({
@@ -2247,26 +2247,17 @@ function create38(context) {
   };
 }
 var RULE_NAME39 = "no-string-refs";
-var RULE_FEATURES39 = [];
-function containsStringLiteral({ value }) {
-  return value?.type === AST_NODE_TYPES.Literal && typeof value.value === "string";
-}
-function containsStringExpressionContainer({ value }) {
-  if (value?.type !== AST_NODE_TYPES.JSXExpressionContainer) {
-    return false;
-  }
-  if (value.expression.type === AST_NODE_TYPES.Literal) {
-    return typeof value.expression.value === "string";
-  }
-  return value.expression.type === AST_NODE_TYPES.TemplateLiteral;
-}
+var RULE_FEATURES39 = [
+  "MOD"
+];
 var no_string_refs_default = createRule({
   meta: {
     type: "problem",
     docs: {
-      description: "Disallow deprecated string `refs`.",
+      description: "Replaces string refs with callback refs.",
       [Symbol.for("rule_features")]: RULE_FEATURES39
     },
+    fixable: "code",
     messages: {
       noStringRefs: "[Deprecated] Use callback refs instead."
     },
@@ -2277,20 +2268,49 @@ var no_string_refs_default = createRule({
   defaultOptions: []
 });
 function create39(context) {
+  const state = {
+    isWithinClassComponent: false
+  };
+  function onClassBodyEnter(node) {
+    if (ER21.isClassComponent(node.parent)) {
+      state.isWithinClassComponent = true;
+    }
+  }
+  function onClassBodyExit() {
+    state.isWithinClassComponent = false;
+  }
   return {
+    ClassBody: onClassBodyEnter,
+    "ClassBody:exit": onClassBodyExit,
     JSXAttribute(node) {
-      if (node.name.name !== "ref") {
-        return;
-      }
-      if (containsStringLiteral(node) || containsStringExpressionContainer(node)) {
-        context.report({
-          messageId: "noStringRefs",
-          node
-        });
-      }
+      if (node.name.name !== "ref") return;
+      const refNameText = getAttributeValueText(context, node.value);
+      if (refNameText == null) return;
+      context.report({
+        messageId: "noStringRefs",
+        node,
+        fix(fixer) {
+          if (node.value == null) return null;
+          if (!state.isWithinClassComponent) return null;
+          return fixer.replaceText(node.value, `{(ref) => { this.refs[${refNameText}] = ref; }}`);
+        }
+      });
     }
   };
 }
+function getAttributeValueText(context, node) {
+  if (node == null) return null;
+  switch (true) {
+    case (node.type === AST_NODE_TYPES.Literal && typeof node.value === "string"):
+      return context.sourceCode.getText(node);
+    case (node.type === AST_NODE_TYPES.JSXExpressionContainer && node.expression.type === AST_NODE_TYPES.Literal && typeof node.expression.value === "string"):
+      return context.sourceCode.getText(node.expression);
+    case (node.type === AST_NODE_TYPES.JSXExpressionContainer && node.expression.type === AST_NODE_TYPES.TemplateLiteral):
+      return context.sourceCode.getText(node.expression);
+    default:
+      return null;
+  }
+}
 var RULE_NAME40 = "no-unsafe-component-will-mount";
 var RULE_FEATURES40 = [];
 function isUnsafeComponentWillMount(node) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "1.42.2-next.5",
+  "version": "1.43.0-next.0",
   "description": "A set of composable ESLint rules for for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -50,13 +50,13 @@
     "is-immutable-type": "^5.0.1",
     "string-ts": "^2.2.1",
     "ts-pattern": "^5.7.0",
-    "@eslint-react/core": "1.42.2-next.5",
-    "@eslint-react/jsx": "1.42.2-next.5",
-    "@eslint-react/eff": "1.42.2-next.5",
-    "@eslint-react/ast": "1.42.2-next.5",
-    "@eslint-react/kit": "1.42.2-next.5",
-    "@eslint-react/shared": "1.42.2-next.5",
-    "@eslint-react/var": "1.42.2-next.5"
+    "@eslint-react/core": "1.43.0-next.0",
+    "@eslint-react/ast": "1.43.0-next.0",
+    "@eslint-react/kit": "1.43.0-next.0",
+    "@eslint-react/jsx": "1.43.0-next.0",
+    "@eslint-react/eff": "1.43.0-next.0",
+    "@eslint-react/shared": "1.43.0-next.0",
+    "@eslint-react/var": "1.43.0-next.0"
   },
   "devDependencies": {
     "@types/react": "^19.1.0",