fix: display transcription-only recaps

Author: Ivo Bellin Salarin

Recap.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved

@@ -85,7 +85,8 @@ final class DependencyContainer {
     func createSummaryViewModel() -> SummaryViewModel {
         SummaryViewModel(
             recordingRepository: recordingRepository,
-            processingCoordinator: processingCoordinator
+            processingCoordinator: processingCoordinator,
+            userPreferencesRepository: userPreferencesRepository
         )
     }
 }

Recap/DependencyContainer/DependencyContainer.swift

@@ -33,7 +33,7 @@ struct SummaryView<ViewModel: SummaryViewModelType>: View {
                         noRecordingView
                     } else if viewModel.isProcessing {
                         processingView(geometry: geometry)
-                    } else if viewModel.hasSummary {
+                    } else if viewModel.isRecordingReady {
                         summaryView
                     } else {
                         errorView(viewModel.currentRecording?.errorMessage ?? "Recording is in an unexpected state")
@@ -147,22 +147,21 @@ struct SummaryView<ViewModel: SummaryViewModelType>: View {
         VStack(spacing: 0) {
             ScrollView {
                 VStack(alignment: .leading, spacing: UIConstants.Spacing.cardSpacing) {
-                    if let recording = viewModel.currentRecording,
-                       let summaryText = recording.summaryText,
-                       let transcriptionText = recording.transcriptionText {
-                        
+                    if let recording = viewModel.currentRecording {
+
                         VStack(alignment: .leading, spacing: UIConstants.Spacing.cardInternalSpacing) {
-                            if !transcriptionText.isEmpty {
+                            if let transcriptionText = recording.transcriptionText, !transcriptionText.isEmpty {
                                 TranscriptDropdownButton(
                                     transcriptText: transcriptionText
                                 )
                             }
-                            
-                            Text("Summary")
-                                .font(UIConstants.Typography.infoCardTitle)
-                                .foregroundColor(UIConstants.Colors.textPrimary)
-                            
-                            Markdown(summaryText)
+
+                            if let summaryText = recording.summaryText, !summaryText.isEmpty {
+                                Text("Summary")
+                                    .font(UIConstants.Typography.infoCardTitle)
+                                    .foregroundColor(UIConstants.Colors.textPrimary)
+
+                                Markdown(summaryText)
                                 .markdownTheme(.docC)
                                 .markdownTextStyle {
                                     ForegroundColor(UIConstants.Colors.textSecondary)
@@ -202,6 +201,14 @@ struct SummaryView<ViewModel: SummaryViewModelType>: View {
                                         }
                                 }
                                 .textSelection(.enabled)
+                            }
+
+                            if recording.summaryText == nil && recording.transcriptionText == nil {
+                                Text("Recording completed without transcription or summary")
+                                    .font(UIConstants.Typography.bodyText)
+                                    .foregroundColor(UIConstants.Colors.textSecondary)
+                                    .padding(.vertical, 20)
+                            }
                         }
                         .padding(.horizontal, UIConstants.Spacing.contentPadding)
                         .padding(.vertical, UIConstants.Spacing.cardSpacing)

Recap/UseCases/Summary/SummaryView.swift

@@ -7,24 +7,41 @@ final class SummaryViewModel: SummaryViewModelType {
     @Published private(set) var isLoadingRecording = false
     @Published private(set) var errorMessage: String?
     @Published var showingCopiedToast = false
-    
+    @Published private(set) var userPreferences: UserPreferencesInfo?
+
     private let recordingRepository: RecordingRepositoryType
     private let processingCoordinator: ProcessingCoordinatorType
+    private let userPreferencesRepository: UserPreferencesRepositoryType
     private var cancellables = Set<AnyCancellable>()
     private var refreshTimer: Timer?
-    
+
     init(
         recordingRepository: RecordingRepositoryType,
-        processingCoordinator: ProcessingCoordinatorType
+        processingCoordinator: ProcessingCoordinatorType,
+        userPreferencesRepository: UserPreferencesRepositoryType
     ) {
         self.recordingRepository = recordingRepository
         self.processingCoordinator = processingCoordinator
+        self.userPreferencesRepository = userPreferencesRepository
+
+        Task {
+            await loadUserPreferences()
+        }
     }
 
+    func loadUserPreferences() async {
+        do {
+            userPreferences = try await userPreferencesRepository.getOrCreatePreferences()
+        } catch {
+            // If we can't load preferences, assume defaults (auto-summarize enabled)
+            userPreferences = nil
+        }
+    }
+
     func loadRecording(withID recordingID: String) {
         isLoadingRecording = true
         errorMessage = nil
-        
+
         Task {
             do {
                 let recording = try await recordingRepository.fetchRecording(id: recordingID)
@@ -75,6 +92,19 @@ final class SummaryViewModel: SummaryViewModelType {
         guard let recording = currentRecording else { return false }
         return recording.state == .completed && recording.summaryText != nil
     }
+
+    var isRecordingReady: Bool {
+        guard let recording = currentRecording else { return false }
+        guard recording.state == .completed else { return false }
+
+        // If auto-summarize is enabled, we need summary text
+        if userPreferences?.autoSummarizeEnabled == true {
+            return recording.summaryText != nil
+        }
+
+        // If auto-summarize is disabled, the recording is valid when completed
+        return true
+    }
 
     func retryProcessing() async {
         guard let recording = currentRecording else { return }

Recap/UseCases/Summary/ViewModel/SummaryViewModel.swift

@@ -8,8 +8,9 @@ protocol SummaryViewModelType: ObservableObject {
     var processingStage: ProcessingStatesCard.ProcessingStage? { get }
     var isProcessing: Bool { get }
     var hasSummary: Bool { get }
+    var isRecordingReady: Bool { get }
     var showingCopiedToast: Bool { get }
-    
+
     func loadRecording(withID recordingID: String)
     func loadLatestRecording()
     func retryProcessing() async