eslint-plugin-react-x 1.24.0-next.14 → 1.24.0-next.16

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.24.0-next.14";
+var version = "1.24.0-next.16";
 var createRule = shared.createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -301,7 +301,7 @@ function isReactChildrenMethod(name2) {
   return reactChildrenMethod.some((method) => method === name2);
 }
 function isUsingReactChildren(node, context) {
-  const settings = shared.unsafeDecodeSettings(context.settings);
+  const { importSource = "react" } = shared.unsafeDecodeSettings(context.settings);
   const { callee } = node;
   if (!("property" in callee) || !("object" in callee) || !("name" in callee.property)) {
     return false;
@@ -314,7 +314,7 @@ function isUsingReactChildren(node, context) {
     return true;
   }
   if (callee.object.type === types.AST_NODE_TYPES.MemberExpression && "name" in callee.object.object) {
-    return core.isInitializedFromReact(callee.object.object.name, initialScope, settings.importSource);
+    return core.isInitializedFromReact(callee.object.object.name, importSource, initialScope);
   }
   return false;
 }
@@ -612,8 +612,8 @@ var no_children_prop_default = createRule({
       JSXElement(node) {
         const attribute = JSX7__namespace.getAttribute(
           "children",
-          context.sourceCode.getScope(node),
-          node.openingElement.attributes
+          node.openingElement.attributes,
+          context.sourceCode.getScope(node)
         );
         if (attribute != null) {
           context.report({
@@ -1438,7 +1438,7 @@ var no_implicit_key_default = createRule({
     return {
       JSXOpeningElement(node) {
         const initialScope = context.sourceCode.getScope(node);
-        const keyPropFound = JSX7__namespace.getAttribute("key", initialScope, node.attributes);
+        const keyPropFound = JSX7__namespace.getAttribute("key", node.attributes, initialScope);
         const keyPropOnElement = node.attributes.some(
           (n) => n.type === types.AST_NODE_TYPES.JSXAttribute && n.name.type === types.AST_NODE_TYPES.JSXIdentifier && n.name.name === "key"
         );
@@ -1702,7 +1702,7 @@ var no_missing_key_default = createRule({
       switch (node.type) {
         case types.AST_NODE_TYPES.JSXElement: {
           const initialScope = context.sourceCode.getScope(node);
-          if (!JSX7__namespace.hasAttribute("key", initialScope, node.openingElement.attributes)) {
+          if (!JSX7__namespace.hasAttribute("key", node.openingElement.attributes, initialScope)) {
             return {
               messageId: "noMissingKey",
               node
@@ -1763,7 +1763,7 @@ var no_missing_key_default = createRule({
         }
         const initialScope = context.sourceCode.getScope(node);
         for (const element of elements) {
-          if (!JSX7__namespace.hasAttribute("key", initialScope, element.openingElement.attributes)) {
+          if (!JSX7__namespace.hasAttribute("key", element.openingElement.attributes, initialScope)) {
             context.report({
               messageId: "noMissingKey",
               node: element
@@ -2893,9 +2893,7 @@ var no_useless_fragment_default = createRule({
     const { allowExpressions = true } = option;
     return {
       JSXElement(node) {
-        if (JSX7__namespace.getElementName(node).split(".").at(-1) !== "Fragment") {
-          return;
-        }
+        if (!JSX7__namespace.isFragmentElement(node)) return;
         checkAndReport(node, context, allowExpressions);
       },
       JSXFragment(node) {
@@ -3007,7 +3005,7 @@ var prefer_react_namespace_import_default = createRule({
   },
   name: RULE_NAME46,
   create(context) {
-    const importSource = shared.getSettingsFromContext(context).importSource ?? "react";
+    const { importSource } = shared.getSettingsFromContext(context);
     return {
       [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
         const hasOtherSpecifiers = node.parent.specifiers.length > 1;
@@ -3153,9 +3151,7 @@ var prefer_shorthand_fragment_default = createRule({
   create(context) {
     return {
       JSXElement(node) {
-        if (JSX7__namespace.getElementName(node).split(".").at(-1) !== "Fragment") {
-          return;
-        }
+        if (!JSX7__namespace.isFragmentElement(node)) return;
         const hasAttributes = node.openingElement.attributes.length > 0;
         if (hasAttributes) {
           return;

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.24.0-next.14";
+var version = "1.24.0-next.16";
 var createRule = createRuleForPlugin("x");
 
 // src/rules/avoid-shorthand-boolean.ts
@@ -274,7 +274,7 @@ function isReactChildrenMethod(name2) {
   return reactChildrenMethod.some((method) => method === name2);
 }
 function isUsingReactChildren(node, context) {
-  const settings = unsafeDecodeSettings(context.settings);
+  const { importSource = "react" } = unsafeDecodeSettings(context.settings);
   const { callee } = node;
   if (!("property" in callee) || !("object" in callee) || !("name" in callee.property)) {
     return false;
@@ -287,7 +287,7 @@ function isUsingReactChildren(node, context) {
     return true;
   }
   if (callee.object.type === AST_NODE_TYPES.MemberExpression && "name" in callee.object.object) {
-    return isInitializedFromReact(callee.object.object.name, initialScope, settings.importSource);
+    return isInitializedFromReact(callee.object.object.name, importSource, initialScope);
   }
   return false;
 }
@@ -585,8 +585,8 @@ var no_children_prop_default = createRule({
       JSXElement(node) {
         const attribute = JSX7.getAttribute(
           "children",
-          context.sourceCode.getScope(node),
-          node.openingElement.attributes
+          node.openingElement.attributes,
+          context.sourceCode.getScope(node)
         );
         if (attribute != null) {
           context.report({
@@ -1411,7 +1411,7 @@ var no_implicit_key_default = createRule({
     return {
       JSXOpeningElement(node) {
         const initialScope = context.sourceCode.getScope(node);
-        const keyPropFound = JSX7.getAttribute("key", initialScope, node.attributes);
+        const keyPropFound = JSX7.getAttribute("key", node.attributes, initialScope);
         const keyPropOnElement = node.attributes.some(
           (n) => n.type === AST_NODE_TYPES.JSXAttribute && n.name.type === AST_NODE_TYPES.JSXIdentifier && n.name.name === "key"
         );
@@ -1675,7 +1675,7 @@ var no_missing_key_default = createRule({
       switch (node.type) {
         case AST_NODE_TYPES.JSXElement: {
           const initialScope = context.sourceCode.getScope(node);
-          if (!JSX7.hasAttribute("key", initialScope, node.openingElement.attributes)) {
+          if (!JSX7.hasAttribute("key", node.openingElement.attributes, initialScope)) {
             return {
               messageId: "noMissingKey",
               node
@@ -1736,7 +1736,7 @@ var no_missing_key_default = createRule({
         }
         const initialScope = context.sourceCode.getScope(node);
         for (const element of elements) {
-          if (!JSX7.hasAttribute("key", initialScope, element.openingElement.attributes)) {
+          if (!JSX7.hasAttribute("key", element.openingElement.attributes, initialScope)) {
             context.report({
               messageId: "noMissingKey",
               node: element
@@ -2866,9 +2866,7 @@ var no_useless_fragment_default = createRule({
     const { allowExpressions = true } = option;
     return {
       JSXElement(node) {
-        if (JSX7.getElementName(node).split(".").at(-1) !== "Fragment") {
-          return;
-        }
+        if (!JSX7.isFragmentElement(node)) return;
         checkAndReport(node, context, allowExpressions);
       },
       JSXFragment(node) {
@@ -2980,7 +2978,7 @@ var prefer_react_namespace_import_default = createRule({
   },
   name: RULE_NAME46,
   create(context) {
-    const importSource = getSettingsFromContext(context).importSource ?? "react";
+    const { importSource } = getSettingsFromContext(context);
     return {
       [`ImportDeclaration[source.value="${importSource}"] ImportDefaultSpecifier`](node) {
         const hasOtherSpecifiers = node.parent.specifiers.length > 1;
@@ -3126,9 +3124,7 @@ var prefer_shorthand_fragment_default = createRule({
   create(context) {
     return {
       JSXElement(node) {
-        if (JSX7.getElementName(node).split(".").at(-1) !== "Fragment") {
-          return;
-        }
+        if (!JSX7.isFragmentElement(node)) return;
         const hasAttributes = node.openingElement.attributes.length > 0;
         if (hasAttributes) {
           return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-x",
-  "version": "1.24.0-next.14",
+  "version": "1.24.0-next.16",
   "description": "A set of composable ESLint rules for libraries and frameworks that use React as a UI runtime.",
   "keywords": [
     "react",
@@ -48,12 +48,12 @@
     "is-immutable-type": "^5.0.1",
     "string-ts": "^2.2.0",
     "ts-pattern": "^5.6.0",
-    "@eslint-react/ast": "1.24.0-next.14",
-    "@eslint-react/eff": "1.24.0-next.14",
-    "@eslint-react/jsx": "1.24.0-next.14",
-    "@eslint-react/core": "1.24.0-next.14",
-    "@eslint-react/shared": "1.24.0-next.14",
-    "@eslint-react/var": "1.24.0-next.14"
+    "@eslint-react/ast": "1.24.0-next.16",
+    "@eslint-react/eff": "1.24.0-next.16",
+    "@eslint-react/core": "1.24.0-next.16",
+    "@eslint-react/jsx": "1.24.0-next.16",
+    "@eslint-react/shared": "1.24.0-next.16",
+    "@eslint-react/var": "1.24.0-next.16"
   },
   "devDependencies": {
     "@types/react": "^19.0.7",