@diegovelasquezweb/a11y-engine 0.11.41 → 0.11.43

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diegovelasquezweb/a11y-engine",
-  "version": "0.11.41",
+  "version": "0.11.43",
   "description": "WCAG 2.2 accessibility audit engine — scanner, analyzer, and report builders",
   "type": "module",
   "license": "MIT",
@@ -32,9 +32,6 @@
     "CHANGELOG.md",
     "LICENSE"
   ],
-  "scripts": {
-    "test": "vitest run"
-  },
   "dependencies": {
     "@axe-core/playwright": "^4.11.1",
     "axe-core": "^4.11.1",
@@ -44,5 +41,7 @@
   "devDependencies": {
     "vitest": "^4.0.18"
   },
-  "packageManager": "pnpm@10.22.0+sha512.bf049efe995b28f527fd2b41ae0474ce29186f7edcb3bf545087bd61fbbebb2bf75362d1307fda09c2d288e1e499787ac12d4fcb617a974718a6051f2eee741c"
-}
+  "scripts": {
+    "test": "vitest run"
+  }
+}

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

@@ -157,14 +157,40 @@ function buildPatternAiInput({ finding, candidate, projectHints }) {
   const originalLine = typeof finding.line === "number" ? finding.line : null;
   const rawMatch = typeof finding.match === "string" ? finding.match.trim() : "";
 
+  // rawMatch is often just the regex prefix (e.g. 'placeholder="') and is not unique
+  // when multiple elements match the same pattern in the same file.
+  // Use finding.context to build a more specific anchor so sequential fixes on the
+  // same file don't end up targeting an already-patched sibling element.
+  const contextLines = typeof finding.context === "string"
+    ? finding.context.split("\n").map((l) => l.trim()).filter(Boolean)
+    : [];
+  const matchPrefix = rawMatch.slice(0, 30);
+  // 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
+    ? (fileLines[effectiveLineIndex] || "").trim()
+    : "";
+  // Prefer a context line that exactly matches the current file content at originalLine.
+  // This disambiguates siblings (e.g. two inputs with placeholder=) when the file has
+  // not yet been modified. Falls back to the longest context line for post-shift cases
+  // where the original position now holds different content (lines were inserted before it).
+  const exactContextMatch = matchPrefix
+    ? contextLines.find((l) => l === currentAtOriginal && l.includes(matchPrefix)) || null
+    : null;
+  const longestContextMatch = contextLines.reduce((best, line) => {
+    if (!matchPrefix || !line.includes(matchPrefix)) return best;
+    return !best || line.length > best.length ? line : best;
+  }, null);
+  const anchor = exactContextMatch || longestContextMatch || matchPrefix;
+
   // The file may have been modified by a previous sequential fix (lines shifted).
-  // Search for the actual current line containing the match string instead of
+  // 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 (rawMatch && effectiveLineIndex >= 0) {
+  if (anchor && effectiveLineIndex >= 0) {
     const lineAtOriginal = (fileLines[effectiveLineIndex] || "").trim();
-    if (!lineAtOriginal.includes(rawMatch.slice(0, 30))) {
-      const found = fileLines.findIndex((l) => l.trim().includes(rawMatch.slice(0, 30)));
+    if (!lineAtOriginal.includes(anchor)) {
+      const found = fileLines.findIndex((l) => l.trim().includes(anchor));
       if (found !== -1) effectiveLineIndex = found;
     }
   }