Store initial Duck.ai address bar setting (#5972)

Task/Issue URL:
https://app.asana.com/1/137249556945/project/488551667048375/task/1210096722341711?focus=true

### Description

- If the Duck.ai menu setting was initially disabled and the user
updates, then the address bar setting is also disabled.
- However, if the the menu setting is toggled, then both settings toggle
because there is no initial stored value for the address bar setting.
- This fixes that use-case.

### Steps to test this PR

- [ ] Checkout 5.230.1
- [ ] Disable the Duck.ai menu setting
- [ ] Update to this branch
- [ ] Verify that both settings are disabled
- [ ] Toggle the menu setting
- [ ] Verify that only the menu setting is toggled

Author: joshliebe

duckchat/duckchat-impl/src/main/java/com/duckduckgo/duckchat/impl/store/DuckChatDataStore.kt

@@ -22,6 +22,8 @@ import androidx.datastore.preferences.core.booleanPreferencesKey
 import androidx.datastore.preferences.core.edit
 import androidx.datastore.preferences.core.stringPreferencesKey
 import com.duckduckgo.app.di.AppCoroutineScope
+import com.duckduckgo.app.di.IsMainProcess
+import com.duckduckgo.common.utils.DispatcherProvider
 import com.duckduckgo.di.scopes.AppScope
 import com.duckduckgo.duckchat.impl.di.DuckChat
 import com.duckduckgo.duckchat.impl.store.SharedPreferencesDuckChatDataStore.Keys.DUCK_CHAT_OPENED
@@ -38,6 +40,7 @@ import kotlinx.coroutines.flow.distinctUntilChanged
 import kotlinx.coroutines.flow.firstOrNull
 import kotlinx.coroutines.flow.map
 import kotlinx.coroutines.flow.stateIn
+import kotlinx.coroutines.launch
 
 interface DuckChatDataStore {
     suspend fun setShowInBrowserMenu(showDuckChat: Boolean)
@@ -56,6 +59,8 @@ interface DuckChatDataStore {
 @SingleInstanceIn(AppScope::class)
 class SharedPreferencesDuckChatDataStore @Inject constructor(
     @DuckChat private val store: DataStore<Preferences>,
+    private val dispatchers: DispatcherProvider,
+    @IsMainProcess private val isMainProcess: Boolean,
     @AppCoroutineScope private val appCoroutineScope: CoroutineScope,
 ) : DuckChatDataStore {
 
@@ -72,6 +77,21 @@ class SharedPreferencesDuckChatDataStore @Inject constructor(
             ?: true
     }
 
+    init {
+        if (isMainProcess) {
+            storeDerivedValues()
+        }
+    }
+
+    private fun storeDerivedValues() = appCoroutineScope.launch(dispatchers.io()) {
+        store.data.firstOrNull()?.let { prefs ->
+            if (prefs[DUCK_CHAT_SHOW_IN_ADDRESS_BAR] == null) {
+                val default = prefs[DUCK_CHAT_SHOW_IN_MENU] ?: true
+                store.edit { it[DUCK_CHAT_SHOW_IN_ADDRESS_BAR] = default }
+            }
+        }
+    }
+
     private val duckChatShowInBrowserMenu: StateFlow<Boolean> = store.data
         .map { prefs -> prefs[DUCK_CHAT_SHOW_IN_MENU] ?: true }
         .distinctUntilChanged()

duckchat/duckchat-impl/src/test/kotlin/com/duckduckgo/duckchat/impl/store/SharedPreferencesDuckChatDataStoreTest.kt

@@ -50,7 +50,12 @@ class SharedPreferencesDuckChatDataStoreTest {
         )
 
     private val testee: DuckChatDataStore =
-        SharedPreferencesDuckChatDataStore(testDataStore, coroutineRule.testScope)
+        SharedPreferencesDuckChatDataStore(
+            testDataStore,
+            coroutineRule.testDispatcherProvider,
+            true,
+            coroutineRule.testScope,
+        )
 
     companion object {
         val DUCK_CHAT_USER_PREFERENCES = stringPreferencesKey("DUCK_CHAT_USER_PREFERENCES")
@@ -107,9 +112,14 @@ class SharedPreferencesDuckChatDataStoreTest {
     }
 
     @Test
-    fun whenMenuFlagChangesThenAddressBarDefaultsToMenuValue() = runTest {
+    fun whenMenuFlagChangesLaterThenAddressBarRemainsUnchanged() = runTest {
+        assertTrue(testee.getShowInBrowserMenu())
+        assertTrue(testee.getShowInAddressBar())
+
         testee.setShowInBrowserMenu(false)
-        assertFalse(testee.getShowInAddressBar())
+
+        assertFalse(testee.getShowInBrowserMenu())
+        assertTrue(testee.getShowInAddressBar())
     }
 
     @Test