Refactor DuckChatSettingsViewModel to improve AI settings handling

Author: mikescamell

duckchat/duckchat-impl/src/main/java/com/duckduckgo/duckchat/impl/ui/settings/DuckChatSettingsViewModel.kt

@@ -44,14 +44,15 @@ import kotlinx.coroutines.flow.flowOn
 import kotlinx.coroutines.flow.receiveAsFlow
 import kotlinx.coroutines.flow.stateIn
 import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 
 class DuckChatSettingsViewModel @AssistedInject constructor(
     @Assisted duckChatActivityParams: GlobalActivityStarter.ActivityParams,
     private val duckChat: DuckChatInternal,
     private val pixel: Pixel,
     private val inputScreenDiscoveryFunnel: InputScreenDiscoveryFunnel,
     private val settingsPageFeature: SettingsPageFeature,
-    dispatcherProvider: DispatcherProvider,
+    private val dispatcherProvider: DispatcherProvider,
 ) : ViewModel() {
     private val commandChannel = Channel<Command>(capacity = 1, onBufferOverflow = DROP_OLDEST)
     val commands = commandChannel.receiveAsFlow()
@@ -142,11 +143,15 @@ class DuckChatSettingsViewModel @AssistedInject constructor(
 
     fun duckChatSearchAISettingsClicked() {
         viewModelScope.launch {
+            val hideAiGeneratedImagesOptionEnabled = withContext(dispatcherProvider.io()) {
+                settingsPageFeature.hideAiGeneratedImagesOption().isEnabled()
+            }
+
             if (settingsPageFeature.embeddedSettingsWebView().isEnabled()) {
                 commandChannel.send(
                     OpenLink(
                         link = DUCK_CHAT_SEARCH_AI_SETTINGS_LINK_EMBEDDED,
-                        titleRes = if (settingsPageFeature.hideAiGeneratedImagesOption().isEnabled()) {
+                        titleRes = if (hideAiGeneratedImagesOptionEnabled) {
                             R.string.duckAiSerpSettingsTitle
                         } else {
                             R.string.duck_chat_assist_settings_title