refactor(datagrid): replace RowBuffer with TableRows; delete legacy row stack

Author: datlechin

CHANGELOG.md

@@ -22,7 +22,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
-- Introduced TableRows, Row, and Delta value types in TablePro/Models/Query/ as the foundation for the data grid row model rewrite. NSTableView delegate reads cell values and row count from TableRows; sidebar, JSON view, and exports now read from TableRows. Cell edits route through TableRows.edit and apply NSTableView updates via the Delta-driven TableRowsController. Row operations (add, duplicate, delete, paste, undo) mutate TableRows and apply the returned Delta through TableViewCoordinator.applyDelta. Sort state moved from InMemoryRowProvider's positional sortIndices to a TableViewCoordinator.sortedIDs permutation keyed by Row.id, so cell edits under sort hit the correct storage row and inserted rows survive at the end of the sorted view without re-sorting. RowBuffer still backs the display cache pending later phases (Phase C.2 of the DataGrid refactor).
+- Replaced RowBuffer / InMemoryRowProvider / RowDataStore with TableRows / TableRowsStore / TableRowsController. Mutations emit Delta events; the controller drives NSTableView via insertRows / removeRows / reloadData(forRowIndexes:). Sort and the display cache moved off the row provider into the data grid coordinator, keyed by Row.id.
 - DataChangeManager extracted a PendingChanges value type that owns cross-collection invariants for cell edits, row insertions, and deletions. DataChangeManager kept undo/redo registration, plugin SQL generation, and the `@Observable` boundary, dropping from ~960 to ~190 lines. The serialization DTO `TabPendingChanges` is renamed to `TabChangeSnapshot` to distinguish it from the live tracker.
 - AnyChangeManager uses ChangeManaging protocol instead of closure-based type erasure, removing all runtime `[Any]` downcasts
 - Row selection state moved from MainContentView @State to GridSelectionState @Observable class, preventing full view tree invalidation on every row click

TablePro/Core/Plugins/StreamingQueryExportDataSource.swift

@@ -4,7 +4,7 @@
 //
 //  Streaming export data source for query results.
 //  Re-executes the query and streams rows directly from the database to the export plugin,
-//  bypassing RowBuffer. Allows exporting large result sets without loading all rows into memory.
+//  bypassing in-memory storage. Allows exporting large result sets without loading all rows into memory.
 //
 
 import Foundation

TablePro/Core/Services/Formatting/CellDisplayFormatter.swift

@@ -3,7 +3,7 @@
 //  TablePro
 //
 //  Pure formatter that transforms raw cell values into display-ready strings.
-//  Used by InMemoryRowProvider's display cache to compute values once per cell.
+//  Used by the data grid coordinator's display cache to compute values once per cell.
 //
 
 import Foundation

TablePro/Core/Services/Query/RowDataStore.swift

@@ -14,7 +14,7 @@ import os
 final class ResultSet: Identifiable {
     let id: UUID
     var label: String
-    var rowBuffer: RowBuffer
+    var tableRows: TableRows
     var executionTime: TimeInterval?
     var rowsAffected: Int = 0
     var errorMessage: String?
@@ -27,37 +27,11 @@ final class ResultSet: Identifiable {
     var pagination = PaginationState()
     var columnLayout = ColumnLayoutState()
 
-    var columnTypes: [ColumnType] {
-        get { rowBuffer.columnTypes }
-        set { rowBuffer.columnTypes = newValue }
-    }
-
-    var columnDefaults: [String: String?] {
-        get { rowBuffer.columnDefaults }
-        set { rowBuffer.columnDefaults = newValue }
-    }
-
-    var columnForeignKeys: [String: ForeignKeyInfo] {
-        get { rowBuffer.columnForeignKeys }
-        set { rowBuffer.columnForeignKeys = newValue }
-    }
-
-    var columnEnumValues: [String: [String]] {
-        get { rowBuffer.columnEnumValues }
-        set { rowBuffer.columnEnumValues = newValue }
-    }
-
-    var columnNullable: [String: Bool] {
-        get { rowBuffer.columnNullable }
-        set { rowBuffer.columnNullable = newValue }
-    }
-
-    var resultColumns: [String] { rowBuffer.columns }
-    var resultRows: [[String?]] { rowBuffer.rows }
+    var resultColumns: [String] { tableRows.columns }
 
-    init(id: UUID = UUID(), label: String, rowBuffer: RowBuffer = RowBuffer()) {
+    init(id: UUID = UUID(), label: String, tableRows: TableRows = TableRows()) {
         self.id = id
         self.label = label
-        self.rowBuffer = rowBuffer
+        self.tableRows = tableRows
     }
 }