@rickcedwhat/playwright-smart-table 6.23.0 → 6.23.1-next.d229498

package/dist/filterEngine.d.ts

@@ -1,4 +1,4 @@
-import { Locator, Page } from "@playwright/test";
+import type { Locator, Page } from "@playwright/test";
 import { FinalTableConfig, FilterValue } from "./types";
 export declare class FilterEngine {
     private config;

package/dist/packageVersion.d.ts

@@ -3,4 +3,4 @@
  * Generated by scripts/embed-version.mjs from package.json
  */
 /** Semver of this package. Use in logs or diagnostics to confirm the installed build. */
-export declare const PLAYWRIGHT_SMART_TABLE_VERSION: "6.23.0";
+export declare const PLAYWRIGHT_SMART_TABLE_VERSION: "6.23.1-next.d229498";

package/dist/packageVersion.js

@@ -6,4 +6,4 @@ exports.PLAYWRIGHT_SMART_TABLE_VERSION = void 0;
  * Generated by scripts/embed-version.mjs from package.json
  */
 /** Semver of this package. Use in logs or diagnostics to confirm the installed build. */
-exports.PLAYWRIGHT_SMART_TABLE_VERSION = "6.23.0";
+exports.PLAYWRIGHT_SMART_TABLE_VERSION = "6.23.1-next.d229498";

package/dist/presets/glide/headers.d.ts

@@ -1,6 +1,8 @@
 import { StrategyContext } from '../../types';
 /**
- * Scans for headers by finding a scrollable container and setting scrollLeft.
+ * Glide header discovery. Same scroll loop as `HeaderStrategies.horizontalScroll`, but the
+ * `.dvn-scroller` is a sibling of the canvas root rather than an ancestor/descendant, so the
+ * lookup falls back to a document-wide search.
  */
 export declare const scrollRightHeader: (context: StrategyContext, options?: {
     limit?: number;

package/dist/presets/glide/headers.js

@@ -1,59 +1,24 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.scrollRightHeader = void 0;
-const debugUtils_1 = require("../../utils/debugUtils");
+const headerScroll_1 = require("../../utils/headerScroll");
 /**
- * Scans for headers by finding a scrollable container and setting scrollLeft.
+ * Glide header discovery. Same scroll loop as `HeaderStrategies.horizontalScroll`, but the
+ * `.dvn-scroller` is a sibling of the canvas root rather than an ancestor/descendant, so the
+ * lookup falls back to a document-wide search.
  */
 const scrollRightHeader = async (context, options) => {
-    var _a, _b;
-    const { resolve, config, root, page } = context;
-    const limit = (_a = options === null || options === void 0 ? void 0 : options.limit) !== null && _a !== void 0 ? _a : 20;
-    const scrollAmount = (_b = options === null || options === void 0 ? void 0 : options.scrollAmount) !== null && _b !== void 0 ? _b : 300;
-    const collectedHeaders = new Set();
-    const getVisible = async () => {
-        const headerLoc = resolve(config.headerSelector, root);
-        const texts = await headerLoc.allInnerTexts();
-        return texts.map(t => t.trim());
-    };
-    let currentHeaders = await getVisible();
-    currentHeaders.forEach(h => collectedHeaders.add(h));
-    const scrollerHandle = await root.evaluateHandle((el, selector) => {
+    const findScroller = () => context.root.evaluateHandle((el, selector) => {
+        var _a;
         if (selector && el.matches(selector))
             return el;
         const effectiveSelector = selector || '.dvn-scroller';
-        const ancestor = el.closest(effectiveSelector);
-        if (ancestor)
-            return ancestor;
-        return document.querySelector(effectiveSelector);
+        return (_a = el.closest(effectiveSelector)) !== null && _a !== void 0 ? _a : document.querySelector(effectiveSelector);
     }, options === null || options === void 0 ? void 0 : options.selector);
-    const isScrollerFound = await scrollerHandle.evaluate(el => !!el);
-    if (isScrollerFound) {
-        await scrollerHandle.evaluate(el => el.scrollLeft = 0);
-        await page.waitForTimeout(200);
-        for (let i = 0; i < limit; i++) {
-            const sizeBefore = collectedHeaders.size;
-            await scrollerHandle.evaluate((el, amount) => el.scrollLeft += amount, scrollAmount);
-            await page.waitForTimeout(300);
-            const newHeaders = await getVisible();
-            newHeaders.forEach(h => collectedHeaders.add(h));
-            if (collectedHeaders.size === sizeBefore) {
-                await scrollerHandle.evaluate((el, amount) => el.scrollLeft += amount, scrollAmount);
-                await page.waitForTimeout(300);
-                const retryHeaders = await getVisible();
-                retryHeaders.forEach(h => collectedHeaders.add(h));
-                if (collectedHeaders.size === sizeBefore)
-                    break;
-            }
-        }
-    }
-    else {
-        (0, debugUtils_1.logDebug)(config, 'info', "HeaderStrategies.scrollRight: Could not find scroller. Returning visible headers.");
-    }
-    if (isScrollerFound) {
-        await scrollerHandle.evaluate(el => el.scrollLeft = 0);
-        await page.waitForTimeout(200);
-    }
-    return Array.from(collectedHeaders);
+    return (0, headerScroll_1.scrollAndCollectHeaders)(context, findScroller, {
+        limit: options === null || options === void 0 ? void 0 : options.limit,
+        scrollAmount: options === null || options === void 0 ? void 0 : options.scrollAmount,
+        notFoundMessage: 'HeaderStrategies.scrollRight: Could not find scroller. Returning visible headers.',
+    });
 };
 exports.scrollRightHeader = scrollRightHeader;

package/dist/smartRow.d.ts

@@ -1,4 +1,4 @@
-import { type Locator, type Page } from '@playwright/test';
+import type { Locator, Page } from '@playwright/test';
 import { SmartRow as SmartRowType, FinalTableConfig, TableResult } from './types';
 import { NavigationBarrier } from './utils/navigationBarrier';
 /**

package/dist/smartRow.js

@@ -1,6 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-const test_1 = require("@playwright/test");
 const rowResolution_1 = require("./engine/rowResolution");
 const fill_1 = require("./strategies/fill");
 const stringUtils_1 = require("./utils/stringUtils");
@@ -80,7 +79,7 @@ const _navigateToCell = async (params) => {
                         cellAttached = true;
                     }
                     catch (error) {
-                        if (!(error instanceof test_1.errors.TimeoutError))
+                        if ((error === null || error === void 0 ? void 0 : error.name) !== 'TimeoutError')
                             throw error;
                     }
                 }

package/dist/strategies/headers.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.HeaderStrategies = void 0;
 const debugUtils_1 = require("../utils/debugUtils");
+const headerScroll_1 = require("../utils/headerScroll");
 exports.HeaderStrategies = {
     /**
      * Default strategy: Returns only the headers currently visible in the DOM.
@@ -30,62 +31,18 @@ exports.HeaderStrategies = {
      */
     horizontalScroll: (options) => {
         return async (context) => {
-            var _a, _b;
-            const { resolve, config, root, page } = context;
-            const limit = (_a = options === null || options === void 0 ? void 0 : options.limit) !== null && _a !== void 0 ? _a : 20;
-            const scrollAmount = (_b = options === null || options === void 0 ? void 0 : options.scrollAmount) !== null && _b !== void 0 ? _b : 300;
-            const collectedHeaders = new Set();
-            const getVisible = async () => {
-                const headerLoc = resolve(config.headerSelector, root);
-                const texts = await headerLoc.allInnerTexts();
-                return texts.map(t => t.trim());
-            };
-            let currentHeaders = await getVisible();
-            currentHeaders.forEach(h => collectedHeaders.add(h));
             const selector = options === null || options === void 0 ? void 0 : options.selector;
             if (!selector) {
-                (0, debugUtils_1.logDebug)(config, 'info', 'HeaderStrategies.horizontalScroll: no selector provided — returning visible headers only. Pass { selector } for the scroll container (e.g. ".dvn-scroller").');
-                return Array.from(collectedHeaders);
+                (0, debugUtils_1.logDebug)(context.config, 'info', 'HeaderStrategies.horizontalScroll: no selector provided — returning visible headers only. Pass { selector } for the scroll container (e.g. ".dvn-scroller").');
+                return (0, headerScroll_1.readVisibleHeaders)(context);
             }
-            const scrollerHandle = await root.evaluateHandle((el, sel) => {
-                if (el.matches(sel))
-                    return el;
-                const ancestor = el.closest(sel);
-                if (ancestor)
-                    return ancestor;
-                const child = el.querySelector(sel);
-                if (child)
-                    return child;
-                return null;
-            }, selector);
-            const isScrollerFound = await scrollerHandle.evaluate(el => !!el);
-            if (isScrollerFound) {
-                await scrollerHandle.evaluate(el => el.scrollLeft = 0);
-                await page.waitForTimeout(200);
-                for (let i = 0; i < limit; i++) {
-                    const sizeBefore = collectedHeaders.size;
-                    await scrollerHandle.evaluate((el, amount) => el.scrollLeft += amount, scrollAmount);
-                    await page.waitForTimeout(300);
-                    const newHeaders = await getVisible();
-                    newHeaders.forEach(h => collectedHeaders.add(h));
-                    if (collectedHeaders.size === sizeBefore) {
-                        await scrollerHandle.evaluate((el, amount) => el.scrollLeft += amount, scrollAmount);
-                        await page.waitForTimeout(300);
-                        const retryHeaders = await getVisible();
-                        retryHeaders.forEach(h => collectedHeaders.add(h));
-                        if (collectedHeaders.size === sizeBefore)
-                            break;
-                    }
-                }
-            }
-            else {
-                (0, debugUtils_1.logDebug)(config, 'info', `HeaderStrategies.horizontalScroll: Could not find scroller matching "${selector}". Returning visible headers.`);
-            }
-            if (isScrollerFound) {
-                await scrollerHandle.evaluate(el => el.scrollLeft = 0);
-                await page.waitForTimeout(200);
-            }
-            return Array.from(collectedHeaders);
+            // Self, then ancestor, then descendant — scoped to the table, never document-wide.
+            const findScroller = () => context.root.evaluateHandle((el, sel) => { var _a; return el.matches(sel) ? el : (_a = el.closest(sel)) !== null && _a !== void 0 ? _a : el.querySelector(sel); }, selector);
+            return (0, headerScroll_1.scrollAndCollectHeaders)(context, findScroller, {
+                limit: options === null || options === void 0 ? void 0 : options.limit,
+                scrollAmount: options === null || options === void 0 ? void 0 : options.scrollAmount,
+                notFoundMessage: `HeaderStrategies.horizontalScroll: Could not find scroller matching "${selector}". Returning visible headers.`,
+            });
         };
     }
 };

package/dist/utils/elementTracker.d.ts

@@ -1,4 +1,4 @@
-import { Locator } from '@playwright/test';
+import type { Locator } from '@playwright/test';
 export declare class ElementTracker {
     readonly id: string;
     constructor(prefix?: string);

package/dist/utils/headerScroll.d.ts

@@ -0,0 +1,13 @@
+import type { JSHandle } from '@playwright/test';
+import type { StrategyContext } from '../types';
+export declare const readVisibleHeaders: ({ resolve, config, root }: StrategyContext) => Promise<string[]>;
+/**
+ * Horizontal header discovery shared by `HeaderStrategies.horizontalScroll` and the Glide preset:
+ * reset the scroller to the left, step right collecting headers until two consecutive steps add
+ * nothing (or `limit` is hit), then scroll back. Callers differ only in how they find the scroller.
+ */
+export declare function scrollAndCollectHeaders(context: StrategyContext, findScroller: () => Promise<JSHandle<Element | null>>, options: {
+    limit?: number;
+    scrollAmount?: number;
+    notFoundMessage: string;
+}): Promise<string[]>;

package/dist/utils/headerScroll.js

@@ -0,0 +1,47 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.readVisibleHeaders = void 0;
+exports.scrollAndCollectHeaders = scrollAndCollectHeaders;
+const debugUtils_1 = require("./debugUtils");
+const readVisibleHeaders = async ({ resolve, config, root }) => {
+    const texts = await resolve(config.headerSelector, root).allInnerTexts();
+    return texts.map(t => t.trim());
+};
+exports.readVisibleHeaders = readVisibleHeaders;
+/**
+ * Horizontal header discovery shared by `HeaderStrategies.horizontalScroll` and the Glide preset:
+ * reset the scroller to the left, step right collecting headers until two consecutive steps add
+ * nothing (or `limit` is hit), then scroll back. Callers differ only in how they find the scroller.
+ */
+async function scrollAndCollectHeaders(context, findScroller, options) {
+    var _a, _b;
+    const { config, page } = context;
+    const limit = (_a = options.limit) !== null && _a !== void 0 ? _a : 20;
+    const scrollAmount = (_b = options.scrollAmount) !== null && _b !== void 0 ? _b : 300;
+    const collectedHeaders = new Set(await (0, exports.readVisibleHeaders)(context));
+    const scrollerHandle = await findScroller();
+    const isScrollerFound = await scrollerHandle.evaluate(el => !!el);
+    if (!isScrollerFound) {
+        (0, debugUtils_1.logDebug)(config, 'info', options.notFoundMessage);
+        return Array.from(collectedHeaders);
+    }
+    const collectAfterStep = async () => {
+        await scrollerHandle.evaluate((el, amount) => el.scrollLeft += amount, scrollAmount);
+        await page.waitForTimeout(300);
+        (await (0, exports.readVisibleHeaders)(context)).forEach(h => collectedHeaders.add(h));
+    };
+    await scrollerHandle.evaluate(el => el.scrollLeft = 0);
+    await page.waitForTimeout(200);
+    for (let i = 0; i < limit; i++) {
+        const sizeBefore = collectedHeaders.size;
+        await collectAfterStep();
+        if (collectedHeaders.size === sizeBefore) {
+            await collectAfterStep();
+            if (collectedHeaders.size === sizeBefore)
+                break;
+        }
+    }
+    await scrollerHandle.evaluate(el => el.scrollLeft = 0);
+    await page.waitForTimeout(200);
+    return Array.from(collectedHeaders);
+}

package/dist/utils.d.ts

@@ -1,4 +1,4 @@
-import { Page } from '@playwright/test';
+import type { Page } from '@playwright/test';
 /**
  * Internal helper to wait for a condition to be met.
  * Replaces the dependency on 'expect(...).toPass()' to ensure compatibility

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rickcedwhat/playwright-smart-table",
-  "version": "6.23.0",
+  "version": "6.23.1-next.d229498",
   "description": "Smart, column-aware table interactions for Playwright",
   "author": "Cedrick Catalan",
   "license": "MIT",
@@ -29,7 +29,7 @@
     "docs:verify-links": "node scripts/verify-vitepress-links.mjs",
     "embed-version": "node scripts/embed-version.mjs",
     "generate-og": "node scripts/generate-og.mjs",
-    "build": "pnpm run embed-version && pnpm run generate-types && pnpm run generate-config-types && pnpm run generate-docs && pnpm run generate-all-api-docs && pnpm run update-all-api-signatures && tsc",
+    "build": "pnpm run embed-version && pnpm run generate-types && pnpm run generate-config-types && pnpm run generate-docs && pnpm run generate-all-api-docs && pnpm run update-all-api-signatures && tsc && node scripts/check-no-runtime-playwright.mjs",
     "prepublishOnly": "pnpm run build",
     "clean-port": "lsof -ti:3000 | xargs kill -9 || true",
     "pretest": "pnpm run clean-port",