refactor: 기존 수동으로 세션 처리하는 로직 제거

Author: opficdev

DevLog/Data/Repository/AuthSessionRepositoryImpl.swift

@@ -9,34 +9,33 @@ import Combine
 import Foundation
 
 final class AuthSessionRepositoryImpl: AuthSessionRepository {
-    private let authService: AuthService
     private let userDefaultsStore: UserDefaultsStore
     private var cancellables = Set<AnyCancellable>()
+    private let signInSubject: CurrentValueSubject<Bool, Never>
 
     init(authService: AuthService, userDefaultsStore: UserDefaultsStore) {
-        self.authService = authService
         self.userDefaultsStore = userDefaultsStore
-        self.signIn = authService.uid != nil
+        self.signInSubject = .init(authService.uid != nil)
 
         authService.signedInPublisher
             .removeDuplicates()
             .receive(on: DispatchQueue.main)
             .sink { [weak self] signIn in
-                self?.setSession(signIn)
+                self?.applySession(signIn)
             }
             .store(in: &cancellables)
     }
 
-    @Published private var signIn: Bool = false
-
     var signedInPublisher: AnyPublisher<Bool, Never> {
-        $signIn.eraseToAnyPublisher()
+        signInSubject.eraseToAnyPublisher()
     }
+}
 
-    func setSession(_ signedIn: Bool) {
-        if !signedIn {
+private extension AuthSessionRepositoryImpl {
+    func applySession(_ signedIn: Bool) {
+        if signInSubject.value && !signedIn {
             userDefaultsStore.removeAll()
         }
-        self.signIn = signedIn
+        signInSubject.send(signedIn)
     }
 }

DevLog/Domain/Protocol/AuthSessionRepository.swift

@@ -9,5 +9,4 @@ import Combine
 
 protocol AuthSessionRepository {
     var signedInPublisher: AnyPublisher<Bool, Never> { get }
-    func setSession(_ signedIn: Bool)
 }

DevLog/Domain/UseCase/Auth/Session/AuthSessionUseCase.swift

@@ -9,5 +9,4 @@ import Combine
 
 protocol AuthSessionUseCase {
     var signedInPublisher: AnyPublisher<Bool, Never> { get }
-    func execute(_ signIn: Bool)
 }

DevLog/Domain/UseCase/Auth/Session/AuthSessionUseCaseImpl.swift

@@ -17,8 +17,4 @@ final class AuthSessionUseCaseImpl: AuthSessionUseCase {
     init(_ repository: AuthSessionRepository) {
         self.repository = repository
     }
-
-    func execute(_ signIn: Bool) {
-        repository.setSession(signIn)
-    }
 }

DevLog/Presentation/ViewModel/LoginViewModel.swift

@@ -87,11 +87,7 @@ final class LoginViewModel: Store {
                 do {
                     defer { send(.setLoading(false)) }
                     try await self.signInUseCase.execute(authProvider)
-                    send(.setLogined(true))
-                    sessionUseCase.execute(true)
                 } catch {
-                    send(.setLogined(false))
-                    sessionUseCase.execute(false)
                     if error.isSocialLoginCancelled { return }
                     send(.setAlert(true))
                 }
@@ -101,8 +97,6 @@ final class LoginViewModel: Store {
                 do {
                     defer { send(.setLoading(false)) }
                     try await self.signOutUseCase.execute()
-                    send(.setLogined(false))
-                    sessionUseCase.execute(false)
                 } catch {
                     send(.setAlert(true))
                 }

DevLog/Presentation/ViewModel/RootViewModel.swift

@@ -110,8 +110,6 @@ final class RootViewModel: Store {
         case .signOut:
             Task {
                 try? await signOutUseCase.execute()
-                send(.didLogined(false))
-                sessionUseCase.execute(false)
             }
         }
     }

DevLog/Presentation/ViewModel/SettingViewModel.swift

@@ -43,7 +43,6 @@ final class SettingViewModel: Store {
     private(set) var state = State()
     private let deleteAuthuseCase: DeleteAuthUseCase
     private let signOutUseCase: SignOutUseCase
-    private let sessionUseCase: AuthSessionUseCase
     private let observeSystemThemeUseCase: ObserveSystemThemeUseCase
     private let updateSystemThemeUseCase: UpdateSystemThemeUseCase
     private let loadingState = LoadingState()
@@ -56,13 +55,11 @@ final class SettingViewModel: Store {
     init(
         deleteAuthUseCase: DeleteAuthUseCase,
         signOutUseCase: SignOutUseCase,
-        sessionUseCase: AuthSessionUseCase,
         observeSystemThemeUseCase: ObserveSystemThemeUseCase,
         updateSystemThemeUseCase: UpdateSystemThemeUseCase
     ) {
         self.deleteAuthuseCase = deleteAuthUseCase
         self.signOutUseCase = signOutUseCase
-        self.sessionUseCase = sessionUseCase
         self.observeSystemThemeUseCase = observeSystemThemeUseCase
         self.updateSystemThemeUseCase = updateSystemThemeUseCase
         setupThemeMonitoring()
@@ -111,7 +108,6 @@ final class SettingViewModel: Store {
                     send(.setAlert(isPresented: false))
                     defer { endLoading(.delayed) }
                     try await deleteAuthuseCase.execute()
-                    sessionUseCase.execute(false)
                 } catch {
                     send(.setAlert(isPresented: true, type: .error))
                 }
@@ -123,7 +119,6 @@ final class SettingViewModel: Store {
                     send(.setAlert(isPresented: false))
                     defer { endLoading(.delayed) }
                     try await signOutUseCase.execute()
-                    sessionUseCase.execute(false)
                 } catch {
                     send(.setAlert(isPresented: true, type: .error))
                 }

DevLog/UI/Profile/ProfileView.swift

@@ -98,7 +98,6 @@ struct ProfileView: View {
                     SettingView(viewModel: SettingViewModel(
                         deleteAuthUseCase: container.resolve(DeleteAuthUseCase.self),
                         signOutUseCase: container.resolve(SignOutUseCase.self),
-                        sessionUseCase: container.resolve(AuthSessionUseCase.self),
                         observeSystemThemeUseCase: container.resolve(ObserveSystemThemeUseCase.self),
                         updateSystemThemeUseCase: container.resolve(UpdateSystemThemeUseCase.self)
                     ))