fix: correct Shift+Click multi-column sort direction cycling

Author: datlechin

TablePro/Views/Results/DataGridCoordinator.swift

@@ -32,6 +32,7 @@ final class TableViewCoordinator: NSObject, NSTableViewDelegate, NSTableViewData
     var onHideColumn: ((String) -> Void)?
     var onMoveRow: ((Int, Int) -> Void)?
     var onNavigateFK: ((String, ForeignKeyInfo) -> Void)?
+    var sortState = SortState()
     var getVisualState: ((Int) -> RowVisualState)?
     var dropdownColumns: Set<Int>?
     var typePickerColumns: Set<Int>?

TablePro/Views/Results/DataGridView.swift

@@ -179,6 +179,7 @@ struct DataGridView: NSViewRepresentable {
 
         scrollView.documentView = tableView
         context.coordinator.tableView = tableView
+        context.coordinator.sortState = sortState
         context.coordinator.onMoveRow = onMoveRow
         context.coordinator.rebuildColumnMetadataCache()
         if let connectionId {
@@ -232,6 +233,7 @@ struct DataGridView: NSViewRepresentable {
         )
         if currentIdentity == coordinator.lastIdentity {
             // Only refresh closure callbacks — they capture new state on each body eval
+            coordinator.sortState = sortState
             coordinator.onCellEdit = onCellEdit
             coordinator.onSort = onSort
             coordinator.onAddRow = onAddRow
@@ -302,6 +304,7 @@ struct DataGridView: NSViewRepresentable {
         coordinator.onRefresh = onRefresh
         coordinator.onCellEdit = onCellEdit
         coordinator.onDeleteRows = onDeleteRows
+        coordinator.sortState = sortState
         coordinator.onSort = onSort
         coordinator.onAddRow = onAddRow
         coordinator.onUndoInsert = onUndoInsert

TablePro/Views/Results/Extensions/DataGridView+Sort.swift

@@ -24,12 +24,12 @@ extension TableViewCoordinator {
         onSort?(columnIndex, sortDescriptor.ascending, isMultiSort)
     }
 
-    /// Handle Shift+Click on column header for multi-column sort.
-    /// NSTableView swallows Shift+Click on headers (sortDescriptorsDidChange never fires),
-    /// so the custom SortableTableHeaderView calls this directly.
+    // NSTableView swallows Shift+Click on headers, so SortableTableHeaderView calls this directly.
+    // Cycles: not sorted -> ascending -> descending -> remove
     func handleShiftClickSort(columnIndex: Int) {
         guard columnIndex >= 0 && columnIndex < rowProvider.columns.count else { return }
-        onSort?(columnIndex, true, true)
+        let ascending = !sortState.columns.contains(where: { $0.columnIndex == columnIndex })
+        onSort?(columnIndex, ascending, true)
     }
 
     // MARK: - Double-Click Column Divider Auto-Fit