implement

yeonjuan · yeonjuan · commit e5ef2121dee5 · 2025-04-27T13:24:20.000+09:00
diff --git a/packages/eslint-plugin/lib/rules/use-standard-html/check-content-model.js b/packages/eslint-plugin/lib/rules/use-standard-html/check-content-model.js
@@ -17,7 +17,11 @@
 
 const { getElementSpec } = require("html-standard");
 const { isTag } = require("../utils/node");
-const { shouldIgnoreChild, getNodeName } = require("./helpers");
+const {
+  shouldIgnoreChild,
+  getNodeName,
+  getDisplayNodeName,
+} = require("./helpers");
 
 const EXIT = false;
 const CONTINUE = true;
@@ -117,6 +121,10 @@ function checkContentModel(
   if (remain && !contentModel) {
     context.report({
       node: remain,
+      data: {
+        child: getDisplayNodeName(remain),
+        parent: getDisplayNodeName(node),
+      },
       messageId: MESSAGE_IDS.NOT_ALLOWED,
     });
   }
@@ -152,6 +160,10 @@ function required(model, context, state, node) {
   }
   context.report({
     node: node.openStart,
+    data: {
+      child: getDisplayNodeName(child),
+      parent: getDisplayNodeName(node),
+    },
     messageId: MESSAGE_IDS.NOT_ALLOWED,
   });
   return EXIT;
diff --git a/packages/eslint-plugin/lib/rules/use-standard-html/helpers.js b/packages/eslint-plugin/lib/rules/use-standard-html/helpers.js
@@ -38,10 +38,34 @@ function getNodeName(node) {
   if (isComment(node)) {
     return "#comment";
   }
+  return "Unknown";
+}
+
+/**
+ * @param {AnyHTMLNode} node
+ * @returns {string}
+ */
+function getDisplayNodeName(node) {
+  if (isTag(node)) {
+    return node.name;
+  }
+  if (isText(node)) {
+    return "Text";
+  }
+  if (isScript(node)) {
+    return "script";
+  }
+  if (isStyle(node)) {
+    return "style";
+  }
+  if (isComment(node)) {
+    return "Comment";
+  }
   return "#unknown";
 }
 
 module.exports = {
   shouldIgnoreChild,
   getNodeName,
+  getDisplayNodeName,
 };
diff --git a/packages/eslint-plugin/lib/rules/use-standard-html/use-standard-html.js b/packages/eslint-plugin/lib/rules/use-standard-html/use-standard-html.js
@@ -41,7 +41,8 @@ module.exports = {
     ],
     messages: {
       [MESSAGE_IDS.REQUIRED]: "required",
-      [MESSAGE_IDS.NOT_ALLOWED]: "not allowed",
+      [MESSAGE_IDS.NOT_ALLOWED]:
+        "Element '{{child}}' not allowed as child of element '{{parent}}'",
     },
   },
   create(context) {
diff --git a/packages/eslint-plugin/tests/rules/use-standard-html.test.js b/packages/eslint-plugin/tests/rules/use-standard-html.test.js
@@ -76,7 +76,7 @@ ruleTester.run("use-standard-html", rule, {
       code: `<html><div></div></html>`,
       errors: [
         {
-          messageId: "notAllowed",
+          message: `Element 'div' not allowed as child of element 'html'`,
         },
       ],
     },

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ ruleTester.run("use-standard-html", rule, {`
`76`	`76`	code: `<html><div></div></html>`,
`77`	`77`	`errors: [`
`78`	`78`	`{`
`79`		`- messageId: "notAllowed",`
	`79`	+ message: `Element 'div' not allowed as child of element 'html'`,
`80`	`80`	`},`
`81`	`81`	`],`
`82`	`82`	`},`