fix: normalize attr names to lowercase; add isNativeElement scope check; tests for UPPERCASE attrs

Author: johanrd

lib/rules/template-valid-input-attributes.js

@@ -2,6 +2,8 @@
 
 // Logic adapted from html-validate (MIT), Copyright 2017 David Sveningsson.
 
+const { isNativeElement } = require('../utils/is-native-element');
+
 const RESTRICTED = new Map([
   ['accept', new Set(['file'])],
   ['alt', new Set(['image'])],
@@ -124,11 +126,15 @@ module.exports = {
   },
 
   create(context) {
+    const sourceCode = context.sourceCode || context.getSourceCode();
     return {
       GlimmerElementNode(node) {
         if (node.tag !== 'input') {
           return;
         }
+        if (!isNativeElement(node, sourceCode)) {
+          return;
+        }
         // Per HTML §4.10.5, an <input> with missing, valueless, empty, or
         // unknown `type` falls back to the Text state. Only a DYNAMIC type
         // (mustache/concat) is opaque to static analysis — skip those.
@@ -151,7 +157,7 @@ module.exports = {
         const type = KNOWN_INPUT_TYPES.has(normalized) ? normalized : 'text';
 
         for (const attr of node.attributes || []) {
-          const validTypes = RESTRICTED.get(attr.name);
+          const validTypes = RESTRICTED.get(attr.name.toLowerCase());
           if (!validTypes) {
             continue;
           }

tests/lib/rules/template-valid-input-attributes.js

@@ -24,6 +24,9 @@ const validHbs = [
   '<input pattern="\\d+" />',
   '<input type />',
   '<input maxlength="100" size="20" readonly />',
+  // Attribute name case — HTML attributes are case-insensitive; rule normalizes.
+  '<input type="text" MAXLENGTH="5" />',
+  '<input type="text" Pattern="\\d+" />',
   // Not an input — rule doesn't apply.
   '<textarea maxlength="10"></textarea>',
   // Empty/whitespace/unknown type values fall back to the Text state per HTML
@@ -63,6 +66,11 @@ const invalidHbs = [
     code: '<input type="TEXT" accept="image/*" />',
     errors: [{ message: err('accept', 'text') }],
   },
+  // Uppercase attribute name — normalized before restriction lookup.
+  {
+    code: '<input type="number" PATTERN="x" />',
+    errors: [{ message: err('PATTERN', 'number') }],
+  },
   // Text-state fallback — <input> with missing/valueless/empty/unknown type
   // is the Text state per HTML spec. Attributes incompatible with text are
   // flagged as `type="text"` in the error message.