feat(be) : tool, test 작성

[jungdongha]

close #80

[github-actions]

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

🟢 좋은점:

• Kotlin 최적화 측면에서 null safety가 잘 지켜짐 (region 파라미터는 required=true로 명시되어 null 불가능, guides 리스트 처리 시 isEmpty()로 안전하게 확인).
• ktlint 규칙 준수: 코드 포맷팅이 일관되며, 네이밍 컨벤션 (e.g., findGuidesByRegion, camelCase)이 적절함. 확장 함수 (e.g., result.take(200))를 활용해 간결하게 구현.
• Tool 어노테이션 (@tool, @ToolParam)이 명확히 사용되어 AI 도구로서의 역할을 잘 정의함. 로깅이 상세하고 (info/error 구분), 디버깅에 유용함.

🟡 개선사항:

• Kotlin 최적화: try-catch 블록 내 if-return 구조가 약간 중첩되어 가독성이 떨어짐. when 표현식을 사용해 guides.isEmpty() 케이스를 더 명확히 분기하거나, try 블록을 objectMapper.writeValueAsString() 부분으로 한정하고 빈 리스트 처리를 try 밖으로 분리하는 게 좋음 (e.g., if (guides.isEmpty()) return "..." else return try { objectMapper... } catch { ... }).
• 글로벌 익셉션 처리: Tool 함수 내에서 직접 예외를 잡고 있지만, @ControllerAdvice 기반의 표준 에러 응답 (e.g., ErrorResponse 객체)으로 통합되지 않음. AI 챗 컨텍스트에서 이 Tool이 호출될 상위 레이어 (e.g., Controller)에서 글로벌 핸들링을 보장하도록 문서화하거나, 예외 시 ApiResponse 형태로 반환을 고려.
• ApiResponse 사용: 이 코드는 API 엔드포인트가 아닌 Tool 함수이므로 ApiResponse 래핑이 필수가 아닐 수 있으나, JSON 변환 결과 (objectMapper.writeValueAsString)를 ApiResponse<List>로 감싸 반환하면 일관된 응답 형식을 유지할 수 있음 (빈 리스트 시 ApiResponse.empty() 등 활용).
• ktlint 규칙: log.info에서 result.take(200) + "..."는 좋지만, 긴 문자열 로그를 피하기 위해 구조화된 로깅 (e.g., log.info

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/domain/user/repository/UserRepository.kt

🟢 좋은점:

• Kotlin 최적화 측면에서 메서드 시그니처가 간결하고 null safety를 고려한 기본 타입(String, UserRole)을 사용하며, Spring Data JPA의 쿼리 메서드 네이밍 컨벤션(findBy + 필드 + 연산자)을 잘 따름. List 반환 타입도 컬렉션 쿼리에 적합.
• ktlint 규칙 준수: 포맷팅(들여쓰기, 공백)이 일관되며, 네이밍(파라미터명 role, location)이 직관적이고 camelCase를 따름.

🟡 개선사항:

• Kotlin 최적화: 만약 UserRole이 enum이라면 이미 최적화된 상태지만, 메서드에 @query 어노테이션을 추가해 명시적 JPQL을 사용하면 성능이나 복잡한 조건(예: 대소문자 무시 LIKE)에서 더 유연할 수 있음. 예: @Query("SELECT u FROM User u WHERE u.role = :role AND LOWER(u.location) LIKE LOWER(CONCAT('%', :location, '%'))").
• ktlint 규칙: 현재는 문제없으나, 긴 메서드 이름이므로 IDE나 빌드에서 자동 포맷팅 확인 추천.

🔴 문제점:

• 없음. 변경사항이 Repository 인터페이스에 국한되어 글로벌 익셉션 처리나 ApiResponse와 무관하며, 기본 규칙을 위반하지 않음.

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/domain/user/service/GuideService.kt

🟢 좋은점:

• @transactional(readOnly = true)을 사용해 읽기 전용 트랜잭션을 명시적으로 적용하여 불필요한 쓰기 잠금을 피하고 성능을 최적화한 점이 좋습니다.
• Kotlin의 map 함수를 활용해 리스트 변환을 간결하게 처리하며, null safety 측면에서 region 파라미터가 기본적으로 non-null String으로 처리되어 안전합니다.
• 네이밍 컨벤션(ktlint 준수): 메서드명 findGuidesByRegion이 명확하고 camelCase를 따르며, 코드 포맷팅이 깔끔합니다.
• GuideResponse.from(it) 확장 함수(또는 static 메서드)를 재사용하여 DRY 원칙을 지켰습니다.

🟡 개선사항:

• 글로벌 익셉션 처리 측면에서, region이 빈 문자열이나 null(호출 시 가능성)이 들어올 경우 Repository 쿼리가 예상치 못한 결과를 반환할 수 있으니, 서비스 레이어에서 입력 유효성 검사를 추가하는 것을 고려하세요. 예: if (region.isBlank()) throw IllegalArgumentException("Region cannot be blank"). 이는 @ControllerAdvice에서 표준 에러 응답으로 처리될 수 있습니다.
• Kotlin 최적화: findByRoleAndLocationContains의 반환 타입이 List로 가정되는데, UserRole.GUIDE 필터링이 Repository에서 이미 처리되므로 좋지만, 필요 시 when 표현식을 사용해 Role 검증을 추가로 강화할 수 있습니다 (현재는 불필요하지만 확장성 위해).
• ApiResponse는 컨트롤러 레이어에서 적용되는 규칙이므로 서비스에서는 무관하지만, 반환 타입을 List 대신 Single<ApiResponse<List>>으로 변경하지 말고 유지하세요 (서비스는 DTO만 반환하는 게 적합).

🔴 문제점:

• 없음. 추가된 메서드가 간단하고 규칙을 위반하지 않습니다.

[YangHJ2415]

확인 했습니다.