[#284] 탭뷰에 있는 알림 이미지에 읽지 않은 푸시알림 데이터가 있을 경우 해당 데이터의 갯수를 배지로 보이도록 구현한다 (#290)

opficdev · web-flow · commit f20d1a694220 · 2026-03-17T12:22:27.000+09:00
* feat: 불필요 방출 제거

* feat: 앱델리에서 푸시 알림 관련 역학 책임 제거

* feat: 읽지 않은 푸시알림 갯수를 탭바와 앱 배지로 보이도록 구현

* fix: 리스너에 삭제되지 않은 임시 푸시알람 데이터인 경우를 걸러내어 삭제 예정인 데이터 갯수는 제외하도록 수정

* fix: 로그아웃 시 앱 아이콘에 푸시알람 배지가 떠있는 이슈 해결

* test: 코덱스 리뷰용 AGENTS.md 생성

* chore: 불필요 파일 제거

* chore: 제미나이용 가이드 업데이트

* fix: 뷰모델이 생성될 때마다 유즈케이스가 호출되는 이슈 해결

* feat: 에러 로깅 추가

* fix: error 문자열이 포함되었을 경우 빌드가 성공해도 CI가 실패했다고 뜨는 이슈 해결
diff --git a/.gemini/styleguide.md b/.gemini/styleguide.md
@@ -1,2 +1,9 @@
-# 지침사항
-당신은 iOS 수석 개발자 입니다. 당신은 한국인 이므로 코드 리뷰 및 요약을 한국어로 해야 합니다.
+## Review Guidelines
+
+- Write all review comments in Korean.
+- Keep review comments concise and high-signal.
+- Prioritize findings about bugs, performance, and readability.
+- Do not explain obvious, trivial, or low-signal issues.
+- When useful, begin the review with a short summary of the main changes.
+- Focus on actionable feedback rather than broad commentary.
+
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -206,15 +206,6 @@ jobs:
           XC_STATUS=${PIPESTATUS[0]}
           set -e
 
-          if [ -f build.log ]; then
-            echo "== error: lines =="
-            if grep -i "error:" build.log; then
-              if [ "$XC_STATUS" -eq 0 ]; then
-                XC_STATUS=1
-              fi
-            fi
-          fi
-
           exit $XC_STATUS
 
       - name: Comment build failure on PR
@@ -228,9 +219,9 @@ jobs:
             if (fs.existsSync(path)) {
               const log = fs.readFileSync(path, 'utf8');
               const lines = log.split(/\r?\n/);
-              const errorLines = lines.filter((line) => /error:/i.test(line));
+              const errorLines = lines.filter((line) => /^(.*?):(\d+):(\d+):\s+error:/i.test(line));
               if (errorLines.length > 0) {
-                body += "Lines containing 'error:':\n\n```\n" + errorLines.join('\n') + '\n```\n';
+                body += "Compiler error lines:\n\n```\n" + errorLines.join('\n') + '\n```\n';
 
                 const repoRoot = process.env.GITHUB_WORKSPACE || process.cwd();
                 const pathMod = require('path');
@@ -258,7 +249,7 @@ jobs:
                   body += "\nCode excerpts:\n\n```\n" + snippets.join('\n\n') + "\n```\n";
                 }
               } else {
-                body += "No lines containing 'error:' were found in build.log.";
+                body += "No compiler-style error diagnostics were found in build.log.";
               }
             } else {
               body += 'build.log not found.';
diff --git a/DevLog/App/Assembler/DomainAssembler.swift b/DevLog/App/Assembler/DomainAssembler.swift
@@ -103,6 +103,10 @@ private extension DomainAssembler {
             FetchPushNotificationsUseCaseImpl(container.resolve(PushNotificationRepository.self))
         }
 
+        container.register(ObserveUnreadPushCountUseCase.self) {
+            ObserveUnreadPushCountUseCaseImpl(container.resolve(PushNotificationRepository.self))
+        }
+
         container.register(TogglePushNotificationReadUseCase.self) {
             TogglePushNotificationReadUseCaseImpl(container.resolve(PushNotificationRepository.self))
         }
diff --git a/DevLog/App/Delegate/AppDelegate.swift b/DevLog/App/Delegate/AppDelegate.swift
@@ -8,17 +8,10 @@
 import UIKit
 import Firebase
 import FirebaseAuth
-import FirebaseFirestore
-import FirebaseMessaging
 import GoogleSignIn
-import Combine
-import UserNotifications
 
 class AppDelegate: UIResponder, UIApplicationDelegate, MessagingDelegate {
     private let logger = Logger(category: "AppDelegate")
-    private var store: Firestore { Firestore.firestore() }
-    private var authStateListenerHandle: AuthStateDidChangeListenerHandle?
-    private var cancellable: AnyCancellable?
 
     func application(
         _ app: UIApplication,
@@ -52,7 +45,6 @@ class AppDelegate: UIResponder, UIApplicationDelegate, MessagingDelegate {
 
         // Firebase Messaging 설정
         Messaging.messaging().delegate = self
-        observeAuthState()
         
         // 앱이 완전 종료되어도, 알림을 통해 앱이 시작된 경우 처리
         if let remoteNotification = launchOptions?[.remoteNotification] as? [AnyHashable: Any] {
@@ -80,17 +72,12 @@ class AppDelegate: UIResponder, UIApplicationDelegate, MessagingDelegate {
         logger.error("Failed to register APNs token", error: error)
     }
 
-    func applicationDidBecomeActive(_ application: UIApplication) {
-        syncBadgeCount()
-    }
-    
     // FCMToken 갱신
     func messaging(
         _ messaging: Messaging, didReceiveRegistrationToken fcmToken: String?
     ) {
         if let fcmToken = fcmToken {
             logger.info("FCM token: \(fcmToken)")
-            NotificationCenter.default.post(name: .fcmToken, object: nil, userInfo: ["fcmToken": fcmToken])
         }
     }
 }
@@ -108,122 +95,6 @@ private extension AppDelegate {
             }
         }
     }
-
-    func observeAuthState() {
-        authStateListenerHandle = Auth.auth().addStateDidChangeListener { [weak self] _, user in
-            guard let self else { return }
-
-            self.cancellable?.cancel()
-
-            guard user != nil else {
-                self.updateBadgeCount(0)
-                return
-            }
-
-            self.startObservingBadgeCount()
-            self.syncBadgeCount()
-        }
-    }
-
-    func syncBadgeCount() {
-        Task { @MainActor [weak self] in
-            guard let self else { return }
-            guard Auth.auth().currentUser != nil else {
-                self.updateBadgeCount(0)
-                return
-            }
-
-            do {
-                let unreadNotificationCount = try await self.fetchUnreadNotificationCount()
-                self.updateBadgeCount(unreadNotificationCount)
-            } catch {
-                self.logger.error("Failed to fetch unread notification count", error: error)
-            }
-        }
-    }
-
-    private func startObservingBadgeCount() {
-        do {
-            cancellable = try observeUnreadNotificationCount()
-                .receive(on: DispatchQueue.main)
-                .sink(
-                    receiveCompletion: { [weak self] completion in
-                        guard let self else { return }
-
-                        if case .failure(let error) = completion {
-                            self.logger.error("Failed to observe unread notification count", error: error)
-                        }
-                    },
-                    receiveValue: { [weak self] count in
-                        self?.updateBadgeCount(count)
-                    }
-                )
-        } catch {
-            logger.error("Failed to start observing badge count", error: error)
-        }
-    }
-
-    private func fetchUnreadNotificationCount() async throws -> Int {
-        logger.info("Fetching unread notification count")
-
-        guard let uid = Auth.auth().currentUser?.uid else {
-            logger.error("User not authenticated")
-            throw AuthError.notAuthenticated
-        }
-
-        do {
-            let snapshot = try await store.collection("users/\(uid)/notifications")
-                .whereField("isRead", isEqualTo: false)
-                .getDocuments()
-
-            let unreadNotificationCount = snapshot.documents.count
-            logger.info("Unread notification count: \(unreadNotificationCount)")
-            return unreadNotificationCount
-        } catch {
-            logger.error("Failed to fetch unread notification count", error: error)
-            throw error
-        }
-    }
-
-    private func observeUnreadNotificationCount() throws -> AnyPublisher<Int, Error> {
-        logger.info("Observing unread notification count")
-
-        guard let uid = Auth.auth().currentUser?.uid else {
-            logger.error("User not authenticated")
-            throw AuthError.notAuthenticated
-        }
-
-        let subject = PassthroughSubject<Int, Error>()
-        let listener = store.collection("users/\(uid)/notifications")
-            .whereField("isRead", isEqualTo: false)
-            .addSnapshotListener { [weak self] snapshot, error in
-                guard let self else { return }
-                if let error {
-                    self.logger.error("Failed to observe unread notification count", error: error)
-                    subject.send(completion: .failure(error))
-                    return
-                }
-
-                guard let snapshot else { return }
-
-                let unreadNotificationCount = snapshot.documents.count
-                self.logger.info("Observed unread notification count: \(unreadNotificationCount)")
-                subject.send(unreadNotificationCount)
-            }
-
-        return subject
-            .handleEvents(receiveCancel: { listener.remove() })
-            .eraseToAnyPublisher()
-    }
-
-    @MainActor
-    private func updateBadgeCount(_ count: Int) {
-        UNUserNotificationCenter.current().setBadgeCount(count) { [weak self] error in
-            if let error {
-                self?.logger.error("Failed to update badge count", error: error)
-            }
-        }
-    }
 }
 
 extension AppDelegate: UNUserNotificationCenterDelegate {
@@ -251,7 +122,3 @@ extension AppDelegate: UNUserNotificationCenterDelegate {
         completionHandler()
     }
 }
-
-extension Notification.Name {
-    static let fcmToken = Notification.Name("fcmToken")
-}
diff --git a/DevLog/App/RootView.swift b/DevLog/App/RootView.swift
@@ -17,18 +17,17 @@ struct RootView: View {
             Color(UIColor.systemGroupedBackground).ignoresSafeArea()
             if let signIn = viewModel.state.signIn {
                 if signIn && !viewModel.state.isFirstLaunch {
-                    MainView()
+                    MainView(viewModel: MainViewModel(
+                        observeUnreadPushCountUseCase: container.resolve(ObserveUnreadPushCountUseCase.self)
+                    ))
                 } else {
                     LoginView(viewModel: LoginViewModel(
                         signInUseCase: container.resolve(SignInUseCase.self),
                         signOutUseCase: container.resolve(SignOutUseCase.self),
                         sessionUseCase: container.resolve(AuthSessionUseCase.self))
                     )
                     .onAppear {
-                        if viewModel.state.isFirstLaunch {
-                            viewModel.send(.setFirstLaunch(false))
-                            viewModel.send(.signOutAuto)
-                        }
+                        viewModel.send(.onAppear)
                     }
                 }
             } else {
diff --git a/DevLog/Data/Repository/PushNotificationRepositoryImpl.swift b/DevLog/Data/Repository/PushNotificationRepositoryImpl.swift
@@ -51,6 +51,11 @@ final class PushNotificationRepositoryImpl: PushNotificationRepository {
             .eraseToAnyPublisher()
     }
 
+    func observeUnreadPushCount() throws -> AnyPublisher<Int, Error> {
+        try service.observeUnreadPushCount()
+            .eraseToAnyPublisher()
+    }
+
     // 푸시 알림 기록 삭제
     func deleteNotification(_ notificationID: String) async throws {
         try await service.deleteNotification(notificationID)
diff --git a/DevLog/Domain/Protocol/PushNotificationRepository.swift b/DevLog/Domain/Protocol/PushNotificationRepository.swift
@@ -20,6 +20,7 @@ protocol PushNotificationRepository {
         _ query: PushNotificationQuery,
         limit: Int
     ) throws -> AnyPublisher<PushNotificationPage, Error>
+    func observeUnreadPushCount() throws -> AnyPublisher<Int, Error>
     func deleteNotification(_ notificationID: String) async throws
     func undoDeleteNotification(_ notificationID: String) async throws
     func toggleNotificationRead(_ todoId: String) async throws
diff --git a/DevLog/Domain/UseCase/PushNotification/Fetch/ObserveUnreadPushCountUseCase.swift b/DevLog/Domain/UseCase/PushNotification/Fetch/ObserveUnreadPushCountUseCase.swift
@@ -0,0 +1,12 @@
+//
+//  ObserveUnreadPushCountUseCase.swift
+//  DevLog
+//
+//  Created by opfic on 3/17/26.
+//
+
+import Combine
+
+protocol ObserveUnreadPushCountUseCase {
+    func execute() throws -> AnyPublisher<Int, Error>
+}
diff --git a/DevLog/Domain/UseCase/PushNotification/Fetch/ObserveUnreadPushCountUseCaseImpl.swift b/DevLog/Domain/UseCase/PushNotification/Fetch/ObserveUnreadPushCountUseCaseImpl.swift
@@ -0,0 +1,22 @@
+//
+//  ObserveUnreadPushCountUseCaseImpl.swift
+//  DevLog
+//
+//  Created by opfic on 3/17/26.
+//
+
+import Combine
+
+final class ObserveUnreadPushCountUseCaseImpl: ObserveUnreadPushCountUseCase {
+    private let repository: PushNotificationRepository
+
+    init(_ repository: PushNotificationRepository) {
+        self.repository = repository
+    }
+
+    func execute() throws -> AnyPublisher<Int, Error> {
+        try repository.observeUnreadPushCount()
+            .removeDuplicates()
+            .eraseToAnyPublisher()
+    }
+}
diff --git a/DevLog/Infra/Service/PushNotificationService.swift b/DevLog/Infra/Service/PushNotificationService.swift
@@ -178,6 +178,30 @@ final class PushNotificationService {
             .eraseToAnyPublisher()
     }
 
+    func observeUnreadPushCount() throws -> AnyPublisher<Int, Error> {
+        guard let uid = Auth.auth().currentUser?.uid else { throw AuthError.notAuthenticated }
+
+        let subject = PassthroughSubject<Int, Error>()
+        let listener = store.collection("users/\(uid)/notifications")
+            .whereField("isRead", isEqualTo: false)
+            .addSnapshotListener { snapshot, error in
+                if let error {
+                    subject.send(completion: .failure(error))
+                    return
+                }
+
+                guard let snapshot else { return }
+                let unreadPushCount = snapshot.documents.filter { document in
+                    !(document.data()[Key.deletingAt.rawValue] is Timestamp)
+                }.count
+                subject.send(unreadPushCount)
+            }
+
+        return subject
+            .handleEvents(receiveCancel: { listener.remove() })
+            .eraseToAnyPublisher()
+    }
+
     /// 푸시 알림 기록 삭제
     func deleteNotification(_ notificationID: String) async throws {
         do {
diff --git a/DevLog/Presentation/ViewModel/MainViewModel.swift b/DevLog/Presentation/ViewModel/MainViewModel.swift
diff --git a/DevLog/Presentation/ViewModel/RootViewModel.swift b/DevLog/Presentation/ViewModel/RootViewModel.swift
diff --git a/DevLog/UI/Common/MainView.swift b/DevLog/UI/Common/MainView.swift

Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,10 @@ private extension DomainAssembler {`
`103`	`103`	`FetchPushNotificationsUseCaseImpl(container.resolve(PushNotificationRepository.self))`
`104`	`104`	`}`
`105`	`105`
	`106`	`+ container.register(ObserveUnreadPushCountUseCase.self) {`
	`107`	`+ ObserveUnreadPushCountUseCaseImpl(container.resolve(PushNotificationRepository.self))`
	`108`	`+ }`
	`109`	`+`
`106`	`110`	`container.register(TogglePushNotificationReadUseCase.self) {`
`107`	`111`	`TogglePushNotificationReadUseCaseImpl(container.resolve(PushNotificationRepository.self))`
`108`	`112`	`}`