sela-core 1.0.7 → 1.0.8

package/dist/services/HealthReportService.d.ts

@@ -1,4 +1,4 @@
-import type { DnaIndex, HealthReport } from './types.js';
+import type { DnaIndex, HealthReport } from "./types.js";
 export interface IHealthReportService {
     generate(index: DnaIndex): HealthReport;
     format(report: HealthReport): string;

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

@@ -1 +1 @@
-{"version":3,"file":"HealthReportService.d.ts","sourceRoot":"","sources":["../../src/services/HealthReportService.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAa,YAAY,EAAE,MAAM,YAAY,CAAC;AAEpE,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,KAAK,EAAE,QAAQ,GAAG,YAAY,CAAC;IACxC,MAAM,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM,CAAC;CACtC;AAkBD,qBAAa,mBAAoB,YAAW,oBAAoB;IAC9D,QAAQ,CAAC,KAAK,EAAE,QAAQ,GAAG,YAAY;IAgBvC,MAAM,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM;CA6ErC"}
+{"version":3,"file":"HealthReportService.d.ts","sourceRoot":"","sources":["../../src/services/HealthReportService.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,QAAQ,EAAa,YAAY,EAAE,MAAM,YAAY,CAAC;AAEpE,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,KAAK,EAAE,QAAQ,GAAG,YAAY,CAAC;IACxC,MAAM,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM,CAAC;CACtC;AAoBD,qBAAa,mBAAoB,YAAW,oBAAoB;IAC9D,QAAQ,CAAC,KAAK,EAAE,QAAQ,GAAG,YAAY;IAkBvC,MAAM,CAAC,MAAM,EAAE,YAAY,GAAG,MAAM;CA0FrC"}

package/dist/services/HealthReportService.js

@@ -28,48 +28,48 @@ class HealthReportService {
             snapshotDir,
             total: records.length,
             orphaned: records.filter((r) => r.isOrphaned),
-            unverifiedHeals: records.filter((r) => !r.isOrphaned && r.status === 'healed'),
+            unverifiedHeals: records.filter((r) => !r.isOrphaned && r.status === "healed"),
             recentlyHealed: records.filter((r) => r.lastHealed !== null && isWithin7Days(r.lastHealed)),
             neverHealed: records.filter((r) => r.lastHealed === null),
             topHealedFiles: topHealedFiles(records),
         };
     }
     format(report) {
-        const lines = [''];
-        lines.push(chalk_1.default.bold.cyan('  Sela Health Report') +
+        const lines = [""];
+        lines.push(chalk_1.default.bold.cyan("  Sela Health Report") +
             chalk_1.default.dim(`  (${new Date(report.generatedAt).toISOString()})`));
-        lines.push('');
+        lines.push("");
         const row = (label, value) => `  ${chalk_1.default.dim(label.padEnd(20))}  ${value}`;
-        lines.push(row('Total DNAs', chalk_1.default.bold(String(report.total))));
+        lines.push(row("Total DNAs", chalk_1.default.bold(String(report.total))));
         const orphanCount = report.orphaned.length;
-        lines.push(row('Orphaned', orphanCount > 0
-            ? chalk_1.default.yellow(`${orphanCount}  ⚠`)
-            : chalk_1.default.green('0')));
+        lines.push(row("Orphaned", orphanCount > 0 ? chalk_1.default.yellow(`${orphanCount}  ⚠`) : chalk_1.default.green("0")));
         const unverifiedCount = report.unverifiedHeals.length;
-        lines.push(row('Unverified heals', unverifiedCount > 0
-            ? chalk_1.default.yellow(`${unverifiedCount}  ⚠  ${chalk_1.default.dim('(AI-healed, needs review)')}`)
-            : chalk_1.default.green('0')));
+        lines.push(row("Unverified heals", unverifiedCount > 0
+            ? chalk_1.default.yellow(`${unverifiedCount}  ⚠  ${chalk_1.default.dim("(AI-healed, needs review)")}`)
+            : chalk_1.default.green("0")));
         const brokenTotal = orphanCount + unverifiedCount;
         const brokenRatio = report.total > 0
             ? ((brokenTotal / report.total) * 100).toFixed(1)
-            : '0.0';
-        lines.push(row('Broken ratio', (brokenTotal > 0 ? chalk_1.default.yellow : chalk_1.default.green)(`${brokenRatio}%  ${chalk_1.default.dim(`(${brokenTotal}/${report.total})`)}`)));
-        lines.push(row('Healed (7 days)', chalk_1.default.bold(String(report.recentlyHealed.length))));
-        lines.push(row('Healed (all time)', chalk_1.default.bold(String(report.total - report.neverHealed.length))));
-        lines.push(row('Never healed', chalk_1.default.bold(String(report.neverHealed.length))));
+            : "0.0";
+        lines.push(row("Broken ratio", (brokenTotal > 0 ? chalk_1.default.yellow : chalk_1.default.green)(`${brokenRatio}%  ${chalk_1.default.dim(`(${brokenTotal}/${report.total})`)}`)));
+        lines.push(row("Healed (7 days)", chalk_1.default.bold(String(report.recentlyHealed.length))));
+        lines.push(row("Healed (all time)", chalk_1.default.bold(String(report.total - report.neverHealed.length))));
+        lines.push(row("Never healed", chalk_1.default.bold(String(report.neverHealed.length))));
         if (report.topHealedFiles.length > 0) {
-            lines.push('');
-            lines.push(`  ${chalk_1.default.bold('Top healed files:')}`);
+            lines.push("");
+            lines.push(`  ${chalk_1.default.bold("Top healed files:")}`);
             for (const { sourceFile, count } of report.topHealedFiles.slice(0, 5)) {
                 const rel = path_1.default.relative(process.cwd(), sourceFile);
-                const label = (rel.length < sourceFile.length ? rel : sourceFile).slice(-60).padEnd(62);
-                const noun = count === 1 ? 'heal' : 'heals';
-                lines.push(`    ${chalk_1.default.cyan(label)}  ${chalk_1.default.dim('—')}  ${chalk_1.default.bold(String(count))} ${chalk_1.default.dim(noun)}`);
+                const label = (rel.length < sourceFile.length ? rel : sourceFile)
+                    .slice(-60)
+                    .padEnd(62);
+                const noun = count === 1 ? "heal" : "heals";
+                lines.push(`    ${chalk_1.default.cyan(label)}  ${chalk_1.default.dim("-")}  ${chalk_1.default.bold(String(count))} ${chalk_1.default.dim(noun)}`);
             }
         }
         if (report.orphaned.length > 0) {
-            lines.push('');
-            lines.push(`  ${chalk_1.default.yellow.bold('Orphaned DNA files:')}`);
+            lines.push("");
+            lines.push(`  ${chalk_1.default.yellow.bold("Orphaned DNA files:")}`);
             for (const r of report.orphaned.slice(0, 10)) {
                 lines.push(`    ${chalk_1.default.dim(r.key)}  ${chalk_1.default.yellow(r.sourceFile)}`);
             }
@@ -77,8 +77,8 @@ class HealthReportService {
                 lines.push(chalk_1.default.dim(`    … and ${report.orphaned.length - 10} more`));
             }
         }
-        lines.push('');
-        return lines.join('\n');
+        lines.push("");
+        return lines.join("\n");
     }
 }
 exports.HealthReportService = HealthReportService;

package/dist/services/InitializerUpdater.js

@@ -24,14 +24,14 @@ class InitializerUpdater {
             else {
                 return null;
             }
-            // Centralised write gate — mirrors ASTSourceUpdater._persistChanges.
+            // Centralised write gate - mirrors ASTSourceUpdater._persistChanges.
             // InitializerUpdater is the canonical write-head for definition-site
             // mutations (SemanticDispatch → tracer-resolved identifier / property
             // access / cross-file paths all converge here), so the Clean-Room
             // snapshot ledger and DryRunGuard MUST be enforced here too.
             // Without this, an upstream-traced fix (e.g. global StringLiteral at
             // top of file) writes to disk but `WorkspaceSnapshotService` never
-            // sees it — `getMutatedSnapshots()` returns empty and PR automation
+            // sees it - `getMutatedSnapshots()` returns empty and PR automation
             // skips the heal entirely.
             const fp = sourceFile.getFilePath();
             if (DryRunGuard_1.DryRunGuard.active()) {

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

@@ -1 +1 @@
-{"version":3,"file":"IntentAuditor.d.ts","sourceRoot":"","sources":["../../src/services/IntentAuditor.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,mBAAmB,CAAC;AAE1C,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAOpD,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,WAAW,GAAG,QAAQ,CAAC;IAEjC,8EAA8E;IAC9E,WAAW,CAAC,EAAE,YAAY,EAAE,CAAC;IAC7B,+EAA+E;IAC/E,YAAY,CAAC,EAAE,YAAY,EAAE,CAAC;IAC9B,+CAA+C;IAC/C,UAAU,CAAC,EAAE,UAAU,CAAC;IACxB,qEAAqE;IACrE,WAAW,CAAC,EAAE,UAAU,CAAC;CAC1B;AAED,MAAM,MAAM,iBAAiB,GAAG,YAAY,GAAG,cAAc,GAAG,YAAY,CAAC;AAE7E,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,iBAAiB,CAAC;IAC3B,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,UAAU,GAAG,cAAc,GAAG,OAAO,GAAG,UAAU,GAAG,IAAI,CAAC;IACzE,yEAAyE;IACzE,cAAc,CAAC,EAAE;QAAE,aAAa,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC;CAC5E;AAkED;;;;;;;;;GASG;AACH,MAAM,WAAW,oBAAoB;IACnC,SAAS,CAAC,EAAE,SAAS,GAAG,IAAI,CAAC;IAC7B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAID,qBAAa,aAAa;IACxB,OAAO,CAAC,SAAS,CAAmB;IACpC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAU;IACnC,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAS;gBAEnB,IAAI,GAAE,oBAAyB;IA+BrC,WAAW,CAAC,GAAG,EAAE,YAAY,GAAG,OAAO,CAAC,YAAY,CAAC;IA4G3D,sBAAsB,CACpB,OAAO,EAAE,YAAY,EAAE,EACvB,QAAQ,EAAE,YAAY,EAAE,GACvB,MAAM;IA4CT,kBAAkB,CAChB,UAAU,EAAE,UAAU,GAAG,IAAI,GAAG,SAAS,EACzC,WAAW,EAAE,UAAU,GAAG,IAAI,GAAG,SAAS,GACzC,MAAM;IAmCT,OAAO,CAAC,YAAY;CA8CrB"}
+{"version":3,"file":"IntentAuditor.d.ts","sourceRoot":"","sources":["../../src/services/IntentAuditor.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,mBAAmB,CAAC;AAE1C,OAAO,EAAE,YAAY,EAAE,UAAU,EAAE,MAAM,UAAU,CAAC;AAOpD,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,WAAW,GAAG,QAAQ,CAAC;IAEjC,8EAA8E;IAC9E,WAAW,CAAC,EAAE,YAAY,EAAE,CAAC;IAC7B,+EAA+E;IAC/E,YAAY,CAAC,EAAE,YAAY,EAAE,CAAC;IAC9B,+CAA+C;IAC/C,UAAU,CAAC,EAAE,UAAU,CAAC;IACxB,qEAAqE;IACrE,WAAW,CAAC,EAAE,UAAU,CAAC;CAC1B;AAED,MAAM,MAAM,iBAAiB,GAAG,YAAY,GAAG,cAAc,GAAG,YAAY,CAAC;AAE7E,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,iBAAiB,CAAC;IAC3B,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,UAAU,GAAG,cAAc,GAAG,OAAO,GAAG,UAAU,GAAG,IAAI,CAAC;IACzE,yEAAyE;IACzE,cAAc,CAAC,EAAE;QAAE,aAAa,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC;CAC5E;AAkED;;;;;;;;;GASG;AACH,MAAM,WAAW,oBAAoB;IACnC,SAAS,CAAC,EAAE,SAAS,GAAG,IAAI,CAAC;IAC7B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAID,qBAAa,aAAa;IACxB,OAAO,CAAC,SAAS,CAAmB;IACpC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAU;IACnC,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAS;gBAEnB,IAAI,GAAE,oBAAyB;IA+BrC,WAAW,CAAC,GAAG,EAAE,YAAY,GAAG,OAAO,CAAC,YAAY,CAAC;IAyH3D,sBAAsB,CACpB,OAAO,EAAE,YAAY,EAAE,EACvB,QAAQ,EAAE,YAAY,EAAE,GACvB,MAAM;IA4CT,kBAAkB,CAChB,UAAU,EAAE,UAAU,GAAG,IAAI,GAAG,SAAS,EACzC,WAAW,EAAE,UAAU,GAAG,IAAI,GAAG,SAAS,GACzC,MAAM;IAmCT,OAAO,CAAC,YAAY;CA8CrB"}

package/dist/services/IntentAuditor.js

@@ -46,15 +46,15 @@ dotenv.config();
 const FALLBACK_VERDICT = {
     verdict: "SUSPICIOUS",
     confidence: 0,
-    reason: "Auditor unavailable — defaulting to cautious SUSPICIOUS verdict",
+    reason: "Auditor unavailable - defaulting to cautious SUSPICIOUS verdict",
     inversionType: null,
 };
 // Ancestry drift penalties (spec §2.1).
 // Applied as negative adjustments to the LLM's confidence score.
 // Maximum combined penalty: −30 pts.
 const DRIFT_PENALTIES = [
-    { full: -20, partial: -10 }, // depth 1 — direct parent
-    { full: -10, partial: -5 }, // depth 2 — grandparent
+    { full: -20, partial: -10 }, // depth 1 - direct parent
+    { full: -10, partial: -5 }, // depth 2 - grandparent
 ];
 // Anchor bonuses: positive-only (spec §3.3).
 // closestLabel exact match:  +10 pts
@@ -85,8 +85,8 @@ RULES:
 3. Minor reformatting is irrelevant. "Save Changes" and "Save" are CONSISTENT.
 4. Role/aria context matters. A button that was "Submit" becoming "Cancel" is INCONSISTENT (identity inversion).
 5. When both texts are very short (3 chars or less) and differ, prefer SUSPICIOUS unless an obvious opposite pair.
-6. In "assertion" mode, be stricter — lean toward SUSPICIOUS or INCONSISTENT when the change is ambiguous.
-7. In "action" mode, be more lenient — lean toward CONSISTENT when the change looks like a UI refactor.
+6. In "assertion" mode, be stricter - lean toward SUSPICIOUS or INCONSISTENT when the change is ambiguous.
+7. In "action" mode, be more lenient - lean toward CONSISTENT when the change looks like a UI refactor.
 
 RESPONSE FORMAT (strict JSON, no other text):
 {
@@ -102,7 +102,7 @@ class IntentAuditor {
     model;
     constructor(opts = {}) {
         this.model = opts.model ?? DEFAULT_AUDITOR_MODEL;
-        // Explicit `null` from the caller forces disabled mode — used by tests.
+        // Explicit `null` from the caller forces disabled mode - used by tests.
         if (opts.anthropic === null) {
             this.anthropic = null;
             this.disabled = true;
@@ -117,7 +117,7 @@ class IntentAuditor {
         // Otherwise fall back to env-based construction (legacy default).
         const apiKey = opts.apiKey ?? process.env.ANTHROPIC_API_KEY;
         if (!apiKey) {
-            console.warn("[IntentAuditor] ANTHROPIC_API_KEY not set — auditor disabled, will return SUSPICIOUS for all audited heals");
+            console.warn("[IntentAuditor] ANTHROPIC_API_KEY not set - auditor disabled, will return SUSPICIOUS for all audited heals");
             this.anthropic = null;
             this.disabled = true;
         }
@@ -154,7 +154,7 @@ class IntentAuditor {
             // ── Structural scoring ────────────────────────────────────────
             // Apply ancestry drift penalty and anchor bonus to the LLM's raw
             // confidence.  The semantic verdict category (CONSISTENT / SUSPICIOUS /
-            // INCONSISTENT) is never changed by scoring — only the confidence value.
+            // INCONSISTENT) is never changed by scoring - only the confidence value.
             // The low-confidence CONSISTENT→SUSPICIOUS downgrade is re-evaluated
             // after scoring so the adjusted value is used for the threshold.
             const penalty = this.computeAncestryPenalty(req.dnaAncestry ?? [], req.liveAncestry ?? []);
@@ -172,7 +172,11 @@ class IntentAuditor {
                 // Re-apply low-confidence CONSISTENT downgrade with the adjusted value.
                 // The raw check in parseVerdict used the pre-adjustment confidence; a
                 // penalty may push a borderline CONSISTENT below the 50% threshold.
-                const scoring = { rawConfidence: rawVerdict.confidence, penalty, bonus };
+                const scoring = {
+                    rawConfidence: rawVerdict.confidence,
+                    penalty,
+                    bonus,
+                };
                 if (rawVerdict.verdict === "CONSISTENT" && adjConf < 50) {
                     finalVerdict = {
                         verdict: "SUSPICIOUS",
@@ -184,22 +188,29 @@ class IntentAuditor {
                     console.log(`[IntentAuditor] CONSISTENT→SUSPICIOUS after scoring (${rawVerdict.confidence}%→${adjConf}%)`);
                 }
                 else {
-                    finalVerdict = { ...rawVerdict, confidence: adjConf, scoreBreakdown: scoring };
+                    finalVerdict = {
+                        ...rawVerdict,
+                        confidence: adjConf,
+                        scoreBreakdown: scoring,
+                    };
                 }
             }
             else {
                 finalVerdict = rawVerdict;
             }
-            console.log(`[IntentAuditor] verdict=${finalVerdict.verdict} (${finalVerdict.confidence}%) — ${finalVerdict.reason}`);
+            console.log(`[IntentAuditor] verdict=${finalVerdict.verdict} (${finalVerdict.confidence}%) - ${finalVerdict.reason}`);
             return finalVerdict;
         }
         catch (err) {
-            console.warn(`[IntentAuditor] Warning: Auditor call failed: ${err.message} — returning SUSPICIOUS`);
-            return { ...FALLBACK_VERDICT, reason: `Auditor call failed: ${err.message}` };
+            console.warn(`[IntentAuditor] Warning: Auditor call failed: ${err.message} - returning SUSPICIOUS`);
+            return {
+                ...FALLBACK_VERDICT,
+                reason: `Auditor call failed: ${err.message}`,
+            };
         }
     }
     // ─────────────────────────────────────────────────────────────
-    // computeAncestryPenalty — DNA v2 Ancestry Drift  (spec §2)
+    // computeAncestryPenalty - DNA v2 Ancestry Drift  (spec §2)
     //
     // Compares at most 2 ancestor depths between the DNA baseline and
     // the live candidate.  Returns a value in [−30, 0]; 0 = no drift.
@@ -216,7 +227,7 @@ class IntentAuditor {
     // ─────────────────────────────────────────────────────────────
     computeAncestryPenalty(dnaPath, livePath) {
         if (dnaPath.length === 0)
-            return 0; // v1 migrated snapshot — skip
+            return 0; // v1 migrated snapshot - skip
         let totalPenalty = 0;
         const maxDepth = Math.min(dnaPath.length, livePath.length, 2);
         for (let i = 0; i < maxDepth; i++) {
@@ -229,21 +240,21 @@ class IntentAuditor {
             const depthLabel = i === 0 ? "depth-1 (parent)" : "depth-2 (grandparent)";
             if (dna.tag !== live.tag) {
                 totalPenalty += full;
-                console.log(`[IntentAuditor] 🏗️  Ancestry drift ${full} pts — ${depthLabel}: ` +
+                console.log(`[IntentAuditor] 🏗️  Ancestry drift ${full} pts - ${depthLabel}: ` +
                     `tag mismatch (${dna.fingerprint} → ${live.fingerprint})`);
             }
             else {
                 totalPenalty += partial;
-                console.log(`[IntentAuditor] 🏗️  Ancestry drift ${partial} pts — ${depthLabel}: ` +
+                console.log(`[IntentAuditor] 🏗️  Ancestry drift ${partial} pts - ${depthLabel}: ` +
                     `same tag, id/class changed (${dna.fingerprint} → ${live.fingerprint})`);
             }
         }
         return totalPenalty;
     }
     // ─────────────────────────────────────────────────────────────
-    // computeAnchorBonus — DNA v2 Anchor Confirmation  (spec §3.3)
+    // computeAnchorBonus - DNA v2 Anchor Confirmation  (spec §3.3)
     //
-    // Anchors only provide positive confirmation — no penalty for
+    // Anchors only provide positive confirmation - no penalty for
     // mismatch.  Returns a value in [0, +15].
     //
     // closestLabel exact match (case-insensitive):  +10 pts
@@ -257,13 +268,13 @@ class IntentAuditor {
         const liveLabel = liveAnchors.closestLabel?.toLowerCase().trim();
         if (dnaLabel && liveLabel && dnaLabel === liveLabel) {
             bonus += ANCHOR_BONUS_LABEL;
-            console.log(`[IntentAuditor] 🎯 Anchor +${ANCHOR_BONUS_LABEL} pts — closestLabel match: "${dnaAnchors.closestLabel}"`);
+            console.log(`[IntentAuditor] 🎯 Anchor +${ANCHOR_BONUS_LABEL} pts - closestLabel match: "${dnaAnchors.closestLabel}"`);
         }
         const dnaRow = dnaAnchors.rowAnchor?.toLowerCase().trim();
         const liveRow = liveAnchors.rowAnchor?.toLowerCase().trim();
         if (dnaRow && liveRow && dnaRow === liveRow) {
             bonus += ANCHOR_BONUS_ROW;
-            console.log(`[IntentAuditor] 🎯 Anchor +${ANCHOR_BONUS_ROW} pts — rowAnchor match: "${dnaAnchors.rowAnchor}"`);
+            console.log(`[IntentAuditor] 🎯 Anchor +${ANCHOR_BONUS_ROW} pts - rowAnchor match: "${dnaAnchors.rowAnchor}"`);
         }
         return bonus;
     }

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,CA0B3D"}

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,34 @@ 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) {
+        log(chalk_1.default.bold.bgCyan.black(` [Sela] Local DX Review - ${entries.length} fix${entries.length === 1 ? "" : "es"} applied to disk `));
+        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 +89,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 +119,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/PRAutomationService.d.ts

@@ -37,14 +37,14 @@ export interface ExecutionResult {
  * Clean-Room execution lifecycle.
  *
  * The legacy flow committed the developer's whole dirty working tree onto
- * the PR branch — polluting reviewers with `test.only`, commented debug
+ * the PR branch - polluting reviewers with `test.only`, commented debug
  * blocks, etc. This refactor isolates Sela's specific AST changes onto an
  * ephemeral branch, leaves the developer's local workspace byte-identical
  * to its pre-Sela state, and is wrapped in a try/finally that restores
  * the workspace even on mid-flight crashes.
  */
 export declare function execute(cfg: ResolvedPRAutomation, decision: StrategyDecision, heals: HealedEvent[], branchInfo: BranchInfo, ctx: ExecuteContext, 
-/** Injectable for tests — defaults to the module singleton. */
+/** Injectable for tests - defaults to the module singleton. */
 workspace?: WorkspaceSnapshotService): Promise<ExecutionResult>;
 export declare function handleBugDetected(action: BugAction, events: ProtectedEvent[], branchInfo: BranchInfo, ctx: ExecuteContext): Promise<void>;
 //# sourceMappingURL=PRAutomationService.d.ts.map