eslint-plugin-react-x 1.26.3-next.6 → 1.26.3-next.8

package/dist/index.d.mts

@@ -1,12 +1,8 @@
 import * as _typescript_eslint_utils_ts_eslint from '@typescript-eslint/utils/ts-eslint';
 
-type MessageID$3 = "noUselessFragment" | "noUselessFragmentInBuiltIn";
-
-type MessageID$2 = "noUnstableContextValue" | "noUnstableContextValueWithFunction" | "noUnstableContextValueWithIdentifier";
-
 type MessageID$1 = "nestedComponent" | "nestedComponentInProps";
 
-type MessageID = "noMissingKey" | "noMissingKeyWithFragment";
+type MessageID = "missingKey" | "unexpectedFragmentSyntax";
 
 declare const _default: {
     meta: {
@@ -53,12 +49,12 @@ declare const _default: {
         "no-unsafe-component-will-mount": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeComponentWillMount", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unsafe-component-will-receive-props": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeComponentWillReceiveProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unsafe-component-will-update": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeComponentWillUpdate", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "no-unstable-context-value": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$2, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        "no-unstable-context-value": _typescript_eslint_utils_ts_eslint.RuleModule<"unstableContextValue", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unstable-default-props": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnstableDefaultProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unused-class-component-members": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedClassComponentMembers", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unused-state": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedState", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-use-context": _typescript_eslint_utils_ts_eslint.RuleModule<"noUseContext", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$3, [{
+        "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"uselessFragment", [{
             allowExpressions: boolean;
         }], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "prefer-destructuring-assignment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferDestructuringAssignment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;

package/dist/index.d.ts

@@ -1,12 +1,8 @@
 import * as _typescript_eslint_utils_ts_eslint from '@typescript-eslint/utils/ts-eslint';
 
-type MessageID$3 = "noUselessFragment" | "noUselessFragmentInBuiltIn";
-
-type MessageID$2 = "noUnstableContextValue" | "noUnstableContextValueWithFunction" | "noUnstableContextValueWithIdentifier";
-
 type MessageID$1 = "nestedComponent" | "nestedComponentInProps";
 
-type MessageID = "noMissingKey" | "noMissingKeyWithFragment";
+type MessageID = "missingKey" | "unexpectedFragmentSyntax";
 
 declare const _default: {
     meta: {
@@ -53,12 +49,12 @@ declare const _default: {
         "no-unsafe-component-will-mount": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeComponentWillMount", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unsafe-component-will-receive-props": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeComponentWillReceiveProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unsafe-component-will-update": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeComponentWillUpdate", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "no-unstable-context-value": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$2, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        "no-unstable-context-value": _typescript_eslint_utils_ts_eslint.RuleModule<"unstableContextValue", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unstable-default-props": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnstableDefaultProps", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unused-class-component-members": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedClassComponentMembers", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-unused-state": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnusedState", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "no-use-context": _typescript_eslint_utils_ts_eslint.RuleModule<"noUseContext", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$3, [{
+        "no-useless-fragment": _typescript_eslint_utils_ts_eslint.RuleModule<"uselessFragment", [{
             allowExpressions: boolean;
         }], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         "prefer-destructuring-assignment": _typescript_eslint_utils_ts_eslint.RuleModule<"preferDestructuringAssignment", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;

package/dist/index.js

@@ -42,7 +42,7 @@ var ts__default = /*#__PURE__*/_interopDefault(ts);
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.26.3-next.6";
+var version = "1.26.3-next.8";
 var createRule = shared.createRuleForPlugin("x");
 
 // src/utils/get-associated-tokens.ts
@@ -1693,8 +1693,8 @@ var no_missing_key_default = createRule({
       [Symbol.for("rule_features")]: RULE_FEATURES29
     },
     messages: {
-      noMissingKey: "Missing 'key' for element when rendering list.",
-      noMissingKeyWithFragment: "Use fragment component instead of '<>' because it does not support `key`."
+      missingKey: "Missing 'key' for element when rendering list.",
+      unexpectedFragmentSyntax: "Use fragment component instead of '<>' because it does not support `key`."
     },
     schema: []
   },
@@ -1707,7 +1707,7 @@ var no_missing_key_default = createRule({
           const initialScope = context.sourceCode.getScope(node);
           if (!JSX7__namespace.hasAttribute("key", node.openingElement.attributes, initialScope)) {
             return {
-              messageId: "noMissingKey",
+              messageId: "missingKey",
               node
             };
           }
@@ -1715,7 +1715,7 @@ var no_missing_key_default = createRule({
         }
         case types.AST_NODE_TYPES.JSXFragment: {
           return {
-            messageId: "noMissingKeyWithFragment",
+            messageId: "unexpectedFragmentSyntax",
             node
           };
         }
@@ -1768,7 +1768,7 @@ var no_missing_key_default = createRule({
         for (const element of elements) {
           if (!JSX7__namespace.hasAttribute("key", element.openingElement.attributes, initialScope)) {
             context.report({
-              messageId: "noMissingKey",
+              messageId: "missingKey",
               node: element
             });
           }
@@ -1810,7 +1810,7 @@ var no_missing_key_default = createRule({
         }
         if (node.parent.type === types.AST_NODE_TYPES.ArrayExpression) {
           context.report({
-            messageId: "noMissingKeyWithFragment",
+            messageId: "unexpectedFragmentSyntax",
             node
           });
         }
@@ -2364,9 +2364,7 @@ var no_unstable_context_value_default = createRule({
       [Symbol.for("rule_features")]: RULE_FEATURES40
     },
     messages: {
-      noUnstableContextValue: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render.",
-      noUnstableContextValueWithFunction: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render. Consider wrapping it in a useCallback hook.",
-      noUnstableContextValueWithIdentifier: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render. Consider wrapping it in a useMemo hook."
+      unstableContextValue: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render. {{suggestion}}"
     },
     schema: []
   },
@@ -2406,12 +2404,13 @@ var no_unstable_context_value_default = createRule({
         for (const { node: component } of components) {
           for (const construction of constructions.get(component) ?? []) {
             const { kind, node: constructionNode } = construction;
-            const messageId = kind.startsWith("Function") ? "noUnstableContextValueWithFunction" : "noUnstableContextValueWithIdentifier";
+            const suggestion = kind.startsWith("Function") ? "Consider wrapping it in a useCallback hook." : "Consider wrapping it in a useMemo hook.";
             context.report({
-              messageId,
+              messageId: "unstableContextValue",
               node: constructionNode,
               data: {
-                type: AST11__namespace.toReadableNodeType(constructionNode)
+                type: AST11__namespace.toReadableNodeType(constructionNode),
+                suggestion
               }
             });
           }
@@ -2950,10 +2949,24 @@ function checkAndReport(node, context, allowExpressions) {
     return;
   }
   if (JSX7__namespace.isBuiltInElement(node.parent)) {
-    context.report({ messageId: "noUselessFragmentInBuiltIn", node, fix });
+    context.report({
+      messageId: "uselessFragment",
+      node,
+      data: {
+        reason: "placed inside a built-in component"
+      },
+      fix
+    });
   }
   if (node.children.length === 0) {
-    context.report({ messageId: "noUselessFragment", node, fix });
+    context.report({
+      messageId: "uselessFragment",
+      node,
+      data: {
+        reason: "contains less than two children"
+      },
+      fix
+    });
     return;
   }
   const isChildElement = AST11__namespace.isOneOf([types.AST_NODE_TYPES.JSXElement, types.AST_NODE_TYPES.JSXFragment])(node.parent);
@@ -2964,11 +2977,25 @@ function checkAndReport(node, context, allowExpressions) {
     }
     // <Foo><>hello, world</></Foo>
     case (!allowExpressions && isChildElement): {
-      context.report({ messageId: "noUselessFragment", node, fix });
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix
+      });
       return;
     }
     case (!allowExpressions && !isChildElement && node.children.length === 1): {
-      context.report({ messageId: "noUselessFragment", node, fix });
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix
+      });
       return;
     }
   }
@@ -2977,7 +3004,14 @@ function checkAndReport(node, context, allowExpressions) {
   switch (true) {
     case nonPaddingChildren.length === 0:
     case (nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== types.AST_NODE_TYPES.JSXExpressionContainer): {
-      context.report({ messageId: "noUselessFragment", node, fix });
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix
+      });
       return;
     }
   }
@@ -2988,12 +3022,11 @@ var no_useless_fragment_default = createRule({
     type: "problem",
     defaultOptions: [...defaultOptions],
     docs: {
-      description: "disallow unnecessary fragments"
+      description: "disallow useless fragments"
     },
     fixable: "code",
     messages: {
-      noUselessFragment: "A fragment contains less than two children is unnecessary.",
-      noUselessFragmentInBuiltIn: "A fragment placed inside a built-in component is unnecessary."
+      uselessFragment: "A fragment {{reason}} is useless."
     },
     schema: [{
       type: "object",

package/dist/index.mjs

@@ -15,7 +15,7 @@ import { getTypeImmutability, isUnknown, isImmutable, isReadonlyShallow, isReado
 
 // package.json
 var name = "eslint-plugin-react-x";
-var version = "1.26.3-next.6";
+var version = "1.26.3-next.8";
 var createRule = createRuleForPlugin("x");
 
 // src/utils/get-associated-tokens.ts
@@ -1666,8 +1666,8 @@ var no_missing_key_default = createRule({
       [Symbol.for("rule_features")]: RULE_FEATURES29
     },
     messages: {
-      noMissingKey: "Missing 'key' for element when rendering list.",
-      noMissingKeyWithFragment: "Use fragment component instead of '<>' because it does not support `key`."
+      missingKey: "Missing 'key' for element when rendering list.",
+      unexpectedFragmentSyntax: "Use fragment component instead of '<>' because it does not support `key`."
     },
     schema: []
   },
@@ -1680,7 +1680,7 @@ var no_missing_key_default = createRule({
           const initialScope = context.sourceCode.getScope(node);
           if (!JSX7.hasAttribute("key", node.openingElement.attributes, initialScope)) {
             return {
-              messageId: "noMissingKey",
+              messageId: "missingKey",
               node
             };
           }
@@ -1688,7 +1688,7 @@ var no_missing_key_default = createRule({
         }
         case AST_NODE_TYPES.JSXFragment: {
           return {
-            messageId: "noMissingKeyWithFragment",
+            messageId: "unexpectedFragmentSyntax",
             node
           };
         }
@@ -1741,7 +1741,7 @@ var no_missing_key_default = createRule({
         for (const element of elements) {
           if (!JSX7.hasAttribute("key", element.openingElement.attributes, initialScope)) {
             context.report({
-              messageId: "noMissingKey",
+              messageId: "missingKey",
               node: element
             });
           }
@@ -1783,7 +1783,7 @@ var no_missing_key_default = createRule({
         }
         if (node.parent.type === AST_NODE_TYPES.ArrayExpression) {
           context.report({
-            messageId: "noMissingKeyWithFragment",
+            messageId: "unexpectedFragmentSyntax",
             node
           });
         }
@@ -2337,9 +2337,7 @@ var no_unstable_context_value_default = createRule({
       [Symbol.for("rule_features")]: RULE_FEATURES40
     },
     messages: {
-      noUnstableContextValue: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render.",
-      noUnstableContextValueWithFunction: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render. Consider wrapping it in a useCallback hook.",
-      noUnstableContextValueWithIdentifier: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render. Consider wrapping it in a useMemo hook."
+      unstableContextValue: "A/an '{{type}}' passed as the value prop to the context provider should not be constructed. It will change on every render. {{suggestion}}"
     },
     schema: []
   },
@@ -2379,12 +2377,13 @@ var no_unstable_context_value_default = createRule({
         for (const { node: component } of components) {
           for (const construction of constructions.get(component) ?? []) {
             const { kind, node: constructionNode } = construction;
-            const messageId = kind.startsWith("Function") ? "noUnstableContextValueWithFunction" : "noUnstableContextValueWithIdentifier";
+            const suggestion = kind.startsWith("Function") ? "Consider wrapping it in a useCallback hook." : "Consider wrapping it in a useMemo hook.";
             context.report({
-              messageId,
+              messageId: "unstableContextValue",
               node: constructionNode,
               data: {
-                type: AST11.toReadableNodeType(constructionNode)
+                type: AST11.toReadableNodeType(constructionNode),
+                suggestion
               }
             });
           }
@@ -2923,10 +2922,24 @@ function checkAndReport(node, context, allowExpressions) {
     return;
   }
   if (JSX7.isBuiltInElement(node.parent)) {
-    context.report({ messageId: "noUselessFragmentInBuiltIn", node, fix });
+    context.report({
+      messageId: "uselessFragment",
+      node,
+      data: {
+        reason: "placed inside a built-in component"
+      },
+      fix
+    });
   }
   if (node.children.length === 0) {
-    context.report({ messageId: "noUselessFragment", node, fix });
+    context.report({
+      messageId: "uselessFragment",
+      node,
+      data: {
+        reason: "contains less than two children"
+      },
+      fix
+    });
     return;
   }
   const isChildElement = AST11.isOneOf([AST_NODE_TYPES.JSXElement, AST_NODE_TYPES.JSXFragment])(node.parent);
@@ -2937,11 +2950,25 @@ function checkAndReport(node, context, allowExpressions) {
     }
     // <Foo><>hello, world</></Foo>
     case (!allowExpressions && isChildElement): {
-      context.report({ messageId: "noUselessFragment", node, fix });
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix
+      });
       return;
     }
     case (!allowExpressions && !isChildElement && node.children.length === 1): {
-      context.report({ messageId: "noUselessFragment", node, fix });
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix
+      });
       return;
     }
   }
@@ -2950,7 +2977,14 @@ function checkAndReport(node, context, allowExpressions) {
   switch (true) {
     case nonPaddingChildren.length === 0:
     case (nonPaddingChildren.length === 1 && firstNonPaddingChild?.type !== AST_NODE_TYPES.JSXExpressionContainer): {
-      context.report({ messageId: "noUselessFragment", node, fix });
+      context.report({
+        messageId: "uselessFragment",
+        node,
+        data: {
+          reason: "contains less than two children"
+        },
+        fix
+      });
       return;
     }
   }
@@ -2961,12 +2995,11 @@ var no_useless_fragment_default = createRule({
     type: "problem",
     defaultOptions: [...defaultOptions],
     docs: {
-      description: "disallow unnecessary fragments"
+      description: "disallow useless fragments"
     },
     fixable: "code",
     messages: {
-      noUselessFragment: "A fragment contains less than two children is unnecessary.",
-      noUselessFragmentInBuiltIn: "A fragment placed inside a built-in component is unnecessary."
+      uselessFragment: "A fragment {{reason}} is useless."
     },
     schema: [{
       type: "object",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "1.26.3-next.6",
+  "version": "1.26.3-next.8",
   "description": "A set of composable linting rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -50,12 +50,12 @@
     "is-immutable-type": "^5.0.1",
     "string-ts": "^2.2.1",
     "ts-pattern": "^5.6.2",
-    "@eslint-react/ast": "1.26.3-next.6",
-    "@eslint-react/core": "1.26.3-next.6",
-    "@eslint-react/jsx": "1.26.3-next.6",
-    "@eslint-react/eff": "1.26.3-next.6",
-    "@eslint-react/shared": "1.26.3-next.6",
-    "@eslint-react/var": "1.26.3-next.6"
+    "@eslint-react/ast": "1.26.3-next.8",
+    "@eslint-react/core": "1.26.3-next.8",
+    "@eslint-react/eff": "1.26.3-next.8",
+    "@eslint-react/jsx": "1.26.3-next.8",
+    "@eslint-react/shared": "1.26.3-next.8",
+    "@eslint-react/var": "1.26.3-next.8"
   },
   "devDependencies": {
     "@types/react": "^19.0.8",