[Sadturtleman]

📌𝘐𝘴𝘴𝘶𝘦𝘴

Closes #573

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

이슈의 수정 방향대로 응답을 DTO 로 받고 계약을 Result<Long> 으로 올렸습니다.

API      BaseResponse<Unit>  →  BaseResponse<DailyQuestionAnswerResponseDto>
domain   Result<Unit>        →  Result<Long>
data     requireStatus()     →  requireData().userDailyQuestionId

임시저장 성공 시 그 값을 draftId 로 들어, 같은 화면에서 이어 저장할 때 목록을 뒤지지 않고 PATCH 합니다.

📷 서버 계약 실측 (2026-08-23)

이슈의 2026-07-28 실측이 지금도 유효한지 확인했습니다.

POST /api/v1/daily-questions
<-- {"userDailyQuestionId":19,"content":"응답 계약 실측","isDraft":true,"receivers":[]}

PATCH /api/v1/daily-questions/19
<-- {"userDailyQuestionId":19,"content":"응답 계약 실측 수정","isDraft":true,"receivers":[]}

두 응답이 같은 스키마라는 것도 그대로입니다.

📷 앱 실측 — update() 경로가 식별자를 씁니다

1) 작성 화면에서 임시저장       → draft 목록 [(20, '<p>draft-then-save</p>')]
2) 재진입 후 정식 저장          → draft 목록 []      정식 목록 [(20, …)]

새 레코드가 만들어지지 않고 20 번이 그대로 정식 전환됩니다 — update() 가 그 식별자로 PATCH 했다는 뜻입니다.

🧪 회귀 가드

✔ 생성 응답에서 내 답변 식별자를 읽는다
✔ 수정 응답도 같은 스키마다
✔ 식별자가 없으면 실패로 드러난다      ← 기본값 0 으로 메우지 않는다

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

• 검증: :feature:mindrecord:data:testDebugUnitTest + :feature:mindrecord:presentation:testDebugUnitTest + 양쪽 ktlintCheck + :app:assembleDebug BUILD SUCCESSFUL.
• 이슈 노트의 "questionId 와 userDailyQuestionId 가 같은 값" 은 이번 실측에서도 같았습니다(둘 다 19). 두 값을 섞어 쓰지 않도록 DTO KDoc 에 근거를 남겼습니다. 다른 계정·날짜에서 갈리는지는 확인하지 못했습니다.
• ⚠️ "같은 화면에서 이어 저장" 경로는 현재 앱 흐름에서 도달하지 않습니다. 임시저장에 성공하면 화면이 목록으로 빠져나가기 때문입니다(onWriteSubmitSuccess). draftId 대입은 그 흐름이 바뀌는 순간 바로 쓰이는 형태로 두었고, 지금 실제 이득은 재진입 시 update() 가 올바른 식별자로 PATCH 한다는 것입니다(위 실측).

[OnlyReadOneBook]

머지 전 1건 수정이 필요합니다. 계약·DTO·contract test 설계 방향은 좋습니다.

1. [blocking] 인터페이스 변경이 :app 테스트 fake 한 곳을 놓쳐 머지 직후 통합 빌드가 깨집니다.

   • 이 PR이 DailyQuestionRepository.create/update를 Result<Long>로 바꿨는데(DailyQuestionRepository.kt:28,34), app/src/test/java/com/afternote/afternote_fe/screen/HomeTabViewModelTest.kt:464-469의 FakeDailyQuestionRepository override는 여전히 Result<Unit>를 반환합니다. 이 파일은 이 PR에서 변경되지 않아 머지 후에도 그대로입니다.
   • 실측: PR head(ad74e7a)에서 :app:compileDebugUnitTestKotlin 실행 →

     e: HomeTabViewModelTest.kt:464:71 Return type of 'suspend fun create(payload: DailyQuestionCreatePayload): Result<Unit>' is not a subtype of the return type of the overridden member '... Result<Long>' ...
     e: HomeTabViewModelTest.kt:469:8 Return type of 'suspend fun update(id: Long, payload: DailyQuestionUpdatePayload): Result<Unit>' is not a subtype of ...
     

   • 현재 CI가 이걸 못 잡는 이유: "Run Unit Tests"가 Gradle 이전 단계인 develop 정책테스트(supply-chain-policy.test.mjs — lint.yml setup-java 캐싱 모순, 전 PR 공통)에서 죽어 Kotlin 컴파일에 도달하지 못합니다. 본문의 검증 목록(:feature:mindrecord:* 테스트 + :app:assembleDebug)도 :app 테스트 소스는 컴파일하지 않습니다.
   • 수정: 해당 fake의 create/update 반환 타입만 Result<Long>로 바꾸면 됩니다(둘 다 본문이 unexpected(...)라 시그니처만 수정). create(payload:) 구현은 저장소에 4곳뿐이고 나머지 3곳은 이미 갱신돼 있는 걸 확인했습니다.

2. [normal] 새 동작(draftId 보존)을 단언하는 테스트가 없습니다.

   • DailyQuestionWriteViewModel.kt:173의 draftId = if (isDraft) savedId else null이 이 PR의 목적인데, DailyQuestionWriteViewModelTest 5개 시나리오에는 create/update 성공 경로가 없습니다(fake의 19L 반환은 createCallCount == 0 단언에만 쓰임). 본문 스스로 이 경로가 현재 앱 흐름에서 도달 불가라고 밝힌 만큼 회귀를 잡을 수단은 테스트뿐입니다 — 임시저장 성공 후 draftId == savedId, 정식 저장 성공 후 draftId == null 단언 추가를 권합니다.

검증:

• PR head에서 :feature:mindrecord:data:testDebugUnitTest + :feature:mindrecord:presentation:testDebugUnitTest 로컬 통과, :app:compileDebugUnitTestKotlin 로컬 실패(위 원문)
• 이슈 #573 실측 스키마와 DTO·KDoc 대조, requireData()의 오류 경로(status!=200 → ApiException → Result.failure, data null 계약 위반도 예외) 확인 — 크래시 경로 없음
• contract test가 실측 응답 원문을 쓰고 프로덕션 Json과 동일한 ignoreUnknownKeys로 미선언 필드를 무시함을 확인, userDailyQuestionId 무기본값 설계라 누락 검증이 실질적
• create/update/getList 호출부 전수 추적(DraftListViewModel·GetHomeSummaryUseCase·DiaryWriteViewModel 파급 없음), 임시저장→onSubmitSuccess 화면 이탈 경로의 본문 서술 정확성 확인

[OnlyReadOneBook]

반영 커밋(9f88772f) 기준 재리뷰했습니다. [blocking] 이 처방대로 정확히 수정돼 승인합니다 — HomeTabViewModelTest의 FakeDailyQuestionRepository.create/update 반환 타입만 Result<Long>로 올렸고 본문은 그대로 unexpected(…) 입니다.

검증:

• PR head(9f88772, develop 머지 포함)에서 :app:testDebugUnitTest --tests '*HomeTabViewModelTest*' 로컬 통과 — 이전 리뷰에서 실패 원문을 남겼던 게이트(:app:compileDebugUnitTestKotlin 경유)가 green 입니다.
• exact-HEAD CI 전부 green. develop 에 #913(lint.yml 캐시 소유권) 머지 후로는 "Run Unit Tests"가 정책테스트 단계를 넘어 실제 Gradle 컴파일·테스트까지 도달해 성공합니다 — 이전 리뷰에서 "CI가 이걸 못 잡는다"고 했던 공백 자체가 develop 쪽에서 해소된 것도 확인했습니다.
• 실기동 QA·시안 대조 해당 없음: 이번 델타는 테스트 fake 시그니처 2줄이라 런타임 거동·UI 변화가 없습니다.

[normal](임시저장 성공 후 draftId == savedId, 정식 저장 후 null 단언 테스트)은 미반영이지만 비차단 제안으로 남깁니다 — 현재 도달 불가 경로의 회귀 방지 수단이 테스트뿐이라는 사정은 그대로라, 후속에서 붙일 가치는 여전합니다.

[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): 일기 날짜 선택 제목이 '발송 날짜' — 시안은 '기록 날짜'로 수정됨 #760, fix(mindrecord): 작성 툴바 '임시저장' 카운트가 항상 0 — 두 작성 화면 모두 draftCount 미전달 #769, refactor(mindrecord): DTO 기본값·불필요 nullable 제거 — 누락 계약을 정상값으로 은폐 #789, fix(core): presigned URL 요청에 필수 contentLength 가 빠져 이미지 업로드가 전부 실패 #950, fix(mindrecord): 수신자 데일리질문 isDraft 를 파싱하지 않고 false 하드코딩 — draft 노출 방어가 무효 #956, 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
• 판단 근거: 고위험 변경이 포함됐습니다: .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(mindrecord): 데일리질문 생성·수정 응답의 userDailyQuestionId 를 BaseResponse<Unit> 으로 버림 #573 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인
• 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 관련 동작을 재현하고 수정 후 기대 결과가 충족되는지 확인

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