@diegovelasquezweb/a11y-engine 0.11.43 → 0.11.45

package/package.json

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

package/src/fixes/apply-finding-fix.mjs

@@ -165,6 +165,10 @@ function buildPatternAiInput({ finding, candidate, projectHints }) {
     ? finding.context.split("\n").map((l) => l.trim()).filter(Boolean)
     : [];
   const matchPrefix = rawMatch.slice(0, 30);
+
+  // Declare effectiveLineIndex first so it can be used when computing currentAtOriginal.
+  let effectiveLineIndex = originalLine !== null ? originalLine - 1 : -1;
+
   // The current content at the original line position — use this to pin the anchor
   // to THIS specific element rather than a sibling with a similar match prefix.
   const currentAtOriginal = effectiveLineIndex >= 0
@@ -186,7 +190,6 @@ function buildPatternAiInput({ finding, candidate, projectHints }) {
   // The file may have been modified by a previous sequential fix (lines shifted).
   // Search for the actual current line containing the anchor instead of
   // relying solely on the original line number from the scan.
-  let effectiveLineIndex = originalLine !== null ? originalLine - 1 : -1;
   if (anchor && effectiveLineIndex >= 0) {
     const lineAtOriginal = (fileLines[effectiveLineIndex] || "").trim();
     if (!lineAtOriginal.includes(anchor)) {
@@ -668,6 +671,39 @@ export async function applyFindingFix(input) {
 
     const validation = validateAiPatchOutput(patchOutput, projectDir, candidateSet);
     if (!validation.ok) {
+      // When Claude returns no changes, it may be because a prior fix (e.g. the DOM
+      // batch) already resolved this issue. Verify by checking if the pattern's
+      // context_reject_regex now matches the surroundingLines of the target element.
+      // If it does, the element is already accessible — count as resolved.
+      if (validation.reason === "AI patch output has no changes") {
+        const patternId = finding.pattern_id || finding.patternId || "";
+        const patternDef = (ASSETS.remediation.codePatterns?.patterns || [])
+          .find((p) => p.id === patternId);
+        const rejectRegex = patternDef?.context_reject_regex;
+        if (rejectRegex) {
+          const context = [aiInput.finding.surroundingLines, aiInput.finding.matchLine]
+            .filter(Boolean)
+            .join("\n");
+          try {
+            if (new RegExp(rejectRegex, "i").test(context)) {
+              return buildResult({
+                applied: true,
+                reason: "",
+                message: "Already resolved by a prior fix.",
+                changedFiles: [],
+                patch: "",
+                verifyRule: "",
+                verifyRoute: "/",
+                findingTitle: finding.title || "",
+                branchSlug: slugify(`${findingId}-${patternId}`),
+                usage: claudeUsage,
+              });
+            }
+          } catch {
+            // Invalid regex in pattern definition — fall through to the error return
+          }
+        }
+      }
       return buildResult({
         applied: false,
         reason: FIX_ERROR_CODES.PATCH_GENERATION_FAILED,