기존 공통 응답 제거 후 RESTful 응답 구조로 변경 + 소나 클라우드 에러 해결

[zzaekkii]

close #12

☑️ 완료 태스크

• ApiResponse 삭제
• 에러 응답 구조에서 data 필드 삭제
• 소나클라우드 에러 해결 👈 24일부터 잡고 있던 에러인데, 드디어 해결!

🔎 PR 내용

issue 만들 때조차 Spring에 오염된 뇌로 ResponseEntity.ok()같은 구조를 떠올리면서,
"흠.. ok()랑 no_content()랑 이것저것 추가해주면 되겠군" 거리고 있었는데요.

💩 ApiResponse 자체가 필요하지 않음

생각해보니까 ApiResponse 자체가 필요가 없었다는 걸 뒤늦게 떠올렸습니다.
그럼 ApiResponse가 기존에 왜 필요했냐?

• 이전에 Moyoy 프로젝트하면서 프론트랑 응답 구조 맞추다가, 프론트에서 다루기 쉬운 구조라는 걸 알게 됨
• 프로젝트 초기에 svelte로 내가 프론트 다 만들 계획이었음
• 나 프론트 처음인 초보라 다루기 쉬운 구조 필요

근데? 서버 만드는 데 더 기력을 쏟기로 마음을 바꾸면서 (과제 마감 기한도 촉박했음)
프론트는 바이브 코딩으로 전략을 바꿨고 해당 사항이 없어지게 된 거죠.

💀 ReponseEntity? Axum엔 그런 거 없음!

Spring에 오염된 채, 사고를 안 하고 그냥 손가락을 움직인 셈이었는데요.
Axum에서는 IntoResponse 트레잇만 구현하면 HTTP 응답이 자동으로 만들어지는 방식입니다.

그래서 응답 데이터 반환 타입을 IntoResponse만 구현하면, 만사 OK인 거죠.
ResponseEntity.ok()라든가, ResponseEntity.nocontent() 이런 게 그냥 전혀 필요가 없다는 겁니다.

그러면 반환할 때 어떻게 해야 되냐,
👉 그냥 IntoResponse trait 구현된 데이터만 반환 '딸깍'하면 끝입니다.

그래서 기존에 불필요하게 status와 커스텀 code, message를 보내고 있던 걸 모두 삭제했습니다.
얘네들은 에러에서나 필요하지 정상 응답에서는 아무 필요도 의미도 없이 자리만 차지하기 때문이죠.

1. ApiResponse 구조체 삭제 ❌

성공 응답을 { status, code, message, data }로 통일해서 wrapper로 감싸던 기존 방식 제거
성공 응답은 이제 HTTP status + 실제 데이터만 반환

- 조회: Ok(Json(data))
- 생성: Ok((StatusCode::CREATED, Json(data)))
- 수정/삭제: Ok(StatusCode::NO_CONTENT) (body 없음)

2. 에러 응답 구조에서 data: {} 삭제 ❌

에러는 이제 HTTP status + { code, message } 구조로 단순화
에러 body에서 data: {} 제거

impl<E: ErrorCode> IntoResponse for AppError<E> {
    fn into_response(self) -> Response {
        let reason = self.code.reason();

        let body = Json(json!({
            "code": reason.code,
            "message": reason.message,
        }));

        (StatusCode::from_u16(reason.status).unwrap(), body).into_response()
    }
}

[sonarqubecloud]

Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud