sela-core 1.0.7 → 1.0.9

package/dist/services/SelectorSanitizer.js

@@ -0,0 +1,318 @@
+"use strict";
+// src/services/SelectorSanitizer.ts
+//
+// THE ABSOLUTE RULE (Ticket #2 hardening):
+// Playwright's `internal:` engine selectors (e.g.
+// `internal:role=button[name="Update System"i]`) are a PRIVATE runtime
+// serialization. They must NEVER be written verbatim into user source — not
+// as a `.locator()` string argument, and not into a string-literal constant
+// consumed by `.locator()`. Although the engine happens to accept them at
+// runtime, they are undocumented, version-fragile, and leak internals into
+// the developer's clean code (bad DX, the exact bug this module prevents).
+//
+// This module is the single source of truth for:
+//   1. DETECTING an internal engine selector (`isInternalEngineSelector`).
+//   2. TRANSLATING it to the idiomatic public method form
+//      (`internalToMethodExpression`) — `getByRole(...)`, `getByText(...)`, …
+//   3. SANITIZING a full locator expression (`sanitizeLocatorExpression`):
+//      rewriting `recv.locator("internal:…")` → `recv.getByRole(…)`, and
+//      reporting whether anything internal remains (unconvertible ⇒ unsafe).
+//
+// Pure, dependency-light (ts-morph only for expression rewriting). Every
+// write-head (InitializerUpdater, MutationApplier, SourceUpdater regex gate)
+// consults this so the invariant holds across all heal paths.
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.isInternalEngineSelector = isInternalEngineSelector;
+exports.parseInternalSelector = parseInternalSelector;
+exports.buildMethodCall = buildMethodCall;
+exports.internalToMethodExpression = internalToMethodExpression;
+exports.internalToPublicSelectorString = internalToPublicSelectorString;
+exports.sanitizeLocatorExpression = sanitizeLocatorExpression;
+const ts_morph_1 = require("ts-morph");
+// ═══════════════════════════════════════════════════════════════════
+// DETECTION
+// ═══════════════════════════════════════════════════════════════════
+/**
+ * True when `value` is (or contains, in a `>>` chain) a Playwright internal
+ * engine selector. These start with the `internal:` engine prefix.
+ */
+function isInternalEngineSelector(value) {
+    if (!value)
+        return false;
+    return value
+        .split(">>")
+        .some((seg) => seg.trim().toLowerCase().startsWith("internal:"));
+}
+/**
+ * Parse a SINGLE internal selector segment into a method descriptor, or null
+ * when it is not a recognised internal form (so callers REFUSE rather than
+ * guess). Mirrors the engine→public mapping used by ASTSourceUpdater.
+ */
+function parseInternalSelector(selector) {
+    // Match against the raw (trimmed) selector. The case-insensitivity `i` flag
+    // is significant per-pattern (trailing for text/label, inside `[…]` for
+    // role), so it must NOT be stripped globally up front.
+    const s = selector.trim();
+    // internal:role=button[name="X"i][level=2]
+    const roleMatch = s.match(/^internal:role=([a-zA-Z]+)(\[.*\])?$/);
+    if (roleMatch) {
+        const desc = {
+            method: "getByRole",
+            primaryArg: roleMatch[1],
+        };
+        const attrs = roleMatch[2] ?? "";
+        const nameMatch = attrs.match(/name="([^"]*)"(i)?/);
+        const options = {};
+        if (nameMatch) {
+            options.name = nameMatch[1];
+            if (!nameMatch[2])
+                options.exact = true; // no `i` flag → exact match
+        }
+        const levelMatch = attrs.match(/level=(\d+)/);
+        if (levelMatch)
+            options.level = Number(levelMatch[1]);
+        if (Object.keys(options).length)
+            desc.options = options;
+        return desc;
+    }
+    // internal:label="X"i  |  internal:label=X
+    const labelQuoted = s.match(/^internal:label="([^"]*)"(i)?$/);
+    if (labelQuoted) {
+        return labelQuoted[2]
+            ? { method: "getByLabel", primaryArg: labelQuoted[1] }
+            : { method: "getByLabel", primaryArg: labelQuoted[1], options: { exact: true } };
+    }
+    const labelBare = s.match(/^internal:label=([^[]+)$/);
+    if (labelBare) {
+        return { method: "getByLabel", primaryArg: labelBare[1].trim() };
+    }
+    // internal:text="X"i
+    const textMatch = s.match(/^internal:text="([^"]*)"(i)?$/);
+    if (textMatch) {
+        return textMatch[2]
+            ? { method: "getByText", primaryArg: textMatch[1] }
+            : { method: "getByText", primaryArg: textMatch[1], options: { exact: true } };
+    }
+    // internal:attr=[placeholder="X"] / [alt="X"] / [title="X"]
+    const placeholder = s.match(/^internal:attr=\[placeholder="([^"]*)"\]$/);
+    if (placeholder)
+        return { method: "getByPlaceholder", primaryArg: placeholder[1] };
+    const alt = s.match(/^internal:attr=\[alt="([^"]*)"\]$/);
+    if (alt)
+        return { method: "getByAltText", primaryArg: alt[1] };
+    const title = s.match(/^internal:attr=\[title="([^"]*)"\]$/);
+    if (title)
+        return { method: "getByTitle", primaryArg: title[1] };
+    // internal:testid=X  |  internal:attr=[data-testid="X"]
+    const testId = s.match(/^internal:testid=(.+)$/) ||
+        s.match(/^internal:attr=\[data-testid="([^"]*)"\]$/);
+    if (testId)
+        return { method: "getByTestId", primaryArg: testId[1].trim() };
+    return null; // unrecognised internal form → caller must refuse
+}
+// ═══════════════════════════════════════════════════════════════════
+// BUILD  (method descriptor → idiomatic call text)
+// ═══════════════════════════════════════════════════════════════════
+function q(s) {
+    // Prefer double quotes, escaping any embedded ones.
+    return `"${s.replace(/"/g, '\\"')}"`;
+}
+function buildMethodCall(receiver, desc) {
+    switch (desc.method) {
+        case "getByRole": {
+            const opts = [];
+            if (desc.options?.name !== undefined)
+                opts.push(`name: ${q(String(desc.options.name))}`);
+            if (desc.options?.exact === true)
+                opts.push(`exact: true`);
+            if (desc.options?.level !== undefined)
+                opts.push(`level: ${desc.options.level}`);
+            const optStr = opts.length ? `, { ${opts.join(", ")} }` : "";
+            return `${receiver}.getByRole(${q(desc.primaryArg)}${optStr})`;
+        }
+        case "getByText":
+        case "getByLabel": {
+            const exact = desc.options?.exact === true ? `, { exact: true }` : "";
+            return `${receiver}.${desc.method}(${q(desc.primaryArg)}${exact})`;
+        }
+        case "getByPlaceholder":
+        case "getByAltText":
+        case "getByTitle":
+        case "getByTestId":
+            return `${receiver}.${desc.method}(${q(desc.primaryArg)})`;
+        default:
+            return `${receiver}.locator(${q(desc.primaryArg)})`;
+    }
+}
+// ═══════════════════════════════════════════════════════════════════
+// TRANSLATE  (full internal selector string → method expression)
+// ═══════════════════════════════════════════════════════════════════
+/**
+ * Translate an internal selector string into an idiomatic Playwright method
+ * expression rooted at `receiver`. Handles `>>` chains by chaining each
+ * segment. Returns null when ANY segment is an unrecognised internal form
+ * (frame `internal:control=…` plumbing, unknown engines) — the caller must
+ * then refuse rather than emit something fragile.
+ */
+function internalToMethodExpression(selector, receiver = "page") {
+    const segments = selector
+        .split(">>")
+        .map((s) => s.trim())
+        .filter(Boolean);
+    if (segments.length === 0)
+        return null;
+    let expr = receiver;
+    for (const seg of segments) {
+        if (seg.toLowerCase().startsWith("internal:")) {
+            const desc = parseInternalSelector(seg);
+            if (!desc)
+                return null; // unconvertible internal form
+            expr = buildMethodCall(expr, desc);
+        }
+        else {
+            // Plain CSS / text segment — stays a public .locator() string.
+            expr = `${expr}.locator(${q(seg)})`;
+        }
+    }
+    return expr;
+}
+// ═══════════════════════════════════════════════════════════════════
+// TRANSLATE  (internal selector → valid PUBLIC .locator() string)
+// ═══════════════════════════════════════════════════════════════════
+function escAttr(s) {
+    return s.replace(/"/g, '\\"');
+}
+/**
+ * Convert a single internal segment to a valid PUBLIC `.locator()` selector
+ * string (the documented selector engines: `role=`, `text=`, css attribute).
+ * Returns null for forms that have NO public string engine (label is an
+ * associative relationship, not an attribute — only `getByLabel(...)` works).
+ */
+function segmentToPublicSelector(seg) {
+    const desc = parseInternalSelector(seg);
+    if (!desc)
+        return null;
+    switch (desc.method) {
+        case "getByRole": {
+            const attrs = [];
+            if (desc.options?.name !== undefined)
+                attrs.push(`name="${escAttr(String(desc.options.name))}"`);
+            if (desc.options?.level !== undefined)
+                attrs.push(`level=${desc.options.level}`);
+            const attrStr = attrs.length ? `[${attrs.join("][")}]` : "";
+            return `role=${desc.primaryArg}${attrStr}`;
+        }
+        case "getByText":
+            // Quoted text engine = exact-ish; unquoted = substring (i-flag default).
+            return desc.options?.exact
+                ? `text="${escAttr(desc.primaryArg)}"`
+                : `text=${desc.primaryArg}`;
+        case "getByPlaceholder":
+            return `[placeholder="${escAttr(desc.primaryArg)}"]`;
+        case "getByAltText":
+            return `[alt="${escAttr(desc.primaryArg)}"]`;
+        case "getByTitle":
+            return `[title="${escAttr(desc.primaryArg)}"]`;
+        case "getByTestId":
+            return `[data-testid="${escAttr(desc.primaryArg)}"]`;
+        case "getByLabel":
+        default:
+            return null; // no public string-engine equivalent
+    }
+}
+/**
+ * Convert an internal engine selector into a VALID PUBLIC `.locator()` string,
+ * preserving `>>` chaining. Returns null when ANY internal segment has no
+ * public string engine (so the caller must method-shift or refuse instead of
+ * stuffing an unusable string into a constant).
+ *
+ * Example: `internal:role=button[name="Update System"i]`
+ *        → `role=button[name="Update System"]`
+ */
+function internalToPublicSelectorString(selector) {
+    const segments = selector
+        .split(">>")
+        .map((s) => s.trim())
+        .filter(Boolean);
+    if (segments.length === 0)
+        return null;
+    const out = [];
+    for (const seg of segments) {
+        if (seg.toLowerCase().startsWith("internal:")) {
+            const pub = segmentToPublicSelector(seg);
+            if (!pub)
+                return null;
+            out.push(pub);
+        }
+        else {
+            out.push(seg); // already a public css/text segment
+        }
+    }
+    return out.join(" >> ");
+}
+let probeCounter = 0;
+/**
+ * Rewrite any `recv.locator("internal:…")` / `recv.frameLocator("internal:…")`
+ * inside `expr` to its idiomatic method form. Returns `safe:false` when an
+ * internal selector remains that cannot be translated, so the caller refuses
+ * to write it.
+ */
+function sanitizeLocatorExpression(expr) {
+    // Fast path: nothing internal anywhere.
+    if (!expr.includes("internal:")) {
+        return { text: expr, sanitized: false, safe: true };
+    }
+    const project = new ts_morph_1.Project({
+        useInMemoryFileSystem: true,
+        compilerOptions: { allowJs: true, noEmit: true, skipLibCheck: true, noResolve: true },
+    });
+    let current = expr;
+    let sanitized = false;
+    // Iteratively rewrite the first internal locator call until none remain.
+    // replaceWithText invalidates nodes, so we re-parse each pass.
+    for (let pass = 0; pass < 32; pass++) {
+        let sf;
+        try {
+            sf = project.createSourceFile(`__san${probeCounter++}.ts`, `const __x = ${current};`, {
+                overwrite: true,
+            });
+        }
+        catch {
+            return { text: expr, sanitized, safe: false };
+        }
+        const call = sf
+            .getDescendantsOfKind(ts_morph_1.SyntaxKind.CallExpression)
+            .find((c) => {
+            const callee = c.getExpression();
+            if (!ts_morph_1.Node.isPropertyAccessExpression(callee))
+                return false;
+            const m = callee.getName();
+            if (m !== "locator" && m !== "frameLocator")
+                return false;
+            const arg = c.getArguments()[0];
+            if (!arg)
+                return false;
+            const lit = ts_morph_1.Node.isStringLiteral(arg) || ts_morph_1.Node.isNoSubstitutionTemplateLiteral(arg)
+                ? arg.getLiteralValue()
+                : null;
+            return lit !== null && isInternalEngineSelector(lit);
+        });
+        if (!call)
+            break; // no more internal locator calls
+        const callee = call.getExpression();
+        const receiver = callee.getExpression().getText();
+        const arg = call.getArguments()[0];
+        const internalSel = arg.getLiteralValue();
+        const methodExpr = internalToMethodExpression(internalSel, receiver);
+        if (!methodExpr) {
+            // Unconvertible — leave as-is and report unsafe.
+            return { text: current, sanitized, safe: false };
+        }
+        call.replaceWithText(methodExpr);
+        current = sf.getVariableDeclaration("__x").getInitializer().getText();
+        sanitized = true;
+    }
+    // Final integrity check: nothing internal must survive.
+    const safe = !isInternalEngineSelector(current) && !current.includes("internal:");
+    return { text: current, sanitized, safe };
+}

package/dist/services/SnapshotService.js

@@ -39,7 +39,7 @@ const path = __importStar(require("path"));
 class SnapshotService {
     baseDir;
     constructor() {
-        this.baseDir = path.join(process.cwd(), "fixwright-snapshots");
+        this.baseDir = path.join(process.cwd(), "sela-snapshots");
         if (!fs.existsSync(this.baseDir)) {
             fs.mkdirSync(this.baseDir, { recursive: true });
         }
@@ -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;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;IAiT1B,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;AA6BtD,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;IAsU1B,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"}