refactor(prisma): soft-delete 활성 필터 공용 where 조각으로 중앙화

[chanwoo7]

무엇을 왜

soft-delete extension이 자동으로 deleted_at: null을 넣어주는 건 루트 READ 쿼리뿐입니다. nested relation, relation 필터(some/is), mutation, raw SQL은 전부 손으로 명시해야 하고, 실제로 이 지점에서 회귀가 반복돼 CLAUDE.md에 경고까지 박혀 있었습니다. 주의력으로 버티는 구조는 코드가 늘수록 확률적으로 뚫리니, 인라인 리터럴을 공용 조각으로 바꿔 한곳에서 통제하도록 했습니다(#207 단기 단계).

작업 전에 repository/service 전체의 deleted_at 277라인을 전수 분류했습니다.

분류	건수	이번 처리
루트 where 중복 명시 (extension이 어차피 주입)	59	제거
updateMany 루트 (extension 미커버)	14	조각 교체 (복원 조건 not: null 2건은 유지)
nested include/select 내부 where	54	조각 교체
relation 필터 (some/is 등)	46	조각 교체
raw SQL	25	제외 — 관용구가 이미 일관, Prisma 7(#202) 때 재검토
기타 (mutation data · 타입 · 주석)	79	무변경

공용 조각과 새 컨벤션

src/prisma/active-where.ts에 activeWhere({ deleted_at: null })와 visibleWhere({ is_active: true, deleted_at: null }, 노출 플래그 보유 모델용) 둘을 두고 @/prisma 배럴로 내보냈습니다. 가장 많이 반복되던 store: { is_active: true, deleted_at: null } 같은 리터럴(11회)이 store: visibleWhere 한 단어로 줄어듭니다.

컨벤션은 이렇게 바뀝니다: 루트 READ는 extension을 신뢰해 명시하지 않고, 미커버 경로는 반드시 조각을 명시한다. 중복 명시 59곳을 지운 것도 이 원칙의 일부입니다. 예외로 publicReviewWhere · visibleWishlistWhere 같은 공유 where 헬퍼는 루트 가드를 지우지 않고 조각 조합으로 유지했는데, 이 헬퍼들이 나중에 nested 위치에서 재사용되면 그때는 extension이 못 지켜주기 때문입니다. CLAUDE.md의 soft-delete 안내도 새 컨벤션으로 교체했습니다.

덤으로 잡은 것: Region

deleted_at 컬럼을 가진 모델이 스키마에 50개인데 extension 등록 목록에는 49개뿐이었습니다. 빠진 하나가 Region — 이력을 추적해 보니 의도적 제외가 아니라 모델을 나중에 추가하면서 목록 갱신을 빠뜨린 것이었습니다. 다행히 Region 루트 조회 4곳이 전부 수동 필터를 하고 있어서 사고는 없었고, 목록에 등록해도 동작이 변하지 않음을 확인하고 추가했습니다. 같은 실수가 재발하면 이제 테스트가 잡습니다 — dmmf에서 deleted_at 보유 모델을 뽑아 등록 목록과 diff가 0임을 단언하는 대조 테스트를 넣었습니다.

검증

동작 변경이 0이어야 하는 PR이라 기존 spec 전체의 무변경 통과가 곧 증명입니다. yarn validate green — 191 suites / 1,649 tests(기존 1,647 전부 무변경, dmmf 대조 2건 신규). 전수 조사에서 함께 발견된 가드 누락(형제 필드는 가드가 있는데 일부만 빠진 곳들)은 동작이 바뀌는 수정이라 별도 fix PR로 분리했습니다.

Refs #207

[coderabbitai]

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 Plus

Run ID: b9c25fb2-4f2f-4883-a749-b06e9d3a2784

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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

🧹 knip — dead-code 리포트

Unused exports (2)

전체 리포트

Unused exports (2)
RegionRepository  src/features/region/index.ts:3:10
StoreRepository   src/features/store/index.ts:3:10

청소 후보(오탐 가능) · 기준 docs/guide/architecture-conventions.md

[github-actions]

🩺 NestJS Doctor — 89/100 (Good)

진단 283건 (error 0).

Category	error	warning	info
architecture	0	0	13
correctness	0	130	0
performance	0	24	17
schema	0	0	86
security	0	13	0

architecture / security 상위 항목

• info architecture/architecture/no-barrel-export-internals: Barrel file re-exports internal type 'IAuditLogRepository'.
• warning security/security/no-exposed-env-vars: Direct 'process.env.NODE_ENV' access in 'AuthController'. Use ConfigService instead.
• warning security/security/require-guards-on-endpoints: Endpoint 'start' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'callback' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'refresh' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'logout' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'sellerLogin' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'sellerRefresh' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'sellerLogout' has no @UseGuards() at class or method level.
• warning security/security/require-guards-on-endpoints: Endpoint 'devIssueToken' has no @UseGuards() at class or method level.
• info architecture/architecture/no-barrel-export-internals: Barrel file re-exports internal module '@/features/conversation/repositories/conversation.repository'.
• info architecture/architecture/no-barrel-export-internals: Barrel file re-exports internal type 'ConversationRepository'.
• info architecture/architecture/no-barrel-export-internals: Barrel file re-exports internal module '@/features/order/repositories/order.repository'.
• info architecture/architecture/no-barrel-export-internals: Barrel file re-exports internal type 'OrderRepository'.
• info architecture/architecture/no-barrel-export-internals: Barrel file re-exports internal module '@/features/product/repositories/product.repository'.

오탐 포함 가능 · 기준 docs/guide/architecture-conventions.md

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟢	Statements	97.66%	5013/5133
🟢	Branches	93.96%	2037/2168
🟢	Functions	96.22%	967/1005
🟢	Lines	98.21%	4564/4647

Test suite run success

1649 tests passing in 191 suites.

Report generated by 🧪jest coverage report action from 39cbe9f

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!