chore: avoid print

Author: Ivo Bellin Salarin

Recap/Audio/Processing/FileManagement/RecordingFileManagerHelper.swift

@@ -1,4 +1,5 @@
 import Foundation
+import OSLog
 
 protocol RecordingFileManagerHelperType {
     func getBaseDirectory() -> URL
@@ -8,6 +9,7 @@ protocol RecordingFileManagerHelperType {
 
 final class RecordingFileManagerHelper: RecordingFileManagerHelperType {
     private let userPreferencesRepository: UserPreferencesRepositoryType
+    private let logger = Logger(subsystem: AppConstants.Logging.subsystem, category: String(describing: RecordingFileManagerHelper.self))
 
     init(userPreferencesRepository: UserPreferencesRepositoryType) {
         self.userPreferencesRepository = userPreferencesRepository
@@ -28,33 +30,33 @@ final class RecordingFileManagerHelper: RecordingFileManagerHelperType {
                     bookmarkDataIsStale: &isStale
                 )
 
-                print("📂 Resolved bookmark to: \(url.path), isStale: \(isStale)")
+                logger.info("📂 Resolved bookmark to: \(url.path, privacy: .public), isStale: \(isStale, privacy: .public)")
 
                 // Start accessing the security-scoped resource
                 guard url.startAccessingSecurityScopedResource() else {
-                    print("❌ Failed to start accessing security-scoped resource")
+                    logger.error("❌ Failed to start accessing security-scoped resource")
                     // Fall through to default if we can't access
                     return defaultDirectory()
                 }
 
-                print("✅ Successfully started accessing security-scoped resource")
+                logger.info("✅ Successfully started accessing security-scoped resource")
                 return url
             } catch {
-                print("❌ Bookmark resolution failed: \(error)")
+                logger.error("❌ Bookmark resolution failed: \(error.localizedDescription, privacy: .public)")
                 // Fall through to default if bookmark resolution fails
             }
         }
 
         // Fallback: try the path string (won't work for sandboxed access but kept for backwards compatibility)
         if let customPath = defaults.string(forKey: "customTmpDirectoryPath") {
-            print("📂 Trying fallback path: \(customPath)")
+            logger.info("📂 Trying fallback path: \(customPath, privacy: .public)")
             let url = URL(fileURLWithPath: customPath)
             if FileManager.default.fileExists(atPath: url.path) {
                 return url
             }
         }
 
-        print("📂 Using default directory")
+        logger.info("📂 Using default directory")
         return defaultDirectory()
     }
 

Recap/Helpers/GlobalShortcut/GlobalShortcutManager.swift

@@ -1,5 +1,6 @@
 import Cocoa
 import Carbon
+import OSLog
 
 @MainActor
 protocol GlobalShortcutDelegate: AnyObject {
@@ -14,6 +15,7 @@ final class GlobalShortcutManager {
 
     // Default shortcut: Cmd+R
     private var currentShortcut: (keyCode: UInt32, modifiers: UInt32) = (keyCode: 15, modifiers: UInt32(cmdKey)) // 'R' key with Cmd
+    private let logger = Logger(subsystem: AppConstants.Logging.subsystem, category: String(describing: GlobalShortcutManager.self))
 
     init() {
         setupEventHandling()
@@ -55,7 +57,7 @@ final class GlobalShortcutManager {
         )
 
         guard status == noErr else {
-            print("Failed to install event handler: \(status)")
+            logger.error("Failed to install event handler: \(status, privacy: .public)")
             return
         }
 
@@ -70,11 +72,11 @@ final class GlobalShortcutManager {
         )
 
         guard status2 == noErr else {
-            print("Failed to register hot key: \(status2)")
+            logger.error("Failed to register hot key: \(status2, privacy: .public)")
             return
         }
-        
-        print("Global shortcut registered: Cmd+R")
+
+        logger.info("Global shortcut registered: Cmd+R")
     }
 
     private func unregisterShortcut() {

Recap/MenuBar/Manager/MenuBarPanelManager.swift

@@ -1,6 +1,7 @@
 import SwiftUI
 import AppKit
 import Combine
+import OSLog
 
 @MainActor
 final class MenuBarPanelManager: MenuBarPanelManagerType, ObservableObject {
@@ -35,6 +36,7 @@ final class MenuBarPanelManager: MenuBarPanelManagerType, ObservableObject {
     let generalSettingsViewModel: GeneralSettingsViewModel
     let userPreferencesRepository: UserPreferencesRepositoryType
     let meetingDetectionService: any MeetingDetectionServiceType
+    private let logger = Logger(subsystem: AppConstants.Logging.subsystem, category: String(describing: MenuBarPanelManager.self))
 
     init(
         statusBarManager: StatusBarManagerType,
@@ -71,7 +73,7 @@ final class MenuBarPanelManager: MenuBarPanelManagerType, ObservableObject {
         recapViewModel.$isRecording
             .receive(on: DispatchQueue.main)
             .sink { [weak self] isRecording in
-                print("🔴 Recording state changed to: \(isRecording)")
+                self?.logger.info("🔴 Recording state changed to: \(isRecording, privacy: .public)")
                 self?.statusBarManager.setRecordingState(isRecording)
             }
             .store(in: &cancellables)

Recap/MenuBar/Manager/StatusBar/StatusBarManager.swift

@@ -1,4 +1,5 @@
 import AppKit
+import OSLog
 
 @MainActor
 protocol StatusBarDelegate: AnyObject {
@@ -15,6 +16,7 @@ final class StatusBarManager: StatusBarManagerType {
     weak var delegate: StatusBarDelegate?
     private var themeObserver: NSObjectProtocol?
     private var isRecording = false
+    private let logger = Logger(subsystem: AppConstants.Logging.subsystem, category: String(describing: StatusBarManager.self))
 
     init() {
         setupStatusItem()
@@ -43,7 +45,7 @@ final class StatusBarManager: StatusBarManagerType {
     private func updateIconForCurrentTheme() {
         guard let button = statusItem?.button else { return }
 
-        print("🎨 updateIconForCurrentTheme called, isRecording: \(isRecording)")
+        logger.debug("🎨 updateIconForCurrentTheme called, isRecording: \(self.isRecording, privacy: .public)")
 
         // Always use the black icon, regardless of theme
         if let image = NSImage(named: "barIcon-dark") {
@@ -53,29 +55,29 @@ final class StatusBarManager: StatusBarManagerType {
                 tintedImage.isTemplate = false
                 button.image = tintedImage
                 button.contentTintColor = nil
-                print("🎨 Applied red tinted image")
+                logger.debug("🎨 Applied red tinted image")
             } else {
                 // Use original image
                 let workingImage = image.copy() as! NSImage
                 workingImage.isTemplate = true
                 button.image = workingImage
                 button.contentTintColor = nil
-                print("🎨 Applied normal image")
+                logger.debug("🎨 Applied normal image")
             }
         } else if let fallback = NSImage(named: "barIcon") {
             if isRecording {
                 // Create red-tinted version
                 let tintedImage = createTintedImage(from: fallback, tint: .systemRed)
                 button.image = tintedImage
                 button.contentTintColor = nil
-                print("🎨 Applied red tinted fallback image")
+                logger.debug("🎨 Applied red tinted fallback image")
             } else {
                 // Use original image
                 let workingImage = fallback.copy() as! NSImage
                 workingImage.isTemplate = true
                 button.image = workingImage
                 button.contentTintColor = nil
-                print("🎨 Applied normal fallback image")
+                logger.debug("🎨 Applied normal fallback image")
             }
         }
     }
@@ -99,10 +101,10 @@ final class StatusBarManager: StatusBarManagerType {
     }
 
     func setRecordingState(_ recording: Bool) {
-        print("🎯 StatusBarManager.setRecordingState called with: \(recording)")
+        logger.info("🎯 StatusBarManager.setRecordingState called with: \(recording, privacy: .public)")
         isRecording = recording
         updateIconForCurrentTheme()
-        print("🎯 Icon updated, isRecording = \(isRecording)")
+        logger.info("🎯 Icon updated, isRecording = \(self.isRecording, privacy: .public)")
     }
 
     @objc private func handleButtonClick(_ sender: NSStatusBarButton) {

Recap/Services/Processing/ProcessingCoordinator.swift

@@ -278,7 +278,7 @@ final class ProcessingCoordinator: ProcessingCoordinatorType {
             )
             delegate?.processingStateDidChange(recordingID: recording.id, newState: failureState)
         } catch {
-            print("Failed to update recording state after error: \(error)")
+            logger.error("Failed to update recording state after error: \(error.localizedDescription, privacy: .public)")
         }
 
         delegate?.processingDidFail(recordingID: recording.id, error: error)

Recap/Services/Transcription/TranscriptionService.swift

@@ -1,11 +1,13 @@
 import Foundation
+import OSLog
 import WhisperKit
 
 @MainActor
 final class TranscriptionService: TranscriptionServiceType {
     private let whisperModelRepository: WhisperModelRepositoryType
     private var whisperKit: WhisperKit?
     private var loadedModelName: String?
+    private let logger = Logger(subsystem: AppConstants.Logging.subsystem, category: String(describing: TranscriptionService.self))
 
     init(whisperModelRepository: WhisperModelRepositoryType) {
         self.whisperModelRepository = whisperModelRepository
@@ -77,7 +79,7 @@ final class TranscriptionService: TranscriptionServiceType {
 
     private func loadModel(_ modelName: String, isDownloaded: Bool) async throws {
         do {
-            print("Loading WhisperKit model: \(modelName), isDownloaded: \(isDownloaded)")
+            logger.info("Loading WhisperKit model: \(modelName, privacy: .public), isDownloaded: \(isDownloaded, privacy: .public)")
 
             // Always try to download/load the model, as WhisperKit will handle caching
             // The isDownloaded flag is just for UI purposes, but WhisperKit manages its own cache
@@ -86,24 +88,24 @@ final class TranscriptionService: TranscriptionServiceType {
                 modelRepo: "argmaxinc/whisperkit-coreml",
                 modelFolder: nil,
                 download: true, // Always allow download, WhisperKit will use cache if available
-                progressCallback: { progress in
-                    print("WhisperKit download progress: \(progress.fractionCompleted)")
+                progressCallback: { [weak self] progress in
+                    self?.logger.info("WhisperKit download progress: \(progress.fractionCompleted, privacy: .public)")
                 }
             )
 
-            print("WhisperKit model loaded successfully: \(modelName)")
+            logger.info("WhisperKit model loaded successfully: \(modelName, privacy: .public)")
             self.whisperKit = newWhisperKit
             self.loadedModelName = modelName
 
             // Mark as downloaded in our repository if not already marked
             if !isDownloaded {
                 let modelInfo = await WhisperKit.getModelSizeInfo(for: modelName)
                 try await whisperModelRepository.markAsDownloaded(name: modelName, sizeInMB: Int64(modelInfo.totalSizeMB))
-                print("Model marked as downloaded: \(modelName), size: \(modelInfo.totalSizeMB) MB")
+                logger.info("Model marked as downloaded: \(modelName, privacy: .public), size: \(modelInfo.totalSizeMB, privacy: .public) MB")
             }
 
         } catch {
-            print("Failed to load WhisperKit model \(modelName): \(error)")
+            logger.error("Failed to load WhisperKit model \(modelName, privacy: .public): \(error.localizedDescription, privacy: .public)")
             throw TranscriptionError.modelLoadingFailed("Failed to load model \(modelName): \(error.localizedDescription)")
         }
     }

Recap/UIComponents/Buttons/DownloadPillButton.swift

@@ -1,4 +1,7 @@
 import SwiftUI
+import OSLog
+
+private let downloadPillButtonPreviewLogger = Logger(subsystem: AppConstants.Logging.subsystem, category: "DownloadPillButtonPreview")
 
 struct DownloadPillButton: View {
     let text: String
@@ -78,31 +81,31 @@ struct DownloadPillButton: View {
             isDownloading: false,
             downloadProgress: 0.0
         ) {
-            print("Download started")
+            downloadPillButtonPreviewLogger.info("Download started")
         }
 
         DownloadPillButton(
             text: "Downloading",
             isDownloading: true,
             downloadProgress: 0.3
         ) {
-            print("Download in progress")
+            downloadPillButtonPreviewLogger.info("Download in progress (0.3)")
         }
 
         DownloadPillButton(
             text: "Downloading",
             isDownloading: true,
             downloadProgress: 0.7
         ) {
-            print("Download in progress")
+            downloadPillButtonPreviewLogger.info("Download in progress (0.7)")
         }
 
         DownloadPillButton(
             text: "Downloaded",
             isDownloading: false,
             downloadProgress: 1.0
         ) {
-            print("Download complete")
+            downloadPillButtonPreviewLogger.info("Download complete")
         }
     }
     .padding()

Recap/UIComponents/Buttons/PillButton.swift

@@ -1,4 +1,7 @@
 import SwiftUI
+import OSLog
+
+private let pillButtonPreviewLogger = Logger(subsystem: AppConstants.Logging.subsystem, category: "PillButtonPreview")
 
 struct PillButton: View {
     let text: String
@@ -54,11 +57,11 @@ struct PillButton: View {
 #Preview {
     VStack(spacing: 20) {
         PillButton(text: "Start Recording", icon: "mic.fill") {
-            print("Recording started")
+            pillButtonPreviewLogger.info("Recording started")
         }
 
         PillButton(text: "Button", icon: nil) {
-            print("Button tapped")
+            pillButtonPreviewLogger.info("Button tapped")
         }
     }
     .padding()

Recap/UIComponents/Buttons/SummaryActionButton.swift

@@ -1,4 +1,7 @@
 import SwiftUI
+import OSLog
+
+private let summaryActionButtonPreviewLogger = Logger(subsystem: AppConstants.Logging.subsystem, category: "SummaryActionButtonPreview")
 
 struct SummaryActionButton: View {
     let text: String
@@ -95,15 +98,15 @@ struct SummaryActionButton: View {
                 text: "Copy",
                 icon: "doc.on.doc"
             ) {
-                print("Copy tapped")
+                summaryActionButtonPreviewLogger.info("Copy tapped")
             }
 
             SummaryActionButton(
                 text: "Retry",
                 icon: "arrow.clockwise",
                 isSecondary: true
             ) {
-                print("Retry tapped")
+                summaryActionButtonPreviewLogger.info("Retry tapped")
             }
         }
 
@@ -113,4 +116,4 @@ struct SummaryActionButton: View {
     }
     .padding(40)
     .background(Color.black)
-}
+}

Recap/UIComponents/Buttons/TabButton.swift

@@ -1,4 +1,7 @@
 import SwiftUI
+import OSLog
+
+private let tabButtonPreviewLogger = Logger(subsystem: AppConstants.Logging.subsystem, category: "TabButtonPreview")
 
 struct TabButton: View {
     let text: String
@@ -41,13 +44,13 @@ struct TabButton: View {
 #Preview {
     HStack(spacing: 8) {
         TabButton(text: "General", isSelected: true) {
-            print("General selected")
+            tabButtonPreviewLogger.info("General selected")
         }
 
         TabButton(text: "Whisper Models", isSelected: false) {
-            print("Whisper Models selected")
+            tabButtonPreviewLogger.info("Whisper Models selected")
         }
     }
     .padding()
     .background(Color.black)
-}
+}