@surea11y/core 1.2.0 → 1.4.0

package/src/checks/automatic/css-orientation-lock.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -20,25 +22,17 @@
  *   display orientation unless that orientation is essential.
  * @implementation-notes
  * - The rotation DEGREE is what makes this the exploit signature, not
- *   merely the presence of a `rotate()` function — verified against a
- *   widely-used reference engine's own `css-orientation-lock` check
- *   (`cssOrientationLockEvaluate`), which computes the actual rotation angle and only
- *   flags ~90/~270 degrees, explicitly excluding ~0/~180 (a no-op or a
- *   flip, neither of which changes portrait<->landscape). A prior
- *   version of this rule flagged the bare presence of "rotate(" in an
- *   orientation-media-query rule regardless of angle — a real, confirmed
- *   false positive found via the live-DOM cross-engine corpus 2026-07-23
- *   on a real page (Times of India) whose `@media (orientation:portrait)`
- *   block only rotated a small decorative arrow icon 45 degrees, nothing
- *   resembling a page-orientation lock.
+ *   merely the presence of a `rotate()` function: a small decorative icon
+ *   rotated 45 degrees inside an orientation media query is not a page
+ *   lock. Compute the actual rotation angle and only flag ~90/~270 degrees,
+ *   excluding ~0/~180 (a no-op or a flip, neither of which changes
+ *   portrait<->landscape).
  * - Only `rotate`/`rotateZ` (transform functions) and the standalone CSS
- *   `rotate` property are parsed for degrees — matches that reference engine's
- *   own two sources (`style.transform`/`style.rotate`) but, like that engine,
- *   does not attempt to decompose `matrix()`/`matrix3d()`/`rotate3d()` into an
- *   equivalent angle (that engine does handle those; deliberately deferred here
- *   as the same class of "genuinely more complex, lower real-world
- *   value" work already deferred elsewhere in this engine, e.g.
- *   `table-th-has-data-cells`'s narrower positional-header algorithm).
+ *   `rotate` property are parsed for degrees, from the two sources
+ *   `style.transform`/`style.rotate`; `matrix()`/`matrix3d()`/`rotate3d()`
+ *   are not decomposed into an equivalent angle (deliberately deferred as
+ *   the same class of higher-complexity/lower-value work deferred elsewhere,
+ *   e.g. `table-th-has-data-cells`'s narrower positional-header algorithm).
  * - Cross-origin stylesheets throw on `.cssRules` access (browser
  *   security model) and are skipped — same class of limitation as any
  *   check that can only see same-origin/inspectable content (compare
@@ -64,7 +58,13 @@ const meta = {
   tags: ['wcag21aa', 'wcag134', 'structure', 'atomic', 'automatic'],
   wcagSc: ['1.3.4'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '1.3.4', title: 'Orientation', conformanceLevel: 'AA' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '1.3.4',
+      title: 'Orientation',
+      conformanceLevel: 'AA'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'perceivable',
@@ -87,12 +87,14 @@ function runInPage(ctx) {
   const CSS_MEDIA_RULE = 4;
   const CSS_STYLE_RULE = 1;
 
-  function trim(v) { return (v == null ? '' : String(v)).trim(); }
+  function trim(v) {
+    return (v == null ? '' : String(v)).trim();
+  }
 
   // Converts a rotate()/rotateZ() angle argument (with unit) to degrees.
-  // Matches a widely-used reference engine's own getAngleInDegrees: only deg/grad/rad/turn are
-  // recognized; a unitless or unrecognized value contributes 0 (ignored,
-  // not treated as a lock -- an unparseable angle isn't a confirmed exploit).
+  // Only deg/grad/rad/turn are recognized; a unitless or unrecognized value
+  // contributes 0 (ignored, not treated as a lock -- an unparseable angle
+  // isn't a confirmed exploit).
   function angleToDegrees(raw) {
     const m = /(-?[\d.]+)\s*(deg|grad|rad|turn)/i.exec(raw);
     if (!m) return 0;
@@ -119,16 +121,20 @@ function runInPage(ctx) {
     return total;
   }
 
-  // Whole-page-orientation-lock detection, matching a widely-used reference engine's own degree
-  // math exactly (see @implementation-notes): a rotation near 0 or 180
-  // degrees (mod 180) is a no-op or a flip, neither of which changes
-  // portrait<->landscape, so it's explicitly NOT a lock; only a rotation
-  // near 90 or 270 degrees (mod 90, once the 0/180 case is excluded) is.
+  // Whole-page-orientation-lock detection (see @implementation-notes): a
+  // rotation near 0 or 180 degrees (mod 180) is a no-op or a flip, neither
+  // of which changes portrait<->landscape, so it's NOT a lock; only a
+  // rotation near 90 or 270 degrees (mod 90, once the 0/180 case is
+  // excluded) is.
   function isLockingRotation(styleDecl) {
     if (!styleDecl) return false;
-    const transformVal = trim(styleDecl.transform) || trim(styleDecl.getPropertyValue ? styleDecl.getPropertyValue('-webkit-transform') : '');
-    // The standalone CSS `rotate` property (distinct from `transform: rotate()`) -- that reference engine checks this too.
-    const rotatePropVal = trim(styleDecl.getPropertyValue ? styleDecl.getPropertyValue('rotate') : '');
+    const transformVal =
+      trim(styleDecl.transform) ||
+      trim(styleDecl.getPropertyValue ? styleDecl.getPropertyValue('-webkit-transform') : '');
+    // The standalone CSS `rotate` property (distinct from `transform: rotate()`).
+    const rotatePropVal = trim(
+      styleDecl.getPropertyValue ? styleDecl.getPropertyValue('rotate') : ''
+    );
 
     let degrees = rotateDegreesFromTransform(transformVal);
     if (!degrees && rotatePropVal) degrees = angleToDegrees(rotatePropVal);
@@ -189,9 +195,10 @@ function runInPage(ctx) {
     return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
   }
 
-  const target = document.documentElement || (document.body || null);
+  const target = document.documentElement || document.body || null;
   const stableSelector = helpers.buildSelector && target ? helpers.buildSelector(target) : 'html';
-  const html = helpers.getOuterHtmlSnippet && target ? helpers.getOuterHtmlSnippet(target) : '<html>';
+  const html =
+    helpers.getOuterHtmlSnippet && target ? helpers.getOuterHtmlSnippet(target) : '<html>';
 
   const occurrences = findings.map((f) => ({
     selector: stableSelector,
@@ -204,11 +211,20 @@ function runInPage(ctx) {
       params: { mediaText: f.mediaText, selectorText: f.selectorText }
     },
     data: {
-      details: { reasonCode: 'ORIENTATION_MEDIA_ROTATE_TRANSFORM', mediaText: f.mediaText, selectorText: f.selectorText }
+      details: {
+        reasonCode: 'ORIENTATION_MEDIA_ROTATE_TRANSFORM',
+        mediaText: f.mediaText,
+        selectorText: f.selectorText
+      }
     }
   }));
 
-  return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'serious', occurrences };
+  return {
+    ruleId: rule.ruleId,
+    outcome: 'fail',
+    severity: rule.defaultSeverity || 'serious',
+    occurrences
+  };
 }
 
 module.exports = { id, meta, runInPage, applicability };

package/src/checks/automatic/definition-list-children-valid.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -18,12 +20,11 @@
  *   violation (see implementation-notes). Any other direct or wrapped child
  *   breaks the description-list semantics assistive technologies rely on.
  * @implementation-notes
- * - Only one level of <div> wrapping is flattened, matching a widely-used
- *   reference engine's definition-list algorithm — a <div> nested inside
+ * - Only one level of <div> wrapping is flattened — a <div> nested inside
  *   another wrapping <div> is not flattened further and its contents are
  *   reported invalid.
- * - Verified 2026-07-20 against a widely-used reference engine: the dt/dd pairing is
- *   only required "when not empty". A flattened set with NEITHER dt nor dd
+ * - The dt/dd pairing is only required "when not empty". A flattened set
+ *   with NEITHER dt nor dd
  *   — whether from an empty wrapping <div>, only <script>/<template>/
  *   <style> content, or a genuinely childless <dl> — is not flagged; only
  *   an unbalanced dt/dd pairing is a real structural problem.
@@ -35,7 +36,8 @@ const id = 'definition-list-children-valid';
 
 const meta = {
   title: 'Description lists must be structured correctly',
-  description: 'Checks that <dl> elements only directly contain <dt>/<dd> groups (optionally wrapped in one <div>), <script>, <template>, or <style>.',
+  description:
+    'Checks that <dl> elements only directly contain <dt>/<dd> groups (optionally wrapped in one <div>), <script>, <template>, or <style>.',
   i18n: {
     titleKey: 'definitionListChildrenValid_title',
     descriptionKey: 'definitionListChildrenValid_description'
@@ -44,7 +46,13 @@ const meta = {
   tags: ['wcag2a', 'wcag131', 'structure', 'atomic', 'automatic', 'list'],
   wcagSc: ['1.3.1'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '1.3.1', title: 'Info and Relationships', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '1.3.1',
+      title: 'Info and Relationships',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'perceivable',
@@ -54,7 +62,7 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
   // Declared inside runInPage — see scripts/build-core.js header
   // ("runInPage MUST be self-contained").
@@ -89,31 +97,39 @@ function runInPage(ctx) {
     const invalidTags = [];
     for (const node of flattened) {
       const tag = node.tagName.toLowerCase();
-      if (tag === 'dt') { hasDt = true; continue; }
-      if (tag === 'dd') { hasDd = true; continue; }
+      if (tag === 'dt') {
+        hasDt = true;
+        continue;
+      }
+      if (tag === 'dd') {
+        hasDd = true;
+        continue;
+      }
       if (!PASSTHROUGH_TAGS.has(tag)) invalidTags.push(tag);
     }
+    const dedupedInvalidTags = [...new Set(invalidTags)];
 
-    // Verified 2026-07-20 against a widely-used reference engine's definition-list check:
-    // the dt/dd pairing is only required "when not empty" — a <dl> with
+    // The dt/dd pairing is only required "when not empty" — a <dl> with
     // NEITHER dt nor dd (whether genuinely childless after flattening, only
     // passthrough script/template/style content, or an empty wrapping div)
     // is vacuously fine, not a violation. Only an UNBALANCED pairing (dt
     // present without any dd, or vice versa) is a real structural problem.
     const reasonCode = invalidTags.length
-        ? 'DL_INVALID_CHILD'
-        : ((hasDt || hasDd) && !(hasDt && hasDd) ? 'DL_NO_DT_DD' : null);
+      ? 'DL_INVALID_CHILD'
+      : (hasDt || hasDd) && !(hasDt && hasDd)
+        ? 'DL_NO_DT_DD'
+        : null;
     if (!reasonCode) 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 || '';
 
     const summary = invalidTags.length
-        ? 'This description list contains a direct or wrapped child that is not part of a dt/dd group.'
-        : 'This description list has no <dt>/<dd> term-definition group.';
+      ? 'This description list contains a direct or wrapped child that is not part of a dt/dd group.'
+      : 'This description list has no <dt>/<dd> term-definition group.';
     const hint = invalidTags.length
-        ? 'Only use <dt>/<dd> (optionally wrapped in one <div>), <script>, <template>, or <style> inside <dl>.'
-        : 'Add at least one <dt>/<dd> pair inside this <dl>.';
+      ? 'Only use <dt>/<dd> (optionally wrapped in one <div>), <script>, <template>, or <style> inside <dl>.'
+      : 'Add at least one <dt>/<dd> pair inside this <dl>.';
 
     occurrences.push({
       selector: stableSelector,
@@ -122,12 +138,12 @@ function runInPage(ctx) {
       hint,
       i18n: {
         summaryKey: invalidTags.length
-            ? 'definitionListChildrenValid_summary_fail_invalidChild'
-            : 'definitionListChildrenValid_summary_fail_noDtDd',
+          ? 'definitionListChildrenValid_summary_fail_invalidChild'
+          : 'definitionListChildrenValid_summary_fail_noDtDd',
         hintKey: invalidTags.length
-            ? 'definitionListChildrenValid_hint_fail_invalidChild'
-            : 'definitionListChildrenValid_hint_fail_noDtDd',
-        params: { invalidChildren: invalidTags.join(', ') }
+          ? 'definitionListChildrenValid_hint_fail_invalidChild'
+          : 'definitionListChildrenValid_hint_fail_noDtDd',
+        params: { invalidChildren: dedupedInvalidTags.join(', ') }
       },
       data: {
         details: { reasonCode, invalidChildren: invalidTags }
@@ -139,9 +155,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 || 'serious', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'serious',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }
 
-module.exports = { id, meta, runInPage };
+module.exports = { id, meta, runInPage };

package/src/checks/automatic/deprecated-elements-not-used.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -14,8 +16,8 @@
  * @expectation
  *   Neither element is present. Since their movement can never be paused,
  *   stopped, or hidden by the user, presence is itself the violation — this
- *   rule has no partial-pass case, matching a widely-used reference engine's `blink`/`marquee`
- *   rules (which report only when the element is found).
+ *   rule has no partial-pass case (it reports only when the element is
+ *   found).
  * @implementation-notes
  * - Not rule-gated on isAccTreeEligible: presence in markup is itself the
  *   violation, independent of visibility (moving/blinking content inside a
@@ -29,7 +31,8 @@ const id = 'deprecated-elements-not-used';
 
 const meta = {
   title: 'Obsolete non-stoppable elements (<blink>, <marquee>) must not be used',
-  description: 'Checks that deprecated, non-standard HTML elements whose blinking/scrolling content cannot be paused, stopped, or hidden by the user (<blink>, <marquee>) are not present.',
+  description:
+    'Checks that deprecated, non-standard HTML elements whose blinking/scrolling content cannot be paused, stopped, or hidden by the user (<blink>, <marquee>) are not present.',
   i18n: {
     titleKey: 'deprecatedElements_title',
     descriptionKey: 'deprecatedElements_description'
@@ -38,7 +41,13 @@ const meta = {
   tags: ['wcag2a', 'wcag222', 'structure', 'atomic', 'automatic'],
   wcagSc: ['2.2.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '2.2.2', title: 'Pause, Stop, Hide', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '2.2.2',
+      title: 'Pause, Stop, Hide',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'operable',
@@ -48,9 +57,11 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart('blink, marquee') : helpers.queryAll('blink, marquee');
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart('blink, marquee')
+    : helpers.queryAll('blink, marquee');
 
   const occurrences = [];
   let applicableCount = 0;
@@ -62,7 +73,7 @@ function runInPage(ctx) {
 
     const tag = el.tagName.toLowerCase();
     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,
@@ -84,9 +95,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 || 'serious', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'serious',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }
 
-module.exports = { id, meta, runInPage };
+module.exports = { id, meta, runInPage };

package/src/checks/automatic/dialog-name-present.js

@@ -1,19 +1,28 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 const id = 'dialog-name-present';
 
 const meta = {
   title: 'Dialogs have an accessible name',
-  description: 'Checks that elements with role="dialog" or role="alertdialog" expose a non-empty accessible name.',
+  description:
+    'Checks that elements with role="dialog" or role="alertdialog" expose a non-empty accessible name.',
   i18n: {
     titleKey: 'dialogNamePresent_title',
     descriptionKey: 'dialogNamePresent_description'
   },
   helpUrl: null,
-  tags: ['wcag2a','wcag412','structure','atomic','automatic','name','dialog'],
+  tags: ['wcag2a', 'wcag412', 'structure', 'atomic', 'automatic', 'name', 'dialog'],
   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',
@@ -24,69 +33,22 @@ const meta = {
 
 function runInPage(ctx) {
   const { document, helpers, rule } = ctx;
-  const getEligibilityInfo = helpers && typeof helpers.getEligibilityInfo === 'function'
-      ? helpers.getEligibilityInfo
-      : null;
-
+  const getEligibilityInfo =
+    helpers && typeof helpers.getEligibilityInfo === 'function' ? helpers.getEligibilityInfo : null;
 
   function normalizeWs(s) {
-    return String(s || '').replace(/\s+/g, ' ').trim();
+    return String(s || '')
+      .replace(/\s+/g, ' ')
+      .trim();
   }
 
   function getAttr(el, name) {
     try {
       if (!el || !el.getAttribute) return '';
       return normalizeWs(el.getAttribute(name));
-    } catch { return ''; }
-  }
-
-  function hasAttr(el, name) {
-    try { return !!(el && el.hasAttribute && el.hasAttribute(name)); } catch { return false; }
-  }
-
-  function isExplicitProgrammatic(el) {
-    // Programmatic name mechanisms we treat as authoritative (presence-only):
-    // - aria-label (non-empty)
-    // - aria-labelledby (non-empty)
-    // - title (non-empty) [weak but allowed for presence]
-    const al = getAttr(el, 'aria-label');
-    if (al) return true;
-    const alb = getAttr(el, 'aria-labelledby');
-    if (alb) return true;
-    const t = getAttr(el, 'title');
-    if (t) return true;
-    return false;
-  }
-
-  function buildLabelForMap(doc) {
-    const map = new Map(); // id -> label element (first)
-    try {
-      const labels = doc && doc.getElementsByTagName ? doc.getElementsByTagName('label') : [];
-      for (let i = 0; i < labels.length; i += 1) {
-        const lab = labels[i];
-        if (!lab || !lab.getAttribute) continue;
-        const f = normalizeWs(lab.getAttribute('for'));
-        if (!f) continue;
-        if (!map.has(f)) map.set(f, lab);
-      }
-    } catch {}
-    return map;
-  }
-
-  function getConservativeSubtreeText(document, container) {
-    // "Name from content" — recurses into descendants and uses each one's
-    // own accessible name (img alt, aria-label/aria-labelledby, title) when
-    // it has one, not just literal text nodes. See getContentNameInfo's
-    // header comment in src/core/dom-helpers.js for the full rationale
-    // (this replaced a text-node-only TreeWalker that missed the common
-    // "<a><img alt='...'></a>" logo-link / "<button><img alt='...'></button>"
-    // icon-button pattern).
-    if (helpers.getContentNameInfo) {
-      const info = helpers.getContentNameInfo(container, ctx);
-      return info && info.present ? info.value : '';
+    } catch {
+      return '';
     }
-    const t = (container && container.textContent) ? String(container.textContent) : '';
-    return t.replace(/\s+/g, ' ').trim();
   }
 
   function resolveAriaLabelledbyText(document, el, maxRefs) {
@@ -95,16 +57,14 @@ function runInPage(ctx) {
     // Resolve via the shared getTextFromIdRefs helper — computes each
     // referenced element's own ACCESSIBLE NAME (aria-label, then
     // aria-labelledby, then a value-like name, then content, then title),
-    // not just its content text. Found via a real page (BBC News' cookie-
-    // consent dialog, 2026-07-22): aria-labelledby pointed at an
-    // <iframe title="SP Consent Message">, whose only name source is its
-    // title attribute (an iframe's content is opaque/cross-origin per
-    // HTML-AAM, so "name from content" is always empty). The previous
-    // version here only ever computed name-from-content of the referenced
-    // node (via getConservativeSubtreeText), silently missing the title
-    // fallback and reporting no accessible name at all — the identical
-    // pattern was hand-copied into 15 other *-name-present rules; all
-    // fixed the same way.
+    // not just its content text. This matters when aria-labelledby points
+    // at an <iframe title="..."> (e.g. BBC News' cookie-consent dialog,
+    // pointing at an `<iframe title="SP Consent Message">`), whose only
+    // name source is its title attribute — an iframe's content is
+    // opaque/cross-origin per HTML-AAM, so "name from content" is always
+    // empty. Computing only name-from-content of the referenced node would
+    // silently miss the title fallback and report no accessible name at
+    // all, the same pattern every other *-name-present rule guards against.
     if (helpers.getTextFromIdRefs) {
       try {
         const r = helpers.getTextFromIdRefs(raw, ctx, { maxRefs: maxRefs || 8 });
@@ -114,17 +74,9 @@ function runInPage(ctx) {
     return '';
   }
 
-  function getInputValueName(el) {
-    try {
-      const type = getAttr(el, 'type').toLowerCase();
-      if (type !== 'button' && type !== 'submit' && type !== 'reset') return '';
-      const v = getAttr(el, 'value');
-      return v;
-    } catch { return ''; }
-  }
-
   function isEligibleAcc(helpers, el, ctx) {
-    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);
@@ -135,13 +87,13 @@ function runInPage(ctx) {
     }
   }
 
-
   const occurrences = [];
   let applicableCount = 0;
 
   const selector = '[role="dialog"],[role="alertdialog"]';
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart(selector) : helpers.queryAll(selector);
-
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart(selector)
+    : helpers.queryAll(selector);
 
   function evaluate(el) {
     const ariaLabel = getAttr(el, 'aria-label');
@@ -173,10 +125,16 @@ function runInPage(ctx) {
     if (res.ok) continue;
 
     const eligInfo = getEligibilityInfo
-        ? (() => { try { return getEligibilityInfo(el, ctx, { targetSet: 'acc' }); } catch { return null; } })()
-        : null;
+      ? (() => {
+          try {
+            return getEligibilityInfo(el, ctx, { targetSet: 'acc' });
+          } catch {
+            return null;
+          }
+        })()
+      : null;
     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,
@@ -195,12 +153,16 @@ function runInPage(ctx) {
     });
   }
 
-
   if (applicableCount === 0) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
   if (occurrences.length) {
-    return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'minor', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'minor',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }

package/src/checks/automatic/dlitem-parent-valid.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -23,7 +25,8 @@ const id = 'dlitem-parent-valid';
 
 const meta = {
   title: 'Description-list items must be inside a description list',
-  description: 'Checks that <dt>/<dd> elements are contained by a <dl>, directly or via one wrapping <div>.',
+  description:
+    'Checks that <dt>/<dd> elements are contained by a <dl>, directly or via one wrapping <div>.',
   i18n: {
     titleKey: 'dlitemParentValid_title',
     descriptionKey: 'dlitemParentValid_description'
@@ -32,7 +35,13 @@ const meta = {
   tags: ['wcag2a', 'wcag131', 'structure', 'atomic', 'automatic', 'list'],
   wcagSc: ['1.3.1'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '1.3.1', title: 'Info and Relationships', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '1.3.1',
+      title: 'Info and Relationships',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'perceivable',
@@ -42,9 +51,11 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
+  const { helpers, rule } = ctx;
 
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart('dt, dd') : helpers.queryAll('dt, dd');
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart('dt, dd')
+    : helpers.queryAll('dt, dd');
 
   const occurrences = [];
   let applicableCount = 0;
@@ -61,7 +72,8 @@ function runInPage(ctx) {
 
     if (!valid && parentTag === 'div') {
       const grandparent = parent.parentElement;
-      const grandparentTag = grandparent && grandparent.tagName ? grandparent.tagName.toLowerCase() : '';
+      const grandparentTag =
+        grandparent && grandparent.tagName ? grandparent.tagName.toLowerCase() : '';
       valid = grandparentTag === 'dl';
     }
 
@@ -69,7 +81,7 @@ function runInPage(ctx) {
 
     const tag = el.tagName.toLowerCase();
     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,
@@ -91,9 +103,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 || 'serious', occurrences };
+    return {
+      ruleId: rule.ruleId,
+      outcome: 'fail',
+      severity: rule.defaultSeverity || 'serious',
+      occurrences
+    };
   }
   return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
 }
 
-module.exports = { id, meta, runInPage };
+module.exports = { id, meta, runInPage };