eslint-plugin-react-dom 2.0.0-beta.4 → 2.0.0-beta.41

package/dist/index.d.ts

@@ -35,6 +35,7 @@ declare const _default: {
                         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>;
                         readonly "no-script-url": _typescript_eslint_utils_ts_eslint.RuleModule<"noScriptUrl", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+                        readonly "no-string-style-prop": _typescript_eslint_utils_ts_eslint.RuleModule<"noStringStyleProp", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-unknown-property": _typescript_eslint_utils_ts_eslint.RuleModule<"dataLowercaseRequired" | "invalidPropOnTag" | "unknownProp" | "unknownPropWithStandardName", unknown[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-unsafe-iframe-sandbox": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeIframeSandbox", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
                         readonly "no-unsafe-target-blank": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$2, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
@@ -67,6 +68,7 @@ declare const _default: {
         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>;
         readonly "no-script-url": _typescript_eslint_utils_ts_eslint.RuleModule<"noScriptUrl", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+        readonly "no-string-style-prop": _typescript_eslint_utils_ts_eslint.RuleModule<"noStringStyleProp", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-unknown-property": _typescript_eslint_utils_ts_eslint.RuleModule<"dataLowercaseRequired" | "invalidPropOnTag" | "unknownProp" | "unknownPropWithStandardName", unknown[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-unsafe-iframe-sandbox": _typescript_eslint_utils_ts_eslint.RuleModule<"noUnsafeIframeSandbox", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
         readonly "no-unsafe-target-blank": _typescript_eslint_utils_ts_eslint.RuleModule<MessageID$2, [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;

package/dist/index.js

@@ -2,7 +2,6 @@ import { getDocsUrl, getSettingsFromContext, DEFAULT_ESLINT_REACT_SETTINGS } fro
 import * as ER from '@eslint-react/core';
 import { ESLintUtils } from '@typescript-eslint/utils';
 import { unit } from '@eslint-react/eff';
-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';
@@ -33,6 +32,7 @@ var rules = {
   "react-dom/no-render": "error",
   "react-dom/no-render-return-value": "error",
   "react-dom/no-script-url": "warn",
+  "react-dom/no-string-style-prop": "error",
   "react-dom/no-unsafe-iframe-sandbox": "warn",
   "react-dom/no-unsafe-target-blank": "warn",
   "react-dom/no-use-form-state": "error",
@@ -44,7 +44,7 @@ var settings = {
 
 // package.json
 var name2 = "eslint-plugin-react-dom";
-var version = "2.0.0-beta.4";
+var version = "2.0.0-beta.41";
 function createJsxElementResolver(context) {
   const { components, polymorphicPropName } = getSettingsFromContext(context);
   return {
@@ -186,7 +186,7 @@ function create2(context) {
     JSXElement(node) {
       const attributes = node.openingElement.attributes;
       const initialScope = context.sourceCode.getScope(node);
-      const hasChildren = hasChildrenWithin(node) || ER.hasAttribute(context, "children", attributes, initialScope);
+      const hasChildren = node.children.some(isSignificantChildren) || ER.hasAttribute(context, "children", attributes, initialScope);
       if (hasChildren && ER.hasAttribute(context, dangerouslySetInnerHTML2, attributes, initialScope)) {
         context.report({
           messageId: "noDangerouslySetInnerhtmlWithChildren",
@@ -196,8 +196,14 @@ function create2(context) {
     }
   };
 }
-function hasChildrenWithin(node) {
-  return node.children.length > 0 && node.children[0] != null && !AST.isLineBreak(node.children[0]);
+function isWhiteSpace(node) {
+  return typeof node.value === "string" && node.raw.trim() === "";
+}
+function isPaddingSpaces(node) {
+  return ER.isJsxText(node) && isWhiteSpace(node) && node.raw.includes("\n");
+}
+function isSignificantChildren(node) {
+  return node.type !== AST_NODE_TYPES.JSXText || !isPaddingSpaces(node);
 }
 var RULE_NAME3 = "no-find-dom-node";
 var RULE_FEATURES3 = [];
@@ -690,7 +696,51 @@ function create11(context) {
     }
   };
 }
-var RULE_NAME12 = "no-unknown-property";
+var RULE_NAME12 = "no-string-style-prop";
+var RULE_FEATURES12 = [];
+var no_string_style_prop_default = createRule({
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Disallows the use of string style prop.",
+      [Symbol.for("rule_features")]: RULE_FEATURES12
+    },
+    messages: {
+      noStringStyleProp: "Do not use string style prop. Use an object instead."
+    },
+    schema: []
+  },
+  name: RULE_NAME12,
+  create: create12,
+  defaultOptions: []
+});
+function create12(context) {
+  return {
+    JSXAttribute(node) {
+      if (node.name.name !== "style") return;
+      const styleText = getAttributeValueText(context, node.value);
+      if (styleText == null) return;
+      context.report({
+        messageId: "noStringStyleProp",
+        node
+      });
+    }
+  };
+}
+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_NAME13 = "no-unknown-property";
 var DEFAULTS = {
   ignore: [],
   requireDataLowercase: false
@@ -1653,11 +1703,11 @@ var no_unknown_property_default = createRule({
       }
     }]
   },
-  name: RULE_NAME12,
-  create: create12,
+  name: RULE_NAME13,
+  create: create13,
   defaultOptions: []
 });
-function create12(context) {
+function create13(context) {
   const report = Reporter.make(context);
   function getIgnoreConfig() {
     return context.options[0]?.ignore || DEFAULTS.ignore;
@@ -1750,8 +1800,8 @@ function testReactVersion(context, comparator, version2) {
 }
 
 // src/rules/no-unsafe-iframe-sandbox.ts
-var RULE_NAME13 = "no-unsafe-iframe-sandbox";
-var RULE_FEATURES12 = [];
+var RULE_NAME14 = "no-unsafe-iframe-sandbox";
+var RULE_FEATURES13 = [];
 var unsafeSandboxValues = [
   ["allow-scripts", "allow-same-origin"]
 ];
@@ -1766,18 +1816,18 @@ var no_unsafe_iframe_sandbox_default = createRule({
     type: "problem",
     docs: {
       description: "Enforces `sandbox` attribute for `iframe` elements is not set to unsafe combinations.",
-      [Symbol.for("rule_features")]: RULE_FEATURES12
+      [Symbol.for("rule_features")]: RULE_FEATURES13
     },
     messages: {
       noUnsafeIframeSandbox: "Unsafe 'sandbox' attribute value on 'iframe' component."
     },
     schema: []
   },
-  name: RULE_NAME13,
-  create: create13,
+  name: RULE_NAME14,
+  create: create14,
   defaultOptions: []
 });
-function create13(context) {
+function create14(context) {
   const resolver = createJsxElementResolver(context);
   return {
     JSXElement(node) {
@@ -1795,8 +1845,8 @@ function create13(context) {
 }
 
 // src/rules/no-unsafe-target-blank.ts
-var RULE_NAME14 = "no-unsafe-target-blank";
-var RULE_FEATURES13 = [
+var RULE_NAME15 = "no-unsafe-target-blank";
+var RULE_FEATURES14 = [
   "FIX"
 ];
 function isExternalLinkLike(value) {
@@ -1812,7 +1862,7 @@ var no_unsafe_target_blank_default = createRule({
     type: "problem",
     docs: {
       description: 'Disallow `target="_blank"` without `rel="noreferrer noopener"`.',
-      [Symbol.for("rule_features")]: RULE_FEATURES13
+      [Symbol.for("rule_features")]: RULE_FEATURES14
     },
     fixable: "code",
     hasSuggestions: true,
@@ -1822,11 +1872,11 @@ var no_unsafe_target_blank_default = createRule({
     },
     schema: []
   },
-  name: RULE_NAME14,
-  create: create14,
+  name: RULE_NAME15,
+  create: create15,
   defaultOptions: []
 });
-function create14(context) {
+function create15(context) {
   const resolver = createJsxElementResolver(context);
   return {
     JSXElement(node) {
@@ -1874,8 +1924,8 @@ function create14(context) {
     }
   };
 }
-var RULE_NAME15 = "no-use-form-state";
-var RULE_FEATURES14 = [
+var RULE_NAME16 = "no-use-form-state";
+var RULE_FEATURES15 = [
   "MOD"
 ];
 var no_use_form_state_default = createRule({
@@ -1883,7 +1933,7 @@ var no_use_form_state_default = createRule({
     type: "problem",
     docs: {
       description: "Replaces usages of `useFormState` with `useActionState`.",
-      [Symbol.for("rule_features")]: RULE_FEATURES14
+      [Symbol.for("rule_features")]: RULE_FEATURES15
     },
     fixable: "code",
     messages: {
@@ -1891,11 +1941,11 @@ var no_use_form_state_default = createRule({
     },
     schema: []
   },
-  name: RULE_NAME15,
-  create: create15,
+  name: RULE_NAME16,
+  create: create16,
   defaultOptions: []
 });
-function create15(context) {
+function create16(context) {
   if (!context.sourceCode.text.includes("useFormState")) return {};
   const settings2 = getSettingsFromContext(context);
   if (compare(settings2.version, "19.0.0", "<")) return {};
@@ -1950,8 +2000,8 @@ function getFix3(context, node) {
     ];
   };
 }
-var RULE_NAME16 = "no-void-elements-with-children";
-var RULE_FEATURES15 = [];
+var RULE_NAME17 = "no-void-elements-with-children";
+var RULE_FEATURES16 = [];
 var voidElements = /* @__PURE__ */ new Set([
   "area",
   "base",
@@ -1975,18 +2025,18 @@ var no_void_elements_with_children_default = createRule({
     type: "problem",
     docs: {
       description: "Disallow `children` in void DOM elements.",
-      [Symbol.for("rule_features")]: RULE_FEATURES15
+      [Symbol.for("rule_features")]: RULE_FEATURES16
     },
     messages: {
       noVoidElementsWithChildren: "'{{element}}' is a void element tag and must not have children."
     },
     schema: []
   },
-  name: RULE_NAME16,
-  create: create16,
+  name: RULE_NAME17,
+  create: create17,
   defaultOptions: []
 });
-function create16(context) {
+function create17(context) {
   const resolver = createJsxElementResolver(context);
   return {
     JSXElement(node) {
@@ -2037,6 +2087,7 @@ var plugin = {
     "no-render": no_render_default,
     "no-render-return-value": no_render_return_value_default,
     "no-script-url": no_script_url_default,
+    "no-string-style-prop": no_string_style_prop_default,
     "no-unknown-property": no_unknown_property_default,
     "no-unsafe-iframe-sandbox": no_unsafe_iframe_sandbox_default,
     "no-unsafe-target-blank": no_unsafe_target_blank_default,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-react-dom",
-  "version": "2.0.0-beta.4",
+  "version": "2.0.0-beta.41",
   "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.35.0",
-    "@typescript-eslint/types": "^8.35.0",
-    "@typescript-eslint/utils": "^8.35.0",
+    "@typescript-eslint/scope-manager": "^8.40.0",
+    "@typescript-eslint/types": "^8.40.0",
+    "@typescript-eslint/utils": "^8.40.0",
     "compare-versions": "^6.1.1",
     "string-ts": "^2.2.1",
-    "ts-pattern": "^5.7.1",
-    "@eslint-react/ast": "2.0.0-beta.4",
-    "@eslint-react/core": "2.0.0-beta.4",
-    "@eslint-react/eff": "2.0.0-beta.4",
-    "@eslint-react/kit": "2.0.0-beta.4",
-    "@eslint-react/shared": "2.0.0-beta.4",
-    "@eslint-react/var": "2.0.0-beta.4"
+    "ts-pattern": "^5.8.0",
+    "@eslint-react/ast": "2.0.0-beta.41",
+    "@eslint-react/core": "2.0.0-beta.41",
+    "@eslint-react/eff": "2.0.0-beta.41",
+    "@eslint-react/var": "2.0.0-beta.41",
+    "@eslint-react/shared": "2.0.0-beta.41",
+    "@eslint-react/kit": "2.0.0-beta.41"
   },
   "devDependencies": {
-    "@types/react": "^19.1.8",
-    "@types/react-dom": "^19.1.6",
+    "@types/react": "^19.1.11",
+    "@types/react-dom": "^19.1.7",
     "tsup": "^8.5.0",
     "@local/configs": "0.0.0"
   },
   "peerDependencies": {
-    "eslint": "^9.29.0",
+    "eslint": "^9.33.0",
     "typescript": "^4.9.5 || ^5.4.5"
   },
   "peerDependenciesMeta": {
@@ -67,7 +67,6 @@
     }
   },
   "engines": {
-    "bun": ">=1.0.15",
     "node": ">=20.19.0"
   },
   "publishConfig": {