feat(be): dto 들을 캡슐화 시킴

[Mrbaeksang]

No description provided.

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/domain/ai/aiChat/controller/AiChatController.kt

🟢 좋은점:

• Kotlin 최적화: 기존의 !! (non-null assertion) 연산자를 제거하고 SessionsResponse.from(it)으로 대체하여 null safety를 강화했습니다. 이는 Kotlin의 안전한 객체 변환(확장 함수나 팩토리 메서드 활용)을 통해 런타임 오류를 방지하는 우수한 접근입니다. data class나 확장 함수를 활용한 것으로 보아 코드의 가독성과 유지보수성을 높였습니다.
• ApiResponse 사용: 변경 전후로 모든 API 응답이 ResponseEntity<ApiResponse<T>>로 일관되게 감싸여 있으며, 성공 메시지도 표준적으로 유지되어 규칙을 잘 준수합니다.
• ktlint 규칙: 코드 포맷팅이 깔끔하며, 네이밍 컨벤션(예: from 메서드 이름)이 자연스럽습니다. 변경으로 인한 불필요한 줄바꿈이나 들여쓰기 이슈가 없습니다.

🟡 개선사항:

• from() 메서드가 내부적으로 null 체크를 어떻게 처리하는지 (예: Optional이나 safe call 사용) 확인해보세요. 만약 SessionsResponse가 data class라면, 생성자에서 기본값을 제공하는 방식으로 더 명시적으로 null을 다루는 것을 고려할 수 있습니다. 이는 추가적인 안전성을 더할 수 있지만, 현재 변경만으로는 큰 문제가 아닙니다.

🔴 문제점:

• 없음. 변경이 필수 규칙을 모두 준수하며, 글로벌 익셉션 처리(@ControllerAdvice)는 컨트롤러 수준에서 직접 보이지 않으나 전체 프로젝트에서 적용된 것으로 가정합니다.

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/domain/ai/aiChat/dto/AiChatResponse.kt

🟢 좋은점:

• Kotlin 최적화 측면에서 data class를 적절히 사용하며, companion object를 통해 팩토리 메서드(from)를 제공하여 엔티티(AiChatMessage)로부터 DTO 변환을 간결하게 처리함. 이는 불필요한 boilerplate 코드를 줄이고, Kotlin의 관용적인 패턴을 잘 따름.
• ktlint 규칙 준수: 코드 포맷팅(들여쓰기, 줄 바꿈)이 일관되며, 네이밍 컨벤션(camelCase for properties and methods)이 적절함. import 문도 필요한 부분만 추가되어 깔끔함.

🟡 개선사항:

• Kotlin 최적화 측면에서 from 메서드의 파라미터 타입(AiChatMessage)이 null safety를 보장하도록 content 속성이 non-null(String)으로 가정되지만, 만약 엔티티의 content가 nullable할 수 있다면 requireNotNull이나 Elvis 연산자(?:)를 추가해 명시적으로 처리하는 것이 더 안전할 수 있음 (e.g., userMessage.content ?: throw IllegalArgumentException("User message content cannot be null")).
• from 메서드의 파라미터 이름(userMessage, aiMessage)이 반환 타입의 프로퍼티 이름과 동일해 약간의 혼동을 줄 수 있으니, 더 구체적으로 fromUserMessage와 fromAiMessage로 변경 고려 (ktlint 네이밍 관점에서 선택적).

🔴 문제점:

• 없음. 변경사항이 DTO의 확장을 위한 안전하고 논리적인 수정으로 보이며, 필수 검토 항목(글로벌 익셉션 처리, ApiResponse 사용)과 직접 관련되지 않음.

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/domain/ai/aiChat/dto/SessionMessagesResponse.kt

🟢 좋은점:

• Kotlin 최적화 측면에서 data class를 적절히 사용하며, companion object를 통해 entity(AiChatMessage)로부터 DTO로의 변환을 위한 팩토리 메서드(from)를 구현하여 코드의 가독성과 재사용성을 높였습니다. 이는 Kotlin의 idiomatic한 패턴(확장함수나 companion object 활용)을 잘 따릅니다.
• ktlint 규칙 준수: 코드 포맷팅이 일관되며, 네이밍 컨벤션(예: camelCase, 명확한 메서드명)이 적절합니다. 불필요한 공백이나 들여쓰기 오류가 없습니다.

🟡 개선사항:

• Kotlin 최적화 측면에서 from 메서드의 매개변수(sessionMessage)를 non-null로 가정하고 있지만, entity의 content나 senderType이 null 가능하다면 null safety를 강화하기 위해 ? 타입이나 Elvis 연산자(예: sessionMessage.content ?: "")를 고려할 수 있습니다. 현재는 안전하지만, API 응답(ApiResponse)에서 일관성을 위해 DTO의 불변성을 더 명확히 할 수 있습니다.
• ktlint 규칙 측면에서 companion object 블록의 중괄호 위치가 표준이지만, 팀 스타일 가이드에 따라 trailing comma(예: return 문 뒤)를 추가하면 더 깔끔해질 수 있습니다.

🔴 문제점:

• 없음. 변경사항이 간단하고 규칙을 위반하지 않습니다. 글로벌 익셉션 처리나 ApiResponse 사용은 이 DTO 파일과 직접 관련이 없어 검토 대상에서 제외됩니다.