@surea11y/core 1.2.0 → 1.3.0

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

@@ -61,7 +61,8 @@ const id = 'aria-required-children';
 
 const meta = {
   title: 'Container roles must own at least one required child role',
-  description: 'Checks that container roles with a documented "required owned elements" entry (list, listbox, menu, radiogroup, table, grid, tablist, tree, row, ...) contain at least one descendant or aria-owns-referenced element with an acceptable owned role.',
+  description:
+    'Checks that container roles with a documented "required owned elements" entry (list, listbox, menu, radiogroup, table, grid, tablist, tree, row, ...) contain at least one descendant or aria-owns-referenced element with an acceptable owned role.',
   i18n: {
     titleKey: 'ariaRequiredChildren_title',
     descriptionKey: 'ariaRequiredChildren_description'
@@ -70,7 +71,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',
@@ -80,7 +87,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) {
@@ -88,7 +95,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);
@@ -109,9 +117,12 @@ function runInPage(ctx) {
   // (kept in sync with aria-helpers.js NATIVE_CONTAINMENT_ROLE_BY_ELEMENT).
   // Declared inside runInPage — see scripts/build-core.js header
   // ("runInPage MUST be self-contained").
-  const CANDIDATE_SELECTOR = '[role], li, option, tr, td, th, thead, tbody, tfoot, ul, ol, table, select, input[type="radio"]';
+  const CANDIDATE_SELECTOR =
+    '[role], li, option, tr, td, th, thead, tbody, tfoot, ul, ol, table, select, input[type="radio"]';
 
-  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;
@@ -139,8 +150,11 @@ function runInPage(ctx) {
       if (out.length >= limit) return;
       if (seen.has(child)) continue;
 
-      if ((child.tagName || '').toLowerCase() === 'slot' && typeof child.assignedElements === 'function') {
-        let assigned = [];
+      if (
+        (child.tagName || '').toLowerCase() === 'slot' &&
+        typeof child.assignedElements === 'function'
+      ) {
+        let assigned;
         try {
           assigned = child.assignedElements({ flatten: true }) || [];
         } catch {
@@ -184,7 +198,7 @@ function runInPage(ctx) {
     // selector, exactly as before this fix — covers the overwhelming
     // majority of containers (no shadow DOM involved at all) with zero
     // added cost.
-    let descendants = [];
+    let descendants;
     try {
       descendants = el.querySelectorAll(CANDIDATE_SELECTOR);
     } catch {
@@ -202,7 +216,7 @@ function runInPage(ctx) {
     // <slot> somewhere in the subtree to expand — bounds the extra cost to
     // exactly the containers that could possibly need it.
     if (!found) {
-      let hasSlot = false;
+      let hasSlot;
       try {
         hasSlot = !!el.querySelector('slot');
       } catch {
@@ -243,7 +257,7 @@ function runInPage(ctx) {
     if (found) 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,
@@ -256,7 +270,11 @@ function runInPage(ctx) {
         params: { role, requiredRoles: requiredOwned.join(', ') }
       },
       data: {
-        details: { reasonCode: 'ARIA_REQUIRED_CHILD_MISSING', role, requiredOwnedRoles: requiredOwned }
+        details: {
+          reasonCode: 'ARIA_REQUIRED_CHILD_MISSING',
+          role,
+          requiredOwnedRoles: requiredOwned
+        }
       }
     });
   }
@@ -265,9 +283,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-required-parent.js

@@ -47,7 +47,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 +57,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 +73,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 +81,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);
@@ -120,14 +128,25 @@ function runInPage(ctx) {
   // 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; };
+  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) — mirroring that
+    // reference engine's getMissingContext, which pushes explicitRole into
+    // reqContext at the same point. 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 +158,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 +173,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 +188,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 +216,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 +229,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 +242,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

@@ -15,7 +15,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 +25,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 +48,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 +69,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 +87,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 +133,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 +173,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 +216,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

@@ -25,7 +25,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 +35,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 +51,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 +80,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 +92,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 +106,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

@@ -38,7 +38,8 @@ 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 +48,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 +64,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 +92,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 +117,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 +132,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

@@ -27,7 +27,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 +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',
@@ -46,7 +53,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 +85,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 +109,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: [] };
 }