Repository navigation
Fix/#56 fix error in test - #57
Conversation
danjee2009
left a comment
danjee2009
left a comment
There was a problem hiding this comment.
검토 중 확인한 문제 1건입니다. 일반 생성·정상 생략과 오류 폴백을 구분하는 진단 로그가 실패 상황에서 성공으로 기록될 수 있습니다.
| ) | ||
| .doOnNext(result -> log.info( | ||
| "follow_up.decision status=completed reason={} sessionId={} interviewId={} questionId={}", | ||
| Boolean.TRUE.equals(result.getRequired()) ? "generated" : "model_not_required", |
There was a problem hiding this comment.
[P2] 기업 꼬리질문 생성 실패를 generated로 기록하지 않도록 해주세요. 기업 재작성 단계가 timeout/빈 응답/잘못된 응답으로 실패하면 CompanyFollowUpGenerator.fallback()은 decision을 그대로 반환합니다. decision이 required=true인 경우 이 doOnNext는 해당 실패도 status=completed reason=generated로 남겨, 새 진단 로그의 핵심 목적(생성 성공과 fallback 구분)을 깨뜨립니다. 생성 결과에 별도 outcome을 전달하거나 성공 로그를 실제 B단계 적용 성공 시점으로 옮기고, fallback은 실패/부분 성공 사유로 남겨주세요.
kimwuju
left a comment
kimwuju
left a comment
There was a problem hiding this comment.
다시 확인했습니다. 앞서 지적된 [P2] 문제는 03d6944·b759831에서 해결됐습니다. 기업 재작성이 실패해 A 단계 질문으로 돌아간 경우가 이제 generated가 아니라 follow_up.decision status=fallback reason=company_*로 기록됩니다. timeout·빈 응답·잘못된 응답과 재작성 성공을 나눠 확인하는 회귀 테스트도 추가됐습니다. 그 외에 추가로 발견한 문제는 없습니다.
56