Fix template-no-arguments-for-html-elements: align HTML-element detection with upstream

Replace static htmlTags package check with inverse-of-component-detection
(reject tags with : / . / uppercase / @ / -). Matches upstream's
isAngleBracketComponent semantics. Custom elements are now correctly
excluded.

Author: johanrd

lib/rules/template-no-arguments-for-html-elements.js

@@ -1,5 +1,34 @@
 /** @type {import('eslint').Rule.RuleModule} */
-const htmlTags = require('html-tags');
+
+// Mirror upstream ember-template-lint's inverse-of-isAngleBracketComponent logic.
+// A tag is treated as an HTML element only when it:
+//   - does NOT contain ':' (named blocks like <:slot>)
+//   - does NOT contain '.' (path/namespaced invocations like <foo.bar>)
+//   - does NOT start with '@' (argument invocations like <@foo>)
+//   - has NO uppercase letters (component invocations like <MyThing>)
+//   - does NOT contain '-' (HTML custom elements like <my-element>)
+// Everything else is a component / custom-element / slot — not a plain HTML element.
+function isHtmlElement(tagName) {
+  if (!tagName) {
+    return false;
+  }
+  if (tagName.startsWith('@')) {
+    return false;
+  }
+  if (tagName.includes(':')) {
+    return false;
+  }
+  if (tagName.includes('.')) {
+    return false;
+  }
+  if (tagName.includes('-')) {
+    return false;
+  }
+  if (tagName !== tagName.toLowerCase()) {
+    return false;
+  }
+  return true;
+}
 
 module.exports = {
   meta: {
@@ -20,12 +49,11 @@ module.exports = {
 
   create(context) {
     const sourceCode = context.sourceCode;
-    const HTML_ELEMENTS = new Set(htmlTags);
 
     return {
       GlimmerElementNode(node) {
-        // Check if this is an HTML element (lowercase)
-        if (!HTML_ELEMENTS.has(node.tag)) {
+        // Check if this is an HTML element (not a component / custom element / slot)
+        if (!isHtmlElement(node.tag)) {
           return;
         }
 

tests/lib/rules/template-no-arguments-for-html-elements.js

@@ -13,6 +13,13 @@ ruleTester.run('template-no-arguments-for-html-elements', rule, {
     '<template><MyComponent @title="Hello" @onClick={{this.handler}} /></template>',
     '<template><CustomButton @disabled={{true}} /></template>',
     '<template><input value={{this.value}} /></template>',
+    // Custom elements (with a dash) are not HTML elements per upstream's
+    // isAngleBracketComponent inverse; they should not be flagged.
+    '<template><my-element @foo="x" /></template>',
+    // Namespaced/path component invocations should not be treated as HTML elements.
+    '<template><NS.Foo @bar="baz" /></template>',
+    // Named blocks (colon-prefixed) should not be treated as HTML elements.
+    '<template><Thing><:slot @item="x">content</:slot></Thing></template>',
     `let div = <template>{{@greeting}}</template>
 
 <template>