sela-core 1.0.5 → 1.0.7

package/dist/services/HealReportService.js

@@ -96,6 +96,52 @@ class HealReportService {
     getProtectedEvents() {
         return this.events.filter((e) => e.kind === "PROTECTED");
     }
+    /** All FAILED events accumulated in this session. */
+    getFailedEvents() {
+        return this.events.filter((e) => e.kind === "FAILED");
+    }
+    /**
+     * Inject a Playwright failure screenshot (Base64 PNG, no data: prefix)
+     * onto the FAILED event(s) matching the supplied identity hints. Used
+     * by `SelaReporter` to enrich engine-emitted FAILED events with the
+     * Playwright-captured screenshot from `TestResult.attachments`.
+     *
+     * Match policy (all supplied hints must agree):
+     *   • `sourceFile` — suffix-matched both directions to absorb
+     *     project-relative vs absolute path mismatches between the engine
+     *     (relative) and Playwright TestInfo.location.file (absolute).
+     *   • `testTitle` — exact match.
+     *   • `stableId`  — exact match.
+     *
+     * Returns the number of events updated. Zero means the caller should
+     * keep the screenshot pending until the matching FailedEvent is
+     * recorded (worker-crash edge case where the heal pipeline did not
+     * run before the Reporter received the failure).
+     */
+    attachFailureScreenshot(match, base64Png) {
+        if (!base64Png)
+            return 0;
+        const norm = (p) => (p ?? "").replace(/\\/g, "/").toLowerCase();
+        const targetFile = norm(match.sourceFile);
+        let touched = 0;
+        for (const ev of this.events) {
+            if (ev.kind !== "FAILED")
+                continue;
+            const evFile = norm(ev.sourceFile);
+            const fileMatch = !targetFile ||
+                !evFile ||
+                evFile === targetFile ||
+                evFile.endsWith(targetFile) ||
+                targetFile.endsWith(evFile);
+            const titleMatch = !match.testTitle || ev.testTitle === match.testTitle;
+            const idMatch = !match.stableId || ev.stableId === match.stableId;
+            if (fileMatch && titleMatch && idMatch) {
+                ev.failureScreenshotPng = base64Png;
+                touched++;
+            }
+        }
+        return touched;
+    }
     /**
      * Write `sela-report.html` (+ `sela-report.json` debug dump) to cwd.
      * Returns the absolute path of the HTML report, or null if nothing to write.
@@ -684,6 +730,23 @@ a:hover { text-decoration: underline; }
 .tag.ghost   { color: var(--warn);    border-color: rgba(251,191,36,0.3); }
 .tag.occlude { color: var(--bad);     border-color: rgba(248,113,113,0.3); }
 
+/* Failure context — Playwright screenshot rendered inline on FAILED cards */
+.failure-img {
+  background: var(--bg-1); border: 1px solid var(--line);
+  border-radius: var(--radius-sm); padding: 8px;
+  overflow: hidden;
+}
+.failure-img img {
+  display: block; max-width: 100%; height: auto;
+  border-radius: 4px;
+  background: #000;
+  outline: 1px solid var(--line);
+}
+.failure-img .img-meta {
+  font-family: var(--mono); font-size: 11px; color: var(--text-muted);
+  padding: 6px 4px 0;
+}
+
 /* Empty state */
 .empty {
   margin-top: 32px; padding: 60px 20px; text-align: center;
@@ -752,6 +815,13 @@ const REPORT_JS = String.raw `
     try { return new Date(iso).toLocaleString(); } catch { return iso; }
   }
   function pluralize(n, one, many) { return n === 1 ? one : (many || one + "s"); }
+  // Graceful percentage formatter — mirrors the server-side helper in
+  // PRAutomationService. Anything that is not a finite number renders "n/a"
+  // instead of being coerced to 0%. Used in every spot that previously
+  // hard-coded a plain "x + %" concatenation.
+  function fmtPct(v) {
+    return (typeof v === "number" && isFinite(v)) ? (v + "%") : "n/a";
+  }
 
   // ── Hero meta + KPIs ──────────────────────────────────────────
   $("#meta-generated").textContent = "Generated " + fmtTime(DATA.generatedAt);
@@ -905,15 +975,16 @@ const REPORT_JS = String.raw `
   function renderAuditor(a) {
     if (!a) return '<div class="muted-block">The Intent Auditor was not consulted for this event (no AI fix attempted, or auditor unavailable).</div>';
     const verdictCls = a.verdict.toLowerCase();
-    const pct = Math.max(0, Math.min(100, a.confidence));
+    const hasPct = typeof a.confidence === "number" && isFinite(a.confidence);
+    const pct = hasPct ? Math.max(0, Math.min(100, a.confidence)) : 0;
     const radius = 56;
     const circ = 2 * Math.PI * radius;
-    const dash = (pct / 100) * circ;
+    const dash = hasPct ? (pct / 100) * circ : 0;
     const adjustments = [];
     if (a.penalty != null && a.penalty !== 0)  adjustments.push("ancestry drift " + a.penalty);
     if (a.bonus   != null && a.bonus   !== 0)  adjustments.push("anchor +" + a.bonus);
     const adjLine = adjustments.length
-      ? '<div class="audit-row"><span class="label">Score adjustments</span><span class="value">raw ' + a.rawConfidence + '% → ' + pct + '% (' + adjustments.join(", ") + ')</span></div>'
+      ? '<div class="audit-row"><span class="label">Score adjustments</span><span class="value">raw ' + fmtPct(a.rawConfidence) + ' → ' + fmtPct(hasPct ? pct : undefined) + ' (' + adjustments.join(", ") + ')</span></div>'
       : "";
     return (
       '<div class="auditor-grid">' +
@@ -924,7 +995,7 @@ const REPORT_JS = String.raw `
               'stroke-dasharray="' + dash.toFixed(2) + ' ' + circ.toFixed(2) + '"/>' +
           '</svg>' +
           '<div class="center">' +
-            '<div class="pct">' + pct + '%</div>' +
+            '<div class="pct">' + (hasPct ? pct + "%" : "n/a") + '</div>' +
             '<div class="pct-label">Consistency</div>' +
           '</div>' +
         '</div>' +
@@ -938,6 +1009,58 @@ const REPORT_JS = String.raw `
     );
   }
 
+  // Parse "git diff --unified=N" output into structured hunks. Skips file
+  // headers ("diff ", "index ", "--- ", "+++ ", rename/similarity lines) and
+  // walks each "@@ -a,b +c,d @@" block, tracking real old/new line numbers
+  // so the rendered diff shows correct line refs even when the AST mutation
+  // landed several lines above the failure site (JumpToDef scenario).
+  function parseUnifiedDiff(raw) {
+    const out = [];
+    if (!raw) return out;
+    const rows = raw.split(/\r?\n/);
+    let cur = null;
+    let oldLn = 0, newLn = 0;
+    for (const r of rows) {
+      if (
+        r.startsWith("diff ") ||
+        r.startsWith("index ") ||
+        r.startsWith("--- ") ||
+        r.startsWith("+++ ") ||
+        r.startsWith("new file mode") ||
+        r.startsWith("deleted file mode") ||
+        r.startsWith("similarity ") ||
+        r.startsWith("rename ") ||
+        r.startsWith("Binary files")
+      ) continue;
+      const hunk = r.match(/^@@ -(\d+)(?:,(\d+))? \+(\d+)(?:,(\d+))? @@/);
+      if (hunk) {
+        if (cur) out.push(cur);
+        cur = { header: r, lines: [] };
+        oldLn = parseInt(hunk[1], 10);
+        newLn = parseInt(hunk[3], 10);
+        continue;
+      }
+      if (!cur) continue;
+      // "" + truly empty trailing lines: ignore
+      if (r.startsWith("\\")) continue;
+      if (r.length === 0) continue;
+      const head = r.charAt(0);
+      const text = r.slice(1);
+      if (head === "+") {
+        cur.lines.push({ type: "add", text, ln: newLn });
+        newLn++;
+      } else if (head === "-") {
+        cur.lines.push({ type: "del", text, ln: oldLn });
+        oldLn++;
+      } else if (head === " ") {
+        cur.lines.push({ type: "ctx", text, ln: newLn });
+        oldLn++; newLn++;
+      }
+    }
+    if (cur) out.push(cur);
+    return out;
+  }
+
   function renderDiff(ev) {
     if (ev.kind !== "HEALED") {
       if (ev.kind === "PROTECTED") {
@@ -947,8 +1070,45 @@ const REPORT_JS = String.raw `
       }
       return '<div class="muted-block">No source code was modified — the AI could not propose a confident fix for this selector.</div>';
     }
-    const oldLine = ev.oldCodeLine || "";
-    const newLine = ev.newCodeLine || "";
+
+    const raw = ev.gitUnifiedDiff || "";
+    const hunks = parseUnifiedDiff(raw);
+
+    let bodyHtml;
+    let sourceTag;
+    if (hunks.length > 0) {
+      sourceTag = "native git diff";
+      bodyHtml = hunks.map(h => {
+        const hdr =
+          '<div class="diff-line ctx">' +
+            '<span class="gutter">@</span><span class="ln"></span><span class="code">' + esc(h.header) + '</span>' +
+          '</div>';
+        const body = h.lines.map(L => {
+          const gutter = L.type === "add" ? "+" : L.type === "del" ? "-" : " ";
+          return '<div class="diff-line ' + L.type + '">' +
+                   '<span class="gutter">' + gutter + '</span>' +
+                   '<span class="ln">' + L.ln + '</span>' +
+                   '<span class="code">' + esc(L.text) + '</span>' +
+                 '</div>';
+        }).join("");
+        return hdr + body;
+      }).join("");
+    } else {
+      // Fallback when git diff capture was unavailable (no git, untracked
+      // file, etc.). Render the single before/after pair the engine carried
+      // over from the legacy code path.
+      sourceTag = "fallback (no git diff captured)";
+      const oldLine = ev.oldCodeLine || "";
+      const newLine = ev.newCodeLine || "";
+      bodyHtml =
+        '<div class="diff-line del">' +
+          '<span class="gutter">-</span><span class="ln">' + (ev.newLineNumber || "") + '</span><span class="code">' + esc(oldLine) + '</span>' +
+        '</div>' +
+        '<div class="diff-line add">' +
+          '<span class="gutter">+</span><span class="ln">' + (ev.newLineNumber || "") + '</span><span class="code">' + esc(newLine) + '</span>' +
+        '</div>';
+    }
+
     return (
       '<div class="diff">' +
         '<div class="diff-head">' +
@@ -957,14 +1117,21 @@ const REPORT_JS = String.raw `
           '<span>line ' + ev.newLineNumber + '</span>' +
           (ev.diffStrategy ? '<span class="muted">·</span><span>strategy: ' + esc(ev.diffStrategy) + '</span>' : "") +
           (ev.blastRadius != null ? '<span class="muted">·</span><span>blast radius: ' + ev.blastRadius + '</span>' : "") +
+          '<span class="muted">·</span><span>' + sourceTag + '</span>' +
         '</div>' +
-        '<div class="diff-body">' +
-          '<div class="diff-line del">' +
-            '<span class="gutter">-</span><span class="ln">' + ev.newLineNumber + '</span><span class="code">' + esc(oldLine) + '</span>' +
-          '</div>' +
-          '<div class="diff-line add">' +
-            '<span class="gutter">+</span><span class="ln">' + ev.newLineNumber + '</span><span class="code">' + esc(newLine) + '</span>' +
-          '</div>' +
+        '<div class="diff-body">' + bodyHtml + '</div>' +
+      '</div>'
+    );
+  }
+
+  function renderFailureContext(ev) {
+    if (ev.kind !== "FAILED" || !ev.failureScreenshotPng) return "";
+    return (
+      '<div class="section">' +
+        '<div class="section-title"><span class="marker"></span>Failure Context · Playwright Screenshot</div>' +
+        '<div class="failure-img">' +
+          '<img src="data:image/png;base64,' + ev.failureScreenshotPng + '" alt="Playwright failure screenshot" loading="lazy" />' +
+          '<div class="img-meta">captured by SelaReporter at test failure · base64-embedded · survives Playwright temp cleanup</div>' +
         '</div>' +
       '</div>'
     );
@@ -1059,7 +1226,10 @@ const REPORT_JS = String.raw `
       '</div>'
     );
 
-    // 5. Alternatives
+    // 5. Failure Context — Playwright screenshot (FAILED only)
+    blocks.push(renderFailureContext(ev));
+
+    // 6. Alternatives
     blocks.push(renderAlts(ev));
 
     // 6. Actions

package/dist/services/HealingCacheService.d.ts

@@ -0,0 +1,135 @@
+import type { SmartChainSegment, SelectorSegment } from "./LLMService";
+export type HealCacheMode = "action" | "assertion";
+export interface CacheKeyInput {
+    /** Absolute or project-relative spec path containing the failing locator. */
+    filePath: string;
+    /** 1-based line of the failing call site — paired with filePath as a fingerprint. */
+    line: number;
+    /**
+     * Stable content hash of the DNA snapshot used to drive the heal. When the
+     * DNA rewrites, this changes — every cached entry for the old DNA
+     * silently stops matching and ages out via TTL.
+     */
+    dnaContentHash: string;
+    /** The original broken selector. */
+    brokenSelector: string;
+    /** Heal mode the cached fix was produced under. */
+    mode: HealCacheMode;
+}
+export interface HealingCacheEntry {
+    /** Full SHA1 of the CacheKeyInput tuple. */
+    key: string;
+    filePath: string;
+    line: number;
+    dnaContentHash: string;
+    brokenSelector: string;
+    mode: HealCacheMode;
+    healedSelector: string;
+    chainSegments?: SmartChainSegment[];
+    segments?: SelectorSegment[];
+    confidence?: number;
+    contentChange?: {
+        oldText: string;
+        newText: string;
+    };
+    explanation?: string;
+    alternatives?: string[];
+    /**
+     * True when SafetyGuard cleared this fix in the originating session
+     * (SAFE or REQUIRES_REVIEW). Unsafe entries are never replayed.
+     */
+    safe: boolean;
+    createdAt: string;
+    /** ISO timestamp after which the entry is considered expired and ignored. */
+    ttlExpiresAt: string;
+    lastHitAt?: string;
+    hitCount: number;
+}
+interface CacheFile {
+    schemaVersion: 1;
+    entries: Record<string, HealingCacheEntry>;
+}
+export interface HealingCacheSetInput extends CacheKeyInput {
+    healedSelector: string;
+    chainSegments?: SmartChainSegment[];
+    segments?: SelectorSegment[];
+    confidence?: number;
+    contentChange?: {
+        oldText: string;
+        newText: string;
+    };
+    explanation?: string;
+    alternatives?: string[];
+    safe: boolean;
+    /** Optional override for entry-level TTL (ms). Defaults to env / 7d. */
+    ttlMs?: number;
+}
+export declare function computeCacheKey(input: CacheKeyInput): string;
+/**
+ * Convenience helper — fingerprint the JSON form of a DNA snapshot so
+ * the cache key changes whenever the DNA is rewritten.
+ */
+export declare function hashDnaContent(dna: unknown): string;
+export declare class HealingCacheService {
+    private readonly cacheDir;
+    private readonly filename;
+    private readonly lockDir;
+    private readonly tmpSuffix;
+    private readonly bakSuffix;
+    private inFlight;
+    constructor(opts?: {
+        cacheDir?: string;
+        filename?: string;
+    });
+    /**
+     * Read-only lookup. Bumps `hitCount` / `lastHitAt` IN-MEMORY only on
+     * the returned entry reference — never persisted on the get() path.
+     * This eliminates the read-write race where parallel workers each
+     * call get() and serialise full entries-map snapshots back to disk,
+     * overwriting peer set() operations.
+     */
+    get(input: CacheKeyInput): HealingCacheEntry | null;
+    /**
+     * Persist a new cache entry. Serialises against in-process writers
+     * AND across processes via the mkdir lock. The in-process chain
+     * isolates each `await` so a rejected upstream Promise never
+     * deadlocks downstream callers.
+     */
+    set(input: HealingCacheSetInput): Promise<void>;
+    /** Synchronous variant — used by teardown / CLI paths. */
+    setSync(input: HealingCacheSetInput): void;
+    /** Wipe every entry. Used by tests and the (forthcoming) `sela cache clear` CLI. */
+    clear(): void;
+    /** Returns the parsed cache file (or an empty shell on failure). */
+    read(): CacheFile;
+    /** Absolute path of the on-disk cache file. */
+    filePath(): string;
+    /**
+     * Prunes expired entries. Cheap O(n) scan — intended for engine
+     * startup. Returns the number of entries removed.
+     */
+    pruneExpired(): number;
+    private isExpired;
+    private lockedWrite;
+    private buildEntry;
+    /**
+     * Reads + parses the cache file. Tolerates every realistic failure
+     * (missing, empty, malformed, wrong schema version) by returning a
+     * fresh empty shell — the contract is "cache misses are silent".
+     */
+    private safeRead;
+    /**
+     * Atomic write: write to `<file>.tmp`, copy current to `<file>.bak`,
+     * then rename `<file>.tmp` → `<file>`. The rename is the single
+     * commit point — a crash between bak and rename leaves the bak file
+     * intact for future recovery.
+     */
+    private atomicWrite;
+    private acquireLockWithBackoff;
+    private tryAcquireLock;
+    private withFileLockSync;
+    private releaseLock;
+}
+export declare const sharedHealingCache: HealingCacheService;
+export {};
+//# sourceMappingURL=HealingCacheService.d.ts.map

package/dist/services/HealingCacheService.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"HealingCacheService.d.ts","sourceRoot":"","sources":["../../src/services/HealingCacheService.ts"],"names":[],"mappings":"AA8DA,OAAO,KAAK,EAAE,iBAAiB,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAMvE,MAAM,MAAM,aAAa,GAAG,QAAQ,GAAG,WAAW,CAAC;AAEnD,MAAM,WAAW,aAAa;IAC5B,6EAA6E;IAC7E,QAAQ,EAAE,MAAM,CAAC;IACjB,qFAAqF;IACrF,IAAI,EAAE,MAAM,CAAC;IACb;;;;OAIG;IACH,cAAc,EAAE,MAAM,CAAC;IACvB,oCAAoC;IACpC,cAAc,EAAE,MAAM,CAAC;IACvB,mDAAmD;IACnD,IAAI,EAAE,aAAa,CAAC;CACrB;AAED,MAAM,WAAW,iBAAiB;IAChC,4CAA4C;IAC5C,GAAG,EAAE,MAAM,CAAC;IACZ,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,cAAc,EAAE,MAAM,CAAC;IACvB,cAAc,EAAE,MAAM,CAAC;IACvB,IAAI,EAAE,aAAa,CAAC;IACpB,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,CAAC,EAAE,iBAAiB,EAAE,CAAC;IACpC,QAAQ,CAAC,EAAE,eAAe,EAAE,CAAC;IAC7B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IACrD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB;;;OAGG;IACH,IAAI,EAAE,OAAO,CAAC;IACd,SAAS,EAAE,MAAM,CAAC;IAClB,6EAA6E;IAC7E,YAAY,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,UAAU,SAAS;IACjB,aAAa,EAAE,CAAC,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,iBAAiB,CAAC,CAAC;CAC5C;AAED,MAAM,WAAW,oBAAqB,SAAQ,aAAa;IACzD,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,CAAC,EAAE,iBAAiB,EAAE,CAAC;IACpC,QAAQ,CAAC,EAAE,eAAe,EAAE,CAAC;IAC7B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IACrD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,IAAI,EAAE,OAAO,CAAC;IACd,wEAAwE;IACxE,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAmBD,wBAAgB,eAAe,CAAC,KAAK,EAAE,aAAa,GAAG,MAAM,CAc5D;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,GAAG,EAAE,OAAO,GAAG,MAAM,CAQnD;AAMD,qBAAa,mBAAmB;IAC9B,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAS;IAClC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAS;IAClC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAS;IACjC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAU;IACpC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAU;IAIpC,OAAO,CAAC,QAAQ,CAAoC;gBAExC,IAAI,CAAC,EAAE;QAAE,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;KAAE;IAU3D;;;;;;OAMG;IACH,GAAG,CAAC,KAAK,EAAE,aAAa,GAAG,iBAAiB,GAAG,IAAI;IAgBnD;;;;;OAKG;IACG,GAAG,CAAC,KAAK,EAAE,oBAAoB,GAAG,OAAO,CAAC,IAAI,CAAC;IAgCrD,0DAA0D;IAC1D,OAAO,CAAC,KAAK,EAAE,oBAAoB,GAAG,IAAI;IAU1C,oFAAoF;IACpF,KAAK,IAAI,IAAI;IAeb,oEAAoE;IACpE,IAAI,IAAI,SAAS;IAIjB,+CAA+C;IAC/C,QAAQ,IAAI,MAAM;IAIlB;;;OAGG;IACH,YAAY,IAAI,MAAM;IAmBtB,OAAO,CAAC,SAAS;YAOH,WAAW;IAkBzB,OAAO,CAAC,UAAU;IA8BlB;;;;OAIG;IACH,OAAO,CAAC,QAAQ;IAqChB;;;;;OAKG;IACH,OAAO,CAAC,WAAW;YA4BL,sBAAsB;IAUpC,OAAO,CAAC,cAAc;IAgBtB,OAAO,CAAC,gBAAgB;IAmBxB,OAAO,CAAC,WAAW;CAOpB;AAqBD,eAAO,MAAM,kBAAkB,qBAA4B,CAAC"}