fix(datagrid): scope pendingScrollToTopAfterReplace per-tabId, add dispatch tests

The original Bool flag could strand if pagination's runQuery failed before setActiveTableRows fired — the flag would stay set, and the next setActiveTableRows from any other code path (undo, FK navigation, multi-statement query) would unexpectedly scroll to top.

Changing to Set<UUID> scopes the intent per-tab. Pagination inserts the tabId; notifyFullReplaceIfActive checks via remove() and only dispatches scrollToTop on a matching hit. A stranded entry stays scoped to that one tab and only fires when that tab next receives a full replace — narrow blast radius.

Adds three regression tests:
- scrollToTopFiresOnPendingFlag: setActiveTableRows on the active tab WITH the flag set dispatches scrollToTop and clears the flag
- scrollToTopFlagIsScopedPerTab: a flag set for tab A doesn't fire when tab B is replaced; tab A's entry stays
- scrollToTopSkippedWhenFlagAbsent: setActiveTableRows without the flag doesn't scroll

Author: datlechin

TablePro/Views/Main/Extensions/MainContentCoordinator+Pagination.swift

@@ -97,7 +97,7 @@ extension MainContentCoordinator {
 
             mutate(&self.tabManager.tabs[idx].pagination)
             self.tabManager.tabs[idx].paginationVersion += 1
-            self.pendingScrollToTopAfterReplace = true
+            self.pendingScrollToTopAfterReplace.insert(tabId)
             self.reloadCurrentPage()
         }
     }

TablePro/Views/Main/Extensions/MainContentCoordinator+TableRowsMutation.swift

@@ -45,8 +45,7 @@ extension MainContentCoordinator {
               idx < tabManager.tabs.count,
               tabManager.tabs[idx].id == tabId else { return }
         dataTabDelegate?.tableViewCoordinator?.applyFullReplace()
-        if pendingScrollToTopAfterReplace {
-            pendingScrollToTopAfterReplace = false
+        if pendingScrollToTopAfterReplace.remove(tabId) != nil {
             dataTabDelegate?.tableViewCoordinator?.scrollToTop()
         }
     }

TablePro/Views/Main/MainContentCoordinator.swift

@@ -147,7 +147,7 @@ final class MainContentCoordinator {
     /// Cache for async-sorted query tab rows (large datasets sorted on background thread)
     @ObservationIgnored var querySortCache: [UUID: QuerySortCacheEntry] = [:]
 
-    @ObservationIgnored var pendingScrollToTopAfterReplace: Bool = false
+    @ObservationIgnored var pendingScrollToTopAfterReplace: Set<UUID> = []
 
     // MARK: - Internal State
 

TableProTests/Views/Main/TableRowsMutationTests.swift

@@ -111,6 +111,45 @@ struct TableRowsMutationTests {
         #expect(f.fake.fullReplaceCount == 2)
     }
 
+    @Test("setActiveTableRows dispatches scrollToTop when pendingScrollToTopAfterReplace contains tabId")
+    func scrollToTopFiresOnPendingFlag() {
+        let f = makeFixture()
+        f.tabManager.addTableTab(tableName: "users")
+        let activeTabId = f.tabManager.tabs[0].id
+
+        f.coordinator.pendingScrollToTopAfterReplace.insert(activeTabId)
+        f.coordinator.setActiveTableRows(makeTableRows(rowCount: 3), for: activeTabId)
+
+        #expect(f.fake.scrollToTopCount == 1)
+        #expect(f.coordinator.pendingScrollToTopAfterReplace.contains(activeTabId) == false)
+    }
+
+    @Test("scrollToTop pending flag for tab A does not fire when tab B is replaced")
+    func scrollToTopFlagIsScopedPerTab() {
+        let f = makeFixture()
+        f.tabManager.addTableTab(tableName: "users")
+        let firstTabId = f.tabManager.tabs[0].id
+        f.tabManager.addTableTab(tableName: "orders")
+        let secondTabId = f.tabManager.tabs[1].id
+
+        f.coordinator.pendingScrollToTopAfterReplace.insert(firstTabId)
+        f.coordinator.setActiveTableRows(makeTableRows(rowCount: 3), for: secondTabId)
+
+        #expect(f.fake.scrollToTopCount == 0)
+        #expect(f.coordinator.pendingScrollToTopAfterReplace.contains(firstTabId) == true)
+    }
+
+    @Test("setActiveTableRows without pending flag does not scroll to top")
+    func scrollToTopSkippedWhenFlagAbsent() {
+        let f = makeFixture()
+        f.tabManager.addTableTab(tableName: "users")
+        let activeTabId = f.tabManager.tabs[0].id
+
+        f.coordinator.setActiveTableRows(makeTableRows(rowCount: 3), for: activeTabId)
+
+        #expect(f.fake.scrollToTopCount == 0)
+    }
+
     @Test("setActiveTableRows is a no-op when delegate is unwired")
     func unwiredDelegateIsNoOp() {
         let tabManager = QueryTabManager()