@diegovelasquezweb/a11y-engine 0.11.3 → 0.11.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegovelasquezweb/a11y-engine",
-  "version": "0.11.3",
+  "version": "0.11.5",
   "description": "WCAG 2.2 accessibility audit engine — scanner, analyzer, and report builders",
   "type": "module",
   "license": "MIT",

package/src/enrichment/analyzer.mjs

@@ -888,7 +888,10 @@ function buildFindings(inputPayload, cliArgs) {
           primary_selector: bestSelector,
           actual: (() => {
             const raw = firstNode?.failureSummary || `Found ${nodes.length} instance(s).`;
-            return raw.replace(/^Fix any of the following:\s*/i, "").trim();
+            return raw
+              .replace(/^Fix (any|all) of the following:\s*/i, "")
+              .replace(/^(See|Reference):\s*https?:\/\/\S+\s*/i, "")
+              .trim();
           })(),
           primary_failure_mode: failureInsights.primaryFailureMode,
           relationship_hint: failureInsights.relationshipHint,

package/src/pipeline/dom-scanner.mjs

@@ -1003,10 +1003,26 @@ async function runPa11yChecks(routeUrl, axeTags, sharedBrowser = null, includeWa
 
     const results = await pa11y(routeUrl, pa11yOptions);
 
+    // Group issues by ruleId so each rule produces one violation (axe-style)
+    const groupedByRule = new Map();
+
+    /**
+     * Cleans raw pa11y messages — strips URL references, "Fix all/any of the following:" prefixes,
+     * and other boilerplate that produces ugly titles in reports.
+     */
+    function cleanPa11yMessage(msg) {
+      if (!msg) return "Accessibility issue";
+      return msg
+        .replace(/^(See|Reference):\s*https?:\/\/\S+\s*/i, "")
+        .replace(/^Fix (all|any) of the following:\s*/i, "")
+        .trim();
+    }
+
     for (const issue of results.issues || []) {
       if (issue.type === "notice") continue;
 
       const impact = impactMap[issue.typeCode] || "moderate";
+      const isWarning = issue.type === "warning";
 
       let wcagCriterion = "";
       const wcagMatch = issue.code?.match(/Guideline(\d+)_(\d+)\.(\d+)_(\d+)_(\d+)/);
@@ -1026,35 +1042,47 @@ async function runPa11yChecks(routeUrl, axeTags, sharedBrowser = null, includeWa
 
       const ruleId = axeEquivId || `pa11y-${((issue.code || "unknown").split(".").pop() || "unknown").toLowerCase()}`;
       const originalCode = issue.code || "unknown";
-
-      violations.push({
-        id: ruleId,
-        impact,
-        tags: ["pa11y-check", ...(wcagCriterion ? [`wcag${wcagCriterion.replace(/\./g, "")}`] : [])],
-        description: issue.message || "pa11y detected an accessibility issue",
-        help: issue.message?.split(".")[0] || "Accessibility issue detected by HTML CodeSniffer",
-        helpUrl: wcagCriterion
-          ? `https://www.w3.org/WAI/WCAG21/Understanding/${wcagCriterion.replace(/\./g, "")}`
-          : "https://squizlabs.github.io/HTML_CodeSniffer/",
-        source: "pa11y",
-        source_rule_id: originalCode,
-        nodes: [{
-          any: [],
-          all: [{
-            id: "pa11y-check",
-            data: { code: originalCode, context: issue.context?.slice(0, 200) },
-            relatedNodes: [],
-            impact,
-            message: issue.message || "",
-          }],
-          none: [],
+      const cleanMessage = cleanPa11yMessage(issue.message);
+
+      const node = {
+        any: [],
+        all: [{
+          id: "pa11y-check",
+          data: { code: originalCode, context: issue.context?.slice(0, 200) },
+          relatedNodes: [],
           impact,
-          html: issue.context || "",
-          target: issue.selector ? [issue.selector] : [],
-          failureSummary: `Fix all of the following:\n  ${issue.message || "Accessibility issue"}`,
+          message: cleanMessage,
         }],
-      });
+        none: [],
+        impact,
+        html: issue.context || "",
+        target: issue.selector ? [issue.selector] : [],
+        failureSummary: cleanMessage,
+      };
+
+      if (groupedByRule.has(ruleId)) {
+        // Add node to existing violation
+        groupedByRule.get(ruleId).nodes.push(node);
+      } else {
+        // Create new violation for this rule
+        groupedByRule.set(ruleId, {
+          id: ruleId,
+          impact,
+          tags: ["pa11y-check", ...(wcagCriterion ? [`wcag${wcagCriterion.replace(/\./g, "")}`] : [])],
+          description: cleanMessage,
+          help: cleanMessage.split(".")[0] || "Accessibility issue detected by HTML CodeSniffer",
+          helpUrl: wcagCriterion
+            ? `https://www.w3.org/WAI/WCAG21/Understanding/${wcagCriterion.replace(/\./g, "")}`
+            : "https://squizlabs.github.io/HTML_CodeSniffer/",
+          source: "pa11y",
+          source_rule_id: originalCode,
+          needs_verification: isWarning,
+          nodes: [node],
+        });
+      }
     }
+
+    violations.push(...groupedByRule.values());
   } catch (err) {
     log.warn(`pa11y checks failed (non-fatal): ${err.message}`);
   }