refactor(afternote): 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 구현 1개로 (closes 938) [base: develop]

[OnlyReadOneBook]

Closes #938

추모 미디어 업로드 경로가 사진·영상으로 타입 이름만 다른 동일한 코드 두 벌로 갈라져 있었고, 각 *UploadOutcome 의 3상태 중 Existing · FreshlyUploaded 는 유일한 소비처에서 같은 처리를 받았다. #258 이 BE 실코드로 영상 URL 영구성을 확정해 PATCH 전용 분기가 사라진 뒤 구분이 무의미해졌는데 타입만 남아 있었다.

변경

항목	내용
신설	MediaKind enum · MemorialMediaUploadRepository(resolve(input, kind): Result<String?>) · MemorialMediaUploadRepositoryImpl
삭제	PhotoUploadOutcome · VideoUploadOutcome · 인터페이스 2개 · Impl 2개
수정	UseCase 의 urlOrNull() 2개 제거 · DI 바인딩 2 → 1 · 낡은 KDoc 정리

• Result<String?> 로 대체한 근거 — Empty + url 을 가진 두 상태는 String? 과 동형이고, 실패는 Result 가 이미 표현한다. 두 축(성공/실패, 값 있음/없음)이 직교하므로 성공 축에 sealed 계층이 필요하지 않다.
• DIRECTORY_AFTERNOTES 이전 — 삭제되는 MemorialPhotoUploadRepositoryImpl.kt 에 있던 internal const 다. 썸네일 구현이 계속 참조하므로 통합 Impl 로 옮겼다.
• 입력단 MediaInput 3상태는 유지 — Local 은 업로드하고 Remote 는 통과시켜 동작이 실제로 갈린다. 출력단만 갈리지 않았다.
• UseCase KDoc 의 BE 계약 서술 제거 — S3 URL 영구성 · POST/PATCH 동일 규칙 설명을 지웠다. 정본은 Swagger·BE 이고 FE 에 복제하면 낡아 거짓이 된다. 근거는 커밋 메시지에 남겼다.

리뷰어에게

이슈 작업 목록에 없던 androidTest 3종을 함께 이관했다 — AfternoteAuthorAndroidTest · AfternoteAuthorImplementedCoverageAndroidTest · AfternoteAuthorExtendedAndroidTest. 이 파일들은 이슈가 기준으로 삼은 커밋 104462fd 에는 없었고 그 뒤 develop 에 들어왔다. 지금은 androidTest 가 PR 게이트 밖이지만 #949 가 :app:compileDebugAndroidTestKotlin 을 필수 게이트에 넣으므로, 빼놓으면 그 시점에 깨진다.

후속(이 PR 범위 밖) — 미디어 삭제 신호가 이 타입에 없다. 미첨부와 삭제가 둘 다 None → null → 필드 생략으로 뭉개진다. BE AfternotePlaylist.update() 가 null 가드를 두어 생략된 필드를 유지하므로 현재 계약으로는 지울 수단 자체가 없다. 편집 화면에 제거 UI 가 없어 도달 불가 경로이고 Outcome.Empty 시절과 동일한 상태이므로 그대로 뒀다. 삭제 기능을 붙일 때 BE 계약과 함께 정할 일이다.

검증

• :feature:afternote:domain:test — 6/6 통과 (ResolveMemorialMediaForSaveUseCaseTest, Fake 2개 → kind 별 호출을 기록하는 1개로 재작성. short-circuit·cause 보존 단언 유지)
• :feature:afternote:data:compileDebugKotlin · :app:compileDebugUnitTestKotlin — 통과
• :app:compileDebugAndroidTestKotlin — 실패하나 44건 전부 develop 기존 드리프트와 완전 일치. develop 을 별도 worktree 로 뽑아 같은 태스크를 돌린 뒤 에러 집합(줄번호 제거)을 대조해 확인했다. 본 변경이 만든 새 에러 0건, 없앤 것도 0건 — androidTest 도메인 API 드리프트 44건+런타임 7건 수정, PR 컴파일 게이트 추가 #949 가 고치는 그 44건이다.
• ktlintFormat 통과, pre-commit ktlint 훅 통과
• 시각 검증 skip — 브랜치 diff 에 Compose 본문·디자인 토큰·리소스 변경이 0건이다(domain·data·androidTest 만). screenshotTest 커버를 지나지 않아 baseline 사이클 불필요.

[github-actions]

🚀 QA 배포 권장

• 위험도: high
• 포함 이슈: fix(mindrecord): 임시저장 선택 삭제 실패가 무시되고 완료 처리됨 #442, 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, develop androidTest 컴파일 실패 44건 — 배치 머지 API 드리프트가 미검증 착륙, 컴파일 게이트 부재 #936, 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 중복 구현 1개로 #938, fix(core): presigned URL 요청에 필수 contentLength 가 빠져 이미지 업로드가 전부 실패 #950, fix(mindrecord): 수신자 데일리질문 isDraft 를 파싱하지 않고 false 하드코딩 — draft 노출 방어가 무효 #956, fix(build): org.bouncycastle:bcpkix-jdk18on 보안 권고 대응 필요 #983, fix(build): org.bouncycastle:bcprov-jdk18on 보안 권고 대응 필요 #984, 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
• 판단 근거: 고위험 변경이 포함됐습니다: .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 포인트

• 추모 미디어 업로드 경로 통합 — Outcome 타입 제거, 사진·영상 중복 구현 1개로 #938 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• 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 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인

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