feat/#80 ootd 및 스타일 기반 추천 수정#81
Conversation
jychoi0831
left a comment
There was a problem hiding this comment.
현재 head commit 201feb2 기준으로 확인했습니다.
리뷰 결과: COMMENT
Findings
P1. 최신 develop 기준 merge conflict가 남아 있습니다
GitHub mergeable이 CONFLICTING이고, 로컬에서도 git merge-tree --write-tree b29aeda23ec6bf5144cfab4d4010957b417b2943 201feb2bc3ce3b476399c65f62c12ed4663d7dce가 src/components/HomeTab.tsx content conflict로 실패했습니다. 이 상태에서는 develop에 병합할 수 없고, 충돌 해소 후 다시 검증해야 합니다.
P1. npm run build가 TypeScript 오류로 실패합니다
src/components/HomeTab.tsx:211에서 it.shoes?.clothesId를 읽지만, 같은 파일의 RecommendItem 타입에는 shoes 필드가 선언되어 있지 않습니다. 실제 검증 결과 src/components/HomeTab.tsx(211,12): error TS2339: Property 'shoes' does not exist on type 'RecommendItem'.로 tsc -b가 중단됩니다. OOTD 조합에 신발을 포함하려면 RecommendItem 타입에도 shoes를 반영하거나, 별도 OOTD combo 타입으로 분리해야 합니다.
Validation
- PASS:
npm ci --offline - PASS:
git diff --check b29aeda23ec6bf5144cfab4d4010957b417b2943...201feb2bc3ce3b476399c65f62c12ed4663d7dce -- - FAIL:
git merge-tree --write-tree b29aeda23ec6bf5144cfab4d4010957b417b2943 201feb2bc3ce3b476399c65f62c12ed4663d7dce(src/components/HomeTab.tsxconflict) - PASS:
npm run lint(0 errors, 20 warnings) - FAIL:
npm run build(TypeScript 오류) - GitHub CI: 등록된 status check 없음
Sync 확인
- 대상 레포: BE
- Sync 이슈 기록: 불필요
- 사유: 이번 PR은 FE 화면/타입 내부 변경이며 BE API 계약, docs, mock 기준 변경을 새로 만들지 않습니다.
본 리뷰는 Codex를 사용해 작성했습니다.
…BE5_FinalProject_Team4_FE into feat/#80 # Conflicts: # src/components/HomeTab.tsx
jychoi0831
left a comment
There was a problem hiding this comment.
현재 head commit 67ba0b6 기준으로 확인했습니다.
리뷰 결과: COMMENT
Findings
P2. 추천 API 실패 후 다시 시도 버튼이 실제 요청을 재실행하지 않습니다
src/components/HomeTab.tsx:657-660의 에러 상태 버튼은 onRefreshWardrobe?.()만 호출합니다. 그런데 OOTD/style 추천을 다시 가져오는 effect는 src/components/HomeTab.tsx:397 기준으로 [activeLabel, userId, authReady, refreshSignal]에만 의존하고, 이 버튼은 refreshSignal을 증가시키거나 ootdError/styleError를 clear하지 않습니다. 따라서 OOTD나 스타일 추천 API가 일시 실패하면 사용자가 화면의 다시 시도를 눌러도 fetchOotdRecommendations/fetchWardrobeRecommendations가 재호출되지 않아 에러 상태에서 복구할 수 없습니다. 버튼에서 해당 에러를 초기화하고 setRefreshSignal을 증가시키거나, 추천 fetch 함수를 명시적으로 재실행하도록 수정해야 합니다.
확인한 이전 지적 사항
- 해소 확인: 최신
develop기준 merge conflict가 사라졌습니다. - 해소 확인:
RecommendItem.shoes누락으로 인한 TypeScript build 오류가 사라졌습니다.
Validation
- PASS:
git diff --check b29aeda23ec6bf5144cfab4d4010957b417b2943...67ba0b691187cd567406a9366c9d03f622bdbf2b -- - PASS:
git merge-tree --write-tree b29aeda23ec6bf5144cfab4d4010957b417b2943 67ba0b691187cd567406a9366c9d03f622bdbf2b(221de6df1b477eacbd83fd6577dc0d331531179f) - PASS:
npm ci --offline - PASS:
npm run lint(0 errors, 36 warnings) - PASS:
npm run build - PASS: GitHub CI
Lint & Build
Sync 확인
- 대상 레포: BE
- Sync 이슈 기록: 불필요
- 사유: 이번 변경은 FE 화면 복구 흐름과 OOTD 표시 로직 내부 변경이며 BE API 계약, docs, mock 기준 변경을 새로 만들지 않습니다.
본 리뷰는 Codex를 사용해 작성했습니다.
jychoi0831
left a comment
There was a problem hiding this comment.
현재 head commit 2f2a8f1 기준으로 확인했습니다.
리뷰 결과: APPROVE
Summary
- 이전 지적 사항 확인: merge conflict,
RecommendItem.shoesbuild 오류, 추천 API 실패 후다시 시도미동작 문제가 모두 해소됐습니다. - 추가 merge-blocking finding은 없습니다.
Validation
- PASS:
git diff --check b29aeda23ec6bf5144cfab4d4010957b417b2943...2f2a8f1495ddefcdbbac70b74175579eb15fa53a -- - PASS:
git merge-tree --write-tree b29aeda23ec6bf5144cfab4d4010957b417b2943 2f2a8f1495ddefcdbbac70b74175579eb15fa53a(5886b32f588ef3edabb81c16423d772b6116a41c) - PASS:
npm ci --offline - PASS:
npm run lint(0 errors, 36 warnings) - PASS:
npm run build - PASS: GitHub CI
Lint & Build
Sync 확인
- 대상 레포: BE
- Sync 이슈 기록: 불필요
- 사유: 이번 변경은 FE 화면 복구 흐름과 OOTD 표시 로직 내부 변경이며 BE API 계약, docs, mock 기준 변경을 새로 만들지 않습니다.
본 리뷰는 Codex를 사용해 작성했습니다.
📌 관련 이슈
Closes #80
🛠️ 작업 내용
✅ 변경 사항
🔍 테스트 내용
📷 스크린샷 (선택사항)
💬 리뷰어에게
📋 PR 체크리스트
develop브랜치를 base로 설정했나요?