Skip to content

fix: 경기 진행률을 미리보기와 추천 픽 박스에 표시 - #183

Open
choigod1023 wants to merge 1 commit into
mainfrom
codex/restore-live-progress-preview-20260905
Open

fix: 경기 진행률을 미리보기와 추천 픽 박스에 표시#183
choigod1023 wants to merge 1 commit into
mainfrom
codex/restore-live-progress-preview-20260905

Conversation

@choigod1023

Copy link
Copy Markdown
Owner

원인

  • 운영 화면은 점수/이닝과 저장된 픽의 확률만 렌더링하며 경기 진행률 UI는 없었습니다.
  • 기존 내부 확률 추정기의 progress는 별도 표시용 계약이 아니며, 미확인 정보에 50%를 사용하므로 그대로 노출하면 부정확합니다.

수정

  • 공통 LiveProgress를 접힌 경기 미리보기, 펼친 상세, 상단 오늘의 추천 픽에 연결했습니다.
  • 저장된 픽, 사전 확률, 점수 없이도 최신 이닝/시계가 확인되면 진행률을 표시합니다.
  • 야구는 정규 9회의 완료 반이닝, 축구는 정규 90분, 농구는 확인된 4쿼터 단위로 기준을 명시합니다.
  • 연장/세트/정보 누락은 임의의 비율을 만들지 않으며, 개별 행이 10분 넘게 낡으면 마지막 상태와 갱신 지연만 표시합니다.
  • 혼합된 이닝/공격측 보강 자료를 추정에 사용하지 않습니다. 오래된 중단 정보에도 신선도 검증을 적용했습니다.
  • 예측 산식, 사전 픽 원장, DB, 수집기 설정은 변경하지 않았습니다. 프로덕션 docs 빌드 포함.

검증

  • npm ci + npm test: 169 passed
  • python -m pytest -q: 698 passed, 58 subtests passed
  • npm run build: passed
  • 운영 API 동일 스냅샷(2026-09-05 17:17:04 KST)으로 운영 화면과 로컬 프로덕션 빌드 비교: 진행 중 6경기 유지, 미리보기 진행률 0→6, 상단 박스 진행률 0→3.
  • 1280px/390px 브라우저 QA: 상세 열기/닫기, 진행률 표시, 가로 넘침 없음, JS 오류 없음.
  • 리뷰 후 혼합 이닝/낡은 중단 정보 회귀 테스트도 통과.
  • 참고: 기존 lockfile npm audit에 nanoid high 1건이 있으며 이번 표시 변경과 무관해 의존성 변경은 포함하지 않았습니다.

반영 상태

  • PR 단계이며 아직 병합/운영 Pages 배포 전입니다.
  • 프론트 변경만 있어 병합 후 Pages 배포가 필요합니다. Fly 재시작은 필요 없습니다.

@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 2135be6f-d9ac-426a-b7fa-46c399a23a3e


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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