Skip to content

fix: 종합 피드백 평가 축과 면접 유형별 섹션 표시 수정 (#168) - #169

Merged
danjee2009 merged 2 commits into
devfrom
fix/#168-feedback-metrics-visibility
Oct 19, 2026
Merged

danjee2009 merged 2 commits into
devfrom
fix/#168-feedback-metrics-visibility

Conversation

@danjee2009

@danjee2009 danjee2009 commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

변경 내용

종합 피드백의 깊이·구체성·정확성이 항상 분석 대기로 보이고, SOLO의 면접관 코멘트와 TEXT의 음성 영역이 노출되던 문제를 수정합니다.

  • API의 scoreBreakdown을 축 이름으로 매핑합니다. 0점·해당 없음·미제공을 구분하며 종합 점수와 기존 채점은 유지합니다.
  • 면접관 코멘트는 MULTI에만 렌더링합니다. SOLO는 빈 안내와 여백까지 제외합니다.
  • TEXT는 음성 영역과 조회를 생략합니다. VOICE는 녹음 실패에도 상태를 보여 줍니다. 새 답변 방식을 생성 요청·조회·재진입에 보존합니다.
  • 답변 방식 없는 옛 기록은 녹음 유무로 표시를 보완합니다. 조회 오류는 TEXT로 단정하지 않으며 재확인을 제공합니다. PENDING만 5초·최대 5분 조회하고 이탈·ID 변경 시 취소합니다.
  • 실제 점수 API가 없는 음성 지표를 고정 분석 대기로 만들지 않고 분석 상태와 준비 중 안내로 표시합니다. 키워드·신뢰성·PDF는 유지합니다.
  • 기존 UNKNOWN/SUBMITTING 제출의 본문·키는 답변 방식 주입 없이 재전송합니다.

관련 이슈

테스트 방법

  • 단위 테스트 추가/수정: node --test tests/*.test.mjs 40 passed
  • 통합 테스트 완료: 독립 HTTP fixture + 실제 브라우저에서 네 조합, 새로고침, 옛 기록, 조회 실패/재확인, PENDING→READY 및 화면 이탈 확인
  • 수동 테스트 완료: SOLO TEXT PDF 내보내기 후 Poppler 렌더링 검수

npm run lint: 오류 0, 기존 router.tsx unused eslint-disable 경고 1개. npm run build 통과.

Fixture의 네 축 100/60/95/100·종합 89점 확인. TEXT 음성 요청 0회, VOICE 1회, 오류 재확인 2회, PENDING→READY 2회, 이탈 후 폴링 중단을 요청 기록으로 확인했습니다. PDF에서도 숨김 조건과 나머지 결과 유지, 겹침·잘림 없음을 확인했습니다.

스크린샷 (UI 변경 시)

Before

의도 충족 외 세 축이 null 고정으로 “분석 대기”. SOLO와 TEXT에도 빈 면접관 코멘트·음성 영역이 표시됩니다.

After

수정한 실제 페이지에 API fixture를 연결한 화면입니다. 실모델 결과로 제시하지 않습니다.

SOLO TEXT: 실제 축 값 연결과 유형별 숨김

같은 조건으로 내보낸 PDF 렌더링

체크리스트

  • 코드가 프로젝트의 코딩 스타일을 따릅니다
  • 자체 코드 리뷰를 완료했습니다
  • 변경 사항에 대한 테스트를 추가했습니다
  • 문서를 업데이트했습니다: docs/feedback-visibility.md
  • Breaking Changes가 없습니다: 신규 필드는 선택이며 기존 제출 본문을 보존합니다

기타 사항

선행 Web #165·#167 병합 후 base를 dev로 변경하고 최신 dev를 통합했습니다. lint/build·단위 40개 재통과. API #108과 함께 로컬에 적용해 실연동 검증을 진행합니다.

실제 서버를 연결한 별도 포트에서 기존 56번 면접을 열었으나 로그인 화면으로 이동해 수정 후 실서버 재면접·실모델 연동은 미검증입니다. localhost:5173의 코드·실행 프로세스와 기존 DB는 변경하지 않았습니다.

한계: 답변 방식이 없는 옛 VOICE 기록에서 녹음 업로드가 전부 실패했다면 TEXT 기록과 구분할 수 없습니다. 음성 점수·코멘트 본구현, 목록 정렬·문항 수·완료 후 이동·피드백 문항 번호 수정은 이번 범위 밖입니다.

@danjee2009
danjee2009 changed the base branch from feature/#166-company-dataset to dev October 19, 2026 04:38
@danjee2009
danjee2009 marked this pull request as ready for review October 19, 2026 04:38
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-19T04:40:47.924183Z cbc4c24 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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.

1 participant