OCI용 CD 파이프라인 마이그레이션

[Hwanvely]

No description provided.

[gemini-code-assist]

Code Review

This pull request implements containerization with Jib, adds deployment scripts for development and rolling updates, and introduces anti-cheating logic for the Fever Time mechanic. It also refactors session management to include event timestamps and adds a global exception handler for multipart requests. Key feedback includes correcting a logic error in the percentile calculation query, optimizing property types for null-safety in the domain model, removing redundant transaction propagation, and improving the reliability of deployment health checks.

I am having trouble creating individual review comments. Click here to see my feedback.

src/main/java/com/snackgame/server/game/snackgame/core/domain/SnackgameRepository.kt (15-28)

현재 쿼리는 WHERE 절에서 session_id를 필터링한 후 percent_rank()를 계산하므로, 항상 단일 행에 대한 순위인 0.0을 반환하게 됩니다. 올바른 백분위를 계산하려면 모든 대상 세션에 대해 먼저 순위를 매긴 후 특정 session_id를 선택해야 합니다.

        value = """
        SELECT percentile FROM (
            SELECT session_id, percent_rank() over (order by score desc) as percentile
            FROM snackgame 
            WHERE expires_at <= now() + INTERVAL 1 SECOND
        ) t WHERE session_id = :sessionId
        
        UNION ALL
        
        SELECT 0.0 as percentile
        FROM snackgame
        WHERE session_id = :sessionId
          AND expires_at > now() + INTERVAL 1 SECOND
        LIMIT 1
        """,

src/main/java/com/snackgame/server/game/snackgame/core/domain/item/FeverTime.kt (13)

feverStreakCount는 항상 0으로 초기화되며 도메인 로직에서 널이 될 가능성이 없으므로, 비널(non-nullable) 타입인 Int로 선언하여 불필요한 엘비스 연산자(?: 0) 사용을 줄이는 것이 좋습니다.

    var feverStreakCount: Int = 0

src/main/java/com/snackgame/server/game/snackgame/core/domain/item/FeverTime.kt (34-38)

feverStreakCount를 비널 타입으로 변경함에 따라 관련 로직의 널 안전성 연산자를 제거할 수 있습니다.

    fun canApplyFeverMultiplier(): Boolean = feverStreakCount < MAX_FEVER_STREAKS

    fun incrementFeverStreak() {
        feverStreakCount += 1
    }

src/main/java/com/snackgame/server/game/snackgame/core/service/SnackgameService.kt (115)

getEndResponse는 단순히 결과를 조회하는 메서드이며, 호출부인 컨트롤러에서 이미 end() 트랜잭션이 완료된 후 호출하므로 Propagation.REQUIRES_NEW는 불필요합니다. 이는 불필요한 데이터베이스 커넥션 점유를 유발할 수 있습니다.

    @Transactional(readOnly = true)

scripts/deploy-rolling.sh (15)

헬스 체크 URL에 특정 리소스 ID(rankings/1)가 포함되어 있습니다. 해당 데이터의 존재 여부에 따라 배포 성공 여부가 결정되는 것은 위험할 수 있으므로, 애플리케이션의 상태를 직접 나타내는 /actuator/health와 같은 엔드포인트를 사용하는 것이 안전합니다.