Skip to content

Feature/#50 behavior interview flow - #52

Merged
2-dayeon merged 3 commits into
mainfrom
feature/#50-behavior-interview-flow
Oct 20, 2026
Merged

2-dayeon merged 3 commits into
mainfrom
feature/#50-behavior-interview-flow

Conversation

@2-dayeon

Copy link
Copy Markdown
Contributor

50

@kimwuju kimwuju left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 남깁니다. 머지를 막을 이슈는 없습니다.

  • setIfAbsent가 실패했을 때만 기존 세션의 interviewId·userId를 비교하므로, 같은 요청을 재시도하면 기존 진행 상태가 그대로 반환되고 식별자가 다르면 AlreadyUsingException 경로로 409 ALREADY_IN_USE가 나가는 것을 확인했습니다. 충돌 시 세션을 덮어쓰거나 TTS를 다시 등록하지 않는 것도 맞습니다.
  • 문서의 종료 본문 필드 목록이 ChatInterviewAllRequest와 일치합니다.
  • 로컬에서 Java 21로 전체 테스트 263개가 통과했고 CI도 통과했습니다. #53과 같은 파일을 수정하지만 충돌 없이 합쳐집니다.

@2-dayeon
2-dayeon merged commit 85a0454 into main Oct 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants