@surea11y/core 1.2.0 → 1.3.0

package/src/checks/automatic/embed-text-alternative-present.js

@@ -32,7 +32,13 @@ const meta = {
   tags: ['wcag2a', 'wcag111', 'nontext', 'embed', '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',
@@ -45,33 +51,39 @@ 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) => {
-      try { return safeRoot && safeRoot.querySelectorAll ? Array.from(safeRoot.querySelectorAll(sel)) : []; }
-      catch { return []; }
-    };
-
-  const getEligibilityInfo = helpers && typeof helpers.getEligibilityInfo === 'function'
-    ? helpers.getEligibilityInfo
-    : null;
-
-  const isAccTreeEligible = helpers && typeof helpers.isAccTreeEligible === 'function'
-    ? helpers.isAccTreeEligible
-    : null;
-
-  const getFocusableInfo = helpers && typeof helpers.getFocusableInfo === 'function'
-    ? helpers.getFocusableInfo
-    : null;
-
-  const getAriaNameInfo = helpers && typeof helpers.getAriaNameInfo === 'function'
-    ? helpers.getAriaNameInfo
-    : null;
+  const queryAllSmart =
+    helpers && typeof helpers.queryAllSmart === 'function' ? helpers.queryAllSmart : null;
+  const queryAll =
+    helpers && typeof helpers.queryAll === 'function'
+      ? helpers.queryAll
+      : (sel) => {
+          try {
+            return safeRoot && safeRoot.querySelectorAll
+              ? Array.from(safeRoot.querySelectorAll(sel))
+              : [];
+          } catch {
+            return [];
+          }
+        };
+
+  const getEligibilityInfo =
+    helpers && typeof helpers.getEligibilityInfo === 'function' ? helpers.getEligibilityInfo : null;
+
+  const isAccTreeEligible =
+    helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
+
+  const getFocusableInfo =
+    helpers && typeof helpers.getFocusableInfo === 'function' ? helpers.getFocusableInfo : null;
+
+  const getAriaNameInfo =
+    helpers && typeof helpers.getAriaNameInfo === 'function' ? helpers.getAriaNameInfo : null;
 
   const embeds = (() => {
-    try { return Array.from((queryAllSmart ? queryAllSmart('embed') : queryAll('embed')) || []); }
-    catch { return queryAll('embed'); }
+    try {
+      return Array.from((queryAllSmart ? queryAllSmart('embed') : queryAll('embed')) || []);
+    } catch {
+      return queryAll('embed');
+    }
   })();
 
   if (!embeds.length) {
@@ -89,11 +101,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'));
@@ -120,20 +143,43 @@ function runInPage(ctx) {
     if (!el || !el.getAttribute) continue;
 
     if (isAccTreeEligible) {
-      const elig = (() => { try { return isAccTreeEligible(el, ctx); } catch { return { eligible: true, reasons: [] }; } })();
+      const elig = (() => {
+        try {
+          return isAccTreeEligible(el, ctx);
+        } catch {
+          return { eligible: true, reasons: [] };
+        }
+      })();
       if (elig && elig.eligible === false) continue;
     }
 
     // role presentation/none exclusion only when not focusable
-    const role = (() => { try { return String(el.getAttribute('role') || '').trim().toLowerCase(); } catch { return ''; } })();
+    const role = (() => {
+      try {
+        return String(el.getAttribute('role') || '')
+          .trim()
+          .toLowerCase();
+      } catch {
+        return '';
+      }
+    })();
     if (role === 'presentation' || role === 'none') {
-      let focusable = false;
+      let focusable;
       if (getFocusableInfo) {
-        const fi = (() => { try { return getFocusableInfo(el, ctx); } catch { return null; } })();
+        const fi = (() => {
+          try {
+            return getFocusableInfo(el, ctx);
+          } catch {
+            return null;
+          }
+        })();
         focusable = !!(fi && fi.focusable);
       } else {
         const tabindex = el.getAttribute('tabindex');
-        focusable = tabindex != null && String(tabindex).trim() !== '' && !Number.isNaN(Number(String(tabindex).trim()));
+        focusable =
+          tabindex != null &&
+          String(tabindex).trim() !== '' &&
+          !Number.isNaN(Number(String(tabindex).trim()));
       }
       if (!focusable) continue;
     }
@@ -150,7 +196,7 @@ function runInPage(ctx) {
 
     const baseOccurrence = {
       summary: 'Missing text alternative for <embed>.',
-      hint: 'Add an accessible name to <embed> (aria-label/aria-labelledby).',
+      hint: 'Add an accessible name to <embed> (aria-label/aria-labelledby preferred, or a title attribute as a best-effort fallback).',
       i18n: {
         summaryKey: 'embed_textAltPresent_summary_fail',
         hintKey: 'embed_textAltPresent_hint_fail',
@@ -184,7 +230,12 @@ function runInPage(ctx) {
     };
   }
 
-  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/form-control-programmatic-label-present.js

@@ -1,20 +1,13 @@
 'use strict';
 
 /**
- * Optimized, regression-safe rewrite of:
- * form-control-programmatic-label-present
- *
- * Goals:
- * - Preserve semantics, schema, determinism, and no-throw behavior.
- * - Reduce per-element DOM queries and avoid unnecessary helper calls.
- *
- * Key safety points:
- * - Control discovery goes through helpers.queryAllSmart/queryAll (multi-root
- *   and shadow-DOM aware), not a direct root/safeRoot DOM query -- ctx.root
- *   is an array (multi-region contextSelector support), not a single element.
- * - label[for] association uses a single precomputed Set of `for` values (document-wide, like original).
- * - getFocusableInfo is only invoked when role is presentational and thus relevant.
- * - aria-labelledby / aria-label helper resolution is only invoked when the attribute is present.
+ * Control discovery goes through helpers.queryAllSmart/queryAll (multi-root
+ * and shadow-DOM aware), not a direct root/safeRoot DOM query -- ctx.root
+ * is an array (multi-region contextSelector support), not a single element.
+ * Label-method resolution (label/aria-labelledby/aria-label/title/
+ * placeholder priority) delegates entirely to the shared dom-helpers
+ * bundle's getLabelMethod, which dom-runner.js always provides to every
+ * rule execution -- see getLabelMethodSafe below.
  *
  * WCAG mapping: matches technique H44 ("Using label elements to associate
  * text labels with form controls"), which WCAG's own Techniques document
@@ -40,9 +33,27 @@ const meta = {
   tags: ['wcag2a', 'wcag131', 'wcag332', 'wcag412', 'forms', 'labels', 'atomic', 'automatic'],
   wcagSc: ['1.3.1', '3.3.2', '4.1.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '1.3.1', title: 'Info and Relationships', conformanceLevel: 'A' },
-    { standard: 'WCAG', version: '2.2', requirement: '3.3.2', title: 'Labels or Instructions', conformanceLevel: 'A' },
-    { standard: 'WCAG', version: '2.2', requirement: '4.1.2', title: 'Name, Role, Value', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '1.3.1',
+      title: 'Info and Relationships',
+      conformanceLevel: 'A'
+    },
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '3.3.2',
+      title: 'Labels or Instructions',
+      conformanceLevel: 'A'
+    },
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '4.1.2',
+      title: 'Name, Role, Value',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'serious',
   category: 'robust',
@@ -58,48 +69,17 @@ const meta = {
 };
 
 function runInPage(ctx) {
-  const { document, helpers, rule } = ctx;
-
-  const isAccTreeEligible = helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
-  const getEligibilityInfo = helpers && typeof helpers.getEligibilityInfo === 'function' ? helpers.getEligibilityInfo : null;
+  const { helpers, rule } = ctx;
 
-  const getFocusableInfo = helpers && typeof helpers.getFocusableInfo === 'function' ? helpers.getFocusableInfo : null;
-  const getAriaLabelInfo = helpers && typeof helpers.getAriaLabelInfo === 'function' ? helpers.getAriaLabelInfo : null;
-  const getAriaLabelledByInfo =
-    helpers && typeof helpers.getAriaLabelledByInfo === 'function' ? helpers.getAriaLabelledByInfo : null;
+  const isAccTreeEligible =
+    helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
+  const getEligibilityInfo =
+    helpers && typeof helpers.getEligibilityInfo === 'function' ? helpers.getEligibilityInfo : null;
 
-  const getAttributeInfo = helpers && typeof helpers.getAttributeInfo === 'function' ? helpers.getAttributeInfo : null;
-  const getLabelMethod = helpers && typeof helpers.getLabelMethod === 'function' ? helpers.getLabelMethod : null;
-  const getContentNameInfo = helpers && typeof helpers.getContentNameInfo === 'function' ? helpers.getContentNameInfo : null;
-  const getAriaNameInfo = helpers && typeof helpers.getAriaNameInfo === 'function' ? helpers.getAriaNameInfo : null;
-
-  // A <label> counts as an association if it contributes a name either via
-  // its own aria-label/aria-labelledby (checked first, same ARIA-over-
-  // content precedence any element's accessible name gives — verified
-  // against a real page: <label aria-label="Toggle Navigation"><svg
-  // aria-hidden="true">...</svg></label> names its control "Toggle
-  // Navigation" even though the label's only child content is aria-hidden)
-  // or, failing that, non-empty ACCESSIBLE content (getContentNameInfo
-  // already excludes aria-hidden/display:none/inert descendants — a label
-  // whose only text is aria-hidden gives the control no real accessible
-  // name even though the structural association exists). See
-  // dom-helpers.js's labelContributesAccessibleName for the primary
-  // (shared-helper) path; this is the fallback-only mirror.
-  function labelHasAccessibleContent(labelEl) {
-    if (getAriaNameInfo) {
-      try {
-        const aria = getAriaNameInfo(labelEl, ctx);
-        if (aria && aria.present && trim(aria.value)) return true;
-      } catch {}
-    }
-    if (!getContentNameInfo) return true; // conservative: don't newly fail without the helper
-    try {
-      const info = getContentNameInfo(labelEl, ctx);
-      return !!(info && info.present && trim(info.value));
-    } catch {
-      return true;
-    }
-  }
+  const getFocusableInfo =
+    helpers && typeof helpers.getFocusableInfo === 'function' ? helpers.getFocusableInfo : null;
+  const getLabelMethod =
+    helpers && typeof helpers.getLabelMethod === 'function' ? helpers.getLabelMethod : null;
 
   const trim = (v) => (v == null ? '' : String(v)).trim();
 
@@ -107,7 +87,14 @@ function runInPage(ctx) {
     applicableCount: 0,
     passCount: 0,
     failCount: 0,
-    byMethod: { label: 0, 'aria-labelledby': 0, 'aria-label': 0, title: 0, placeholder: 0, none: 0 },
+    byMethod: {
+      label: 0,
+      'aria-labelledby': 0,
+      'aria-label': 0,
+      title: 0,
+      placeholder: 0,
+      none: 0
+    },
     weakPassCount: 0
   };
 
@@ -118,35 +105,6 @@ function runInPage(ctx) {
     return 'none';
   }
 
-  function getNonEmptyAttrViaHelper(el, name) {
-    if (!getAttributeInfo) return '';
-    try {
-      const info = getAttributeInfo(el, name);
-      return info && info.present ? trim(info.value) : '';
-    } catch {
-      return '';
-    }
-  }
-
-  // Build a document-wide Map of label[for] value -> label element once
-  // (O(#labels) instead of O(#controls) selector queries). Keeps the
-  // element reference (not just a presence flag) so hasLabelAssociation can
-  // check the label's actual accessible content.
-  const labelForMap = new Map();
-  try {
-    if (document && typeof document.getElementsByTagName === 'function') {
-      const labels = document.getElementsByTagName('label');
-      for (let i = 0; i < labels.length; i++) {
-        const lab = labels[i];
-        if (!lab || !lab.getAttribute) continue;
-        const f = trim(lab.getAttribute('for'));
-        if (f && !labelForMap.has(f)) labelForMap.set(f, lab);
-      }
-    }
-  } catch {
-    // no-throw
-  }
-
   function isEligibleAcc(el) {
     if (!isAccTreeEligible) return true;
     try {
@@ -158,110 +116,23 @@ function runInPage(ctx) {
     }
   }
 
-  function hasLabelAssociation(el) {
-    // 1) Native labels API — resolves both wrapping <label> and
-    // <label for="id"> as real elements in one call.
-    try {
-      if (el && 'labels' in el && el.labels && el.labels.length) {
-        return Array.prototype.some.call(el.labels, labelHasAccessibleContent);
-      }
-    } catch {}
-
-    // 2) Wrapped by <label>
-    try {
-      if (el && el.closest) {
-        const wrap = el.closest('label');
-        if (wrap) return labelHasAccessibleContent(wrap);
-      }
-    } catch {}
-
-    // 3) <label for="id">
-    try {
-      const idAttr = el && el.getAttribute ? trim(el.getAttribute('id')) : '';
-      if (!idAttr) return false;
-      const lab = labelForMap.get(idAttr);
-      return !!lab && labelHasAccessibleContent(lab);
-    } catch {
-      return false;
-    }
-  }
-
-  function computeLabelMethodFallback(el) {
-    // Deterministic priority order
-    if (hasLabelAssociation(el)) return { method: 'label', value: '' };
-
-    // Only resolve aria-labelledby if attribute exists
-    let raw = '';
-    try {
-      raw = el && el.getAttribute ? trim(el.getAttribute('aria-labelledby')) : '';
-    } catch {
-      raw = '';
-    }
-    if (raw && getAriaLabelledByInfo) {
-      try {
-        const info = getAriaLabelledByInfo(el, ctx, { maxRefs: 8 });
-        const v = info && info.present ? trim(info.value) : '';
-        if (v) return { method: 'aria-labelledby', value: v };
-      } catch {}
-    }
-
-    // Only resolve aria-label if attribute exists
-    raw = '';
+  // getLabelMethod is provided by the shared dom-helpers bundle that
+  // dom-runner.js always constructs for every rule execution (built-in or
+  // custom) — see createDomHelpers's own getLabelMethod, which implements
+  // this exact <label>/aria-labelledby/aria-label/title/placeholder
+  // priority order. No local reimplementation is needed as a fallback.
+  function getLabelMethodSafe(el) {
+    if (!getLabelMethod) return { method: 'none', value: '' };
     try {
-      raw = el && el.getAttribute ? trim(el.getAttribute('aria-label')) : '';
+      const r = getLabelMethod(el, ctx);
+      const m = r && typeof r.method === 'string' ? r.method : 'none';
+      const v = r && r.value != null ? trim(r.value) : '';
+      if (!Object.prototype.hasOwnProperty.call(metrics.byMethod, m))
+        return { method: 'none', value: '' };
+      return { method: m, value: v };
     } catch {
-      raw = '';
-    }
-    if (raw && getAriaLabelInfo) {
-      try {
-        const info = getAriaLabelInfo(el, ctx);
-        const v = info && info.present ? trim(info.value) : '';
-        if (v) return { method: 'aria-label', value: v };
-      } catch {}
+      return { method: 'none', value: '' };
     }
-
-    // title / placeholder
-    const titleV = getNonEmptyAttrViaHelper(el, 'title') || (() => {
-      try { return trim(el.getAttribute('title')); } catch { return ''; }
-    })();
-    if (titleV) return { method: 'title', value: titleV };
-
-    // `placeholder` is only a valid name/hint source for text-entry input
-    // types and <textarea> — not checkbox/radio/range/color/date/... or
-    // <select>, which browsers/AT never read a placeholder from.
-    const isPlaceholderCapable = (() => {
-      try {
-        const tag = (el.tagName || '').toLowerCase();
-        if (tag === 'textarea') return true;
-        if (tag !== 'input') return false;
-        const type = trim(el.getAttribute('type') || 'text').toLowerCase() || 'text';
-        return ['text', 'search', 'tel', 'url', 'email', 'password', 'number'].includes(type);
-      } catch {
-        return false;
-      }
-    })();
-
-    const phV = isPlaceholderCapable && (getNonEmptyAttrViaHelper(el, 'placeholder') || (() => {
-      try { return trim(el.getAttribute('placeholder')); } catch { return ''; }
-    })());
-    if (phV) return { method: 'placeholder', value: phV };
-
-    return { method: 'none', value: '' };
-  }
-
-  function getLabelMethodSafe(el) {
-    if (getLabelMethod) {
-      try {
-        const r = getLabelMethod(el, ctx);
-        const m = r && typeof r.method === 'string' ? r.method : 'none';
-        const v = r && r.value != null ? trim(r.value) : '';
-        if (!Object.prototype.hasOwnProperty.call(metrics.byMethod, m)) return { method: 'none', value: '' };
-        return { method: m, value: v };
-      } catch {
-        // fall through
-      }
-    }
-    return computeLabelMethodFallback(el);
   }
 
   // Collect candidate nodes via the shared queryAllSmart/queryAll helpers
@@ -272,17 +143,21 @@ function runInPage(ctx) {
   const nodes = [];
   try {
     const sel = 'input,select,textarea';
-    const candidates = (helpers && typeof helpers.queryAllSmart === 'function')
-      ? helpers.queryAllSmart(sel)
-      : (helpers && typeof helpers.queryAll === 'function' ? helpers.queryAll(sel) : []);
-
-    for (const el of (candidates || [])) {
+    const candidates =
+      helpers && typeof helpers.queryAllSmart === 'function'
+        ? helpers.queryAllSmart(sel)
+        : helpers && typeof helpers.queryAll === 'function'
+          ? helpers.queryAll(sel)
+          : [];
+
+    for (const el of candidates || []) {
       if (!el || !el.getAttribute) continue;
       const tag = (el.tagName || '').toLowerCase();
       if (tag === 'input') {
         const t = trim(el.getAttribute('type')).toLowerCase();
         // exclude hidden|submit|reset|button|image
-        if (t === 'hidden' || t === 'submit' || t === 'reset' || t === 'button' || t === 'image') continue;
+        if (t === 'hidden' || t === 'submit' || t === 'reset' || t === 'button' || t === 'image')
+          continue;
       }
       nodes.push(el);
     }
@@ -309,7 +184,7 @@ function runInPage(ctx) {
     if (!isEligibleAcc(el)) continue;
 
     // role="presentation"/"none" exclusion only when NOT focusable
-    let role = '';
+    let role;
     try {
       role = trim(el.getAttribute('role')).toLowerCase();
     } catch {
@@ -350,7 +225,15 @@ function runInPage(ctx) {
 
     metrics.failCount += 1;
 
-    const vf = getEligibilityInfo ? (() => { try { return getEligibilityInfo(el, ctx, { targetSet: 'acc' }); } catch { return null; } })() : null;
+    const vf = getEligibilityInfo
+      ? (() => {
+          try {
+            return getEligibilityInfo(el, ctx, { targetSet: 'acc' });
+          } catch {
+            return null;
+          }
+        })()
+      : null;
 
     const baseOccurrence = {
       summary: 'Form control is missing a programmatic label.',

package/src/checks/automatic/form-control-single-label.js

@@ -21,13 +21,20 @@
  *   decision from form-control-programmatic-label-present (that
  *   rule checks a label exists at all; this one checks there is at most
  *   one).
+ * - A label that isn't accessibility-tree-eligible (display:none,
+ *   aria-hidden, etc.) is excluded before counting: it can't contribute to
+ *   the control's accessible name, so it can't create the ambiguity this
+ *   rule exists to catch. Matches the reference engine's own
+ *   multiple-label check, which filters hidden labels the same way before
+ *   deciding whether there's more than one.
  */
 
 const id = 'form-control-single-label';
 
 const meta = {
   title: 'Form controls must not have multiple labels',
-  description: 'Checks that a form control is associated with at most one <label> (by wrapping or by label[for]).',
+  description:
+    'Checks that a form control is associated with at most one <label> (by wrapping or by label[for]).',
   i18n: {
     titleKey: 'formControlSingleLabel_title',
     descriptionKey: 'formControlSingleLabel_description'
@@ -36,7 +43,13 @@ const meta = {
   tags: ['wcag2a', 'wcag332', 'forms', 'atomic', 'automatic'],
   wcagSc: ['3.3.2'],
   normativeMappings: [
-    { standard: 'WCAG', version: '2.2', requirement: '3.3.2', title: 'Labels or Instructions', conformanceLevel: 'A' }
+    {
+      standard: 'WCAG',
+      version: '2.2',
+      requirement: '3.3.2',
+      title: 'Labels or Instructions',
+      conformanceLevel: 'A'
+    }
   ],
   defaultSeverity: 'moderate',
   category: 'understandable',
@@ -48,8 +61,14 @@ const meta = {
 function runInPage(ctx) {
   const { document, helpers, rule } = ctx;
 
-  const selector = 'input:not([type="hidden"]):not([type="submit"]):not([type="reset"]):not([type="button"]):not([type="image"]),select,textarea';
-  const nodes = helpers.queryAllSmart ? helpers.queryAllSmart(selector) : helpers.queryAll(selector);
+  const isAccTreeEligible =
+    helpers && typeof helpers.isAccTreeEligible === 'function' ? helpers.isAccTreeEligible : null;
+
+  const selector =
+    'input:not([type="hidden"]):not([type="submit"]):not([type="reset"]):not([type="button"]):not([type="image"]),select,textarea';
+  const nodes = helpers.queryAllSmart
+    ? helpers.queryAllSmart(selector)
+    : helpers.queryAll(selector);
 
   // Build a for-value -> label[] map once (avoids per-control dynamic
   // attribute-selector construction / CSS.escape, which is not guaranteed
@@ -82,11 +101,20 @@ function runInPage(ctx) {
       for (const lab of labelsByFor.get(controlId)) labels.add(lab);
     }
 
-    if (labels.size <= 1) continue;
+    const eligibleLabels = isAccTreeEligible
+      ? new Set(
+          [...labels].filter((lab) => {
+            const elig = isAccTreeEligible(lab);
+            return !elig || elig.eligible !== false;
+          })
+        )
+      : labels;
+
+    if (eligibleLabels.size <= 1) continue;
 
     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,
@@ -96,10 +124,14 @@ function runInPage(ctx) {
       i18n: {
         summaryKey: 'formControlSingleLabel_summary_fail',
         hintKey: 'formControlSingleLabel_hint_fail',
-        params: { element: tag, labelCount: String(labels.size) }
+        params: { element: tag, labelCount: String(eligibleLabels.size) }
       },
       data: {
-        details: { reasonCode: 'FORM_FIELD_MULTIPLE_LABELS', element: tag, labelCount: labels.size }
+        details: {
+          reasonCode: 'FORM_FIELD_MULTIPLE_LABELS',
+          element: tag,
+          labelCount: eligibleLabels.size
+        }
       }
     });
   }
@@ -108,9 +140,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 };