Conversation
Walkthrough심박수(heart-beat) 엔드포인트가 Void에서 Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
src/main/java/com/gpt/geumpumtabackend/study/service/StudySessionService.java (2)
35-35: 상수명을 더 명확하게 개선하는 것을 고려하세요.
MAX_FOCUS_TIME상수가 시간 단위를 나타내지만 상수명만으로는 단위가 명확하지 않습니다.다음과 같이 개선할 수 있습니다:
- private static final Integer MAX_FOCUS_TIME = 3; + private static final Integer MAX_FOCUS_TIME_HOURS = 3;그리고 Line 105에서도 이에 맞춰 변경:
- if(elapsed.compareTo(Duration.ofHours(MAX_FOCUS_TIME)) >= 0) { + if(elapsed.compareTo(Duration.ofHours(MAX_FOCUS_TIME_HOURS)) >= 0) {
104-111: 최대 집중 시간 제한 로직이 올바르게 구현되었습니다.3시간 제한 로직이 의도대로 구현되었습니다:
- 세션 시작부터 현재까지의 경과 시간을 계산
- 3시간 이상이면 세션을 시작 시점 + 3시간으로 종료 (소급 종료)
- 적절한
HeartBeatResponse반환다만, 다음 개선 사항을 고려하세요:
하드코딩된 메시지를 i18n으로 관리: Line 107과 Line 110의 한글 메시지를 메시지 리소스 파일로 외부화하면 다국어 지원 및 유지보수성이 향상됩니다.
로깅 추가: 3시간 제한으로 세션이 자동 종료될 때 로그를 남기면 모니터링에 유용합니다.
Duration elapsed = Duration.between(studySession.getStartTime(), LocalDateTime.now()); if(elapsed.compareTo(Duration.ofHours(MAX_FOCUS_TIME)) >= 0) { + log.info("Session {} for user {} exceeded max focus time (3 hours), auto-terminating", sessionId, userId); studySession.endStudySession(studySession.getStartTime().plusHours(MAX_FOCUS_TIME)); return new HeartBeatResponse(false, "최대 집중시간은 3시간입니다."); }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java(3 hunks)src/main/java/com/gpt/geumpumtabackend/study/controller/StudySessionController.java(2 hunks)src/main/java/com/gpt/geumpumtabackend/study/dto/response/HeartBeatResponse.java(1 hunks)src/main/java/com/gpt/geumpumtabackend/study/service/StudySessionSchedulerService.java(0 hunks)src/main/java/com/gpt/geumpumtabackend/study/service/StudySessionService.java(5 hunks)
💤 Files with no reviewable changes (1)
- src/main/java/com/gpt/geumpumtabackend/study/service/StudySessionSchedulerService.java
🧰 Additional context used
🧬 Code graph analysis (1)
src/main/java/com/gpt/geumpumtabackend/study/controller/StudySessionController.java (1)
src/main/java/com/gpt/geumpumtabackend/global/response/ResponseUtil.java (1)
ResponseUtil(5-28)
🔇 Additional comments (3)
src/main/java/com/gpt/geumpumtabackend/study/dto/response/HeartBeatResponse.java (1)
3-4: LGTM!하트비트 응답을 위한 간결하고 명확한 record 타입입니다.
sessionActive로 세션 상태를 전달하고message로 상세 정보를 제공하는 구조가 적절합니다.src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java (1)
173-173: API 반환 타입 변경을 확인하세요.반환 타입이
ResponseBody<Void>에서ResponseBody<HeartBeatResponse>로 변경되어 클라이언트가 이제 세션 상태(sessionActive)와 메시지를 받게 됩니다. 이는 의도된 breaking change이지만, 모든 클라이언트 구현체가 이 변경에 대응하도록 업데이트되었는지 확인이 필요합니다.src/main/java/com/gpt/geumpumtabackend/study/controller/StudySessionController.java (1)
68-70: LGTM!하트비트 엔드포인트가 이제
HeartBeatResponse를 반환하도록 올바르게 구현되었습니다. 서비스 계층의 반환값을 적절하게 감싸서 클라이언트에 전달합니다.
src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java
Outdated
Show resolved
Hide resolved
…pi.java Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java (2)
141-159: 하트비트 Javadoc에서 항목 1, 2가 중복되어 있습니다.
🔄 동작 원리아래에서:
- 149–150행에 1, 2번 항목이 있고
- 151–152행에 동일한 1, 2번 항목이 한 번 더 반복됩니다.
문서 가독성을 위해 중복된 항목은 제거하고, 1–3번을 한 번만 나열하도록 정리하는 것이 좋겠습니다.
예시:
- 🔄 **동작 원리:** - 1. Wi-Fi 연결 상태 재검증 (Gateway IP + IP 대역 확인) - 2. 클라이언트 실제 IP 주소 재확인 (서버에서 추출) - 1. Wi-Fi 연결 상태 재검증 (Gateway IP + IP 대역 확인) - 2. 클라이언트 실제 IP 주소 재확인 (서버에서 추출) - 3. 최대 집중 시간(3시간) 초과 여부 확인 및 자동 세션 종료 + 🔄 **동작 원리:** + 1. Wi-Fi 연결 상태 재검증 (Gateway IP + IP 대역 확인) + 2. 클라이언트 실제 IP 주소 재확인 (서버에서 추출) + 3. 최대 집중 시간(3시간) 초과 여부 확인 및 자동 세션 종료
28-41: 90초 자동 종료 설명과 3시간 최대 집중 시간 설명을 한 번 더 정합성 체크해 주세요.상단
⚠️ 중요사항(38–40행)에는:
- “하트비트 중단시 90초 후 자동 세션 종료”
라고 되어 있고, 하트비트 Javadoc(148–153행)에는:
- “최대 집중 시간(3시간) 초과 여부 확인 및 자동 세션 종료”
가 추가되었습니다.
실제 백엔드 로직이 “하트비트 미수신 90초 타임아웃”과 “3시간 최대 집중 시간”을 둘 다 사용한다면, 두 제약이 함께 동작한다는 점을 명시해 주시면 좋고,
만약 기존 90초 타임아웃이 제거/완화되고 3시간 제한만 유효하다면, 40행의 90초 설명을 최신 동작에 맞게 수정하거나 삭제하는 편이 혼선을 줄일 수 있습니다.Also applies to: 141-159
🧹 Nitpick comments (1)
src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java (1)
161-176: 하트비트 응답 타입을 Swagger 문서에도 명시하는 것을 고려해 주세요.
processHeartBeat의 실제 반환 타입이ResponseEntity<ResponseBody<HeartBeatResponse>>로 변경되었는데,
@SwaggerApiSuccessResponse에서는response타입을 명시하지 않고 있습니다. 위쪽getTodayStudySession,startStudySession과의 일관성을 위해 아래와 같이 응답 타입을 명시하면 Swagger 문서에서 스키마가 더 명확해질 것 같습니다.예시:
- @SwaggerApiResponses( - success = @SwaggerApiSuccessResponse( - description = "하트비트 전송 성공 - 세션 유지"), + @SwaggerApiResponses( + success = @SwaggerApiSuccessResponse( + response = HeartBeatResponse.class, + description = "하트비트 전송 성공 - 세션 유지"),
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java(3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (1)
src/main/java/com/gpt/geumpumtabackend/study/api/StudySessionApi.java (1)
12-13: HeartBeatResponse 도입 및 반환 타입 변경 방향은 적절해 보입니다.
HeartBeatResponseimport 추가와processHeartBeat의 반환 타입을ResponseBody<HeartBeatResponse>로 바꾼 부분은 PR 설명(세션 활성/만료 여부를 응답으로 전달)에 잘 부합하고, API 사용성도 좋아졌습니다.별다른 이슈 없어 보입니다.
Also applies to: 172-177
2 participants
🚀 1. 개요
기존의 백그라운드 스케줄러를 이용한 좀비 세션 관리 방식을 제거하고, 클라이언트의 하트비트 요청 시점에최대 집중 시간을 검증하여 세션을 자동 종료하도록 로직을 변경했습니다.
📝 2. 주요 변경 사항
1. 좀비 세션 스케줄러 제거
StudySessionSchedulerService.java2. 최대 집중 시간(Max Focus Time) 로직 적용
StudySessionService에MAX_FOCUS_TIME(3시간) 상수를 추가했습니다.updateHeartBeat):void)sessionActive: true반환sessionActive: false반환Summary by CodeRabbit
릴리스 노트
새로운 기능
문서/표현 정렬
✏️ Tip: You can customize this high-level summary in your review settings.