@surea11y/core 1.2.0 → 1.4.0

package/src/checks/automatic/video-poster-text-alternative-present.js

@@ -1,3 +1,5 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
@@ -37,7 +39,13 @@ const meta = {
   tags: ['wcag2a', 'wcag111', 'media', 'video', 'atomic', 'automatic'],
   wcagSc: ['1.1.1'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '1.1.1', title: 'Non-text Content', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '1.1.1',
+      title: 'Non-text Content',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'perceivable',
@@ -54,32 +62,28 @@ function runInPage(ctx) {
   const { document, root, helpers, rule } = ctx;
   const safeRoot = root || document;
 
-  const reportOccurrence = helpers && typeof helpers.reportOccurrence === 'function'
-      ? helpers.reportOccurrence
-      : null;
+  const reportOccurrence =
+    helpers && typeof helpers.reportOccurrence === 'function' ? helpers.reportOccurrence : null;
 
-  const queryAllSmart = helpers && typeof helpers.queryAllSmart === 'function' ? helpers.queryAllSmart : null;
-  const queryAll = helpers && typeof helpers.queryAll === 'function'
-      ? helpers.queryAll
-      : null;
+  const queryAllSmart =
+    helpers && typeof helpers.queryAllSmart === 'function' ? helpers.queryAllSmart : null;
+  const queryAll = helpers && typeof helpers.queryAll === 'function' ? helpers.queryAll : null;
 
-  const getEligibilityInfo = helpers && typeof helpers.getEligibilityInfo === 'function'
-    ? helpers.getEligibilityInfo
-    : null;
+  const getEligibilityInfo =
+    helpers && typeof helpers.getEligibilityInfo === 'function' ? helpers.getEligibilityInfo : null;
 
-  const isAccTreeEligible = helpers && typeof helpers.isAccTreeEligible === 'function'
-    ? helpers.isAccTreeEligible
-    : null;
+  const isAccTreeEligible =
+    helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
 
-  const isFocusableInfo = helpers && typeof helpers.getFocusableInfo === 'function'
-    ? helpers.getFocusableInfo
-    : null;
+  const isFocusableInfo =
+    helpers && typeof helpers.getFocusableInfo === 'function' ? helpers.getFocusableInfo : null;
 
-  const getAriaNameInfo = helpers && typeof helpers.getAriaNameInfo === 'function'
-    ? helpers.getAriaNameInfo
-    : null;
+  const getAriaNameInfo =
+    helpers && typeof helpers.getAriaNameInfo === 'function' ? helpers.getAriaNameInfo : null;
 
-  function trim(v) { return (v == null ? '' : String(v)).trim(); }
+  function trim(v) {
+    return (v == null ? '' : String(v)).trim();
+  }
 
   function computeNameInfo(el) {
     // <video> is not a labelable element (no browser computes an accessible
@@ -89,11 +93,22 @@ function runInPage(ctx) {
     let aria = null;
 
     if (getAriaNameInfo) {
-      aria = (() => { try { return getAriaNameInfo(el, ctx); } catch { return null; } })();
+      aria = (() => {
+        try {
+          return getAriaNameInfo(el, ctx);
+        } catch {
+          return null;
+        }
+      })();
     }
 
     if (aria && aria.present && trim(aria.value)) {
-      return { present: true, value: trim(aria.value), mechanism: aria.mechanism || 'aria', flags: (aria.flags || []).slice(0) };
+      return {
+        present: true,
+        value: trim(aria.value),
+        mechanism: aria.mechanism || 'aria',
+        flags: (aria.flags || []).slice(0)
+      };
     }
 
     const title = trim(el.getAttribute && el.getAttribute('title'));
@@ -113,14 +128,22 @@ function runInPage(ctx) {
     try {
       if (queryAllSmart) return Array.from(queryAllSmart('video') || []);
       if (queryAll) return Array.from(queryAll('video') || []);
-      return safeRoot && safeRoot.querySelectorAll ? Array.from(safeRoot.querySelectorAll('video')) : [];
+      return safeRoot && safeRoot.querySelectorAll
+        ? Array.from(safeRoot.querySelectorAll('video'))
+        : [];
     } catch {
-      try { return safeRoot && safeRoot.querySelectorAll ? Array.from(safeRoot.querySelectorAll('video')) : []; }
-      catch { return []; }
+      try {
+        return safeRoot && safeRoot.querySelectorAll
+          ? Array.from(safeRoot.querySelectorAll('video'))
+          : [];
+      } catch {
+        return [];
+      }
     }
   })();
 
-  if (!videos.length) return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
+  if (!videos.length)
+    return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
 
   const occurrences = [];
   let applicableCount = 0;
@@ -134,7 +157,11 @@ function runInPage(ctx) {
     // Eligibility: only elements exposed to AT (with helper exceptions)
     if (isAccTreeEligible) {
       const elig = (() => {
-        try { return isAccTreeEligible(el, ctx); } catch { return { eligible: true, reasons: [] }; }
+        try {
+          return isAccTreeEligible(el, ctx);
+        } catch {
+          return { eligible: true, reasons: [] };
+        }
       })();
       if (elig && elig.eligible === false) continue;
     }
@@ -142,13 +169,20 @@ function runInPage(ctx) {
     // Role presentation/none excluded ONLY if not focusable (mirrors img behavior)
     const role = trim(el.getAttribute('role')).toLowerCase();
     if (role === 'presentation' || role === 'none') {
-      let focusable = false;
+      let focusable;
       if (isFocusableInfo) {
-        const fi = (() => { try { return isFocusableInfo(el, ctx); } catch { return null; } })();
+        const fi = (() => {
+          try {
+            return isFocusableInfo(el, ctx);
+          } catch {
+            return null;
+          }
+        })();
         focusable = !!(fi && fi.focusable);
       } else {
         const tabindex = el.getAttribute('tabindex');
-        focusable = tabindex != null && trim(tabindex) !== '' && !Number.isNaN(Number(trim(tabindex)));
+        focusable =
+          tabindex != null && trim(tabindex) !== '' && !Number.isNaN(Number(trim(tabindex)));
       }
       if (!focusable) continue;
     }
@@ -160,12 +194,16 @@ function runInPage(ctx) {
 
     let eligInfo = null;
     if (getEligibilityInfo) {
-      try { eligInfo = getEligibilityInfo(el, ctx, { targetSet: 'acc' }); } catch { eligInfo = null; }
+      try {
+        eligInfo = getEligibilityInfo(el, ctx, { targetSet: 'acc' });
+      } catch {
+        eligInfo = null;
+      }
     }
 
     const baseOccurrence = {
       summary: 'Missing text alternative for <video> poster.',
-      hint: 'Provide an accessible name (e.g., aria-label/aria-labelledby) for the poster image.',
+      hint: 'Provide an accessible name for the poster image (aria-label/aria-labelledby preferred, or a title attribute as a fallback).',
       i18n: {
         summaryKey: 'videoPoster_textAltPresent_summary_fail',
         hintKey: 'videoPoster_textAltPresent_hint_fail',
@@ -185,9 +223,16 @@ function runInPage(ctx) {
     }
   }
 
-  if (applicableCount === 0) return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
-  if (!occurrences.length) return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
-  return { ruleId: rule.ruleId, outcome: 'fail', severity: rule.defaultSeverity || 'minor', occurrences };
+  if (applicableCount === 0)
+    return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
+  if (!occurrences.length)
+    return { ruleId: rule.ruleId, outcome: 'pass', severity: 'minor', occurrences: [] };
+  return {
+    ruleId: rule.ruleId,
+    outcome: 'fail',
+    severity: rule.defaultSeverity || 'minor',
+    occurrences
+  };
 }
 
 module.exports = { id, meta, runInPage };

package/src/checks/manual/accesskeys-manual.js

@@ -1,10 +1,12 @@
+/* SPDX-License-Identifier: MPL-2.0 */
+
 'use strict';
 
 /**
  * @check accesskeys
  * @atomic true
  * @summary accesskey values must be unique on the page
- * @standard Best Practices (a widely-used reference engine's classification; no formal WCAG Success Criterion — see ROADMAP.md Tier 1b)
+ * @standard Best Practices (no formal WCAG Success Criterion — see ROADMAP.md Tier 1b)
  * @applicability
  *   Applies whenever two or more elements share the same non-empty
  *   accesskey attribute value (case-insensitive).
@@ -40,8 +42,10 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart('[accesskey]') : helpers.queryAll('[accesskey]');
+  const { helpers, rule } = ctx;
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart('[accesskey]')
+    : helpers.queryAll('[accesskey]');
 
   const groups = new Map(); // normalized key -> elements[]
   for (const el of nodes) {
@@ -59,12 +63,14 @@ function runInPage(ctx) {
     if (els.length <= 1) continue;
     for (const el of els) {
       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,
         html,
-        summary: 'This element\'s accesskey is shared with another element on the page.',
+        summary: "This element's accesskey is shared with another element on the page.",
         hint: 'Make each accesskey value unique across the page.',
         i18n: {
           summaryKey: 'accesskeys_summary_cantTell',
@@ -81,7 +87,12 @@ function runInPage(ctx) {
   if (!occurrences.length) {
     return { ruleId: rule.ruleId, outcome: 'notApplicable', severity: 'minor', occurrences: [] };
   }
-  return { ruleId: rule.ruleId, outcome: 'cantTell', severity: rule.defaultSeverity || 'minor', occurrences };
+  return {
+    ruleId: rule.ruleId,
+    outcome: 'cantTell',
+    severity: rule.defaultSeverity || 'minor',
+    occurrences
+  };
 }
 
 module.exports = { id, meta, runInPage };