@rickcedwhat/playwright-smart-table 6.23.1-next.f369c63 → 6.24.0-next.531615

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.f369c63";
+export declare const PLAYWRIGHT_SMART_TABLE_VERSION: "6.24.0-next.531615";

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.f369c63";
+exports.PLAYWRIGHT_SMART_TABLE_VERSION = "6.24.0-next.531615";

package/dist/presets/glide/index.js

@@ -56,7 +56,6 @@ const glideFillSimple = async ({ value, page }) => {
 };
 const glidePaginationStrategy = pagination_1.PaginationStrategies.infiniteScroll({
     scrollTarget: 'xpath=//ancestor::body//div[contains(@class, "dvn-scroller")]',
-    scrollAmount: 500,
     action: 'js-scroll',
     stabilization: stabilization_1.StabilizationStrategies.contentChanged({ timeout: 5000 }),
     timeout: 5000

package/dist/presets/rdg.js

@@ -50,7 +50,6 @@ const rdgGetCellLocator = ({ row, columnIndex }) => {
 };
 const rdgPaginationStrategy = pagination_1.PaginationStrategies.infiniteScroll({
     action: 'js-scroll',
-    scrollAmount: 500,
     stabilization: stabilization_1.StabilizationStrategies.contentChanged({ timeout: 5000 })
 });
 const rdgNavigation = {

package/dist/strategies/pagination.d.ts

@@ -30,7 +30,10 @@ export declare const PaginationStrategies: {
      *
      * @param options.action 'scroll' (mouse wheel) or 'js-scroll' (direct scrollTop).
      * @param options.scrollTarget Selector for the scroll container (defaults to table root).
-     * @param options.scrollAmount Amount to scroll in pixels (default 500).
+     * @param options.scrollAmount Amount to scroll in pixels. Defaults to 80% of the scroll target's
+     *        visible height, so a step never jumps past the rendered window of a virtualized list.
+     *        An explicit value larger than the visible height logs a one-time warning, because
+     *        virtualizers never mount the rows that get skipped.
      * @param options.stabilization Strategy to determine if new content loaded.
      *        Defaults to `rowCountIncreased` (simple append).
      *        Use `contentChanged` for virtualization.

package/dist/strategies/pagination.js

@@ -67,23 +67,46 @@ exports.PaginationStrategies = {
      *
      * @param options.action 'scroll' (mouse wheel) or 'js-scroll' (direct scrollTop).
      * @param options.scrollTarget Selector for the scroll container (defaults to table root).
-     * @param options.scrollAmount Amount to scroll in pixels (default 500).
+     * @param options.scrollAmount Amount to scroll in pixels. Defaults to 80% of the scroll target's
+     *        visible height, so a step never jumps past the rendered window of a virtualized list.
+     *        An explicit value larger than the visible height logs a one-time warning, because
+     *        virtualizers never mount the rows that get skipped.
      * @param options.stabilization Strategy to determine if new content loaded.
      *        Defaults to `rowCountIncreased` (simple append).
      *        Use `contentChanged` for virtualization.
      */
     infiniteScroll: (options = {}) => {
-        var _a, _b;
+        var _a;
         // Default stabilization: Wait for row count to increase (Append mode)
         const stabilization = (_a = options.stabilization) !== null && _a !== void 0 ? _a : stabilization_1.StabilizationStrategies.rowCountIncreased({ timeout: options.timeout });
-        const amount = (_b = options.scrollAmount) !== null && _b !== void 0 ? _b : 500;
+        let warnedOversizedStep = false;
         const createScroller = (directionMultiplier) => {
             return async (context) => {
                 const { root, resolve, page } = context;
                 const scrollTarget = options.scrollTarget
                     ? resolve(options.scrollTarget, root)
                     : root;
-                const beforeScrollTop = await scrollTarget.evaluate((el) => el.scrollTop);
+                const metrics = await scrollTarget.evaluate((el) => {
+                    const viewportHeight = window.innerHeight || document.documentElement.clientHeight;
+                    return {
+                        scrollTop: el.scrollTop,
+                        // A non-scrolling target scrolls the page, so the window bounds what is rendered.
+                        visibleHeight: el.clientHeight > 0 ? Math.min(el.clientHeight, viewportHeight) : viewportHeight,
+                        isScrollContainer: el.scrollHeight > el.clientHeight + 1,
+                    };
+                });
+                const beforeScrollTop = metrics.scrollTop;
+                let amount;
+                if (options.scrollAmount !== undefined) {
+                    amount = options.scrollAmount;
+                    if (!warnedOversizedStep && metrics.isScrollContainer && amount > metrics.visibleHeight) {
+                        warnedOversizedStep = true;
+                        console.warn(`[SmartTable] infiniteScroll scrollAmount (${amount}px) is larger than the scroll target's visible height (${Math.round(metrics.visibleHeight)}px). Virtualized lists only render rows near the viewport, so rows between steps are never mounted and will be silently skipped. Omit scrollAmount to step by the visible height, or pass a smaller value.`);
+                    }
+                }
+                else {
+                    amount = Math.floor(metrics.visibleHeight * 0.8) || 500;
+                }
                 const doScroll = async () => {
                     const box = await scrollTarget.boundingBox();
                     const scrollValue = amount * directionMultiplier;

package/dist/strategies/resolution.d.ts

@@ -1,4 +1,5 @@
 import { StrategyContext } from '../types';
+/** @deprecated Not used by the library. Will be removed in v7. */
 export interface ColumnResolutionStrategy {
     /**
      * Resolves a column name (string or Regex) to a column index.
@@ -17,6 +18,3 @@ export interface ColumnResolutionStrategy {
         headerMap: Map<string, number>;
     }): string;
 }
-export declare const ResolutionStrategies: {
-    default: ColumnResolutionStrategy;
-};

package/dist/strategies/resolution.js

@@ -1,30 +1,2 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ResolutionStrategies = void 0;
-exports.ResolutionStrategies = {
-    default: {
-        resolveIndex: ({ query, headerMap }) => {
-            // 1. Exact / String Match
-            if (typeof query === 'string') {
-                if (headerMap.has(query))
-                    return headerMap.get(query);
-            }
-            // 2. Regex Match
-            if (query instanceof RegExp) {
-                for (const [colName, idx] of headerMap.entries()) {
-                    if (query.test(colName))
-                        return idx;
-                }
-                return undefined;
-            }
-            // 3. (Optional) Fuzzy String Match fallback could go here
-            // But for strict default strategy, we might want to keep it simple first
-            // The original code didn't do fuzzy *resolution* logic inside the get(), it just did strict get().
-            // The fuzzy logic was only for *suggestions* on error.
-            return undefined;
-        },
-        resolveName: ({ query }) => {
-            return query.toString();
-        }
-    }
-};

package/dist/strategies/sorting.js

@@ -12,7 +12,7 @@ exports.SortingStrategies = {
      */
     AriaSort: () => {
         return {
-            async doSort({ columnName, direction, context }) {
+            async doSort({ columnName, context }) {
                 // getHeaderCell is always present on TableContext after table is initialized
                 const targetHeader = await context.getHeaderCell(columnName);
                 // The table engine handles verify-and-retry. We only provide the trigger here.

package/dist/useTable.js

@@ -38,7 +38,7 @@ const smartRowInternals_1 = require("./utils/smartRowInternals");
 // Helper to safely serialize objects containing functions for logging
 const safeStringify = (obj) => {
     try {
-        return JSON.stringify(obj, (key, value) => {
+        return JSON.stringify(obj, (_key, value) => {
             if (typeof value === 'function') {
                 return '[Function]';
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rickcedwhat/playwright-smart-table",
-  "version": "6.23.1-next.f369c63",
+  "version": "6.24.0-next.531615",
   "description": "Smart, column-aware table interactions for Playwright",
   "author": "Cedrick Catalan",
   "license": "MIT",
@@ -53,9 +53,7 @@
     },
     "./types": {
       "types": "./dist/types.d.ts",
-      "import": "./dist/index.js",
-      "require": "./dist/index.js",
-      "default": "./dist/index.js"
+      "default": "./dist/types.js"
     },
     "./presets": {
       "types": "./dist/presets/index.d.ts",