Merge pull request #226 from YAPP-Github/BOOK-437-refactor/#225

refactor: FCM 토큰 및 알림 설정 동기화 로직 개선

Author: seoyoon513

core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/NotificationSyncUtils.kt

@@ -0,0 +1,4 @@
+package com.ninecraft.booket.core.common.utils
+
+fun shouldSyncNotification(effectiveEnabled: Boolean, lastSynced: Boolean?): Boolean =
+    lastSynced == null || lastSynced != effectiveEnabled

core/data/impl/src/main/kotlin/com/ninecraft/booket/core/data/impl/repository/DefaultUserRepository.kt

@@ -40,20 +40,11 @@ internal class DefaultUserRepository @Inject constructor(
 
     override suspend fun syncFcmToken() = runSuspendCatching {
         val newToken = getRemoteFcmToken()
-        val localToken = getLocalFcmToken()
-
-        if (newToken == localToken) {
-            Logger.d("Skip FCM token sync (already up-to-date)")
-            return@runSuspendCatching
-        }
-
         registerDevice(newToken)
-        setFcmToken(newToken)
     }
 
     override suspend fun syncFcmToken(fcmToken: String): Result<Unit> = runSuspendCatching {
         registerDevice(fcmToken)
-        setFcmToken(fcmToken)
     }
 
     override val isUserNotificationEnabled = notificationDataSource.isUserNotificationEnabled
@@ -93,12 +84,6 @@ internal class DefaultUserRepository @Inject constructor(
         }
     }
 
-    private suspend fun getLocalFcmToken(): String = notificationDataSource.fcmToken.first()
-
-    private suspend fun setFcmToken(fcmToken: String) {
-        notificationDataSource.setFcmToken(fcmToken)
-    }
-
     private suspend fun getDeviceId(): String {
         return try {
             firebaseInstallations.id.await()

core/datastore/api/src/main/kotlin/com/ninecraft/booket/core/datastore/api/datasource/NotificationDataSource.kt

@@ -3,9 +3,6 @@ package com.ninecraft.booket.core.datastore.api.datasource
 import kotlinx.coroutines.flow.Flow
 
 interface NotificationDataSource {
-    val fcmToken: Flow<String>
-    suspend fun setFcmToken(fcmToken: String)
-
     val isUserNotificationEnabled: Flow<Boolean>
     suspend fun setUserNotificationEnabled(isEnabled: Boolean)
 

core/datastore/impl/src/main/kotlin/com/ninecraft/booket/core/datastore/impl/datasource/DefaultNotificationDataSource.kt

@@ -4,7 +4,6 @@ import androidx.datastore.core.DataStore
 import androidx.datastore.preferences.core.Preferences
 import androidx.datastore.preferences.core.booleanPreferencesKey
 import androidx.datastore.preferences.core.edit
-import androidx.datastore.preferences.core.stringPreferencesKey
 import com.ninecraft.booket.core.datastore.api.datasource.NotificationDataSource
 import com.ninecraft.booket.core.datastore.impl.di.NotificationDataStore
 import com.ninecraft.booket.core.datastore.impl.util.handleIOException
@@ -15,18 +14,6 @@ import javax.inject.Inject
 class DefaultNotificationDataSource @Inject constructor(
     @NotificationDataStore private val dataStore: DataStore<Preferences>,
 ) : NotificationDataSource {
-    override val fcmToken: Flow<String> = dataStore.data
-        .handleIOException()
-        .map { prefs ->
-            prefs[FCM_TOKEN] ?: ""
-        }
-
-    override suspend fun setFcmToken(fcmToken: String) {
-        dataStore.edit { prefs ->
-            prefs[FCM_TOKEN] = fcmToken
-        }
-    }
-
     override val isUserNotificationEnabled: Flow<Boolean> = dataStore.data
         .handleIOException()
         .map { prefs ->
@@ -58,7 +45,6 @@ class DefaultNotificationDataSource @Inject constructor(
     }
 
     companion object Companion {
-        private val FCM_TOKEN = stringPreferencesKey("FCM_TOKEN")
         private val USER_NOTIFICATION_ENABLED = booleanPreferencesKey("USER_NOTIFICATION_ENABLED")
         private val LAST_SYNCED_NOTIFICATION_ENABLED = booleanPreferencesKey("LAST_SYNCED_NOTIFICATION_ENABLED")
     }

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomePresenter.kt

@@ -7,6 +7,7 @@ import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
 import com.ninecraft.booket.core.common.analytics.AnalyticsHelper
 import com.ninecraft.booket.core.common.utils.handleException
+import com.ninecraft.booket.core.common.utils.shouldSyncNotification
 import com.ninecraft.booket.core.data.api.repository.AuthRepository
 import com.ninecraft.booket.core.data.api.repository.BookRepository
 import com.ninecraft.booket.core.data.api.repository.UserRepository
@@ -115,14 +116,15 @@ class HomePresenter @AssistedInject constructor(
                 is HomeUiEvent.OnNotificationPermissionResult -> {
                     scope.launch {
                         val isPermissionGranted = event.granted
-                        val userEnabled = userRepository.getUserNotificationEnabled()
+                        val userSettingEnabled = userRepository.getUserNotificationEnabled()
                         val lastSyncedServerEnabled = userRepository.getLastSyncedNotificationEnabled()
 
-                        val shouldSync = (!isPermissionGranted && lastSyncedServerEnabled != false) ||
-                            (userEnabled && (lastSyncedServerEnabled == null || lastSyncedServerEnabled != isPermissionGranted))
+                        val effectiveNotificationEnabled = userSettingEnabled && isPermissionGranted
+
+                        val shouldSync = shouldSyncNotification(effectiveNotificationEnabled, lastSyncedServerEnabled)
 
                         if (shouldSync) {
-                            syncNotificationSettings(isPermissionGranted)
+                            syncNotificationSettings(effectiveNotificationEnabled)
                         }
                     }
                 }

feature/settings/src/main/kotlin/com/ninecraft/booket/feature/settings/notification/NotificationPresenter.kt

@@ -6,6 +6,7 @@ import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
 import com.ninecraft.booket.core.common.utils.handleException
+import com.ninecraft.booket.core.common.utils.shouldSyncNotification
 import com.ninecraft.booket.core.data.api.repository.UserRepository
 import com.ninecraft.booket.feature.screens.LoginScreen
 import com.ninecraft.booket.feature.screens.NotificationScreen
@@ -92,14 +93,15 @@ class NotificationPresenter @AssistedInject constructor(
                 is NotificationUiEvent.OnNotificationPermissionResult -> {
                     scope.launch {
                         val isPermissionGranted = event.granted
-                        val userEnabled = userRepository.getUserNotificationEnabled()
+                        val userSettingEnabled = userRepository.getUserNotificationEnabled()
                         val lastSyncedServerEnabled = userRepository.getLastSyncedNotificationEnabled()
 
-                        val shouldSync = (!isPermissionGranted && lastSyncedServerEnabled != false) ||
-                            (userEnabled && (lastSyncedServerEnabled == null || lastSyncedServerEnabled != isPermissionGranted))
+                        val effectiveNotificationEnabled = userSettingEnabled && isPermissionGranted
+
+                        val shouldSync = shouldSyncNotification(effectiveNotificationEnabled, lastSyncedServerEnabled)
 
                         if (shouldSync) {
-                            syncNotificationSettings(isPermissionGranted)
+                            syncNotificationSettings(effectiveNotificationEnabled)
                         }
                     }
                 }