Repository navigation
feat: 문항별 예상 꼬리질문 저장·조회와 기업 코멘트 진단 로그 - #119
Merged
Merged
Conversation
분석 콜백의 feedbacks[].followUpQuestion을 받아 feedback_item에 저장하고 단건·목록 조회의 data.feedbacks[]로 돌려준다. 이전 피드백과 이 필드를 보내지 않는 분석 서버의 콜백은 null로 남는다. - V41: feedback_item.follow_up_question TEXT 추가 - 콜백 필드는 JsonNode로 받아 문자열이 아닌 값이 와도 본문 전체가 400이 되지 않게 한다 - 앞뒤 공백을 뺀 1~200 code point 문자열만 저장하고, 그 밖은 그 문항만 비운다 - 기업을 골랐는데 companyFit이 오지 않으면 FIT_NOT_RECEIVED를 남긴다 - 채점 요청·결과 저장 시 기업 여부, fact 수, 근거 문항 수, 꼬리질문 수를 로그로 남긴다 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
danjee2009
reviewed
Oct 20, 2026
danjee2009
left a comment
Member
There was a problem hiding this comment.
검토 결과: 확인된 수정 필요 결함은 없습니다. 콜백 역직렬화, 꼬리질문 정규화·길이 검증과 저장·조회 경로, 기업 관점 진단 로그를 검토했습니다. 테스트는 직접 재실행하지 않았습니다.
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.
연관 이슈 없음 (부스 운영 전 수정안 — API 서버 담당분)
Task
followUpQuestion추가 (신규 계약)V41__add_feedback_item_follow_up_question.sql:feedback_item.follow_up_question TEXT(nullable) 추가result.feedbacks[].followUpQuestion→FeedbackItemEntity→ 단건·목록 조회data.feedbacks[].followUpQuestioncompanyFit이 null로 오면FIT_NOT_RECEIVED를 남깁니다. 지금까지는 이 경우 로그가 하나도 없어, 분석 서버 쪽 문제인지 API 검증에서 버린 것인지 가릴 수 없었습니다.NO_ANSWERED_QUESTION으로 빠지는 경우를 가릴 수 있습니다.ETC
followUpQuestion은String대신JsonNode로 받습니다.String이면 분석 서버가 객체를 보냈을 때 역직렬화가 실패해 콜백 전체가 400이 되고, 분석 서버는 재시도 뒤 채점 결과를 버립니다. 기존axisScores·companyFit과 같은 방식입니다.followUpQuestion생성 → 웹feedbackDisplay.ts의 고정 문구를 응답값으로 교체. 웹을 고치지 않으면 화면은 계속 "예상 꼬리질문이 아직 제공되지 않았습니다."로 나옵니다.SUCCEEDED된 면접은 재생성되지 않으므로, 부스 검수는 새 면접으로 해야 합니다../gradlew test964개 통과🤖 Generated with Claude Code