From 205cfe08991bc7ba4c612dffeeaae8a1a3ea26ef Mon Sep 17 00:00:00 2001 From: Klaus Date: Fri, 13 Mar 2026 22:57:15 +0900 Subject: [PATCH] =?UTF-8?q?docs(push-notification):=20=ED=91=B8=EC=8B=9C?= =?UTF-8?q?=20=EC=8B=9C=EC=8A=A4=ED=85=9C=20=EC=B9=B4=ED=85=8C=EA=B3=A0?= =?UTF-8?q?=EB=A6=AC=20=EC=A0=80=EC=9E=A5=20=EC=A0=95=EC=B1=85=20=EB=B3=B4?= =?UTF-8?q?=EC=99=84=20=EC=9E=91=EC=97=85=20=EB=AC=B8=EC=84=9C=EB=A5=BC=20?= =?UTF-8?q?=EC=B6=94=EA=B0=80=ED=95=9C=EB=8B=A4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/20260313_푸시시스템카테고리저장정책보완.md | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 docs/20260313_푸시시스템카테고리저장정책보완.md diff --git a/docs/20260313_푸시시스템카테고리저장정책보완.md b/docs/20260313_푸시시스템카테고리저장정책보완.md new file mode 100644 index 00000000..949719da --- /dev/null +++ b/docs/20260313_푸시시스템카테고리저장정책보완.md @@ -0,0 +1,15 @@ +- [x] 리뷰 결과 요약 및 수정 범위 확정 +- [x] FcmEvent 저장 조건 제거 및 서비스 계층으로 정책 이동 +- [x] PushNotificationService에서 SYSTEM 저장 제외 보장 +- [x] category null 회귀 방지 테스트 추가 +- [x] 검증 실행 (LSP, 테스트, 빌드) + +## 검증 기록 + +### 1차 구현 +- 무엇을: `SYSTEM` 카테고리 저장 제외 정책을 Listener에서 Service로 이동하고, `category = null` 회귀를 막는 테스트를 추가했다. +- 왜: 현재 Listener 조건은 `category != null`을 요구해 타입 기반 카테고리 보정(`resolveCategory`)을 우회할 수 있어, 비SYSTEM 이벤트의 저장 누락 위험이 있었다. +- 어떻게: + - `lsp_diagnostics` 실행: Kotlin LSP 미설정으로 불가(환경상 `.kt` 진단 서버 없음). + - `./gradlew test --tests kr.co.vividnext.sodalive.fcm.notification.PushNotificationServiceTest` 실행: 성공. + - `./gradlew build` 실행: 성공.