feat: 전역 예외처리 및 공통 ApiResult 구현

[eric-jy-park]

📌 PR 제목

• [Common] 전역 예외처리 및 공통 ApiResult 구현 #25

📋 작업 내용

• 전역 예외 처리를 추가했습니다.
• 에러 코드들도 따로 관리할까 싶었지만,,, 일단은 이렇게 갈게요!

✅ 체크리스트

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

🚨 주의사항

• 이거 PR 먼저 올리고, 클로드 코드 리뷰 받고, 리뷰 반영한 다음에 공통 ApiResult까지 만들고 머지할게요!

[claude]

Claude finished @eric-jy-park's task —— View job

---

전역 예외처리 PR 코드 리뷰 완료

작업 체크리스트

• 전역 예외처리 PR 분석 및 체크리스트 생성
• 변경된 파일들 읽고 구현 내용 파악
• 버그, 보안 취약점, 성능 문제 검토
• 최종 코드 리뷰 결과 작성 및 제출

---

변경사항 요약

전역 예외처리 구조를 잘 설계하셨고, ErrorResponse DTO와 BusinessException 추상 클래스를 통해 일관된 에러 응답 구조를 구축했습니다. GlobalExceptionHandler에서 다양한 예외 상황을 체계적으로 처리하고 있으며, 각 예외마다 적절한 에러 코드와 메시지를 제공합니다. Swagger 문서에도 에러 응답 예시를 포함하여 API 문서화가 잘 되어 있습니다. 서버 포트를 8080에서 9090으로 변경하는 설정도 모든 관련 파일에 일관되게 적용되었습니다.

코드 리뷰 결과

GlobalExceptionHandler.java:17 - issue: 잘못된 AccessDeniedException import - java.nio.file.AccessDeniedException 대신 org.springframework.security.access.AccessDeniedException을 사용해야 웹 애플리케이션 보안 컨텍스트에 적합합니다.

GlobalExceptionHandler.java:147 - issue: IllegalArgumentException 메시지 직접 노출 - 내부 구현 세부사항이 클라이언트에게 노출될 수 있으므로 일반적인 메시지로 변경하거나 메시지 필터링 로직을 추가하세요.

OnboardingController.java:36 - issue: HTTP 상태 코드 불일치 - API 문서에는 responseCode "400"으로 되어 있지만 실제 UserOnboardingAlreadyCompleteException은 HttpStatus.CONFLICT(409)를 사용하므로 문서를 "409"로 수정하세요.

OnboardingController.java:80 - issue: HTTP 상태 코드 불일치 - 동일한 문제로 responseCode를 "409"로 수정하세요.

OnboardingController.java:122 - issue: HTTP 상태 코드 불일치 - 동일한 문제로 responseCode를 "409"로 수정하세요.

---

[claude]

Claude encountered an error —— View job

---

전역 예외처리 PR 코드 리뷰 진행 중

작업 체크리스트

• 변경된 파일들 읽고 구현 내용 파악
• 버그, 보안 취약점, 성능 문제 검토
• 이전 리뷰 지적사항 해결 여부 확인
• 최종 코드 리뷰 결과 작성

---

[claude]

Claude finished @eric-jy-park's task —— View job

---

전역 예외처리 PR 코드 리뷰 완료

작업 체크리스트

• 변경된 파일들 읽고 구현 내용 파악
• 이전 리뷰 지적사항 해결 여부 확인
• 버그, 보안 취약점, 성능 문제 검토
• 최종 코드 리뷰 결과 작성

---

변경사항 요약

전역 예외처리 구조가 체계적으로 잘 구현되었습니다. GlobalExceptionHandler에서 다양한 예외 상황을 일관되게 처리하고, ErrorResponse와 ApiResult DTO를 통해 통일된 응답 형식을 제공합니다. 이전 리뷰에서 지적된 AccessDeniedException import 문제는 해결되었으나, 일부 보안 취약점과 API 문서 불일치 문제가 여전히 남아있습니다. BusinessException 추상 클래스와 공통 ApiResult 구조는 확장성과 유지보수성 측면에서 훌륭한 설계입니다.

코드 리뷰 결과

GlobalExceptionHandler.java:148 - issue: IllegalArgumentException 메시지 직접 노출 - 내부 구현 세부사항이 클라이언트에게 노출될 수 있으므로 일반적인 메시지로 변경하거나 메시지 필터링 로직을 추가하세요.

OnboardingController.java:37 - issue: HTTP 상태 코드 불일치 - API 문서에는 responseCode "400"으로 되어 있지만 실제 UserOnboardingAlreadyCompleteException은 HttpStatus.CONFLICT(409)를 사용하므로 문서를 "409"로 수정하세요.

OnboardingController.java:80 - issue: HTTP 상태 코드 불일치 - 동일한 문제로 responseCode를 "409"로 수정하세요.

OnboardingController.java:123 - issue: HTTP 상태 코드 불일치 - 동일한 문제로 responseCode를 "409"로 수정하세요.

---