feat: 유저 도전기록 등록/조회 API

[1117mg]

📌 PR 제목

• [Route Mission] 유저의 루트 미션 도전 영상 등록, 조회 #12

📋 작업 내용

• feat: 유저 도전기록 등록/조회 API
• 업로드 상태별 관리 (실패 포함)

✅ 체크리스트

• 테스트를 완료했나요?
• 코드 컨벤션을 지켰나요?
• 관련 문서를 업데이트 했나요?

[claude]

Claude finished @1117mg's task —— View job

---

코드 리뷰 완료 ✅

Todo List:

• 변경된 파일들 분석
• 코드 리뷰 수행 (버그, 보안, 성능, 불명확한 로직 중심)
• 리뷰 결과 작성

변경 사항 요약

이번 PR은 유저 도전기록 등록/조회 API와 업로드 상태별 관리 기능을 추가합니다. AttemptStatus enum과 관련 응답 DTO들을 추가하여 도전기록의 업로드 상태를 추적할 수 있게 되었습니다. 주요 기능으로는 세션별 도전기록 조회, 특정 도전 상태 조회, 미완료 도전기록 목록 조회가 포함됩니다. Repository에서는 복잡한 JOIN 쿼리를 통해 필요한 데이터를 효율적으로 조회하도록 구현되었습니다.

코드 리뷰

AttemptServiceImpl.java:40 - issue: ZeroDivisionException 위험성 - upload.getChunkLength()가 0일 때 나눗셈 연산에서 예외 발생 가능, 사전에 0 체크 후 기본값 반환하도록 수정 필요

AttemptServiceImpl.java:284-288 - suggestion: 트랜잭션 내 예외 처리 문제 - IOException 발생 시 uploadSession 상태를 FAILED로 변경하지만 바로 예외를 던져 롤백될 수 있음, 별도 트랜잭션으로 분리하거나 상태 변경 로직 재검토 필요

UserMissionAttemptRepository.java:33-55 - question: N+1 쿼리 발생 가능성 - 복잡한 JOIN 쿼리에서 SessionAttemptDetail 생성자 파라미터 순서와 실제 필드 매핑 검증 필요, 특히 gl.imageUrl과 m.difficulty 매핑 확인 필요

---

[claude]

Claude finished @1117mg's task —— View job

---

코드 리뷰 완료 ✅

Todo List:

• 변경된 파일들 분석
• 코드 리뷰 수행 (버그, 보안, 성능, 불명확한 로직 중심)
• 리뷰 결과 작성

변경 사항 요약

이번 PR은 유저 도전기록 등록/조회 API와 업로드 상태별 관리 기능을 추가합니다. AttemptStatus enum과 관련 응답 DTO들을 추가하여 도전기록의 업로드 상태를 추적할 수 있게 되었습니다. 주요 기능으로는 세션별 도전기록 조회, 특정 도전 상태 조회, 미완료 도전기록 목록 조회가 포함됩니다. Repository에서는 복잡한 JOIN 쿼리를 통해 필요한 데이터를 효율적으로 조회하도록 구현되었습니다.

코드 리뷰

AttemptStatusResponse.java:40 - issue: ZeroDivisionException 위험성 - upload.getChunkLength()가 0일 때 나눗셈 연산에서 예외 발생 가능, 사전에 0 체크 후 기본값 반환하도록 수정 필요

AttemptServiceImpl.java:282-285 - suggestion: 트랜잭션 내 예외 처리 문제 - IOException 발생 시 uploadSession 상태를 FAILED로 변경하지만 바로 예외를 던져 롤백될 수 있음, 별도 트랜잭션으로 분리하거나 상태 변경 로직 재검토 필요

UserMissionAttemptRepository.java:33-55 - question: 복잡한 JPQL 쿼리 검증 필요 - SessionAttemptDetail 생성자 파라미터와 실제 SELECT 절의 필드 매핑 순서 확인 필요, 특히 gl.imageUrl과 m.difficulty 순서 불일치 가능성

---

[claude]

Claude finished @1117mg's task —— View job

---

코드 리뷰 완료 ✅

Todo List:

• 변경된 파일들 분석
• 코드 리뷰 수행 (버그, 보안, 성능, 불명확한 로직 중심)
• 리뷰 결과 작성

변경 사항 요약

이번 PR은 유저 도전기록 등록/조회 API와 업로드 상태별 관리 기능을 추가합니다. AttemptStatus enum과 관련 응답 DTO들을 추가하여 도전기록의 업로드 상태를 추적할 수 있게 되었습니다. 주요 기능으로는 세션별 도전기록 조회, 특정 도전 상태 조회, 미완료 도전기록 목록 조회가 포함됩니다. Repository에서는 복잡한 JOIN 쿼리를 통해 필요한 데이터를 효율적으로 조회하도록 구현되었습니다.

코드 리뷰

AttemptStatusResponse.java:40 - issue: ZeroDivisionException 위험성 - upload.getChunkLength()가 0일 때 나눗셈 연산에서 예외 발생 가능, 사전에 0 체크 후 기본값 반환하도록 수정 필요

AttemptServiceImpl.java:282-285 - suggestion: 트랜잭션 내 예외 처리 문제 - IOException 발생 시 uploadSession 상태를 FAILED로 변경하지만 바로 예외를 던져 롤백될 수 있음, 별도 트랜잭션으로 분리하거나 상태 변경 로직 재검토 필요

UserMissionAttemptRepository.java:33-55 - question: 복잡한 JPQL 쿼리 검증 필요 - SessionAttemptDetail 생성자 파라미터와 실제 SELECT 절의 필드 매핑 순서 확인 필요, 특히 gl.imageUrl과 m.difficulty 순서 불일치 가능성

---

[claude]

Claude finished @1117mg's task —— View job

---

코드 리뷰 완료 ✅

Todo List:

• 변경된 파일들 분석
• 코드 리뷰 수행 (버그, 보안, 성능, 불명확한 로직 중심)
• 리뷰 결과 작성

변경 사항 요약

이번 PR은 유저 도전기록 등록/조회 API와 업로드 상태별 관리 기능을 추가합니다. AttemptStatus enum과 관련 응답 DTO들을 추가하여 도전기록의 업로드 상태를 추적할 수 있게 되었습니다. 주요 기능으로는 세션별 도전기록 조회, 특정 도전 상태 조회, 미완료 도전기록 목록 조회가 포함됩니다. Repository에서는 복잡한 JOIN 쿼리를 통해 필요한 데이터를 효율적으로 조회하도록 구현되었습니다.

코드 리뷰

AttemptStatusResponse.java:40 - issue: ZeroDivisionException 위험성 - upload.getChunkLength()가 0일 때 나눗셈 연산에서 예외 발생 가능, 사전에 0 체크 후 기본값 반환하도록 수정 필요

AttemptServiceImpl.java:294-299 - suggestion: 트랜잭션 내 예외 처리 문제 - IOException 발생 시 uploadSession 상태를 FAILED로 변경하지만 바로 예외를 던져 롤백될 수 있음, 별도 트랜잭션으로 분리하거나 상태 변경 로직 재검토 필요

UserMissionAttemptRepository.java:33-55 - question: 복잡한 JPQL 쿼리 검증 필요 - SessionAttemptDetail 생성자 파라미터와 실제 SELECT 절의 필드 매핑 순서 확인 필요, 특히 gl.imageUrl과 m.difficulty 순서 불일치 가능성

---