fix(template-no-noninteractive-tabindex): case-insensitive attribute-name lookup for tabIndex / TABINDEX (Copilot review)

Author: johanrd

lib/rules/template-no-noninteractive-tabindex.js

@@ -34,7 +34,11 @@ function buildInteractiveRoleSet() {
 }
 
 function findAttr(node, name) {
-  return node.attributes?.find((a) => a.name === name);
+  // HTML attribute names are case-insensitive. Normalize both sides so
+  // `tabindex`, `tabIndex`, and `TABINDEX` all match the same lookup.
+  // LLM-generated templates commonly emit `tabIndex` (React convention).
+  const target = name.toLowerCase();
+  return node.attributes?.find((a) => a.name?.toLowerCase() === target);
 }
 
 function getStaticTabindexValue(attr) {

tests/lib/rules/template-no-noninteractive-tabindex.js

@@ -86,6 +86,19 @@ ruleTester.run('template-no-noninteractive-tabindex', rule, {
       output: null,
       errors: [{ messageId: 'noNonInteractiveTabindex' }],
     },
+    // HTML attribute names are case-insensitive. LLM-generated Ember
+    // templates often emit the React-style camelCase `tabIndex`; a fully
+    // uppercase `TABINDEX` is also valid per spec. Both must still flag.
+    {
+      code: '<template><div tabIndex="0"></div></template>',
+      output: null,
+      errors: [{ messageId: 'noNonInteractiveTabindex' }],
+    },
+    {
+      code: '<template><div TABINDEX="0"></div></template>',
+      output: null,
+      errors: [{ messageId: 'noNonInteractiveTabindex' }],
+    },
     {
       code: '<template><article tabindex="0">Story</article></template>',
       output: null,