sela-core 1.0.7 → 1.0.8

package/dist/services/ASTSourceUpdater.js

@@ -50,7 +50,7 @@ const CrossFileHealer_1 = require("./CrossFileHealer");
 const HealingAdvisory_1 = require("./HealingAdvisory");
 const ChainValidator_1 = require("./ChainValidator");
 // ═══════════════════════════════════════════════════════════════════
-// FIX #1 — normalizeSelector utility
+// FIX #1 - normalizeSelector utility
 // ═══════════════════════════════════════════════════════════════════
 function normalizeSelector(s) {
     return s
@@ -70,7 +70,7 @@ function normalizeSelector(s) {
 // of "varName", or null otherwise.
 //
 // Why this matters:
-//   const shadowBtn = shadowBtn.getByRole(…)   ← CIRCULAR — must be caught
+//   const shadowBtn = shadowBtn.getByRole(…)   ← CIRCULAR - must be caught
 //   const shadowBtn = page.getByRole(…)        ← CORRECT
 //
 // We walk UP the ancestor chain from the call node looking for a
@@ -83,7 +83,7 @@ function getEnclosingVariableDeclaration(node) {
         if (ts_morph_1.Node.isVariableDeclaration(current)) {
             return current;
         }
-        // Stop climbing past statement boundaries — we only want the immediate
+        // Stop climbing past statement boundaries - we only want the immediate
         // enclosing declaration, not a declaration somewhere further up the tree.
         if (ts_morph_1.Node.isBlock(current) ||
             ts_morph_1.Node.isSourceFile(current) ||
@@ -121,7 +121,7 @@ function isInsideOwnInitializer(callNode, varName) {
 function resolveSafeReceiver(proposedReceiver, rootReceiver, callNode) {
     if (isInsideOwnInitializer(callNode, proposedReceiver)) {
         console.log(`[ScopeGuard] 🔒 Circular reference prevented: ` +
-            `"${proposedReceiver}" is being initialized — ` +
+            `"${proposedReceiver}" is being initialized - ` +
             `falling back to root receiver "${rootReceiver}"`);
         return rootReceiver;
     }
@@ -554,7 +554,7 @@ class MethodMapper {
         for (const seg of segments) {
             const callText = MethodMapper.build(currentReceiver, seg);
             if (callText === null) {
-                console.warn(`[MethodMapper] Cannot build segment type "${seg.type}" — aborting chain`);
+                console.warn(`[MethodMapper] Cannot build segment type "${seg.type}" - aborting chain`);
                 return null;
             }
             const methodPart = callText.slice(currentReceiver.length);
@@ -617,7 +617,7 @@ function deduplicateChainSegments(aiSegments, contextSegments) {
     if (overlap === 0)
         return { deduped: aiSegments, stripped: 0 };
     console.log(`[ChainDedup] ✂️  Stripping ${overlap}/${contextSegments.length} duplicate leading ` +
-        `segment(s) — context already encodes them`);
+        `segment(s) - context already encodes them`);
     return { deduped: aiSegments.slice(overlap), stripped: overlap };
 }
 const PAGE_ROOT_NAMES = new Set([
@@ -731,15 +731,15 @@ function resolveVariableToSegments(sourceFile, varName, beforeLine, depth = 0) {
 // Module-level write gate shared between `ASTSourceUpdater` and the
 // internal `NodeTransformationStrategy`. Centralises three concerns
 // every `saveSync` call site needs:
-//   1. `DryRunGuard.active()` short-circuit — keeps the mutation in
+//   1. `DryRunGuard.active()` short-circuit - keeps the mutation in
 //      memory but suppresses the on-disk write under SELA_DRY_RUN.
-//   2. `sharedWorkspaceSnapshot.preWrite()` — captures `contentBefore`
+//   2. `sharedWorkspaceSnapshot.preWrite()` - captures `contentBefore`
 //      so the Clean-Room ledger / Local DX rollback can rewind.
-//   3. `sharedWorkspaceSnapshot.postWrite()` — refreshes `contentAfter`
+//   3. `sharedWorkspaceSnapshot.postWrite()` - refreshes `contentAfter`
 //      so `getMutatedSnapshots()` (driving the pending-prompts ledger
 //      and the interactive CLI prompt) sees the new disk state.
 //
-// Strategies do NOT receive the persistence service via DI — they call
+// Strategies do NOT receive the persistence service via DI - they call
 // this helper directly. Same singleton, zero coupling, one code path.
 // ═══════════════════════════════════════════════════════════════════
 function persistSourceFile(sourceFile) {
@@ -753,7 +753,7 @@ function persistSourceFile(sourceFile) {
     WorkspaceSnapshotService_1.sharedWorkspaceSnapshot.postWrite(fp);
 }
 // ═══════════════════════════════════════════════════════════════════
-// NODE TRANSFORMATION STRATEGY — Scope-Aware, Zero-Redundancy
+// NODE TRANSFORMATION STRATEGY - Scope-Aware, Zero-Redundancy
 // ═══════════════════════════════════════════════════════════════════
 class NodeTransformationStrategy {
     chainValidator;
@@ -823,8 +823,10 @@ class NodeTransformationStrategy {
         // When AI's first segment TYPE matches the receiver variable's defining type
         // but with a different selector, the AI is replacing the receiver context.
         // Strategy: update the receiver variable's definition, then use remaining
-        // segments with the receiver var — preserving developer's variable structure.
-        if (!forcedReceiver && discoveredAstRoot && !PAGE_ROOT_NAMES.has(discoveredAstRoot)) {
+        // segments with the receiver var - preserving developer's variable structure.
+        if (!forcedReceiver &&
+            discoveredAstRoot &&
+            !PAGE_ROOT_NAMES.has(discoveredAstRoot)) {
             const typeCheckRes = resolveVariableToSegments(sourceFile, discoveredAstRoot, caller.line);
             if (typeCheckRes.segments.length > 0 && workingSegments.length > 0) {
                 const receiverDefType = typeCheckRes.segments[typeCheckRes.segments.length - 1].type;
@@ -923,8 +925,9 @@ class NodeTransformationStrategy {
             // to prevent.  Skip the candidate; the semantic gate owns this case.
             const firstArg = call.getArguments()[0];
             if (firstArg &&
-                (ts_morph_1.Node.isIdentifier(firstArg) || ts_morph_1.Node.isPropertyAccessExpression(firstArg))) {
-                console.log(`[NodeTransform] 🛑 skip — first arg is ${firstArg.getKindName()} ` +
+                (ts_morph_1.Node.isIdentifier(firstArg) ||
+                    ts_morph_1.Node.isPropertyAccessExpression(firstArg))) {
+                console.log(`[NodeTransform] 🛑 skip - first arg is ${firstArg.getKindName()} ` +
                     `"${firstArg.getText()}" (semantic dispatch owns definition-site mutations)`);
                 continue;
             }
@@ -936,12 +939,12 @@ class NodeTransformationStrategy {
             // Before we build code, we must verify that the proposed receiver
             // does not create a circular self-reference.
             //
-            // Case A — forcedReceiver is set (strip-and-force path):
+            // Case A - forcedReceiver is set (strip-and-force path):
             //   Check if the chainTop lives inside the initializer of that var.
             //   If yes, fall back to the actual AST root (e.g. "page").
             //
-            // Case B — no forced receiver:
-            //   Use astRoot directly — it was walked up from the call node and
+            // Case B - no forced receiver:
+            //   Use astRoot directly - it was walked up from the call node and
             //   is always either "page" or a stable variable reference.
             // ══════════════════════════════════════════════════════════════
             let effectiveReceiver;
@@ -1191,7 +1194,7 @@ class NodeTransformationStrategy {
     }
     updateReceiverDefinition(sourceFile, varName, segType, newSelector, callerLine) {
         // Collect all matching declarations, then pick the closest one that
-        // appears strictly BEFORE callerLine — prevents accidentally updating
+        // appears strictly BEFORE callerLine - prevents accidentally updating
         // a same-named variable in a different test block that happens to be
         // within the 100-line distance window.
         const declarations = sourceFile.getDescendantsOfKind(ts_morph_1.SyntaxKind.VariableDeclaration);
@@ -1302,7 +1305,7 @@ class NodeTransformationStrategy {
                         .map((d) => `[TS${d.getCode()}] ${d.getMessageText()}`)
                         .join(", "));
                 if (/\.(getBy\w+|frameLocator)\s*\(/.test(code)) {
-                    console.warn(`[NodeTransform] ⚠️  Semantic method detected — bypassing validation`);
+                    console.warn(`[NodeTransform] ⚠️  Semantic method detected - bypassing validation`);
                     return true;
                 }
                 return false;
@@ -1562,7 +1565,7 @@ class ASTSourceUpdater {
     /**
      * Centralised write hook for every ts-morph `saveSync` call in this class.
      * When `SELA_DRY_RUN` is active the AST mutation stays in memory but the
-     * on-disk file is left untouched — the heal pipeline can still compute a
+     * on-disk file is left untouched - the heal pipeline can still compute a
      * candidate diff via `sourceFile.getFullText()` and the report layer
      * tags the event with `dryRun: true`.
      */
@@ -1579,7 +1582,7 @@ class ASTSourceUpdater {
     // BEFORE the LLM call and record a FailedEvent in the "Skipped by
     // Developer" category.
     //
-    // Implementation note — ts-morph's `Node.getLeadingCommentRanges()`
+    // Implementation note - ts-morph's `Node.getLeadingCommentRanges()`
     // returns `ts.CommentRange[]` (offsets, not text). We slice the full
     // source text by `(pos, end)` to recover the raw comment for each
     // range, matching both `//` line comments and `/* */` block comments
@@ -1643,7 +1646,7 @@ class ASTSourceUpdater {
         return { failFast: false };
     }
     /**
-     * Convenience helper — runs `scanFailFastDirective` and throws the
+     * Convenience helper - runs `scanFailFastDirective` and throws the
      * canonical SelaError when the directive is present. Used by SelaEngine
      * as the very first gate inside `heal()`.
      */
@@ -1790,7 +1793,7 @@ class ASTSourceUpdater {
             sourceFile = this.project.addSourceFileAtPath(filePath);
         }
         if (caller.line <= 0) {
-            console.log(`[ASTUpdater] ⚠️  line ${caller.line} invalid — Attempting Global Template Strategy`);
+            console.log(`[ASTUpdater] ⚠️  line ${caller.line} invalid - Attempting Global Template Strategy`);
             const globalResult = this.strategyTemplateLiteral(sourceFile, caller, oldSelector, newSelector);
             if (globalResult && globalResult.success) {
                 if (contentChange)
@@ -1814,7 +1817,7 @@ class ASTSourceUpdater {
         // node when they differ) for `// sela-fail-fast`. When present, abort
         // all heal work with SelaError(AST_HEAL_DISABLED_BY_DIRECTIVE) so the
         // engine records a FailedEvent in the "Skipped by Developer" bucket.
-        // Defense-in-depth — SelaEngine also calls this at the very start of
+        // Defense-in-depth - SelaEngine also calls this at the very start of
         // heal() to skip the LLM round-trip entirely.
         this.enforceFailFastDirective(caller);
         const suspectIdentifier = statement
@@ -1827,7 +1830,7 @@ class ASTSourceUpdater {
         // HARD EXIT: any non-null result (success OR failure) stops all
         // subsequent AST strategies.  When the locator argument is an
         // Identifier, PropertyAccess, or TemplateLiteral, the gate owns
-        // the mutation — NT/H must never override the symbolic reference.
+        // the mutation - NT/H must never override the symbolic reference.
         const semanticResult = this.semanticDispatch(sourceFile, caller, oldSelector, newSelector);
         if (semanticResult !== null) {
             if (semanticResult.success) {
@@ -1836,7 +1839,7 @@ class ASTSourceUpdater {
                 }
                 return this.broadcastAndReturn(semanticResult, oldSelector, newSelector, caller);
             }
-            // Failure (blast-radius block, trace failed, syntax error) — hard stop.
+            // Failure (blast-radius block, trace failed, syntax error) - hard stop.
             console.log(`[SemanticDispatch] 🚫 Hard stop (${semanticResult.strategy}): ${semanticResult.reason}`);
             return semanticResult;
         }
@@ -1885,7 +1888,7 @@ class ASTSourceUpdater {
         };
     }
     // ═══════════════════════════════════════════════════════════════
-    // STRATEGY H — Smart Chain (Legacy fallback)
+    // STRATEGY H - Smart Chain (Legacy fallback)
     // ═══════════════════════════════════════════════════════════════
     strategySmartChain(sourceFile, caller, oldSelector, oldSem, smartSegments, originalChainHint, suspectIdentifier) {
         console.log(`[ASTUpdater] 🔗 Strategy H v2: Smart Chain (${smartSegments.length} segments)`);
@@ -1959,13 +1962,14 @@ class ASTSourceUpdater {
             return b.chainDepth - a.chainDepth;
         });
         for (const { leafCall, chainTop, rootReceiver, chainDepth, originalText, } of candidates) {
-            // Identifier/PropertyAccess guard — mirrors the NT guard.
+            // Identifier/PropertyAccess guard - mirrors the NT guard.
             // Strategy H must never substitute a hardcoded string for a symbolic
             // reference; the semantic dispatch gate owns definition-site mutations.
             const firstArgH = leafCall.getArguments()[0];
             if (firstArgH &&
-                (ts_morph_1.Node.isIdentifier(firstArgH) || ts_morph_1.Node.isPropertyAccessExpression(firstArgH))) {
-                console.log(`[StrategyH] 🛑 skip — first arg is ${firstArgH.getKindName()} ` +
+                (ts_morph_1.Node.isIdentifier(firstArgH) ||
+                    ts_morph_1.Node.isPropertyAccessExpression(firstArgH))) {
+                console.log(`[StrategyH] 🛑 skip - first arg is ${firstArgH.getKindName()} ` +
                     `"${firstArgH.getText()}" (semantic dispatch owns definition-site mutations)`);
                 continue;
             }
@@ -2035,8 +2039,9 @@ class ASTSourceUpdater {
         // Block H-partial from substituting a literal for an Identifier/PropertyAccess arg.
         const targetFirstArg = targetCall.getArguments()[0];
         if (targetFirstArg &&
-            (ts_morph_1.Node.isIdentifier(targetFirstArg) || ts_morph_1.Node.isPropertyAccessExpression(targetFirstArg))) {
-            console.log(`[H-partial] 🛑 skip — arg is ${targetFirstArg.getKindName()} ` +
+            (ts_morph_1.Node.isIdentifier(targetFirstArg) ||
+                ts_morph_1.Node.isPropertyAccessExpression(targetFirstArg))) {
+            console.log(`[H-partial] 🛑 skip - arg is ${targetFirstArg.getKindName()} ` +
                 `"${targetFirstArg.getText()}" (semantic dispatch owns this)`);
             return null;
         }
@@ -2072,7 +2077,7 @@ class ASTSourceUpdater {
         }
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY F — Chain Collapse
+    // STRATEGY F - Chain Collapse
     // ─────────────────────────────────────────────────────────────
     strategyChainCollapse(sourceFile, caller, oldSelector, newSelector, oldSem, newSem) {
         console.log(`[ASTUpdater] ⛓️  Strategy F: Chain Collapse (upward traversal)`);
@@ -2138,7 +2143,7 @@ class ASTSourceUpdater {
         };
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY G — Assertion Healing
+    // STRATEGY G - Assertion Healing
     // ─────────────────────────────────────────────────────────────
     runStrategyG(sourceFile, caller, locatorVarName, contentChange) {
         if (!contentChange)
@@ -2170,7 +2175,7 @@ class ASTSourceUpdater {
         return null;
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY A — Frame Upstream Healing
+    // STRATEGY A - Frame Upstream Healing
     // ─────────────────────────────────────────────────────────────
     strategyFrameUpstream(sourceFile, caller, oldElementSelector, aiSegments, fullSelectorContext) {
         console.log(`[ASTUpdater] 🧠 Strategy A: Frame Upstream`);
@@ -2246,7 +2251,7 @@ class ASTSourceUpdater {
         };
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY B — Deep Symbol Resolution
+    // STRATEGY B - Deep Symbol Resolution
     // ─────────────────────────────────────────────────────────────
     strategyDeepSymbolResolution(sourceFile, caller, oldSelector, newSelector, oldSem, newSem, suspectIdentifier) {
         console.log(`[ASTUpdater] 🔭 Strategy B: Deep Symbol Resolution`);
@@ -2430,7 +2435,7 @@ class ASTSourceUpdater {
         };
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY C — Inline Semantic Fix
+    // STRATEGY C - Inline Semantic Fix
     // ─────────────────────────────────────────────────────────────
     strategyInlineSemanticFix(sourceFile, caller, oldSelector, newSelector, oldSem, newSem) {
         console.log(`[ASTUpdater] 🎯 Strategy C: Inline Semantic Fix`);
@@ -2488,7 +2493,7 @@ class ASTSourceUpdater {
         };
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY D — Template Literal
+    // STRATEGY D - Template Literal
     // ─────────────────────────────────────────────────────────────
     strategyTemplateLiteral(sourceFile, caller, oldSelector, newSelector) {
         console.log(`[ASTUpdater] 🔤 Strategy D: Template Literal`);
@@ -2545,7 +2550,7 @@ class ASTSourceUpdater {
         };
     }
     // ─────────────────────────────────────────────────────────────
-    // STRATEGY E — Semantic Fragment Scan
+    // STRATEGY E - Semantic Fragment Scan
     // ─────────────────────────────────────────────────────────────
     strategySemanticFragmentScan(sourceFile, caller, oldSelector, newSelector, oldSem, newSem) {
         console.log(`[ASTUpdater] 🔎 Strategy E: Semantic Fragment Scan`);
@@ -2666,7 +2671,7 @@ class ASTSourceUpdater {
     /**
      * Guard: returns false when replacing oldNode with proposedNewText would
      * substitute a hardcoded string literal for an Identifier or
-     * PropertyAccessExpression — which is the definition-site mutation bug.
+     * PropertyAccessExpression - which is the definition-site mutation bug.
      * All other replacements are allowed.
      */
     static isSafeReplacement(oldNode, proposedNewText) {
@@ -2690,7 +2695,7 @@ class ASTSourceUpdater {
                 firstArg.replaceWithText(`"${safe}"`);
             }
             else {
-                console.warn(`[ASTUpdater] ⚠️ replaceCallArgument: blocked unsafe replacement — ` +
+                console.warn(`[ASTUpdater] ⚠️ replaceCallArgument: blocked unsafe replacement - ` +
                     `arg is ${firstArg.getKindName()}, not a string literal. ` +
                     `Heal at definition site instead.`);
                 return false;
@@ -2992,10 +2997,10 @@ class ASTSourceUpdater {
                     shape === "PROPERTY_ACCESS" ||
                     shape === "TEMPLATE_LITERAL") {
                     console.log(`[SemanticDispatch] 🛑 hard-block: "${node.getText()}" is ${shape} ` +
-                        `but trace failed — preserving call site`);
+                        `but trace failed - preserving call site`);
                     return {
                         success: false,
-                        reason: `semantic guard: could not trace "${node.getText()}" — call site preserved`,
+                        reason: `semantic guard: could not trace "${node.getText()}" - call site preserved`,
                         strategy: "semantic-guard",
                     };
                 }
@@ -3018,21 +3023,21 @@ class ASTSourceUpdater {
             if (blast.risk === "high") {
                 this.advisoryBuffer.push({
                     code: HealingAdvisory_1.AdvisoryCode.BLAST_RADIUS_BLOCKED,
-                    message: `Mutation blocked — ${blast.testFunctionCount} tests reference this constant.`,
+                    message: `Mutation blocked - ${blast.testFunctionCount} tests reference this constant.`,
                     affectedFile: targetFile.getFilePath(),
                     affectedLine: targetNode.getStartLineNumber() - 1,
                     suggestedAction: "Review all referencing tests before changing this constant.",
                 });
                 return {
                     success: false,
-                    reason: `BlastRadius BLOCKED: ${blast.testFunctionCount} tests reference this constant — manual review required`,
+                    reason: `BlastRadius BLOCKED: ${blast.testFunctionCount} tests reference this constant - manual review required`,
                     strategy: "semantic-identifier",
                 };
             }
             if (blast.risk === "medium") {
                 this.advisoryBuffer.push({
                     code: HealingAdvisory_1.AdvisoryCode.BLAST_RADIUS_WARNING,
-                    message: `Mutated constant used in ${blast.testFunctionCount} tests — verify no regressions.`,
+                    message: `Mutated constant used in ${blast.testFunctionCount} tests - verify no regressions.`,
                     affectedFile: targetFile.getFilePath(),
                     affectedLine: targetNode.getStartLineNumber() - 1,
                     suggestedAction: "Run the full test suite to confirm all references still pass.",
@@ -3044,7 +3049,7 @@ class ASTSourceUpdater {
             if (!this.validateFileSyntax(targetFile)) {
                 return {
                     success: false,
-                    reason: "post-mutation syntax validation failed — reverted",
+                    reason: "post-mutation syntax validation failed - reverted",
                     strategy: "semantic-identifier",
                 };
             }
@@ -3054,7 +3059,10 @@ class ASTSourceUpdater {
                 strategy: "semantic-identifier",
                 lineUpdated: result.lineUpdated,
                 healedLocatorString: result.healedLocatorString,
-                definitionSite: { file: targetFile.getFilePath(), line: result.lineUpdated },
+                definitionSite: {
+                    file: targetFile.getFilePath(),
+                    line: result.lineUpdated,
+                },
                 blastRadius: blast.testFunctionCount,
             };
         }
@@ -3083,21 +3091,21 @@ class ASTSourceUpdater {
             if (blast.risk === "high") {
                 this.advisoryBuffer.push({
                     code: HealingAdvisory_1.AdvisoryCode.BLAST_RADIUS_BLOCKED,
-                    message: `Mutation blocked — ${blast.testFunctionCount} tests reference this property.`,
+                    message: `Mutation blocked - ${blast.testFunctionCount} tests reference this property.`,
                     affectedFile: targetFile.getFilePath(),
                     affectedLine: targetNode.getStartLineNumber() - 1,
                     suggestedAction: "Review all referencing tests before changing this property.",
                 });
                 return {
                     success: false,
-                    reason: `BlastRadius BLOCKED: ${blast.testFunctionCount} tests reference this property — manual review required`,
+                    reason: `BlastRadius BLOCKED: ${blast.testFunctionCount} tests reference this property - manual review required`,
                     strategy: "semantic-property-access",
                 };
             }
             if (blast.risk === "medium") {
                 this.advisoryBuffer.push({
                     code: HealingAdvisory_1.AdvisoryCode.BLAST_RADIUS_WARNING,
-                    message: `Mutated property used in ${blast.testFunctionCount} tests — verify no regressions.`,
+                    message: `Mutated property used in ${blast.testFunctionCount} tests - verify no regressions.`,
                     affectedFile: targetFile.getFilePath(),
                     affectedLine: targetNode.getStartLineNumber() - 1,
                     suggestedAction: "Run the full test suite to confirm all references still pass.",
@@ -3109,7 +3117,7 @@ class ASTSourceUpdater {
             if (!this.validateFileSyntax(targetFile)) {
                 return {
                     success: false,
-                    reason: "post-mutation syntax validation failed — reverted",
+                    reason: "post-mutation syntax validation failed - reverted",
                     strategy: "semantic-property-access",
                 };
             }
@@ -3119,7 +3127,10 @@ class ASTSourceUpdater {
                 strategy: "semantic-property-access",
                 lineUpdated: result.lineUpdated,
                 healedLocatorString: result.healedLocatorString,
-                definitionSite: { file: targetFile.getFilePath(), line: result.lineUpdated },
+                definitionSite: {
+                    file: targetFile.getFilePath(),
+                    line: result.lineUpdated,
+                },
                 blastRadius: blast.testFunctionCount,
             };
         }
@@ -3163,10 +3174,11 @@ class ASTSourceUpdater {
                     }
                 }
             }
-            if (patchResult.reason.includes("structural") || patchResult.reason.includes("not found")) {
+            if (patchResult.reason.includes("structural") ||
+                patchResult.reason.includes("not found")) {
                 this.advisoryBuffer.push({
                     code: HealingAdvisory_1.AdvisoryCode.MANUAL_REVIEW_TEMPLATE_COMPLEX,
-                    message: `Template literal requires structural change — automated patch skipped.`,
+                    message: `Template literal requires structural change - automated patch skipped.`,
                     affectedFile: sourceFile.getFilePath(),
                     affectedLine: node.getStartLineNumber() - 1,
                     suggestedAction: "Manually update the template literal and its callers.",
@@ -3180,7 +3192,7 @@ class ASTSourceUpdater {
         if (!this.validateFileSyntax(sourceFile)) {
             return {
                 success: false,
-                reason: "post-mutation syntax validation failed — reverted",
+                reason: "post-mutation syntax validation failed - reverted",
                 strategy: "semantic-template",
             };
         }
@@ -3208,7 +3220,8 @@ class ASTSourceUpdater {
             if (ts_morph_1.Node.isCallExpression(init)) {
                 return this.handleCallExpressionArg(init, sourceFile, oldSpanVal, newSpanVal);
             }
-            if (ts_morph_1.Node.isStringLiteral(init) || ts_morph_1.Node.isNoSubstitutionTemplateLiteral(init)) {
+            if (ts_morph_1.Node.isStringLiteral(init) ||
+                ts_morph_1.Node.isNoSubstitutionTemplateLiteral(init)) {
                 const result = InitializerUpdater_1.InitializerUpdater.apply(init, sourceFile, newSpanVal);
                 if (!result)
                     return null;
@@ -3219,7 +3232,10 @@ class ASTSourceUpdater {
                     strategy: "semantic-template",
                     lineUpdated: result.lineUpdated,
                     healedLocatorString: result.healedLocatorString,
-                    definitionSite: { file: sourceFile.getFilePath(), line: result.lineUpdated },
+                    definitionSite: {
+                        file: sourceFile.getFilePath(),
+                        line: result.lineUpdated,
+                    },
                 };
             }
             if (ts_morph_1.Node.isTemplateExpression(init)) {
@@ -3233,7 +3249,7 @@ class ASTSourceUpdater {
         if (fnAnalysis.kind === "complex") {
             this.advisoryBuffer.push({
                 code: HealingAdvisory_1.AdvisoryCode.MANUAL_REVIEW_CONDITIONAL_RETURN,
-                message: `Function has conditional/complex return — automated patch skipped. Reason: ${fnAnalysis.reason}`,
+                message: `Function has conditional/complex return - automated patch skipped. Reason: ${fnAnalysis.reason}`,
                 affectedFile: sourceFile.getFilePath(),
                 suggestedAction: "Manually update the function's return expression.",
             });
@@ -3246,7 +3262,7 @@ class ASTSourceUpdater {
             if (!this.validateFileSyntax(fnAnalysis.functionFile)) {
                 return {
                     success: false,
-                    reason: "post-mutation syntax validation failed — reverted",
+                    reason: "post-mutation syntax validation failed - reverted",
                     strategy: "semantic-call-expression",
                 };
             }
@@ -3256,7 +3272,10 @@ class ASTSourceUpdater {
                 strategy: "semantic-call-expression",
                 lineUpdated: result.lineUpdated,
                 healedLocatorString: result.healedLocatorString,
-                definitionSite: { file: fnAnalysis.functionFile.getFilePath(), line: result.lineUpdated },
+                definitionSite: {
+                    file: fnAnalysis.functionFile.getFilePath(),
+                    line: result.lineUpdated,
+                },
             };
         }
         if (fnAnalysis.kind === "template") {
@@ -3266,7 +3285,7 @@ class ASTSourceUpdater {
                 if (patchResult.reason.includes("structural")) {
                     this.advisoryBuffer.push({
                         code: HealingAdvisory_1.AdvisoryCode.MANUAL_REVIEW_TEMPLATE_COMPLEX,
-                        message: `Function template return requires structural change — automated patch skipped.`,
+                        message: `Function template return requires structural change - automated patch skipped.`,
                         affectedFile: fnAnalysis.functionFile.getFilePath(),
                         affectedLine: fnAnalysis.functionLine - 1,
                         suggestedAction: "Manually update the function's template literal.",
@@ -3280,7 +3299,7 @@ class ASTSourceUpdater {
             if (!this.validateFileSyntax(fnAnalysis.functionFile)) {
                 return {
                     success: false,
-                    reason: "post-mutation syntax validation failed — reverted",
+                    reason: "post-mutation syntax validation failed - reverted",
                     strategy: "semantic-call-expression",
                 };
             }
@@ -3290,7 +3309,10 @@ class ASTSourceUpdater {
                 strategy: "semantic-call-expression",
                 lineUpdated,
                 healedLocatorString: newSelector,
-                definitionSite: { file: fnAnalysis.functionFile.getFilePath(), line: lineUpdated },
+                definitionSite: {
+                    file: fnAnalysis.functionFile.getFilePath(),
+                    line: lineUpdated,
+                },
             };
         }
         return null;
@@ -3306,11 +3328,13 @@ class ASTSourceUpdater {
             return !ALLOWED_CODES.has(d.getCode());
         });
         if (syntaxErrors.length > 0) {
-            const msgs = syntaxErrors.map((d) => String(d.getMessageText())).join("; ");
-            console.warn(`[SemanticDispatch] ⚠️ syntax errors after mutation: ${msgs} — reverting`);
+            const msgs = syntaxErrors
+                .map((d) => String(d.getMessageText()))
+                .join("; ");
+            console.warn(`[SemanticDispatch] ⚠️ syntax errors after mutation: ${msgs} - reverting`);
             this.advisoryBuffer.push({
                 code: HealingAdvisory_1.AdvisoryCode.POST_MUTATION_TYPE_ERROR,
-                message: `Post-mutation type errors detected — mutation reverted. Errors: ${msgs}`,
+                message: `Post-mutation type errors detected - mutation reverted. Errors: ${msgs}`,
                 affectedFile: sourceFile.getFilePath(),
                 suggestedAction: "Manually inspect and fix the type errors introduced by this heal.",
             });

package/dist/services/ArgumentTypeAnalyzer.js

@@ -20,7 +20,7 @@ exports.ALL_LOCATOR_METHODS = new Set([
 // ArgumentTypeAnalyzer
 //
 // Classifies the first argument of a Playwright locator CallExpression.
-// Stateless — no file I/O, no side effects.
+// Stateless - no file I/O, no side effects.
 // ═══════════════════════════════════════════════════════════════════
 class ArgumentTypeAnalyzer {
     /**
@@ -64,28 +64,28 @@ class ArgumentTypeAnalyzer {
     // Private helpers
     // ─────────────────────────────────────────────────────────────────
     static classifyNode(node) {
-        // Plain string — value is self-contained in this node.
+        // Plain string - value is self-contained in this node.
         if (ts_morph_1.Node.isStringLiteral(node) ||
             ts_morph_1.Node.isNoSubstitutionTemplateLiteral(node)) {
             return "STRING_LITERAL";
         }
-        // Template with ${...} spans — static segments are healable, spans are not.
+        // Template with ${...} spans - static segments are healable, spans are not.
         if (ts_morph_1.Node.isTemplateExpression(node)) {
             return "TEMPLATE_LITERAL";
         }
-        // Bare variable name — must trace to declaration.
+        // Bare variable name - must trace to declaration.
         if (ts_morph_1.Node.isIdentifier(node)) {
             return "IDENTIFIER";
         }
-        // OBJECT.property — must trace to object literal property value.
+        // OBJECT.property - must trace to object literal property value.
         if (ts_morph_1.Node.isPropertyAccessExpression(node)) {
             return "PROPERTY_ACCESS";
         }
-        // fn(...) — must enter function body to find return value.
+        // fn(...) - must enter function body to find return value.
         if (ts_morph_1.Node.isCallExpression(node)) {
             return "CALL_EXPRESSION";
         }
-        // Binary expressions, conditionals, spreads, etc. — not healable here.
+        // Binary expressions, conditionals, spreads, etc. - not healable here.
         return "COMPLEX";
     }
 }

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

@@ -1 +1 @@
-{"version":3,"file":"BlastRadiusAnalyzer.d.ts","sourceRoot":"","sources":["../../src/services/BlastRadiusAnalyzer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAc,MAAM,UAAU,CAAC;AAMjE,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC;AAElD,MAAM,WAAW,iBAAiB;IAChC,eAAe,EAAE,MAAM,CAAC;IACxB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,aAAa,EAAE,MAAM,EAAE,CAAC;IACxB,IAAI,EAAE,SAAS,CAAC;CACjB;AAWD,qBAAa,mBAAmB;IAC9B,MAAM,CAAC,OAAO,CACZ,eAAe,EAAE,IAAI,EACrB,WAAW,EAAE,UAAU,EACvB,QAAQ,EAAE,OAAO,GAChB,iBAAiB;IAgBpB,OAAO,CAAC,MAAM,CAAC,SAAS;IAwDxB,OAAO,CAAC,MAAM,CAAC,WAAW;IAY1B,OAAO,CAAC,MAAM,CAAC,UAAU;CAiB1B"}
+{"version":3,"file":"BlastRadiusAnalyzer.d.ts","sourceRoot":"","sources":["../../src/services/BlastRadiusAnalyzer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,UAAU,EAAE,OAAO,EAAc,MAAM,UAAU,CAAC;AAMjE,MAAM,MAAM,SAAS,GAAG,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC;AAElD,MAAM,WAAW,iBAAiB;IAChC,eAAe,EAAE,MAAM,CAAC;IACxB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,aAAa,EAAE,MAAM,EAAE,CAAC;IACxB,IAAI,EAAE,SAAS,CAAC;CACjB;AAWD,qBAAa,mBAAmB;IAC9B,MAAM,CAAC,OAAO,CACZ,eAAe,EAAE,IAAI,EACrB,WAAW,EAAE,UAAU,EACvB,QAAQ,EAAE,OAAO,GAChB,iBAAiB;IAgBpB,OAAO,CAAC,MAAM,CAAC,SAAS;IA2DxB,OAAO,CAAC,MAAM,CAAC,WAAW;IAY1B,OAAO,CAAC,MAAM,CAAC,UAAU;CAiB1B"}

package/dist/services/BlastRadiusAnalyzer.js

@@ -8,7 +8,7 @@ const ts_morph_1 = require("ts-morph");
 // Before mutating a definition-site node, determine how many tests
 // reference it.  Uses ts-morph's findReferences() for cross-file
 // accuracy.  On any analysis failure, defaults to risk:'low' and logs
-// a warning — never blocks healing due to analysis error.
+// a warning - never blocks healing due to analysis error.
 // ═══════════════════════════════════════════════════════════════════
 class BlastRadiusAnalyzer {
     static analyze(declarationNode, _sourceFile, _project) {
@@ -35,7 +35,10 @@ class BlastRadiusAnalyzer {
                 risk: "low",
             };
         }
-        const refs = nameNode.getProject().getLanguageService().findReferences(nameNode);
+        const refs = nameNode
+            .getProject()
+            .getLanguageService()
+            .findReferences(nameNode);
         const affectedFilesSet = new Set();
         let totalRefs = 0;
         // Key = filePath:startLine of the enclosing test() / it() call.