AXE-3625: Trim bulk-review code comments to one-liners

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: rajathmr2000

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: {

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: {

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];
 
     /**

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 };

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'],

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]);

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' } },