🐛 로컬 draft→S3 promote가 DB 트랜잭션과 분리되어 orphan 파일/유실 위험

[si-zero]

🐛 버그 설명

배틀/옵션 이미지를 로컬 draft로 올려뒀다가 발행(PUBLISHED) 시점에 S3로 승격하는 로직(LocalDraftFileStorageService.promoteLocalDraftToS3)이, DB 저장을 담당하는 @Transactional 메서드(BattleServiceImpl.createBattle/updateBattle) 중간에서 즉시 S3 업로드 + 로컬 draft 삭제까지 동기적으로 수행하고 있었다.

🔍 원인

S3 업로드/로컬 파일 삭제는 DB 트랜잭션의 롤백 대상이 아니다. 따라서 승격 이후 같은 트랜잭션에서 다른 이유로 저장이 실패해 롤백되더라도:

• 이미 S3에 올라간 파일은 그대로 남아 orphan이 되고,
• 이미 지워진 로컬 draft는 복구할 수 없어 사용자가 재시도할 때 참조할 원본이 사라진다.

[[s3_railway_bucket_migration]] 작업 중 발견해서 배틀 발행 자동화 때 같이 고치기로 미뤄뒀던 기존 버그.

✅ 해결

• LocalDraftFileStorageService를 resolveS3Key(목적지 key 계산만, 부수효과 없음)와 promoteToS3(실제 업로드+로컬 삭제)로 분리
• BattleServiceImpl에서 목적지 key는 트랜잭션 중 동기적으로 계산해 엔티티에 저장하되, 실제 승격(promoteToS3)은 기존에 이미 쓰이던 TransactionSynchronizationManager afterCommit 패턴으로 커밋 이후로 미룸 — 롤백 시에는 아예 호출되지 않아 draft가 그대로 남는다.