Repository navigation
Feature/#50 behavior interview flow - #52
Merged
Merged
Conversation
kimwuju
reviewed
Oct 20, 2026
kimwuju
left a comment
There was a problem hiding this comment.
리뷰 남깁니다. 머지를 막을 이슈는 없습니다.
setIfAbsent가 실패했을 때만 기존 세션의interviewId·userId를 비교하므로, 같은 요청을 재시도하면 기존 진행 상태가 그대로 반환되고 식별자가 다르면AlreadyUsingException경로로 409ALREADY_IN_USE가 나가는 것을 확인했습니다. 충돌 시 세션을 덮어쓰거나 TTS를 다시 등록하지 않는 것도 맞습니다.- 문서의 종료 본문 필드 목록이
ChatInterviewAllRequest와 일치합니다. - 로컬에서 Java 21로 전체 테스트 263개가 통과했고 CI도 통과했습니다. #53과 같은 파일을 수정하지만 충돌 없이 합쳐집니다.
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.
50