sela-core 1.0.4 → 1.0.5

package/dist/engine/SelaEngine.d.ts

@@ -14,6 +14,14 @@ 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;
+    /**
+     * Snapshot the entire file as a string[] of lines.  Returns an empty array
+     * if the file is missing or unreadable.  Used to preserve the PRE-mutation
+     * source so the diff in HealedEvent reflects the line that was actually
+     * mutated (which may differ from the failure line when JumpToDef lands on
+     * a const declaration above).
+     */
+    private _safeReadAllLines;
     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;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;IAwclB,OAAO,CAAC,aAAa;IAQrB,OAAO,CAAC,aAAa;IAarB;;;;;;OAMG;IACH,OAAO,CAAC,iBAAiB;IAWzB,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"}

package/dist/engine/SelaEngine.js

@@ -262,7 +262,7 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
                         reason: safetyDecision.reason,
                         safetyLevel: safetyDecision.level,
                         auditor: this._auditorBlockFromDecision(safetyDecision.meta),
-                        aiConfidence: aiFix.confidence,
+                        aiConfidence: typeof aiFix.confidence === "number" ? aiFix.confidence : 0,
                         aiExplanation: aiFix.explanation ?? "",
                         dnaBefore: (0, HealReportService_1.summariseSnapshot)(lastKnownState),
                         dnaAfter: (0, HealReportService_1.summariseSnapshot)(liveSnapshot ?? null),
@@ -286,12 +286,37 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
                         `(raw ${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);
+                // Snapshot the ENTIRE source file BEFORE the AST mutation.
+                // We don't know which line will actually be mutated until update()
+                // returns (JumpToDef may land on the const-decl line, not the failure
+                // line) — so we keep the pre-image around and slice into it after.
+                const preMutationLines = this._safeReadAllLines(filePath);
                 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;
+                // Old line: PRE-mutation content at the line that ended up being mutated.
+                // If the mutation was cross-file, read the pre-image of THAT file by
+                // recovering it from disk minus our own change (we don't track that —
+                // fall back to current pre-image if same file, else empty string).
+                const oldCodeLine = sameFileDef && preMutationLines.length > 0
+                    ? (preMutationLines[writtenLine - 1] ?? "")
+                    : this._safeReadLine(mutatedFilePath, writtenLine);
+                // New line: POST-mutation content at the same line in the mutated file.
+                const newCodeLine = 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,7 +364,7 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
                     oldCodeLine,
                     newCodeLine,
                     newLineNumber: writtenLine,
-                    aiConfidence: aiFix.confidence,
+                    aiConfidence: typeof aiFix.confidence === "number" ? aiFix.confidence : 0,
                     aiExplanation: aiFix.explanation ?? "",
                     aiAlternatives: this._buildAlternatives(aiFix.chainSegments, aiFix.segments),
                     reasoningSteps: this._buildReasoningSteps({
@@ -435,6 +460,25 @@ If the text has NOT changed, omit the "contentChange" field entirely.`;
             return "";
         }
     }
+    /**
+     * Snapshot the entire file as a string[] of lines.  Returns an empty array
+     * if the file is missing or unreadable.  Used to preserve the PRE-mutation
+     * source so the diff in HealedEvent reflects the line that was actually
+     * mutated (which may differ from the failure line when JumpToDef lands on
+     * a const declaration above).
+     */
+    _safeReadAllLines(filePath) {
+        try {
+            const cleaned = this._toReportFile(filePath);
+            const abs = path.isAbsolute(cleaned) ? cleaned : path.resolve(process.cwd(), cleaned);
+            if (!fs.existsSync(abs))
+                return [];
+            return fs.readFileSync(abs, "utf8").split(/\r?\n/);
+        }
+        catch {
+            return [];
+        }
+    }
     _auditorBlockFromDecision(meta) {
         if (!meta || !meta.auditorVerdict)
             return 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,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;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,CAuDlB;AA6DD,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

@@ -82,6 +82,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 +122,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 +142,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,11 +160,27 @@ 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
@@ -191,6 +232,9 @@ function decideEffectiveStrategy(cfg, branch, heals) {
 // ═══════════════════════════════════════════════════════════════════
 // PR BODY RENDERING
 // ═══════════════════════════════════════════════════════════════════
+function fmtPct(v) {
+    return typeof v === "number" && Number.isFinite(v) ? `${v}%` : "n/a";
+}
 function buildDefaultBody(heals, branchInfo, reportLink, decision) {
     const lines = [];
     lines.push(`# 🤖 Sela Insights — Automation Suite Healed!`);
@@ -199,8 +243,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 +252,17 @@ 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()}`);
+        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/package.json

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