@rickcedwhat/playwright-smart-table 2.0.2 → 2.0.6

package/dist/strategies/index.d.ts

@@ -15,9 +15,14 @@ export declare const TableStrategies: {
      */
     clickLoadMore: (buttonSelector: Selector, timeout?: number) => PaginationStrategy;
     /**
-     * Strategy: Scrolls to the bottom of the table and waits for more rows to appear.
+     * Strategy: Scrolls a specific container (or the window) to the bottom.
      * Best for: Infinite Scroll grids (Ag-Grid, Virtual Lists)
-     * * @param timeout - Wait timeout (default: 5000ms)
+     * * @param options.timeout - Wait timeout (default: 5000ms)
+     * @param options.scrollerSelector - (Optional) Selector for the scrollable container.
+     * If omitted, tries to scroll the table root.
      */
-    infiniteScroll: (timeout?: number) => PaginationStrategy;
+    infiniteScroll: (options?: {
+        timeout?: number;
+        scrollerSelector?: Selector;
+    }) => PaginationStrategy;
 };

package/dist/strategies/index.js

@@ -74,24 +74,37 @@ exports.TableStrategies = {
         });
     },
     /**
-     * Strategy: Scrolls to the bottom of the table and waits for more rows to appear.
+     * Strategy: Scrolls a specific container (or the window) to the bottom.
      * Best for: Infinite Scroll grids (Ag-Grid, Virtual Lists)
-     * * @param timeout - Wait timeout (default: 5000ms)
+     * * @param options.timeout - Wait timeout (default: 5000ms)
+     * @param options.scrollerSelector - (Optional) Selector for the scrollable container.
+     * If omitted, tries to scroll the table root.
      */
-    infiniteScroll: (timeout = 5000) => {
+    infiniteScroll: (options) => {
+        const { timeout = 5000, scrollerSelector } = options || {};
         return (_a) => __awaiter(void 0, [_a], void 0, function* ({ root, config, resolve, page }) {
             const rows = resolve(config.rowSelector, root);
             const oldCount = yield rows.count();
             if (oldCount === 0)
                 return false;
-            // Aggressive Scroll Logic:
-            // We use expect.poll to RETRY the scroll action if the count hasn't increased.
-            // This fixes flakiness where the first scroll might be missed by the intersection observer.
             try {
                 yield test_1.expect.poll(() => __awaiter(void 0, void 0, void 0, function* () {
-                    // 1. Trigger: Scroll the last row into view
-                    yield rows.last().scrollIntoViewIfNeeded();
-                    // 2. Force: Press "End" to help with stubborn window-scrollers
+                    // 1. Determine target container
+                    // If user provided a specific scroller (e.g. the div WRAPPING the table), use it.
+                    // Otherwise, default to the root locator.
+                    const scroller = scrollerSelector
+                        ? resolve(scrollerSelector, root)
+                        : root;
+                    // 2. Perform the Scroll
+                    // Method A: DOM Manipulation (Fastest/Most Reliable for containers)
+                    // We set scrollTop to a huge number to force it to the bottom
+                    yield scroller.evaluate((el) => {
+                        el.scrollTop = el.scrollHeight;
+                    }).catch(() => { }); // Ignore if element doesn't support scrollTop (e.g. it's a window wrapper)
+                    // Method B: Playwright Native (Fallback)
+                    // Scroll the last row into view (good for Window scroll)
+                    yield rows.last().scrollIntoViewIfNeeded().catch(() => { });
+                    // Method C: Keyboard (Desperation)
                     yield page.keyboard.press('End');
                     // 3. Return count for assertion
                     return rows.count();

package/dist/typeContext.d.ts

@@ -3,4 +3,4 @@
  * This file is generated by scripts/embed-types.js
  * It contains the raw text of types.ts to provide context for LLM prompts.
  */
-export declare const TYPE_CONTEXT = "\n// src/types.ts\n\nexport type Selector = string | ((root: Locator | Page) => Locator);\n\nexport type SmartRow = Locator & {\n  getCell(column: string): Locator;\n  toJSON(): Promise<Record<string, string>>;\n};\n\nexport interface TableContext {\n  root: Locator;\n  config: Required<TableConfig>;\n  page: Page;\n  resolve: (selector: Selector, parent: Locator | Page) => Locator;\n}\n\n/**\n * A function that handles pagination logic.\n * Returns true if more data was loaded (navigation occurred), false if end of data.\n */\nexport type PaginationStrategy = (context: TableContext) => Promise<boolean>;\n\nexport interface PromptOptions {\n  /** * Where to output the prompt.\n   * - 'console': Logs to stdout (default).\n   * - 'report': Attaches to the Playwright HTML report (best for CI/VMs).\n   * - 'file': Writes to a local file (e.g. 'smart-table-prompt.md').\n   */\n  output?: 'console' | 'report' | 'file';\n  \n  /**\n   * Include TypeScript interfaces in the prompt?\n   * Helps LLMs generate correct code structure.\n   * Default: true\n   */\n  includeTypes?: boolean;\n}\n\nexport interface TableConfig {\n  rowSelector?: Selector;\n  headerSelector?: Selector;\n  cellSelector?: Selector;\n  /**\n   * Strategy for handling pagination.\n   * Use presets from TableStrategies or write your own.\n   */\n  pagination?: PaginationStrategy;\n  maxPages?: number;\n  /**\n   * Optional hook to rename columns dynamically.\n   * Useful for naming empty columns (like '__col_0') to something semantic like 'Actions'.\n   */\n  headerTransformer?: (text: string, index: number) => string;\n  /**\n   * Automatically scroll the table into view on first interaction.\n   * Helps ensure the table is visible in traces/videos.\n   * Default: true\n   */\n  autoScroll?: boolean;\n}\n\nexport interface TableResult {\n  getHeaders: () => Promise<string[]>;\n  getHeaderCell: (columnName: string) => Promise<Locator>;\n\n  /** * Find a specific row by its content.\n   * Default: Returns SmartRow (Locator).\n   * Option { asJSON: true }: Returns Record<string, string> (Data).\n   */\n  getByRow: <T extends { asJSON?: boolean }>(\n    filters: Record<string, string | RegExp | number>, \n    options?: { exact?: boolean, maxPages?: number } & T\n  ) => Promise<T['asJSON'] extends true ? Record<string, string> : SmartRow>;\n\n  /** * Get all rows on the current page.\n   * Can be filtered by column values.\n   * * @param options.asJSON - Returns data objects instead of SmartRows\n   * @param options.filter - Key-Value pairs to filter rows (e.g. { Status: 'Active' })\n   * @param options.exact - Exact match for filters (default: false)\n   */\n  getAllRows: <T extends { asJSON?: boolean }>(\n    options?: { \n      filter?: Record<string, string | RegExp | number>; \n      exact?: boolean \n    } & T\n  ) => Promise<T['asJSON'] extends true ? Record<string, string>[] : SmartRow[]>;\n\n  generateConfigPrompt: (options?: PromptOptions) => Promise<void>;\n  generateStrategyPrompt: (options?: PromptOptions) => Promise<void>;\n}\n";
+export declare const TYPE_CONTEXT = "\nexport type Selector = string | ((root: Locator | Page) => Locator);\n\nexport type SmartRow = Locator & {\n  getCell(column: string): Locator;\n  toJSON(): Promise<Record<string, string>>;\n};\n\nexport interface TableContext {\n  root: Locator;\n  config: Required<TableConfig>;\n  page: Page;\n  resolve: (selector: Selector, parent: Locator | Page) => Locator;\n}\n\nexport type PaginationStrategy = (context: TableContext) => Promise<boolean>;\n\nexport interface PromptOptions {\n  /**\n   * Output Strategy:\n   * - 'error': Throws an error with the prompt (Best for Cloud/QA Wolf to get clean text).\n   * - 'console': Standard console logs (Default).\n   * - 'report': Attaches to Playwright Report (Requires testInfo).\n   */\n  output?: 'console' | 'report' | 'error';\n  includeTypes?: boolean;\n  testInfo?: TestInfo;\n}\n\nexport interface TableConfig {\n  rowSelector?: Selector;\n  headerSelector?: Selector;\n  cellSelector?: Selector;\n  pagination?: PaginationStrategy;\n  maxPages?: number;\n  headerTransformer?: (text: string, index: number) => string;\n  autoScroll?: boolean;\n}\n\nexport interface TableResult {\n  getHeaders: () => Promise<string[]>;\n  getHeaderCell: (columnName: string) => Promise<Locator>;\n\n  getByRow: <T extends { asJSON?: boolean }>(\n    filters: Record<string, string | RegExp | number>, \n    options?: { exact?: boolean, maxPages?: number } & T\n  ) => Promise<T['asJSON'] extends true ? Record<string, string> : SmartRow>;\n\n  getAllRows: <T extends { asJSON?: boolean }>(\n    options?: { filter?: Record<string, any>, exact?: boolean } & T\n  ) => Promise<T['asJSON'] extends true ? Record<string, string>[] : SmartRow[]>;\n\n  generateConfigPrompt: (options?: PromptOptions) => Promise<void>;\n  generateStrategyPrompt: (options?: PromptOptions) => Promise<void>;\n}\n";

package/dist/typeContext.js

@@ -7,8 +7,6 @@ exports.TYPE_CONTEXT = void 0;
  * It contains the raw text of types.ts to provide context for LLM prompts.
  */
 exports.TYPE_CONTEXT = `
-// src/types.ts
-
 export type Selector = string | ((root: Locator | Page) => Locator);
 
 export type SmartRow = Locator & {
@@ -23,48 +21,27 @@ export interface TableContext {
   resolve: (selector: Selector, parent: Locator | Page) => Locator;
 }
 
-/**
- * A function that handles pagination logic.
- * Returns true if more data was loaded (navigation occurred), false if end of data.
- */
 export type PaginationStrategy = (context: TableContext) => Promise<boolean>;
 
 export interface PromptOptions {
-  /** * Where to output the prompt.
-   * - 'console': Logs to stdout (default).
-   * - 'report': Attaches to the Playwright HTML report (best for CI/VMs).
-   * - 'file': Writes to a local file (e.g. 'smart-table-prompt.md').
-   */
-  output?: 'console' | 'report' | 'file';
-  
   /**
-   * Include TypeScript interfaces in the prompt?
-   * Helps LLMs generate correct code structure.
-   * Default: true
+   * Output Strategy:
+   * - 'error': Throws an error with the prompt (Best for Cloud/QA Wolf to get clean text).
+   * - 'console': Standard console logs (Default).
+   * - 'report': Attaches to Playwright Report (Requires testInfo).
    */
+  output?: 'console' | 'report' | 'error';
   includeTypes?: boolean;
+  testInfo?: TestInfo;
 }
 
 export interface TableConfig {
   rowSelector?: Selector;
   headerSelector?: Selector;
   cellSelector?: Selector;
-  /**
-   * Strategy for handling pagination.
-   * Use presets from TableStrategies or write your own.
-   */
   pagination?: PaginationStrategy;
   maxPages?: number;
-  /**
-   * Optional hook to rename columns dynamically.
-   * Useful for naming empty columns (like '__col_0') to something semantic like 'Actions'.
-   */
   headerTransformer?: (text: string, index: number) => string;
-  /**
-   * Automatically scroll the table into view on first interaction.
-   * Helps ensure the table is visible in traces/videos.
-   * Default: true
-   */
   autoScroll?: boolean;
 }
 
@@ -72,26 +49,13 @@ export interface TableResult {
   getHeaders: () => Promise<string[]>;
   getHeaderCell: (columnName: string) => Promise<Locator>;
 
-  /** * Find a specific row by its content.
-   * Default: Returns SmartRow (Locator).
-   * Option { asJSON: true }: Returns Record<string, string> (Data).
-   */
   getByRow: <T extends { asJSON?: boolean }>(
     filters: Record<string, string | RegExp | number>, 
     options?: { exact?: boolean, maxPages?: number } & T
   ) => Promise<T['asJSON'] extends true ? Record<string, string> : SmartRow>;
 
-  /** * Get all rows on the current page.
-   * Can be filtered by column values.
-   * * @param options.asJSON - Returns data objects instead of SmartRows
-   * @param options.filter - Key-Value pairs to filter rows (e.g. { Status: 'Active' })
-   * @param options.exact - Exact match for filters (default: false)
-   */
   getAllRows: <T extends { asJSON?: boolean }>(
-    options?: { 
-      filter?: Record<string, string | RegExp | number>; 
-      exact?: boolean 
-    } & T
+    options?: { filter?: Record<string, any>, exact?: boolean } & T
   ) => Promise<T['asJSON'] extends true ? Record<string, string>[] : SmartRow[]>;
 
   generateConfigPrompt: (options?: PromptOptions) => Promise<void>;

package/dist/types.d.ts

@@ -1,4 +1,4 @@
-import { Locator, Page } from '@playwright/test';
+import { Locator, Page, TestInfo } from '@playwright/test';
 export type Selector = string | ((root: Locator | Page) => Locator);
 export type SmartRow = Locator & {
     getCell(column: string): Locator;
@@ -10,70 +10,40 @@ export interface TableContext {
     page: Page;
     resolve: (selector: Selector, parent: Locator | Page) => Locator;
 }
-/**
- * A function that handles pagination logic.
- * Returns true if more data was loaded (navigation occurred), false if end of data.
- */
 export type PaginationStrategy = (context: TableContext) => Promise<boolean>;
 export interface PromptOptions {
-    /** * Where to output the prompt.
-     * - 'console': Logs to stdout (default).
-     * - 'report': Attaches to the Playwright HTML report (best for CI/VMs).
-     * - 'file': Writes to a local file (e.g. 'smart-table-prompt.md').
-     */
-    output?: 'console' | 'report' | 'file';
     /**
-     * Include TypeScript interfaces in the prompt?
-     * Helps LLMs generate correct code structure.
-     * Default: true
+     * Output Strategy:
+     * - 'error': Throws an error with the prompt (Best for Cloud/QA Wolf to get clean text).
+     * - 'console': Standard console logs (Default).
+     * - 'report': Attaches to Playwright Report (Requires testInfo).
      */
+    output?: 'console' | 'report' | 'error';
     includeTypes?: boolean;
+    testInfo?: TestInfo;
 }
 export interface TableConfig {
     rowSelector?: Selector;
     headerSelector?: Selector;
     cellSelector?: Selector;
-    /**
-     * Strategy for handling pagination.
-     * Use presets from TableStrategies or write your own.
-     */
     pagination?: PaginationStrategy;
     maxPages?: number;
-    /**
-     * Optional hook to rename columns dynamically.
-     * Useful for naming empty columns (like '__col_0') to something semantic like 'Actions'.
-     */
     headerTransformer?: (text: string, index: number) => string;
-    /**
-     * Automatically scroll the table into view on first interaction.
-     * Helps ensure the table is visible in traces/videos.
-     * Default: true
-     */
     autoScroll?: boolean;
 }
 export interface TableResult {
     getHeaders: () => Promise<string[]>;
     getHeaderCell: (columnName: string) => Promise<Locator>;
-    /** * Find a specific row by its content.
-     * Default: Returns SmartRow (Locator).
-     * Option { asJSON: true }: Returns Record<string, string> (Data).
-     */
     getByRow: <T extends {
         asJSON?: boolean;
     }>(filters: Record<string, string | RegExp | number>, options?: {
         exact?: boolean;
         maxPages?: number;
     } & T) => Promise<T['asJSON'] extends true ? Record<string, string> : SmartRow>;
-    /** * Get all rows on the current page.
-     * Can be filtered by column values.
-     * * @param options.asJSON - Returns data objects instead of SmartRows
-     * @param options.filter - Key-Value pairs to filter rows (e.g. { Status: 'Active' })
-     * @param options.exact - Exact match for filters (default: false)
-     */
     getAllRows: <T extends {
         asJSON?: boolean;
     }>(options?: {
-        filter?: Record<string, string | RegExp | number>;
+        filter?: Record<string, any>;
         exact?: boolean;
     } & T) => Promise<T['asJSON'] extends true ? Record<string, string>[] : SmartRow[]>;
     generateConfigPrompt: (options?: PromptOptions) => Promise<void>;

package/dist/useTable.js

@@ -13,7 +13,7 @@ exports.useTable = void 0;
 const test_1 = require("@playwright/test");
 const typeContext_1 = require("./typeContext");
 const useTable = (rootLocator, configOptions = {}) => {
-    const config = Object.assign({ rowSelector: "tbody tr", headerSelector: "th", cellSelector: "td", pagination: undefined, maxPages: 1, headerTransformer: undefined, autoScroll: true }, configOptions);
+    const config = Object.assign({ rowSelector: "tbody tr", headerSelector: "th", cellSelector: "td", pagination: () => __awaiter(void 0, void 0, void 0, function* () { return false; }), maxPages: 1, headerTransformer: (text) => text, autoScroll: true }, configOptions);
     const resolve = (item, parent) => {
         if (typeof item === 'string')
             return parent.locator(item);
@@ -26,7 +26,10 @@ const useTable = (rootLocator, configOptions = {}) => {
         if (_headerMap)
             return _headerMap;
         if (config.autoScroll) {
-            yield rootLocator.scrollIntoViewIfNeeded();
+            try {
+                yield rootLocator.scrollIntoViewIfNeeded({ timeout: 1000 });
+            }
+            catch (e) { }
         }
         const headerLoc = resolve(config.headerSelector, rootLocator);
         try {
@@ -36,8 +39,8 @@ const useTable = (rootLocator, configOptions = {}) => {
         const texts = yield headerLoc.allInnerTexts();
         _headerMap = new Map(texts.map((t, i) => {
             let text = t.trim() || `__col_${i}`;
-            if (config.headerTransformer)
-                text = config.headerTransformer(text, i);
+            // Safe call: config.headerTransformer is always defined now
+            text = config.headerTransformer(text, i);
             return [text, i];
         }));
         return _headerMap;
@@ -96,7 +99,7 @@ const useTable = (rootLocator, configOptions = {}) => {
                 throw new Error(`Strict Mode Violation: Found ${count} rows matching ${JSON.stringify(filters)}.`);
             if (count === 1)
                 return matchedRows.first();
-            if (config.pagination && currentPage < effectiveMaxPages) {
+            if (currentPage < effectiveMaxPages) {
                 const context = {
                     root: rootLocator,
                     config: config,
@@ -113,27 +116,33 @@ const useTable = (rootLocator, configOptions = {}) => {
         }
     });
     const _handlePrompt = (promptName_1, content_1, ...args_1) => __awaiter(void 0, [promptName_1, content_1, ...args_1], void 0, function* (promptName, content, options = {}) {
-        const { output = 'console', includeTypes = true } = options;
+        const { output = 'console', includeTypes = true, testInfo } = options;
         let finalPrompt = content;
         if (includeTypes) {
             finalPrompt += `\n\n👇 Useful TypeScript Definitions 👇\n\`\`\`typescript\n${typeContext_1.TYPE_CONTEXT}\n\`\`\`\n`;
         }
-        if (output === 'console') {
-            console.log(finalPrompt);
+        if (output === 'error') {
+            console.log(`⚠️ Throwing error to display [${promptName}] cleanly...`);
+            throw new Error(finalPrompt);
         }
-        else if (output === 'report') {
-            if (test_1.test.info()) {
-                yield test_1.test.info().attach(promptName, {
-                    body: finalPrompt,
-                    contentType: 'text/markdown'
-                });
-                console.log(`✅ Attached '${promptName}' to Playwright Report.`);
+        if (output === 'report') {
+            let activeInfo = testInfo;
+            if (!activeInfo) {
+                try {
+                    activeInfo = test_1.test.info();
+                }
+                catch (e) { }
+            }
+            if (activeInfo) {
+                yield activeInfo.attach(promptName, { body: finalPrompt, contentType: 'text/markdown' });
+                console.log(`✅ [${promptName}] Attached to Playwright Report.`);
+                return;
             }
             else {
-                console.warn('⚠️ Cannot attach to report: No active test info found. Logging to console instead.');
-                console.log(finalPrompt);
+                console.warn(`⚠️ [${promptName}] Cannot attach to report (Context unavailable). Logging to console instead.`);
             }
         }
+        console.log(finalPrompt);
     });
     return {
         getHeaders: () => __awaiter(void 0, void 0, void 0, function* () { return Array.from((yield _getMap()).keys()); }),

package/package.json

@@ -1,7 +1,11 @@
 {
   "name": "@rickcedwhat/playwright-smart-table",
-  "version": "2.0.2",
+  "version": "2.0.6",
   "description": "A smart table utility for Playwright with built-in pagination strategies that are fully extensible.",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/rickcedwhat/playwright-smart-table.git"
+  },
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "files": [
@@ -11,7 +15,6 @@
     "generate-types": "node scripts/embed-types.mjs",
     "build": "npm run generate-types && tsc",
     "prepublishOnly": "npm run build",
-    "release": "npm run build && npm publish --access public",
     "test": "npx playwright test"
   },
   "keywords": [