eslint-plugin-react-dom 2.0.0-next.5 → 2.0.0-next.51

package/dist/index.d.ts

@@ -1,6 +1,15 @@
+import { CamelCase } from 'string-ts';
 import * as _typescript_eslint_utils_ts_eslint from '@typescript-eslint/utils/ts-eslint';
 import { RulePreset } from '@eslint-react/kit';
 
+declare const RULE_NAME$1 = "no-missing-iframe-sandbox";
+type MessageID$1 = CamelCase<typeof RULE_NAME$1> | RuleSuggestMessageID$1;
+type RuleSuggestMessageID$1 = "addIframeSandbox";
+
+declare const RULE_NAME = "no-missing-button-type";
+type MessageID = CamelCase<typeof RULE_NAME> | RuleSuggestMessageID;
+type RuleSuggestMessageID = "addButtonType";
+
 declare const _default: {
     configs: {
         recommended: {
@@ -16,8 +25,8 @@ declare const _default: {
                         readonly "no-find-dom-node": _typescript_eslint_utils_ts_eslint.RuleModule<"noFindDomNode", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-flush-sync": _typescript_eslint_utils_ts_eslint.RuleModule<"noFlushSync", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-hydrate": _typescript_eslint_utils_ts_eslint.RuleModule<"noHydrate", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-                        readonly "no-missing-button-type": _typescript_eslint_utils_ts_eslint.RuleModule<"noMissingButtonType", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-                        readonly "no-missing-iframe-sandbox": _typescript_eslint_utils_ts_eslint.RuleModule<"noMissingIframeSandbox", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-missing-button-type": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-missing-iframe-sandbox": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$1, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-namespace": _typescript_eslint_utils_ts_eslint.RuleModule<"noNamespace", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-render": _typescript_eslint_utils_ts_eslint.RuleModule<"noRender", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-render-return-value": _typescript_eslint_utils_ts_eslint.RuleModule<"noRenderReturnValue", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
@@ -48,8 +57,8 @@ declare const _default: {
         readonly "no-find-dom-node": _typescript_eslint_utils_ts_eslint.RuleModule<"noFindDomNode", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-flush-sync": _typescript_eslint_utils_ts_eslint.RuleModule<"noFlushSync", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-hydrate": _typescript_eslint_utils_ts_eslint.RuleModule<"noHydrate", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        readonly "no-missing-button-type": _typescript_eslint_utils_ts_eslint.RuleModule<"noMissingButtonType", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        readonly "no-missing-iframe-sandbox": _typescript_eslint_utils_ts_eslint.RuleModule<"noMissingIframeSandbox", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "no-missing-button-type": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "no-missing-iframe-sandbox": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$1, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-namespace": _typescript_eslint_utils_ts_eslint.RuleModule<"noNamespace", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-render": _typescript_eslint_utils_ts_eslint.RuleModule<"noRender", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-render-return-value": _typescript_eslint_utils_ts_eslint.RuleModule<"noRenderReturnValue", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;

package/dist/index.js

@@ -5,7 +5,7 @@ import * as AST from '@eslint-react/ast';
 import { AST_NODE_TYPES } from '@typescript-eslint/types';
 import { compare } from 'compare-versions';
 import { RegExp, Reporter } from '@eslint-react/kit';
-import { _ } from '@eslint-react/eff';
+import { unit } from '@eslint-react/eff';
 
 var __defProp = Object.defineProperty;
 var __export = (target, all) => {
@@ -44,7 +44,7 @@ var settings = {
 
 // package.json
 var name2 = "eslint-plugin-react-dom";
-var version = "2.0.0-next.5";
+var version = "2.0.0-next.51";
 function createJsxElementResolver(context) {
   const { components, polymorphicPropName } = getSettingsFromContext(context);
   return {
@@ -323,7 +323,10 @@ function getFix(context, node) {
   };
 }
 var RULE_NAME6 = "no-missing-button-type";
-var RULE_FEATURES6 = [];
+var RULE_FEATURES6 = [
+  "FIX"
+];
+var BUTTON_TYPES = ["button", "submit", "reset"];
 var no_missing_button_type_default = createRule({
   meta: {
     type: "problem",
@@ -331,7 +334,9 @@ var no_missing_button_type_default = createRule({
       description: "Enforces explicit `type` attribute for `button` elements.",
       [Symbol.for("rule_features")]: RULE_FEATURES6
     },
+    hasSuggestions: true,
     messages: {
+      addButtonType: "Add 'type' attribute with value '{{type}}'.",
       noMissingButtonType: "Add missing 'type' attribute on 'button' component."
     },
     schema: []
@@ -360,10 +365,13 @@ function create6(context) {
           attributeNode,
           propNameOnJsx
         );
-        if (attributeValue.kind === "some" && typeof attributeValue.value !== "string") {
+        if (attributeValue.kind !== "some" || typeof attributeValue.value !== "string") {
           context.report({
             messageId: "noMissingButtonType",
-            node: attributeNode
+            node: attributeNode,
+            suggest: getSuggest((type) => (fixer) => {
+              return fixer.replaceText(attributeNode, `${propNameOnJsx}="${type}"`);
+            })
           });
         }
         return;
@@ -371,35 +379,26 @@ function create6(context) {
       if (typeof customComponentProp?.defaultValue !== "string") {
         context.report({
           messageId: "noMissingButtonType",
-          node
+          node,
+          suggest: getSuggest((type) => (fixer) => {
+            return fixer.insertTextAfter(node.openingElement.name, ` type="${type}"`);
+          })
         });
       }
     }
   };
 }
+function getSuggest(getFix4) {
+  return BUTTON_TYPES.map((type) => ({
+    messageId: "addButtonType",
+    data: { type },
+    fix: getFix4(type)
+  }));
+}
 var RULE_NAME7 = "no-missing-iframe-sandbox";
-var RULE_FEATURES7 = [];
-var validTypes = [
-  "",
-  "allow-downloads",
-  "allow-downloads-without-user-activation",
-  "allow-forms",
-  "allow-modals",
-  "allow-orientation-lock",
-  "allow-pointer-lock",
-  "allow-popups",
-  "allow-popups-to-escape-sandbox",
-  "allow-presentation",
-  "allow-same-origin",
-  "allow-scripts",
-  "allow-storage-access-by-user-activation",
-  "allow-top-navigation",
-  "allow-top-navigation-by-user-activation",
-  "allow-top-navigation-to-custom-protocols"
+var RULE_FEATURES7 = [
+  "FIX"
 ];
-function hasValidSandBox(value) {
-  return typeof value === "string" && value.split(" ").every((value2) => validTypes.some((valid) => valid === value2));
-}
 var no_missing_iframe_sandbox_default = createRule({
   meta: {
     type: "problem",
@@ -407,7 +406,10 @@ var no_missing_iframe_sandbox_default = createRule({
       description: "Enforces explicit `sandbox` attribute for `iframe` elements.",
       [Symbol.for("rule_features")]: RULE_FEATURES7
     },
+    fixable: "code",
+    hasSuggestions: true,
     messages: {
+      addIframeSandbox: "Add 'sandbox' attribute with value '{{value}}'.",
       noMissingIframeSandbox: "Add missing 'sandbox' attribute on 'iframe' component."
     },
     schema: []
@@ -436,17 +438,36 @@ function create7(context) {
           attributeNode,
           propNameOnJsx
         );
-        if (attributeValue.kind === "some" && hasValidSandBox(attributeValue.value)) return;
-        context.report({
-          messageId: "noMissingIframeSandbox",
-          node: attributeNode
-        });
+        if (attributeValue.kind !== "some" || typeof attributeValue.value !== "string") {
+          context.report({
+            messageId: "noMissingIframeSandbox",
+            node: attributeNode,
+            suggest: [
+              {
+                messageId: "addIframeSandbox",
+                data: { value: "" },
+                fix(fixer) {
+                  return fixer.replaceText(attributeNode, `${propNameOnJsx}=""`);
+                }
+              }
+            ]
+          });
+        }
         return;
       }
-      if (!hasValidSandBox(customComponentProp?.defaultValue)) {
+      if (typeof customComponentProp?.defaultValue !== "string") {
         context.report({
           messageId: "noMissingIframeSandbox",
-          node
+          node,
+          suggest: [
+            {
+              messageId: "addIframeSandbox",
+              data: { value: "" },
+              fix(fixer) {
+                return fixer.insertTextAfter(node.openingElement.name, ` ${propNameOnJsx}=""`);
+              }
+            }
+          ]
         });
       }
     }
@@ -1834,7 +1855,7 @@ function create14(context) {
         if (attributeValue.kind === "some" && typeof attributeValue.value === "string") {
           return attributeValue.value;
         }
-        return _;
+        return unit;
       };
       if (getAttributeStringValue("target") !== "_blank") {
         return;
@@ -1965,9 +1986,10 @@ var no_void_elements_with_children_default = createRule({
   defaultOptions: []
 });
 function create16(context) {
+  const resolver = createJsxElementResolver(context);
   return {
     JSXElement(node) {
-      const elementName = ER.getElementType(context, node);
+      const { domElementType: elementName } = resolver.resolve(node);
       if (elementName.length === 0 || !voidElements.has(elementName)) {
         return;
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-dom",
-  "version": "2.0.0-next.5",
+  "version": "2.0.0-next.51",
   "description": "ESLint React's ESLint plugin for React DOM related rules.",
   "keywords": [
     "react",
@@ -35,27 +35,27 @@
     "./package.json"
   ],
   "dependencies": {
-    "@typescript-eslint/scope-manager": "^8.31.0",
-    "@typescript-eslint/types": "^8.31.0",
-    "@typescript-eslint/utils": "^8.31.0",
+    "@typescript-eslint/scope-manager": "^8.34.1",
+    "@typescript-eslint/types": "^8.34.1",
+    "@typescript-eslint/utils": "^8.34.1",
     "compare-versions": "^6.1.1",
     "string-ts": "^2.2.1",
-    "ts-pattern": "^5.7.0",
-    "@eslint-react/ast": "2.0.0-next.5",
-    "@eslint-react/kit": "2.0.0-next.5",
-    "@eslint-react/shared": "2.0.0-next.5",
-    "@eslint-react/core": "2.0.0-next.5",
-    "@eslint-react/var": "2.0.0-next.5",
-    "@eslint-react/eff": "2.0.0-next.5"
+    "ts-pattern": "^5.7.1",
+    "@eslint-react/ast": "2.0.0-next.51",
+    "@eslint-react/core": "2.0.0-next.51",
+    "@eslint-react/kit": "2.0.0-next.51",
+    "@eslint-react/shared": "2.0.0-next.51",
+    "@eslint-react/eff": "2.0.0-next.51",
+    "@eslint-react/var": "2.0.0-next.51"
   },
   "devDependencies": {
-    "@types/react": "^19.1.2",
-    "@types/react-dom": "^19.1.2",
-    "tsup": "^8.4.0",
+    "@types/react": "^19.1.8",
+    "@types/react-dom": "^19.1.6",
+    "tsup": "^8.5.0",
     "@local/configs": "0.0.0"
   },
   "peerDependencies": {
-    "eslint": "^9.25.0",
+    "eslint": "^9.29.0",
     "typescript": "^4.9.5 || ^5.4.5"
   },
   "peerDependenciesMeta": {