@rickcedwhat/playwright-smart-table 6.20.1-next.230dedf → 6.20.1-next.6671496

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.20.1-next.230dedf";
+export declare const PLAYWRIGHT_SMART_TABLE_VERSION: "6.20.1-next.6671496";

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.20.1-next.230dedf";
+exports.PLAYWRIGHT_SMART_TABLE_VERSION = "6.20.1-next.6671496";

package/dist/strategies/viewport.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ViewportStrategies = void 0;
+const debugUtils_1 = require("../utils/debugUtils");
 /**
  * Viewport strategies for tables where row and cell elements carry a DOM attribute
  * equal to their index (e.g. Braintrust logs table, TanStack-style grids).
@@ -11,6 +12,10 @@ exports.ViewportStrategies = void 0;
  *   - scrollToColumn       — aligns header's left edge to container left, waits for cell mount
  *   - scrollToRow          — scrolls row into view, waits for row mount
  *
+ * **Important:** `rowAttribute` must be present on the elements matched by `rowSelector`,
+ * not on child cells. If row elements lack the attribute, `getVisibleRowRange` returns
+ * `{first:0, last:0}` and logs a warning at the `error` log level.
+ *
  * @example
  * // TanStack / Braintrust (data-index, 0-based — default)
  * viewport: Strategies.Viewport.dataAttribute()
@@ -34,17 +39,22 @@ const dataAttribute = (options) => {
         getVisibleColumnRange: async ({ root, config }) => {
             const rowSel = config.rowSelector;
             const cellSel = typeof config.cellSelector === 'string' ? config.cellSelector : `[${colAttr}]`;
-            return root.evaluate((el, { rowSel, cellSel, colAttr, colOffset }) => {
+            const result = await root.evaluate((el, { rowSel, cellSel, colAttr, colOffset }) => {
                 const firstRow = el.querySelector(rowSel);
                 if (!firstRow)
-                    return { first: 0, last: 0 };
-                const indices = Array.from(firstRow.querySelectorAll(cellSel))
+                    return { first: 0, last: 0, _cellCount: 0, _validCount: 0 };
+                const cells = Array.from(firstRow.querySelectorAll(cellSel));
+                const indices = cells
                     .map(c => Number(c.getAttribute(colAttr)) - colOffset)
                     .filter(n => !isNaN(n));
                 if (!indices.length)
-                    return { first: 0, last: 0 };
-                return { first: Math.min(...indices), last: Math.max(...indices) };
+                    return { first: 0, last: 0, _cellCount: cells.length, _validCount: 0 };
+                return { first: Math.min(...indices), last: Math.max(...indices), _cellCount: cells.length, _validCount: indices.length };
             }, { rowSel, cellSel, colAttr, colOffset });
+            if (result._cellCount > 0 && result._validCount === 0) {
+                (0, debugUtils_1.logDebug)(config, 'error', `dataAttribute viewport: ${result._cellCount} cell(s) found but none have attribute "${colAttr}". Check that columnAttribute targets cell elements matched by cellSelector.`);
+            }
+            return { first: result.first, last: result.last };
         },
         getVisibleRowIndices: async ({ root, config }) => {
             const rowSel = config.rowSelector;
@@ -72,33 +82,30 @@ const dataAttribute = (options) => {
         },
         getVisibleRowRange: async ({ root, config }) => {
             const rowSel = config.rowSelector;
-            return root.evaluate((el, { rowSel, rowAttr, rowOffset, containerSel }) => {
+            const result = await root.evaluate((el, { rowSel, rowAttr, rowOffset, containerSel }) => {
                 const rows = Array.from(el.querySelectorAll(rowSel));
-                // #353: report only rows actually within the scroll container's vertical
-                // bounds, not every mounted row. Previously overscan rows (kept mounted by
-                // the virtual scroller above/below the fold) were counted as "visible".
-                // Inclusive: a row with ANY vertical overlap counts as visible; only rows
-                // entirely above or below the container are dropped — so we never drop a
-                // partially-visible real row.
                 const container = el.closest(containerSel);
                 const containerRect = container ? container.getBoundingClientRect() : null;
-                const indices = rows
+                const visibleRows = rows
                     .filter(r => {
-                    // Container not resolvable — can't measure, so keep all rows (safe
-                    // fallback, preserves pre-#353 behavior rather than risk dropping real rows).
                     if (!containerRect)
                         return true;
                     const rect = r.getBoundingClientRect();
                     if (rect.height === 0)
-                        return false; // unrendered / detached
+                        return false;
                     return rect.bottom > containerRect.top && rect.top < containerRect.bottom;
-                })
+                });
+                const indices = visibleRows
                     .map(r => Number(r.getAttribute(rowAttr)) - rowOffset)
                     .filter(n => !isNaN(n));
                 if (!indices.length)
-                    return { first: 0, last: 0 };
-                return { first: Math.min(...indices), last: Math.max(...indices) };
+                    return { first: 0, last: 0, _rowCount: visibleRows.length, _validCount: 0 };
+                return { first: Math.min(...indices), last: Math.max(...indices), _rowCount: visibleRows.length, _validCount: indices.length };
             }, { rowSel, rowAttr, rowOffset, containerSel });
+            if (result._rowCount > 0 && result._validCount === 0) {
+                (0, debugUtils_1.logDebug)(config, 'error', `dataAttribute viewport: ${result._rowCount} row(s) found but none have attribute "${rowAttr}". Check that rowAttribute targets row elements matched by rowSelector, not child cells. getVisibleRowRange returning {first:0, last:0}.`);
+            }
+            return { first: result.first, last: result.last };
         },
         scrollToColumn: async ({ root, config }, colIndex) => {
             const headerSel = typeof config.headerSelector === 'string' ? config.headerSelector : null;

package/package.json

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