[CARECODE-017] P0-P2 기능 개선 커밋 PR 생성 및 리뷰 체크리스트

[RosieOh]

🧑‍💻 오늘의 개발하면서 생긴 크고 작은 이슈사항을 기록해주세요

💚 어떤 기능인가요?

이번 브랜치의 대규모 커밋(도메인별 기능 개선, 예외/검증/DTO/조회 흐름 정비)을 PR로 통합 제출하기 위한 이슈입니다.
리뷰어가 변경 범위를 빠르게 파악하고, 머지 전 필수 확인 항목을 체크할 수 있도록 PR 기준선을 정의합니다.

PR 범위(핵심)

• Health / Notification / CareFacility / Policy / Chatbot 도메인 개선
• Controller/Service/Repository/DTO/Facade 역할별 구조 정리
• 커밋 메시지 표준화(한글 상세 + #이슈번호)

✅ To Dos

• PR 본문에 변경 범위(도메인별) 요약 작성
• 위험 구간(예외 처리/DTO 스키마/조회 조건) 명시
• 테스트 실행 결과(./gradlew test) 및 실패/보류 항목 기록
• API 영향 여부(스키마/응답 포맷) 체크리스트 첨부
• 리뷰 포인트(우선 확인 파일) 지정
• 머지 전 최종 확인(라벨/이슈 링크/마일스톤) 완료

완료 조건

• PR에 리뷰 체크리스트가 포함되어 있고, 필수 검증 항목이 모두 확인됨
• 리뷰어가 도메인별 변경 영향도를 명확히 파악 가능한 상태로 제출됨

[RosieOh]

PR #66 으로 통합 제출했고, 리뷰 기준선은 PR 코멘트로 단계마다 남겼습니다 — 실기동 검증 결과, 회귀 방지 추가, 알림 루프 2종, 설계 문서 정리.
공통 근거

리팩토링 시점으로부터 상당한 작업이 누적됐고, 그동안 아래가 갖춰졌습니다.

• 테스트 43종 (단위 · H2 통합 · Testcontainers · 실연동)
• ./gradlew clean build 통과 (JaCoCo 커버리지 검증 포함)
• prod 프로파일 실기동 검증 — 마이그레이션 18건 적용, 공개 경로 200 / 보호 경로 401
• 스키마 정합성과 접근제어를 CI 가 회귀 감시 ([ Test ] 스키마·접근제어 회귀를 CI 가 잡도록 계약 테스트 추가 #72)

특히 실기동 검증에서 리팩토링 잔재를 실제로 찾아 고쳤습니다 (#70) — 참조 0건인 HealthRecordType 매핑, 허구 데이터를 만들던 기동 러너, 테이블 없이 존재하던 정책 북마크 API.

이 이슈들은 당시의 불확실성을 추적하려던 것이고 그 불확실성은 해소됐습니다. 새 문제가 나오면 구체적인 이슈로 여는 편이 낫습니다.