fix: 릴리즈 리뷰 반영 (계약 설명 3건 정정) - #292
Conversation
전부 코드의 일부만 보고 적어 실제 계약과 어긋난 설명이다. 동작은 바꾸지 않는다.
- sellerConversationUpdated — "활성 매장"이라 적었으나 findStoreBySellerAccount는
activeWhere({ deleted_at: null })만 쓴다. is_active를 포함하는 건 visibleWhere다
(prisma/active-where.ts:15,18). 비활성 매장도 구독이 열린다.
코드가 아니라 설명을 고친 이유: 매장을 비활성으로 돌려도 진행 중인 고객 문의는
계속 받아야 한다. is_active를 넣으면 노출만 잠시 끈 사이 메시지를 놓친다.
- SellerProductListInput.search — "상품명 부분일치"만 적었으나 OR로 상품명과
product_tags.tag.name을 함께 본다 (product.repository.ts:145-157)
- SellerAuditLog.storeId — "비밀번호 변경 등이면 null"이라 적었으나
seller-credential.service.ts:205가 store?.id를 넣어, 매장을 가진 판매자의
비밀번호 변경 로그는 storeId가 채워진다. 이 로그를 잘못 분류하게 만드는 설명이었다
전체 1,874건 통과.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
🧹 knip — dead-code 리포트전체 리포트
|
🩺 NestJS Doctor — 90/100 (Excellent)진단 313건 (error 0).
architecture / security 상위 항목
|
Coverage report
Test suite run success1874 tests passing in 225 suites. Report generated by 🧪jest coverage report action from 2659a4d |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
릴리즈 PR #289 리뷰 추가분. 동작은 바꾸지 않고 설명만 실제 계약에 맞춘다.
셋 다 원인이 같다 — 코드의 일부만 보고 적었다.
sellerConversationUpdatedactiveWhere는{ deleted_at: null }뿐.is_active를 포함하는 건visibleWhere(prisma/active-where.ts:15,18) → 비활성 매장도 구독 열림SellerProductListInput.searchOR로 상품명 + 태그명을 함께 본다 (product.repository.ts:145-157)SellerAuditLog.storeIdstore?.id를 넣어 매장 보유 판매자면 채워진다 (seller-credential.service.ts:205)첫 건에서 코드가 아니라 설명을 고친 이유
is_active를 조건에 넣는 선택지도 있었지만, 매장을 비활성으로 돌려도 진행 중인 고객 문의는 계속 받아야 한다. 노출만 잠시 끈 사이 메시지를 놓치는 쪽이 더 나쁘다. 지금 동작이 맞고 내 설명이 틀렸다.검증
전체 225 suites 1,874건 통과.
tsc·docs:check통과.