sela-core 1.0.4 → 1.0.6

package/dist/engine/SelaEngine.d.ts

@@ -14,6 +14,31 @@ export declare class SelaEngine {
     heal(page: Page, fullSelector: string, elementSelectorOnly: string, stableId: string, filePath: string, line: number, healMode?: HealMode): Promise<string>;
     private _toReportFile;
     private _safeReadLine;
+    /**
+     * Capture the native `git diff --unified=1 HEAD -- <file>` for the file the
+     * AST mutation just wrote to. Returns an empty string when git is not
+     * available, the file is not tracked, or any other failure — callers must
+     * treat empty output as "no diff captured" and fall back to a direct disk
+     * read of the mutated line.
+     *
+     * Path safety: the absolute path is always wrapped in double quotes after
+     * escaping any embedded double quotes, which makes the call safe on Windows
+     * paths containing spaces or backslashes as well as POSIX paths.
+     */
+    private _captureGitDiff;
+    /**
+     * Walk a unified diff and pull out the first removed line and the first
+     * added line so the legacy report fields (`oldCodeLine`, `newCodeLine`)
+     * stay populated for callers that haven't migrated to the full
+     * `gitUnifiedDiff` payload yet.
+     *
+     * Carefully skips the diff preamble (--- a/file, +++ b/file, "diff ",
+     * "index ", rename headers) — those lines would otherwise be mistaken for
+     * a -/+ change pair and produce nonsense like the original "line replaced
+     * by itself" output. Real change lines only count once the parser has seen
+     * a "@@" hunk header.
+     */
+    private _extractFirstChangeFromDiff;
     private _auditorBlockFromDecision;
     private _buildAlternatives;
     private _buildReasoningSteps;

package/dist/engine/SelaEngine.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"SelaEngine.d.ts","sourceRoot":"","sources":["../../src/engine/SelaEngine.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAMxC,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAE7C,OAAO,EAAe,QAAQ,EAAE,MAAM,yBAAyB,CAAC;AAgChE,qBAAa,UAAU;IACrB,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,eAAe,CAAkB;IACzC,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,MAAM,CAAiB;IAC/B,OAAO,CAAC,SAAS,CAA6C;IAC9D,OAAO,CAAC,cAAc,CAAoC;IAC1D,OAAO,CAAC,eAAe,CAAkC;;IAanD,YAAY,CAChB,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,MAAM,EACd,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IAiDnB,IAAI,CACR,IAAI,EAAE,IAAI,EACV,YAAY,EAAE,MAAM,EACpB,mBAAmB,EAAE,MAAM,EAC3B,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,EACZ,QAAQ,GAAE,QAAmB,GAC5B,OAAO,CAAC,MAAM,CAAC;IAyalB,OAAO,CAAC,aAAa;IAQrB,OAAO,CAAC,aAAa;IAarB,OAAO,CAAC,yBAAyB;IAiBjC,OAAO,CAAC,kBAAkB;IA8B1B,OAAO,CAAC,oBAAoB;IAuC5B,gBAAgB,CAAC,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,IAAI;IAIrD,YAAY,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC;IAItE,aAAa,IAAI,OAAO,CAAC,IAAI,CAAC;YA4FtB,aAAa;IAsB3B,OAAO,CAAC,0BAA0B;IAiClC,OAAO,CAAC,WAAW;IA6Bb,wBAAwB,CAC5B,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM;CAqBnB;AAED,OAAO,EAAE,CAAC"}
+{"version":3,"file":"SelaEngine.d.ts","sourceRoot":"","sources":["../../src/engine/SelaEngine.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,kBAAkB,CAAC;AAOxC,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAE7C,OAAO,EAAe,QAAQ,EAAE,MAAM,yBAAyB,CAAC;AAyChE,qBAAa,UAAU;IACrB,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,eAAe,CAAkB;IACzC,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,MAAM,CAAiB;IAC/B,OAAO,CAAC,SAAS,CAA6C;IAC9D,OAAO,CAAC,cAAc,CAAoC;IAC1D,OAAO,CAAC,eAAe,CAAkC;;IAanD,YAAY,CAChB,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,MAAM,EACd,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IAiDnB,IAAI,CACR,IAAI,EAAE,IAAI,EACV,YAAY,EAAE,MAAM,EACpB,mBAAmB,EAAE,MAAM,EAC3B,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,EACZ,QAAQ,GAAE,QAAmB,GAC5B,OAAO,CAAC,MAAM,CAAC;IAyclB,OAAO,CAAC,aAAa;IAQrB,OAAO,CAAC,aAAa;IAarB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,eAAe;IAwBvB;;;;;;;;;;;OAWG;IACH,OAAO,CAAC,2BAA2B;IA4BnC,OAAO,CAAC,yBAAyB;IAwBjC,OAAO,CAAC,kBAAkB;IA8B1B,OAAO,CAAC,oBAAoB;IAuC5B,gBAAgB,CAAC,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,IAAI;IAIrD,YAAY,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,iBAAiB,GAAG,OAAO,CAAC,IAAI,CAAC;IAItE,aAAa,IAAI,OAAO,CAAC,IAAI,CAAC;YA6FtB,aAAa;IAsB3B,OAAO,CAAC,0BAA0B;IAiClC,OAAO,CAAC,WAAW;IA6Bb,wBAAwB,CAC5B,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM;CAqBnB;AAED,OAAO,EAAE,CAAC"}

package/dist/engine/SelaEngine.js

@@ -36,6 +36,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SelaEngine = void 0;
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
+const child_process_1 = require("child_process");
 const LLMService_1 = require("../services/LLMService");
 const SnapshotService_1 = require("../services/SnapshotService");
 const DOMUtils_1 = require("../utils/DOMUtils");
@@ -44,6 +45,14 @@ const SafetyGuard_1 = require("../services/SafetyGuard");
 const ConfigLoader_1 = require("../config/ConfigLoader");
 const HealReportService_1 = require("../services/HealReportService");
 const PRAutomationService_1 = require("../services/PRAutomationService");
+/**
+ * Graceful percentage formatter for engine-side log lines. Mirrors the helper
+ * in PRAutomationService / HealReportService so a missing AI/auditor
+ * confidence reads as "n/a" instead of "undefined%" or "0%".
+ */
+function fmtPctEngine(v) {
+    return typeof v === "number" && Number.isFinite(v) ? `${v}%` : "n/a";
+}
 class SelaEngine {
     llmService;
     snapshotService;
@@ -281,17 +290,42 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
                         parts.push(`${breakdown.penalty} ancestry drift`);
                     if (breakdown.bonus !== 0)
                         parts.push(`+${breakdown.bonus} anchor match`);
-                    console.log(`[Sela] 📊 Score — AI: ${aiFix.confidence}% | ` +
-                        `Auditor: ${breakdown.adjustedConfidence}% ` +
-                        `(raw ${breakdown.rawConfidence}%, ${parts.join(", ")})`);
+                    console.log(`[Sela] 📊 Score — AI: ${fmtPctEngine(aiFix.confidence)} | ` +
+                        `Auditor: ${fmtPctEngine(breakdown.adjustedConfidence)} ` +
+                        `(raw ${fmtPctEngine(breakdown.rawConfidence)}, ${parts.join(", ")})`);
                 }
                 console.log(`[Sela] 🚀 Rebuilt Full Runtime Selector (Clean): ${newFullSelector}`);
-                // Capture old source line BEFORE the AST mutation — needed for the diff.
-                const oldCodeLine = this._safeReadLine(filePath, line);
                 const updateResult = SourceUpdater_1.SourceUpdater.update({ filePath, line }, elementSelectorOnly, newFullSelector, neighborhoodDom, undefined, fullSelector, aiFix.segments, aiFix.contentChange, aiFix.chainSegments, aiFix.originalChainHint, this.config.updateStrategy, this.config.autoCommit);
-                // Capture new source line AFTER mutation, at the line the updater landed on.
-                const writtenLine = updateResult.lineUpdated ?? line;
-                const newCodeLine = this._safeReadLine(filePath, writtenLine);
+                // Resolve the line that was actually mutated. Order of precedence:
+                //   1. updateResult.lineUpdated  (the strategy's literal write location)
+                //   2. updateResult.definitionSite.line  (semantic JumpToDef target,
+                //      used when the mutation crossed into a const/variable declaration)
+                //   3. line  (the original failure line — last-resort fallback)
+                // For cross-file definitionSite.file, we still read the new line from
+                // the SAME file that was mutated.
+                const defSite = updateResult.definitionSite;
+                const sameFileDef = defSite && defSite.file
+                    ? path.resolve(defSite.file) ===
+                        (SourceUpdater_1.SourceUpdater.resolveFilePath(filePath) ?? "")
+                    : true;
+                const writtenLine = updateResult.lineUpdated ??
+                    (sameFileDef ? defSite?.line : undefined) ??
+                    line;
+                const mutatedFilePath = !sameFileDef && defSite ? defSite.file : filePath;
+                // Capture the native unified diff straight from git. This runs AFTER
+                // SourceUpdater flushed the file to disk and BEFORE any `git add` —
+                // so `git diff HEAD` reports the exact pending change at the real
+                // mutated line. No in-process line-index math, no off-by-one when
+                // JumpToDef lands above the failure site, and cross-file healing
+                // falls out for free.
+                const gitUnifiedDiff = this._captureGitDiff(mutatedFilePath);
+                const firstChange = this._extractFirstChangeFromDiff(gitUnifiedDiff);
+                // Backward-compat: keep oldCodeLine/newCodeLine populated from the
+                // first -/+ pair inside the diff. When git produced no diff (file
+                // untracked, git unavailable, …) fall back to a direct disk read of
+                // the mutated line — strictly inferior but preserves old reports.
+                const oldCodeLine = firstChange.oldLine ?? this._safeReadLine(mutatedFilePath, writtenLine);
+                const newCodeLine = firstChange.newLine ?? this._safeReadLine(mutatedFilePath, writtenLine);
                 const canonicalSelector = updateResult.healedLocatorString ?? newFullSelector;
                 console.log(`[Sela] ✅ Canonical selector (disk == runtime): "${canonicalSelector}"`);
                 // Buffer CLI-ready metadata — merged into DNA at commitUpdates() time.
@@ -339,6 +373,7 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
                     oldCodeLine,
                     newCodeLine,
                     newLineNumber: writtenLine,
+                    gitUnifiedDiff: gitUnifiedDiff || undefined,
                     aiConfidence: aiFix.confidence,
                     aiExplanation: aiFix.explanation ?? "",
                     aiAlternatives: this._buildAlternatives(aiFix.chainSegments, aiFix.segments),
@@ -435,12 +470,95 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
             return "";
         }
     }
+    /**
+     * Capture the native `git diff --unified=1 HEAD -- <file>` for the file the
+     * AST mutation just wrote to. Returns an empty string when git is not
+     * available, the file is not tracked, or any other failure — callers must
+     * treat empty output as "no diff captured" and fall back to a direct disk
+     * read of the mutated line.
+     *
+     * Path safety: the absolute path is always wrapped in double quotes after
+     * escaping any embedded double quotes, which makes the call safe on Windows
+     * paths containing spaces or backslashes as well as POSIX paths.
+     */
+    _captureGitDiff(filePath) {
+        try {
+            const cleaned = this._toReportFile(filePath);
+            const abs = path.isAbsolute(cleaned)
+                ? cleaned
+                : path.resolve(process.cwd(), cleaned);
+            if (!abs || !fs.existsSync(abs))
+                return "";
+            const quoted = `"${abs.replace(/"/g, '\\"')}"`;
+            const out = (0, child_process_1.execSync)(`git diff --unified=1 HEAD -- ${quoted}`, {
+                cwd: process.cwd(),
+                encoding: "utf-8",
+                stdio: ["pipe", "pipe", "pipe"],
+                windowsHide: true,
+            });
+            return out.toString();
+        }
+        catch (err) {
+            // git missing / not a repo / file untracked vs HEAD → log & fall back.
+            console.debug?.(`[Sela] git diff capture skipped: ${err?.message ?? err}`);
+            return "";
+        }
+    }
+    /**
+     * Walk a unified diff and pull out the first removed line and the first
+     * added line so the legacy report fields (`oldCodeLine`, `newCodeLine`)
+     * stay populated for callers that haven't migrated to the full
+     * `gitUnifiedDiff` payload yet.
+     *
+     * Carefully skips the diff preamble (--- a/file, +++ b/file, "diff ",
+     * "index ", rename headers) — those lines would otherwise be mistaken for
+     * a -/+ change pair and produce nonsense like the original "line replaced
+     * by itself" output. Real change lines only count once the parser has seen
+     * a "@@" hunk header.
+     */
+    _extractFirstChangeFromDiff(diff) {
+        if (!diff)
+            return {};
+        let inHunk = false;
+        let oldLine;
+        let newLine;
+        for (const row of diff.split(/\r?\n/)) {
+            if (!inHunk) {
+                if (row.startsWith("@@"))
+                    inHunk = true;
+                continue;
+            }
+            // Defensive: a fresh @@ header starts a new hunk but does not reset
+            // the values we've already captured.
+            if (row.startsWith("@@"))
+                continue;
+            if (row.startsWith("---") || row.startsWith("+++"))
+                continue;
+            if (row.startsWith("\\"))
+                continue;
+            if (row.startsWith("-") && oldLine === undefined) {
+                oldLine = row.slice(1);
+            }
+            else if (row.startsWith("+") && newLine === undefined) {
+                newLine = row.slice(1);
+            }
+            if (oldLine !== undefined && newLine !== undefined)
+                break;
+        }
+        return { oldLine, newLine };
+    }
     _auditorBlockFromDecision(meta) {
         if (!meta || !meta.auditorVerdict)
             return null;
+        // Keep auditor confidence as `undefined` when SafetyGuard did not produce
+        // one — the renderer fmtPct helper turns it into "n/a" rather than 0%.
+        const conf = typeof meta.auditorConfidence === "number" &&
+            Number.isFinite(meta.auditorConfidence)
+            ? meta.auditorConfidence
+            : undefined;
         return {
             verdict: meta.auditorVerdict,
-            confidence: meta.auditorConfidence ?? 0,
+            confidence: conf,
             reason: meta.auditScoreBreakdown
                 ? `Adjusted ${meta.auditScoreBreakdown.rawConfidence}% → ${meta.auditScoreBreakdown.adjustedConfidence}% (penalty ${meta.auditScoreBreakdown.penalty}, bonus +${meta.auditScoreBreakdown.bonus}).`
                 : "Auditor verdict recorded by SafetyGuard.",
@@ -518,7 +636,7 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
         if (ctx.aiFix.explanation) {
             steps.push(`AI reasoning: ${ctx.aiFix.explanation}`);
         }
-        steps.push(`Confidence: ${ctx.aiFix.confidence}%.`);
+        steps.push(`Confidence: ${fmtPctEngine(ctx.aiFix.confidence)}.`);
         if (ctx.aiFix.contentChange) {
             steps.push(`Detected content drift: "${ctx.aiFix.contentChange.oldText}" → "${ctx.aiFix.contentChange.newText}".`);
         }
@@ -571,7 +689,8 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
                 if (decision.downgradeReason) {
                     console.warn(`[Sela PR] ⚠️  Strategy downgraded: '${decision.configured}' → '${decision.effective}' ` +
                         `(reason: ${decision.downgradeReason}, ` +
-                        `minAI: ${decision.minAiConfidence}%, minAuditor: ${decision.minAuditorConfidence}%)`);
+                        `minAI: ${fmtPctEngine(decision.minAiConfidence)}, ` +
+                        `minAuditor: ${fmtPctEngine(decision.minAuditorConfidence)})`);
                 }
                 else {
                     console.log(`[Sela PR] 🎯 Effective strategy: '${decision.effective}' on branch '${branchInfo.branch ?? "unknown"}'`);

package/dist/services/HealReportService.d.ts

@@ -17,7 +17,11 @@ export interface DnaSummary {
 }
 export interface AuditorBlock {
     verdict: AuditVerdictValue;
-    confidence: number;
+    /**
+     * Adjusted confidence 0-100. `undefined` when the auditor produced no score
+     * (e.g. SafetyGuard bypassed it) — reports MUST render this as "n/a".
+     */
+    confidence?: number;
     reason: string;
     inversionType: AuditVerdict["inversionType"];
     rawConfidence?: number;
@@ -39,10 +43,22 @@ export interface HealedEvent extends HealEventBase {
     kind: "HEALED";
     oldSelector: string;
     newSelector: string;
+    /** First removed line from the native git diff (backward-compat fallback). */
     oldCodeLine: string;
+    /** First added line from the native git diff (backward-compat fallback). */
     newCodeLine: string;
     newLineNumber: number;
-    aiConfidence: number;
+    /**
+     * Raw `git diff --unified=1` output captured immediately after the AST
+     * mutation hit disk and BEFORE any `git add`. Includes headers (---/+++)
+     * and one or more @@ hunks. Renderers strip headers; PR body emits the
+     * hunks verbatim inside a ```diff fence so cross-file healing, multi-line
+     * edits, and template-literal changes are all displayed natively without
+     * any in-process line-index math.
+     */
+    gitUnifiedDiff?: string;
+    /** AI-reported confidence 0-100. `undefined` when the model omitted it. */
+    aiConfidence?: number;
     aiExplanation: string;
     aiAlternatives: string[];
     reasoningSteps: string[];
@@ -75,7 +91,8 @@ export interface ProtectedEvent extends HealEventBase {
     reason: string;
     safetyLevel: string;
     auditor: AuditorBlock | null;
-    aiConfidence: number;
+    /** AI-reported confidence 0-100. `undefined` when the model omitted it. */
+    aiConfidence?: number;
     aiExplanation: string;
     dnaBefore: DnaSummary | null;
     dnaAfter: DnaSummary | null;

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

@@ -1 +1 @@
-{"version":3,"file":"HealReportService.d.ts","sourceRoot":"","sources":["../../src/services/HealReportService.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,YAAY,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACvE,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAMlD,MAAM,MAAM,aAAa,GAAG,QAAQ,GAAG,QAAQ,GAAG,WAAW,CAAC;AAE9D,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,UAAU,EAAE,OAAO,CAAC;IACpB,aAAa,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,iBAAiB,CAAC;IAC3B,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,YAAY,CAAC,eAAe,CAAC,CAAC;IAC7C,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,UAAU,aAAa;IACrB,IAAI,EAAE,aAAa,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,QAAQ,EAAE,OAAO,CAAC;IAClB,WAAW,EAAE,OAAO,CAAC;CACtB;AAED,MAAM,WAAW,WAAY,SAAQ,aAAa;IAChD,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,OAAO,EAAE,YAAY,GAAG,IAAI,CAAC;IAC7B,SAAS,EAAE,UAAU,CAAC;IACtB,QAAQ,EAAE,UAAU,GAAG,IAAI,CAAC;IAC5B,YAAY,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IAChD,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CACtD;AAED,MAAM,WAAW,WAAY,SAAQ,aAAa;IAChD,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,SAAS,EAAE,UAAU,GAAG,IAAI,CAAC;CAC9B;AAED,MAAM,WAAW,cAAe,SAAQ,aAAa;IACnD,IAAI,EAAE,WAAW,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,YAAY,GAAG,IAAI,CAAC;IAC7B,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,UAAU,GAAG,IAAI,CAAC;IAC7B,QAAQ,EAAE,UAAU,GAAG,IAAI,CAAC;IAC5B,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CACtD;AAED,MAAM,MAAM,SAAS,GAAG,WAAW,GAAG,WAAW,GAAG,cAAc,CAAC;AAWnE,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,iBAAiB,GAAG,IAAI,GAAG,SAAS,GAAG,UAAU,GAAG,IAAI,CAgB5F;AAMD,qBAAa,iBAAiB;IAC5B,OAAO,CAAC,MAAM,CAAmB;IACjC,OAAO,CAAC,SAAS,CAAoC;IAErD,MAAM,CAAC,KAAK,EAAE,SAAS,GAAG,IAAI;IAI9B,IAAI,IAAI,MAAM;IAId,KAAK,IAAI,IAAI;IAKb,UAAU,IAAI,OAAO;IAIrB,qDAAqD;IACrD,eAAe,IAAI,WAAW,EAAE;IAIhC,6EAA6E;IAC7E,kBAAkB,IAAI,cAAc,EAAE;IAItC;;;OAGG;IACH,WAAW,CAAC,SAAS,GAAE,MAAsB,GAAG,MAAM,GAAG,IAAI;IAyB7D,OAAO,CAAC,UAAU;IAWlB,OAAO,CAAC,YAAY;CAwBrB;AAMD,eAAO,MAAM,gBAAgB,mBAA0B,CAAC"}
+{"version":3,"file":"HealReportService.d.ts","sourceRoot":"","sources":["../../src/services/HealReportService.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,YAAY,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACvE,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAMlD,MAAM,MAAM,aAAa,GAAG,QAAQ,GAAG,QAAQ,GAAG,WAAW,CAAC;AAE9D,MAAM,WAAW,UAAU;IACzB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,UAAU,EAAE,OAAO,CAAC;IACpB,aAAa,EAAE,OAAO,CAAC;CACxB;AAED,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,iBAAiB,CAAC;IAC3B;;;OAGG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,YAAY,CAAC,eAAe,CAAC,CAAC;IAC7C,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,UAAU,aAAa;IACrB,IAAI,EAAE,aAAa,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,QAAQ,EAAE,OAAO,CAAC;IAClB,WAAW,EAAE,OAAO,CAAC;CACtB;AAED,MAAM,WAAW,WAAY,SAAQ,aAAa;IAChD,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,8EAA8E;IAC9E,WAAW,EAAE,MAAM,CAAC;IACpB,4EAA4E;IAC5E,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB;;;;;;;OAOG;IACH,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,2EAA2E;IAC3E,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,EAAE,MAAM,CAAC;IACtB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,cAAc,EAAE,MAAM,EAAE,CAAC;IACzB,OAAO,EAAE,YAAY,GAAG,IAAI,CAAC;IAC7B,SAAS,EAAE,UAAU,CAAC;IACtB,QAAQ,EAAE,UAAU,GAAG,IAAI,CAAC;IAC5B,YAAY,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IAChD,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CACtD;AAED,MAAM,WAAW,WAAY,SAAQ,aAAa;IAChD,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,SAAS,EAAE,UAAU,GAAG,IAAI,CAAC;CAC9B;AAED,MAAM,WAAW,cAAe,SAAQ,aAAa;IACnD,IAAI,EAAE,WAAW,CAAC;IAClB,WAAW,EAAE,MAAM,CAAC;IACpB,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,OAAO,EAAE,YAAY,GAAG,IAAI,CAAC;IAC7B,2EAA2E;IAC3E,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,UAAU,GAAG,IAAI,CAAC;IAC7B,QAAQ,EAAE,UAAU,GAAG,IAAI,CAAC;IAC5B,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;CACtD;AAED,MAAM,MAAM,SAAS,GAAG,WAAW,GAAG,WAAW,GAAG,cAAc,CAAC;AAWnE,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,iBAAiB,GAAG,IAAI,GAAG,SAAS,GAAG,UAAU,GAAG,IAAI,CAgB5F;AAMD,qBAAa,iBAAiB;IAC5B,OAAO,CAAC,MAAM,CAAmB;IACjC,OAAO,CAAC,SAAS,CAAoC;IAErD,MAAM,CAAC,KAAK,EAAE,SAAS,GAAG,IAAI;IAI9B,IAAI,IAAI,MAAM;IAId,KAAK,IAAI,IAAI;IAKb,UAAU,IAAI,OAAO;IAIrB,qDAAqD;IACrD,eAAe,IAAI,WAAW,EAAE;IAIhC,6EAA6E;IAC7E,kBAAkB,IAAI,cAAc,EAAE;IAItC;;;OAGG;IACH,WAAW,CAAC,SAAS,GAAE,MAAsB,GAAG,MAAM,GAAG,IAAI;IAyB7D,OAAO,CAAC,UAAU;IAWlB,OAAO,CAAC,YAAY;CAwBrB;AAMD,eAAO,MAAM,gBAAgB,mBAA0B,CAAC"}

package/dist/services/HealReportService.js

@@ -752,6 +752,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 +912,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 +932,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 +946,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 +1007,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,15 +1054,9 @@ 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>' +
+        '<div class="diff-body">' + bodyHtml + '</div>' +
       '</div>'
     );
   }

package/dist/services/LLMService.d.ts

@@ -76,7 +76,12 @@ export interface FixResponse {
      */
     chainSegments?: SmartChainSegment[];
     originalChainHint?: SmartChainSegment[];
-    confidence: number;
+    /**
+     * Confidence score 0-100. Undefined when the model omitted it or returned a
+     * non-finite value — downstream code must treat undefined as "unknown" and
+     * render "n/a" rather than synthesising a 0.
+     */
+    confidence?: number;
     explanation: string;
     new_selector?: string | null;
     contentChange?: ContentChange;

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

@@ -1 +1 @@
-{"version":3,"file":"LLMService.d.ts","sourceRoot":"","sources":["../../src/services/LLMService.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAQtC,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1B,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,MAAM,iBAAiB,GACzB;IACE,IAAI,EAAE,SAAS,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IAEjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GACD;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,GAC1C;IACE,IAAI,EAAE,WAAW,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,OAAO,CAAC;IAEhB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB,GACD;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GACrD;IAAE,IAAI,EAAE,WAAW,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GAEpD;IAAE,IAAI,EAAE,kBAAkB,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GAC3D;IAAE,IAAI,EAAE,aAAa,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,GACvC;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GACvD;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GACrD;IACE,IAAI,EAAE,QAAQ,CAAC;IACf,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GACD;IAAE,IAAI,EAAE,OAAO,CAAA;CAAE,GACjB;IAAE,IAAI,EAAE,MAAM,CAAA;CAAE,GAChB;IAAE,IAAI,EAAE,KAAK,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAEnC,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,OAAO,GAAG,WAAW,CAAC;IAC9B;;;OAGG;IACH,QAAQ,EAAE,eAAe,EAAE,CAAC;IAC5B;;OAEG;IACH,aAAa,CAAC,EAAE,iBAAiB,EAAE,CAAC;IACpC,iBAAiB,CAAC,EAAE,iBAAiB,EAAE,CAAC;IACxC,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,aAAa,CAAC,EAAE,aAAa,CAAC;CAC/B;AAwKD,qBAAa,UAAU;IACrB,OAAO,CAAC,SAAS,CAAY;;IAQ7B,OAAO,CAAC,SAAS;IAejB;;;;;;;;OAQG;IACH,OAAO,CAAC,kBAAkB;IA6FpB,MAAM,CAAC,OAAO,EAAE,UAAU,GAAG,OAAO,CAAC,WAAW,CAAC;CAsKxD"}
+{"version":3,"file":"LLMService.d.ts","sourceRoot":"","sources":["../../src/services/LLMService.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAQtC,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1B,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,MAAM,iBAAiB,GACzB;IACE,IAAI,EAAE,SAAS,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IAEjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GACD;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,GAC1C;IACE,IAAI,EAAE,WAAW,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,OAAO,CAAC;IAEhB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB,GACD;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GACrD;IAAE,IAAI,EAAE,WAAW,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GAEpD;IAAE,IAAI,EAAE,kBAAkB,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GAC3D;IAAE,IAAI,EAAE,aAAa,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,GACvC;IAAE,IAAI,EAAE,cAAc,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GACvD;IAAE,IAAI,EAAE,YAAY,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,OAAO,CAAA;CAAE,GACrD;IACE,IAAI,EAAE,QAAQ,CAAC;IACf,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,GACD;IAAE,IAAI,EAAE,OAAO,CAAA;CAAE,GACjB;IAAE,IAAI,EAAE,MAAM,CAAA;CAAE,GAChB;IAAE,IAAI,EAAE,KAAK,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAEnC,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,OAAO,GAAG,WAAW,CAAC;IAC9B;;;OAGG;IACH,QAAQ,EAAE,eAAe,EAAE,CAAC;IAC5B;;OAEG;IACH,aAAa,CAAC,EAAE,iBAAiB,EAAE,CAAC;IACpC,iBAAiB,CAAC,EAAE,iBAAiB,EAAE,CAAC;IACxC;;;;OAIG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,aAAa,CAAC,EAAE,aAAa,CAAC;CAC/B;AAqLD,qBAAa,UAAU;IACrB,OAAO,CAAC,SAAS,CAAY;;IAQ7B,OAAO,CAAC,SAAS;IAejB;;;;;;;;OAQG;IACH,OAAO,CAAC,kBAAkB;IA6FpB,MAAM,CAAC,OAAO,EAAE,UAAU,GAAG,OAAO,CAAC,WAAW,CAAC;CAqLxD"}

package/dist/services/LLMService.js

@@ -103,21 +103,34 @@ The healing system depends on the attribute value being present in your response
 If the element's text changed, you MUST include:
   "contentChange": { "oldText": "<previous>", "newText": "<current>" }
 
+### CONFIDENCE SCORE (MANDATORY)
+You MUST include a numeric "confidence" field — an INTEGER in the range 0-100 —
+representing how certain you are that the returned chain will resolve the
+intended element on the next test run. Use this rubric:
+  - 90-100: stable id, data-testid, or unique role+name — zero ambiguity in the DOM.
+  - 70-89:  strong semantic match, but some structural drift or competing candidates.
+  - 50-69:  best-guess match — multiple plausible candidates required disambiguation.
+  - 1-49:   speculative match — return this only when no better candidate exists.
+NEVER omit this field. NEVER return it as a string, boolean, or null.
+Reports rendered downstream will show "n/a" when this field is missing — which
+is a clear failure signal to the developer reviewing the heal.
+
 ### MANDATORY OUTPUT FORMAT
 Return ONLY valid JSON.
 
 Example output — Identifying a specific button in a list row:
 {
   "status": "FIXED",
+  "confidence": 92,
   "chainSegments": [
     { "type": "locator", "selector": ".user-item" },
     { "type": "filter", "hasText": "Bob Smith" },
     { "type": "getByRole", "role": "button", "name": "Modify Bob" }
   ],
   "segments": [
-    { 
-      "type": "element", 
-      "selector": ".user-item >> internal:has-text=\"Bob Smith\"i >> internal:role=button[name=\"Modify Bob\"i]" 
+    {
+      "type": "element",
+      "selector": ".user-item >> internal:has-text=\"Bob Smith\"i >> internal:role=button[name=\"Modify Bob\"i]"
     }
   ],
   "explanation": "Updated container to .user-item and targeted the button by its new semantic name 'Modify Bob' to ensure a unique match."
@@ -343,6 +356,19 @@ Return ONLY a raw JSON object — no markdown, no preamble.`;
             if (!jsonString)
                 throw new Error("AI did not return a valid JSON object");
             const parsed = JSON.parse(jsonString);
+            // ── Normalise confidence: keep only finite numbers in [0,100] ─────
+            // Anything else (undefined, NaN, null, string, out-of-range) collapses
+            // to `undefined` so reports render "n/a" instead of synthesising 0.
+            const rawConf = parsed.confidence;
+            if (typeof rawConf === "number" && Number.isFinite(rawConf)) {
+                parsed.confidence = Math.max(0, Math.min(100, Math.round(rawConf)));
+            }
+            else {
+                if (rawConf !== undefined) {
+                    console.warn(`[LLMService] ⚠️  Discarded non-numeric confidence: ${JSON.stringify(rawConf)}`);
+                }
+                parsed.confidence = undefined;
+            }
             // ── Normalize chainSegments → segments (legacy) ───────────────────
             if (parsed.chainSegments && Array.isArray(parsed.chainSegments)) {
                 // Reconstruct a flat new_selector from chain segments for legacy paths
@@ -430,7 +456,7 @@ Return ONLY a raw JSON object — no markdown, no preamble.`;
                 status: "NOT_FOUND",
                 segments: [],
                 new_selector: null,
-                confidence: 0,
+                confidence: undefined,
                 explanation: `AI Error: ${error.message}`,
             };
         }

package/dist/services/PRAutomationService.d.ts

@@ -10,8 +10,14 @@ export interface StrategyDecision {
     effective: PRStrategy;
     configured: PRStrategy;
     downgradeReason: "PROTECTED_BRANCH" | "LOW_CONFIDENCE" | "LOW_AUDITOR" | null;
-    minAiConfidence: number;
-    minAuditorConfidence: number;
+    /**
+     * Lowest AI confidence observed across this session's heals, or `undefined`
+     * when no heal carried a numeric confidence (LLM omitted the field). Render
+     * via `fmtPct` so the absence reads as "n/a" instead of 0%.
+     */
+    minAiConfidence: number | undefined;
+    /** Same semantics as `minAiConfidence`, for the auditor's adjusted score. */
+    minAuditorConfidence: number | undefined;
 }
 export interface ExecuteContext {
     cwd: string;
@@ -19,6 +25,7 @@ export interface ExecuteContext {
 }
 export declare function detectBranch(cwd?: string): BranchInfo;
 export declare function decideEffectiveStrategy(cfg: ResolvedPRAutomation, branch: string | null, heals: HealedEvent[]): StrategyDecision;
+export declare function fmtPct(v: unknown): string;
 export interface ExecutionResult {
     effective: PRStrategy;
     prUrl: string | null;

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

@@ -1 +1 @@
-{"version":3,"file":"PRAutomationService.d.ts","sourceRoot":"","sources":["../../src/services/PRAutomationService.ts"],"names":[],"mappings":"AAiBA,OAAO,KAAK,EACV,oBAAoB,EACpB,UAAU,EACV,SAAS,EACV,MAAM,sBAAsB,CAAC;AAC9B,OAAO,KAAK,EACV,WAAW,EACX,cAAc,EACf,MAAM,qBAAqB,CAAC;AAM7B,MAAM,WAAW,UAAU;IACzB,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,IAAI,EAAE,OAAO,CAAC;IACd,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,MAAM,EAAE,KAAK,GAAG,KAAK,GAAG,MAAM,CAAC;CAChC;AAED,MAAM,WAAW,gBAAgB;IAC/B,SAAS,EAAE,UAAU,CAAC;IACtB,UAAU,EAAE,UAAU,CAAC;IACvB,eAAe,EAAE,kBAAkB,GAAG,gBAAgB,GAAG,aAAa,GAAG,IAAI,CAAC;IAC9E,eAAe,EAAE,MAAM,CAAC;IACxB,oBAAoB,EAAE,MAAM,CAAC;CAC9B;AAED,MAAM,WAAW,cAAc;IAC7B,GAAG,EAAE,MAAM,CAAC;IACZ,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B;AAiDD,wBAAgB,YAAY,CAAC,GAAG,GAAE,MAAsB,GAAG,UAAU,CAqDpE;AAMD,wBAAgB,uBAAuB,CACrC,GAAG,EAAE,oBAAoB,EACzB,MAAM,EAAE,MAAM,GAAG,IAAI,EACrB,KAAK,EAAE,WAAW,EAAE,GACnB,gBAAgB,CAuDlB;AAsDD,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,UAAU,CAAC;IACtB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,wBAAsB,OAAO,CAC3B,GAAG,EAAE,oBAAoB,EACzB,QAAQ,EAAE,gBAAgB,EAC1B,KAAK,EAAE,WAAW,EAAE,EACpB,UAAU,EAAE,UAAU,EACtB,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,eAAe,CAAC,CA4J1B;AAMD,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,cAAc,EAAE,EACxB,UAAU,EAAE,UAAU,EACtB,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,IAAI,CAAC,CA0Ff"}
+{"version":3,"file":"PRAutomationService.d.ts","sourceRoot":"","sources":["../../src/services/PRAutomationService.ts"],"names":[],"mappings":"AAiBA,OAAO,KAAK,EACV,oBAAoB,EACpB,UAAU,EACV,SAAS,EACV,MAAM,sBAAsB,CAAC;AAC9B,OAAO,KAAK,EACV,WAAW,EACX,cAAc,EACf,MAAM,qBAAqB,CAAC;AAM7B,MAAM,WAAW,UAAU;IACzB,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,IAAI,EAAE,OAAO,CAAC;IACd,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,MAAM,EAAE,KAAK,GAAG,KAAK,GAAG,MAAM,CAAC;CAChC;AAED,MAAM,WAAW,gBAAgB;IAC/B,SAAS,EAAE,UAAU,CAAC;IACtB,UAAU,EAAE,UAAU,CAAC;IACvB,eAAe,EAAE,kBAAkB,GAAG,gBAAgB,GAAG,aAAa,GAAG,IAAI,CAAC;IAC9E;;;;OAIG;IACH,eAAe,EAAE,MAAM,GAAG,SAAS,CAAC;IACpC,6EAA6E;IAC7E,oBAAoB,EAAE,MAAM,GAAG,SAAS,CAAC;CAC1C;AAED,MAAM,WAAW,cAAc;IAC7B,GAAG,EAAE,MAAM,CAAC;IACZ,cAAc,EAAE,MAAM,GAAG,IAAI,CAAC;CAC/B;AA2ED,wBAAgB,YAAY,CAAC,GAAG,GAAE,MAAsB,GAAG,UAAU,CA8EpE;AAMD,wBAAgB,uBAAuB,CACrC,GAAG,EAAE,oBAAoB,EACzB,MAAM,EAAE,MAAM,GAAG,IAAI,EACrB,KAAK,EAAE,WAAW,EAAE,GACnB,gBAAgB,CAgElB;AAMD,wBAAgB,MAAM,CAAC,CAAC,EAAE,OAAO,GAAG,MAAM,CAEzC;AAuFD,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,UAAU,CAAC;IACtB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,wBAAsB,OAAO,CAC3B,GAAG,EAAE,oBAAoB,EACzB,QAAQ,EAAE,gBAAgB,EAC1B,KAAK,EAAE,WAAW,EAAE,EACpB,UAAU,EAAE,UAAU,EACtB,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,eAAe,CAAC,CA4J1B;AAMD,wBAAsB,iBAAiB,CACrC,MAAM,EAAE,SAAS,EACjB,MAAM,EAAE,cAAc,EAAE,EACxB,UAAU,EAAE,UAAU,EACtB,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,IAAI,CAAC,CA0Ff"}

package/dist/services/PRAutomationService.js

@@ -48,6 +48,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.detectBranch = detectBranch;
 exports.decideEffectiveStrategy = decideEffectiveStrategy;
+exports.fmtPct = fmtPct;
 exports.execute = execute;
 exports.handleBugDetected = handleBugDetected;
 const fs = __importStar(require("fs"));
@@ -82,6 +83,30 @@ function gitAvailable(cwd) {
 function isDetachedHead(cwd) {
     return !sh("git symbolic-ref HEAD", cwd).ok;
 }
+const SELA_BRANCH_PREFIX = "sela/heal-";
+function isSelaHealBranch(branch) {
+    return !!branch && branch.startsWith(SELA_BRANCH_PREFIX);
+}
+/**
+ * Resolve the repo's default integration branch.
+ *   1. origin/HEAD symbolic-ref (set by `git clone`).
+ *   2. First existing local ref among: main, master, develop, dev.
+ * Returns null if none found — caller must handle that.
+ */
+function findDefaultBranch(cwd) {
+    const headRef = sh("git symbolic-ref refs/remotes/origin/HEAD", cwd);
+    if (headRef.ok) {
+        const m = headRef.stdout.trim().match(/refs\/remotes\/origin\/(.+)$/);
+        if (m)
+            return m[1];
+    }
+    for (const candidate of ["main", "master", "develop", "dev"]) {
+        if (sh(`git rev-parse --verify --quiet ${candidate}`, cwd).ok) {
+            return candidate;
+        }
+    }
+    return null;
+}
 // ═══════════════════════════════════════════════════════════════════
 // TEMPLATING
 // ═══════════════════════════════════════════════════════════════════
@@ -98,14 +123,17 @@ function detectBranch(cwd = process.cwd()) {
     const ghEventName = process.env.GITHUB_EVENT_NAME;
     const ghRef = process.env.GITHUB_REF; // "refs/pull/123/merge"
     // Prefer HEAD_REF on PRs — REF_NAME is the synthetic merge ref.
-    if (ghHeadRef && ghHeadRef.length > 0) {
+    if (ghHeadRef && ghHeadRef.length > 0 && !isSelaHealBranch(ghHeadRef)) {
         const prNumber = (() => {
             const m = ghRef?.match(/refs\/pull\/(\d+)\//);
             return m ? Number(m[1]) : null;
         })();
         return { branch: ghHeadRef, isPR: true, prNumber, source: "env" };
     }
-    if (ghRefName && ghRefName.length > 0 && !ghRefName.endsWith("/merge")) {
+    if (ghRefName &&
+        ghRefName.length > 0 &&
+        !ghRefName.endsWith("/merge") &&
+        !isSelaHealBranch(ghRefName)) {
         return {
             branch: ghRefName,
             isPR: ghEventName === "pull_request",
@@ -115,18 +143,16 @@ function detectBranch(cwd = process.cwd()) {
     }
     // GitLab CI fallback
     const gitlabRef = process.env.CI_COMMIT_REF_NAME;
-    if (gitlabRef && gitlabRef.length > 0) {
+    if (gitlabRef && gitlabRef.length > 0 && !isSelaHealBranch(gitlabRef)) {
         return { branch: gitlabRef, isPR: false, prNumber: null, source: "env" };
     }
     // Generic Jenkins / local fallback
     const jenkinsBranch = process.env.GIT_BRANCH;
     if (jenkinsBranch && jenkinsBranch.length > 0) {
-        return {
-            branch: jenkinsBranch.replace(/^origin\//, ""),
-            isPR: false,
-            prNumber: null,
-            source: "env",
-        };
+        const cleaned = jenkinsBranch.replace(/^origin\//, "");
+        if (!isSelaHealBranch(cleaned)) {
+            return { branch: cleaned, isPR: false, prNumber: null, source: "env" };
+        }
     }
     // Local: git CLI
     if (!gitAvailable(cwd)) {
@@ -135,22 +161,45 @@ function detectBranch(cwd = process.cwd()) {
     const r = sh("git rev-parse --abbrev-ref HEAD", cwd);
     if (!r.ok)
         return { branch: null, isPR: false, prNumber: null, source: "none" };
-    const branch = r.stdout.trim();
-    if (!branch || branch === "HEAD") {
+    const localBranch = r.stdout.trim();
+    if (!localBranch || localBranch === "HEAD") {
         return { branch: null, isPR: false, prNumber: null, source: "none" };
     }
-    return { branch, isPR: false, prNumber: null, source: "git" };
+    // ── Sela self-branch guard ─────────────────────────────────────────
+    // If we're sitting on a previously-created `sela/heal-*` branch (typical
+    // local-rerun scenario), do NOT use it as the PR base. Fall back to the
+    // repo's default integration branch (main / master / develop) so the new
+    // PR targets a real branch, not another sela patch branch.
+    if (isSelaHealBranch(localBranch)) {
+        const fallback = findDefaultBranch(cwd);
+        if (fallback) {
+            console.warn(`[Sela PR] ⚠️  HEAD is on '${localBranch}' (sela patch branch) — ` +
+                `using '${fallback}' as PR base instead.`);
+            return { branch: fallback, isPR: false, prNumber: null, source: "git" };
+        }
+        console.warn(`[Sela PR] ⚠️  HEAD is on '${localBranch}' and no default branch found — ` +
+            `skipping PR automation to avoid self-targeting.`);
+        return { branch: null, isPR: false, prNumber: null, source: "none" };
+    }
+    return { branch: localBranch, isPR: false, prNumber: null, source: "git" };
 }
 // ═══════════════════════════════════════════════════════════════════
 // STRATEGY DECISION
 // ═══════════════════════════════════════════════════════════════════
 function decideEffectiveStrategy(cfg, branch, heals) {
-    const minAi = heals.length === 0
-        ? 100
-        : Math.min(...heals.map((h) => h.aiConfidence ?? 100));
-    const minAud = heals.length === 0
-        ? 100
-        : Math.min(...heals.map((h) => h.auditor?.confidence ?? 100));
+    // Compute MIN across only the heals that reported a finite numeric
+    // confidence. When *no* heal carried one, the result is `undefined` —
+    // surfaced as "n/a" in reports rather than synthesised as 100% (which
+    // would silently mask the absence of evidence) or 0% (which would
+    // incorrectly force a downgrade for every clean run on Sonnet).
+    const aiVals = heals
+        .map((h) => h.aiConfidence)
+        .filter((v) => typeof v === "number" && Number.isFinite(v));
+    const audVals = heals
+        .map((h) => h.auditor?.confidence)
+        .filter((v) => typeof v === "number" && Number.isFinite(v));
+    const minAi = aiVals.length > 0 ? Math.min(...aiVals) : undefined;
+    const minAud = audVals.length > 0 ? Math.min(...audVals) : undefined;
     const baseDecision = {
         effective: cfg.strategy,
         configured: cfg.strategy,
@@ -168,8 +217,11 @@ function decideEffectiveStrategy(cfg, branch, heals) {
             downgradeReason: "PROTECTED_BRANCH",
         };
     }
-    // Rule 2: low AI confidence
+    // Rule 2: low AI confidence — only triggers when the measured min is
+    // actually below the threshold. Missing confidence is "unknown", not
+    // "zero", so we do NOT downgrade purely on absence.
     if (cfg.reviewThresholds.minConfidenceForDirectCommit > 0 &&
+        typeof minAi === "number" &&
         minAi < cfg.reviewThresholds.minConfidenceForDirectCommit) {
         return {
             ...baseDecision,
@@ -177,8 +229,9 @@ function decideEffectiveStrategy(cfg, branch, heals) {
             downgradeReason: "LOW_CONFIDENCE",
         };
     }
-    // Rule 3: low auditor score
+    // Rule 3: low auditor score — same "absence ≠ zero" rule as Rule 2.
     if (cfg.reviewThresholds.minAuditorScoreForDirectCommit > 0 &&
+        typeof minAud === "number" &&
         minAud < cfg.reviewThresholds.minAuditorScoreForDirectCommit) {
         return {
             ...baseDecision,
@@ -191,6 +244,43 @@ function decideEffectiveStrategy(cfg, branch, heals) {
 // ═══════════════════════════════════════════════════════════════════
 // PR BODY RENDERING
 // ═══════════════════════════════════════════════════════════════════
+function fmtPct(v) {
+    return typeof v === "number" && Number.isFinite(v) ? `${v}%` : "n/a";
+}
+/**
+ * Strip the file-level preamble from a unified diff so the GitHub Markdown
+ * renderer doesn't show `--- a/file` / `+++ b/file` as red/green lines (which
+ * read as fake deletions/additions in a ```diff fence). Keeps the `@@` hunk
+ * header and every -/+/space content row intact, so the rendered block is
+ * still syntactically a valid unified diff that GitHub highlights correctly.
+ */
+function stripDiffPreamble(raw) {
+    return raw
+        .split(/\r?\n/)
+        .filter((ln) => {
+        if (ln.startsWith("diff "))
+            return false;
+        if (ln.startsWith("index "))
+            return false;
+        if (ln.startsWith("--- "))
+            return false;
+        if (ln.startsWith("+++ "))
+            return false;
+        if (ln.startsWith("new file mode"))
+            return false;
+        if (ln.startsWith("deleted file mode"))
+            return false;
+        if (ln.startsWith("similarity "))
+            return false;
+        if (ln.startsWith("rename "))
+            return false;
+        if (ln.startsWith("Binary files"))
+            return false;
+        return true;
+    })
+        .join("\n")
+        .replace(/\n+$/, "");
+}
 function buildDefaultBody(heals, branchInfo, reportLink, decision) {
     const lines = [];
     lines.push(`# 🤖 Sela Insights — Automation Suite Healed!`);
@@ -199,8 +289,8 @@ function buildDefaultBody(heals, branchInfo, reportLink, decision) {
     lines.push("");
     lines.push(`## 📊 Executive Summary`);
     lines.push(`* **Heals:** ${heals.length}`);
-    lines.push(`* **Min AI Confidence:** ${decision.minAiConfidence}%`);
-    lines.push(`* **Min Auditor Confidence:** ${decision.minAuditorConfidence}%`);
+    lines.push(`* **Min AI Confidence:** ${fmtPct(decision.minAiConfidence)}`);
+    lines.push(`* **Min Auditor Confidence:** ${fmtPct(decision.minAuditorConfidence)}`);
     lines.push(`* **Branch:** \`${branchInfo.branch ?? "unknown"}\``);
     if (decision.downgradeReason) {
         lines.push(`* **⚠️ Strategy Downgrade:** \`${decision.configured}\` → \`${decision.effective}\` (${decision.downgradeReason})`);
@@ -208,16 +298,26 @@ function buildDefaultBody(heals, branchInfo, reportLink, decision) {
     lines.push("");
     lines.push(`## 💻 Per-Heal Diff`);
     for (const h of heals) {
-        lines.push(`### \`${h.sourceFile}:${h.sourceLine}\` — ${h.testTitle ?? "(no test title)"}`);
+        const lineRef = h.newLineNumber ?? h.sourceLine;
+        lines.push(`### \`${h.sourceFile}:${lineRef}\` — ${h.testTitle ?? "(no test title)"}`);
         lines.push("");
         lines.push("```diff");
-        lines.push(`- ${h.oldCodeLine.trim()}`);
-        lines.push(`+ ${h.newCodeLine.trim()}`);
+        if (h.gitUnifiedDiff && h.gitUnifiedDiff.trim().length > 0) {
+            // Emit the real git diff verbatim. This natively handles cross-file
+            // healing, multi-line edits, and template-literal changes — none of
+            // which the legacy single-line -/+ pair could represent.
+            lines.push(stripDiffPreamble(h.gitUnifiedDiff));
+        }
+        else {
+            // Fallback when git was unavailable / the file was untracked.
+            lines.push(`- ${(h.oldCodeLine ?? "").trim()}`);
+            lines.push(`+ ${(h.newCodeLine ?? "").trim()}`);
+        }
         lines.push("```");
         if (h.aiExplanation) {
             lines.push(`> 🧠 ${h.aiExplanation}`);
         }
-        lines.push(`> AI Confidence: **${h.aiConfidence}%** · Auditor: **${h.auditor?.confidence ?? "n/a"}%**`);
+        lines.push(`> AI Confidence: **${fmtPct(h.aiConfidence)}** · Auditor: **${fmtPct(h.auditor?.confidence)}**`);
         lines.push("");
     }
     lines.push(`## 🔍 Visual Evidence`);

package/dist/services/SafetyGuard.d.ts

@@ -8,7 +8,13 @@ export interface HealContext {
     branch?: string;
 }
 export interface AIFixSummary {
-    confidence: number;
+    /**
+     * Raw AI confidence 0-100. `undefined` when the LLM omitted the field —
+     * SafetyGuard's hard-stop / review-threshold logic treats absence as the
+     * conservative worst case (0) so unverified heals are NOT silently let
+     * through, while downstream report renderers display "n/a".
+     */
+    confidence?: number;
     contentChange?: {
         oldText: string;
         newText: string;

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

@@ -1 +1 @@
-{"version":3,"file":"SafetyGuard.d.ts","sourceRoot":"","sources":["../../src/services/SafetyGuard.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAC1D,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAMpE,MAAM,MAAM,QAAQ,GAAG,WAAW,GAAG,QAAQ,CAAC;AAE9C,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,QAAQ,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,yEAAyE;IACzE,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,YAAY;IAC3B,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IACrD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,4EAA4E;IAC5E,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,4EAA4E;IAC5E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,oEAAoE;IACpE,OAAO,CAAC,EAAE,MAAM,CAAC;IAEjB,0EAA0E;IAC1E,cAAc,CAAC,EAAE,cAAc,CAAC;IAChC,uEAAuE;IACvE,YAAY,CAAC,EAAE,YAAY,EAAE,CAAC;IAC9B,6DAA6D;IAC7D,WAAW,CAAC,EAAE,UAAU,CAAC;IACzB,4CAA4C;IAC5C,WAAW,CAAC,EAAE,YAAY,EAAE,CAAC;IAC7B,+CAA+C;IAC/C,UAAU,CAAC,EAAE,UAAU,CAAC;CACzB;AAED,MAAM,MAAM,SAAS,GAAG,MAAM,GAAG,iBAAiB,GAAG,WAAW,GAAG,SAAS,CAAC;AAE7E,MAAM,WAAW,cAAc;IAC7B,KAAK,EAAE,SAAS,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,OAAO,CAAC;IACpB,wEAAwE;IACxE,IAAI,CAAC,EAAE;QACL,cAAc,CAAC,EAAE,YAAY,GAAG,YAAY,GAAG,cAAc,CAAC;QAC9D,iBAAiB,CAAC,EAAE,MAAM,CAAC;QAC3B,qFAAqF;QACrF,mBAAmB,CAAC,EAAE;YACpB,aAAa,EAAE,MAAM,CAAC;YACtB,OAAO,EAAE,MAAM,CAAC;YAChB,KAAK,EAAE,MAAM,CAAC;YACd,kBAAkB,EAAE,MAAM,CAAC;SAC5B,CAAC;KACH,CAAC;CACH;AA6DD,KAAK,cAAc,GACf,QAAQ,GACR,QAAQ,GACR,QAAQ,GACR,UAAU,GACV,MAAM,GACN,iBAAiB,GACjB,iBAAiB,GACjB,SAAS,CAAC;AA2Id,qBAAa,WAAW;IACtB,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,UAAU,CAAqB;gBAE3B,UAAU,CAAC,EAAE,kBAAkB;IASrC,QAAQ,CAAC,OAAO,EAAE,WAAW,EAAE,KAAK,EAAE,YAAY,GAAG,OAAO,CAAC,cAAc,CAAC;IAkTlF,OAAO,CAAC,kBAAkB;IAY1B,sBAAsB,CAAC,IAAI,EAAE,MAAM,GAAG,cAAc;IAsBpD,yBAAyB,CAAC,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,GAAG,MAAM;IAiBvD,OAAO,CAAC,mBAAmB;IAyB3B,OAAO,CAAC,iBAAiB;CAY1B"}
+{"version":3,"file":"SafetyGuard.d.ts","sourceRoot":"","sources":["../../src/services/SafetyGuard.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAC1D,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAMpE,MAAM,MAAM,QAAQ,GAAG,WAAW,GAAG,QAAQ,CAAC;AAE9C,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,QAAQ,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,yEAAyE;IACzE,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,YAAY;IAC3B;;;;;OAKG;IACH,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,4EAA4E;IAC5E,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,4EAA4E;IAC5E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,oEAAoE;IACpE,OAAO,CAAC,EAAE,MAAM,CAAC;IAEjB,0EAA0E;IAC1E,cAAc,CAAC,EAAE,cAAc,CAAC;IAChC,uEAAuE;IACvE,YAAY,CAAC,EAAE,YAAY,EAAE,CAAC;IAC9B,6DAA6D;IAC7D,WAAW,CAAC,EAAE,UAAU,CAAC;IACzB,4CAA4C;IAC5C,WAAW,CAAC,EAAE,YAAY,EAAE,CAAC;IAC7B,+CAA+C;IAC/C,UAAU,CAAC,EAAE,UAAU,CAAC;CACzB;AAED,MAAM,MAAM,SAAS,GAAG,MAAM,GAAG,iBAAiB,GAAG,WAAW,GAAG,SAAS,CAAC;AAE7E,MAAM,WAAW,cAAc;IAC7B,KAAK,EAAE,SAAS,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,UAAU,EAAE,OAAO,CAAC;IACpB,wEAAwE;IACxE,IAAI,CAAC,EAAE;QACL,cAAc,CAAC,EAAE,YAAY,GAAG,YAAY,GAAG,cAAc,CAAC;QAC9D,iBAAiB,CAAC,EAAE,MAAM,CAAC;QAC3B,qFAAqF;QACrF,mBAAmB,CAAC,EAAE;YACpB,aAAa,EAAE,MAAM,CAAC;YACtB,OAAO,EAAE,MAAM,CAAC;YAChB,KAAK,EAAE,MAAM,CAAC;YACd,kBAAkB,EAAE,MAAM,CAAC;SAC5B,CAAC;KACH,CAAC;CACH;AA6DD,KAAK,cAAc,GACf,QAAQ,GACR,QAAQ,GACR,QAAQ,GACR,UAAU,GACV,MAAM,GACN,iBAAiB,GACjB,iBAAiB,GACjB,SAAS,CAAC;AA2Id,qBAAa,WAAW;IACtB,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,UAAU,CAAqB;gBAE3B,UAAU,CAAC,EAAE,kBAAkB;IASrC,QAAQ,CAAC,OAAO,EAAE,WAAW,EAAE,KAAK,EAAE,YAAY,GAAG,OAAO,CAAC,cAAc,CAAC;IAyTlF,OAAO,CAAC,kBAAkB;IAY1B,sBAAsB,CAAC,IAAI,EAAE,MAAM,GAAG,cAAc;IAsBpD,yBAAyB,CAAC,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,MAAM,GAAG,MAAM;IAiBvD,OAAO,CAAC,mBAAmB;IAyB3B,OAAO,CAAC,iBAAiB;CAY1B"}

package/dist/services/SafetyGuard.js

@@ -199,7 +199,13 @@ class SafetyGuard {
     // PUBLIC ENTRY POINT
     // ─────────────────────────────────────────────────────────────
     async evaluate(context, aiFix) {
-        const { confidence, liveText, dnaBaselineText } = aiFix;
+        const { liveText, dnaBaselineText } = aiFix;
+        // Conservative fallback for missing confidence: treat as 0 so the
+        // hardStop / reviewThreshold rules still gate the heal. Reporting layers
+        // separately render the absence as "n/a" via fmtPct.
+        const confidence = typeof aiFix.confidence === "number" && Number.isFinite(aiFix.confidence)
+            ? aiFix.confidence
+            : 0;
         const { mode } = context;
         // ── 0. Visibility Gate ───────────────────────────────────────
         // Runs before any confidence or content checks.  A ghost element

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sela-core",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "AI self-healing Playwright wrapper — drop-in replacement for @playwright/test",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",