Gemini 특정 모델을 사용할 수 없는 경우, 다른 모델을 사용하도록 구현

Gemini 특정 모델을 사용할 수 없는 경우, 다른 모델을 사용하도록 수정

Author: Jwhyee

src/main/kotlin/com/project/codereview/client/github/GithubDiffUtils.kt

@@ -14,14 +14,13 @@ object GithubDiffUtils {
         val side: String,      // "RIGHT" | "LEFT"
         val snippet: String
     ) {
-        fun toReviewType(): ReviewType.ByMultiline =
-            ReviewType.ByMultiline(
-                path = path,
-                line = endLine,
-                side = side,
-                start_line = startLine,
-                start_side = side
-            )
+        fun toReviewType() = ReviewType.ByMultiline(
+            path = path,
+            line = endLine,
+            side = side,
+            start_line = startLine,
+            start_side = side
+        )
     }
 
     // 파일 단위 컨텍스트(파일별로 묶기)

src/main/kotlin/com/project/codereview/client/google/GoogleGeminiClient.kt

@@ -1,18 +1,16 @@
 package com.project.codereview.client.google
 
 import com.google.genai.Client
-import com.google.genai.types.Content
-import com.google.genai.types.GenerateContentConfig
-import com.google.genai.types.Part
-import com.google.genai.types.ThinkingConfig
-import com.project.codereview.client.util.MODEL
+import com.google.genai.types.*
+import com.project.codereview.client.util.GeminiTextModel
 import com.project.codereview.client.util.ReviewLanguage
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
 import org.slf4j.LoggerFactory
 import org.springframework.beans.factory.annotation.Value
 import org.springframework.stereotype.Component
 import java.util.concurrent.ConcurrentHashMap
+import kotlin.jvm.optionals.getOrNull
 
 @Component
 class GoogleGeminiClient(
@@ -46,6 +44,7 @@ class GoogleGeminiClient(
     suspend fun chat(
         filePath: String,
         prompt: String,
+        model: GeminiTextModel,
         instruction: Content = ReviewLanguage.fromExtension(filePath).let { language ->
             instructionMap[language] ?: instructionMap[ReviewLanguage.KT]!!
         },
@@ -54,7 +53,7 @@ class GoogleGeminiClient(
         try {
             logger.info("[Gemini] request started = {}", filePath)
             client.models.generateContentStream(
-                MODEL,
+                model.modelName,
                 prompt,
                 GenerateContentConfig.builder()
                     .systemInstruction(instruction)

src/main/kotlin/com/project/codereview/client/util/ReviewLanguage.kt

@@ -32,112 +32,4 @@ enum class ReviewLanguage(
             } ?: KT
         }
     }
-}
-
-const val REJECT_REVIEW = "REJECT_REVIEW"
-
-private const val SYSTEM_PROMPT_JS_TS = """
-
-"""
-
-private const val SYSTEM_PROMPT_COMMON = """
-## 이름과 역할
-
-- 너의 이름은 Review Bot.
-- 한국의 코틀린/스프링 생태계를 잘 아는 CTO 동료처럼, 간결하고 실무 친화적으로 말한다.
-- 입력은 Pull Request의 Git diff 결과이며, 독자는 주니어 개발자다.
-
-## 응답 원칙
-
-삭제된 파일이거나, 리뷰가 필요 없는 단순한 코드인 경우, ${REJECT_REVIEW}를 응답한다.
-그렇지 않을 경우, 다음 원칙에 따라 **출력 형식**에 맞게 응답한다.
-
-- 모든 지적에는 중요도(High/Medium/Low)와 영향 범주(안정성/성능/가독성/보안/호환)를 명시한다.
-- 최소 3개 이상의 범주(언어·스타일, API·설계, 동시성/코루틴, 성능/컬렉션, I/O·경계, 로깅·보안, 테스트)를 동시에 다룬다.
-- 실제로 바꿔야 할 코드만 30줄 이내 스니펫으로 제시하고, 적용될 파일/라인 범위를 함께 적는다.
-
-## 길이·톤 제약
-
-- 전체 350~900자 권장(스니펫 제외), 항목별 6줄 이내.
-- 과장·군더더기 배제, 동료에게 직접 적용 가능한 실무 톤 유지.
-
-## 스타일 가이드 선택
-
-- JetBrains Kotlin 컨벤션을 기본으로 가정한다.
-
-## 리뷰 체크리스트(범용)
-
-1) 언어·스타일
-   - 불변(val) 우선, 의미 있는 이름, 가시성 최소화(public 지양, internal/private 선호).
-   - 널 안정성(안전 호출/엘비스, require/check, 널 가능 타입 축소).
-   - 타입 설계: value/inline class, typealias, sealed 계층으로 도메인 명확화.
-   - 스코프/확장 함수(let/run/apply/also/with) 남용 방지.
-
-2) API·설계
-   - 단일 책임, 레이어 경계, 의존 역전.
-   - 오류 모델 일관성(예외 vs Result vs sealed error).
-   - equals/hashCode/copy 의미·불변성 보장.
-   - 공개 API 변경 시 호환성 영향 표기(바이너리/소스/직렬화 스키마).
-
-3) 동시성/코루틴
-   - 구조화된 동시성(스코프, 취소 전파, timeout) 준수, GlobalScope 지양.
-   - Dispatcher 선택과 blocking 호출 유입 점검(withContext(Dispatchers.IO) vs 비차단).
-   - Flow/Channel의 backpressure, 오류·취소 전파, 리소스 해제 확인.
-
-4) 성능/컬렉션
-   - 불필요한 객체/복사 제거, sequence/lazy 적정 사용.
-   - 알고리즘 복잡도, 배치/캐시 전략, boxing·toList() 남용 점검.
-
-5) I/O·경계
-   - 네트워크/DB/파일 타임아웃, 재시도, 연결/자원 해제.
-   - 직렬화/역직렬화 안전성, 입력 검증, 경계값 처리.
-
-6) 로깅·보안·관측
-   - 파라미터화 로깅 사용, 로그 레벨 일관성, MDC/추적ID.
-   - 비밀/PII 로그 금지, 예외 메시지에 민감정보 포함 금지.
-   - 메트릭·트레이싱 노출로 관측 가능성 확보.
-
-7) 테스트
-   - 단위/통합/계약/경계/동시성/직렬화 라운드트립.
-   - 순수 함수에는 property-based test를 최소 1개 요구.
-   - Given-When-Then, 결정적 실행, 명확한 네이밍.
-
-## 출력 형식
-
-인사 한 줄
-(예: 안녕하세요. ~에 대한 PR 잘 보았습니다.)
-
-###  좋은 점
-
-- 1~3개. 구체적 변화와 팀 가치(가독성/안정성/성능)를 연결해 말한다.
-
-### 개선 및 제안
-
-- 아래 서브포맷으로 항목을 작성한다. 각 항목은 다른 범주를 우선적으로 커버한다.
-    - 설명: 무엇이 왜 문제인지, 맥락을 2~4문장으로  
-    - AS-IS: 필요한 부분만 코드 스니펫(≤30줄)  
-    - TO-BE: 즉시 적용 가능한 대안 스니펫(≤30줄)  
-
-간단한 마무리 인사 한 줄
-(예: 수고하셨습니다 👍)
-"""
-
-const val SUMMARY_PROMPT = """
-## 이름과 역할
-
-- 너의 이름은 Review Bot.
-- 너는 다재다능한 CTO이고, 간결하고 실무 친화적인 말투로 작업 결과물에 대해서 요약을 해야한다.
-- 입력은 Pull Request의 Git diff 결과이며, 독자는 주니어 개발자다.
-
-## 응답 원칙
-
-- 이 Pull Request에 어떤 변경 사항이 있는지 정리한다.
-- 절대로 향후 방향성에 대한 내용은 작성하지 않고, 단순히 코드 변경 사항을 요약한다.
-
-## 출력 형식
-
-인사 한 줄
-(예: 안녕하세요. 코드 변경 사항에 대해서 요약해드리겠습니다.)
-
-주요 변경 사항 요약
-"""
+}

src/main/kotlin/com/project/codereview/client/util/ReviewModels.kt

@@ -0,0 +1,55 @@
+package com.project.codereview.client.util
+
+enum class GeminiTextModel(
+    val modelName: String,
+    val maxRpm: Int,
+    val maxTpm: Int,
+    val maxRpd: Int
+) {
+    GEMINI_2_5_FLASH(
+        modelName = "gemini-2.5-flash",
+        maxRpm = 10,
+        maxTpm = 250_000,
+        maxRpd = 250
+    ),
+
+    GEMINI_2_5_FLASH_LITE(
+        modelName = "gemini-2.5-flash-lite",
+        maxRpm = 15,
+        maxTpm = 250_000,
+        maxRpd = 1_000
+    ),
+
+    GEMINI_2_5_PRO(
+        modelName = "gemini-2.5-pro",
+        maxRpm = 2,
+        maxTpm = 125_000,
+        maxRpd = 50
+    ),
+
+    GEMINI_2_0_FLASH(
+        modelName = "gemini-2.0-flash",
+        maxRpm = 15,
+        maxTpm = 1_000_000,
+        maxRpd = 200
+    ),
+
+    GEMINI_2_0_FLASH_LITE(
+        modelName = "gemini-2.0-flash-lite",
+        maxRpm = 30,
+        maxTpm = 1_000_000,
+        maxRpd = 200
+    );
+
+    fun toRateLimit(): RateLimit = RateLimit(
+        rpm = maxRpm,
+        tpm = maxTpm,
+        rpd = maxRpd
+    )
+}
+
+data class RateLimit(
+    val rpm: Int,
+    val tpm: Int,
+    val rpd: Int
+)

src/main/kotlin/com/project/codereview/client/util/ReviewPrompts.kt

@@ -0,0 +1,59 @@
+package com.project.codereview.client.util
+
+const val REJECT_REVIEW = "REJECT_REVIEW"
+
+const val SYSTEM_PROMPT_COMMON = """
+## 이름과 역할
+
+- 너의 이름은 Review Bot.
+- 한국의 코틀린/스프링 생태계를 잘 아는 CTO 동료처럼, 간결하고 실무 친화적으로 말한다.
+- 입력은 Pull Request의 Git diff 결과이며, 독자는 주니어 개발자다.
+
+## 응답 원칙
+
+삭제된 파일이거나, 리뷰가 필요 없는 단순한 코드인 경우, ${REJECT_REVIEW}를 응답한다.
+그렇지 않을 경우, 다음 원칙에 따라 **출력 형식**에 맞게 응답한다.
+
+- 모든 지적에는 중요도(High/Medium/Low)와 영향 범주(안정성/성능/가독성/보안/호환)를 명시한다.
+- 최소 3개 이상의 범주(언어·스타일, API·설계, 동시성/코루틴, 성능/컬렉션, I/O·경계, 로깅·보안, 테스트)를 동시에 다룬다.
+- 실제로 바꿔야 할 코드만 5줄 이내 스니펫으로 제시한다.
+
+## 출력 형식
+
+인사 한 줄
+(예: 안녕하세요. ~에 대한 PR 잘 보았습니다.)
+
+###  좋은 점
+
+- 1~3개. 구체적 변화와 팀 가치(가독성/안정성/성능)를 연결해 말한다.
+
+### 개선 및 제안
+
+- 아래 서브포맷으로 항목을 작성한다. 각 항목은 다른 범주를 우선적으로 커버한다.
+    - 설명: 무엇이 왜 문제인지, 맥락을 2~4문장으로  
+    - AS-IS: 필요한 부분만 코드 스니펫(≤30줄)  
+    - TO-BE: 즉시 적용 가능한 대안 스니펫(≤30줄)  
+
+간단한 마무리 인사 한 줄
+(예: 수고하셨습니다 👍)
+"""
+
+const val SUMMARY_PROMPT = """
+## 이름과 역할
+
+- 너의 이름은 Review Bot.
+- 너는 다재다능한 CTO이고, 간결하고 실무 친화적인 말투로 작업 결과물에 대해서 요약을 해야한다.
+- 입력은 Pull Request의 Git diff 결과이며, 독자는 주니어 개발자다.
+
+## 응답 원칙
+
+- 이 Pull Request에 어떤 변경 사항이 있는지 정리한다.
+- 절대로 향후 방향성에 대한 내용은 작성하지 않고, 단순히 코드 변경 사항을 요약한다.
+
+## 출력 형식
+
+인사 한 줄
+(예: 안녕하세요. 코드 변경 사항에 대해서 요약해드리겠습니다.)
+
+주요 변경 사항 요약
+"""

src/main/kotlin/com/project/codereview/client/util/ReviewUtils.kt

@@ -1,8 +1,7 @@
 package com.project.codereview.core.controller
 
-import com.project.codereview.core.dto.GithubActionType
 import com.project.codereview.core.dto.parsePayload
-import com.project.codereview.core.service.CodeReviewService
+import com.project.codereview.core.service.PullRequestEventEntry
 import com.project.codereview.core.util.GithubSignature
 import org.slf4j.LoggerFactory
 import org.springframework.beans.factory.annotation.Value
@@ -16,52 +15,32 @@ import org.springframework.web.bind.annotation.RestController
 @RestController
 class CodeReviewController(
     @param:Value("\${app.github.webhook.secret-key}") private val secret: String,
-    private val codeReviewService: CodeReviewService
+    private val entry: PullRequestEventEntry
 ) {
-    private val logger = LoggerFactory.getLogger(CodeReviewController::class.java)
+    private val log = LoggerFactory.getLogger(CodeReviewController::class.java)
 
     @PostMapping("/api/code/review")
-    suspend fun net(
+    suspend fun handleWebhook(
         @RequestHeader("X-GitHub-Event") event: String,
         @RequestHeader("X-Hub-Signature-256", required = false) sig256: String?,
         @RequestBody rawBody: ByteArray
     ): ResponseEntity<String> {
-        if (sig256 == null) {
-            logger.warn("Missing signature header for GitHub webhook")
-            return fail("Missing signature")
-        }
-        // 서명 검증
-        if (!GithubSignature.isValid(sig256, secret, rawBody)) {
-            logger.warn("Invalid signature detected from GitHub webhook")
-            return fail("Invalid signature")
-        }
+        if (sig256.isNullOrBlank()) return fail("Missing signature")
+        if (!GithubSignature.isValid(sig256, secret, rawBody)) return fail("Invalid signature")
 
         val payload = try {
             parsePayload(rawBody)
         } catch (e: Exception) {
-            return fail("Invalid payload : ${e.message}", HttpStatus.NOT_ACCEPTABLE)
+            return fail("Invalid payload: ${e.message}", HttpStatus.NOT_ACCEPTABLE)
         }
 
-        val action = GithubActionType(payload.action)
-        logger.info("payload = {}, action = {}", payload, action)
+        entry.handle(event, payload)
 
-        // PR 생성 시에만 리뷰 진행
-        when (event) {
-            "pull_request" -> when (GithubActionType(payload.action)) {
-                GithubActionType.OPENED,
-                GithubActionType.REOPENED -> {
-                    codeReviewService.review(payload)
-                }
-                else -> logger.info("Ignored pull_request event: ${payload.action}")
-            }
-            else -> logger.debug("Unhandled GitHub event: $event")
-        }
-
-        return ResponseEntity.ok("Request completed")
+        return ResponseEntity.ok("Accepted")
     }
 
-    private fun fail(
-        message: String,
-        status: HttpStatus = HttpStatus.UNAUTHORIZED
-    ) = ResponseEntity.status(status).body(message)
+    private fun fail(message: String, status: HttpStatus = HttpStatus.UNAUTHORIZED): ResponseEntity<String> {
+        log.error("[Api request fail] reason = {}", message)
+        return ResponseEntity.status(status).body(message)
+    }
 }