fix: check driver.supportsTransactions before wrapping saves in transactions

Author: datlechin

TablePro/Core/Database/DatabaseManager+Schema.swift

@@ -60,15 +60,20 @@ extension DatabaseManager {
             )
             let statements = try generator.generate(changes: changes)
 
-            // Execute in transaction
-            try await driver.beginTransaction()
+            let useTransaction = driver.supportsTransactions
+
+            if useTransaction {
+                try await driver.beginTransaction()
+            }
 
             do {
                 for stmt in statements {
                     _ = try await driver.execute(query: stmt.sql)
                 }
 
-                try await driver.commitTransaction()
+                if useTransaction {
+                    try await driver.commitTransaction()
+                }
 
                 // Record each statement in query history
                 let connId = connectionId
@@ -87,10 +92,12 @@ extension DatabaseManager {
                 // Post notification to refresh UI
                 NotificationCenter.default.post(name: .refreshData, object: nil)
             } catch {
-                do {
-                    try await driver.rollbackTransaction()
-                } catch {
-                    Self.logger.error("Rollback failed after schema change error: \(error.localizedDescription)")
+                if useTransaction {
+                    do {
+                        try await driver.rollbackTransaction()
+                    } catch {
+                        Self.logger.error("Rollback failed after schema change error: \(error.localizedDescription)")
+                    }
                 }
                 throw DatabaseError.queryFailed("Schema change failed: \(error.localizedDescription)")
             }

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

@@ -35,7 +35,11 @@ extension MainContentCoordinator {
             throw DatabaseError.notConnected
         }
 
-        try await driver.beginTransaction()
+        let useTransaction = driver.supportsTransactions
+
+        if useTransaction {
+            try await driver.beginTransaction()
+        }
 
         do {
             for stmt in statements {
@@ -45,12 +49,16 @@ extension MainContentCoordinator {
                     _ = try await driver.executeParameterized(query: stmt.sql, parameters: stmt.parameters)
                 }
             }
-            try await driver.commitTransaction()
+            if useTransaction {
+                try await driver.commitTransaction()
+            }
         } catch {
-            do {
-                try await driver.rollbackTransaction()
-            } catch {
-                discardLogger.error("Rollback failed: \(error.localizedDescription, privacy: .public)")
+            if useTransaction {
+                do {
+                    try await driver.rollbackTransaction()
+                } catch {
+                    discardLogger.error("Rollback failed: \(error.localizedDescription, privacy: .public)")
+                }
             }
             throw error
         }

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

@@ -187,9 +187,7 @@ extension MainContentCoordinator {
                     throw DatabaseError.notConnected
                 }
 
-                // Redis MULTI/EXEC is not a true transaction (no rollback on failure),
-                // so execute statements individually without wrapping.
-                let useTransaction = dbType != .redis
+                let useTransaction = driver.supportsTransactions
 
                 if useTransaction {
                     try await driver.beginTransaction()