@surea11y/core 1.2.0 → 1.4.0

package/src/checks/automatic/aria-required-parent.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -47,7 +49,8 @@ const id = 'aria-required-parent';
 
 const meta = {
   title: 'Roles requiring a specific context role must be in that context',
-  description: 'Checks that roles with a documented "required context role" entry (listitem, option, tab, treeitem, row, cell, ...) have an ancestor or aria-owns owner with an acceptable context role.',
+  description:
+    'Checks that roles with a documented "required context role" entry (listitem, option, tab, treeitem, row, cell, ...) have an ancestor or aria-owns owner with an acceptable context role.',
   i18n: {
     titleKey: 'ariaRequiredParent_title',
     descriptionKey: 'ariaRequiredParent_description'
@@ -56,7 +59,13 @@ const meta = {
   tags: ['wcag2a', 'wcag412', 'aria', 'structure', 'atomic', 'automatic'],
   wcagSc: ['4.1.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '4.1.2', title: 'Name, Role, Value', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '4.1.2',
+      title: 'Name, Role, Value',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'moderate',
   category: 'robust',
@@ -66,7 +75,7 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
   const ariaHelpers = helpers && helpers.aria ? helpers.aria : null;
   if (!ariaHelpers) {
@@ -74,7 +83,8 @@ function runInPage(ctx) {
   }
 
   function isEligibleAcc(el) {
-    const fn = helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
+    const fn =
+      helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
     if (!fn) return true;
     try {
       const r = fn(el, ctx);
@@ -86,24 +96,20 @@ function runInPage(ctx) {
   }
 
   // Roles that may host a nested listitem/treeitem group without breaking
-  // the required-context chain (verified against a widely-used reference
-  // engine's own getMissingContext, which special-cases exactly these two
-  // roles via its ownGroupRoles option).
+  // the required-context chain — the group role is transparent for exactly
+  // these two roles.
   const GROUP_TRANSPARENT_FOR_ROLES = new Set(['listitem', 'treeitem']);
 
   // A real ancestor role — not "no role at all" and not the two roles that
   // strip an element from the accessibility tree's parent/child chain
-  // entirely (presentation/none) — stops the search, matching that reference
-  // engine's getMissingContext. This is stricter than "any ancestor with the right
-  // role anywhere up the tree": the required-context relationship is about
-  // the accessibility tree's actual PARENT, so an intervening ancestor with
-  // its OWN distinct real role (e.g. a plain <li>'s native "listitem" role)
-  // blocks the search even if a further-up ancestor has the correct role.
-  // Found via a real page — Le Monde's review-carousel tablist, where each
-  // <button role="tab"> sits inside a plain <li> (native listitem) inside
-  // <ul role="tablist">: that reference engine correctly fails this (the tablist is never the
-  // tab's accessible-tree parent, listitem is), which the old "walk every
-  // ancestor" version here missed entirely.
+  // entirely (presentation/none) — stops the search. This is stricter than
+  // "any ancestor with the right role anywhere up the tree": the
+  // required-context relationship is about the accessibility tree's actual
+  // PARENT, so an intervening ancestor with its OWN distinct real role
+  // (e.g. a plain <li>'s native "listitem" role) blocks the search even if
+  // a further-up ancestor has the correct role. E.g. a <button role="tab">
+  // inside a plain <li> (native listitem) inside <ul role="tablist"> fails:
+  // the tablist is never the tab's accessible-tree parent, the listitem is.
   function getRealContextRole(el) {
     const role = ariaHelpers.getContainmentRole(el);
     if (!role || role === 'presentation' || role === 'none') return '';
@@ -113,21 +119,29 @@ function runInPage(ctx) {
   // Flat-tree ancestor walk (ctx.helpers.composedParent — assignedSlot wins
   // over parentNode, then shadow host). A slotted light-DOM element's real
   // rendered ancestor is whatever the shadow tree wraps its <slot> in (e.g.
-  // a role="list" container), not its own light-DOM parentElement — found
-  // via Adobe Spectrum Web Components' <sp-sidenav-item role="listitem">,
-  // distributed via slot="descendant" into its parent's shadow root, which
-  // wraps that slot in a <div role="list">. composedParent can return a
-  // non-Element node (a ShadowRoot, nodeType 11) when climbing out of a
-  // shadow tree that has no further light-DOM parent — skip those and keep
-  // climbing rather than treating them as a (roleless) context.
-  const getComposedParent = helpers && typeof helpers.composedParent === 'function'
-    ? helpers.composedParent
-    : function (n) { return n && n.parentElement ? n.parentElement : null; };
+  // a role="list" container), not its own light-DOM parentElement.
+  // composedParent can return a non-Element node (a ShadowRoot, nodeType
+  // 11) when climbing out of a shadow tree that has no further light-DOM
+  // parent — skip those and keep climbing rather than treating them as a
+  // (roleless) context.
+  const getComposedParent =
+    helpers && typeof helpers.composedParent === 'function'
+      ? helpers.composedParent
+      : function (n) {
+          return n && n.parentElement ? n.parentElement : null;
+        };
 
   function hasAcceptableAncestorContext(el, acceptableRoles, ownRole) {
     const allowsGroup = acceptableRoles.has('group');
     let cur = getComposedParent(el);
     let guard = 0;
+    // Mutable working copy: passing a transparent "group" ancestor also
+    // makes the element's OWN role an acceptable context from that point on
+    // (a nested treeitem-under-group-under-treeitem chain is a normal,
+    // arbitrarily-deep ARIA tree/list, not just one level). Cloned lazily
+    // so the caller's Set (built once per element in runInPage) is never
+    // mutated.
+    let roles = acceptableRoles;
     while (cur && guard++ < 200) {
       if (cur.nodeType !== 1) {
         cur = getComposedParent(cur);
@@ -139,10 +153,12 @@ function runInPage(ctx) {
         continue;
       }
       if (role === 'group' && allowsGroup && GROUP_TRANSPARENT_FOR_ROLES.has(ownRole)) {
+        if (roles === acceptableRoles) roles = new Set(acceptableRoles);
+        roles.add(ownRole);
         cur = getComposedParent(cur);
         continue;
       }
-      return acceptableRoles.has(role);
+      return roles.has(role);
     }
     return false;
   }
@@ -152,7 +168,9 @@ function runInPage(ctx) {
     const idTok = elId && String(elId).trim();
     if (!idTok) return false;
 
-    const owners = helpers.queryAllSmart ? helpers.queryAllSmart('[aria-owns]') : helpers.queryAll('[aria-owns]');
+    const owners = helpers.queryAllSmart
+      ? helpers.queryAllSmart('[aria-owns]')
+      : helpers.queryAll('[aria-owns]');
     for (const owner of owners) {
       if (!owner || !owner.getAttribute) continue;
       const ownsAttr = owner.getAttribute('aria-owns') || '';
@@ -165,7 +183,9 @@ function runInPage(ctx) {
     return false;
   }
 
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart('[role]') : helpers.queryAll('[role]');
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart('[role]')
+    : helpers.queryAll('[role]');
 
   const occurrences = [];
   let applicableCount = 0;
@@ -191,7 +211,7 @@ function runInPage(ctx) {
     if (hasContext) continue;
 
     const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : (el.outerHTML || '');
+    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
 
     occurrences.push({
       selector: stableSelector,
@@ -204,7 +224,11 @@ function runInPage(ctx) {
         params: { role, requiredRoles: requiredContext.join(', ') }
       },
       data: {
-        details: { reasonCode: 'ARIA_REQUIRED_PARENT_MISSING', role, requiredContextRoles: requiredContext }
+        details: {
+          reasonCode: 'ARIA_REQUIRED_PARENT_MISSING',
+          role,
+          requiredContextRoles: requiredContext
+        }
       }
     });
   }
@@ -213,9 +237,14 @@ function runInPage(ctx) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
   if (occurrences.length) {
-    return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'moderate', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'moderate',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }
 
-module.exports = { id, meta, runInPage };
+module.exports = { id, meta, runInPage };

package/src/checks/automatic/aria-role-name-present.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -15,7 +17,8 @@ const id = 'aria-role-name-present';
 
 const meta = {
   title: 'ARIA widget/container roles have an accessible name',
-  description: 'Checks that selected ARIA widget/container roles expose a non-empty accessible name.',
+  description:
+    'Checks that selected ARIA widget/container roles expose a non-empty accessible name.',
   i18n: {
     titleKey: 'ariaRoleNamePresent_title',
     descriptionKey: 'ariaRoleNamePresent_description'
@@ -24,7 +27,13 @@ const meta = {
   tags: ['wcag2a', 'wcag412', 'navigation', 'atomic', 'automatic', 'name'],
   wcagSc: ['4.1.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '4.1.2', title: 'Name, Role, Value', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '4.1.2',
+      title: 'Name, Role, Value',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'robust',
@@ -41,17 +50,19 @@ function runInPage(ctx) {
   const { document, root, helpers, rule } = ctx;
   const safeRoot = root || document;
 
-  const queryAllSmart = helpers && typeof helpers.queryAllSmart === 'function' ? helpers.queryAllSmart : null;
-  const queryAll = helpers && typeof helpers.queryAll === 'function'
-    ? helpers.queryAll
-    : (sel, rt) => {
-        try {
-          const scope = rt || safeRoot;
-          return scope && scope.querySelectorAll ? Array.from(scope.querySelectorAll(sel)) : [];
-        } catch {
-          return [];
-        }
-      };
+  const queryAllSmart =
+    helpers && typeof helpers.queryAllSmart === 'function' ? helpers.queryAllSmart : null;
+  const queryAll =
+    helpers && typeof helpers.queryAll === 'function'
+      ? helpers.queryAll
+      : (sel, rt) => {
+          try {
+            const scope = rt || safeRoot;
+            return scope && scope.querySelectorAll ? Array.from(scope.querySelectorAll(sel)) : [];
+          } catch {
+            return [];
+          }
+        };
 
   const isAccTreeEligible =
     helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
@@ -60,9 +71,14 @@ function runInPage(ctx) {
     helpers && typeof helpers.getEligibilityInfo === 'function' ? helpers.getEligibilityInfo : null;
 
   const getAriaLabelledByInfo =
-    helpers && typeof helpers.getAriaLabelledByInfo === 'function' ? helpers.getAriaLabelledByInfo : null;
+    helpers && typeof helpers.getAriaLabelledByInfo === 'function'
+      ? helpers.getAriaLabelledByInfo
+      : null;
 
-  const normalizeWs = (s) => String(s || '').replace(/\s+/g, ' ').trim();
+  const normalizeWs = (s) =>
+    String(s || '')
+      .replace(/\s+/g, ' ')
+      .trim();
 
   const getAttr = (el, name) => {
     try {
@@ -73,7 +89,6 @@ function runInPage(ctx) {
     }
   };
 
-
   // aria-labelledby resolution: delegate to the shared helper (used by
   // button-name-present et al.), which correctly includes hidden/aria-hidden
   // referenced nodes per the Accessible Name and Description Computation
@@ -120,7 +135,9 @@ function runInPage(ctx) {
 
   const nodes = (() => {
     try {
-      return (queryAllSmart ? queryAllSmart(selector, safeRoot) : queryAll(selector, safeRoot)) || [];
+      return (
+        (queryAllSmart ? queryAllSmart(selector, safeRoot) : queryAll(selector, safeRoot)) || []
+      );
     } catch {
       return queryAll(selector, safeRoot);
     }
@@ -158,9 +175,15 @@ function runInPage(ctx) {
     const ok = !!(ariaLabel || labelled || title);
     if (ok) continue;
 
-    const eligInfo = getEligibilityInfo ? (() => {
-      try { return getEligibilityInfo(el, ctx, { targetSet: 'acc' }); } catch { return null; }
-    })() : null;
+    const eligInfo = getEligibilityInfo
+      ? (() => {
+          try {
+            return getEligibilityInfo(el, ctx, { targetSet: 'acc' });
+          } catch {
+            return null;
+          }
+        })()
+      : null;
 
     const baseOccurrence = {
       summary: 'This element has no accessible name.',
@@ -195,7 +218,12 @@ function runInPage(ctx) {
     return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
   }
 
-  return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'minor', occurrences };
+  return {
+    ruleId: rule.ruleId,
+    outcome: 'fail',
+    severity: rule.defaultSeverity || 'minor',
+    occurrences
+  };
 }
 
 module.exports = { id, meta, runInPage };

package/src/checks/automatic/aria-roles-valid.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -25,7 +27,8 @@ const id = 'aria-roles-valid';
 
 const meta = {
   title: 'role attribute must be a valid, non-abstract ARIA role',
-  description: 'Checks that an explicit role="" attribute resolves to a real, non-abstract WAI-ARIA role.',
+  description:
+    'Checks that an explicit role="" attribute resolves to a real, non-abstract WAI-ARIA role.',
   i18n: {
     titleKey: 'ariaRolesValid_title',
     descriptionKey: 'ariaRolesValid_description'
@@ -34,7 +37,13 @@ const meta = {
   tags: ['wcag2a', 'wcag412', 'aria', 'structure', 'atomic', 'automatic'],
   wcagSc: ['4.1.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '4.1.2', title: 'Name, Role, Value', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '4.1.2',
+      title: 'Name, Role, Value',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'robust',
@@ -44,14 +53,16 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
   const ariaHelpers = helpers && helpers.aria ? helpers.aria : null;
   if (!ariaHelpers) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
 
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart('[role]') : helpers.queryAll('[role]');
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart('[role]')
+    : helpers.queryAll('[role]');
 
   const occurrences = [];
   let applicableCount = 0;
@@ -71,7 +82,7 @@ function runInPage(ctx) {
 
     const reasonCode = !isKnown ? 'ARIA_ROLE_INVALID' : 'ARIA_ROLE_ABSTRACT';
     const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : (el.outerHTML || '');
+    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
 
     occurrences.push({
       selector: stableSelector,
@@ -83,12 +94,8 @@ function runInPage(ctx) {
         ? 'Use a valid ARIA role token, or remove the role attribute if none applies.'
         : 'Replace this abstract role with a concrete role appropriate for the widget/structure.',
       i18n: {
-        summaryKey: !isKnown
-          ? 'ariaRolesValid_summary_invalid'
-          : 'ariaRolesValid_summary_abstract',
-        hintKey: !isKnown
-          ? 'ariaRolesValid_hint_invalid'
-          : 'ariaRolesValid_hint_abstract',
+        summaryKey: !isKnown ? 'ariaRolesValid_summary_invalid' : 'ariaRolesValid_summary_abstract',
+        hintKey: !isKnown ? 'ariaRolesValid_hint_invalid' : 'ariaRolesValid_hint_abstract',
         params: { role }
       },
       data: {
@@ -101,7 +108,12 @@ function runInPage(ctx) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
   if (occurrences.length) {
-    return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'serious', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'serious',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }

package/src/checks/automatic/aria-valid-attr-value.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -13,32 +15,34 @@
  * @expectation
  *   Each attribute's value conforms to its WAI-ARIA-declared value type:
  *   boolean ("true"/"false"), tristate ("true"/"false"/"mixed"), a token
- *   from a fixed enumerated set, an integer, a real number, or a non-empty
- *   ID reference (list) that resolves to an existing element in the
- *   document.
+ *   from a fixed enumerated set, an integer, a real number, or an empty
+ *   value or ID reference (list) that resolves to an existing element in
+ *   the document.
  * @implementation-notes
  * - Not rule-gated on isAccTreeEligible: this remains a static-markup
  *   property, while engine-level hidden-subtree filtering still applies
  *   unless engineOptions.includeHiddenElements is true.
- * - ID-reference resolution (added 2026-07-20, see aria-helpers.js's
- *   idExists) only flags idref-list attributes (aria-labelledby,
- *   aria-describedby, aria-controls, aria-owns, etc.) when NONE of the
- *   space-separated ids resolve — a partially-dangling list (some ids
- *   exist, some don't) is left unflagged. Verified 2026-07-21 directly
- *   against a widely-used reference engine's own `validateAttrValue` source:
- *   this is not a conservative guess, it's an exact match for that engine's
- *   own behavior (`idrefs(vNode, attr).some(node => !!node)` — that engine
- *   itself only invalidates when every token fails to resolve). Single-idref
- *   attributes (aria-activedescendant, aria-errormessage) are flagged
- *   whenever their one id doesn't resolve, also matching that engine's `idref`
- *   case exactly.
+ * - ID-reference resolution (see aria-helpers.js's idExists) only flags
+ *   idref-list attributes (aria-labelledby, aria-describedby,
+ *   aria-controls, aria-owns, etc.) when NONE of the space-separated ids
+ *   resolve — a partially-dangling list (some ids exist, some don't) is
+ *   left unflagged (only invalidate when every token fails to resolve).
+ *   Single-idref attributes (aria-activedescendant, aria-errormessage) are
+ *   flagged whenever their one id doesn't resolve.
+ * - An explicitly-EMPTY idref/idref-list value (e.g.
+ *   `aria-describedby=""`) is valid, not a violation — `allowEmpty` holds
+ *   for every idref/idref-list ARIA attribute. A common, deliberate pattern
+ *   in templated markup (e.g. React conditionally rendering
+ *   `aria-describedby={hasError ? errorId : ''}`); flagging it would be a
+ *   false positive.
  */
 
 const id = 'aria-valid-attr-value';
 
 const meta = {
   title: 'aria-* attribute values must match their declared type',
-  description: 'Checks that every recognized aria-* attribute has a value conforming to its WAI-ARIA-declared value type (boolean, tristate, token, integer, number, or ID reference).',
+  description:
+    'Checks that every recognized aria-* attribute has a value conforming to its WAI-ARIA-declared value type (boolean, tristate, token, integer, number, or ID reference).',
   i18n: {
     titleKey: 'ariaValidAttrValue_title',
     descriptionKey: 'ariaValidAttrValue_description'
@@ -47,7 +51,13 @@ const meta = {
   tags: ['wcag2a', 'wcag412', 'aria', 'structure', 'atomic', 'automatic'],
   wcagSc: ['4.1.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '4.1.2', title: 'Name, Role, Value', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '4.1.2',
+      title: 'Name, Role, Value',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'robust',
@@ -57,7 +67,7 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
   const ariaHelpers = helpers && helpers.aria ? helpers.aria : null;
   if (!ariaHelpers) {
@@ -85,14 +95,18 @@ function runInPage(ctx) {
       const result = ariaHelpers.validateAttrValue(name, rawValue);
       if (!result.valid) {
         if (!invalid) invalid = [];
-        invalid.push({ name, value: rawValue == null ? '' : String(rawValue), reason: result.reason });
+        invalid.push({
+          name,
+          value: rawValue == null ? '' : String(rawValue),
+          reason: result.reason
+        });
       }
     }
 
     if (!invalid || !invalid.length) continue;
 
     const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : (el.outerHTML || '');
+    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
 
     for (const item of invalid) {
       occurrences.push({
@@ -106,7 +120,12 @@ function runInPage(ctx) {
           params: { attr: item.name, value: item.value }
         },
         data: {
-          details: { reasonCode: 'ARIA_ATTR_VALUE_INVALID', attr: item.name, value: item.value, valueReason: item.reason }
+          details: {
+            reasonCode: 'ARIA_ATTR_VALUE_INVALID',
+            attr: item.name,
+            value: item.value,
+            valueReason: item.reason
+          }
         }
       });
     }
@@ -116,7 +135,12 @@ function runInPage(ctx) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
   if (occurrences.length) {
-    return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'serious', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'serious',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }

package/src/checks/automatic/aria-valid-attr.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -27,7 +29,8 @@ const id = 'aria-valid-attr';
 
 const meta = {
   title: 'aria-* attributes must be real, defined ARIA attributes',
-  description: 'Checks that every aria-* attribute name present in the DOM is a real attribute defined by the WAI-ARIA specification.',
+  description:
+    'Checks that every aria-* attribute name present in the DOM is a real attribute defined by the WAI-ARIA specification.',
   i18n: {
     titleKey: 'ariaValidAttr_title',
     descriptionKey: 'ariaValidAttr_description'
@@ -36,7 +39,13 @@ const meta = {
   tags: ['wcag2a', 'wcag412', 'aria', 'structure', 'atomic', 'automatic'],
   wcagSc: ['4.1.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '4.1.2', title: 'Name, Role, Value', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '4.1.2',
+      title: 'Name, Role, Value',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'robust',
@@ -46,7 +55,7 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
   const ariaHelpers = helpers && helpers.aria ? helpers.aria : null;
   if (!ariaHelpers) {
@@ -78,7 +87,7 @@ function runInPage(ctx) {
     if (!invalidNames || !invalidNames.length) continue;
 
     const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : (el.outerHTML || '');
+    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
 
     for (const name of invalidNames) {
       occurrences.push({
@@ -102,7 +111,12 @@ function runInPage(ctx) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
   if (occurrences.length) {
-    return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'serious', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'serious',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }