fix(template-no-invalid-role): preserve author casing in invalid-role message (Copilot review)

Keep the original token alongside the lowercased one; validation stays case-
insensitive (lowercases against VALID_ROLES) but the reported-back token uses
the author's raw casing so the error surfaces their input verbatim rather
than the normalized form.

Also drop a 'line 229 of rule' hard-coded reference from the aria-role
peer-parity fixture — line numbers rot on every refactor.

Author: johanrd

lib/rules/template-no-invalid-role.js

@@ -162,15 +162,19 @@ module.exports = {
 
         // ARIA role attribute is a whitespace-separated list of tokens
         // (role-fallback pattern per ARIA 1.2 §5.4). Validate each token.
-        const tokens = raw.split(/\s+/u).map((t) => t.toLowerCase());
+        // Keep the original casing alongside the normalized (lowercase) form
+        // so reported-back tokens preserve author intent — the validation
+        // is case-insensitive, the ERROR MESSAGE isn't.
+        const rawTokens = raw.split(/\s+/u);
+        const tokens = rawTokens.map((t) => t.toLowerCase());
 
         if (catchNonexistentRoles) {
-          const invalidToken = tokens.find((token) => !VALID_ROLES.has(token));
-          if (invalidToken) {
+          const invalidIdx = tokens.findIndex((token) => !VALID_ROLES.has(token));
+          if (invalidIdx !== -1) {
             context.report({
               node: roleAttr,
               messageId: 'invalid',
-              data: { role: invalidToken },
+              data: { role: rawTokens[invalidIdx] },
             });
             return;
           }

tests/audit/aria-role/peer-parity.js

@@ -95,7 +95,7 @@ ruleTester.run('audit:aria-role (gts)', rule, {
     // === DIVERGENCE — empty role string ===
     // jsx-a11y: INVALID — `<div role="" />` flagged.
     // vue-a11y: INVALID — same.
-    // Our rule: early-return on empty/whitespace role (line 229 of rule). NO FLAG.
+    // Our rule: early-return on empty/whitespace role. NO FLAG.
     // So this case reflects OUR (non-flagging) behavior with an explicit note.
     // (No invalid assertion possible here — we'd need to move this to valid,
     //  or fix the rule to flag.)

tests/lib/rules/template-no-invalid-role.js

@@ -204,7 +204,9 @@ ruleTester.run('template-no-invalid-role', rule, {
     {
       code: '<template><div role="COMMAND INTERFACE"></div></template>',
       output: null,
-      errors: [{ message: "Invalid ARIA role 'command'. Must be a valid ARIA role." }],
+      // Validation is case-insensitive, but the error message echoes the
+      // author-provided token verbatim so authors see their own text.
+      errors: [{ message: "Invalid ARIA role 'COMMAND'. Must be a valid ARIA role." }],
     },
     {
       code: '<template><div role="command interface"></div></template>',
@@ -363,7 +365,9 @@ hbsRuleTester.run('template-no-invalid-role', rule, {
     {
       code: '<div role="COMMAND INTERFACE"></div>',
       output: null,
-      errors: [{ message: "Invalid ARIA role 'command'. Must be a valid ARIA role." }],
+      // Validation is case-insensitive, but the error message echoes the
+      // author-provided token verbatim so authors see their own text.
+      errors: [{ message: "Invalid ARIA role 'COMMAND'. Must be a valid ARIA role." }],
     },
     // Newly added SEMANTIC_ELEMENTS: presentation/none on iframe, video, audio, embed
     {