sela-core 1.0.6 → 1.0.8

package/dist/services/SafetyGuard.js

@@ -3,18 +3,18 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SafetyGuard = void 0;
 const IntentAuditor_1 = require("./IntentAuditor");
+const SelaError_1 = require("../errors/SelaError");
 // ═══════════════════════════════════════════════════════════════════
 // DEFAULT THRESHOLDS  (match pre-config hardcoded values exactly)
 // ═══════════════════════════════════════════════════════════════════
 const DEFAULT_THRESHOLDS = {
     confidenceHardStop: 85,
     confidenceReviewThreshold: 95,
-    assertionSimilarityFloor: 0.7,
     auditorConfidenceMin: 75,
     allowSuspicious: false,
     auditorStrictness: "balanced",
 };
-// Binary semantic opposites — always trigger FAIL_HARD regardless of mode,
+// Binary semantic opposites - always trigger FAIL_HARD regardless of mode,
 // because they indicate a state inversion (pass→fail, success→error, etc.)
 // that almost certainly masks a real logical regression.
 const SEMANTIC_OPPOSITE_PAIRS = new Set([
@@ -55,6 +55,9 @@ const SEMANTIC_OPPOSITE_PAIRS = new Set([
     "valid|invalid",
     "invalid|valid",
 ]);
+function escapeRegExp(s) {
+    return s.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}
 const ROLE_KEYWORDS = new Map([
     [
         "SUBMIT",
@@ -185,15 +188,35 @@ const ROLE_KEYWORDS = new Map([
         ]),
     ],
 ]);
-// ═══════════════════════════════════════════════════════════════════
-// SafetyGuard
-// ═══════════════════════════════════════════════════════════════════
+// Pre-compiled word-boundary regexes for the role classifier.  Built
+// once at module load - classifyFunctionalRole() is on the hot heal
+// path, so we don't pay for regex construction per call.
+const ROLE_KEYWORD_REGEXES = Array.from(ROLE_KEYWORDS, ([role, keywords]) => ({
+    role,
+    regexes: Array.from(keywords, (kw) => new RegExp(`\\b${escapeRegExp(kw)}\\b`, "i")),
+}));
 class SafetyGuard {
     intentAuditor;
     thresholds;
-    constructor(thresholds) {
-        this.thresholds = thresholds ?? DEFAULT_THRESHOLDS;
-        this.intentAuditor = new IntentAuditor_1.IntentAuditor();
+    /**
+     * Accepts either the legacy positional `thresholds` argument or an
+     * options bag (DI-friendly).  Existing call sites keep working
+     * unchanged.
+     */
+    constructor(arg) {
+        const opts = SafetyGuard.normalizeArgs(arg);
+        this.thresholds = opts.thresholds ?? DEFAULT_THRESHOLDS;
+        this.intentAuditor = opts.intentAuditor ?? new IntentAuditor_1.IntentAuditor();
+    }
+    static normalizeArgs(arg) {
+        if (!arg)
+            return {};
+        // Heuristic: an options bag has either `intentAuditor` or a nested
+        // `thresholds` field; otherwise the value itself is a ResolvedThresholds.
+        if ("intentAuditor" in arg || "thresholds" in arg) {
+            return arg;
+        }
+        return { thresholds: arg };
     }
     // ─────────────────────────────────────────────────────────────
     // PUBLIC ENTRY POINT
@@ -217,7 +240,8 @@ class SafetyGuard {
             if (vis.isGhost) {
                 return {
                     level: "FAIL_HARD",
-                    reason: `Visibility Gate: candidate element is a Ghost (${vis.ghostReason}) — ` +
+                    code: SelaError_1.SAFETY_GUARD_CODES.FAIL_HARD_VISIBILITY_GHOST,
+                    reason: `Visibility Gate: candidate element is a Ghost (${vis.ghostReason}) - ` +
                         `functionally absent from the user's perspective`,
                     canProceed: false,
                 };
@@ -227,12 +251,13 @@ class SafetyGuard {
                 if (isMainBranch) {
                     return {
                         level: "FAIL_HARD",
-                        reason: `Visibility Gate: candidate element is occluded by an overlay on branch "${context.branch}" — ` +
+                        code: SelaError_1.SAFETY_GUARD_CODES.FAIL_HARD_VISIBILITY_OCCLUDED,
+                        reason: `Visibility Gate: candidate element is occluded by an overlay on branch "${context.branch}" - ` +
                             `hard-fail policy active on main`,
                         canProceed: false,
                     };
                 }
-                console.warn(`[SafetyGuard] ⚠️  Occlusion detected on branch "${context.branch ?? "unknown"}" — ` +
+                console.warn(`[SafetyGuard] ⚠️  Occlusion detected on branch "${context.branch ?? "unknown"}" - ` +
                     `escalating to REQUIRES_REVIEW (not hard-failing on non-main branch)`);
                 return {
                     level: "REQUIRES_REVIEW",
@@ -246,7 +271,8 @@ class SafetyGuard {
         if (confidence < this.thresholds.confidenceHardStop) {
             return {
                 level: "BLOCKED",
-                reason: `Confidence too low (${confidence}/100 < ${this.thresholds.confidenceHardStop}) — refusing to heal`,
+                code: SelaError_1.SAFETY_GUARD_CODES.BLOCKED_LOW_CONFIDENCE,
+                reason: `Confidence too low (${confidence}/100 < ${this.thresholds.confidenceHardStop}) - refusing to heal`,
                 canProceed: false,
             };
         }
@@ -257,16 +283,18 @@ class SafetyGuard {
         //
         // When liveText + dnaBaselineText are both available and differ, we
         // synthesise a contentChange pair and apply the full inversion-check
-        // pipeline on it — regardless of what the AI reported.
+        // pipeline on it - regardless of what the AI reported.
         //
         // If the AI DID report contentChange we use that (more precise than raw
         // innerText, which may include child element noise).
         let effectiveContentChange = aiFix.contentChange;
-        if (!effectiveContentChange && liveText !== undefined && dnaBaselineText !== undefined) {
+        if (!effectiveContentChange &&
+            liveText !== undefined &&
+            dnaBaselineText !== undefined) {
             const normLive = liveText.trim();
             const normDNA = dnaBaselineText.trim();
             if (normLive !== normDNA && normLive.length > 0 && normDNA.length > 0) {
-                console.warn(`[SafetyGuard] 🚨 Zero-Trust override — AI omitted contentChange but live DOM differs:\n` +
+                console.warn(`[SafetyGuard] 🚨 Zero-Trust override - AI omitted contentChange but live DOM differs:\n` +
                     `  DNA baseline : "${normDNA}"\n` +
                     `  Live text    : "${normLive}"`);
                 effectiveContentChange = { oldText: normDNA, newText: normLive };
@@ -274,7 +302,9 @@ class SafetyGuard {
         }
         // ── 3. Structural-only heal (no text change detected) ────────
         if (!effectiveContentChange) {
-            const level = confidence < this.thresholds.confidenceReviewThreshold ? "REQUIRES_REVIEW" : "SAFE";
+            const level = confidence < this.thresholds.confidenceReviewThreshold
+                ? "REQUIRES_REVIEW"
+                : "SAFE";
             return {
                 level,
                 reason: `Structural selector change only (confidence=${confidence})`,
@@ -286,11 +316,12 @@ class SafetyGuard {
         console.log(`[SafetyGuard] 🔬 Content change (${source}): "${oldText}" → "${newText}"`);
         // ── 4. Binary semantic-opposite guard ────────────────────────
         // Applies in ALL modes: "Enabled"→"Disabled", "Success"→"Error", etc.
-        // These always indicate a state inversion — never safe to auto-heal.
+        // These always indicate a state inversion - never safe to auto-heal.
         if (this.isSemanticOpposite(oldText, newText)) {
             return {
                 level: "FAIL_HARD",
-                reason: `Semantic inversion detected: "${oldText}" ↔ "${newText}" — ` +
+                code: SelaError_1.SAFETY_GUARD_CODES.FAIL_HARD_SEMANTIC_INVERSION,
+                reason: `Semantic inversion detected: "${oldText}" ↔ "${newText}" - ` +
                     `state inversions mask logical regressions`,
                 canProceed: false,
             };
@@ -305,29 +336,29 @@ class SafetyGuard {
         if (isStatusInversion) {
             return {
                 level: "FAIL_HARD",
-                reason: `Status inversion: "${oldText}" (${oldRole}) → "${newText}" (${newRole}) — ` +
+                code: SelaError_1.SAFETY_GUARD_CODES.FAIL_HARD_STATUS_INVERSION,
+                reason: `Status inversion: "${oldText}" (${oldRole}) → "${newText}" (${newRole}) - ` +
                     `masking a logic failure is not permitted`,
                 canProceed: false,
             };
         }
         // Cross-role swap (e.g. DELETE → CANCEL, SUBMIT → NAVIGATE)
-        if (oldRole !== "UNKNOWN" &&
-            newRole !== "UNKNOWN" &&
-            oldRole !== newRole) {
+        if (oldRole !== "UNKNOWN" && newRole !== "UNKNOWN" && oldRole !== newRole) {
             if (mode === "assertion") {
                 // Assertion explicitly checked the old functional text; a different
                 // function being found is almost certainly a wrong element fix.
                 return {
                     level: "BLOCKED",
-                    reason: `Functional role swap in assertion: "${oldText}" (${oldRole}) → "${newText}" (${newRole}) — ` +
+                    code: SelaError_1.SAFETY_GUARD_CODES.BLOCKED_ROLE_SWAP_ASSERTION,
+                    reason: `Functional role swap in assertion: "${oldText}" (${oldRole}) → "${newText}" (${newRole}) - ` +
                         `the assertion was verifying a specific functional state`,
                     canProceed: false,
                 };
             }
-            // Action mode: allow with warning — a UI refactor may rename a control
+            // Action mode: allow with warning - a UI refactor may rename a control
             // (e.g. "Delete" button renamed "Remove" or merged into a menu)
             console.warn(`[SafetyGuard] ⚠️  Functional role change in action: ` +
-                `"${oldText}" (${oldRole}) → "${newText}" (${newRole}) — review recommended`);
+                `"${oldText}" (${oldRole}) → "${newText}" (${newRole}) - review recommended`);
             return {
                 level: "REQUIRES_REVIEW",
                 reason: `Functional role change in action: "${oldText}" (${oldRole}) → "${newText}" (${newRole})`,
@@ -361,7 +392,8 @@ class SafetyGuard {
             if (auditVerdict.verdict === "INCONSISTENT") {
                 return {
                     level: "FAIL_HARD",
-                    reason: `Intent Auditor — INCONSISTENT: ${auditVerdict.reason}` +
+                    code: SelaError_1.SAFETY_GUARD_CODES.FAIL_HARD_AUDITOR_INCONSISTENT,
+                    reason: `Intent Auditor - INCONSISTENT: ${auditVerdict.reason}` +
                         (auditVerdict.inversionType
                             ? ` [inversion: ${auditVerdict.inversionType}]`
                             : ""),
@@ -377,23 +409,24 @@ class SafetyGuard {
                 if (!this.thresholds.allowSuspicious && mode === "assertion") {
                     return {
                         level: "BLOCKED",
-                        reason: `Intent Auditor — SUSPICIOUS in assertion mode: ${auditVerdict.reason}`,
+                        code: SelaError_1.SAFETY_GUARD_CODES.BLOCKED_AUDITOR_SUSPICIOUS,
+                        reason: `Intent Auditor - SUSPICIOUS in assertion mode: ${auditVerdict.reason}`,
                         canProceed: false,
                         meta,
                     };
                 }
                 if (this.thresholds.allowSuspicious && mode === "assertion") {
-                    console.log(`[SafetyGuard] ℹ️  allowSuspicious=true — SUSPICIOUS in assertion mode escalated to REQUIRES_REVIEW`);
+                    console.log(`[SafetyGuard] ℹ️  allowSuspicious=true - SUSPICIOUS in assertion mode escalated to REQUIRES_REVIEW`);
                 }
                 return {
                     level: "REQUIRES_REVIEW",
-                    reason: `Intent Auditor — SUSPICIOUS in ${mode} mode: ${auditVerdict.reason}`,
+                    reason: `Intent Auditor - SUSPICIOUS in ${mode} mode: ${auditVerdict.reason}`,
                     canProceed: true,
                     meta,
                 };
             }
-            // CONSISTENT — fall through to similarity gate
-            console.log(`[SafetyGuard] ✅ Intent Auditor — CONSISTENT (${auditVerdict.confidence}%): ${auditVerdict.reason}`);
+            // CONSISTENT - fall through to confidence tiers
+            console.log(`[SafetyGuard] ✅ Intent Auditor - CONSISTENT (${auditVerdict.confidence}%): ${auditVerdict.reason}`);
             if (auditVerdict.scoreBreakdown) {
                 const { rawConfidence, penalty, bonus } = auditVerdict.scoreBreakdown;
                 if (penalty !== 0 || bonus !== 0) {
@@ -408,31 +441,11 @@ class SafetyGuard {
                 }
             }
         }
-        // ── 8. Semantic similarity gate (same-role or both UNKNOWN) ──
-        // For same-role pairs (e.g. SUBMIT→SUBMIT: "Submit"→"Send") we skip
-        // the similarity gate entirely — the role match is sufficient proof
-        // of functional equivalence.
-        //
-        // The gate is only applied when BOTH roles are UNKNOWN in assertion
-        // mode, where we have no functional category to fall back on.
-        if (mode === "assertion" && oldRole === "UNKNOWN" && newRole === "UNKNOWN") {
-            const similarity = this.computeSemanticSimilarity(oldText, newText);
-            console.log(`[SafetyGuard] 📊 Similarity: "${oldText}" ↔ "${newText}" = ${(similarity * 100).toFixed(1)}%`);
-            if (similarity < this.thresholds.assertionSimilarityFloor) {
-                return {
-                    level: "BLOCKED",
-                    reason: `Text divergence too high in assertion: "${oldText}" → "${newText}" ` +
-                        `(similarity=${(similarity * 100).toFixed(1)}% < ${(this.thresholds.assertionSimilarityFloor * 100).toFixed(0)}%) — ` +
-                        `possible content regression`,
-                    canProceed: false,
-                };
-            }
-        }
-        // ── 9. Confidence tiers ──────────────────────────────────────
+        // ── 8. Confidence tiers ──────────────────────────────────────
         if (confidence < this.thresholds.confidenceReviewThreshold) {
             return {
                 level: "REQUIRES_REVIEW",
-                reason: `Low-confidence fix (${confidence}/100) — tagging for manual review`,
+                reason: `Low-confidence fix (${confidence}/100) - tagging for manual review`,
                 canProceed: true,
                 meta: localAuditMeta,
             };
@@ -455,76 +468,32 @@ class SafetyGuard {
     // ─────────────────────────────────────────────────────────────
     // FUNCTIONAL ROLE CLASSIFICATION
     //
-    // Returns the first role whose keyword set contains the text
-    // (exact match or substring). Returns UNKNOWN when no role matches.
+    // Returns the first role whose keyword matches the text as a whole
+    // word (delimited by \b boundaries).  Returns UNKNOWN when no role
+    // matches.
+    //
+    // Why \b instead of `.includes(kw)`:
+    //   - "unauthorized".includes("authorize")  → true   ← bug
+    //   - "offline".includes("on")              → true   ← bug
+    //   - "disconnected".includes("connected")  → true   ← bug
+    //   - "inactive".includes("active")         → true   ← bug
+    // The substring approach mis-classified negations into the
+    // positive role of their root word, silently bypassing
+    // FAIL_HARD_STATUS_INVERSION.  Word-boundary regex eliminates
+    // these collisions while keeping multi-word keywords ("sign in")
+    // and punctuation-flanked matches ("Submit!") working.
     // ─────────────────────────────────────────────────────────────
     classifyFunctionalRole(text) {
         const normalized = text.toLowerCase().trim();
-        for (const [role, keywords] of ROLE_KEYWORDS) {
-            for (const kw of keywords) {
-                if (normalized === kw || normalized.includes(kw)) {
+        if (normalized.length === 0)
+            return "UNKNOWN";
+        for (const { role, regexes } of ROLE_KEYWORD_REGEXES) {
+            for (const re of regexes) {
+                if (re.test(normalized))
                     return role;
-                }
             }
         }
         return "UNKNOWN";
     }
-    // ─────────────────────────────────────────────────────────────
-    // SEMANTIC SIMILARITY  (no external dependencies)
-    //
-    // Combined score: 0.5 × levenshtein_similarity + 0.5 × jaccard_similarity
-    //
-    // Levenshtein captures character-level similarity (typos, abbreviations).
-    // Jaccard captures word-level overlap (shared vocabulary, different order).
-    // The 50/50 blend is robust against short labels where one metric misleads.
-    // ─────────────────────────────────────────────────────────────
-    computeSemanticSimilarity(a, b) {
-        const na = a.toLowerCase().trim();
-        const nb = b.toLowerCase().trim();
-        if (na === nb)
-            return 1;
-        const maxLen = Math.max(na.length, nb.length);
-        const levSim = maxLen === 0 ? 1 : 1 - this.levenshteinDistance(na, nb) / maxLen;
-        const jaccard = this.jaccardSimilarity(na, nb);
-        return 0.5 * levSim + 0.5 * jaccard;
-    }
-    // ─────────────────────────────────────────────────────────────
-    // PRIVATE: Levenshtein distance (space-optimised two-row DP)
-    // ─────────────────────────────────────────────────────────────
-    levenshteinDistance(a, b) {
-        const m = a.length;
-        const n = b.length;
-        if (m === 0)
-            return n;
-        if (n === 0)
-            return m;
-        let prev = Array.from({ length: n + 1 }, (_, i) => i);
-        let curr = new Array(n + 1);
-        for (let i = 1; i <= m; i++) {
-            curr[0] = i;
-            for (let j = 1; j <= n; j++) {
-                const cost = a[i - 1] === b[j - 1] ? 0 : 1;
-                curr[j] = Math.min(curr[j - 1] + 1, prev[j] + 1, prev[j - 1] + cost);
-            }
-            [prev, curr] = [curr, prev];
-        }
-        return prev[n];
-    }
-    // ─────────────────────────────────────────────────────────────
-    // PRIVATE: Jaccard word-set similarity
-    // ─────────────────────────────────────────────────────────────
-    jaccardSimilarity(a, b) {
-        const setA = new Set(a.split(/\s+/).filter(Boolean));
-        const setB = new Set(b.split(/\s+/).filter(Boolean));
-        if (setA.size === 0 && setB.size === 0)
-            return 1;
-        let intersection = 0;
-        for (const w of setA) {
-            if (setB.has(w))
-                intersection++;
-        }
-        const union = setA.size + setB.size - intersection;
-        return union === 0 ? 1 : intersection / union;
-    }
 }
 exports.SafetyGuard = SafetyGuard;

package/dist/services/SnapshotService.js

@@ -67,12 +67,12 @@ class SnapshotService {
         return raw;
     }
     // ─────────────────────────────────────────────────────────────
-    // captureElement — single-pass atomic extraction of all v2 metrics.
+    // captureElement - single-pass atomic extraction of all v2 metrics.
     //
     // All data collection (core props, ancestry, visibility, anchors,
     // shadow-DOM flag) runs inside ONE page.evaluate() call so the
     // browser serialises a single IPC payload. Returns null on any
-    // failure — callers must treat null as "capture skipped" and not
+    // failure - callers must treat null as "capture skipped" and not
     // block the heal.
     // ─────────────────────────────────────────────────────────────
     async captureElement(page, selector, framePath = []) {
@@ -219,7 +219,7 @@ class SnapshotService {
                             closestLabel = t;
                     }
                 }
-                // P4: implicit <label> wrapper — clone and strip the form element
+                // P4: implicit <label> wrapper - clone and strip the form element
                 if (!closestLabel) {
                     const wrapper = el.closest("label");
                     if (wrapper) {
@@ -318,10 +318,10 @@ class SnapshotService {
         }
     }
     // ─────────────────────────────────────────────────────────────
-    // migrateToV2 — backfills default v2 fields onto a v1 snapshot.
+    // migrateToV2 - backfills default v2 fields onto a v1 snapshot.
     //
     // ancestry, visibility, and anchors are empty / safe defaults.
-    // The migrated snapshot is NOT re-saved to disk automatically —
+    // The migrated snapshot is NOT re-saved to disk automatically -
     // the engine decides when to flush.
     // ─────────────────────────────────────────────────────────────
     migrateToV2(v1) {

package/dist/services/SourceLinkService.d.ts

@@ -1,5 +1,5 @@
-import { type ASTUpdateResult } from './ASTSourceUpdater.js';
-import type { DnaRecord, DnaEditPatch } from './types.js';
+import { type ASTUpdateResult } from "./ASTSourceUpdater.js";
+import type { DnaRecord, DnaEditPatch } from "./types.js";
 interface CallerContext {
     filePath: string;
     line: number;

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

@@ -1 +1 @@
-{"version":3,"file":"SourceLinkService.d.ts","sourceRoot":"","sources":["../../src/services/SourceLinkService.ts"],"names":[],"mappings":"AAGA,OAAO,EAAoB,KAAK,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC/E,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAG1D,UAAU,aAAa;IACrB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;CACd;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;CAChB;AAiDD,MAAM,WAAW,kBAAkB;IACjC,oBAAoB,CAAC,MAAM,EAAE,SAAS,GAAG,aAAa,CAAC;IACvD,uBAAuB,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,kBAAkB,EAAE,CAAC;IAC9E,iBAAiB,CAAC,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,YAAY,GAAG,OAAO,CAAC,eAAe,CAAC,CAAC;CACrF;AAED,qBAAa,iBAAkB,YAAW,kBAAkB;IAC1D,OAAO,CAAC,QAAQ,CAAC,OAAO,CAA0B;IAElD,OAAO,CAAC,QAAQ,CAAC,OAAO,CAGrB;IAEH,oBAAoB,CAAC,MAAM,EAAE,SAAS,GAAG,aAAa;IAItD,uBAAuB,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,kBAAkB,EAAE;IAyDvE,iBAAiB,CAAC,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,YAAY,GAAG,OAAO,CAAC,eAAe,CAAC;CAyD1F"}
+{"version":3,"file":"SourceLinkService.d.ts","sourceRoot":"","sources":["../../src/services/SourceLinkService.ts"],"names":[],"mappings":"AAQA,OAAO,EAAoB,KAAK,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC/E,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAG1D,UAAU,aAAa;IACrB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;CACd;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;CAChB;AAoDD,MAAM,WAAW,kBAAkB;IACjC,oBAAoB,CAAC,MAAM,EAAE,SAAS,GAAG,aAAa,CAAC;IACvD,uBAAuB,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,kBAAkB,EAAE,CAAC;IAC9E,iBAAiB,CACf,MAAM,EAAE,SAAS,EACjB,KAAK,EAAE,YAAY,GAClB,OAAO,CAAC,eAAe,CAAC,CAAC;CAC7B;AAED,qBAAa,iBAAkB,YAAW,kBAAkB;IAC1D,OAAO,CAAC,QAAQ,CAAC,OAAO,CAA0B;IAElD,OAAO,CAAC,QAAQ,CAAC,OAAO,CAGrB;IAEH,oBAAoB,CAAC,MAAM,EAAE,SAAS,GAAG,aAAa;IAItD,uBAAuB,CACrB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,GACX,kBAAkB,EAAE;IAyDjB,iBAAiB,CACrB,MAAM,EAAE,SAAS,EACjB,KAAK,EAAE,YAAY,GAClB,OAAO,CAAC,eAAe,CAAC;CAuE5B"}

package/dist/services/SourceLinkService.js

@@ -10,15 +10,15 @@ const ts_morph_1 = require("ts-morph");
 const ASTSourceUpdater_js_1 = require("./ASTSourceUpdater.js");
 // Mirror of ASTSourceUpdater's module-scoped ALL_SELECTOR_METHODS.
 const ALL_SELECTOR_METHODS = new Set([
-    'locator',
-    'frameLocator',
-    'getByRole',
-    'getByLabel',
-    'getByText',
-    'getByTestId',
-    'getByPlaceholder',
-    'getByAltText',
-    'getByTitle',
+    "locator",
+    "frameLocator",
+    "getByRole",
+    "getByLabel",
+    "getByText",
+    "getByTestId",
+    "getByPlaceholder",
+    "getByAltText",
+    "getByTitle",
 ]);
 function dnaError(message, code) {
     const err = new Error(message);
@@ -33,11 +33,12 @@ function extractSelector(call) {
         return null;
     const first = args[0];
     const kind = first.getKind();
-    if (kind === ts_morph_1.SyntaxKind.StringLiteral || kind === ts_morph_1.SyntaxKind.NoSubstitutionTemplateLiteral) {
+    if (kind === ts_morph_1.SyntaxKind.StringLiteral ||
+        kind === ts_morph_1.SyntaxKind.NoSubstitutionTemplateLiteral) {
         const raw = first.getText();
         return raw.slice(1, -1); // strip surrounding quotes/backticks
     }
-    // For template literals with expressions, identifiers, etc. — return raw text.
+    // For template literals with expressions, identifiers, etc. - return raw text.
     return first.getText();
 }
 // Get the method name from a CallExpression's callee.
@@ -68,7 +69,7 @@ class SourceLinkService {
             return [];
         let sf;
         try {
-            // addSourceFileAtPath throws if already added — use getOrAdd pattern.
+            // addSourceFileAtPath throws if already added - use getOrAdd pattern.
             sf =
                 this.project.getSourceFile(absPath) ??
                     this.project.addSourceFileAtPath(absPath);
@@ -91,7 +92,7 @@ class SourceLinkService {
             if (selector === null)
                 return;
             const raw = call.getText();
-            const preview = raw.length > 80 ? raw.slice(0, 79) + '…' : raw;
+            const preview = raw.length > 80 ? raw.slice(0, 79) + "…" : raw;
             candidates.push({
                 col: call.getStart(),
                 preview,
@@ -110,16 +111,24 @@ class SourceLinkService {
     }
     async applySourceUpdate(record, patch) {
         if (!record.sourceFile) {
-            return { success: false, reason: 'DNA record has no sourceFile', strategy: 'none' };
+            return {
+                success: false,
+                reason: "DNA record has no sourceFile",
+                strategy: "none",
+            };
         }
         if (!(0, fs_1.existsSync)(path_1.default.resolve(record.sourceFile))) {
-            throw dnaError(`Source file not found: ${record.sourceFile}`, 'SOURCE_FILE_MISSING');
+            throw dnaError(`Source file not found: ${record.sourceFile}`, "SOURCE_FILE_MISSING");
         }
         if (record.sourceLine <= 0) {
-            return { success: false, reason: 'DNA record has no valid sourceLine', strategy: 'none' };
+            return {
+                success: false,
+                reason: "DNA record has no valid sourceLine",
+                strategy: "none",
+            };
         }
         const caller = this.resolveCallerContext(record);
-        if (patch.field === 'selector') {
+        if (patch.field === "selector") {
             const newSelector = patch.newValue;
             // Resolve oldSelector: prefer patch.oldValue, fall back to the locator
             // at patch.locatorIndex on the recorded line.
@@ -131,25 +140,25 @@ class SourceLinkService {
                     return {
                         success: false,
                         reason: `No locator found at index ${patch.locatorIndex} on line ${record.sourceLine}`,
-                        strategy: 'none',
+                        strategy: "none",
                     };
                 }
                 oldSelector = target.selector;
             }
             return this.updater.update(caller, oldSelector, newSelector);
         }
-        if (patch.field === 'text') {
+        if (patch.field === "text") {
             // Use ASTSourceUpdater's contentChange path for text-content edits.
             const contentChange = {
-                oldText: String(patch.oldValue ?? ''),
-                newText: String(patch.newValue ?? ''),
+                oldText: String(patch.oldValue ?? ""),
+                newText: String(patch.newValue ?? ""),
             };
             return this.updater.update(caller, record.selector, record.selector, undefined, undefined, contentChange);
         }
         return {
             success: false,
             reason: `Source update not supported for field: ${patch.field}`,
-            strategy: 'none',
+            strategy: "none",
         };
     }
 }

package/dist/services/SourceUpdater.d.ts

@@ -4,7 +4,7 @@ interface UpdateResult {
     success: boolean;
     reason: string;
     lineUpdated?: number;
-    /** Canonical locator string written to disk — use this for runtime execution. */
+    /** Canonical locator string written to disk - use this for runtime execution. */
     healedLocatorString?: string;
     /** Populated by semantic dispatch: where the definition was mutated. */
     definitionSite?: {

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

@@ -1 +1 @@
-{"version":3,"file":"SourceUpdater.d.ts","sourceRoot":"","sources":["../../src/services/SourceUpdater.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAE/C,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,cAAc,CAAC;AAMtD,UAAU,YAAY;IACpB,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,iFAAiF;IACjF,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,wEAAwE;IACxE,cAAc,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IAChD,iEAAiE;IACjE,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,UAAU,aAAa;IACrB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;CACd;AA0YD,qBAAa,aAAa;IAMxB,MAAM,CAAC,MAAM,CACX,MAAM,EAAE,aAAa,EACrB,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,EACnB,UAAU,CAAC,EAAE,MAAM,EACnB,WAAW,CAAC,EAAE,MAAM,EACpB,mBAAmB,CAAC,EAAE,MAAM,EAC5B,UAAU,CAAC,EAAE,eAAe,EAAE,EAC9B,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,EACpD,kBAAkB,CAAC,EAAE,iBAAiB,EAAE,EACxC,iBAAiB,CAAC,EAAE,iBAAiB,EAAE,EACvC,cAAc,CAAC,EAAE,SAAS,GAAG,aAAa,GAAG,QAAQ,EACrD,UAAU,CAAC,EAAE,OAAO,GACnB,YAAY;IAoDf,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAwC/B,OAAO,CAAC,MAAM,CAAC,WAAW;IAwS1B,OAAO,CAAC,MAAM,CAAC,2BAA2B;IA8E1C,OAAO,CAAC,MAAM,CAAC,wBAAwB;IA8EvC,OAAO,CAAC,MAAM,CAAC,sBAAsB;IAoErC,OAAO,CAAC,MAAM,CAAC,kBAAkB;IAsBjC,OAAO,CAAC,MAAM,CAAC,qBAAqB;IA6DpC,OAAO,CAAC,MAAM,CAAC,qBAAqB;IAuCpC,OAAO,CAAC,MAAM,CAAC,wBAAwB;IA6CvC,OAAO,CAAC,MAAM,CAAC,sBAAsB;IAqCrC,OAAO,CAAC,MAAM,CAAC,sBAAsB;IAqGrC,OAAO,CAAC,MAAM,CAAC,kBAAkB;IA0BjC,OAAO,CAAC,MAAM,CAAC,uBAAuB;IAatC,MAAM,CAAC,eAAe,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAgBlD,MAAM,CAAC,cAAc,CACnB,MAAM,EAAE,aAAa,EACrB,MAAM,EAAE,MAAM,EACd,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,GAClB,YAAY;IA0Cf,0FAA0F;IAC1F,MAAM,CAAC,eAAe,IAAI,IAAI;CAG/B"}
+{"version":3,"file":"SourceUpdater.d.ts","sourceRoot":"","sources":["../../src/services/SourceUpdater.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAE/C,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,cAAc,CAAC;AA4BtD,UAAU,YAAY;IACpB,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,iFAAiF;IACjF,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,wEAAwE;IACxE,cAAc,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IAChD,iEAAiE;IACjE,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,UAAU,aAAa;IACrB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;CACd;AAqZD,qBAAa,aAAa;IAMxB,MAAM,CAAC,MAAM,CACX,MAAM,EAAE,aAAa,EACrB,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,EACnB,UAAU,CAAC,EAAE,MAAM,EACnB,WAAW,CAAC,EAAE,MAAM,EACpB,mBAAmB,CAAC,EAAE,MAAM,EAC5B,UAAU,CAAC,EAAE,eAAe,EAAE,EAC9B,aAAa,CAAC,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,EACpD,kBAAkB,CAAC,EAAE,iBAAiB,EAAE,EACxC,iBAAiB,CAAC,EAAE,iBAAiB,EAAE,EACvC,cAAc,CAAC,EAAE,SAAS,GAAG,aAAa,GAAG,QAAQ,EACrD,UAAU,CAAC,EAAE,OAAO,GACnB,YAAY;IAoDf,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAwC/B,OAAO,CAAC,MAAM,CAAC,WAAW;IAkT1B,OAAO,CAAC,MAAM,CAAC,2BAA2B;IA8E1C,OAAO,CAAC,MAAM,CAAC,wBAAwB;IA8EvC,OAAO,CAAC,MAAM,CAAC,sBAAsB;IAoErC,OAAO,CAAC,MAAM,CAAC,kBAAkB;IAsBjC,OAAO,CAAC,MAAM,CAAC,qBAAqB;IA6DpC,OAAO,CAAC,MAAM,CAAC,qBAAqB;IAuCpC,OAAO,CAAC,MAAM,CAAC,wBAAwB;IA6CvC,OAAO,CAAC,MAAM,CAAC,sBAAsB;IAqCrC,OAAO,CAAC,MAAM,CAAC,sBAAsB;IAqGrC,OAAO,CAAC,MAAM,CAAC,kBAAkB;IA0BjC,OAAO,CAAC,MAAM,CAAC,uBAAuB;IAatC,MAAM,CAAC,eAAe,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAgBlD,MAAM,CAAC,cAAc,CACnB,MAAM,EAAE,aAAa,EACrB,MAAM,EAAE,MAAM,EACd,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,GAClB,YAAY;IA0Cf,0FAA0F;IAC1F,MAAM,CAAC,eAAe,IAAI,IAAI;CAG/B"}