[WTH-203] 동아리 개설/가입 수 제한

[hyxklee]

📌 Summary

어떤 작업인지 한 줄 요약해 주세요.

MVP 동아리 개설/가입 수 제한 정책을 수립했습니다.

📝 Changes

변경사항을 what, why, how로 구분해 작성해 주세요.

What

동아리 개설/가입 수 제한 정책 수립

Why

무분별한 생성과 가입을 막기 위해

How

동아리 생성 1개, 가입 1개로 정책 수립
MVP만 유지

📸 Screenshots / Logs

필요시 스크린샷 or 로그를 첨부해주세요.

💡 Reviewer 참고사항

리뷰에 참고할 내용을 작성해주세요.

일단은 MVP에서만 유효한 설정입니다. 추후 BM 설정에 따라 개수를 바꾸는 식으로 변경될 수도 있습니다.

✅ Checklist

• PR 제목 설정 완료 (WTH-123 인증 필터 설정)
• 테스트 구현 완료
• 리뷰어 등록 완료
• 자체 코드 리뷰 완료

Summary by CodeRabbit

• New Features

  • 동아리 생성(LEAD) 및 가입(USER)에 대해 역할 기반으로 각각 최대 1개로 제한합니다.
  • 생성/가입 시 제한을 사전검증하여 초과 시 명확한 충돌 오류를 반환합니다.
  • 생성 흐름에서 사용자 조회 시 잠금을 적용해 정책 검사 전에 처리합니다.

• Tests

  • 제한 정책 관련 단위/통합 테스트를 추가·갱신해 실패·성공 시나리오를 검증합니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8a20e695-7284-4643-b023-28a8872704dc

📥 Commits

Reviewing files that changed from the base of the PR and between 77027d4 and 2c85251.

📒 Files selected for processing (8)

• src/main/kotlin/com/weeth/domain/club/application/exception/ClubErrorCode.kt
• src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubMemberUsecase.kt
• src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCase.kt
• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberReader.kt
• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberRepository.kt
• src/main/kotlin/com/weeth/domain/club/domain/service/ClubMemberPolicy.kt
• src/test/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubMemberUseCaseTest.kt
• src/test/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCaseTest.kt

🚧 Files skipped from review as they are similar to previous changes (6)

• src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCase.kt
• src/test/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubMemberUseCaseTest.kt
• src/main/kotlin/com/weeth/domain/club/domain/service/ClubMemberPolicy.kt
• src/main/kotlin/com/weeth/domain/club/application/exception/ClubErrorCode.kt
• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberReader.kt
• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberRepository.kt

---

📝 Walkthrough

Walkthrough

동아리 도메인에 가입 및 생성 제한 로직을 도입했습니다. 새로운 예외/에러 코드, 멤버 집계 카운트 메서드, 정책 검증(validateJoinLimit/validateCreateLimit)을 추가하고, 유스케이스에서 해당 정책을 호출하도록 수정했습니다.

Changes

Cohort / File(s)	Summary
Exception Classes src/main/kotlin/com/weeth/domain/club/application/exception/ClubCreateLimitExceededException.kt, src/main/kotlin/com/weeth/domain/club/application/exception/ClubJoinLimitExceededException.kt	새 예외 추가(ClubCreateLimitExceededException) 및 가입 예외명 변경(기존 ClubCantJoinException → ClubJoinLimitExceededException).
Error Code Enum src/main/kotlin/com/weeth/domain/club/application/exception/ClubErrorCode.kt	CLUB_CANT_JOIN 제거; CLUB_JOIN_LIMIT_EXCEEDED(21110) 및 CLUB_CREATE_LIMIT_EXCEEDED(21111) 추가(HTTP CONFLICT).
Domain Repository / Reader src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberReader.kt, src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberRepository.kt	사용자/상태/역할로 필터링한 멤버 수를 반환하는 countByUserIdAndMemberStatusAndMemberRole(...) 메서드 추가(Repo에 쿼리 포함).
Domain Policy src/main/kotlin/com/weeth/domain/club/domain/service/ClubMemberPolicy.kt	validateJoinLimit(userId) 및 validateCreateLimit(userId) 추가; 내부 상수 MAX_USER_CLUBS = 1, MAX_LEAD_CLUBS = 1; 카운트 기반으로 예외 던짐.
Use Cases src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubMemberUsecase.kt, src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCase.kt	가입 시 clubMemberPolicy.validateJoinLimit() 호출로 기존 체크 대체; 생성 시 userReader.getByIdWithLock(...) 사용 및 clubMemberPolicy.validateCreateLimit() 호출 추가.
Tests src/test/kotlin/com/weeth/domain/club/.../ManageClubMemberUseCaseTest.kt, .../ManageClubUseCaseTest.kt, .../ClubMemberPolicyTest.kt	정책 기반 예외 검증을 반영하도록 테스트 수정/추가(예외 타입 변경, 정책 모킹, 신규 실패/성공 시나리오).

Sequence Diagram(s)

sequenceDiagram
    participant Client as Client
    participant UseCase as ManageClubUseCase
    participant Policy as ClubMemberPolicy
    participant Repo as ClubMemberRepository
    participant DB as Database

    Client->>UseCase: 요청 (create/join)
    UseCase->>Repo: userReader.getByIdWithLock(userId) / getById(userId)
    UseCase->>Policy: validateCreateLimit(userId) / validateJoinLimit(userId)
    Policy->>Repo: countByUserIdAndMemberStatusAndMemberRole(userId, status, role)
    Repo->>DB: SELECT COUNT(...)
    DB-->>Repo: count
    Repo-->>Policy: count
    alt count >= MAX
        Policy-->>UseCase: throw LimitExceededException
        UseCase-->>Client: 409 Conflict (예외)
    else count < MAX
        UseCase->>Repo: save club / save clubMember
        Repo->>DB: INSERT ...
        DB-->>Repo: ok
        Repo-->>UseCase: saved
        UseCase-->>Client: 성공 응답
    end

Loading

🎯 Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

📌 Possibly Related PRs

• [WTH-189] club 추가 마무리 작업 #25 — ClubMemberPolicy와 멤버 제한/예외 관련 변경을 다룬 PR로 코드 레벨 연관성 높음.
• [WTH-176] club 도메인 엔티티 추가 작업 #21 — 클럽 도메인과 저장소/타입 도입 관련 변경을 포함, 본 PR이 활용하는 기초 변경과 연결됨.
• [WTH-181] club 도메인 추가 작업 2 #22 — ClubErrorCode 및 정책/예외 관련 수정 이력과 겹치는 파일을 수정함.

👥 Suggested Reviewers

• JIN921
• dalzzy
• woneeeee
• soo0711

🐰 Poem

🐇 동아리 문을 지키네, 한 걸음씩,
가입은 셈해 보고, 창설도 한정,
정책이 말하네: "한도는 하나!",
예외 휙 던지고, 테스트가 박수,
당근처럼 튀어나온 안정의 코드.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 44.44% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 동아리 개설/가입 수 제한이라는 주요 변경사항을 명확하게 요약하고 있으며, 작업번호 WTH-203를 포함하여 구체적이고 간결합니다.
Description check	✅ Passed	PR 설명은 템플릿의 모든 주요 섹션(Summary, What/Why/How, 체크리스트)을 포함하고 있으며, 작업 내용과 의도가 명확하게 기술되어 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/WTH-203-동아리-생성-수-제한

📝 Coding Plan

• Generate coding plan for human review comments

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCase.kt`:
- Around line 39-43: Acquire and hold the per-user lock before running the
create-limit check to avoid TOCTOU: obtain the user lock for userId, then call
clubMemberPolicy.validateCreateLimit(userId), then load the user with
userReader.getById(userId), and finally release the lock (ensure release in a
try/finally or equivalent). Update the flow in ManageClubUseCase so the lock is
taken prior to validateCreateLimit and always released after the validation and
subsequent use of user to serialize the policy upper bound.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 35219ae0-cdc8-4aa0-be79-389dd69e6190

📥 Commits

Reviewing files that changed from the base of the PR and between 76eee80 and 77027d4.

📒 Files selected for processing (11)

• src/main/kotlin/com/weeth/domain/club/application/exception/ClubCreateLimitExceededException.kt
• src/main/kotlin/com/weeth/domain/club/application/exception/ClubErrorCode.kt
• src/main/kotlin/com/weeth/domain/club/application/exception/ClubJoinLimitExceededException.kt
• src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubMemberUsecase.kt
• src/main/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCase.kt
• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberReader.kt
• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberRepository.kt
• src/main/kotlin/com/weeth/domain/club/domain/service/ClubMemberPolicy.kt
• src/test/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubMemberUseCaseTest.kt
• src/test/kotlin/com/weeth/domain/club/application/usecase/command/ManageClubUseCaseTest.kt
• src/test/kotlin/com/weeth/domain/club/domain/service/ClubMemberPolicyTest.kt

[soo0711]

수고하셧습니다!! 코드래빗이 말한 동시성 이슈만 체크하면 될 것 같아요!