[WTH-454] 마이페이지 수정사항 반영

[soo0711]

📌 Summary

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

마이페이지 수정사항 반영

📝 Changes

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

What

프로필 사용할 동아리 목록 조회
동아리별로 내가 쓴 글, 내가 출석한 세션 개수 맞는지 확인
마이페이지 요약 조회

Why

마이페이지 수정사항이 생김

How

프로필 사용할 동아리 목록 api 추가
마이페이지 요약을 clubId path를 추가해서 동아리별 쓴 글, 세션 개수와 현재 사용중인 멀티프로필 응답 추가

📸 Screenshots / Logs

필요시 스크린샷 or 로그를 첨부해주세요.
현재 동아리 마이페이지 요약 조회

-> 동아리별 쓴 글, 세션 개수로 수정, 현재 진입한 동아리에서 사용중인 멀티프로필 응답 추가

프로필 사용할 수 있는 동아리 목록

💡 Reviewer 참고사항

리뷰에 참고할 내용을 작성해주세요.
지원님 생각해보니까 현재 진입한 동아리에서 사용 중인 멀티프로필 정보는 없더라고요! 그래서 마이페이지 요약 응답에 추가해뒀습니다!
마이페이지 진입시에 사용하면 됩니다!

✅ Checklist

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

Summary by CodeRabbit

• 새로운 기능

  • 사용자가 프로필에 사용할 수 있는 활동 중인 동아리 목록을 조회할 수 있습니다.
  • 동아리별 이름, 이미지, 활성 회원 수가 함께 제공됩니다.
  • 동아리별 마이페이지 요약에서 현재 사용 중인 프로필 정보가 표시됩니다.
  • 마이페이지 요약 조회 시 선택한 동아리 기준의 회원 활동 정보가 제공됩니다.

• 변경 사항

  • 프로필 할당 가능 동아리 조회 경로가 새로 제공됩니다.
  • 기존 사용자 마이페이지 요약 조회 경로는 동아리 회원 영역으로 이동되었습니다.

• 테스트

  • 동아리 목록, 프로필 정보, 마이페이지 요약 조회에 대한 검증을 추가했습니다.

[coderabbitai]

📝 Walkthrough

Walkthrough

클럽별 활성 멤버 수 집계와 프로필 할당 가능 클럽 조회 API가 추가되었다. 마이페이지는 clubId 기준으로 활성 멤버와 현재 프로필을 조회하며, 요약 엔드포인트가 클럽 멤버 컨트롤러로 이동했다.

Changes

프로필 할당 가능 클럽 조회

Layer / File(s)	Summary
클럽별 멤버 집계 및 응답 구성 src/main/kotlin/com/weeth/domain/club/domain/repository/*, src/main/kotlin/com/weeth/domain/user/application/dto/response/UserProfileAssignableClubsResponse.kt, src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryService.kt, src/test/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryServiceTest.kt	활성 멤버 수를 클럽별로 집계하고, 활성 클럽·인코딩된 ID·이미지 URL·멤버 수를 응답으로 매핑한다.

현재 프로필 마이페이지 반영

Layer / File(s)	Summary
클럽 기준 마이페이지와 현재 프로필 매핑 src/main/kotlin/com/weeth/domain/user/application/dto/response/UserMyPageResponse.kt, src/main/kotlin/com/weeth/domain/user/application/mapper/UserMyPageMapper.kt, src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserMyPageQueryService.kt, src/test/kotlin/com/weeth/domain/user/application/usecase/query/GetUserMyPageQueryServiceTest.kt	getMyPage가 clubId와 활성 멤버를 사용하고, 현재 프로필 정보와 해당 멤버 기준 통계를 응답하며 이를 테스트한다.

API 엔드포인트 연결

Layer / File(s)	Summary
마이페이지 및 프로필 클럽 API wiring src/main/kotlin/com/weeth/domain/user/presentation/*, src/test/kotlin/com/weeth/domain/user/presentation/*	GET /me/profiles/assignable-clubs를 추가하고 GET /me/mypage를 클럽 멤버 요약 엔드포인트로 대체했으며, 응답 코드와 컨트롤러 테스트를 갱신한다.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Client
  participant UserController
  participant GetUserProfileAssignableClubQueryService
  participant ClubMemberReader
  participant FileAccessUrlPort
  Client->>UserController: GET /me/profiles/assignable-clubs
  UserController->>GetUserProfileAssignableClubQueryService: findAll(userId)
  GetUserProfileAssignableClubQueryService->>ClubMemberReader: 활성 클럽 및 멤버 수 조회
  ClubMemberReader-->>GetUserProfileAssignableClubQueryService: 클럽별 멤버 수
  GetUserProfileAssignableClubQueryService->>FileAccessUrlPort: 이미지 URL 해석
  FileAccessUrlPort-->>GetUserProfileAssignableClubQueryService: 이미지 URL
  GetUserProfileAssignableClubQueryService-->>UserController: UserProfileAssignableClubsResponse
  UserController-->>Client: 성공 응답

Loading

Possibly related PRs

• Team-Weeth/weeth-server#93: 클럽 기준 마이페이지 API와 게시물·출석 세션 조회 흐름을 함께 확장한다.

Suggested labels: ✨ Feature, 📬 API

Suggested reviewers: hyxklee

Poem

깡충 토끼가 클럽을 세고
프로필 당근을 골라 담네
현재 멤버 길을 따라
마이페이지 꽃이 피어요
API도 폴짝, 테스트도 폴짝 🐰

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	PR 제목이 핵심 변경사항인 마이페이지 수정 반영을 간결하게 잘 요약합니다.
Description check	✅ Passed	요약, 변경사항의 What/Why/How, 스크린샷, 참고사항, 체크리스트가 모두 있어 템플릿을 대부분 충족합니다.

✨ 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 fix/WTH-454-마이페이지-수정사항-반영

---

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.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/test/kotlin/com/weeth/domain/user/presentation/UserControllerTest.kt (1)

218-219: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

새로운 엔드포인트에 대한 테스트 누락을 보완해 주세요.

UserController에 새로 추가된 getUserProfileAssignableClubs 엔드포인트에 대한 테스트 블록이 누락되어 있습니다. 컨트롤러 로직 검증과 테스트 커버리지를 위해 해당 테스트를 추가하는 것을 권장합니다.

테스트 추가 제안

             }
         }
+
+        describe("getUserProfileAssignableClubs") {
+            it("프로필을 사용할 수 있는 동아리 목록을 조회한다") {
+                val responseDto = UserProfileAssignableClubsResponse(emptyList())
+                io.mockk.every { getUserProfileAssignableClubQueryService.findAll(1L) } returns responseDto
+
+                val response = controller.getUserProfileAssignableClubs(1L)
+
+                response.code shouldBe UserResponseCode.USER_PROFILE_ASSIGNABLE_CLUBS_FIND_SUCCESS.code
+                response.message shouldBe UserResponseCode.USER_PROFILE_ASSIGNABLE_CLUBS_FIND_SUCCESS.message
+                response.data shouldBe responseDto
+            }
+        }
     })

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/test/kotlin/com/weeth/domain/user/presentation/UserControllerTest.kt`
around lines 218 - 219, 새 엔드포인트 getUserProfileAssignableClubs에 대한 테스트 블록을
UserControllerTest에 추가하세요. UserController의 해당 메서드가 정상 응답과 필요한 요청 조건을 올바르게 처리하는지
기존 테스트 패턴과 픽스처를 재사용해 검증하고, 엔드포인트 동작을 커버하도록 구성하세요.

🧹 Nitpick comments (1)

src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryService.kt (1)

20-24: 🎯 Functional Correctness | 🔵 Trivial | 💤 Low value

중복 클럽 응답 방지를 위한 방어적 로직 추가를 고려해 주세요.

사용자가 동일한 클럽에서 여러 개의 활성 멤버십(예: 다른 역할)을 가질 수 있는 엣지 케이스가 존재할 경우, 응답에 동일한 클럽이 중복해서 포함될 수 있습니다. 이를 방어하기 위해 distinctBy { it.id }를 추가하는 것을 권장합니다.

💡 제안하는 리팩토링

         val clubs =
             clubMemberReader
                 .findAllByUserIdAndMemberStatusWithClub(userId, MemberStatus.ACTIVE)
                 .map { it.club }
+                .distinctBy { it.id }
                 .sortedBy { it.id }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryService.kt`
around lines 20 - 24, Update the club collection transformation in
GetUserProfileAssignableClubQueryService to apply distinctBy using each club’s
id after mapping memberships to clubs, before sorting or returning the results.
Preserve the existing active-membership filtering and ascending id ordering
while ensuring each club appears only once.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@src/test/kotlin/com/weeth/domain/user/presentation/UserControllerTest.kt`:
- Around line 218-219: 새 엔드포인트 getUserProfileAssignableClubs에 대한 테스트 블록을
UserControllerTest에 추가하세요. UserController의 해당 메서드가 정상 응답과 필요한 요청 조건을 올바르게 처리하는지
기존 테스트 패턴과 픽스처를 재사용해 검증하고, 엔드포인트 동작을 커버하도록 구성하세요.

---

Nitpick comments:
In
`@src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryService.kt`:
- Around line 20-24: Update the club collection transformation in
GetUserProfileAssignableClubQueryService to apply distinctBy using each club’s
id after mapping memberships to clubs, before sorting or returning the results.
Preserve the existing active-membership filtering and ascending id ordering
while ensuring each club appears only once.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1eeb4a06-90e5-4e4b-9d1f-46eb76acbc14

📥 Commits

Reviewing files that changed from the base of the PR and between 68298a4 and 8716f84.

📒 Files selected for processing (15)

• src/main/kotlin/com/weeth/domain/club/domain/repository/ClubMemberCount.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/user/application/dto/response/UserMyPageResponse.kt
• src/main/kotlin/com/weeth/domain/user/application/dto/response/UserProfileAssignableClubsResponse.kt
• src/main/kotlin/com/weeth/domain/user/application/mapper/UserMyPageMapper.kt
• src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserMyPageQueryService.kt
• src/main/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryService.kt
• src/main/kotlin/com/weeth/domain/user/presentation/ClubMemberMyPageController.kt
• src/main/kotlin/com/weeth/domain/user/presentation/UserController.kt
• src/main/kotlin/com/weeth/domain/user/presentation/UserResponseCode.kt
• src/test/kotlin/com/weeth/domain/user/application/usecase/query/GetUserMyPageQueryServiceTest.kt
• src/test/kotlin/com/weeth/domain/user/application/usecase/query/GetUserProfileAssignableClubQueryServiceTest.kt
• src/test/kotlin/com/weeth/domain/user/presentation/ClubMemberMyPageControllerTest.kt
• src/test/kotlin/com/weeth/domain/user/presentation/UserControllerTest.kt

[hyxklee]

머지하구 프론트 작업에 따른 후속 수정사항 대응하면 댈 것 같아요!
고생하셨습니다아아