sela-core 1.0.1 → 1.0.2

package/dist/cli/index.js

@@ -58,7 +58,7 @@ const program = new commander_1.Command();
 program
     .name('sela')
     .description('Sela CLI — proactive DNA management for Fixwright')
-    .version('1.0.1')
+    .version('1.0.2')
     .exitOverride(ErrorHandler_js_1.handleError)
     .configureOutput({ outputError: () => { } });
 // ── sela init ────────────────────────────────────────────────────────────────

package/dist/fixtures/expectProxy.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"expectProxy.d.ts","sourceRoot":"","sources":["../../src/fixtures/expectProxy.ts"],"names":[],"mappings":"AAEA,OAAO,EAEL,OAAO,EAER,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAC;AA2N5D;;;;;;GAMG;AACH,wBAAgB,mBAAmB,CACjC,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,OAAO,kBAAkB,EAAE,IAAI,EACrC,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,EAAE,6CAA6C;AAClE,YAAY,CAAC,EAAE,CAAC,KAAK,EAAE,OAAO,KAAK,OAAO,GAAG,IAAI,GAChD,CAAC,cAAc,EAAE,OAAO,KAAK,GAAG,CA2DlC"}
+{"version":3,"file":"expectProxy.d.ts","sourceRoot":"","sources":["../../src/fixtures/expectProxy.ts"],"names":[],"mappings":"AAEA,OAAO,EAEL,OAAO,EAER,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAC;AAgO5D;;;;;;GAMG;AACH,wBAAgB,mBAAmB,CACjC,MAAM,EAAE,eAAe,EACvB,IAAI,EAAE,OAAO,kBAAkB,EAAE,IAAI,EACrC,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,EAAE,6CAA6C;AAClE,YAAY,CAAC,EAAE,CAAC,KAAK,EAAE,OAAO,KAAK,OAAO,GAAG,IAAI,GAChD,CAAC,cAAc,EAAE,OAAO,KAAK,GAAG,CAiElC"}

package/dist/fixtures/expectProxy.js

@@ -29,6 +29,11 @@ function isRealLocator(value) {
         return false;
     if (typeof value !== "object")
         return false;
+    // Fixwright locator proxies expose `_fixwrightSelector`. The proxy forwards
+    // `_frame` via Reflect.get so `hasFrame` would lie — explicitly reject them
+    // here. Callers should unwrap via `resolveProxy` first.
+    if (value._fixwrightSelector !== undefined)
+        return false;
     const hasFrame = value._frame !== undefined ||
         value._impl?._frame !== undefined;
     const isFrameLocator = value._frameSelector !== undefined ||
@@ -206,6 +211,11 @@ resolveProxy) {
                 resolved = unwrapped;
         }
         const base = (0, test_1.expect)(resolved);
+        // Defensive: if for any reason `base` is not an object, return it as-is
+        // rather than crashing inside `new Proxy(...)`.
+        if (base === null || typeof base !== "object") {
+            return base;
+        }
         // Step 2 — only wrap matchers for genuine page-level Locator objects.
         if (!isRealLocator(resolved)) {
             return base;

package/dist/fixtures/index.d.ts

@@ -1,4 +1,6 @@
+import { Locator } from "@playwright/test";
 import { createHealingExpect } from "./expectProxy";
+export declare function resolveFixwrightProxy(value: unknown): Locator | null;
 export declare const test: import("@playwright/test").TestType<import("@playwright/test").PlaywrightTestArgs & import("@playwright/test").PlaywrightTestOptions & {
     expect: ReturnType<typeof createHealingExpect>;
 }, import("@playwright/test").PlaywrightWorkerArgs & import("@playwright/test").PlaywrightWorkerOptions>;

package/dist/fixtures/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/fixtures/index.ts"],"names":[],"mappings":"AAQA,OAAO,EAAE,mBAAmB,EAAE,MAAM,eAAe,CAAC;AAk2BpD,eAAO,MAAM,IAAI;YACP,UAAU,CAAC,OAAO,mBAAmB,CAAC;wGA+F9C,CAAC;AAGH,OAAO,EAAE,mBAAmB,EAAE,MAAM,eAAe,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/fixtures/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAEL,OAAO,EAER,MAAM,kBAAkB,CAAC;AAI1B,OAAO,EAAE,mBAAmB,EAAE,MAAM,eAAe,CAAC;AAWpD,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,GAAG,IAAI,CAIpE;AA02BD,eAAO,MAAM,IAAI;YACP,UAAU,CAAC,OAAO,mBAAmB,CAAC;wGA+F9C,CAAC;AAGH,OAAO,EAAE,mBAAmB,EAAE,MAAM,eAAe,CAAC"}

package/dist/fixtures/index.js

@@ -34,6 +34,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createHealingExpect = exports.test = void 0;
+exports.resolveFixwrightProxy = resolveFixwrightProxy;
 const test_1 = require("@playwright/test");
 const StackUtils_1 = require("../utils/StackUtils");
 const HealingRegistry_1 = require("../engine/HealingRegistry");
@@ -43,6 +44,15 @@ const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const registry = HealingRegistry_1.HealingRegistry.getInstance();
 const proxyToActiveLocator = new WeakMap();
+// Module-level resolver — moduleExpect.ts uses this to unwrap a Fixwright
+// locator proxy back to its live underlying Playwright Locator without
+// needing access to the WeakMap directly.
+function resolveFixwrightProxy(value) {
+    if (!value || typeof value !== "object")
+        return null;
+    const getActive = proxyToActiveLocator.get(value);
+    return getActive ? getActive() : null;
+}
 // ─── Action classification (identical to original) ────────────────
 const INTERCEPTED_ACTIONS = [
     "click",
@@ -461,6 +471,12 @@ function createFrameLocatorProxy(rawFrameLocator, frameSelector, rawPage, testTi
 //  3. Exposes _fixwrightUnsubscribe so the fixture can clean up.
 // ─────────────────────────────────────────────────────────────────
 function createLocatorProxy(rawLocator, selector, rawPage, testTitle, actionCounter, elementSelectorOnly) {
+    // Defensive: Proxy ctor throws "Cannot create proxy with a non-object as target"
+    // when downstream callers (e.g. chained traps) pass a non-Locator return value.
+    // Returning the raw value lets the caller see it unchanged instead of crashing.
+    if (rawLocator === null || rawLocator === undefined || typeof rawLocator !== "object") {
+        return rawLocator;
+    }
     // ── Resolve immediately (handles post-heal proxy creation) ────
     const initialResolved = registry.resolveSelector(selector);
     let currentFullSelector = initialResolved;
@@ -517,9 +533,13 @@ function createLocatorProxy(rawLocator, selector, rawPage, testTitle, actionCoun
             // ── getBy* ───────────────────────────────────────────────
             if (prop in GET_BY_METHODS) {
                 return (...args) => {
+                    const nextLocator = activeLocator[prop](...args);
+                    if (nextLocator === null || nextLocator === undefined || typeof nextLocator !== "object") {
+                        return nextLocator;
+                    }
                     const semanticSelector = GET_BY_METHODS[prop](args);
                     const childFullSelector = `${currentFullSelector} >> ${semanticSelector}`;
-                    return createLocatorProxy(activeLocator[prop](...args), childFullSelector, rawPage, testTitle, actionCounter, semanticSelector);
+                    return createLocatorProxy(nextLocator, childFullSelector, rawPage, testTitle, actionCounter, semanticSelector);
                 };
             }
             // ── Chainable methods ────────────────────────────────────
@@ -527,6 +547,9 @@ function createLocatorProxy(rawLocator, selector, rawPage, testTitle, actionCoun
             if (chainables.includes(prop)) {
                 return (...args) => {
                     const nextLocator = activeLocator[prop](...args);
+                    if (nextLocator === null || nextLocator === undefined || typeof nextLocator !== "object") {
+                        return nextLocator;
+                    }
                     let nextFull = currentFullSelector;
                     let nextElement = currentElementSelector;
                     if (prop === "locator") {

package/dist/fixtures/moduleExpect.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"moduleExpect.d.ts","sourceRoot":"","sources":["../../src/fixtures/moduleExpect.ts"],"names":[],"mappings":"AAsCA,wBAAgB,MAAM,CAAC,KAAK,EAAE,OAAO,GAAG,GAAG,CAO1C"}
+{"version":3,"file":"moduleExpect.d.ts","sourceRoot":"","sources":["../../src/fixtures/moduleExpect.ts"],"names":[],"mappings":"AAuCA,wBAAgB,MAAM,CAAC,KAAK,EAAE,OAAO,GAAG,GAAG,CAa1C"}

package/dist/fixtures/moduleExpect.js

@@ -7,6 +7,7 @@ exports.expect = expect;
 const test_1 = require("@playwright/test");
 const singleton_js_1 = require("../engine/singleton.js");
 const expectProxy_js_1 = require("./expectProxy.js");
+const index_js_1 = require("./index.js");
 function extractPage(value) {
     if (!value || typeof value !== 'object')
         return null;
@@ -40,7 +41,7 @@ function expect(value) {
     const page = extractPage(value);
     if (page) {
         const filePath = getCallerFile();
-        return (0, expectProxy_js_1.createHealingExpect)(singleton_js_1.sharedEngine, page, filePath, 0)(value);
+        return (0, expectProxy_js_1.createHealingExpect)(singleton_js_1.sharedEngine, page, filePath, 0, index_js_1.resolveFixwrightProxy)(value);
     }
     return (0, test_1.expect)(value);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sela-core",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "AI self-healing Playwright wrapper — drop-in replacement for @playwright/test",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",