@surea11y/core 1.4.1 → 1.5.0

package/src/checks/automatic/aria-hidden-focus.js

@@ -146,6 +146,54 @@ function runInPage(ctx) {
     return false;
   }
 
+  // An open modal is expected to trap focus, so a focusable background behind
+  // it may be unreachable; that can't be proven statically, so the finding is
+  // downgraded to cantTell instead of fail. A modal that is display:none is
+  // not open, so it does not count.
+  function isRenderedForModal(node) {
+    if (!node) return false;
+    if (!isDomVisibleEligible) return true;
+    try {
+      const vis = isDomVisibleEligible(node, ctx, {
+        visibilityMode: 'styleOnly',
+        disableGeometry: true
+      });
+      if (vis && vis.eligible === false) return false;
+    } catch {
+      // treat as rendered
+    }
+    return true;
+  }
+
+  // Native <dialog>, aria-modal="true", or a dialog/alertdialog role, so
+  // libraries that leave aria-modal off (e.g. Angular Material defaults to
+  // aria-modal="false") still count as an open modal.
+  function collectOpenModalCandidates() {
+    const nodes = qAll('dialog[open],[aria-modal="true"],[role="dialog"],[role="alertdialog"]');
+    const out = [];
+    for (let i = 0; i < nodes.length; i++) {
+      const n = nodes[i];
+      if (!n || !n.getAttribute) continue;
+      if (!isRenderedForModal(n)) continue;
+      out.push(n);
+    }
+    return out;
+  }
+
+  // Only a modal in a separate subtree is the "hidden background behind an
+  // open dialog" case. A modal inside the aria-hidden subtree (or an
+  // aria-hidden root inside the modal) is a genuine defect, so keep it a fail.
+  function hasSeparateOpenModal(rootEl, candidates) {
+    for (let i = 0; i < candidates.length; i++) {
+      const m = candidates[i];
+      if (!m || m === rootEl) continue;
+      if (isWithinComposedSubtree(m, rootEl)) continue;
+      if (isWithinComposedSubtree(rootEl, m)) continue;
+      return true;
+    }
+    return false;
+  }
+
   function getDeepActiveElement() {
     let cur = document && document.activeElement ? document.activeElement : null;
     let guard = 0;
@@ -713,6 +761,8 @@ function runInPage(ctx) {
     entry.rootIsFocusable = isActuallyFocusable(el);
   }
 
+  const modalCandidates = collectOpenModalCandidates();
+
   // 3) Report one occurrence per aria-hidden root that contains focusable content.
   const failOccurrences = [];
   const uncertainOccurrences = [];
@@ -765,24 +815,56 @@ function runInPage(ctx) {
         : `aria-hidden ${tagName} is focusable (${totalFocusable} focusable element(s)).`
       : `aria-hidden ${tagName} contains ${totalFocusable} focusable element(s).`;
 
-    const runtimeProbe = probeImmediateFocusRedirect(entry);
-    const downgradedToCantTell = !!(runtimeProbe && runtimeProbe.redirected);
-
-    const cantTellSummary = `aria-hidden ${tagName} received focus but focus moved immediately to another element. Verify sentinel/focus-trap behavior.`;
+    // A modal takes precedence over the redirect probe; skip the probe when a
+    // modal already explains the background.
+    const modalOpenOutside = modalCandidates.length
+      ? hasSeparateOpenModal(el, modalCandidates)
+      : false;
+
+    const runtimeProbe = modalOpenOutside ? null : probeImmediateFocusRedirect(entry);
+    const downgradedByRedirect = !!(runtimeProbe && runtimeProbe.redirected);
+    const downgradedByModal = modalOpenOutside;
+    const downgradedToCantTell = downgradedByModal || downgradedByRedirect;
+
+    const cantTellRedirectSummary = `aria-hidden ${tagName} received focus but focus moved immediately to another element. Verify sentinel/focus-trap behavior.`;
+    const cantTellModalSummary = `aria-hidden ${tagName} contains ${totalFocusable} focusable element(s) while a modal dialog is open. If the modal keeps keyboard focus trapped they may be unreachable; verify focus cannot land on them.`;
+
+    let occSummary;
+    let occHint;
+    let occSummaryKey;
+    let occHintKey;
+    let occReasonCode;
+
+    if (downgradedByModal) {
+      occSummary = cantTellModalSummary;
+      occHint =
+        'A modal dialog appears to be open. Prefer making the background inert (or a native <dialog> opened with showModal()) so it leaves the tab order, then verify keyboard focus stays within the dialog.';
+      occSummaryKey = 'ariaHidden_focus_summary_cantTell_modal';
+      occHintKey = 'ariaHidden_focus_hint_cantTell_modal';
+      occReasonCode = 'ariaHiddenFocusable_modalOpen_needsReview';
+    } else if (downgradedByRedirect) {
+      occSummary = cantTellRedirectSummary;
+      occHint =
+        'Verify this is an intentional focus sentinel/focus-trap handoff and that keyboard users never remain on hidden focus targets.';
+      occSummaryKey = 'ariaHidden_focus_summary_cantTell_redirect';
+      occHintKey = 'ariaHidden_focus_hint_cantTell_redirect';
+      occReasonCode = 'ariaHiddenFocusable_runtimeRedirect_needsReview';
+    } else {
+      occSummary = summaryText;
+      occHint =
+        'Remove focusability from descendants or remove aria-hidden; ensure focus and accessibility trees stay aligned.';
+      occSummaryKey = summaryKey;
+      occHintKey = 'ariaHidden_focus_hint_fail';
+      occReasonCode = reasonCode;
+    }
 
     const baseOccurrence = {
-      summary: downgradedToCantTell ? cantTellSummary : summaryText,
-      hint: downgradedToCantTell
-        ? 'Verify this is an intentional focus sentinel/focus-trap handoff and that keyboard users never remain on hidden focus targets.'
-        : 'Remove focusability from descendants or remove aria-hidden; ensure focus and accessibility trees stay aligned.',
+      summary: occSummary,
+      hint: occHint,
       occurrenceOutcome: downgradedToCantTell ? 'cantTell' : 'fail',
       i18n: {
-        summaryKey: downgradedToCantTell
-          ? 'ariaHidden_focus_summary_cantTell_redirect'
-          : summaryKey,
-        hintKey: downgradedToCantTell
-          ? 'ariaHidden_focus_hint_cantTell_redirect'
-          : 'ariaHidden_focus_hint_fail',
+        summaryKey: occSummaryKey,
+        hintKey: occHintKey,
         params: {
           element: tagName,
           focusableCount: String(totalFocusable),
@@ -795,15 +877,14 @@ function runInPage(ctx) {
       },
       data: {
         details: {
-          reasonCode: downgradedToCantTell
-            ? 'ariaHiddenFocusable_runtimeRedirect_needsReview'
-            : reasonCode,
+          reasonCode: occReasonCode,
           metrics: {
             focusableTotal: totalFocusable,
             focusableDescendants: descendantFocusable,
             rootIsFocusable: selfFocusable,
             offendersCaptured: entry.offenders.length,
-            visibilityHints: hintsArr.slice(0)
+            visibilityHints: hintsArr.slice(0),
+            modalOpen: !!downgradedByModal
           },
           runtimeProbe: runtimeProbe || null,
           offenders: entry.offenders.slice(0)

package/src/checks/automatic/aria-prohibited-attr.js

@@ -167,25 +167,22 @@ function runInPage(ctx) {
 
     if (!present.length) continue;
 
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
     for (const attr of present) {
-      failOccurrences.push({
-        selector: stableSelector,
-        html,
-        occurrenceOutcome: 'fail',
-        summary: 'This attribute is prohibited on this element’s role.',
-        hint: 'Remove this attribute; this role must not carry an accessible name.',
-        i18n: {
-          summaryKey: 'ariaProhibitedAttr_summary_fail',
-          hintKey: 'ariaProhibitedAttr_hint_fail',
-          params: { attr, role }
-        },
-        data: {
-          details: { reasonCode: 'ARIA_ATTR_PROHIBITED', attr, role }
-        }
-      });
+      failOccurrences.push(
+        helpers.reportOccurrence(el, {
+          occurrenceOutcome: 'fail',
+          summary: 'This attribute is prohibited on this element’s role.',
+          hint: 'Remove this attribute; this role must not carry an accessible name.',
+          i18n: {
+            summaryKey: 'ariaProhibitedAttr_summary_fail',
+            hintKey: 'ariaProhibitedAttr_hint_fail',
+            params: { attr, role }
+          },
+          data: {
+            details: { reasonCode: 'ARIA_ATTR_PROHIBITED', attr, role }
+          }
+        })
+      );
     }
   }
 
@@ -361,47 +358,49 @@ function runInPage(ctx) {
       String(nameInfo.value || '').trim() !== ''
     );
 
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
     for (const attr of present) {
       if (hasContentFallback) {
-        cantTellOccurrences.push({
-          selector: stableSelector,
-          html,
-          occurrenceOutcome: 'cantTell',
-          summary: `This ${tag} has no role, so ${attr} may not be exposed as its accessible name by assistive technology — but the element's own content already provides one.`,
-          hint: 'Verify whether the existing text content already serves as this element’s label; if so the naming attribute is redundant, otherwise give the element a role that supports naming (e.g. role="img").',
-          i18n: {
-            summaryKey: 'ariaProhibitedAttr_summary_cantTell_roleless',
-            hintKey: 'ariaProhibitedAttr_hint_cantTell_roleless',
-            params: { attr, element: tag }
-          },
-          data: {
-            details: {
-              reasonCode: 'ARIA_ATTR_PROHIBITED_ROLELESS_NEEDS_REVIEW',
-              attr,
-              role: null,
-              element: tag
+        cantTellOccurrences.push(
+          helpers.reportOccurrence(el, {
+            occurrenceOutcome: 'cantTell',
+            summary: `This ${tag} has no role, so ${attr} may not be exposed as its accessible name by assistive technology — but the element's own content already provides one.`,
+            hint: 'Verify whether the existing text content already serves as this element’s label; if so the naming attribute is redundant, otherwise give the element a role that supports naming (e.g. role="img").',
+            i18n: {
+              summaryKey: 'ariaProhibitedAttr_summary_cantTell_roleless',
+              hintKey: 'ariaProhibitedAttr_hint_cantTell_roleless',
+              params: { attr, element: tag }
+            },
+            data: {
+              details: {
+                reasonCode: 'ARIA_ATTR_PROHIBITED_ROLELESS_NEEDS_REVIEW',
+                attr,
+                role: null,
+                element: tag
+              }
             }
-          }
-        });
+          })
+        );
       } else {
-        failOccurrences.push({
-          selector: stableSelector,
-          html,
-          occurrenceOutcome: 'fail',
-          summary: `This ${tag} has no role and no other accessible-name source, so ${attr} is not reliably exposed to assistive technology.`,
-          hint: 'Give this element a role that supports an accessible name (e.g. role="img"/"button"), or remove this attribute if it serves no purpose without one.',
-          i18n: {
-            summaryKey: 'ariaProhibitedAttr_summary_fail_roleless',
-            hintKey: 'ariaProhibitedAttr_hint_fail_roleless',
-            params: { attr, element: tag }
-          },
-          data: {
-            details: { reasonCode: 'ARIA_ATTR_PROHIBITED_ROLELESS', attr, role: null, element: tag }
-          }
-        });
+        failOccurrences.push(
+          helpers.reportOccurrence(el, {
+            occurrenceOutcome: 'fail',
+            summary: `This ${tag} has no role and no other accessible-name source, so ${attr} is not reliably exposed to assistive technology.`,
+            hint: 'Give this element a role that supports an accessible name (e.g. role="img"/"button"), or remove this attribute if it serves no purpose without one.',
+            i18n: {
+              summaryKey: 'ariaProhibitedAttr_summary_fail_roleless',
+              hintKey: 'ariaProhibitedAttr_hint_fail_roleless',
+              params: { attr, element: tag }
+            },
+            data: {
+              details: {
+                reasonCode: 'ARIA_ATTR_PROHIBITED_ROLELESS',
+                attr,
+                role: null,
+                element: tag
+              }
+            }
+          })
+        );
       }
     }
   }

package/src/checks/automatic/aria-prohibited-children.js

@@ -256,10 +256,6 @@ function runInPage(ctx) {
     for (const entry of owned) {
       if (entry.role && requiredSet.has(entry.role)) continue;
 
-      const stableSelector = helpers.buildSelector ? helpers.buildSelector(entry.el) : 'html';
-      const html = helpers.getOuterHtmlSnippet
-        ? helpers.getOuterHtmlSnippet(entry.el)
-        : entry.el.outerHTML || '';
       const containerSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
 
       const isRoleless = !entry.role;
@@ -286,29 +282,33 @@ function runInPage(ctx) {
         hintKey = 'ariaProhibitedChildren_hint_fail';
       }
 
-      occurrences.push({
-        selector: stableSelector,
-        html,
-        summary,
-        hint,
-        i18n: {
-          summaryKey,
-          hintKey,
-          params: isRoleless
-            ? { attr: entry.attr, containerRole: role }
-            : { childRole: entry.role, containerRole: role, allowedRoles: requiredOwned.join(', ') }
-        },
-        data: {
-          details: {
-            reasonCode: isRoleless ? 'ARIA_PROHIBITED_CHILD_ROLELESS' : 'ARIA_PROHIBITED_CHILD',
-            childRole: entry.role,
-            attr: entry.attr,
-            containerRole: role,
-            containerSelector,
-            allowedOwnedRoles: requiredOwned
+      occurrences.push(
+        helpers.reportOccurrence(entry.el, {
+          summary,
+          hint,
+          i18n: {
+            summaryKey,
+            hintKey,
+            params: isRoleless
+              ? { attr: entry.attr, containerRole: role }
+              : {
+                  childRole: entry.role,
+                  containerRole: role,
+                  allowedRoles: requiredOwned.join(', ')
+                }
+          },
+          data: {
+            details: {
+              reasonCode: isRoleless ? 'ARIA_PROHIBITED_CHILD_ROLELESS' : 'ARIA_PROHIBITED_CHILD',
+              childRole: entry.role,
+              attr: entry.attr,
+              containerRole: role,
+              containerSelector,
+              allowedOwnedRoles: requiredOwned
+            }
           }
-        }
-      });
+        })
+      );
     }
   }
 

package/src/checks/automatic/aria-required-attr.js

@@ -129,24 +129,21 @@ function runInPage(ctx) {
 
     if (!missing.length) continue;
 
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
     for (const attr of missing) {
-      occurrences.push({
-        selector: stableSelector,
-        html,
-        summary: 'This attribute is required for this element’s role, but is missing.',
-        hint: 'Add this attribute with a valid value for this role.',
-        i18n: {
-          summaryKey: 'ariaRequiredAttr_summary_fail',
-          hintKey: 'ariaRequiredAttr_hint_fail',
-          params: { attr, role }
-        },
-        data: {
-          details: { reasonCode: 'ARIA_ATTR_REQUIRED_MISSING', attr, role }
-        }
-      });
+      occurrences.push(
+        helpers.reportOccurrence(el, {
+          summary: 'This attribute is required for this element’s role, but is missing.',
+          hint: 'Add this attribute with a valid value for this role.',
+          i18n: {
+            summaryKey: 'ariaRequiredAttr_summary_fail',
+            hintKey: 'ariaRequiredAttr_hint_fail',
+            params: { attr, role }
+          },
+          data: {
+            details: { reasonCode: 'ARIA_ATTR_REQUIRED_MISSING', attr, role }
+          }
+        })
+      );
     }
   }
 

package/src/checks/automatic/aria-required-children.js

@@ -253,27 +253,24 @@ function runInPage(ctx) {
 
     if (found) continue;
 
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
-    occurrences.push({
-      selector: stableSelector,
-      html,
-      summary: 'This container role has no owned child with a required role.',
-      hint: 'Add a descendant (or aria-owns-referenced element) with one of the required owned roles.',
-      i18n: {
-        summaryKey: 'ariaRequiredChildren_summary_fail',
-        hintKey: 'ariaRequiredChildren_hint_fail',
-        params: { role, requiredRoles: requiredOwned.join(', ') }
-      },
-      data: {
-        details: {
-          reasonCode: 'ARIA_REQUIRED_CHILD_MISSING',
-          role,
-          requiredOwnedRoles: requiredOwned
+    occurrences.push(
+      helpers.reportOccurrence(el, {
+        summary: 'This container role has no owned child with a required role.',
+        hint: 'Add a descendant (or aria-owns-referenced element) with one of the required owned roles.',
+        i18n: {
+          summaryKey: 'ariaRequiredChildren_summary_fail',
+          hintKey: 'ariaRequiredChildren_hint_fail',
+          params: { role, requiredRoles: requiredOwned.join(', ') }
+        },
+        data: {
+          details: {
+            reasonCode: 'ARIA_REQUIRED_CHILD_MISSING',
+            role,
+            requiredOwnedRoles: requiredOwned
+          }
         }
-      }
-    });
+      })
+    );
   }
 
   if (applicableCount === 0) {

package/src/checks/automatic/aria-required-parent.js

@@ -210,27 +210,24 @@ function runInPage(ctx) {
 
     if (hasContext) continue;
 
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
-    occurrences.push({
-      selector: stableSelector,
-      html,
-      summary: 'This role requires a specific ancestor/owner context role, which was not found.',
-      hint: 'Place this element inside (or aria-owns-reference it from) an element with an acceptable context role.',
-      i18n: {
-        summaryKey: 'ariaRequiredParent_summary_fail',
-        hintKey: 'ariaRequiredParent_hint_fail',
-        params: { role, requiredRoles: requiredContext.join(', ') }
-      },
-      data: {
-        details: {
-          reasonCode: 'ARIA_REQUIRED_PARENT_MISSING',
-          role,
-          requiredContextRoles: requiredContext
+    occurrences.push(
+      helpers.reportOccurrence(el, {
+        summary: 'This role requires a specific ancestor/owner context role, which was not found.',
+        hint: 'Place this element inside (or aria-owns-reference it from) an element with an acceptable context role.',
+        i18n: {
+          summaryKey: 'ariaRequiredParent_summary_fail',
+          hintKey: 'ariaRequiredParent_hint_fail',
+          params: { role, requiredRoles: requiredContext.join(', ') }
+        },
+        data: {
+          details: {
+            reasonCode: 'ARIA_REQUIRED_PARENT_MISSING',
+            role,
+            requiredContextRoles: requiredContext
+          }
         }
-      }
-    });
+      })
+    );
   }
 
   if (applicableCount === 0) {

package/src/checks/automatic/aria-roles-valid.js

@@ -68,14 +68,28 @@ function runInPage(ctx) {
   let applicableCount = 0;
 
   // Programmatically hidden per the ACT glossary: display:none, visibility not
-  // visible, or aria-hidden on the element or an ancestor.
+  // visible, or aria-hidden on the element or an ancestor. inert is treated
+  // the same although the glossary predates it -- an inert subtree is out of
+  // the accessibility tree entirely, so a role on it reaches no one.
   function isHidden(el) {
     try {
       if (typeof helpers.isDomVisibleEligible === 'function') {
         if (!helpers.isDomVisibleEligible(el, ctx)) return true;
       }
-      for (let n = el; n && n.getAttribute; n = n.parentElement) {
+      // Walk the composed tree, not parentElement: that stops at a shadow
+      // root, so a host carrying aria-hidden or inert would never be seen
+      // from inside its own shadow content.
+      const up =
+        typeof helpers.composedParent === 'function'
+          ? helpers.composedParent
+          : (n) => n.parentElement;
+
+      // A shadow root has no getAttribute, so skip past it rather than
+      // stopping: the host one step further up is the node that matters.
+      for (let n = el; n; n = up(n)) {
+        if (!n.getAttribute) continue;
         if (String(n.getAttribute('aria-hidden') || '').toLowerCase() === 'true') return true;
+        if (n.hasAttribute && n.hasAttribute('inert')) return true;
       }
     } catch {
       return false;
@@ -106,27 +120,27 @@ function runInPage(ctx) {
     const role = tokens[0];
     const isKnown = tokens.some((t) => ariaHelpers.isKnownRole(t));
     const reasonCode = !isKnown ? 'ARIA_ROLE_INVALID' : 'ARIA_ROLE_ABSTRACT';
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
-    occurrences.push({
-      selector: stableSelector,
-      html,
-      summary: !isKnown
-        ? 'The role attribute value is not a recognized ARIA role.'
-        : 'The role attribute value is an abstract ARIA role, which must not be used directly.',
-      hint: !isKnown
-        ? 'Use a valid ARIA role token, or remove the role attribute if none applies.'
-        : 'Replace this abstract role with a concrete role appropriate for the widget/structure.',
-      i18n: {
-        summaryKey: !isKnown ? 'ariaRolesValid_summary_invalid' : 'ariaRolesValid_summary_abstract',
-        hintKey: !isKnown ? 'ariaRolesValid_hint_invalid' : 'ariaRolesValid_hint_abstract',
-        params: { role }
-      },
-      data: {
-        details: { reasonCode, role }
-      }
-    });
+
+    occurrences.push(
+      helpers.reportOccurrence(el, {
+        summary: !isKnown
+          ? 'The role attribute value is not a recognized ARIA role.'
+          : 'The role attribute value is an abstract ARIA role, which must not be used directly.',
+        hint: !isKnown
+          ? 'Use a valid ARIA role token, or remove the role attribute if none applies.'
+          : 'Replace this abstract role with a concrete role appropriate for the widget/structure.',
+        i18n: {
+          summaryKey: !isKnown
+            ? 'ariaRolesValid_summary_invalid'
+            : 'ariaRolesValid_summary_abstract',
+          hintKey: !isKnown ? 'ariaRolesValid_hint_invalid' : 'ariaRolesValid_hint_abstract',
+          params: { role }
+        },
+        data: {
+          details: { reasonCode, role }
+        }
+      })
+    );
   }
 
   if (applicableCount === 0) {

package/src/checks/automatic/aria-valid-attr-value.js

@@ -105,29 +105,26 @@ function runInPage(ctx) {
 
     if (!invalid || !invalid.length) continue;
 
-    const stableSelector = helpers.buildSelector ? helpers.buildSelector(el) : 'html';
-    const html = helpers.getOuterHtmlSnippet ? helpers.getOuterHtmlSnippet(el) : el.outerHTML || '';
-
     for (const item of invalid) {
-      occurrences.push({
-        selector: stableSelector,
-        html,
-        summary: 'This element has an ARIA attribute with an invalid value.',
-        hint: 'Use a value that matches the attribute’s expected type (see the WAI-ARIA specification for this attribute).',
-        i18n: {
-          summaryKey: 'ariaValidAttrValue_summary_fail',
-          hintKey: 'ariaValidAttrValue_hint_fail',
-          params: { attr: item.name, value: item.value }
-        },
-        data: {
-          details: {
-            reasonCode: 'ARIA_ATTR_VALUE_INVALID',
-            attr: item.name,
-            value: item.value,
-            valueReason: item.reason
+      occurrences.push(
+        helpers.reportOccurrence(el, {
+          summary: 'This element has an ARIA attribute with an invalid value.',
+          hint: 'Use a value that matches the attribute’s expected type (see the WAI-ARIA specification for this attribute).',
+          i18n: {
+            summaryKey: 'ariaValidAttrValue_summary_fail',
+            hintKey: 'ariaValidAttrValue_hint_fail',
+            params: { attr: item.name, value: item.value }
+          },
+          data: {
+            details: {
+              reasonCode: 'ARIA_ATTR_VALUE_INVALID',
+              attr: item.name,
+              value: item.value,
+              valueReason: item.reason
+            }
           }
-        }
-      });
+        })
+      );
     }
   }