fix(#27): address round-2 Copilot review (normalize role via getRole; pin gridcell/row/columnheader/rowheader)

Author: johanrd

lib/rules/template-no-nested-interactive.js

@@ -146,8 +146,10 @@ module.exports = {
         return true;
       }
 
-      // Check role
-      const role = getTextAttr(node, 'role');
+      // Check role — normalize via getRole so that `role="BUTTON"` and
+      // `role="button checkbox"` match INTERACTIVE_ROLES. This mirrors the
+      // resolution used by isCompositeWidgetPattern below.
+      const role = getRole(node);
       if (role && INTERACTIVE_ROLES.has(role)) {
         return true;
       }
@@ -190,7 +192,7 @@ module.exports = {
       if (tag === 'a' && hasAttr(node, 'href')) {
         return false;
       }
-      const role = getTextAttr(node, 'role');
+      const role = getRole(node);
       if (role && INTERACTIVE_ROLES.has(role)) {
         return false;
       }

tests/lib/utils/interactive-roles-test.js

@@ -40,6 +40,17 @@ describe('INTERACTIVE_ROLES', () => {
     }
   });
 
+  describe('composite widget children (widget-descended per aria-query)', () => {
+    // Pin behaviorally-important widget-descended roles both of the rules
+    // previously relied on. Explicit assertions here make taxonomy drift
+    // easier to diagnose than the size-only drift check below.
+    for (const role of ['gridcell', 'row', 'columnheader', 'rowheader']) {
+      it(`includes "${role}"`, () => {
+        expect(INTERACTIVE_ROLES.has(role)).toBe(true);
+      });
+    }
+  });
+
   describe('manual override', () => {
     it('includes "toolbar" (not widget-descended per aria-query; added per APG convention)', () => {
       expect(INTERACTIVE_ROLES.has('toolbar')).toBe(true);