Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughgRPC 및 protobuf 관련 의존성 추가와 버전 관리가 이루어졌으며, casper-feed 모듈의 Gradle 빌드 스크립트와 proto 파일, gRPC 클라이언트 및 응답 DTO, 통합 테스트 클래스가 새로 도입되었습니다. 또한 detekt 정적 분석 규칙 일부가 비활성화되었고, Kotlin 버전이 다운그레이드되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant AdminGrpcClient
participant gRPC_Server
Client->>AdminGrpcClient: getAdminInfoByAdminId(adminId)
AdminGrpcClient->>gRPC_Server: GetAdminByUUID(admin_id)
gRPC_Server-->>AdminGrpcClient: GetAdminIdResponse(admin_id)
AdminGrpcClient-->>Client: InternalAdminResponse(id)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15-20 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 7
🔭 Outside diff range comments (1)
buildSrc/src/main/kotlin/DependencyVersion.kt (1)
2-2: Kotlin 버전 상수 불일치
앞서 지적한PluginVersion와 값이 달라 빌드 파이프라인에서 혼용될 가능성이 있습니다. 한쪽 값을 수정하거나, 단일 소스-오브-트루스만 남기는 리팩터를 제안드립니다.
🧹 Nitpick comments (4)
buildSrc/src/main/kotlin/DependencyVersion.kt (1)
15-17: 상수명 오타 및 중복 버전 상수
GOGGLE_PROTOBUF는 오타로 보이며, 이미PROTOBUF(3.25.3) 상수가 존재합니다. 중복 정의는 혼란을 유발하므로 다음과 같이 정리하는 편이 좋습니다.-const val GOGGLE_PROTOBUF = "3.25.3" +// 중복 제거: 필요 시 PROTOBUF 상수만 사용하거나 +const val GOOGLE_PROTOBUF = "3.25.3"detekt.yml (1)
14-15: MagicNumber 규칙 전면 해제
완전 비활성화보다는ignoreNumbers목록 확장, 테스트 소스 전용 disable 등 세분화 설정을 권장합니다. 매직 넘버 유입 시 유지보수 난이도가 급격히 높아질 수 있습니다.casper-feed/src/test/kotlin/hs/kr/entrydsm/feed/CasperFeedApplicationTests.kt (1)
22-25: 의미 있는 검증 로직 추가 제안
contextLoads()가 빈 메서드인 경우 실패 조건이 애매합니다. 간단히 주요 Bean 존재 여부를 확인하면 테스트 신뢰도가 상승합니다.@Autowired lateinit var adminGrpcClient: AdminGrpcClient @Test fun contextLoads() { assertNotNull(adminGrpcClient) }casper-feed/build.gradle.kts (1)
88-88: 하드코딩된 버전 대신 상수를 사용해주세요.Spring Kafka 의존성에서 버전이 하드코딩되어 있습니다. 일관성을 위해 Dependencies 상수를 사용하는 것이 좋습니다.
다음과 같이 수정하는 것을 권장합니다:
- implementation("org.springframework.kafka:spring-kafka:3.1.2") + // 위의 Dependencies.KAFKA가 이미 선언되어 있으므로 이 줄은 제거하거나 버전 상수를 사용하세요
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
📒 Files selected for processing (9)
buildSrc/src/main/kotlin/Dependencies.kt(1 hunks)buildSrc/src/main/kotlin/DependencyVersion.kt(1 hunks)buildSrc/src/main/kotlin/PluginVersion.kt(1 hunks)casper-feed/build.gradle.kts(1 hunks)casper-feed/src/main/kotlin/hs/kr/entrydsm/feed/infrastructure/grpc/client/AdminGrpcClient.kt(1 hunks)casper-feed/src/main/kotlin/hs/kr/entrydsm/feed/infrastructure/grpc/client/dto/response/InternalAdminResponse.kt(1 hunks)casper-feed/src/main/proto/admin.proto(1 hunks)casper-feed/src/test/kotlin/hs/kr/entrydsm/feed/CasperFeedApplicationTests.kt(1 hunks)detekt.yml(2 hunks)
🧰 Additional context used
🪛 Buf (1.55.1)
casper-feed/src/main/proto/admin.proto
3-3: Files with package "casper.feed" must be within a directory "casper/feed" relative to root but were in directory "casper-feed/src/main/proto".
(PACKAGE_DIRECTORY_MATCH)
🔇 Additional comments (4)
casper-feed/src/main/kotlin/hs/kr/entrydsm/feed/infrastructure/grpc/client/dto/response/InternalAdminResponse.kt (1)
10-12: LGTM – 간결하고 명확한 DTO
UUID 필드만 노출하여 도메인 경량화를 잘 수행하셨습니다. 추후 확장(예: 이름·권한) 시에도 가변성 최소화를 위해 별도 DTO 추가를 고려해 주세요.detekt.yml (1)
4-7: LongParameterList 규칙 비활성화 사유 확인 필요
gRPC stub 생성 코드 등에서 매개변수 길이가 길어질 수 있지만, 전역적으로 규칙을 끄면 핵심 비즈니스 코드 품질이 저하될 수 있습니다. 모듈/패키지 단위 제외 또는@Suppress로 국소화하는 방안을 검토해 주세요.buildSrc/src/main/kotlin/Dependencies.kt (1)
41-42: gRPC Spring Boot starter 의존성이 올바르게 추가되었습니다.gRPC 서버 및 클라이언트를 위한 Spring Boot starter 의존성이 적절히 정의되었습니다.
casper-feed/build.gradle.kts (1)
94-116: Protobuf 설정이 올바르게 구성되었습니다.gRPC와 gRPC Kotlin 플러그인이 적절히 설정되어 코드 생성이 제대로 작동할 것입니다.
casper-feed/src/main/kotlin/hs/kr/entrydsm/feed/infrastructure/grpc/client/AdminGrpcClient.kt
Show resolved
Hide resolved
casper-feed/src/main/kotlin/hs/kr/entrydsm/feed/infrastructure/grpc/client/AdminGrpcClient.kt
Show resolved
Hide resolved
1 participant
📌 개요
🧩 주요 변경 사항
✅ 기능 구현
AdminGrpcClient 클래스 생성
getAdminInfoByAdminId(UUID): InternalAdminResponse 메서드 구현
CoroutineStub을 사용한 비동기 gRPC 호출
InternalAdminResponse DTO 생성
gRPC 응답 데이터를 도메인 내부에서 사용할 수 있도록 변환
admin.proto 정의
GetAdminByUUID RPC 메서드 및 관련 메시지 정의
Summary by CodeRabbit
New Features
Bug Fixes
Tests
Chores