sela-core 1.0.7 → 1.0.9

package/dist/services/InteractiveReview.d.ts

@@ -1,24 +1,15 @@
 import type { PendingPromptEntry } from "./PendingPromptLedger";
-export interface InteractiveReviewDeps {
+export interface PrintReviewDeps {
     entries: PendingPromptEntry[];
-    /** Override the prompt — tests stub Y/N answers without spinning up readline. */
-    prompt?: (question: string) => Promise<string>;
-    /** Override the console sink — tests capture output for assertions. */
+    /** Override the console sink - tests capture output for assertions. */
     log?: (msg: string) => void;
-    /** Override the filesystem rollback (rewrites contentBefore). Tests stub. */
-    rollback?: (entry: PendingPromptEntry) => void;
-}
-export interface InteractiveReviewOutcome {
-    file: string;
-    applied: boolean;
+    /** Absolute path of the rendered Sela Insights HTML report, when present. */
+    reportHtmlPath?: string | null;
 }
 /**
- * Run the post-run interactive review loop sequentially across every
- * pending entry. Returns one outcome per entry so callers can log / test
- * the resulting Apply / Reject decisions.
- *
- * Sequential by design — every prompt `await`s before the next entry is
- * rendered. Concurrent prompts on a single TTY would interleave output.
+ * Print the post-run review summary. Returns immediately - never blocks
+ * on stdin. Safe to call from the reporter's onEnd() in any environment
+ * (TTY, piped CI, IDE-integrated runner).
  */
-export declare function runInteractiveReview(deps: InteractiveReviewDeps): Promise<InteractiveReviewOutcome[]>;
+export declare function printHealReview(deps: PrintReviewDeps): void;
 //# sourceMappingURL=InteractiveReview.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"InteractiveReview.d.ts","sourceRoot":"","sources":["../../src/services/InteractiveReview.ts"],"names":[],"mappings":"AAuBA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAEhE,MAAM,WAAW,qBAAqB;IACpC,OAAO,EAAE,kBAAkB,EAAE,CAAC;IAC9B,iFAAiF;IACjF,MAAM,CAAC,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IAC/C,uEAAuE;IACvE,GAAG,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5B,6EAA6E;IAC7E,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,kBAAkB,KAAK,IAAI,CAAC;CAChD;AAED,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;CAClB;AAID;;;;;;;GAOG;AACH,wBAAsB,oBAAoB,CACxC,IAAI,EAAE,qBAAqB,GAC1B,OAAO,CAAC,wBAAwB,EAAE,CAAC,CA+CrC"}
+{"version":3,"file":"InteractiveReview.d.ts","sourceRoot":"","sources":["../../src/services/InteractiveReview.ts"],"names":[],"mappings":"AAkBA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAEhE,MAAM,WAAW,eAAe;IAC9B,OAAO,EAAE,kBAAkB,EAAE,CAAC;IAC9B,uEAAuE;IACvE,GAAG,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5B,6EAA6E;IAC7E,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAChC;AAED;;;;GAIG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,eAAe,GAAG,IAAI,CA6B3D"}

package/dist/services/InteractiveReview.js

@@ -1,21 +1,18 @@
 "use strict";
 // src/services/InteractiveReview.ts
 //
-// Local DX — Interactive CLI review of post-run heals.
+// Local DX - Post-run summary printer (MVP write-and-notify).
 //
-// Runs in the REPORTER (main process) only — workers must never block on
-// stdin. The reporter reads cross-process state from
-// `PendingPromptLedger` and calls `runInteractiveReview()` with the
-// merged entry list.
+// The MVP flow is write-on-disk-then-notify: workers (and the engine)
+// already mutated the source files in-place during execution. The
+// reporter calls printHealReview() at the end purely to display:
+//   • a banner with the number of mutated files,
+//   • per-file isolated unified diff (clean room - never the working tree),
+//   • a vscode://file deeplink to the mutated line,
+//   • an optional deeplink to the generated HTML insights report.
 //
-// For every entry: prints `[Sela] 🔧 Healed selector in <FileName>`, the
-// isolated unified diff, a clickable `vscode://file/<abs>:<line>`
-// deeplink, then prompts the developer. 'Y' / 'y' / Enter keeps the
-// on-disk fix (the worker already wrote it); 'N' / 'n' rewrites the file
-// back to `contentBefore` (the pre-heal state captured by
-// `WorkspaceSnapshotService.preWrite()` in the worker).
-//
-// All I/O is injectable so this can be unit-tested without a real TTY.
+// No prompts. No rollback. No blocking on stdin. SIGINT handling lives
+// in SelaReporter - this module is pure stdout/stderr.
 var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
     if (k2 === undefined) k2 = k;
     var desc = Object.getOwnPropertyDescriptor(m, k);
@@ -53,55 +50,37 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.runInteractiveReview = runInteractiveReview;
-const fs = __importStar(require("fs"));
+exports.printHealReview = printHealReview;
 const path = __importStar(require("path"));
-const readline = __importStar(require("readline"));
 const chalk_1 = __importDefault(require("chalk"));
 const IsolatedDiff_1 = require("../utils/IsolatedDiff");
-const DEFAULT_PROMPT = "[Sela] Apply this fix? (Y/n): ";
 /**
- * Run the post-run interactive review loop sequentially across every
- * pending entry. Returns one outcome per entry so callers can log / test
- * the resulting Apply / Reject decisions.
- *
- * Sequential by design — every prompt `await`s before the next entry is
- * rendered. Concurrent prompts on a single TTY would interleave output.
+ * Print the post-run review summary. Returns immediately - never blocks
+ * on stdin. Safe to call from the reporter's onEnd() in any environment
+ * (TTY, piped CI, IDE-integrated runner).
  */
-async function runInteractiveReview(deps) {
+function printHealReview(deps) {
     const log = deps.log ?? ((m) => process.stdout.write(m + "\n"));
-    const prompt = deps.prompt ?? defaultPrompt;
-    const rollback = deps.rollback ?? defaultRollback;
     const entries = deps.entries ?? [];
-    const outcomes = [];
-    if (entries.length === 0)
-        return outcomes;
-    // Headline banner — surfaces even when downstream chalk/readline
-    // output is suppressed by the host terminal multiplexer (e.g.
-    // Playwright's reporter-level stdio interception).
+    const reportHtmlPath = deps.reportHtmlPath ?? null;
+    if (entries.length === 0 && !reportHtmlPath)
+        return;
     log("");
-    log(chalk_1.default.bold.bgCyan.black(` [Sela] Local DX Review — ${entries.length} pending fix(es) `));
-    for (const entry of entries) {
-        renderEntry(entry, log);
-        const answer = await prompt(chalk_1.default.cyan(DEFAULT_PROMPT));
-        const decision = parseDecision(answer);
-        if (decision === "apply") {
-            log(chalk_1.default.green(`[Sela] ✅ Fix kept on disk: ${entry.relativePath}`));
-            outcomes.push({ file: entry.absolutePath, applied: true });
-        }
-        else {
-            try {
-                rollback(entry);
-                log(chalk_1.default.yellow(`[Sela] ↩️  Rolled back: ${entry.relativePath} (restored to pre-heal state)`));
-            }
-            catch (err) {
-                log(chalk_1.default.red(`[Sela] ⚠️  Rollback failed for ${entry.relativePath}: ${err.message}`));
-            }
-            outcomes.push({ file: entry.absolutePath, applied: false });
+    if (entries.length > 0) {
+        // Count actual healed selectors (a file can hold several), not files.
+        const fileCount = entries.length;
+        const fixCount = entries.reduce((n, e) => n + (e.healCount ?? 1), 0);
+        log(chalk_1.default.bold.bgCyan.black(` [Sela] Local DX Review - ${fixCount} fix${fixCount === 1 ? "" : "es"} applied across ${fileCount} file${fileCount === 1 ? "" : "s"} `));
+        for (const entry of entries) {
+            renderEntry(entry, log);
         }
     }
+    if (reportHtmlPath) {
+        log("");
+        log(chalk_1.default.bold(`[Sela] 📄 Insights report:`));
+        log(`   ${chalk_1.default.cyan(buildFileLink(reportHtmlPath))}`);
+    }
     log("");
-    return outcomes;
 }
 function renderEntry(entry, log) {
     const fileName = path.basename(entry.absolutePath);
@@ -113,9 +92,8 @@ function renderEntry(entry, log) {
     log("");
     log(colorizeDiff(diff));
     log("");
-    log(`👉 To review and accept this fix directly in VS Code, click here:`);
+    log(`👉 Open in VS Code:`);
     log(`   ${chalk_1.default.cyan(buildVsCodeLink(entry.absolutePath, entry.lineNumber))}`);
-    log("");
 }
 function colorizeDiff(diff) {
     if (!diff)
@@ -144,75 +122,7 @@ function buildVsCodeLink(absPath, line) {
         ? `vscode://file/${normalised}:${line}`
         : `vscode://file/${normalised}`;
 }
-function parseDecision(raw) {
-    const trimmed = (raw ?? "").trim().toLowerCase();
-    // Empty (Enter) ⇒ apply. 'y' / 'yes' ⇒ apply. 'n' / 'no' ⇒ reject.
-    // Anything else ⇒ apply (default-safe — never silently destroy work).
-    if (trimmed === "" || trimmed === "y" || trimmed === "yes")
-        return "apply";
-    if (trimmed === "n" || trimmed === "no")
-        return "reject";
-    return "apply";
-}
-/**
- * Default rollback — rewrites `contentBefore` to disk. Honours the
- * `createdBySela` flag so files Sela CREATED from nothing get unlinked
- * instead of being replaced with an empty buffer.
- */
-function defaultRollback(entry) {
-    if (entry.createdBySela) {
-        if (fs.existsSync(entry.absolutePath)) {
-            fs.unlinkSync(entry.absolutePath);
-        }
-        return;
-    }
-    fs.writeFileSync(entry.absolutePath, entry.contentBefore, "utf8");
-}
-/**
- * Bind `readline` to the live stdin/stdout. Reporter-only — workers must
- * never reach this code path (they have no TTY and would deadlock).
- *
- * Hardening:
- *   • If stdin is not a TTY (CI, non-interactive shell, Playwright reporter
- *     context that detached stdio), default to Apply ('y') without blocking.
- *   • Explicitly `resume()` stdin — Node pauses it by default once any other
- *     consumer (readline-from-an-earlier-iteration, raw-mode toggles) lets go.
- *   • Write the prompt to `stderr` so Playwright's stdout summary buffer does
- *     not overwrite or strip the question line.
- */
-function defaultPrompt(question) {
-    const stdinIsTty = process.stdin.isTTY === true;
-    if (!stdinIsTty) {
-        process.stderr.write(question + "y (no TTY — auto-apply)\n");
-        return Promise.resolve("y");
-    }
-    return new Promise((resolve) => {
-        try {
-            process.stdin.resume();
-        }
-        catch {
-            // Already resumed / not resumable — readline will still try.
-        }
-        // `terminal: false` reads stdin line-buffered without raw-mode tricks.
-        // This is reliable for both real TTYs AND piped harnesses (CI smoke
-        // tests, child-process drivers). `terminal: true` only adds history /
-        // arrow-key support, which we don't need for a single Y/N prompt and
-        // which mis-parses piped input on Windows.
-        process.stderr.write(question);
-        const rl = readline.createInterface({
-            input: process.stdin,
-            output: process.stderr,
-            terminal: false,
-        });
-        let settled = false;
-        const finalise = (answer) => {
-            if (settled)
-                return;
-            settled = true;
-            rl.close();
-            resolve(answer);
-        };
-        rl.once("line", (line) => finalise(line));
-        rl.once("close", () => finalise("y"));
-    });
+function buildFileLink(absPath) {
+    const normalised = absPath.split(path.sep).join("/");
+    return `file://${normalised}`;
 }

package/dist/services/LLMService.d.ts

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

package/dist/services/LLMService.js

@@ -71,11 +71,11 @@ Narrowing: filter, first, last, nth
 You MUST return a "chainSegments" array for the permanent code fix AND a matching "segments" array for the runtime proxy.
 
 Build the chain in this priority order:
-  PRIORITY 1 — getByTestId
-  PRIORITY 2 — getByRole (ALWAYS include "name" if it exists in the DOM to ensure uniqueness)
-  PRIORITY 3 — getByLabel
-  PRIORITY 4 — getByText (use exact: true if possible)
-  NARROWING — Use filter({ hasText: "..." }) to isolate an element within a repeated container.
+  PRIORITY 1 - getByTestId
+  PRIORITY 2 - getByRole (ALWAYS include "name" if it exists in the DOM to ensure uniqueness)
+  PRIORITY 3 - getByLabel
+  PRIORITY 4 - getByText (use exact: true if possible)
+  NARROWING - Use filter({ hasText: "..." }) to isolate an element within a repeated container.
 
 ### PROXY SELECTOR CONSTRUCTION (segments field)
 The "segments[0].selector" string is used for an immediate "live" retry. 
@@ -90,10 +90,10 @@ Preserve the original code's structural pattern from "originalChainHint":
 
 ### TEMPLATE LITERAL / FUNCTION SELECTOR PRESERVATION
 When the failing selector originates from a Template Literal or a function returning a template,
-preserve the attribute structure — update the value inside the attribute rather than removing it.
+preserve the attribute structure - update the value inside the attribute rather than removing it.
 For chain selectors: if the original was "#section-B >> input[data-role='user_legacy']"
 and the DNA shows data-role changed to "user_active", return "#section-B >> input[data-role='user_active']".
-NEVER simplify to "#section-B >> input" — that loses the attribute and breaks automated source healing.
+NEVER simplify to "#section-B >> input" - that loses the attribute and breaks automated source healing.
 The healing system depends on the attribute value being present in your response to patch the template variable.
 
 ### TYPE SAFETY RULES
@@ -105,21 +105,21 @@ If the element's text changed, you MUST include:
   "contentChange": { "oldText": "<previous>", "newText": "<current>" }
 
 ### CONFIDENCE SCORE (MANDATORY)
-You MUST include a numeric "confidence" field — an INTEGER in the range 0-100 —
+You MUST include a numeric "confidence" field - an INTEGER in the range 0-100 -
 representing how certain you are that the returned chain will resolve the
 intended element on the next test run. Use this rubric:
-  - 90-100: stable id, data-testid, or unique role+name — zero ambiguity in the DOM.
+  - 90-100: stable id, data-testid, or unique role+name - zero ambiguity in the DOM.
   - 70-89:  strong semantic match, but some structural drift or competing candidates.
-  - 50-69:  best-guess match — multiple plausible candidates required disambiguation.
-  - 1-49:   speculative match — return this only when no better candidate exists.
+  - 50-69:  best-guess match - multiple plausible candidates required disambiguation.
+  - 1-49:   speculative match - return this only when no better candidate exists.
 NEVER omit this field. NEVER return it as a string, boolean, or null.
-Reports rendered downstream will show "n/a" when this field is missing — which
+Reports rendered downstream will show "n/a" when this field is missing - which
 is a clear failure signal to the developer reviewing the heal.
 
 ### MANDATORY OUTPUT FORMAT
 Return ONLY valid JSON.
 
-Example output — Identifying a specific button in a list row:
+Example output - Identifying a specific button in a list row:
 {
   "status": "FIXED",
   "confidence": 92,
@@ -215,7 +215,7 @@ class LLMService {
             throw new SelaError_1.SelaError({
                 subsystem: "LLMService",
                 code: SelaError_1.LLM_SERVICE_CODES.NO_API_KEY,
-                reason: "ANTHROPIC_API_KEY not found — pass an `anthropic` client or `apiKey` to LLMService(), or set the env var",
+                reason: "ANTHROPIC_API_KEY not found - pass an `anthropic` client or `apiKey` to LLMService(), or set the env var",
             });
         }
         this.anthropic = new sdk_1.default({ apiKey });
@@ -339,7 +339,7 @@ ${request.targetIntent}
 ### PREVIOUS DNA (Element Fingerprint):
 ${dnaSummary && Object.keys(dnaSummary).length > 0
             ? JSON.stringify(dnaSummary, null, 2)
-            : "NO DNA (First run — infer from selector and DOM)"}
+            : "NO DNA (First run - infer from selector and DOM)"}
 
 ### CURRENT DOM SNIPPET:
 ${request.currentDom}
@@ -351,7 +351,7 @@ ${request.currentDom}
 4. Populate "contentChange" if the element's visible text changed.
 5. Also populate "segments" (legacy format) for backward compatibility.
 
-Return ONLY a raw JSON object — no markdown, no preamble.`;
+Return ONLY a raw JSON object - no markdown, no preamble.`;
         try {
             const msg = await this.anthropic.messages.create({
                 model: this.model,

package/dist/services/MutationApplier.d.ts

@@ -0,0 +1,55 @@
+import { AnchorKey, SourceSpan } from "./AnchorResolver";
+/**
+ * A single edit. `replace` edits carry an `anchor` and are RE-RESOLVED at
+ * apply-time (span recomputed from the fresh parse). `insert` edits (forks)
+ * carry a literal `span` computed against the same content being applied.
+ */
+export interface AnchoredEdit {
+    kind: "replace" | "insert";
+    replacement: string;
+    /** Replace target — re-resolved via resolveAnchor at apply time. */
+    anchor?: AnchorKey;
+    /** Literal span for inserts (and as a fallback for anchorless replaces). */
+    span?: SourceSpan;
+    /**
+     * Require the replacement to be a Locator-returning expression. Defaults to
+     * true for `replace`. Set false for inserts / non-locator edits.
+     */
+    requireLocator?: boolean;
+}
+export interface FileBatch {
+    filePath: string;
+    edits: AnchoredEdit[];
+}
+export type ApplyStatus = "applied" | "rolled-back" | "no-op" | "skipped";
+export interface ApplyOutcome {
+    filePath: string;
+    status: ApplyStatus;
+    reason: string;
+    written: boolean;
+    /** Content after the run — the new content when applied, else the original. */
+    finalContent: string;
+    /** Edits that could not be applied (stale anchor, type-incompat, …). */
+    skipped: Array<{
+        reason: string;
+    }>;
+}
+export interface ApplyOptions {
+    /** When false, compute + gate but never touch disk. Default true. */
+    write?: boolean;
+    /** Honour SELA_DRY_RUN by leaving disk untouched. Default true. */
+    respectDryRun?: boolean;
+}
+export declare class MutationApplier {
+    /** Apply many batches; each file is processed independently. */
+    static applyBatches(batches: FileBatch[], opts?: ApplyOptions): ApplyOutcome[];
+    static applyFileBatch(batch: FileBatch, opts?: ApplyOptions): ApplyOutcome;
+}
+/**
+ * Cheap return-type pre-check. The replacement must be an expression whose
+ * OUTERMOST call is a Locator-returning Playwright method (locator / getBy* /
+ * frameLocator / nth / first / last / filter / and / or). Rejects actions
+ * (.click()), string queries (.title(), .textContent()), bare literals, etc.
+ */
+export declare function isLocatorReturningExpr(text: string): boolean;
+//# sourceMappingURL=MutationApplier.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"MutationApplier.d.ts","sourceRoot":"","sources":["../../src/services/MutationApplier.ts"],"names":[],"mappings":"AA2BA,OAAO,EACL,SAAS,EAET,UAAU,EACX,MAAM,kBAAkB,CAAC;AAS1B;;;;GAIG;AACH,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,SAAS,GAAG,QAAQ,CAAC;IAC3B,WAAW,EAAE,MAAM,CAAC;IACpB,oEAAoE;IACpE,MAAM,CAAC,EAAE,SAAS,CAAC;IACnB,4EAA4E;IAC5E,IAAI,CAAC,EAAE,UAAU,CAAC;IAClB;;;OAGG;IACH,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAED,MAAM,WAAW,SAAS;IACxB,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,YAAY,EAAE,CAAC;CACvB;AAED,MAAM,MAAM,WAAW,GAAG,SAAS,GAAG,aAAa,GAAG,OAAO,GAAG,SAAS,CAAC;AAE1E,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,WAAW,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;IACjB,+EAA+E;IAC/E,YAAY,EAAE,MAAM,CAAC;IACrB,wEAAwE;IACxE,OAAO,EAAE,KAAK,CAAC;QAAE,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACpC;AAED,MAAM,WAAW,YAAY;IAC3B,qEAAqE;IACrE,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,mEAAmE;IACnE,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAeD,qBAAa,eAAe;IAC1B,gEAAgE;IAChE,MAAM,CAAC,YAAY,CACjB,OAAO,EAAE,SAAS,EAAE,EACpB,IAAI,GAAE,YAAiB,GACtB,YAAY,EAAE;IAIjB,MAAM,CAAC,cAAc,CACnB,KAAK,EAAE,SAAS,EAChB,IAAI,GAAE,YAAiB,GACtB,YAAY;CA6JhB;AA2CD;;;;;GAKG;AACH,wBAAgB,sBAAsB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAkB5D"}