[Sadturtleman]

Closes #506

무엇이 문제였나

onRecipientChipClick 본문이 // TODO: 수신인 지정 화면 Route 추가 후 연결 주석뿐이었다. 칩은 탭 가능한 UI 로 보이는데 눌러도 아무 일이 없다. 목적지 라우트(SettingRoute.RecipientRegisterRoute)와 그리로 가는 액션은 이미 있었고 배선만 빠져 있었다.

목적지를 등록으로 고른 근거

이슈 Note 가 «등록인지 목록인지 기획 확정 전» 이라고 적어 둔 지점이다. 코드가 답을 좁혀 준다.

• 이 칩은 미완료일 때만 눌린다 (Completed 는 onClick = null)
• 그 «미완료» 판정이 곧 등록된 수신자가 0명 이다 — 홈 요약이 userRepository.getReceivers() 결과로 정한다

즉 목록으로 보내면 빈 화면이 나오고 사용자는 다시 등록 화면을 찾아야 한다. 상태를 해소하는 화면으로 보내는 쪽만 막다른 길이 아니다. 기획이 «목록» 으로 확정되면 한 줄만 바꾸면 되고, 근거는 docs/qa/assumptions.md 에 남긴다.

실기기 시연 (에뮬레이터, 실서버)

수신자를 지워 「미완료」 상태를 만든 뒤 칩을 눌렀다.

$ DELETE /api/v1/users/receivers/17   → 200   (기록에 붙은 수신자 해제 후)
$ GET    /api/v1/users/receivers      → data: []

홈 진입 → '수신인 지정 미완료' @ (254,551)
칩 탭   → 이름 | 이름을 입력하세요 | 연락처 | … | 관계 | 이메일 | 마지막 인사말
          (수신자 등록 화면)

고치기 전은 이 diff 가 그대로 보여 준다 — 본문이 주석 한 줄이라 탭이 아무 데도 가지 않았다.

검증

ktlintCheck · :app:assembleDebug → BUILD SUCCESSFUL

내비게이션 배선이라 단위 테스트로 고정할 자리가 마땅치 않다(이 레포에 AppNavigationActions 테스트 선례가 없고, NavController 를 세우는 것은 androidTest 범위다). 대신 위 실기 시연으로 확인했다.

🤖 Generated with Claude Code

[OnlyReadOneBook]

총평

승인합니다. 배선 한 줄이 빠져 있던 자리를 정확히 채웠고, 목적지 선택 근거도 코드로 좁혀 놨습니다.

검증한 것:

• SettingRoute.RecipientRegisterRoute 는 SettingNavGraph.kt:124 에 등록돼 있고, settingNavGraph 가 최상위 NavHost 에 붙어 있어 홈 탭에서 직접 navigate 가 성립합니다 (AppNavigation.kt:123).
• 같은 파일 AppNavigationActions.kt:269 에 이미 동일한 호출 선례가 있어 관행과도 맞습니다.
• 등록 성공 시 onRegisterSuccess = actions::onRecipientRegisterBack → popBackStack() 이라 홈으로 되돌아옵니다. 바텀바는 bottomBarRoutes 기준이라 등록 화면에서 감춰집니다 — 막다른 길이 없습니다.

지적

1. [normal] 주석이 존재하지 않는 근거를 가리킨다 → app/src/main/java/com/afternote/afternote_fe/navigation/AppNavigationActions.kt:347 인라인 코멘트.

검증

• 시안 대조: 해당 없음. 렌더 변경이 없는 내비게이션 배선입니다.
• 실기동 QA: 저자 시연으로 갈음. 본문의 «수신자 0명 → 칩 탭 → 등록 화면» 시연이 위 라우트 등록 상태와 일치합니다. 별도 재현은 하지 않았습니다.
• CI: 전 체크 green.

[OnlyReadOneBook]

[normal] 이 주석이 가리키는 docs/qa/assumptions.md 에 #506 항목이 없습니다.

$ grep -n -E "506|수신인 지정|RecipientRegister" docs/qa/assumptions.md
(0건)

PR 본문도 «근거는 docs/qa/assumptions.md 에 남긴다» 라고 적었는데 diff 는 이 파일 하나뿐입니다. 지금 상태로 머지되면 나중에 이 판단을 되짚는 사람이 문서를 열었다가 아무것도 못 찾습니다.

문서에 항목을 추가하거나, 추가하지 않을 거라면 주석에서 그 참조를 빼고 여기서 근거를 끝내 주세요.

[github-actions]

🚀 QA 배포 권장

• 위험도: high
• 포함 이슈: fix(mindrecord): 임시저장 선택 삭제 실패가 무시되고 완료 처리됨 #442, fix(home): 홈 '수신인 지정 미완료' 칩 탭 무동작 — 내비게이션 핸들러 미연결(TODO) #506, refactor(app): 홈·수신자 홈 화면 feature 모듈 이전 — app 잔존 presentation 정리 #546, fix(mindrecord): 홈 MEMORIES 카드·추억 공간 화면이 고정 더미 — 실데이터 미연동 #559, fix(mindrecord): 데일리질문 생성·수정 응답의 userDailyQuestionId 를 BaseResponse<Unit> 으로 버림 #573, fix(mindrecord): 주간리포트 상단 카운트와 요약 기록일수 불일치 — diaryAmount 는 1인데 week[] 에 미포함 #590, fix(mindrecord): 데일리질문 목록 상단 '오늘의 추천 질문' 영역 미구현 — 조회한 todayQuestion 을 화면이 소비하지 않음 #592, refactor(afternote): 수신자 흐름 58개 파일을 feature:receiver 로 이전 — 애프터노트 모듈이 타 도메인 열람 관문 소유 #615, fix(mindrecord): 홈 TODAY'S QUESTION 카드 '데일리질문 답변하기' 무동작 — 카드 데이터는 정상 #627, fix(mindrecord): data 계층 runCatching 이 CancellationException 을 Result 로 소비 — 취소 전파 보존 #670, fix(mindrecord): 화면 배경이 시안 #FAFAFA 대신 Material3 기본 surface #FEF7FF 로 렌더 — 탭 간 배경 불일치 #691, fix(mindrecord): 보기 스위치·캘린더 탐색 상태가 콘텐츠와 불일치 #724, fix(mindrecord): 주간리포트 주차 드롭다운이 5개로 끝나 시안의 추가 이력·스크롤 미노출 #729, fix(mindrecord): 주간리포트 요약 문구 공백 소실 — '이번 주,이름님은N일의…'로 붙어 표시 #732, fix(mindrecord): 주간리포트 캘린더가 감정 이모티콘과 점을 함께 표시 — 시안·디자이너 정의는 배타적 #749, fix(mindrecord): 주간리포트 데일리질문 날짜 파싱 실패가 오늘 날짜로 무음 폴백 — 틀린 날짜를 정상처럼 표시 #751, fix(mindrecord): 일기 날짜 선택 제목이 '발송 날짜' — 시안은 '기록 날짜'로 수정됨 #760, fix(mindrecord): 작성 툴바 '임시저장' 카운트가 항상 0 — 두 작성 화면 모두 draftCount 미전달 #769, refactor(mindrecord): DTO 기본값·불필요 nullable 제거 — 누락 계약을 정상값으로 은폐 #789, 로그아웃 시 로컬 저장소 3개가 지워지지 않음 — DataStore 수명(scope) 계약 도입 #912, chore(build): 빌드·테스트 클래스패스 bouncycastle 1.84 상향 — dependency-review critical(GHSA-574f-3g2m-x479) 해소 #921, NavigationDebugLogger 삭제 — 역할 끝난 일회성 디버그 프로브 #927, core:startup 모듈 삭제 — Initializer 2개를 app 으로 이동 #929, core:di 모듈 삭제 — 바인딩을 구현 옆으로 옮기고 repository impl 을 internal 로 #930, AuthorReceiverRepository 가 동작하지 않음 — 에디터 수신자 목록이 항상 빈 리스트 #935, develop androidTest 컴파일 실패 44건 — 배치 머지 API 드리프트가 미검증 착륙, 컴파일 게이트 부재 #936, 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 중복 구현 1개로 #938, fix(core): presigned URL 요청에 필수 contentLength 가 빠져 이미지 업로드가 전부 실패 #950, fix(mindrecord): 수신자 데일리질문 isDraft 를 파싱하지 않고 false 하드코딩 — draft 노출 방어가 무효 #956, fix(build): androidx-compose-material3 선언 버전과 Gradle 해석 버전을 일치시킨다 #973, fix(build): coroutines-core 선언 버전과 Gradle 해석 버전을 일치시킨다 #974, fix(build): io.netty:netty-codec 보안 권고 대응 필요 #975, fix(build): io.netty:netty-codec-http 보안 권고 대응 필요 #976, fix(build): io.netty:netty-codec-http2 보안 권고 대응 필요 #977, fix(build): io.netty:netty-common 보안 권고 대응 필요 #978, fix(build): io.netty:netty-handler 보안 권고 대응 필요 #979, fix(build): io.netty:netty-handler-proxy 보안 권고 대응 필요 #980, fix(build): org.apache.commons:commons-lang3 보안 권고 대응 필요 #981, fix(build): org.bitbucket.b_c:jose4j 보안 권고 대응 필요 #982, fix(build): org.bouncycastle:bcpkix-jdk18on 보안 권고 대응 필요 #983, fix(build): org.bouncycastle:bcprov-jdk18on 보안 권고 대응 필요 #984, fix(build): org.jdom:jdom2 보안 권고 대응 필요 #985, fix(ci): 릴리스·WIF canary 배포가 같은 Firebase 앱에 동시 업로드 가능 — concurrency group 분리 #995, perf(ci): PR 검증 4개 job이 같은 debug variant 를 각자 컴파일 — build cache 가 전역 off #996, chore(ci): 실서비스 secret workflow 3종의 검증·decode 블록 3변종 중복 — 공유 composite 로 통합 #997, chore(ci): dependency-graph job correlator 불일치 — develop base PR 의 dependency-review 가 의존성 전량을 신규 도입으로 오탐 #1009, perf(ci): ktlint 를 PR 당 두 번 검사 — reviewdog 경로 32초·버전 이원화 정리 #1012, develop androidTest 컴파일 실패 21건 — #936 후속, 게이트 착륙 전에 갈린 도메인 계약 5건 #1022, fix(ci): CodeQL 빌드 스텝이 :app:mergeExtDexDebug 에서 OOM — 산발적 실패로 보안 경고와 구분 불가 #1023, fix(ci): base 와 충돌한 PR 은 pull_request CI 가 트리거되지 않아 최신 커밋이 미검증으로 방치 — 충돌 자동 감지·라벨 필요 #1028
• 판단 근거: 고위험 변경이 포함됐습니다: .github/workflows/release-distribution.yml, PR fix(ci): 릴리스·WIF canary 가 같은 Firebase 앱에 동시 업로드 가능 (closes 995) [base: develop] #1001, PR chore(ci): 실서비스 secret workflow 3종의 검증·decode 3변종을 composite 로 통합 (closes 997) [base: develop] #1002, PR [fix] #627 홈 TODAY'S QUESTION 답변 CTA 를 작성 화면에 연결한다 #894. 다른 변경을 기다리지 않고 QA 배포합니다.

QA 포인트

• fix(home): 홈 '수신인 지정 미완료' 칩 탭 무동작 — 내비게이션 핸들러 미연결(TODO) #506 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• refactor(app): 홈·수신자 홈 화면 feature 모듈 이전 — app 잔존 presentation 정리 #546 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• refactor(afternote): 수신자 흐름 58개 파일을 feature:receiver 로 이전 — 애프터노트 모듈이 타 도메인 열람 관문 소유 #615 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 보기 스위치·캘린더 탐색 상태가 콘텐츠와 불일치 #724 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• perf(ci): PR 검증 4개 job이 같은 debug variant 를 각자 컴파일 — build cache 가 전역 off #996 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(ci): 릴리스·WIF canary 배포가 같은 Firebase 앱에 동시 업로드 가능 — concurrency group 분리 #995 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• chore(ci): 실서비스 secret workflow 3종의 검증·decode 블록 3변종 중복 — 공유 composite 로 통합 #997 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 홈 TODAY'S QUESTION 카드 '데일리질문 답변하기' 무동작 — 카드 데이터는 정상 #627 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): data 계층 runCatching 이 CancellationException 을 Result 로 소비 — 취소 전파 보존 #670 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 작성 툴바 '임시저장' 카운트가 항상 0 — 두 작성 화면 모두 draftCount 미전달 #769 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 화면 배경이 시안 #FAFAFA 대신 Material3 기본 surface #FEF7FF 로 렌더 — 탭 간 배경 불일치 #691 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 요약 문구 공백 소실 — '이번 주,이름님은N일의…'로 붙어 표시 #732 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 데일리질문 목록 상단 '오늘의 추천 질문' 영역 미구현 — 조회한 todayQuestion 을 화면이 소비하지 않음 #592 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 홈 MEMORIES 카드·추억 공간 화면이 고정 더미 — 실데이터 미연동 #559 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 캘린더가 감정 이모티콘과 점을 함께 표시 — 시안·디자이너 정의는 배타적 #749 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 상단 카운트와 요약 기록일수 불일치 — diaryAmount 는 1인데 week[] 에 미포함 #590 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 임시저장 선택 삭제 실패가 무시되고 완료 처리됨 #442 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 일기 날짜 선택 제목이 '발송 날짜' — 시안은 '기록 날짜'로 수정됨 #760 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(core): presigned URL 요청에 필수 contentLength 가 빠져 이미지 업로드가 전부 실패 #950 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• refactor(mindrecord): DTO 기본값·불필요 nullable 제거 — 누락 계약을 정상값으로 은폐 #789 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 수신자 데일리질문 isDraft 를 파싱하지 않고 false 하드코딩 — draft 노출 방어가 무효 #956 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 주차 드롭다운이 5개로 끝나 시안의 추가 이력·스크롤 미노출 #729 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 데일리질문 생성·수정 응답의 userDailyQuestionId 를 BaseResponse<Unit> 으로 버림 #573 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(mindrecord): 주간리포트 데일리질문 날짜 파싱 실패가 오늘 날짜로 무음 폴백 — 틀린 날짜를 정상처럼 표시 #751 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• chore(ci): dependency-graph job correlator 불일치 — develop base PR 의 dependency-review 가 의존성 전량을 신규 도입으로 오탐 #1009 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• develop androidTest 컴파일 실패 44건 — 배치 머지 API 드리프트가 미검증 착륙, 컴파일 게이트 부재 #936 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• 로그아웃 시 로컬 저장소 3개가 지워지지 않음 — DataStore 수명(scope) 계약 도입 #912 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• chore(build): 빌드·테스트 클래스패스 bouncycastle 1.84 상향 — dependency-review critical(GHSA-574f-3g2m-x479) 해소 #921 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.bouncycastle:bcpkix-jdk18on 보안 권고 대응 필요 #983 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.bouncycastle:bcprov-jdk18on 보안 권고 대응 필요 #984 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• NavigationDebugLogger 삭제 — 역할 끝난 일회성 디버그 프로브 #927 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• core:startup 모듈 삭제 — Initializer 2개를 app 으로 이동 #929 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 중복 구현 1개로 #938 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): androidx-compose-material3 선언 버전과 Gradle 해석 버전을 일치시킨다 #973 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): coroutines-core 선언 버전과 Gradle 해석 버전을 일치시킨다 #974 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-codec 보안 권고 대응 필요 #975 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-codec-http 보안 권고 대응 필요 #976 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-codec-http2 보안 권고 대응 필요 #977 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-common 보안 권고 대응 필요 #978 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-handler 보안 권고 대응 필요 #979 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): io.netty:netty-handler-proxy 보안 권고 대응 필요 #980 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.apache.commons:commons-lang3 보안 권고 대응 필요 #981 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.bitbucket.b_c:jose4j 보안 권고 대응 필요 #982 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(build): org.jdom:jdom2 보안 권고 대응 필요 #985 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• core:di 모듈 삭제 — 바인딩을 구현 옆으로 옮기고 repository impl 을 internal 로 #930 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• AuthorReceiverRepository 가 동작하지 않음 — 에디터 수신자 목록이 항상 빈 리스트 #935 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• develop androidTest 컴파일 실패 21건 — #936 후속, 게이트 착륙 전에 갈린 도메인 계약 5건 #1022 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(ci): CodeQL 빌드 스텝이 :app:mergeExtDexDebug 에서 OOM — 산발적 실패로 보안 경고와 구분 불가 #1023 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• fix(ci): base 와 충돌한 PR 은 pull_request CI 가 트리거되지 않아 최신 커밋이 미검증으로 방치 — 충돌 자동 감지·라벨 필요 #1028 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• perf(ci): ktlint 를 PR 당 두 번 검사 — reviewdog 경로 32초·버전 이원화 정리 #1012 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인

이 판단은 배포를 자동 실행하지 않습니다. deploy일 때 Release Distribution을 수동 실행하세요.