refactor(datagrid): consolidate resolvedTableRowsForTab into the existing helper (#936)

resolvedTableRowsForTab(coordinator:tabId:) and resolvedTableRows(for:) returned the same one-liner: coordinator.tableRowsStore.existingTableRows(for: id) ?? TableRows(). The duplicate existed because the dataGridView closure captures [coordinator] and didn't have a QueryTab in scope.

Inlined the store read directly in the closure (already a one-liner, no helper needed). Drops the duplicate method. resolvedTableRows(for tab:) stays for the four other call sites that operate on a QueryTab.

Author: datlechin

TablePro/Views/Main/Child/MainEditorContentView.swift

@@ -518,7 +518,7 @@ struct MainEditorContentView: View {
         let tabId = tab.id
         DataGridView(
             tableRowsProvider: { [coordinator] in
-                resolvedTableRowsForTab(coordinator: coordinator, tabId: tabId)
+                coordinator.tableRowsStore.existingTableRows(for: tabId) ?? TableRows()
             },
             tableRowsMutator: { [coordinator] mutate in
                 coordinator.mutateActiveTableRows(for: tabId) { rows in
@@ -557,11 +557,6 @@ struct MainEditorContentView: View {
         coordinator.tableRowsStore.existingTableRows(for: tab.id) ?? TableRows()
     }
 
-    @MainActor
-    private func resolvedTableRowsForTab(coordinator: MainContentCoordinator, tabId: UUID) -> TableRows {
-        coordinator.tableRowsStore.existingTableRows(for: tabId) ?? TableRows()
-    }
-
     private func displayFormats(for tab: QueryTab) -> [ValueDisplayFormat?] {
         let tableRows = coordinator.tableRowsStore.existingTableRows(for: tab.id)
         let columns = tableRows?.columns ?? []