bug fixes (#149)

# bug fixes

## ⚙️ Release Notes
- fixed: the UserStudyChatView would not properly update its UI
- new: added a `--disableFirebase` feature flag
- fixed: the app would unnecessarily load the Fog and Local LLM
components when launching in user study mode


## 📚 Documentation
n/a


## ✅ Testing
n/a


### Code of Conduct & Contributing Guidelines
By creating and submitting this pull request, you agree to follow our
[Code of
Conduct](https://github.com/StanfordBDHG/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordBDHG/.github/blob/main/CONTRIBUTING.md):
- [x] I agree to follow the [Code of
Conduct](https://github.com/StanfordBDHG/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordBDHG/.github/blob/main/CONTRIBUTING.md).

Author: lukaskollmer

LLMonFHIR.xcodeproj/xcshareddata/xcschemes/LLMonFHIR.xcscheme

@@ -81,6 +81,10 @@
             argument = "--useFirebaseEmulator"
             isEnabled = "NO">
          </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--disableFirebase"
+            isEnabled = "NO">
+         </CommandLineArgument>
          <CommandLineArgument
             argument = "--showOnboarding"
             isEnabled = "NO">

LLMonFHIR/FHIRInterpretation/UserStudy/UserStudyChatView.swift

@@ -19,7 +19,7 @@ struct UserStudyChatView: View {
 
     @LocalPreference(.enableTextToSpeech) private var enableTextToSpeechPrefValue
 
-    private let model: UserStudyChatViewModel
+    @State private var model: UserStudyChatViewModel
     @State private var viewState: ViewState = .idle
 
     private var enableTextToSpeech: Binding<Bool> {

LLMonFHIR/FHIRInterpretation/UserStudy/UserStudyChatViewModel.swift

@@ -87,7 +87,7 @@ final class UserStudyChatViewModel: Sendable {
         interpretationModule.resourceSummarizer
     }
 
-    var processingState: ProcessingState = .processingSystemPrompts
+    private(set) var processingState: ProcessingState = .processingSystemPrompts
 
     /// The current navigation state of the study
     private(set) var navigationState: NavigationState = .introduction
@@ -129,24 +129,18 @@ final class UserStudyChatViewModel: Sendable {
         inProgressStudy: InProgressStudy,
         initialQuestionnaireResponse: ModelsR4.QuestionnaireResponse?,
         interpretationModule: FHIRInterpretationModule,
-//        interpreter: FHIRMultipleResourceInterpreter,
-//        resourceSummarizer: FHIRResourceSummarizer,
         uploader: FirebaseUpload?
     ) {
         self.inProgressStudy = inProgressStudy
         self.initialQuestionnaireResponse = initialQuestionnaireResponse
         self.interpretationModule = interpretationModule
-//        self.interpreter =  interpreter
-//        self.resourceSummarizer = resourceSummarizer
         self.uploader = uploader
         configureMessageLimits()
     }
 
     static func unguided(
         title: String,
         interpretationModule: FHIRInterpretationModule,
-//        interpreter: FHIRMultipleResourceInterpreter,
-//        resourceSummarizer: FHIRResourceSummarizer
     ) -> Self {
         let emptyStudy = Study(
             id: Study.unguidedStudyId,
@@ -166,8 +160,6 @@ final class UserStudyChatViewModel: Sendable {
             ),
             initialQuestionnaireResponse: nil,
             interpretationModule: interpretationModule,
-//            interpreter: interpreter,
-//            resourceSummarizer: resourceSummarizer,
             uploader: nil
         )
     }

LLMonFHIR/LLMonFHIRDelegate.swift

@@ -29,7 +29,7 @@ import SpeziLLMOpenAI
 final class LLMonFHIRDelegate: SpeziAppDelegate {
     override var configuration: Configuration {
         Configuration(standard: LLMonFHIRStandard()) {
-            if let config = AppConfigFile.current().firebaseConfig {
+            if !FeatureFlags.disableFirebase, let config = AppConfigFile.current().firebaseConfig {
                 firebaseModules(using: config)
             }
             let openAIInterceptor = OpenAIRequestInterceptor()
@@ -51,8 +51,13 @@ final class LLMonFHIRDelegate: SpeziAppDelegate {
                     retryPolicy: .attempts(3),  // Automatically perform up to 3 retries on retryable OpenAI API status codes
                     middlewares: [openAIInterceptor]
                 ))
-                LLMFogPlatform(configuration: .init(host: "spezillmfog.local", connectionType: .http, authToken: .none))
-                LLMLocalPlatform()
+                switch AppConfigFile.current().appLaunchMode {
+                case .study:
+                    let _ = () // swiftlint:disable:this redundant_discardable_let
+                case .standalone, .test:
+                    LLMFogPlatform(configuration: .init(host: "spezillmfog.local", connectionType: .http, authToken: .none))
+                    LLMLocalPlatform()
+                }
             }
         }
     }

LLMonFHIR/SharedContext/FeatureFlags.swift

@@ -23,4 +23,9 @@ enum FeatureFlags {
 
     /// Whether the app should use a local firebase emulator
     static let useFirebaseEmulator = CommandLine.arguments.contains("--useFirebaseEmulator")
+    
+    /// Whether the app disable its firebase integration.
+    ///
+    /// - Note: if present, this option will take precedence over ``useFirebaseEmulator``.
+    static let disableFirebase = CommandLine.arguments.contains("--disableFirebase")
 }