fix: soft-delete 가드 누락 지점 보강 (셀러 조회·리뷰 집계) - #222
Conversation
이슈 #207 전수 조사에서 발견된, 같은 파일 내 형제 필드는 가드가 있는데 빠져 있던 명백한 불일치 지점을 보강한다. - product.repository 상품 조회 3경로(listProductsByStore·findProductById· findProductByIdIncludingInactive): product_categories/product_tags에 링크·대상 soft-delete 가드 추가. is_active는 셀러 화면에서 기존 지정을 계속 보여줘야 하므로 걸지 않는다(명세 외 정책 결정). - product.repository 셀러 옵션·템플릿 read 3곳(findOptionGroupById· listOptionGroupsByProduct·findCustomTemplateById): option_items/ text_tokens 가드 추가. - order.repository 셀러 주문 상세(findOrderDetailByStore): 유저측 상세와 동일하게 status_histories·items·option_items·custom_texts·free_edits· attachments 가드 추가. 셀러 목록·상세의 items.some에도 가드를 얹어 soft-delete 아이템만 남은 주문이 노출되지 않게 한다. - order.repository findReviewableOrderIds: 형제 메서드 (listReviewableOrderItems)와 동일하게 order 관계 필터에 가드 추가. - 테스트 팩토리: createOrder/createOrderItem에 deleted_at override 지원. 회귀 테스트 6건: 상품 상세 카테고리·태그 링크 제외(repository), 옵션 아이템·텍스트 토큰 제외(repository), 셀러 주문 상세 아이템·이력 제외, soft-delete 아이템만 있는 주문의 상세 NotFound·목록 제외(service), soft-delete 주문의 리뷰 가능 집계 제외(repository). 전체 validate green (192 suites / 1,655 tests). Claude-Session: https://claude.ai/code/session_01KfiGgWooJdsa4iPQPmqvBj
|
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: Pro Plus 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 |
🩺 NestJS Doctor — 89/100 (Good)진단 283건 (error 0).
architecture / security 상위 항목
|
🧹 knip — dead-code 리포트전체 리포트
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cd20058197
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| product_categories: { | ||
| // 링크·대상 카테고리의 soft-delete 가드. is_active는 셀러 화면에서 | ||
| // 기존 지정을 계속 보여줘야 하므로 걸지 않는다. | ||
| where: { ...activeWhere, category: activeWhere }, |
There was a problem hiding this comment.
When sellerProducts filters by a category or searches by a tag that is linked only through a soft-deleted ProductCategory/ProductTag (or a deleted target), the top-level some predicates still match because they lack activeWhere. These new include guards then hide the matching relation while the product remains in the results, so deleted taxonomy continues to affect seller filtering and search; apply the same link and target guards to the relation predicates in listProductsByStore.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
반영: listProductsByStore의 categoryId·search 태그 predicate에도 include와 동일한 링크·대상 soft-delete 가드 추가. 회귀 테스트 1건 동반.
Coverage report
Test suite run success1656 tests passing in 191 suites. Report generated by 🧪jest coverage report action from da3725d |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Codex 리뷰 반영(PR #222): listProductsByStore의 categoryId·search 태그 predicate가 include와 달리 무가드라, 삭제된 연결만 가진 상품이 필터에 걸린 채 연결은 숨겨지는 불일치가 있었다. include와 동일한 가드를 적용하고 회귀 테스트 1건(삭제된 연결·태그로는 목록 필터 미매칭)을 추가한다. Claude-Session: https://claude.ai/code/session_01KfiGgWooJdsa4iPQPmqvBj
Codex 리뷰 반영(PR #222): listProductsByStore의 categoryId·search 태그 predicate가 include와 달리 무가드라, 삭제된 연결만 가진 상품이 필터에 걸린 채 연결은 숨겨지는 불일치가 있었다. include와 동일한 가드를 적용하고 회귀 테스트 1건(삭제된 연결·태그로는 목록 필터 미매칭)을 추가한다.
fix: soft-delete 가드 누락 지점 보강 (셀러 조회·리뷰 집계)
1 participant
무엇을 왜
#221의 전수 조사에서 soft-delete 가드가 빠진 지점들이 나왔습니다. 이 PR은 그중 "같은 파일에서 형제 필드는 가드하는데 이것만 안 한다"가 명백한 곳만 고칩니다. 예컨대 상품 상세 include에서 images와 option_groups는 걸러주면서 product_categories는 안 거르는 식인데, 이건 정책이 아니라 누락입니다. 반면 유저 본인 리뷰 목록에 삭제된 매장/상품이 보이는 건(to-one 무필터) 이력 표시라는 의도일 수 있어 정책 판단으로 넘겼고, mutation 결과 include는 방금 쓴 행이라 실피해가 낮아 함께 보류했습니다. 둘 다 #207에 기록해 뒀습니다.
고친 곳
product_categories/product_tags에 링크와 대상 양쪽의 가드를 추가했습니다. 삭제된 카테고리에 연결돼 있다는 이유로 상품 화면에 유령 카테고리가 뜨던 경로입니다.option_items/text_tokens가드. 같은 파일의 구매자 노출용 include에는 전부 가드가 있었습니다.items.some에도 가드를 얹어 soft-delete 아이템만 남은 주문이 목록에 뜨거나 상세가 열리는 일이 없게 했습니다.카테고리/태그에서 한 가지 판단이 있었습니다. 가드는
deleted_at만 걸고is_active는 걸지 않았는데, 셀러 화면은 관리자가 카테고리를 비활성화해도 자기 상품의 기존 지정을 계속 볼 수 있어야 하기 때문입니다. 명세에 없는 결정이라 코드 주석에도 남겼습니다.리뷰에서 하나 더
Codex가 유효한 빈틈을 짚었습니다. include만 가드하면 셀러 목록에서 "삭제된 연결로 필터에는 걸리는데 정작 그 연결은 화면에서 숨겨지는" 반쪽 상태가 된다는 것.
listProductsByStore의 categoryId 필터와 태그 검색 predicate에도 같은 가드를 적용해 마무리했습니다.검증
각 수정에 회귀 테스트를 붙여 총 7건입니다 — 카테고리·태그 링크/대상 제외와 삭제된 연결의 필터 미매칭(repository), 옵션 아이템·텍스트 토큰 제외(repository), 셀러 주문 상세의 아이템·이력 제외와 soft-delete 아이템만 남은 주문의 상세 NotFound·목록 제외(service), soft-delete 주문의 리뷰 가능 집계 제외(repository). 테스트를 위해 주문 팩토리에
deleted_atoverride를 추가했습니다.yarn validate전체 green입니다(192 suites / 1,656 tests).Refs #207