From 7b2a6b303b7e26b522314413bbea258ad8453dd4 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Fri, 26 Jun 2026 15:18:20 +0530
Subject: [PATCH 01/13] AXE-3625: Add defaultCategory to Phase 2 bulk-review
 rules

Wire seven axe-core rules to their Phase 2 bulk-review categories via
metadata.defaultCategory (reference data only; no rule logic change):

  aria-hidden-focus            -> aria-hidden
  aria-valid-attr-value        -> aria-attributes
  bypass                       -> bypass-blocks
  css-orientation-lock         -> orientation
  duplicate-id-aria            -> duplicate-ids
  identical-links-same-purpose -> link-consistency
  video-caption                -> media-captions

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 lib/rules/aria-hidden-focus.json            | 1 +
 lib/rules/aria-valid-attr-value.json        | 1 +
 lib/rules/bypass.json                       | 1 +
 lib/rules/css-orientation-lock.json         | 1 +
 lib/rules/duplicate-id-aria.json            | 1 +
 lib/rules/identical-links-same-purpose.json | 1 +
 lib/rules/video-caption.json                | 1 +
 7 files changed, 7 insertions(+)

diff --git a/lib/rules/aria-hidden-focus.json b/lib/rules/aria-hidden-focus.json
index afb1ad8fd..d149e7900 100755
--- a/lib/rules/aria-hidden-focus.json
+++ b/lib/rules/aria-hidden-focus.json
@@ -17,6 +17,7 @@
   ],
   "actIds": ["6cfa84"],
   "metadata": {
+    "defaultCategory": "aria-hidden",
     "description": "Ensure aria-hidden elements are not focusable nor contain focusable elements",
     "help": "ARIA hidden element must not be focusable or contain focusable elements",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/aria-valid-attr-value.json b/lib/rules/aria-valid-attr-value.json
index ac92598e2..dc39253c3 100644
--- a/lib/rules/aria-valid-attr-value.json
+++ b/lib/rules/aria-valid-attr-value.json
@@ -13,6 +13,7 @@
   ],
   "actIds": ["6a7281"],
   "metadata": {
+    "defaultCategory": "aria-attributes",
     "description": "Ensure all ARIA attributes have valid values",
     "help": "ARIA attributes must conform to valid values",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/bypass.json b/lib/rules/bypass.json
index 02666d9da..b05b74210 100644
--- a/lib/rules/bypass.json
+++ b/lib/rules/bypass.json
@@ -20,6 +20,7 @@
   ],
   "actIds": ["cf77f2", "047fe0", "b40fd1", "3e12e1", "ye5d6e"],
   "metadata": {
+    "defaultCategory": "bypass-blocks",
     "description": "Ensure each page has at least one mechanism for a user to bypass navigation and jump straight to the content",
     "help": "Page must have means to bypass repeated blocks",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/css-orientation-lock.json b/lib/rules/css-orientation-lock.json
index 06bc7d426..ed134b978 100644
--- a/lib/rules/css-orientation-lock.json
+++ b/lib/rules/css-orientation-lock.json
@@ -16,6 +16,7 @@
   ],
   "actIds": ["b33eff"],
   "metadata": {
+    "defaultCategory": "orientation",
     "description": "Ensure content is not locked to any specific display orientation, and the content is operable in all display orientations",
     "help": "CSS Media queries must not lock display orientation",
     "violationConfidence": 90,
diff --git a/lib/rules/duplicate-id-aria.json b/lib/rules/duplicate-id-aria.json
index 1c900c3dd..d3921ff55 100644
--- a/lib/rules/duplicate-id-aria.json
+++ b/lib/rules/duplicate-id-aria.json
@@ -16,6 +16,7 @@
   "reviewOnFail": true,
   "actIds": ["3ea0c8"],
   "metadata": {
+    "defaultCategory": "duplicate-ids",
     "description": "Ensure every id attribute value used in ARIA and in labels is unique",
     "help": "IDs used in ARIA and labels must be unique",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/identical-links-same-purpose.json b/lib/rules/identical-links-same-purpose.json
index afde13c5e..cba1facec 100644
--- a/lib/rules/identical-links-same-purpose.json
+++ b/lib/rules/identical-links-same-purpose.json
@@ -8,6 +8,7 @@
   "tags": ["cat.semantics", "wcag2aaa", "wcag249"],
   "actIds": ["b20e66"],
   "metadata": {
+    "defaultCategory": "link-consistency",
     "description": "Ensure that links with the same accessible name serve a similar purpose",
     "help": "Links with the same name must have a similar purpose",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/video-caption.json b/lib/rules/video-caption.json
index b0297ddb0..98f23d092 100644
--- a/lib/rules/video-caption.json
+++ b/lib/rules/video-caption.json
@@ -17,6 +17,7 @@
   ],
   "actIds": ["eac66b"],
   "metadata": {
+    "defaultCategory": "media-captions",
     "description": "Ensure <video> elements have captions",
     "help": "<video> elements must have captions",
     "needsReviewConfidence": 49.99

From 9db3fa6cabd4d75ce188693e225dc006004de245 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Fri, 26 Jun 2026 19:38:23 +0530
Subject: [PATCH 02/13] AXE-3625: Emit bulk-review visualHelperData for
 axe-core rules

Add reviewPayload.visualHelperData emits for aria-valid-attr-value,
aria-hidden-focus, identical-links-same-purpose and duplicate-id-aria
(dedicated evaluate/after), plus their unit specs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../aria/aria-valid-attr-value-evaluate.js    |  6 +-
 .../keyboard/focusable-disabled-evaluate.js   | 24 +++++++-
 .../focusable-not-tabbable-evaluate.js        | 24 +++++++-
 .../identical-links-same-purpose-after.js     | 25 +++++++++
 .../identical-links-same-purpose-evaluate.js  |  1 +
 lib/checks/parsing/duplicate-id-aria-after.js | 18 ++++++
 .../parsing/duplicate-id-aria-evaluate.js     | 55 +++++++++++++++++++
 lib/checks/parsing/duplicate-id-aria.json     |  6 +-
 test/checks/aria/aria-valid-attr-value.js     | 46 ++++++++++++++++
 test/checks/keyboard/focusable-disabled.js    | 15 +++++
 .../checks/keyboard/focusable-not-tabbable.js | 14 +++++
 .../identical-links-same-purpose-after.js     | 52 ++++++++++++++++++
 .../identical-links-same-purpose.js           | 24 +++++++-
 test/checks/parser/duplicate-id-aria.js       | 55 +++++++++++++++++++
 14 files changed, 356 insertions(+), 9 deletions(-)
 create mode 100644 lib/checks/parsing/duplicate-id-aria-after.js
 create mode 100644 lib/checks/parsing/duplicate-id-aria-evaluate.js
 create mode 100644 test/checks/aria/aria-valid-attr-value.js
 create mode 100644 test/checks/parser/duplicate-id-aria.js

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index ad43d1393..b88b8627a 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -140,7 +140,11 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
   }
 
   if (needsReview) {
-    this.data({ messageKey, needsReview });
+    this.data({
+      messageKey,
+      needsReview,
+      reviewPayload: { visualHelperData: { ariaAttribute: needsReview } }
+    });
     return undefined;
   }
 
diff --git a/lib/checks/keyboard/focusable-disabled-evaluate.js b/lib/checks/keyboard/focusable-disabled-evaluate.js
index d58807158..b5a77c1a4 100644
--- a/lib/checks/keyboard/focusable-disabled-evaluate.js
+++ b/lib/checks/keyboard/focusable-disabled-evaluate.js
@@ -25,7 +25,7 @@ function focusableDisabledEvaluate(node, options, virtualNode) {
     return true;
   }
 
-  return relatedNodes.every(vNode => {
+  const verdict = relatedNodes.every(vNode => {
     const pointerEvents = vNode.getComputedStylePropertyValue('pointer-events');
     const width = parseInt(vNode.getComputedStylePropertyValue('width'));
     const height = parseInt(vNode.getComputedStylePropertyValue('height'));
@@ -37,6 +37,28 @@ function focusableDisabledEvaluate(node, options, virtualNode) {
   })
     ? undefined
     : false;
+
+  // a11y-rule-aria-hidden-focus: surface the focusable descendants for the
+  // bulk-review "Review Hidden Elements" visual helper. Reuse DqElement.selector
+  // (the same format-aware path relatedNodes uses) so the list and the overlay
+  // share identical selectors; no new computation. Wrapped so any failure to
+  // build the payload (e.g. an unexpected node) can never fail the check/scan.
+  try {
+    this.data({
+      reviewPayload: {
+        visualHelperData: {
+          focusableChildren: relatedNodes
+            .map(vNode => vNode.actualNode)
+            .filter(Boolean)
+            .map(actualNode => new axe.utils.DqElement(actualNode).selector)
+        }
+      }
+    });
+  } catch {
+    // best-effort emit — preserve the scan result
+  }
+
+  return verdict;
 }
 
 export default focusableDisabledEvaluate;
diff --git a/lib/checks/keyboard/focusable-not-tabbable-evaluate.js b/lib/checks/keyboard/focusable-not-tabbable-evaluate.js
index b5a5d2965..e0e2dfe79 100644
--- a/lib/checks/keyboard/focusable-not-tabbable-evaluate.js
+++ b/lib/checks/keyboard/focusable-not-tabbable-evaluate.js
@@ -25,7 +25,7 @@ function focusableNotTabbableEvaluate(node, options, virtualNode) {
     return true;
   }
 
-  return relatedNodes.every(vNode => {
+  const verdict = relatedNodes.every(vNode => {
     const pointerEvents = vNode.getComputedStylePropertyValue('pointer-events');
     const width = parseInt(vNode.getComputedStylePropertyValue('width'));
     const height = parseInt(vNode.getComputedStylePropertyValue('height'));
@@ -37,6 +37,28 @@ function focusableNotTabbableEvaluate(node, options, virtualNode) {
   })
     ? undefined
     : false;
+
+  // a11y-rule-aria-hidden-focus: surface the focusable descendants for the
+  // bulk-review "Review Hidden Elements" visual helper. Reuse DqElement.selector
+  // (the same format-aware path relatedNodes uses) so the list and the overlay
+  // share identical selectors; no new computation. Wrapped so any failure to
+  // build the payload (e.g. an unexpected node) can never fail the check/scan.
+  try {
+    this.data({
+      reviewPayload: {
+        visualHelperData: {
+          focusableChildren: relatedNodes
+            .map(vNode => vNode.actualNode)
+            .filter(Boolean)
+            .map(actualNode => new axe.utils.DqElement(actualNode).selector)
+        }
+      }
+    });
+  } catch {
+    // best-effort emit — preserve the scan result
+  }
+
+  return verdict;
 }
 
 export default focusableNotTabbableEvaluate;
diff --git a/lib/checks/navigation/identical-links-same-purpose-after.js b/lib/checks/navigation/identical-links-same-purpose-after.js
index 2459be117..702b43d7f 100644
--- a/lib/checks/navigation/identical-links-same-purpose-after.js
+++ b/lib/checks/navigation/identical-links-same-purpose-after.js
@@ -84,6 +84,13 @@ function identicalLinksSamePurposeAfter(results) {
       currentResult.result = undefined;
     }
 
+    /**
+     * The node's own DqElement is stored at `relatedNodes[0]` by the evaluate
+     * fn; capture it before `relatedNodes` is rebuilt so the link group can
+     * include the primary link, not just its same-name siblings.
+     */
+    const currentNode = currentResult.relatedNodes[0];
+
     /**
      *  -> deduplicate results (for both `pass` or `incomplete`) and add `relatedNodes` if any
      */
@@ -92,6 +99,24 @@ function identicalLinksSamePurposeAfter(results) {
       ...sameNameResults.map(node => node.relatedNodes[0])
     );
 
+    if (sameNameResults.length) {
+      try {
+        const identicalLinks = [currentNode, ...currentResult.relatedNodes]
+          .filter(Boolean)
+          .map(dqElm => (dqElm ? dqElm.selector : undefined))
+          .filter(Boolean);
+        const linkText =
+          (currentResult.data && currentResult.data.accessibleText) || name;
+        if (currentResult.data) {
+          currentResult.data.reviewPayload = {
+            visualHelperData: { linkText, identicalLinks }
+          };
+        }
+      } catch {
+        // best-effort emit — preserve the reconciled results
+      }
+    }
+
     /**
      * Update `nodeMap` with `sameNameResults`
      */
diff --git a/lib/checks/navigation/identical-links-same-purpose-evaluate.js b/lib/checks/navigation/identical-links-same-purpose-evaluate.js
index b47400c91..6d5b4c23d 100644
--- a/lib/checks/navigation/identical-links-same-purpose-evaluate.js
+++ b/lib/checks/navigation/identical-links-same-purpose-evaluate.js
@@ -24,6 +24,7 @@ function identicalLinksSamePurposeEvaluate(node, options, virtualNode) {
    */
   const afterData = {
     name,
+    accessibleText: text.sanitize(accText),
     urlProps: dom.urlPropsFromAttribute(node, 'href')
   };
   this.data(afterData);
diff --git a/lib/checks/parsing/duplicate-id-aria-after.js b/lib/checks/parsing/duplicate-id-aria-after.js
new file mode 100644
index 000000000..95034ed88
--- /dev/null
+++ b/lib/checks/parsing/duplicate-id-aria-after.js
@@ -0,0 +1,18 @@
+/**
+ * Mirrors `duplicate-id-after`, but dedupes by `r.data.id` since the aria
+ * evaluate stores its data as an object (`{ id, reviewPayload }`) rather than
+ * a bare id string.
+ */
+function duplicateIdAriaAfter(results) {
+  const uniqueIds = [];
+  return results.filter(r => {
+    const id = r.data && r.data.id;
+    if (uniqueIds.indexOf(id) === -1) {
+      uniqueIds.push(id);
+      return true;
+    }
+    return false;
+  });
+}
+
+export default duplicateIdAriaAfter;
diff --git a/lib/checks/parsing/duplicate-id-aria-evaluate.js b/lib/checks/parsing/duplicate-id-aria-evaluate.js
new file mode 100644
index 000000000..30cb4dd7a
--- /dev/null
+++ b/lib/checks/parsing/duplicate-id-aria-evaluate.js
@@ -0,0 +1,55 @@
+import { getRootNode } from '../../commons/dom';
+import { escapeSelector } from '../../core/utils';
+
+/**
+ * Mirrors `duplicate-id-evaluate`, but additionally emits a `reviewPayload`
+ * for the bulk-review "Review Duplicate IDs" visual helper. Kept separate so
+ * the non-bulk-review `duplicate-id` / `duplicate-id-active` checks (which
+ * share `duplicate-id-evaluate`) are untouched.
+ */
+function duplicateIdAriaEvaluate(node) {
+  const id = node.getAttribute('id').trim();
+  // Since empty ID's are not meaningful and are ignored by Edge, we'll
+  // let those pass.
+  if (!id) {
+    return true;
+  }
+  const root = getRootNode(node);
+  const matchingNodes = Array.from(
+    root.querySelectorAll(`[id="${escapeSelector(id)}"]`)
+  ).filter(foundNode => foundNode !== node);
+
+  if (matchingNodes.length) {
+    this.relatedNodes(matchingNodes);
+  }
+
+  // Verdict is computed independently of the emit below.
+  const verdict = matchingNodes.length === 0;
+
+  // a11y-rule-duplicate-id-aria: surface the duplicate id and the other
+  // elements sharing it for the visual helper. Reuse `DqElement.selector`
+  // (the same path `relatedNodes` serializes with) so the list matches the
+  // reported nodes; no new lookup. Emit only when a duplicate exists, and
+  // wrapped so any failure to build the payload can never fail the check/scan
+  // (the `id` is always carried for the fail message).
+  const data = { id };
+  try {
+    if (matchingNodes.length) {
+      data.reviewPayload = {
+        visualHelperData: {
+          duplicateId: id,
+          elements: matchingNodes
+            .filter(Boolean)
+            .map(foundNode => new axe.utils.DqElement(foundNode).selector)
+        }
+      };
+    }
+  } catch {
+    // best-effort emit — preserve the scan result
+  }
+  this.data(data);
+
+  return verdict;
+}
+
+export default duplicateIdAriaEvaluate;
diff --git a/lib/checks/parsing/duplicate-id-aria.json b/lib/checks/parsing/duplicate-id-aria.json
index 67cfa5cd9..105f4c091 100644
--- a/lib/checks/parsing/duplicate-id-aria.json
+++ b/lib/checks/parsing/duplicate-id-aria.json
@@ -1,12 +1,12 @@
 {
   "id": "duplicate-id-aria",
-  "evaluate": "duplicate-id-evaluate",
-  "after": "duplicate-id-after",
+  "evaluate": "duplicate-id-aria-evaluate",
+  "after": "duplicate-id-aria-after",
   "metadata": {
     "impact": "critical",
     "messages": {
       "pass": "Document has no elements referenced with ARIA or labels that share the same id attribute",
-      "fail": "Document has multiple elements referenced with ARIA with the same id attribute: ${data}"
+      "fail": "Document has multiple elements referenced with ARIA with the same id attribute: ${data.id}"
     }
   }
 }
diff --git a/test/checks/aria/aria-valid-attr-value.js b/test/checks/aria/aria-valid-attr-value.js
new file mode 100644
index 000000000..c0bd0d371
--- /dev/null
+++ b/test/checks/aria/aria-valid-attr-value.js
@@ -0,0 +1,46 @@
+describe('aria-valid-attr-value', () => {
+  'use strict';
+
+  const checkContext = axe.testUtils.MockCheckContext();
+  const checkSetup = axe.testUtils.checkSetup;
+  const checkEvaluate = axe.testUtils.getCheckEvaluate('aria-valid-attr-value');
+
+  afterEach(() => {
+    checkContext.reset();
+  });
+
+  it('emits the flagged aria-current attribute in reviewPayload.visualHelperData', () => {
+    const params = checkSetup(
+      '<div id="target" aria-current="active">Contents</div>'
+    );
+    assert.isUndefined(checkEvaluate.apply(checkContext, params));
+    assert.equal(checkContext._data.messageKey, 'ariaCurrent');
+    assert.equal(checkContext._data.needsReview, 'aria-current="active"');
+    assert.deepEqual(checkContext._data.reviewPayload, {
+      visualHelperData: { ariaAttribute: 'aria-current="active"' }
+    });
+  });
+
+  it('emits the flagged aria-labelledby attribute in reviewPayload.visualHelperData', () => {
+    const params = checkSetup(
+      '<div id="target" aria-labelledby="missing-id">Contents</div>'
+    );
+    assert.isUndefined(checkEvaluate.apply(checkContext, params));
+    assert.equal(
+      checkContext._data.needsReview,
+      'aria-labelledby="missing-id"'
+    );
+    assert.deepEqual(checkContext._data.reviewPayload, {
+      visualHelperData: { ariaAttribute: 'aria-labelledby="missing-id"' }
+    });
+  });
+
+  it('does not add a reviewPayload on the invalid-value (violation) path', () => {
+    const params = checkSetup(
+      '<div id="target" role="checkbox" aria-checked="foo">Contents</div>'
+    );
+    assert.isFalse(checkEvaluate.apply(checkContext, params));
+    assert.isArray(checkContext._data);
+    assert.notProperty(checkContext._data, 'reviewPayload');
+  });
+});
diff --git a/test/checks/keyboard/focusable-disabled.js b/test/checks/keyboard/focusable-disabled.js
index ffe1536f9..ba6dd0512 100644
--- a/test/checks/keyboard/focusable-disabled.js
+++ b/test/checks/keyboard/focusable-disabled.js
@@ -218,4 +218,19 @@ describe('focusable-disabled', function () {
     );
     assert.isUndefined(check.evaluate.apply(checkContext, params));
   });
+
+  it('emits the focusable children selectors in reviewPayload.visualHelperData', () => {
+    var params = checkSetup(
+      '<fieldset id="target" aria-hidden="true"><input /></fieldset>'
+    );
+    assert.isFalse(check.evaluate.apply(checkContext, params));
+    var focusableChildren =
+      checkContext._data.reviewPayload.visualHelperData.focusableChildren;
+    assert.isArray(focusableChildren);
+    assert.lengthOf(focusableChildren, 1);
+    // each entry is a DqElement.selector array, identical to relatedNodes[].selector
+    assert.isArray(focusableChildren[0]);
+    assert.isString(focusableChildren[0][0]);
+    assert.match(focusableChildren[0][0], /input/);
+  });
 });
diff --git a/test/checks/keyboard/focusable-not-tabbable.js b/test/checks/keyboard/focusable-not-tabbable.js
index 60bd891d1..81854cbf1 100644
--- a/test/checks/keyboard/focusable-not-tabbable.js
+++ b/test/checks/keyboard/focusable-not-tabbable.js
@@ -178,4 +178,18 @@ describe('focusable-not-tabbable', function () {
     );
     assert.isUndefined(check.evaluate.apply(checkContext, params));
   });
+
+  it('emits the focusable children selectors in reviewPayload.visualHelperData', () => {
+    var params = checkSetup(
+      '<div id="target" aria-hidden="true"><a href="/">Link</a></div>'
+    );
+    assert.isFalse(check.evaluate.apply(checkContext, params));
+    var focusableChildren =
+      checkContext._data.reviewPayload.visualHelperData.focusableChildren;
+    assert.isArray(focusableChildren);
+    assert.lengthOf(focusableChildren, 1);
+    // each entry is a DqElement.selector array, identical to relatedNodes[].selector
+    assert.isArray(focusableChildren[0]);
+    assert.isString(focusableChildren[0][0]);
+  });
 });
diff --git a/test/checks/navigation/identical-links-same-purpose-after.js b/test/checks/navigation/identical-links-same-purpose-after.js
index cd8ff1e81..21eb030e0 100644
--- a/test/checks/navigation/identical-links-same-purpose-after.js
+++ b/test/checks/navigation/identical-links-same-purpose-after.js
@@ -176,4 +176,56 @@ describe('identical-links-same-purpose-after tests', function () {
     assert.deepEqual(results[1].relatedNodes, []);
     assert.equal(results[1].result, true);
   });
+
+  it('emits the link group selectors in reviewPayload.visualHelperData for identical links', function () {
+    var nodeOneData = {
+      data: {
+        name: 'read more',
+        accessibleText: 'Read More',
+        urlProps: { pathname: '/blog/ai-health' }
+      },
+      relatedNodes: [{ selector: ['#card-1 > a.read-more'] }],
+      result: true
+    };
+    var nodeTwoData = {
+      data: {
+        name: 'read more',
+        accessibleText: 'Read More',
+        urlProps: { pathname: '/blog/ai-finance' }
+      },
+      relatedNodes: [{ selector: ['#card-2 > a.read-more'] }],
+      result: true
+    };
+
+    var results = check.after([nodeOneData, nodeTwoData]);
+    assert.lengthOf(results, 1);
+
+    // full group: the primary link plus every same-name sibling, each entry a
+    // DqElement.selector array identical to relatedNodes[].selector
+    var visualHelperData = results[0].data.reviewPayload.visualHelperData;
+    assert.deepEqual(visualHelperData.identicalLinks, [
+      ['#card-1 > a.read-more'],
+      ['#card-2 > a.read-more']
+    ]);
+    // the actual rendered link text shared by the group (original case)
+    assert.equal(visualHelperData.linkText, 'Read More');
+  });
+
+  it('does not emit reviewPayload when a link has no same-name group', function () {
+    var nodeOneData = {
+      data: { name: 'earth', urlProps: {} },
+      relatedNodes: [{ selector: ['#a'] }],
+      result: true
+    };
+    var nodeTwoData = {
+      data: { name: 'venus', urlProps: {} },
+      relatedNodes: [{ selector: ['#b'] }],
+      result: true
+    };
+
+    var results = check.after([nodeOneData, nodeTwoData]);
+    assert.lengthOf(results, 2);
+    assert.notProperty(results[0].data, 'reviewPayload');
+    assert.notProperty(results[1].data, 'reviewPayload');
+  });
 });
diff --git a/test/checks/navigation/identical-links-same-purpose.js b/test/checks/navigation/identical-links-same-purpose.js
index 8e6a9d20a..f5b1c7c87 100644
--- a/test/checks/navigation/identical-links-same-purpose.js
+++ b/test/checks/navigation/identical-links-same-purpose.js
@@ -60,8 +60,13 @@ describe('identical-links-same-purpose tests', function () {
       vNode
     );
     assert.isTrue(actual);
-    assert.hasAllKeys(checkContext._data, ['name', 'urlProps']);
+    assert.hasAllKeys(checkContext._data, [
+      'name',
+      'accessibleText',
+      'urlProps'
+    ]);
     assert.equal(checkContext._data.name, 'Pass 1'.toLowerCase());
+    assert.equal(checkContext._data.accessibleText, 'Pass 1');
     assert.equal(checkContext._data.urlProps.hash, '#/foo');
     assert.equal(checkContext._data.urlProps.pathname, '/home/');
   });
@@ -80,8 +85,13 @@ describe('identical-links-same-purpose tests', function () {
       vNode
     );
     assert.isTrue(actual);
-    assert.hasAllKeys(checkContext._data, ['name', 'urlProps']);
+    assert.hasAllKeys(checkContext._data, [
+      'name',
+      'accessibleText',
+      'urlProps'
+    ]);
     assert.equal(checkContext._data.name, 'MDN'.toLowerCase());
+    assert.equal(checkContext._data.accessibleText, 'MDN');
     assert.isFalse(!!checkContext._data.resource);
   });
 
@@ -96,11 +106,19 @@ describe('identical-links-same-purpose tests', function () {
       vNode
     );
     assert.isTrue(actual);
-    assert.hasAllKeys(checkContext._data, ['name', 'urlProps']);
+    assert.hasAllKeys(checkContext._data, [
+      'name',
+      'accessibleText',
+      'urlProps'
+    ]);
     assert.equal(
       checkContext._data.name,
       'The is orange the is white'.toLowerCase()
     );
+    assert.equal(
+      checkContext._data.accessibleText,
+      'The ☀️ is orange, the ◓ is white.'
+    );
     assert.equal(checkContext._data.urlProps.filename, 'foo.html');
   });
 });
diff --git a/test/checks/parser/duplicate-id-aria.js b/test/checks/parser/duplicate-id-aria.js
new file mode 100644
index 000000000..9c4a8ca7b
--- /dev/null
+++ b/test/checks/parser/duplicate-id-aria.js
@@ -0,0 +1,55 @@
+describe('duplicate-id-aria', function () {
+  'use strict';
+
+  var fixture = document.getElementById('fixture');
+  var fixtureSetup = axe.testUtils.fixtureSetup;
+  var checkContext = axe.testUtils.MockCheckContext();
+  var checkEvaluate = axe.testUtils.getCheckEvaluate('duplicate-id-aria');
+
+  afterEach(function () {
+    fixture.innerHTML = '';
+    axe._tree = undefined;
+    checkContext.reset();
+  });
+
+  it('returns true and emits no reviewPayload when the id is unique', function () {
+    fixtureSetup('<div id="solo"></div>');
+    var node = fixture.querySelector('#solo');
+    assert.isTrue(checkEvaluate.call(checkContext, node));
+    assert.equal(checkContext._data.id, 'solo');
+    assert.notProperty(checkContext._data, 'reviewPayload');
+    assert.deepEqual(checkContext._relatedNodes, []);
+  });
+
+  it('emits the duplicate id and element selectors in reviewPayload.visualHelperData', function () {
+    fixtureSetup(
+      '<div id="dup"></div><div id="dup"></div><div id="dup"></div>'
+    );
+    var node = fixture.querySelector('div');
+    assert.isFalse(checkEvaluate.call(checkContext, node));
+
+    assert.equal(checkContext._data.id, 'dup');
+    var visualHelperData = checkContext._data.reviewPayload.visualHelperData;
+    assert.equal(visualHelperData.duplicateId, 'dup');
+    // the OTHER elements sharing the id (relatedNodes), each a
+    // DqElement.selector array identical to relatedNodes[].selector
+    assert.lengthOf(visualHelperData.elements, 2);
+    assert.isArray(visualHelperData.elements[0]);
+    assert.isString(visualHelperData.elements[0][0]);
+  });
+
+  it('after dedupes results by data.id', function () {
+    var results = checks['duplicate-id-aria'].after([
+      { data: { id: 'a' } },
+      { data: { id: 'b' } },
+      { data: { id: 'b' } }
+    ]);
+    assert.lengthOf(results, 2);
+    assert.deepEqual(
+      results.map(function (r) {
+        return r.data.id;
+      }),
+      ['a', 'b']
+    );
+  });
+});

From d183a3480684c939bf1c5b1d320d4b95fce9e299 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 10:20:05 +0530
Subject: [PATCH 03/13] AXE-3625: Reuse shared duplicate-id check for the aria
 variant (DRY)

Collapse the dedicated duplicate-id-aria evaluate/after back into the
shared duplicate-id-evaluate/duplicate-id-after, gated by a `reviewPayload`
check option. The aria check enables the bulk-review payload via that
option; the static/active checks keep their original string `data` and are
behaviourally unchanged. Adds tests for both data shapes and verdict parity.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 lib/checks/parsing/duplicate-id-after.js      |  5 +-
 lib/checks/parsing/duplicate-id-aria-after.js | 18 ------
 .../parsing/duplicate-id-aria-evaluate.js     | 55 -------------------
 lib/checks/parsing/duplicate-id-aria.json     |  5 +-
 lib/checks/parsing/duplicate-id-evaluate.js   | 24 +++++++-
 test/checks/parser/duplicate-id-aria.js       | 16 +++++-
 test/checks/parser/duplicate-id.js            | 26 +++++++++
 7 files changed, 69 insertions(+), 80 deletions(-)
 delete mode 100644 lib/checks/parsing/duplicate-id-aria-after.js
 delete mode 100644 lib/checks/parsing/duplicate-id-aria-evaluate.js

diff --git a/lib/checks/parsing/duplicate-id-after.js b/lib/checks/parsing/duplicate-id-after.js
index e6a587185..021dfffdd 100644
--- a/lib/checks/parsing/duplicate-id-after.js
+++ b/lib/checks/parsing/duplicate-id-after.js
@@ -1,8 +1,9 @@
 function duplicateIdAfter(results) {
   const uniqueIds = [];
   return results.filter(r => {
-    if (uniqueIds.indexOf(r.data) === -1) {
-      uniqueIds.push(r.data);
+    const id = r.data && typeof r.data === 'object' ? r.data.id : r.data;
+    if (uniqueIds.indexOf(id) === -1) {
+      uniqueIds.push(id);
       return true;
     }
     return false;
diff --git a/lib/checks/parsing/duplicate-id-aria-after.js b/lib/checks/parsing/duplicate-id-aria-after.js
deleted file mode 100644
index 95034ed88..000000000
--- a/lib/checks/parsing/duplicate-id-aria-after.js
+++ /dev/null
@@ -1,18 +0,0 @@
-/**
- * Mirrors `duplicate-id-after`, but dedupes by `r.data.id` since the aria
- * evaluate stores its data as an object (`{ id, reviewPayload }`) rather than
- * a bare id string.
- */
-function duplicateIdAriaAfter(results) {
-  const uniqueIds = [];
-  return results.filter(r => {
-    const id = r.data && r.data.id;
-    if (uniqueIds.indexOf(id) === -1) {
-      uniqueIds.push(id);
-      return true;
-    }
-    return false;
-  });
-}
-
-export default duplicateIdAriaAfter;
diff --git a/lib/checks/parsing/duplicate-id-aria-evaluate.js b/lib/checks/parsing/duplicate-id-aria-evaluate.js
deleted file mode 100644
index 30cb4dd7a..000000000
--- a/lib/checks/parsing/duplicate-id-aria-evaluate.js
+++ /dev/null
@@ -1,55 +0,0 @@
-import { getRootNode } from '../../commons/dom';
-import { escapeSelector } from '../../core/utils';
-
-/**
- * Mirrors `duplicate-id-evaluate`, but additionally emits a `reviewPayload`
- * for the bulk-review "Review Duplicate IDs" visual helper. Kept separate so
- * the non-bulk-review `duplicate-id` / `duplicate-id-active` checks (which
- * share `duplicate-id-evaluate`) are untouched.
- */
-function duplicateIdAriaEvaluate(node) {
-  const id = node.getAttribute('id').trim();
-  // Since empty ID's are not meaningful and are ignored by Edge, we'll
-  // let those pass.
-  if (!id) {
-    return true;
-  }
-  const root = getRootNode(node);
-  const matchingNodes = Array.from(
-    root.querySelectorAll(`[id="${escapeSelector(id)}"]`)
-  ).filter(foundNode => foundNode !== node);
-
-  if (matchingNodes.length) {
-    this.relatedNodes(matchingNodes);
-  }
-
-  // Verdict is computed independently of the emit below.
-  const verdict = matchingNodes.length === 0;
-
-  // a11y-rule-duplicate-id-aria: surface the duplicate id and the other
-  // elements sharing it for the visual helper. Reuse `DqElement.selector`
-  // (the same path `relatedNodes` serializes with) so the list matches the
-  // reported nodes; no new lookup. Emit only when a duplicate exists, and
-  // wrapped so any failure to build the payload can never fail the check/scan
-  // (the `id` is always carried for the fail message).
-  const data = { id };
-  try {
-    if (matchingNodes.length) {
-      data.reviewPayload = {
-        visualHelperData: {
-          duplicateId: id,
-          elements: matchingNodes
-            .filter(Boolean)
-            .map(foundNode => new axe.utils.DqElement(foundNode).selector)
-        }
-      };
-    }
-  } catch {
-    // best-effort emit — preserve the scan result
-  }
-  this.data(data);
-
-  return verdict;
-}
-
-export default duplicateIdAriaEvaluate;
diff --git a/lib/checks/parsing/duplicate-id-aria.json b/lib/checks/parsing/duplicate-id-aria.json
index 105f4c091..56287276a 100644
--- a/lib/checks/parsing/duplicate-id-aria.json
+++ b/lib/checks/parsing/duplicate-id-aria.json
@@ -1,7 +1,8 @@
 {
   "id": "duplicate-id-aria",
-  "evaluate": "duplicate-id-aria-evaluate",
-  "after": "duplicate-id-aria-after",
+  "evaluate": "duplicate-id-evaluate",
+  "after": "duplicate-id-after",
+  "options": { "reviewPayload": true },
   "metadata": {
     "impact": "critical",
     "messages": {
diff --git a/lib/checks/parsing/duplicate-id-evaluate.js b/lib/checks/parsing/duplicate-id-evaluate.js
index 55b520c09..f4687aee9 100644
--- a/lib/checks/parsing/duplicate-id-evaluate.js
+++ b/lib/checks/parsing/duplicate-id-evaluate.js
@@ -1,7 +1,7 @@
 import { getRootNode } from '../../commons/dom';
 import { escapeSelector } from '../../core/utils';
 
-function duplicateIdEvaluate(node) {
+function duplicateIdEvaluate(node, options = {}) {
   const id = node.getAttribute('id').trim();
   // Since empty ID's are not meaningful and are ignored by Edge, we'll
   // let those pass.
@@ -16,6 +16,28 @@ function duplicateIdEvaluate(node) {
   if (matchingNodes.length) {
     this.relatedNodes(matchingNodes);
   }
+
+  if (options.reviewPayload) {
+    const verdict = matchingNodes.length === 0;
+    const data = { id };
+    try {
+      if (matchingNodes.length) {
+        data.reviewPayload = {
+          visualHelperData: {
+            duplicateId: id,
+            elements: matchingNodes
+              .filter(Boolean)
+              .map(foundNode => new axe.utils.DqElement(foundNode).selector)
+          }
+        };
+      }
+    } catch {
+      // best-effort emit — preserve the scan result
+    }
+    this.data(data);
+    return verdict;
+  }
+
   this.data(id);
 
   return matchingNodes.length === 0;
diff --git a/test/checks/parser/duplicate-id-aria.js b/test/checks/parser/duplicate-id-aria.js
index 9c4a8ca7b..c23ad78e2 100644
--- a/test/checks/parser/duplicate-id-aria.js
+++ b/test/checks/parser/duplicate-id-aria.js
@@ -5,6 +5,8 @@ describe('duplicate-id-aria', function () {
   var fixtureSetup = axe.testUtils.fixtureSetup;
   var checkContext = axe.testUtils.MockCheckContext();
   var checkEvaluate = axe.testUtils.getCheckEvaluate('duplicate-id-aria');
+  // duplicate-id-aria enables the bulk-review payload via this check option
+  var options = { reviewPayload: true };
 
   afterEach(function () {
     fixture.innerHTML = '';
@@ -15,7 +17,7 @@ describe('duplicate-id-aria', function () {
   it('returns true and emits no reviewPayload when the id is unique', function () {
     fixtureSetup('<div id="solo"></div>');
     var node = fixture.querySelector('#solo');
-    assert.isTrue(checkEvaluate.call(checkContext, node));
+    assert.isTrue(checkEvaluate.call(checkContext, node, options));
     assert.equal(checkContext._data.id, 'solo');
     assert.notProperty(checkContext._data, 'reviewPayload');
     assert.deepEqual(checkContext._relatedNodes, []);
@@ -26,7 +28,7 @@ describe('duplicate-id-aria', function () {
       '<div id="dup"></div><div id="dup"></div><div id="dup"></div>'
     );
     var node = fixture.querySelector('div');
-    assert.isFalse(checkEvaluate.call(checkContext, node));
+    assert.isFalse(checkEvaluate.call(checkContext, node, options));
 
     assert.equal(checkContext._data.id, 'dup');
     var visualHelperData = checkContext._data.reviewPayload.visualHelperData;
@@ -38,6 +40,16 @@ describe('duplicate-id-aria', function () {
     assert.isString(visualHelperData.elements[0][0]);
   });
 
+  it('emits additively — the reviewPayload option does not change the verdict', function () {
+    fixtureSetup('<div id="dup"></div><div id="dup"></div>');
+    var node = fixture.querySelector('div');
+    var withOption = checkEvaluate.call(checkContext, node, options);
+    checkContext.reset();
+    var withoutOption = checkEvaluate.call(checkContext, node, {});
+    assert.equal(withOption, withoutOption);
+    assert.isFalse(withOption);
+  });
+
   it('after dedupes results by data.id', function () {
     var results = checks['duplicate-id-aria'].after([
       { data: { id: 'a' } },
diff --git a/test/checks/parser/duplicate-id.js b/test/checks/parser/duplicate-id.js
index f48b8e520..aa8adf4de 100644
--- a/test/checks/parser/duplicate-id.js
+++ b/test/checks/parser/duplicate-id.js
@@ -31,6 +31,19 @@ describe('duplicate-id', function () {
     assert.deepEqual(checkContext._relatedNodes, [node.nextSibling]);
   });
 
+  it('keeps data a plain id string and emits no reviewPayload when the reviewPayload option is off', function () {
+    // Guards the static/active duplicate-id checks: they share this evaluate
+    // but never pass the reviewPayload option, so they must keep string data.
+    fixture.innerHTML = '<div id="target"></div><div id="target"></div>';
+    var node = fixture.querySelector('#target');
+    assert.isFalse(
+      axe.testUtils.getCheckEvaluate('duplicate-id').call(checkContext, node)
+    );
+    assert.isString(checkContext._data);
+    assert.equal(checkContext._data, 'target');
+    assert.isUndefined(checkContext._data.reviewPayload);
+  });
+
   it('should return remove duplicates', function () {
     assert.deepEqual(
       checks['duplicate-id'].after([
@@ -42,6 +55,19 @@ describe('duplicate-id', function () {
     );
   });
 
+  it('removes duplicates for object-shaped data (shared after with the aria variant)', function () {
+    // The shared after dedupes by data.id when data is an object
+    // ({ id, reviewPayload }), as emitted by the aria (bulk-review) variant.
+    assert.deepEqual(
+      checks['duplicate-id'].after([
+        { data: { id: 'a' } },
+        { data: { id: 'b' } },
+        { data: { id: 'b' } }
+      ]),
+      [{ data: { id: 'a' } }, { data: { id: 'b' } }]
+    );
+  });
+
   it('should ignore empty ids', function () {
     fixture.innerHTML =
       '<div data-testelm="1" id=""></div><div data-testelm="2"  id=""></div>';

From 32c3c0574fcc893e89b8724b2dbc2df4ff625380 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 10:26:15 +0530
Subject: [PATCH 04/13] AXE-3625: Trim bulk-review code comments to one-liners

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 lib/checks/keyboard/focusable-disabled-evaluate.js          | 6 +-----
 lib/checks/keyboard/focusable-not-tabbable-evaluate.js      | 6 +-----
 lib/checks/navigation/identical-links-same-purpose-after.js | 6 +-----
 lib/checks/parsing/duplicate-id-evaluate.js                 | 1 +
 .../checks/navigation/identical-links-same-purpose-after.js | 3 +--
 test/checks/parser/duplicate-id-aria.js                     | 3 +--
 test/checks/parser/duplicate-id.js                          | 6 ++----
 7 files changed, 8 insertions(+), 23 deletions(-)

diff --git a/lib/checks/keyboard/focusable-disabled-evaluate.js b/lib/checks/keyboard/focusable-disabled-evaluate.js
index b5a77c1a4..683f26fa5 100644
--- a/lib/checks/keyboard/focusable-disabled-evaluate.js
+++ b/lib/checks/keyboard/focusable-disabled-evaluate.js
@@ -38,11 +38,7 @@ function focusableDisabledEvaluate(node, options, virtualNode) {
     ? undefined
     : false;
 
-  // a11y-rule-aria-hidden-focus: surface the focusable descendants for the
-  // bulk-review "Review Hidden Elements" visual helper. Reuse DqElement.selector
-  // (the same format-aware path relatedNodes uses) so the list and the overlay
-  // share identical selectors; no new computation. Wrapped so any failure to
-  // build the payload (e.g. an unexpected node) can never fail the check/scan.
+  // a11y-rule-aria-hidden-focus: emit focusable descendants for bulk review; additive, guarded.
   try {
     this.data({
       reviewPayload: {
diff --git a/lib/checks/keyboard/focusable-not-tabbable-evaluate.js b/lib/checks/keyboard/focusable-not-tabbable-evaluate.js
index e0e2dfe79..fbc2f6815 100644
--- a/lib/checks/keyboard/focusable-not-tabbable-evaluate.js
+++ b/lib/checks/keyboard/focusable-not-tabbable-evaluate.js
@@ -38,11 +38,7 @@ function focusableNotTabbableEvaluate(node, options, virtualNode) {
     ? undefined
     : false;
 
-  // a11y-rule-aria-hidden-focus: surface the focusable descendants for the
-  // bulk-review "Review Hidden Elements" visual helper. Reuse DqElement.selector
-  // (the same format-aware path relatedNodes uses) so the list and the overlay
-  // share identical selectors; no new computation. Wrapped so any failure to
-  // build the payload (e.g. an unexpected node) can never fail the check/scan.
+  // a11y-rule-aria-hidden-focus: emit focusable descendants for bulk review; additive, guarded.
   try {
     this.data({
       reviewPayload: {
diff --git a/lib/checks/navigation/identical-links-same-purpose-after.js b/lib/checks/navigation/identical-links-same-purpose-after.js
index 702b43d7f..454232106 100644
--- a/lib/checks/navigation/identical-links-same-purpose-after.js
+++ b/lib/checks/navigation/identical-links-same-purpose-after.js
@@ -84,11 +84,7 @@ function identicalLinksSamePurposeAfter(results) {
       currentResult.result = undefined;
     }
 
-    /**
-     * The node's own DqElement is stored at `relatedNodes[0]` by the evaluate
-     * fn; capture it before `relatedNodes` is rebuilt so the link group can
-     * include the primary link, not just its same-name siblings.
-     */
+    // primary link's DqElement is at relatedNodes[0]; capture before rebuild.
     const currentNode = currentResult.relatedNodes[0];
 
     /**
diff --git a/lib/checks/parsing/duplicate-id-evaluate.js b/lib/checks/parsing/duplicate-id-evaluate.js
index f4687aee9..8914fe8f9 100644
--- a/lib/checks/parsing/duplicate-id-evaluate.js
+++ b/lib/checks/parsing/duplicate-id-evaluate.js
@@ -17,6 +17,7 @@ function duplicateIdEvaluate(node, options = {}) {
     this.relatedNodes(matchingNodes);
   }
 
+  // a11y-rule-duplicate-id-aria: emit duplicate id + sharing elements for bulk review when enabled.
   if (options.reviewPayload) {
     const verdict = matchingNodes.length === 0;
     const data = { id };
diff --git a/test/checks/navigation/identical-links-same-purpose-after.js b/test/checks/navigation/identical-links-same-purpose-after.js
index 21eb030e0..7a8a4ff24 100644
--- a/test/checks/navigation/identical-links-same-purpose-after.js
+++ b/test/checks/navigation/identical-links-same-purpose-after.js
@@ -200,8 +200,7 @@ describe('identical-links-same-purpose-after tests', function () {
     var results = check.after([nodeOneData, nodeTwoData]);
     assert.lengthOf(results, 1);
 
-    // full group: the primary link plus every same-name sibling, each entry a
-    // DqElement.selector array identical to relatedNodes[].selector
+    // full group (primary + same-name siblings), each a DqElement.selector array
     var visualHelperData = results[0].data.reviewPayload.visualHelperData;
     assert.deepEqual(visualHelperData.identicalLinks, [
       ['#card-1 > a.read-more'],
diff --git a/test/checks/parser/duplicate-id-aria.js b/test/checks/parser/duplicate-id-aria.js
index c23ad78e2..9655e52db 100644
--- a/test/checks/parser/duplicate-id-aria.js
+++ b/test/checks/parser/duplicate-id-aria.js
@@ -33,8 +33,7 @@ describe('duplicate-id-aria', function () {
     assert.equal(checkContext._data.id, 'dup');
     var visualHelperData = checkContext._data.reviewPayload.visualHelperData;
     assert.equal(visualHelperData.duplicateId, 'dup');
-    // the OTHER elements sharing the id (relatedNodes), each a
-    // DqElement.selector array identical to relatedNodes[].selector
+    // the other elements sharing the id, each a DqElement.selector array
     assert.lengthOf(visualHelperData.elements, 2);
     assert.isArray(visualHelperData.elements[0]);
     assert.isString(visualHelperData.elements[0][0]);
diff --git a/test/checks/parser/duplicate-id.js b/test/checks/parser/duplicate-id.js
index aa8adf4de..57e513169 100644
--- a/test/checks/parser/duplicate-id.js
+++ b/test/checks/parser/duplicate-id.js
@@ -32,8 +32,7 @@ describe('duplicate-id', function () {
   });
 
   it('keeps data a plain id string and emits no reviewPayload when the reviewPayload option is off', function () {
-    // Guards the static/active duplicate-id checks: they share this evaluate
-    // but never pass the reviewPayload option, so they must keep string data.
+    // static/active checks share this evaluate but never set the option.
     fixture.innerHTML = '<div id="target"></div><div id="target"></div>';
     var node = fixture.querySelector('#target');
     assert.isFalse(
@@ -56,8 +55,7 @@ describe('duplicate-id', function () {
   });
 
   it('removes duplicates for object-shaped data (shared after with the aria variant)', function () {
-    // The shared after dedupes by data.id when data is an object
-    // ({ id, reviewPayload }), as emitted by the aria (bulk-review) variant.
+    // shared after dedupes object data by data.id (aria variant shape).
     assert.deepEqual(
       checks['duplicate-id'].after([
         { data: { id: 'a' } },

From 09aa4b0a9fdbc0f3b0b45662d5f219a15e543bcd Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 12:10:46 +0530
Subject: [PATCH 05/13] AXE-3625: Bulk-review priority rules (axe-core subset)

Wire metadata.defaultCategory for the prioritized rules (aria-valid-attr-value,
video-caption, bypass, css-orientation-lock) and emit reviewPayload.visualHelperData
for aria-valid-attr-value. Excludes aria-hidden-focus / identical-links-same-purpose
/ duplicate-id-aria (deferred).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../aria/aria-valid-attr-value-evaluate.js    |  6 ++-
 lib/rules/aria-valid-attr-value.json          |  1 +
 lib/rules/bypass.json                         |  1 +
 lib/rules/css-orientation-lock.json           |  1 +
 lib/rules/video-caption.json                  |  1 +
 test/checks/aria/aria-valid-attr-value.js     | 46 +++++++++++++++++++
 6 files changed, 55 insertions(+), 1 deletion(-)
 create mode 100644 test/checks/aria/aria-valid-attr-value.js

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index ad43d1393..b88b8627a 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -140,7 +140,11 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
   }
 
   if (needsReview) {
-    this.data({ messageKey, needsReview });
+    this.data({
+      messageKey,
+      needsReview,
+      reviewPayload: { visualHelperData: { ariaAttribute: needsReview } }
+    });
     return undefined;
   }
 
diff --git a/lib/rules/aria-valid-attr-value.json b/lib/rules/aria-valid-attr-value.json
index ac92598e2..dc39253c3 100644
--- a/lib/rules/aria-valid-attr-value.json
+++ b/lib/rules/aria-valid-attr-value.json
@@ -13,6 +13,7 @@
   ],
   "actIds": ["6a7281"],
   "metadata": {
+    "defaultCategory": "aria-attributes",
     "description": "Ensure all ARIA attributes have valid values",
     "help": "ARIA attributes must conform to valid values",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/bypass.json b/lib/rules/bypass.json
index 02666d9da..b05b74210 100644
--- a/lib/rules/bypass.json
+++ b/lib/rules/bypass.json
@@ -20,6 +20,7 @@
   ],
   "actIds": ["cf77f2", "047fe0", "b40fd1", "3e12e1", "ye5d6e"],
   "metadata": {
+    "defaultCategory": "bypass-blocks",
     "description": "Ensure each page has at least one mechanism for a user to bypass navigation and jump straight to the content",
     "help": "Page must have means to bypass repeated blocks",
     "needsReviewConfidence": 49.99
diff --git a/lib/rules/css-orientation-lock.json b/lib/rules/css-orientation-lock.json
index 06bc7d426..ed134b978 100644
--- a/lib/rules/css-orientation-lock.json
+++ b/lib/rules/css-orientation-lock.json
@@ -16,6 +16,7 @@
   ],
   "actIds": ["b33eff"],
   "metadata": {
+    "defaultCategory": "orientation",
     "description": "Ensure content is not locked to any specific display orientation, and the content is operable in all display orientations",
     "help": "CSS Media queries must not lock display orientation",
     "violationConfidence": 90,
diff --git a/lib/rules/video-caption.json b/lib/rules/video-caption.json
index b0297ddb0..98f23d092 100644
--- a/lib/rules/video-caption.json
+++ b/lib/rules/video-caption.json
@@ -17,6 +17,7 @@
   ],
   "actIds": ["eac66b"],
   "metadata": {
+    "defaultCategory": "media-captions",
     "description": "Ensure <video> elements have captions",
     "help": "<video> elements must have captions",
     "needsReviewConfidence": 49.99
diff --git a/test/checks/aria/aria-valid-attr-value.js b/test/checks/aria/aria-valid-attr-value.js
new file mode 100644
index 000000000..c0bd0d371
--- /dev/null
+++ b/test/checks/aria/aria-valid-attr-value.js
@@ -0,0 +1,46 @@
+describe('aria-valid-attr-value', () => {
+  'use strict';
+
+  const checkContext = axe.testUtils.MockCheckContext();
+  const checkSetup = axe.testUtils.checkSetup;
+  const checkEvaluate = axe.testUtils.getCheckEvaluate('aria-valid-attr-value');
+
+  afterEach(() => {
+    checkContext.reset();
+  });
+
+  it('emits the flagged aria-current attribute in reviewPayload.visualHelperData', () => {
+    const params = checkSetup(
+      '<div id="target" aria-current="active">Contents</div>'
+    );
+    assert.isUndefined(checkEvaluate.apply(checkContext, params));
+    assert.equal(checkContext._data.messageKey, 'ariaCurrent');
+    assert.equal(checkContext._data.needsReview, 'aria-current="active"');
+    assert.deepEqual(checkContext._data.reviewPayload, {
+      visualHelperData: { ariaAttribute: 'aria-current="active"' }
+    });
+  });
+
+  it('emits the flagged aria-labelledby attribute in reviewPayload.visualHelperData', () => {
+    const params = checkSetup(
+      '<div id="target" aria-labelledby="missing-id">Contents</div>'
+    );
+    assert.isUndefined(checkEvaluate.apply(checkContext, params));
+    assert.equal(
+      checkContext._data.needsReview,
+      'aria-labelledby="missing-id"'
+    );
+    assert.deepEqual(checkContext._data.reviewPayload, {
+      visualHelperData: { ariaAttribute: 'aria-labelledby="missing-id"' }
+    });
+  });
+
+  it('does not add a reviewPayload on the invalid-value (violation) path', () => {
+    const params = checkSetup(
+      '<div id="target" role="checkbox" aria-checked="foo">Contents</div>'
+    );
+    assert.isFalse(checkEvaluate.apply(checkContext, params));
+    assert.isArray(checkContext._data);
+    assert.notProperty(checkContext._data, 'reviewPayload');
+  });
+});

From afa83019401c62c770b437415e39fb314e158609 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 13:25:42 +0530
Subject: [PATCH 06/13] AXE-3625: Fix aria-valid-attr-value test + add
 submodule tag

Update the 8 needs-review deepEqual expectations in valid-attr-value.js
to include the reviewPayload the check now emits on the needs-review
path, and add the // a11y-rule-aria-valid-attr-value tag on the emit.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../aria/aria-valid-attr-value-evaluate.js    |  1 +
 test/checks/aria/valid-attr-value.js          | 36 ++++++++++++++-----
 2 files changed, 29 insertions(+), 8 deletions(-)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index b88b8627a..49d1322da 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -140,6 +140,7 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
   }
 
   if (needsReview) {
+    // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
     this.data({
       messageKey,
       needsReview,
diff --git a/test/checks/aria/valid-attr-value.js b/test/checks/aria/valid-attr-value.js
index 956c3b496..1b8005473 100644
--- a/test/checks/aria/valid-attr-value.js
+++ b/test/checks/aria/valid-attr-value.js
@@ -119,7 +119,10 @@ describe('aria-valid-attr-value', function () {
     );
     assert.deepEqual(checkContext._data, {
       messageKey: 'controlsWithinPopup',
-      needsReview: 'aria-controls="test"'
+      needsReview: 'aria-controls="test"',
+      reviewPayload: {
+        visualHelperData: { ariaAttribute: 'aria-controls="test"' }
+      }
     });
   });
 
@@ -160,7 +163,10 @@ describe('aria-valid-attr-value', function () {
     );
     assert.deepEqual(checkContext._data, {
       messageKey: 'noId',
-      needsReview: 'aria-describedby="test"'
+      needsReview: 'aria-describedby="test"',
+      reviewPayload: {
+        visualHelperData: { ariaAttribute: 'aria-describedby="test"' }
+      }
     });
   });
 
@@ -174,7 +180,10 @@ describe('aria-valid-attr-value', function () {
       assert.isUndefined(validAttrValueCheck.apply(checkContext, params));
       assert.deepEqual(checkContext._data, {
         messageKey: 'noIdShadow',
-        needsReview: 'aria-describedby="test"'
+        needsReview: 'aria-describedby="test"',
+        reviewPayload: {
+          visualHelperData: { ariaAttribute: 'aria-describedby="test"' }
+        }
       });
     }
   );
@@ -188,7 +197,10 @@ describe('aria-valid-attr-value', function () {
     );
     assert.deepEqual(checkContext._data, {
       messageKey: 'noId',
-      needsReview: 'aria-labelledby="test"'
+      needsReview: 'aria-labelledby="test"',
+      reviewPayload: {
+        visualHelperData: { ariaAttribute: 'aria-labelledby="test"' }
+      }
     });
   });
 
@@ -202,7 +214,10 @@ describe('aria-valid-attr-value', function () {
       assert.isUndefined(validAttrValueCheck.apply(checkContext, params));
       assert.deepEqual(checkContext._data, {
         messageKey: 'noIdShadow',
-        needsReview: 'aria-labelledby="test"'
+        needsReview: 'aria-labelledby="test"',
+        reviewPayload: {
+          visualHelperData: { ariaAttribute: 'aria-labelledby="test"' }
+        }
       });
     }
   );
@@ -248,7 +263,8 @@ describe('aria-valid-attr-value', function () {
       );
       assert.deepEqual(checkContext._data, {
         messageKey: 'empty',
-        needsReview: 'aria-checked'
+        needsReview: 'aria-checked',
+        reviewPayload: { visualHelperData: { ariaAttribute: 'aria-checked' } }
       });
     });
 
@@ -261,7 +277,8 @@ describe('aria-valid-attr-value', function () {
       );
       assert.deepEqual(checkContext._data, {
         messageKey: 'empty',
-        needsReview: 'aria-checked'
+        needsReview: 'aria-checked',
+        reviewPayload: { visualHelperData: { ariaAttribute: 'aria-checked' } }
       });
     });
 
@@ -320,7 +337,10 @@ describe('aria-valid-attr-value', function () {
       );
       assert.deepEqual(checkContext._data, {
         messageKey: 'idrefs',
-        needsReview: 'aria-owns="test"'
+        needsReview: 'aria-owns="test"',
+        reviewPayload: {
+          visualHelperData: { ariaAttribute: 'aria-owns="test"' }
+        }
       });
     });
 

From a50068ff58b14b824c3ec009a6e2d64ef8359efc Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 14:27:37 +0530
Subject: [PATCH 07/13] Update aria-valid-attr-value-evaluate.js

---
 lib/checks/aria/aria-valid-attr-value-evaluate.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index 49d1322da..b88b8627a 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -140,7 +140,6 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
   }
 
   if (needsReview) {
-    // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
     this.data({
       messageKey,
       needsReview,

From 8cc8ca18d2bad78e85d71db27b020502859a0706 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 14:32:03 +0530
Subject: [PATCH 08/13] Update aria-valid-attr-value-evaluate.js

---
 lib/checks/aria/aria-valid-attr-value-evaluate.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index b88b8627a..10c9a5231 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -139,6 +139,7 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
     return false;
   }
 
+	// a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
   if (needsReview) {
     this.data({
       messageKey,

From cf9afe020b6d0bcf1cfc0a6eb91f7bcbec76ec00 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 14:32:43 +0530
Subject: [PATCH 09/13] Update aria-valid-attr-value-evaluate.js

---
 lib/checks/aria/aria-valid-attr-value-evaluate.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index 10c9a5231..403627bbd 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -139,11 +139,11 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
     return false;
   }
 
-	// a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
   if (needsReview) {
     this.data({
       messageKey,
       needsReview,
+	  // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
       reviewPayload: { visualHelperData: { ariaAttribute: needsReview } }
     });
     return undefined;

From c29f9704acf2ff01583fc017dc2e9a964cde9d66 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 15:05:52 +0530
Subject: [PATCH 10/13] formated

---
 lib/checks/aria/aria-valid-attr-value-evaluate.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index 403627bbd..bb3e8cc16 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -143,7 +143,7 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
     this.data({
       messageKey,
       needsReview,
-	  // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
+	    // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
       reviewPayload: { visualHelperData: { ariaAttribute: needsReview } }
     });
     return undefined;

From 256e0770c61e312f2776487aaaf59b4800c85eda Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 15:16:05 +0530
Subject: [PATCH 11/13] AXE-3625: Fix prettier formatting on reviewPayload tag
 comment

Normalize the // a11y-rule comment indentation (tab -> spaces) so
fmt:check (prettier --check) passes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 lib/checks/aria/aria-valid-attr-value-evaluate.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index bb3e8cc16..00431bbac 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -143,7 +143,7 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
     this.data({
       messageKey,
       needsReview,
-	    // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
+      // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
       reviewPayload: { visualHelperData: { ariaAttribute: needsReview } }
     });
     return undefined;

From d066b98fd7f1ea24d761cb38eeb2f7bb6d17f982 Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 16:59:47 +0530
Subject: [PATCH 12/13] added changes

---
 test/checks/aria/valid-attr-value.js | 36 +++++++++++++++++++++-------
 1 file changed, 28 insertions(+), 8 deletions(-)

diff --git a/test/checks/aria/valid-attr-value.js b/test/checks/aria/valid-attr-value.js
index 956c3b496..1b8005473 100644
--- a/test/checks/aria/valid-attr-value.js
+++ b/test/checks/aria/valid-attr-value.js
@@ -119,7 +119,10 @@ describe('aria-valid-attr-value', function () {
     );
     assert.deepEqual(checkContext._data, {
       messageKey: 'controlsWithinPopup',
-      needsReview: 'aria-controls="test"'
+      needsReview: 'aria-controls="test"',
+      reviewPayload: {
+        visualHelperData: { ariaAttribute: 'aria-controls="test"' }
+      }
     });
   });
 
@@ -160,7 +163,10 @@ describe('aria-valid-attr-value', function () {
     );
     assert.deepEqual(checkContext._data, {
       messageKey: 'noId',
-      needsReview: 'aria-describedby="test"'
+      needsReview: 'aria-describedby="test"',
+      reviewPayload: {
+        visualHelperData: { ariaAttribute: 'aria-describedby="test"' }
+      }
     });
   });
 
@@ -174,7 +180,10 @@ describe('aria-valid-attr-value', function () {
       assert.isUndefined(validAttrValueCheck.apply(checkContext, params));
       assert.deepEqual(checkContext._data, {
         messageKey: 'noIdShadow',
-        needsReview: 'aria-describedby="test"'
+        needsReview: 'aria-describedby="test"',
+        reviewPayload: {
+          visualHelperData: { ariaAttribute: 'aria-describedby="test"' }
+        }
       });
     }
   );
@@ -188,7 +197,10 @@ describe('aria-valid-attr-value', function () {
     );
     assert.deepEqual(checkContext._data, {
       messageKey: 'noId',
-      needsReview: 'aria-labelledby="test"'
+      needsReview: 'aria-labelledby="test"',
+      reviewPayload: {
+        visualHelperData: { ariaAttribute: 'aria-labelledby="test"' }
+      }
     });
   });
 
@@ -202,7 +214,10 @@ describe('aria-valid-attr-value', function () {
       assert.isUndefined(validAttrValueCheck.apply(checkContext, params));
       assert.deepEqual(checkContext._data, {
         messageKey: 'noIdShadow',
-        needsReview: 'aria-labelledby="test"'
+        needsReview: 'aria-labelledby="test"',
+        reviewPayload: {
+          visualHelperData: { ariaAttribute: 'aria-labelledby="test"' }
+        }
       });
     }
   );
@@ -248,7 +263,8 @@ describe('aria-valid-attr-value', function () {
       );
       assert.deepEqual(checkContext._data, {
         messageKey: 'empty',
-        needsReview: 'aria-checked'
+        needsReview: 'aria-checked',
+        reviewPayload: { visualHelperData: { ariaAttribute: 'aria-checked' } }
       });
     });
 
@@ -261,7 +277,8 @@ describe('aria-valid-attr-value', function () {
       );
       assert.deepEqual(checkContext._data, {
         messageKey: 'empty',
-        needsReview: 'aria-checked'
+        needsReview: 'aria-checked',
+        reviewPayload: { visualHelperData: { ariaAttribute: 'aria-checked' } }
       });
     });
 
@@ -320,7 +337,10 @@ describe('aria-valid-attr-value', function () {
       );
       assert.deepEqual(checkContext._data, {
         messageKey: 'idrefs',
-        needsReview: 'aria-owns="test"'
+        needsReview: 'aria-owns="test"',
+        reviewPayload: {
+          visualHelperData: { ariaAttribute: 'aria-owns="test"' }
+        }
       });
     });
 

From a99cb2934fd0a0472563f6cdece2cab9b4b7d7ba Mon Sep 17 00:00:00 2001
From: Rajath M R <rajath.mr@browserstack.com>
Date: Mon, 29 Jun 2026 17:24:19 +0530
Subject: [PATCH 13/13] Update aria-valid-attr-value-evaluate.js

---
 lib/checks/aria/aria-valid-attr-value-evaluate.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/checks/aria/aria-valid-attr-value-evaluate.js b/lib/checks/aria/aria-valid-attr-value-evaluate.js
index 00431bbac..900442368 100644
--- a/lib/checks/aria/aria-valid-attr-value-evaluate.js
+++ b/lib/checks/aria/aria-valid-attr-value-evaluate.js
@@ -143,7 +143,7 @@ export default function ariaValidAttrValueEvaluate(node, options, virtualNode) {
     this.data({
       messageKey,
       needsReview,
-      // a11y-rule-aria-valid-attr-value: emit reviewPayload for bulk review
+      // a11y-rule-aria-valid-attr-value: emit reviewPayload
       reviewPayload: { visualHelperData: { ariaAttribute: needsReview } }
     });
     return undefined;