Merge branch 'main' into fix/memory-retention-after-tab-close

Author: datlechin

CHANGELOG.md

@@ -7,6 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- MongoDB Atlas connections failing to authenticate (#438)
+- MongoDB TLS certificate verification skipped for SRV connections
+- Active tab data no longer refreshes when switching back to the app window
+- Undo history preserved when switching between database tables
+- Health monitor now detects stuck queries beyond the configured timeout
+- SSH tunnel closure errors now logged instead of silently discarded
+- Schema/database restore errors during reconnect now logged
+
+## [0.23.1] - 2026-03-24
+
 ### Added
 
 - Test Connection button in SSH profile editor to validate SSH connectivity independently
@@ -1003,7 +1015,8 @@ TablePro is a native macOS database client built with SwiftUI and AppKit, design
     - Custom SQL query templates
     - Performance optimized for large datasets
 
-[Unreleased]: https://github.com/datlechin/tablepro/compare/v0.23.0...HEAD
+[Unreleased]: https://github.com/datlechin/tablepro/compare/v0.23.1...HEAD
+[0.23.1]: https://github.com/datlechin/tablepro/compare/v0.23.0...v0.23.1
 [0.23.0]: https://github.com/datlechin/tablepro/compare/v0.22.1...v0.23.0
 [0.22.1]: https://github.com/datlechin/tablepro/compare/v0.22.0...v0.22.1
 [0.22.0]: https://github.com/datlechin/tablepro/compare/v0.21.0...v0.22.0

Plugins/MongoDBDriverPlugin/MongoDBConnection.swift

@@ -190,6 +190,8 @@ final class MongoDBConnection: @unchecked Sendable {
         let effectiveAuthSource: String
         if let source = authSource, !source.isEmpty {
             effectiveAuthSource = source
+        } else if useSrv {
+            effectiveAuthSource = "admin"
         } else if !database.isEmpty {
             effectiveAuthSource = database
         } else {
@@ -206,8 +208,7 @@ final class MongoDBConnection: @unchecked Sendable {
         let sslEnabled = ["Preferred", "Required", "Verify CA", "Verify Identity"].contains(sslMode)
         if sslEnabled {
             params.append("tls=true")
-            let verifiesCert = sslMode == "Verify CA" || sslMode == "Verify Identity"
-            if !verifiesCert {
+            if sslMode == "Preferred" {
                 params.append("tlsAllowInvalidCertificates=true")
             }
             if !sslCACertPath.isEmpty {

Plugins/MongoDBDriverPlugin/MongoDBPluginDriver.swift

@@ -36,7 +36,10 @@ final class MongoDBPluginDriver: PluginDatabaseDriver {
     // MARK: - Connection Management
 
     func connect() async throws {
+        // Auto-enable SRV for Atlas hostnames (*.mongodb.net) even if the toggle wasn't set,
+        // since Atlas clusters only resolve via SRV records.
         let useSrv = config.additionalFields["mongoUseSrv"] == "true"
+            || config.host.hasSuffix(".mongodb.net")
         let authMechanism = config.additionalFields["mongoAuthMechanism"]
         let replicaSet = config.additionalFields["mongoReplicaSet"]
 
@@ -54,7 +57,9 @@ final class MongoDBPluginDriver: PluginDatabaseDriver {
             user: config.username,
             password: config.password,
             database: currentDb,
-            sslMode: config.additionalFields["sslMode"] ?? "Disabled",
+            sslMode: useSrv && (config.additionalFields["sslMode"] ?? "Disabled") == "Disabled"
+                ? "Required"
+                : config.additionalFields["sslMode"] ?? "Disabled",
             sslCACertPath: config.additionalFields["sslCACertPath"] ?? "",
             sslClientCertPath: config.additionalFields["sslClientCertPath"] ?? "",
             authSource: config.additionalFields["mongoAuthSource"],

TablePro.xcodeproj/project.pbxproj

@@ -1920,7 +1920,7 @@
 				CODE_SIGN_IDENTITY = "Apple Development";
 				"CODE_SIGN_IDENTITY[sdk=macosx*]" = "Apple Development";
 				CODE_SIGN_STYLE = Automatic;
-				CURRENT_PROJECT_VERSION = 44;
+				CURRENT_PROJECT_VERSION = 45;
 				DEAD_CODE_STRIPPING = YES;
 				DEVELOPMENT_TEAM = D7HJ5TFYCU;
 				ENABLE_APP_SANDBOX = NO;
@@ -1945,7 +1945,7 @@
 				LD_RUNPATH_SEARCH_PATHS = "@executable_path/Frameworks";
 				"LD_RUNPATH_SEARCH_PATHS[sdk=macosx*]" = "@executable_path/../Frameworks";
 				MACOSX_DEPLOYMENT_TARGET = 14.0;
-				MARKETING_VERSION = 0.23.0;
+				MARKETING_VERSION = 0.23.1;
 				OTHER_LDFLAGS = (
 					"-Wl,-w",
 					"-force_load",
@@ -1992,7 +1992,7 @@
 				"CODE_SIGN_IDENTITY[sdk=macosx*]" = "Apple Development";
 				CODE_SIGN_STYLE = Automatic;
 				COPY_PHASE_STRIP = YES;
-				CURRENT_PROJECT_VERSION = 44;
+				CURRENT_PROJECT_VERSION = 45;
 				DEAD_CODE_STRIPPING = YES;
 				DEPLOYMENT_POSTPROCESSING = YES;
 				DEVELOPMENT_TEAM = D7HJ5TFYCU;
@@ -2018,7 +2018,7 @@
 				LD_RUNPATH_SEARCH_PATHS = "@executable_path/Frameworks";
 				"LD_RUNPATH_SEARCH_PATHS[sdk=macosx*]" = "@executable_path/../Frameworks";
 				MACOSX_DEPLOYMENT_TARGET = 14.0;
-				MARKETING_VERSION = 0.23.0;
+				MARKETING_VERSION = 0.23.1;
 				OTHER_LDFLAGS = (
 					"-Wl,-w",
 					"-force_load",

TablePro/AppDelegate+WindowConfig.swift

@@ -100,18 +100,22 @@ extension AppDelegate {
 
     // MARK: - Window Identification
 
+    private enum WindowId {
+        static let main = "main"
+        static let welcome = "welcome"
+        static let connectionForm = "connection-form"
+    }
+
     func isMainWindow(_ window: NSWindow) -> Bool {
-        guard let identifier = window.identifier?.rawValue else { return false }
-        return identifier.contains("main")
+        window.identifier?.rawValue == WindowId.main
     }
 
     func isWelcomeWindow(_ window: NSWindow) -> Bool {
-        window.identifier?.rawValue == "welcome" ||
-            window.title.lowercased().contains("welcome")
+        window.identifier?.rawValue == WindowId.welcome
     }
 
     private func isConnectionFormWindow(_ window: NSWindow) -> Bool {
-        window.identifier?.rawValue.contains("connection-form") == true
+        window.identifier?.rawValue == WindowId.connectionForm
     }
 
     // MARK: - Welcome Window
@@ -259,10 +263,7 @@ extension AppDelegate {
 
             if remainingMainWindows == 0 {
                 NotificationCenter.default.post(name: .mainWindowWillClose, object: nil)
-
-                DispatchQueue.main.async {
-                    self.openWelcomeWindow()
-                }
+                openWelcomeWindow()
             }
         }
     }
@@ -273,13 +274,9 @@ extension AppDelegate {
 
         if isWelcomeWindow(window),
            window.occlusionState.contains(.visible),
-           NSApp.windows.contains(where: { isMainWindow($0) && $0.isVisible }) {
-            DispatchQueue.main.async { [weak self] in
-                guard let self else { return }
-                if self.isWelcomeWindow(window), window.isVisible {
-                    window.close()
-                }
-            }
+           NSApp.windows.contains(where: { isMainWindow($0) && $0.isVisible }),
+           window.isVisible {
+            window.close()
         }
     }
 

TablePro/Core/ChangeTracking/DataChangeManager.swift

@@ -125,7 +125,8 @@ final class DataChangeManager {
 
     // MARK: - Configuration
 
-    /// Clear all changes (called after successful save)
+    /// Clear all tracked changes, preserving undo/redo history.
+    /// Use when changes are invalidated but undo context may still be relevant.
     func clearChanges() {
         changes.removeAll()
         changeIndex.removeAll()
@@ -134,11 +135,18 @@ final class DataChangeManager {
         modifiedCells.removeAll()
         insertedRowData.removeAll()
         changedRowIndices.removeAll()
-        undoManager.clearAll()
         hasChanges = false
         reloadVersion += 1
     }
 
+    /// Clear all tracked changes AND undo/redo history.
+    /// Use after successful save, explicit discard, or new query execution
+    /// where undo context is no longer meaningful.
+    func clearChangesAndUndoHistory() {
+        clearChanges()
+        undoManager.clearAll()
+    }
+
     /// Atomically configure the manager for a new table
     func configureForTable(
         tableName: String,

TablePro/Core/Database/ConnectionHealthMonitor.swift

@@ -35,7 +35,6 @@ actor ConnectionHealthMonitor {
     // MARK: - Configuration
 
     private static let pingInterval: TimeInterval = 30.0
-    private static let initialBackoffDelays: [TimeInterval] = [2.0, 4.0, 8.0]
     private static let maxBackoffDelay: TimeInterval = 120.0
 
     // MARK: - Dependencies
@@ -227,15 +226,7 @@ actor ConnectionHealthMonitor {
     /// Uses the initial delay table for the first few attempts, then doubles
     /// the previous delay for subsequent attempts, capped at `maxBackoffDelay`.
     private func backoffDelay(for attempt: Int) -> TimeInterval {
-        let delays = Self.initialBackoffDelays
-        if attempt <= delays.count {
-            return delays[attempt - 1]
-        }
-        // Exponential: last seed delay * 2^(attempt - seedCount)
-        let lastSeed = delays[delays.count - 1]
-        let exponent = attempt - delays.count
-        let computed = lastSeed * pow(2.0, Double(exponent))
-        return min(computed, Self.maxBackoffDelay)
+        ExponentialBackoff.delay(for: attempt, maxDelay: Self.maxBackoffDelay)
     }
 
     // MARK: - State Transitions

TablePro/Core/Database/DatabaseDriver.swift

@@ -315,12 +315,15 @@ extension DatabaseDriver {
 /// Factory for creating database drivers via plugin lookup
 @MainActor
 enum DatabaseDriverFactory {
+    private static let logger = Logger(subsystem: "com.TablePro", category: "DatabaseDriverFactory")
+
     static func createDriver(for connection: DatabaseConnection) throws -> DatabaseDriver {
         let pluginId = connection.type.pluginTypeId
         // If the plugin isn't registered yet and background loading hasn't finished,
         // fall back to synchronous loading for this critical code path.
         if PluginManager.shared.driverPlugins[pluginId] == nil,
            !PluginManager.shared.hasFinishedInitialLoad {
+            logger.warning("Plugin '\(pluginId)' not loaded yet — performing synchronous load")
             PluginManager.shared.loadPendingPlugins()
         }
         guard let plugin = PluginManager.shared.driverPlugins[pluginId] else {