Repository navigation
fix: 텍스트 면접 문항 수 표시와 면접 완료 후 종합 피드백 이동 - #170
Merged
Merged
Conversation
SOLO 진행 화면이 면접 생성 시점 스냅샷의 질문 수와 폴백값 10을 써서 "Question 01 / 10"으로 보였다. 질문 준비 완료 때 세션에 들어오는 totalQuestionCount를 쓰고, 개수를 모르면(0) 분모를 "--"로 표시한다. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 완료 이동 state에 interviewId를 넣고, "피드백 보러가기"가
/main/feedback/overall/{id}로 fromCompletion state와 함께 이동한다.
state가 없으면(새로고침 등) 지금처럼 목록으로 간다.
- 면접 COMPLETED는 Chat → API가 비동기로 반영하므로, fromCompletion으로
들어왔는데 아직 IN_PROGRESS면 GET /api/interviews/get을 3초 간격으로
최대 60초 다시 조회하고 그동안 "면접 기록을 저장하고 있어요"를 보여 준다.
COMPLETED가 되면 기존 피드백 조회·폴링으로 넘어가고, 60초를 넘기면
기존 오류 문구를 보여 준다.
- 재조회 요청마다 시간 제한을 두고, 재시도할 수 없는 오류(4xx)는 바로
멈추고 서버 오류 문구를 보여 준다.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- SOLO 텍스트·음성 문항 수, 개수 미상 "--", MULTI 표시 회귀 - 완료 → 종합 피드백 이동, state 없는 완료 화면의 목록 이동 - IN_PROGRESS → COMPLETED 전환 뒤 PENDING 폴링, 60초 초과, 재시도 가능/불가 오류, fromCompletion 없음·ABANDONED·이미 COMPLETED일 때 재조회 없음 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
danjee2009
commented
Oct 19, 2026
danjee2009
left a comment
danjee2009
left a comment
Member
Author
There was a problem hiding this comment.
완료 화면에서 피드백 화면으로 이동한 뒤 상태를 주기적으로 조회하는 흐름, 제한 시간·요청 시간 초과·재시도 처리, 질문 수 전달과 관련 테스트를 검토했습니다. 확인된 실질적 문제는 없습니다. 이 PR에는 CI 체크 결과가 표시되지 않았으며, Chat→API 간 실제 타이밍 통합은 검증하지 못했습니다.
This was referenced Oct 19, 2026
Merged
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.
명세:
Doc/면접완료-피드백-수정계획-2026-10-19.md2-1, 2-2 (4절 해당 행)기준 커밋:
origin/feature/#166-company-dataset92d6cea무엇을 / 왜
2-1. 문항 수
/10InterviewContentView)이 면접 생성 시점 스냅샷preparedInterview.questions.length와 폴백값 10을 써서Question 01 / 10으로 보였다.totalQuestionCount(질문 준비 완료 때SET_QUESTION_COUNT)를 쓰고, 0이면 분모를--로 표시한다.FALLBACK_TOTAL_QUESTION_COUNT삭제.2-2. 완료 후 이동
interviewId추가(useInterviewSession)./main/feedback/overall/{id}+ state{ fromCompletion: true }. state가 없으면 지금처럼 목록.useFeedbackInterview:fromCompletion으로 들어왔고 면접이IN_PROGRESS면GET /api/interviews/get?interviewId=를 3초 간격, 최대 60초 다시 조회. 그동안 종합 화면에 "면접 기록을 저장하고 있어요".COMPLETED가 되면 기존getOrRequestFeedback→ PENDING 폴링, 60초 초과면 기존 오류 문구.계약 변경 없음.
테스트
npm run lint(기존router.tsx경고 1건 외 없음),npm run build통과node --test tests/*.test.mjs32건 통과tests/interview-complete-flow.browser.mjs13건 통과feedback-polling22,interview-recovery10,interview-recording22,question-audio23 통과interview-persona.browser.mjs는 기준 커밋 92d6cea에서도 같은 위치(설정 화면 "다음" 버튼)에서 실패하는 기존 문제라 이번 변경과 무관🤖 Generated with Claude Code
통합 검증 (2026-10-19)
최신 dev와 #169·#170·#171 통합 후보에서 lint 오류 0(기존 router 경고 1), build 및 단위 테스트 51개 통과. dev 대상 원격 PR 검사 실행은 없어 로컬 검증 결과를 기록합니다. API #106·#109·#108은 main에 병합했으며 전체 테스트 888개 통과했습니다. 실서버 UI 검증 결과는 작업 폴더
Doc/PR-정리-2026-10-19.md에 기록합니다.