lint: narrow INTERACTIVE_ROLES util to match jsx-a11y/lit-a11y

Author: johanrd

lib/utils/interactive-roles.js

@@ -1,38 +1,31 @@
 const { roles } = require('aria-query');
 
-// Interactive ARIA roles — the set of concrete roles whose taxonomy in WAI-ARIA
-// includes a widget / command / composite / input / range ancestor. Derived from
-// aria-query so the list stays current with ARIA spec updates (including
-// DPUB-ARIA and Graphics-ARIA additions) without hand maintenance.
+// Interactive ARIA roles — concrete roles whose taxonomy descends from `widget`
+// in aria-query. This is the same derivation jsx-a11y and lit-a11y use (they
+// define the canonical peer-plugin behavior here):
+//   https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/src/util/isInteractiveRole.js
+//   https://github.com/open-wc/open-wc/blob/main/packages/eslint-plugin-lit-a11y/lib/utils/isInteractiveElement.js
 //
-// `tooltip` and `toolbar` are added explicitly:
-//   - `tooltip`: ARIA 1.2 §5.4 lists tooltip among widget roles, but aria-query's
-//     superClass chain doesn't include `widget` for it. Practitioner convention
-//     (and jsx-a11y/vuejs-accessibility) treats it as interactive.
-//   - `toolbar`: does not descend from `widget` in aria-query, but supports
-//     `aria-activedescendant` and is widget-like in practice. jsx-a11y adds it
-//     with the same rationale.
+// `toolbar` is added explicitly — it does not descend from `widget` per
+// aria-query's taxonomy, but supports `aria-activedescendant` and is widget-
+// like in practice. jsx-a11y and lit-a11y add it for the same reason.
+//
+// `tooltip` is also added — ARIA 1.2 doesn't cleanly categorize tooltip under
+// the widget taxonomy (aria-query's superClass is `structure/section`), but
+// tooltips with interactive content (close buttons, links) are common and our
+// existing test suite treats them as interactive.
 module.exports.INTERACTIVE_ROLES = buildInteractiveRoleSet();
 
 function buildInteractiveRoleSet() {
-  const result = new Set(['tooltip', 'toolbar']);
+  const result = new Set(['toolbar', 'tooltip']);
   for (const [role, def] of roles) {
     if (def.abstract) {
       continue;
     }
-    const ancestors = new Set();
-    for (const chain of def.superClass || []) {
-      for (const cls of chain) {
-        ancestors.add(cls);
-      }
-    }
-    if (
-      ancestors.has('widget') ||
-      ancestors.has('command') ||
-      ancestors.has('composite') ||
-      ancestors.has('input') ||
-      ancestors.has('range')
-    ) {
+    const descendsFromWidget = (def.superClass || []).some((chain) =>
+      chain.includes('widget')
+    );
+    if (descendsFromWidget) {
       result.add(role);
     }
   }