Skip to content

fix(user): wishlist 가시성 일관성 + 목록 정렬 안정화 (PR #81 리뷰 반영)#82

Merged
chanwoo7 merged 1 commit intodevelopfrom
fix/coderabbit-codex-pr81-feedback
Apr 29, 2026
Merged

fix(user): wishlist 가시성 일관성 + 목록 정렬 안정화 (PR #81 리뷰 반영)#82
chanwoo7 merged 1 commit intodevelopfrom
fix/coderabbit-codex-pr81-feedback

Conversation

@chanwoo7
Copy link
Copy Markdown
Member

Summary

PR #81 (develop→main 릴리즈)에 달린 Codex/CodeRabbit 리뷰 2건 반영.
머지 후 develop 갱신 → PR #81에 자동 포함됨.

변경 사항

Codex P2: findWishlistedProductIds 가시성 일관성

  • 매핑용 IN 쿼리도 visibleWishlistWhere helper를 사용하도록 통일
  • 이전 fix(PR feat(user): 찜 토글 + isWishlisted 노출 + 카운트 일관성 수정 #80)에서 myWishlist/wishlistCount/findWishlistItems 세 곳을 통일했으나, 이 메서드만 누락되어 있던 상태
  • 효과: 비활성 store에 속한 product가 recent-view에서 isWishlisted=true로 보이지만 myWishlist에는 안 보이는 모순 회피

CodeRabbit Major: findWishlistItems orderBy tie-breaker

  • orderBy에 product_id desc 보조 정렬키 추가
  • 같은 밀리초에 생성된 항목의 페이지 경계 흔들림 방지

회귀 테스트

  • recent-view spec 1건 추가: 비활성 store의 product에 wishlist가 있어도 isWishlisted=false로 매핑됨 (가시성 일관)

Breaking 여부

  • ❌ Breaking 아님. 가시성 조건 좁히기와 정렬 보조키 추가뿐
  • 마이그레이션: 없음

Test plan

  • 로컬 yarn test:cov 통과 (868 tests, +1)
  • CI check 통과
  • CI coverage-report 통과
  • CodeQL 통과

PR #81 develop→main 릴리즈에 달린 Codex/CodeRabbit 리뷰 반영.

- Codex P2: findWishlistedProductIds도 visibleWishlistWhere 사용하여
  myWishlist/wishlistCount와 동일한 가시성 기준 공유. 비활성 store에 속한
  product가 recent-view에서 isWishlisted=true로 보이지만 myWishlist에는
  안 보이는 모순 회피.
- CodeRabbit Major: findWishlistItems orderBy에 product_id 보조 정렬키 추가.
  같은 밀리초에 생성된 항목의 페이지 경계 흔들림 방지.
- 회귀 테스트 1건: 비활성 store의 product에 wishlist가 있어도
  recent-view list에서 isWishlisted=false로 매핑됨 (가시성 일관)
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: cb6e558a-0700-426d-9bd7-69be4d9207be

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/coderabbit-codex-pr81-feedback

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 and usage tips.

@github-actions
Copy link
Copy Markdown

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 96.69% 2894/2993
🟢 Branches 86.5% 1647/1904
🟢 Functions 93.32% 615/659
🟢 Lines 97.02% 2638/2719

Test suite run success

868 tests passing in 77 suites.

Report generated by 🧪jest coverage report action from 4c35731

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@chanwoo7 chanwoo7 merged commit 97af31f into develop Apr 29, 2026
10 checks passed
@chanwoo7 chanwoo7 deleted the fix/coderabbit-codex-pr81-feedback branch April 29, 2026 23:02
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