본문 이미지 fileKey 를 URL 에서 역산하지 않고 upload() 반환값을 쓴다 — #1017 반영 후속 #1125

📜 Overview (Required)

마음의 기록 작성 2곳이 본문 img src 에 실을 fileKey 를 업로드 URL 에서 문자열로 역산한다.

// feature/mindrecord/presentation/.../util/HtmlText.kt
fun String.toUploadedFileKey(): String =
    if (contains("://")) substringAfter("://").substringAfter('/') else this

fun String.toWireContent(uploadedImageUrls: Set<String>): String =
    uploadedImageUrls.fold(this) { content, url -> content.replace(url, url.toUploadedFileKey()) }

이 방식은 fileUrl == "<스킴>://<호스트>/" + fileKey 를 가정한다. CDN 이 경로 프리픽스를 붙이거나(https://cdn/bucket/mindrecords/…) 쿼리스트링이 붙으면 조용히 틀린 키가 나가고, 증상은 #549 가 고친 것과 같은 «저장은 됐는데 이미지가 안 뜬다» 다.

왜 지금까지 역산이었나 — 그리고 무엇이 바뀌나

역산은 몰라서 쓴 게 아니라 코어가 fileKey 를 버려서 쓴 우회다. HtmlText.kt 의 KDoc 이 그 사정을 적어 두고 있다.

권위 있는 출처는 presigned 응답의 fileKey 인데 PhotoUploadRepository.upload() 가 fileUrl 만 돌려주며 버린다 — CDN 이 경로 프리픽스나 쿼리스트링을 붙이면 조용히 틀린 키가 된다. 반환을 넓히는 것은 core:data 범위라 별건으로 둔다 (#1017).

그 별건이 #1017(PR #1094)이다. 머지되면 upload() 가 UploadedFile(fileUrl, fileKey) 를 돌려주므로 역산의 전제가 사라진다. 이 이슈는 그 뒤에 남는 우회를 걷어내는 후속이다.

무엇을 하면 되나

• 업로드 결과를 URL 집합이 아니라 URL → fileKey 대응으로 기억한다 (uploadedImageUrls: Set<String> → 맵).
• toWireContent 가 그 대응으로 치환하게 바꾼다. 서버가 준 키를 그대로 쓰므로 추측이 사라진다.
• 쓰는 곳이 없어지는 toUploadedFileKey() 와 그 단위 테스트(HtmlImageSrcTest)를 지운다.
• 이미 저장돼 본문에 들어 있는 영구 URL 은 계속 건드리지 않는다 — 이번 작성에서 받은 것만 치환하는 현재 규칙은 그대로다 (fix(mindrecord): 데일리질문 imageUrl 이 서버 계약에 없어 이미지 저장·조회 불가 — content HTML img 태그로 전환 #549·fix(mindrecord): 일기 작성의 본문 이미지가 fileKey 로 변환되지 않음 — #953 머지 시 깨진 이미지로 활성화 #1016 에서 같은 이유로 그렇게 했다).

소비처는 DailyQuestionWriteViewModel(데일리질문 #549)과 DiaryWriteViewModel(일기 #1016) 두 곳이다.

📌 Child Issue(Optional)

No response

📍 Note (Optional)

• refactor(core): PhotoUploadRepository.upload() 가 presigned 응답의 fileKey 를 버림 — 호출부가 URL 에서 역산 #1017 (PR core 업로드 반환에 fileKey 를 함께 싣는다 — 호출부의 URL 역산 제거 (closes 1017) [base: develop] #1094) 이 머지된 뒤에 착수한다 — 그 전에는 upload() 가 fileKey 를 주지 않아 이 작업 자체가 불가능하다.
• PR core 업로드 반환에 fileKey 를 함께 싣는다 — 호출부의 URL 역산 제거 (closes 1017) [base: develop] #1094 와 PR fix(mindrecord): 일기 본문 이미지도 fileKey 로 바꿔 보낸다 (#1016) #1120 은 DiaryWriteViewModel.kt 에서 충돌한다. 머지 순서를 정해 한쪽이 다른 쪽을 병합해야 한다.
• fix(mindrecord): 일기 작성의 본문 이미지가 fileKey 로 변환되지 않음 — #953 머지 시 깨진 이미지로 활성화 #1016 (PR fix(mindrecord): 일기 본문 이미지도 fileKey 로 바꿔 보낸다 (#1016) #1120) 이 두 ViewModel 의 변환 로직을 HtmlText.kt 로 공통화해 두었으므로, 이 정리도 그 한 곳에서 끝난다.