Repository navigation
Fix/#49 interview termination state - #53
Merged
Merged
Conversation
kimwuju
reviewed
Oct 20, 2026
kimwuju
left a comment
There was a problem hiding this comment.
리뷰 남깁니다. 머지를 막을 이슈는 없습니다.
- 진행 중인 세션에 남은 질문이 있을 때만 COMPLETE를 거절하고, 이미 확정된 상태는 outbox 결과를 그대로 반환하는 흐름을 확인했습니다. REST는 전용 핸들러로 409
INTERVIEW_NOT_FINISHED를 반환하고, WebSocket은 ERROR를 보낸 뒤 연결을 유지해서 문서와 동작이 일치합니다. - END의
status는 서비스 응답(재전송 시에는 outbox의interviewStatus)에서 가져오므로, QUIT과 COMPLETE가 엇갈려도 실제 상태가 전달됩니다. 삭제한end(String)를 호출하는 곳도 남아 있지 않습니다. - 복구 워커가 잠금 안에서 종료 상태를 다시 확인하므로, 스캔 이후 같은 sessionId로 다시 준비된 IN_PROGRESS 세션을 지우지 않습니다.
- 현재 Web은 명시적인 COMPLETE를 보내지 않으므로, 조기 완료를 거절해도 기존 화면 흐름은 깨지지 않습니다.
- 로컬에서 Java 21로 전체 테스트 267개가 통과했고 CI도 통과했습니다. #52와 충돌 없이 합쳐집니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
49