[leesengjun]

💡 개요

Android production 환경에서 FCM 토큰 발급 및 서버 등록은 정상적으로 이루어지고,
Firebase Console 테스트 메시지도 앱까지 수신되지만
실제 알림이 표시되지 않는 문제를 수정했습니다.

기존에는 AndroidManifest.xml에
laundry_completion 기본 알림 채널 ID만 선언되어 있고
실제 NotificationChannel 생성 코드가 없었습니다.

또한 앱이 foreground 상태일 때
FCM 메시지를 수신해도 시스템 알림으로 표시하는 로직이 없었습니다.

이번 수정으로 Android 알림 채널을 실제로 생성하고,
foreground 상태에서도 수신한 notification payload를 시스템 알림으로 표시하도록 처리했습니다.

🔗 관련 이슈

Close #389

📃 작업내용

• Android laundry_completion NotificationChannel 생성
• Manifest의 기본 notification channel ID와 실제 생성 channel ID 정합성 유지
• Android foreground FCM 메시지 수신 처리 추가
• foreground 메시지를 NotificationManager를 통해 시스템 알림으로 표시
• Android 13+ POST_NOTIFICATIONS 권한 상태 확인
• 앱 전체 알림 및 개별 채널 차단 상태 확인
• 알림용 small icon 및 channel name 리소스 추가
• foreground 알림 표시용 Android MethodChannel 추가
• Android notification 표시 관련 테스트 추가
• 기존 iOS APNs / FCM token 등록 및 foreground presentation 동작 유지
• 기존 FCM token refresh / app start / resume / login 동기화 흐름 유지

🔍 테스트

• POST /api/v2/notifications/fcm-token 200 응답 확인
• Firebase Console 테스트 메시지 수신 로그 확인
  • D/FLTFireMsgReceiver: broadcast received for message
• Android Emulator background 상태에서 테스트 알림 표시 확인
• Android Emulator foreground 상태에서 테스트 알림 표시 확인
• Android notification permission 허용 상태 확인
• laundry_completion 채널 생성 및 HIGH importance 확인
• fvm flutter analyze --no-pub 통과
• notification 관련 테스트 38개 통과
• git diff --check 통과
• 기존 iOS FCM/APNs 로직 회귀 없음 확인

전체 테스트 참고

전체 테스트에서는 기존 History / Report 관련 테스트 2개가
로컬 Flutter SDK / shader 환경 차이로 실패했습니다.

해당 테스트 및 관련 코드는 develop과 동일하며
이번 #389 변경으로 발생한 실패는 아닙니다.

🙋‍♂️ 질문사항

• Android foreground 알림을 native NotificationManager + MethodChannel 방식으로 처리한 구조가 적절한지 확인 부탁드립니다.
• 현재 알림 탭 시 앱만 열리며 특정 화면 이동용 payload 처리는 하지 않고 있습니다.
  추후 알림별 화면 이동이 필요할 경우 별도 이슈로 분리하는 것이 적절한지 확인 부탁드립니다.

✅ 체크리스트

• Android foreground 알림 표시 확인
• Android background 알림 표시 확인
• FCM 토큰 생성 및 production 서버 등록 흐름 유지
• 기존 iOS APNs / FCM 동작 유지
• 관련 테스트 추가
• 관련 없는 예약 polling 코드 수정 없음
• 의존성 추가 없음
• pubspec.lock 변경은 PR에 포함하지 않음
• 🔧 :: Fastlane·CocoaPods gem lock과 배포 환경 재현성 확보 #351 / 🔧 :: Dart 플러그인 의존성과 iOS Podfile.lock 드리프트 정리 #352 범위 수정 없음

[inryeok-bot]

리뷰 결과

변경된 9개 파일을 검토했고 인라인 관찰 1개를 남겼습니다.

심각도	개수
Critical	0
High	0
Medium	1
Low	0

유형: 필수 수정 0, 수정 권장 1, 제안 0, 확인 질문 0, 좋았던 점 0

주요 관찰

• 수정 권장 · MEDIUM 백그라운드 FCM 알림도 새 채널에 연결해야 합니다

검토한 head: 086327d99e98 · 최신 head에서 다시 검토하려면 /review를 작성하세요.

[inryeok-bot]

🟠 수정 권장 · MEDIUM

백그라운드 FCM 알림도 새 채널에 연결해야 합니다

현재 Manifest에는 기본 아이콘만 지정되어 있습니다. 서버가 android_channel_id를 넣지 않은 notification payload를 백그라운드나 종료 상태에서 Firebase SDK가 표시하면 laundry_completion이 아니라 FCM fallback 채널을 사용합니다. 따라서 foreground와 background 알림의 설정·중요도가 서로 달라지고, README에 명시한 단일 채널 동작도 성립하지 않습니다.

이유: 동일한 세탁 완료 알림이 앱 상태에 따라 다른 시스템 채널과 사용자 설정을 따르게 됩니다.

제안: 기본 알림 채널 ID 메타데이터를 추가하고 foreground/background 알림이 모두 laundry_completion을 사용하는지 확인하세요.