[#101] 리캡 status 지정과 AI 카테고리 추가

[mjk25]

작업 내용

• 리캡 생성 실패시 FAILED로 상태를 저장하고, API 응답에도 status를 함께 전달
• AI 카테고리 추가

참고

• env에 RECAP_IMAGE_BASE_URL 수정 예정입니다.

관련 이슈

• close 리캡 status 지정과 AI 카테고리 추가 #101

[github-actions]

🤖 Gemini Code Review

✅ 변경 사항 요약

• 리캡 생성 프로세스에 runCatching 기반의 예외 처리 및 FAILED 상태 기록 로직 추가.
• RecapImage 매핑 번호 변경 및 AI 카테고리 추가.
• RecapStatus 내 미사용 상태값(IN_PROGRESS, PENDING) 제거.

🚨 치명적인 이슈 (Critical Issues)

• 🛑 RecapImage.kt

  • Problem: 기존 DEVELOPMENT에 매핑되었던 11.png를 AI로 변경하고 이후 항목들의 파일 번호를 강제로 밀어냄. DB에 Enum 이름이 저장되어 있다면, 기존 사용자가 보유한 리캡의 이미지가 일괄적으로 변경되는 데이터 정합성 오류 발생.
  • Fix: 기존 매핑(11.png~20.png)은 유지하고, 신규 AI 항목은 마지막 번호(21.png)로 추가할 것.

• 🛑 RecapService.kt

  • Problem: runCatching 블록에서 예외 발생 시 에러 로그를 남기지 않음. AI API 호출 실패, 타임아웃, 파싱 에러 등 운영 환경에서 발생하는 장애의 원인 파악이 불가능함.
  • Fix: onFailure 블록 내에 log.error를 추가하여 예외 스택 트레이스를 반드시 기록할 것.

✏️ 파일별 상세 리뷰

RecapStatus.kt

• 📍 Issue: IN_PROGRESS, PENDING 상태 삭제로 인해, DB에 해당 값을 가진 과거 레코드가 남아있을 경우 조회 시 IllegalArgumentException 발생 위험이 있음.

RecapService.kt

• 📍 Issue: onFailure 내부에서 새 트랜잭션을 열어 FAILED 상태를 저장하고 다시 getOrThrow()를 호출함. 만약 상위 호출부에 트랜잭션이 걸려있다면 전파 설정에 따라 의도치 않은 롤백이나 상태 저장 실패가 발생할 수 있음.
• ✨ Improvement: FAILED 저장이 반드시 보장되어야 한다면 해당 저장 로직에 Propagation.REQUIRES_NEW 적용 검토 필요.

RecapImage.kt

• 📍 Issue: 리소스 매핑 파일 번호가 변경됨에 따라 클라이언트(UI)에서 이전과 다른 이미지를 노출하게 됨.
• ✨ Improvement: 도메인 순서보다 기존 데이터와의 하위 호환성을 우선하여 인덱스를 관리할 것.

[earlgrey02]

수고하셨습니다!