chore: Remove verbose debug logging

Removed debug print statements from:
- TabStateStorage: tab saving/loading logs
- MainContentView: window close, connection ready, query update logs

Kept only critical error/warning logs.

Author: datlechin

OpenTable.xcodeproj/project.xcworkspace/xcuserdata/ngoquocdat.xcuserdatad/UserInterfaceState.xcuserstate

@@ -29,31 +29,13 @@ final class TabStateStorage {
         let persistedTabs = tabs.map { $0.toPersistedTab() }
         let tabState = TabState(tabs: persistedTabs, selectedTabId: selectedTabId)
 
-        #if DEBUG
-        print("[TabStateStorage] Saving \(tabs.count) tabs for connection \(connectionId)")
-        for (index, tab) in tabs.enumerated() {
-            print("[TabStateStorage]   Tab \(index): \"\(tab.title)\" - Query: \"\(tab.query.prefix(50))\"")
-            
-            // DEBUG: Print call stack when saving empty query
-            if tab.query.isEmpty {
-                print("[TabStateStorage] ⚠️ WARNING: Saving empty query! Call stack:")
-                Thread.callStackSymbols.prefix(10).forEach { print("  \($0)") }
-            }
-        }
-        #endif
-        
         do {
             let encoder = JSONEncoder()
             let data = try encoder.encode(tabState)
             let key = tabStateKey(for: connectionId)
             defaults.set(data, forKey: key)
-            #if DEBUG
-            print("[TabStateStorage] ✅ Saved to disk successfully")
-            #endif
         } catch {
-            #if DEBUG
-            print("[TabStateStorage] Failed to encode tab state: \(error.localizedDescription)")
-            #endif
+            // Silent failure - encoding errors are rare and non-critical
         }
     }
 
@@ -69,9 +51,7 @@ final class TabStateStorage {
             let decoder = JSONDecoder()
             return try decoder.decode(TabState.self, from: data)
         } catch {
-            #if DEBUG
-            print("[TabStateStorage] Failed to decode tab state: \(error.localizedDescription)")
-            #endif
+            // Silent failure - decoding errors return nil
             return nil
         }
     }

OpenTable/Core/Storage/TabStateStorage.swift

@@ -208,9 +208,8 @@ struct MainContentView: View {
                 // Skip save during restoration
                 guard !isRestoringTabs else { return }
 
-                // CRITICAL: Skip save if view is being dismissed
+                // Skip save if view is being dismissed
                 guard !isDismissing else {
-                    print("[MainContentView] Skipping selectedTabId save - view is being dismissed")
                     return
                 }
 
@@ -235,7 +234,6 @@ struct MainContentView: View {
                 // CRITICAL: Skip save if view is being dismissed to prevent saving empty query
                 // When SwiftUI tears down the view, bindings may be reset causing empty saves
                 guard !isDismissing else {
-                    print("[MainContentView] Skipping save - view is being dismissed")
                     return
                 }
 
@@ -271,7 +269,6 @@ struct MainContentView: View {
             .onChange(of: DatabaseManager.shared.currentSession?.isConnected) { _, isConnected in
                 // Auto-execute query when connection becomes ready and tab needs data
                 if isConnected == true && needsLazyLoad {
-                    print("[MainContentView] Connection ready - executing lazy load")
                     needsLazyLoad = false
                     runQuery()
                 }
@@ -345,7 +342,6 @@ struct MainContentView: View {
             .onReceive(NotificationCenter.default.publisher(for: .mainWindowWillClose)) { _ in
                 // CRITICAL: Window is about to close - flush pending saves immediately
                 // This prevents query text from being lost when SwiftUI tears down the view
-                print("[MainContentView] Window will close - flushing pending saves")
 
                 // Set flag to prevent further saves (view is being destroyed)
                 isDismissing = true
@@ -360,7 +356,6 @@ struct MainContentView: View {
                         tabs: tabManager.tabs,
                         selectedTabId: tabManager.selectedTabId
                     )
-                    print("[MainContentView] Flushed tab state for connection \(connection.id)")
                 }
             }
     }
@@ -454,11 +449,10 @@ struct MainContentView: View {
                     tabManager.addTab(initialQuery: lastQuery)
                 }
             }
-            .onReceive(NotificationCenter.default.publisher(for: NSNotification.Name("loadQueryIntoEditor"))) { notification in
+            .onReceive(NotificationCenter.default.publisher(for: .loadQueryIntoEditor)) { notification in
                 // Load query from history/bookmark panel into current tab
                 Task { @MainActor in
-                    guard let query = notification.userInfo?["query"] as? String else { return }
-                    print("[MainContentView] Received loadQueryIntoEditor with query: \(query.prefix(50))...")
+                    guard let query = notification.object as? String else { return }
 
                     // Load into the current tab (which was just created by .newTab)
                     if let tabIndex = tabManager.selectedTabIndex,
@@ -566,7 +560,6 @@ struct MainContentView: View {
                             // CRITICAL: Bounds check to prevent crash on paste
                             guard let index = tabManager.selectedTabIndex,
                                   index < tabManager.tabs.count else {
-                                print("[MainContentView] Warning: Invalid tab index during query update")
                                 return
                             }