eslint-plugin-react-dom 1.24.0-beta.3 → 1.24.0-beta.8

package/dist/index.js

@@ -2,12 +2,11 @@
 
 var JSX3 = require('@eslint-react/jsx');
 var eff = require('@eslint-react/eff');
-var VAR = require('@eslint-react/var');
-var tsPattern = require('ts-pattern');
 var shared = require('@eslint-react/shared');
 var types = require('@typescript-eslint/types');
 var core = require('@eslint-react/core');
 var AST = require('@eslint-react/ast');
+var VAR = require('@eslint-react/var');
 var compareVersions = require('compare-versions');
 
 function _interopNamespace(e) {
@@ -29,28 +28,44 @@ function _interopNamespace(e) {
 }
 
 var JSX3__namespace = /*#__PURE__*/_interopNamespace(JSX3);
-var VAR__namespace = /*#__PURE__*/_interopNamespace(VAR);
 var AST__namespace = /*#__PURE__*/_interopNamespace(AST);
+var VAR__namespace = /*#__PURE__*/_interopNamespace(VAR);
 
 // package.json
 var name = "eslint-plugin-react-dom";
-var version = "1.24.0-beta.3";
-function getAdditionalAttributes(name2, additionalComponents) {
-  return additionalComponents.findLast((c) => c.name === name2 || c.re.test(name2))?.attributes ?? [];
+var version = "1.24.0-beta.8";
+function getAdditionalAttributes(componentName, additionalComponents) {
+  return additionalComponents.findLast((c) => c.name === componentName || c.re.test(componentName))?.attributes ?? [];
 }
-function getAttributeStringValue(attributeName, element, context, additionalAttributes) {
-  const attributeConfig = additionalAttributes.findLast((attr) => attr.as === attributeName);
-  const actualAttributeName = attributeConfig?.name ?? attributeName;
-  const actualAttributeNode = JSX3__namespace.findPropInAttributes(
-    actualAttributeName,
+function getAttributeMetaData(name2, attributes) {
+  const settings = attributes.findLast((a) => a.as === name2);
+  return {
+    name: settings?.name ?? name2,
+    controlled: settings?.controlled ?? false,
+    defaultValue: settings?.defaultValue
+  };
+}
+function getAttributeNodeAndStringValue(name2, element, context, additionalAttributes) {
+  const attributeMetaData = getAttributeMetaData(name2, additionalAttributes);
+  const attributeNameOnElement = attributeMetaData.name;
+  const attributeNode = JSX3__namespace.getAttributeNode(
+    attributeNameOnElement,
     context.sourceCode.getScope(element),
     element.openingElement.attributes
   );
-  if (actualAttributeNode == null) return attributeConfig?.defaultValue;
-  const actualAttributeScope = context.sourceCode.getScope(actualAttributeNode);
-  const actualAttributeValue = JSX3__namespace.getPropValue(actualAttributeNode, actualAttributeScope);
-  const actualAttributeValueResolved = VAR__namespace.toResolved(actualAttributeValue).value;
-  return tsPattern.match(actualAttributeValueResolved).with(tsPattern.P.string, eff.identity).with({ [actualAttributeName]: tsPattern.P.select(tsPattern.P.string) }, eff.identity).otherwise(() => eff._);
+  if (attributeNode == null) {
+    return {
+      attributeNode: eff._,
+      attributeValue: attributeMetaData.defaultValue
+    };
+  }
+  const attributeScope = context.sourceCode.getScope(attributeNode);
+  const attributeValue = JSX3__namespace.getAttributeStringValue(
+    attributeNameOnElement,
+    attributeNode,
+    attributeScope
+  );
+  return { attributeNode, attributeValue };
 }
 var createRule = shared.createRuleForPlugin("dom");
 
@@ -73,14 +88,19 @@ var no_dangerously_set_innerhtml_default = createRule({
   },
   name: RULE_NAME,
   create(context) {
+    if (!context.sourceCode.text.includes("dangerouslySetInnerHTML")) return {};
     return {
       JSXElement(node) {
         const attributes = node.openingElement.attributes;
-        const prop = JSX3__namespace.getProp("dangerouslySetInnerHTML", context.sourceCode.getScope(node), attributes);
-        if (prop == null) return;
+        const attribute = JSX3__namespace.getAttributeNode(
+          "dangerouslySetInnerHTML",
+          context.sourceCode.getScope(node),
+          attributes
+        );
+        if (attribute == null) return;
         context.report({
           messageId: "noDangerouslySetInnerhtml",
-          node: prop
+          node: attribute
         });
       }
     };
@@ -105,13 +125,13 @@ var no_dangerously_set_innerhtml_with_children_default = createRule({
   },
   name: RULE_NAME2,
   create(context) {
+    if (!context.sourceCode.text.includes("dangerouslySetInnerHTML")) return {};
     return {
       JSXElement(node) {
         const attributes = node.openingElement.attributes;
         const initialScope = context.sourceCode.getScope(node);
-        const hasChildren = hasChildrenWithin(node) || JSX3__namespace.hasProp("children", initialScope, attributes);
-        const hasDangerouslySetInnerHTML = JSX3__namespace.hasProp("dangerouslySetInnerHTML", initialScope, attributes);
-        if (hasChildren && hasDangerouslySetInnerHTML) {
+        const hasChildren = hasChildrenWithin(node) || JSX3__namespace.hasAttribute("children", initialScope, attributes);
+        if (hasChildren && JSX3__namespace.hasAttribute("dangerouslySetInnerHTML", initialScope, attributes)) {
           context.report({
             messageId: "noDangerouslySetInnerhtmlWithChildren",
             node
@@ -143,6 +163,7 @@ var no_find_dom_node_default = createRule({
   },
   name: RULE_NAME3,
   create(context) {
+    if (!context.sourceCode.text.includes("findDOMNode")) return {};
     return {
       CallExpression(node) {
         const { callee } = node;
@@ -182,29 +203,27 @@ var no_missing_button_type_default = createRule({
   name: RULE_NAME4,
   create(context) {
     const settings = shared.getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "button");
     return {
       JSXElement(node) {
-        const [name2, representName] = core.getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = core.getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "button") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
-            node,
-            context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
-        };
-        const prop = getPropValue3("type");
-        if (typeof prop !== "string") {
+        if (elementNameOnHtml !== "button") return;
+        const { attributeNode, attributeValue } = getAttributeNodeAndStringValue(
+          "type",
+          node,
+          context,
+          getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+        );
+        if (typeof attributeValue !== "string") {
           context.report({
             messageId: "noMissingButtonType",
-            node
+            node: attributeNode ?? node
           });
         }
       }
@@ -249,39 +268,29 @@ var no_missing_iframe_sandbox_default = createRule({
   name: RULE_NAME5,
   create(context) {
     const settings = shared.getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "iframe");
     return {
       JSXElement(node) {
-        const [name2, representName] = core.getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = core.getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "iframe") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
-            node,
-            context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
-        };
-        const sandboxValue = getPropValue3("sandbox");
-        if (sandboxValue == null) {
-          context.report({
-            messageId: "noMissingIframeSandbox",
-            node: node.openingElement
-          });
-          return;
-        }
-        const values = sandboxValue.split(" ");
-        if (!values.every((value) => validTypes.some((validType) => validType === value))) {
-          context.report({
-            messageId: "noMissingIframeSandbox",
-            node: node.openingElement
-          });
-        }
+        if (elementNameOnHtml !== "iframe") return;
+        const { attributeNode, attributeValue } = getAttributeNodeAndStringValue(
+          "sandbox",
+          node,
+          context,
+          getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+        );
+        const hasValidSandboxValue = typeof attributeValue === "string" && attributeValue.split(" ").every((value) => validTypes.some((valid) => valid === value));
+        if (hasValidSandboxValue) return;
+        context.report({
+          messageId: "noMissingIframeSandbox",
+          node: attributeNode ?? node.openingElement
+        });
       }
     };
   },
@@ -399,11 +408,11 @@ var no_script_url_default = createRule({
         if (node.name.type !== types.AST_NODE_TYPES.JSXIdentifier || node.value == null) {
           return;
         }
-        const propScope = context.sourceCode.getScope(node);
-        const propValue = JSX3__namespace.getPropValue(node, propScope);
-        const propValueResolved = VAR__namespace.toResolved(propValue).value;
-        if (typeof propValueResolved !== "string") return;
-        if (shared.RE_JAVASCRIPT_PROTOCOL.test(propValueResolved)) {
+        const attributeScope = context.sourceCode.getScope(node);
+        const attributeValue = JSX3__namespace.getAttributeStaticValue(node, attributeScope);
+        const attributeValueResolved = VAR__namespace.toResolved(attributeValue).value;
+        if (typeof attributeValueResolved !== "string") return;
+        if (shared.RE_JAVASCRIPT_PROTOCOL.test(attributeValueResolved)) {
           context.report({
             messageId: "noScriptUrl",
             node: node.value
@@ -1493,33 +1502,29 @@ var no_unsafe_iframe_sandbox_default = createRule({
   name: RULE_NAME10,
   create(context) {
     const settings = shared.getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "iframe");
     return {
       JSXElement(node) {
-        const [name2, representName] = core.getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = core.getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "iframe") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
-            node,
-            context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
-        };
-        const sandboxValue = getPropValue3("sandbox");
-        if (sandboxValue == null) return;
-        const values = sandboxValue.split(" ");
-        if (unsafeCombinations.some((unsafes) => unsafes.every((x) => values.includes(x)))) {
-          context.report({
-            messageId: "noUnsafeIframeSandbox",
-            node: node.openingElement
-          });
-        }
+        if (elementNameOnHtml !== "iframe") return;
+        const { attributeNode, attributeValue } = getAttributeNodeAndStringValue(
+          "sandbox",
+          node,
+          context,
+          getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+        );
+        if (attributeValue == null) return;
+        if (!unsafeCombinations.some((c) => c.every((v) => attributeValue.includes(v)))) return;
+        context.report({
+          messageId: "noUnsafeIframeSandbox",
+          node: attributeNode ?? node.openingElement
+        });
       }
     };
   },
@@ -1552,36 +1557,38 @@ var no_unsafe_target_blank_default = createRule({
   name: RULE_NAME11,
   create(context) {
     const settings = shared.getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "a");
     return {
       JSXElement(node) {
-        const [name2, representName] = core.getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = core.getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "a") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
+        if (elementNameOnHtml !== "a") return;
+        const getAttributeValue = (name2) => {
+          return getAttributeNodeAndStringValue(
+            name2,
             node,
             context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
+            getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+          ).attributeValue;
         };
-        if (getPropValue3("target") !== "_blank") {
+        if (getAttributeValue("target") !== "_blank") {
           return;
         }
-        if (!isExternalLinkLike(getPropValue3("href"))) {
+        if (!isExternalLinkLike(getAttributeValue("href"))) {
           return;
         }
-        if (!isSafeRel(getPropValue3("rel"))) {
-          context.report({
-            messageId: "noUnsafeTargetBlank",
-            node
-          });
+        if (isSafeRel(getAttributeValue("rel"))) {
+          return;
         }
+        context.report({
+          messageId: "noUnsafeTargetBlank",
+          node
+        });
       }
     };
   },
@@ -1640,8 +1647,8 @@ var no_void_elements_with_children_default = createRule({
         }
         const { attributes } = node.openingElement;
         const initialScope = context.sourceCode.getScope(node);
-        const hasAttribute = (name2) => JSX3__namespace.findPropInAttributes(name2, initialScope, attributes) != null;
-        if (hasAttribute("children") || hasAttribute("dangerouslySetInnerHTML")) {
+        const hasAttribute3 = (name2) => JSX3__namespace.hasAttribute(name2, initialScope, attributes);
+        if (hasAttribute3("children") || hasAttribute3("dangerouslySetInnerHTML")) {
           context.report({
             messageId: "noVoidElementsWithChildren",
             node,

package/dist/index.mjs

@@ -1,32 +1,47 @@
 import * as JSX3 from '@eslint-react/jsx';
-import { identity, _ } from '@eslint-react/eff';
-import * as VAR from '@eslint-react/var';
-import { match, P } from 'ts-pattern';
+import { _ } from '@eslint-react/eff';
 import { createRuleForPlugin, getSettingsFromContext, RE_JAVASCRIPT_PROTOCOL } from '@eslint-react/shared';
 import { AST_NODE_TYPES } from '@typescript-eslint/types';
-import { getElementNameAndRepresentName } from '@eslint-react/core';
+import { getElementNameOnJsxAndHtml } from '@eslint-react/core';
 import * as AST from '@eslint-react/ast';
+import * as VAR from '@eslint-react/var';
 import { compare } from 'compare-versions';
 
 // package.json
 var name = "eslint-plugin-react-dom";
-var version = "1.24.0-beta.3";
-function getAdditionalAttributes(name2, additionalComponents) {
-  return additionalComponents.findLast((c) => c.name === name2 || c.re.test(name2))?.attributes ?? [];
+var version = "1.24.0-beta.8";
+function getAdditionalAttributes(componentName, additionalComponents) {
+  return additionalComponents.findLast((c) => c.name === componentName || c.re.test(componentName))?.attributes ?? [];
+}
+function getAttributeMetaData(name2, attributes) {
+  const settings = attributes.findLast((a) => a.as === name2);
+  return {
+    name: settings?.name ?? name2,
+    controlled: settings?.controlled ?? false,
+    defaultValue: settings?.defaultValue
+  };
 }
-function getAttributeStringValue(attributeName, element, context, additionalAttributes) {
-  const attributeConfig = additionalAttributes.findLast((attr) => attr.as === attributeName);
-  const actualAttributeName = attributeConfig?.name ?? attributeName;
-  const actualAttributeNode = JSX3.findPropInAttributes(
-    actualAttributeName,
+function getAttributeNodeAndStringValue(name2, element, context, additionalAttributes) {
+  const attributeMetaData = getAttributeMetaData(name2, additionalAttributes);
+  const attributeNameOnElement = attributeMetaData.name;
+  const attributeNode = JSX3.getAttributeNode(
+    attributeNameOnElement,
     context.sourceCode.getScope(element),
     element.openingElement.attributes
   );
-  if (actualAttributeNode == null) return attributeConfig?.defaultValue;
-  const actualAttributeScope = context.sourceCode.getScope(actualAttributeNode);
-  const actualAttributeValue = JSX3.getPropValue(actualAttributeNode, actualAttributeScope);
-  const actualAttributeValueResolved = VAR.toResolved(actualAttributeValue).value;
-  return match(actualAttributeValueResolved).with(P.string, identity).with({ [actualAttributeName]: P.select(P.string) }, identity).otherwise(() => _);
+  if (attributeNode == null) {
+    return {
+      attributeNode: _,
+      attributeValue: attributeMetaData.defaultValue
+    };
+  }
+  const attributeScope = context.sourceCode.getScope(attributeNode);
+  const attributeValue = JSX3.getAttributeStringValue(
+    attributeNameOnElement,
+    attributeNode,
+    attributeScope
+  );
+  return { attributeNode, attributeValue };
 }
 var createRule = createRuleForPlugin("dom");
 
@@ -49,14 +64,19 @@ var no_dangerously_set_innerhtml_default = createRule({
   },
   name: RULE_NAME,
   create(context) {
+    if (!context.sourceCode.text.includes("dangerouslySetInnerHTML")) return {};
     return {
       JSXElement(node) {
         const attributes = node.openingElement.attributes;
-        const prop = JSX3.getProp("dangerouslySetInnerHTML", context.sourceCode.getScope(node), attributes);
-        if (prop == null) return;
+        const attribute = JSX3.getAttributeNode(
+          "dangerouslySetInnerHTML",
+          context.sourceCode.getScope(node),
+          attributes
+        );
+        if (attribute == null) return;
         context.report({
           messageId: "noDangerouslySetInnerhtml",
-          node: prop
+          node: attribute
         });
       }
     };
@@ -81,13 +101,13 @@ var no_dangerously_set_innerhtml_with_children_default = createRule({
   },
   name: RULE_NAME2,
   create(context) {
+    if (!context.sourceCode.text.includes("dangerouslySetInnerHTML")) return {};
     return {
       JSXElement(node) {
         const attributes = node.openingElement.attributes;
         const initialScope = context.sourceCode.getScope(node);
-        const hasChildren = hasChildrenWithin(node) || JSX3.hasProp("children", initialScope, attributes);
-        const hasDangerouslySetInnerHTML = JSX3.hasProp("dangerouslySetInnerHTML", initialScope, attributes);
-        if (hasChildren && hasDangerouslySetInnerHTML) {
+        const hasChildren = hasChildrenWithin(node) || JSX3.hasAttribute("children", initialScope, attributes);
+        if (hasChildren && JSX3.hasAttribute("dangerouslySetInnerHTML", initialScope, attributes)) {
           context.report({
             messageId: "noDangerouslySetInnerhtmlWithChildren",
             node
@@ -119,6 +139,7 @@ var no_find_dom_node_default = createRule({
   },
   name: RULE_NAME3,
   create(context) {
+    if (!context.sourceCode.text.includes("findDOMNode")) return {};
     return {
       CallExpression(node) {
         const { callee } = node;
@@ -158,29 +179,27 @@ var no_missing_button_type_default = createRule({
   name: RULE_NAME4,
   create(context) {
     const settings = getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "button");
     return {
       JSXElement(node) {
-        const [name2, representName] = getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "button") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
-            node,
-            context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
-        };
-        const prop = getPropValue3("type");
-        if (typeof prop !== "string") {
+        if (elementNameOnHtml !== "button") return;
+        const { attributeNode, attributeValue } = getAttributeNodeAndStringValue(
+          "type",
+          node,
+          context,
+          getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+        );
+        if (typeof attributeValue !== "string") {
           context.report({
             messageId: "noMissingButtonType",
-            node
+            node: attributeNode ?? node
           });
         }
       }
@@ -225,39 +244,29 @@ var no_missing_iframe_sandbox_default = createRule({
   name: RULE_NAME5,
   create(context) {
     const settings = getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "iframe");
     return {
       JSXElement(node) {
-        const [name2, representName] = getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "iframe") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
-            node,
-            context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
-        };
-        const sandboxValue = getPropValue3("sandbox");
-        if (sandboxValue == null) {
-          context.report({
-            messageId: "noMissingIframeSandbox",
-            node: node.openingElement
-          });
-          return;
-        }
-        const values = sandboxValue.split(" ");
-        if (!values.every((value) => validTypes.some((validType) => validType === value))) {
-          context.report({
-            messageId: "noMissingIframeSandbox",
-            node: node.openingElement
-          });
-        }
+        if (elementNameOnHtml !== "iframe") return;
+        const { attributeNode, attributeValue } = getAttributeNodeAndStringValue(
+          "sandbox",
+          node,
+          context,
+          getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+        );
+        const hasValidSandboxValue = typeof attributeValue === "string" && attributeValue.split(" ").every((value) => validTypes.some((valid) => valid === value));
+        if (hasValidSandboxValue) return;
+        context.report({
+          messageId: "noMissingIframeSandbox",
+          node: attributeNode ?? node.openingElement
+        });
       }
     };
   },
@@ -375,11 +384,11 @@ var no_script_url_default = createRule({
         if (node.name.type !== AST_NODE_TYPES.JSXIdentifier || node.value == null) {
           return;
         }
-        const propScope = context.sourceCode.getScope(node);
-        const propValue = JSX3.getPropValue(node, propScope);
-        const propValueResolved = VAR.toResolved(propValue).value;
-        if (typeof propValueResolved !== "string") return;
-        if (RE_JAVASCRIPT_PROTOCOL.test(propValueResolved)) {
+        const attributeScope = context.sourceCode.getScope(node);
+        const attributeValue = JSX3.getAttributeStaticValue(node, attributeScope);
+        const attributeValueResolved = VAR.toResolved(attributeValue).value;
+        if (typeof attributeValueResolved !== "string") return;
+        if (RE_JAVASCRIPT_PROTOCOL.test(attributeValueResolved)) {
           context.report({
             messageId: "noScriptUrl",
             node: node.value
@@ -1469,33 +1478,29 @@ var no_unsafe_iframe_sandbox_default = createRule({
   name: RULE_NAME10,
   create(context) {
     const settings = getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "iframe");
     return {
       JSXElement(node) {
-        const [name2, representName] = getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "iframe") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
-            node,
-            context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
-        };
-        const sandboxValue = getPropValue3("sandbox");
-        if (sandboxValue == null) return;
-        const values = sandboxValue.split(" ");
-        if (unsafeCombinations.some((unsafes) => unsafes.every((x) => values.includes(x)))) {
-          context.report({
-            messageId: "noUnsafeIframeSandbox",
-            node: node.openingElement
-          });
-        }
+        if (elementNameOnHtml !== "iframe") return;
+        const { attributeNode, attributeValue } = getAttributeNodeAndStringValue(
+          "sandbox",
+          node,
+          context,
+          getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+        );
+        if (attributeValue == null) return;
+        if (!unsafeCombinations.some((c) => c.every((v) => attributeValue.includes(v)))) return;
+        context.report({
+          messageId: "noUnsafeIframeSandbox",
+          node: attributeNode ?? node.openingElement
+        });
       }
     };
   },
@@ -1528,36 +1533,38 @@ var no_unsafe_target_blank_default = createRule({
   name: RULE_NAME11,
   create(context) {
     const settings = getSettingsFromContext(context);
+    const polymorphicPropName = settings.polymorphicPropName;
     const additionalComponents = settings.additionalComponents.filter((c) => c.as === "a");
     return {
       JSXElement(node) {
-        const [name2, representName] = getElementNameAndRepresentName(
+        const [elementNameOnJsx, elementNameOnHtml] = getElementNameOnJsxAndHtml(
           node.openingElement,
           context,
-          settings.polymorphicPropName,
-          settings.additionalComponents
+          polymorphicPropName,
+          additionalComponents
         );
-        if (representName !== "a") return;
-        const getPropValue3 = (propName) => {
-          return getAttributeStringValue(
-            propName,
+        if (elementNameOnHtml !== "a") return;
+        const getAttributeValue = (name2) => {
+          return getAttributeNodeAndStringValue(
+            name2,
             node,
             context,
-            getAdditionalAttributes(name2, additionalComponents)
-          );
+            getAdditionalAttributes(elementNameOnJsx, additionalComponents)
+          ).attributeValue;
         };
-        if (getPropValue3("target") !== "_blank") {
+        if (getAttributeValue("target") !== "_blank") {
           return;
         }
-        if (!isExternalLinkLike(getPropValue3("href"))) {
+        if (!isExternalLinkLike(getAttributeValue("href"))) {
           return;
         }
-        if (!isSafeRel(getPropValue3("rel"))) {
-          context.report({
-            messageId: "noUnsafeTargetBlank",
-            node
-          });
+        if (isSafeRel(getAttributeValue("rel"))) {
+          return;
         }
+        context.report({
+          messageId: "noUnsafeTargetBlank",
+          node
+        });
       }
     };
   },
@@ -1616,8 +1623,8 @@ var no_void_elements_with_children_default = createRule({
         }
         const { attributes } = node.openingElement;
         const initialScope = context.sourceCode.getScope(node);
-        const hasAttribute = (name2) => JSX3.findPropInAttributes(name2, initialScope, attributes) != null;
-        if (hasAttribute("children") || hasAttribute("dangerouslySetInnerHTML")) {
+        const hasAttribute3 = (name2) => JSX3.hasAttribute(name2, initialScope, attributes);
+        if (hasAttribute3("children") || hasAttribute3("dangerouslySetInnerHTML")) {
           context.report({
             messageId: "noVoidElementsWithChildren",
             node,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-dom",
-  "version": "1.24.0-beta.3",
+  "version": "1.24.0-beta.8",
   "description": "ESLint React's ESLint plugin for React DOM related rules.",
   "keywords": [
     "react",
@@ -41,22 +41,21 @@
     "./package.json"
   ],
   "dependencies": {
-    "@typescript-eslint/scope-manager": "^8.19.1",
-    "@typescript-eslint/types": "^8.19.1",
-    "@typescript-eslint/utils": "^8.19.1",
+    "@typescript-eslint/scope-manager": "^8.20.0",
+    "@typescript-eslint/types": "^8.20.0",
+    "@typescript-eslint/utils": "^8.20.0",
     "compare-versions": "^6.1.1",
     "string-ts": "^2.2.0",
     "ts-pattern": "^5.6.0",
-    "@eslint-react/ast": "1.24.0-beta.3",
-    "@eslint-react/eff": "1.24.0-beta.3",
-    "@eslint-react/core": "1.24.0-beta.3",
-    "@eslint-react/jsx": "1.24.0-beta.3",
-    "@eslint-react/types": "1.24.0-beta.3",
-    "@eslint-react/shared": "1.24.0-beta.3",
-    "@eslint-react/var": "1.24.0-beta.3"
+    "@eslint-react/ast": "1.24.0-beta.8",
+    "@eslint-react/jsx": "1.24.0-beta.8",
+    "@eslint-react/eff": "1.24.0-beta.8",
+    "@eslint-react/var": "1.24.0-beta.8",
+    "@eslint-react/core": "1.24.0-beta.8",
+    "@eslint-react/shared": "1.24.0-beta.8"
   },
   "devDependencies": {
-    "@types/react": "^19.0.6",
+    "@types/react": "^19.0.7",
     "@types/react-dom": "^19.0.3",
     "tsup": "^8.3.5",
     "@workspace/configs": "0.0.0"