fix: move onReset to run after goToFirst and autoInit (#404)

docs/.vitepress/tableconfig-signatures.json

@@ -42,7 +42,7 @@
   {
     "name": "onReset",
     "signature": "onReset?: (context: TableContext) => Promise<void>",
-    "comment": "/** Reset hook */"
+    "comment": "/** Hook called after reset completes (after goToFirst, cache clear, and autoInit). */"
   },
   {
     "name": "strategies",

docs/.vitepress/tableresult-signatures.json

@@ -77,7 +77,7 @@
   {
     "name": "reset",
     "signature": "reset: () => Promise<void>",
-    "comment": "/**\n* Resets the table state (clears cache, flags) and invokes the onReset strategy.\n*/"
+    "comment": "/**\n* Resets the table: calls goToFirst (if configured), clears cache, re-inits headers, then calls onReset.\n*/"
   },
   {
     "name": "revalidate",

src/typeContext.ts

@@ -685,7 +685,7 @@ export interface TableConfig<T = any> {
   autoScroll?: boolean;
   /** Debug options for development and troubleshooting */
   debug?: DebugConfig;
-  /** Reset hook */
+  /** Hook called after reset completes (after goToFirst, cache clear, and autoInit). */
   onReset?: (context: TableContext) => Promise<void>;
   /** All interaction strategies */
   strategies?: TableStrategies;
@@ -915,7 +915,7 @@ export interface TableResult<T = any> extends AsyncIterable<{ row: SmartRow<T>;
 
 
   /**
-   * Resets the table state (clears cache, flags) and invokes the onReset strategy.
+   * Resets the table: calls goToFirst (if configured), clears cache, re-inits headers, then calls onReset.
    */
   reset: () => Promise<void>;
 

src/types.ts

@@ -685,7 +685,7 @@ export interface TableConfig<T = any> {
   autoScroll?: boolean;
   /** Debug options for development and troubleshooting */
   debug?: DebugConfig;
-  /** Reset hook */
+  /** Hook called after reset completes (after goToFirst, cache clear, and autoInit). */
   onReset?: (context: TableContext) => Promise<void>;
   /** All interaction strategies */
   strategies?: TableStrategies;
@@ -915,7 +915,7 @@ export interface TableResult<T = any> extends AsyncIterable<{ row: SmartRow<T>;
 
 
   /**
-   * Resets the table state (clears cache, flags) and invokes the onReset strategy.
+   * Resets the table: calls goToFirst (if configured), clears cache, re-inits headers, then calls onReset.
    */
   reset: () => Promise<void>;
 

src/useTable.ts

@@ -444,8 +444,6 @@ export const useTable = <T = any>(rootLocator: Locator, configOptions: TableConf
 
     reset: async () => {
       log("Resetting table...");
-      await config.onReset(createStrategyContext());
-
       if (config.strategies.pagination?.goToFirst) {
         log("Auto-navigating to first page...");
         await config.strategies.pagination.goToFirst(createStrategyContext());
@@ -458,6 +456,7 @@ export const useTable = <T = any>(rootLocator: Locator, configOptions: TableConf
       tableMapper.clear();
       log("Table reset complete. Calling autoInit to restore state.");
       await _autoInit();
+      await config.onReset(createStrategyContext());
     },
 
     revalidate: async () => {

tests/edge-cases.spec.ts

@@ -251,6 +251,50 @@ test.describe('Edge cases and missing coverage', () => {
     await expect(page.locator('#page')).toHaveText('1');
   });
 
+  test('onReset runs after goToFirst and autoInit (#404)', async ({ page }) => {
+    await page.setContent(`
+      <table id="t">
+        <thead><tr><th>ID</th></tr></thead>
+        <tbody id="tbody">
+          <tr><td>1</td></tr>
+          <tr><td>2</td></tr>
+        </tbody>
+        <tfoot><tr><td><button id="first">First</button><button id="next">Next</button></td></tr></tfoot>
+      </table>
+      <div id="page">1</div>
+      <script>
+        let p = 1;
+        function render() {
+          if (p === 1) {
+            document.getElementById('tbody').innerHTML = '<tr><td>1</td></tr><tr><td>2</td></tr>';
+            document.getElementById('page').textContent = '1';
+          } else {
+            document.getElementById('tbody').innerHTML = '<tr><td>3</td></tr><tr><td>4</td></tr>';
+            document.getElementById('page').textContent = '2';
+          }
+        }
+        document.getElementById('next').onclick = () => { if (p === 1) { p = 2; render(); } };
+        document.getElementById('first').onclick = () => { if (p === 2) { p = 1; render(); } };
+      </script>
+    `);
+
+    let pageTextDuringOnReset = '';
+    const table = useTable(page.locator('#t'), {
+      strategies: { pagination: Strategies.Pagination.click({ next: '#next', first: '#first' }) },
+      maxPages: 2,
+      onReset: async () => {
+        pageTextDuringOnReset = await page.locator('#page').textContent() ?? '';
+      },
+    });
+    await table.init();
+    await table.findRows({});
+    expect(table.currentPageIndex).toBe(1);
+    await expect(page.locator('#page')).toHaveText('2');
+
+    await table.reset();
+    expect(pageTextDuringOnReset).toBe('1');
+  });
+
   test('sorting.getState throws when no sorting strategy configured', async ({ page }) => {
     await page.setContent(`
       <table id="t"><thead><tr><th>Name</th></tr></thead><tbody><tr><td>Alice</td></tr></tbody></table>