@rickcedwhat/playwright-smart-table 6.23.1-next.3ae8c6f → 6.23.1

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.1-next.3ae8c6f";
+export declare const PLAYWRIGHT_SMART_TABLE_VERSION: "6.23.1";

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.1-next.3ae8c6f";
+exports.PLAYWRIGHT_SMART_TABLE_VERSION = "6.23.1";

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

@@ -1,8 +1,6 @@
 import { StrategyContext } from '../../types';
 /**
- * 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.
+ * Scans for headers by finding a scrollable container and setting scrollLeft.
  */
 export declare const scrollRightHeader: (context: StrategyContext, options?: {
     limit?: number;

package/dist/presets/glide/headers.js

@@ -1,24 +1,59 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.scrollRightHeader = void 0;
-const headerScroll_1 = require("../../utils/headerScroll");
+const debugUtils_1 = require("../../utils/debugUtils");
 /**
- * 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.
+ * Scans for headers by finding a scrollable container and setting scrollLeft.
  */
 const scrollRightHeader = async (context, options) => {
-    const findScroller = () => context.root.evaluateHandle((el, selector) => {
-        var _a;
+    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) => {
         if (selector && el.matches(selector))
             return el;
         const effectiveSelector = selector || '.dvn-scroller';
-        return (_a = el.closest(effectiveSelector)) !== null && _a !== void 0 ? _a : document.querySelector(effectiveSelector);
+        const ancestor = el.closest(effectiveSelector);
+        if (ancestor)
+            return ancestor;
+        return document.querySelector(effectiveSelector);
     }, options === null || options === void 0 ? void 0 : options.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.scrollRight: Could not find scroller. Returning visible headers.',
-    });
+    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);
 };
 exports.scrollRightHeader = scrollRightHeader;

package/dist/strategies/headers.js

@@ -2,7 +2,6 @@
 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.
@@ -31,18 +30,62 @@ 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)(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);
+                (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);
             }
-            // 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.`,
-            });
+            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);
         };
     }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rickcedwhat/playwright-smart-table",
-  "version": "6.23.1-next.3ae8c6f",
+  "version": "6.23.1",
   "description": "Smart, column-aware table interactions for Playwright",
   "author": "Cedrick Catalan",
   "license": "MIT",

package/dist/utils/headerScroll.d.ts

@@ -1,13 +0,0 @@
-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

@@ -1,47 +0,0 @@
-"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);
-}