Feat : 마이페이지 참여중인 파티 조회

[milowon]

📝 상세 내용

• 마이페이지 참여중인 파티 조회

---

🔗 관련 이슈

• Closes [FEAT] 마이페이지 참여중인 파티 조회 #49

Summary by CodeRabbit

릴리스 노트

• 새 기능

  • 사용자 위치 정보 업데이트 API 추가
  • 계정 삭제 API 추가
  • 활성 파티 목록 조회 기능 추가
  • 사용자 프로필 엔드포인트 응답 형식 업데이트

• 문서

  • API 엔드포인트에 Swagger/OpenAPI 문서 추가로 API 명세 개선

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

마이페이지에서 사용자가 참여 중인 파티 목록을 조회할 수 있는 기능을 추가합니다. PartyParticipantRepository에 활성 파티 조회 및 상태별 카운팅 메서드를 추가하고, UserService에 getActiveParties 메서드를 구현합니다. UserController에 새 엔드포인트 GET /api/v1/user/parties/active를 추가하고, PartyResponse DTO를 도입하여 파티 정보를 응답합니다. 사용자의 인증 정보를 기반으로 활성 및 승인된 상태의 파티만 필터링하여 반환합니다.

Sequence Diagram

sequenceDiagram
    actor Client
    participant UC as UserController
    participant US as UserService
    participant PPR as PartyParticipantRepository
    participant DB as Database
    
    Client->>UC: GET /api/v1/user/parties/active
    activate UC
    UC->>US: getActiveParties(userId)
    deactivate UC
    
    activate US
    US->>PPR: findActivePartiesByUserId(userId, RECRUITING, APPROVED)
    deactivate US
    
    activate PPR
    PPR->>DB: Query PartyParticipant with filters
    DB-->>PPR: List<PartyParticipant>
    deactivate PPR
    
    activate US
    loop For each PartyParticipant
        US->>PPR: countByPartyIdAndStatus(partyId, APPROVED)
        activate PPR
        PPR->>DB: Count APPROVED participants
        DB-->>PPR: int count
        deactivate PPR
        US->>US: Build PartyResponse with<br/>currentParticipants, isHost, status
    end
    US-->>UC: List<PartyResponse>
    deactivate US
    
    activate UC
    UC-->>Client: ResponseEntity<List<PartyResponse>>
    deactivate UC

Loading

Possibly related PRs

• Feature/13 user #27: UserController의 /api/v1/user 경로 및 /me, /me/location 엔드포인트 수정으로 동일한 API 구조 변경과 관련됨
• Feature/#28 gogole apple login #36: UserController와 UserService 레이어의 사용자 API 기능 추가로 닉네임 검증 및 서비스 메서드 변경과 겹침
• Release : 파티 참여, 승인, 거절 기능 추가(v1.0.3) #47: PartyParticipantRepository에 쿼리 메서드를 추가하는 동일한 도메인 변경으로 직접 관련됨

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 마이페이지 참여중인 파티 조회 기능을 명확하게 설명하고 있으며, 변경 사항의 주요 기능과 일치합니다.
Linked Issues check	✅ Passed	PR의 모든 코드 변경사항이 #49 이슈의 요구사항(마이페이지 참여중인 파티 조회)을 충족합니다.
Out of Scope Changes check	✅ Passed	.gitignore 수정을 제외하고, 모든 변경사항이 마이페이지 참여중인 파티 조회 기능 구현과 관련이 있습니다.

---

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: 5

Caution

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

⚠️ Outside diff range comments (1)

src/main/java/ita/tinybite/domain/party/repository/PartyParticipantRepository.java (1)

3-3: 잘못된 @Param 어노테이션 import입니다.

io.lettuce.core.dynamic.annotation.Param은 Lettuce Redis 클라이언트용 어노테이션입니다. Spring Data JPA의 @Query와 함께 사용하려면 org.springframework.data.repository.query.Param을 사용해야 합니다. 현재 코드는 런타임에 파라미터 바인딩 오류가 발생합니다.

🔎 수정 제안

-import io.lettuce.core.dynamic.annotation.Param;
+import org.springframework.data.repository.query.Param;

🧹 Nitpick comments (2)

src/main/java/ita/tinybite/domain/user/controller/UserController.java (2)

88-93: API 응답 형식이 다른 엔드포인트와 일관성이 없습니다.

다른 엔드포인트들은 APIResponse<?> 래퍼를 사용하지만, 이 엔드포인트만 ResponseEntity<List<PartyResponse>>를 직접 반환합니다. 클라이언트 측에서 두 가지 다른 응답 형식을 처리해야 하므로 일관성 있는 API 설계를 위해 APIResponse로 래핑하는 것이 좋습니다.

🔎 수정 제안

     @GetMapping("/parties/active")
-    public ResponseEntity<List<PartyResponse>> getActiveParties(
+    public APIResponse<List<PartyResponse>> getActiveParties(
             @AuthenticationPrincipal Long userId) {
         List<PartyResponse> response = userService.getActiveParties(userId);
-        return ResponseEntity.ok(response);
+        return success(response);
     }

---

9-9: 사용되지 않는 import입니다.

UserDto가 import되어 있지만 이 컨트롤러에서 사용되지 않습니다.

🔎 수정 제안

-import ita.tinybite.domain.auth.dto.response.UserDto;

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6eb0d8d and 596b0ea.

📒 Files selected for processing (6)

• .gitignore
• src/main/java/ita/tinybite/domain/party/repository/PartyParticipantRepository.java
• src/main/java/ita/tinybite/domain/user/controller/UserController.java
• src/main/java/ita/tinybite/domain/user/dto/res/PartyResponse.java
• src/main/java/ita/tinybite/domain/user/service/UserService.java
• src/test/java/ita/tinybite/domain/user/service/UserServiceTest.java

🔇 Additional comments (2)

.gitignore (1)

42-42: LGTM!

Firebase 자격 증명 및 설정 파일이 저장소에 커밋되지 않도록 .gitignore에 추가한 것은 좋은 보안 관행입니다.

src/main/java/ita/tinybite/domain/user/controller/UserController.java (1)

89-91: userId가 null일 경우 처리가 필요합니다.

인증되지 않은 요청이나 인증 정보가 잘못된 경우 @AuthenticationPrincipal이 null을 반환할 수 있습니다. userService.getActiveParties(userId)에 null이 전달되면 예상치 못한 동작이 발생할 수 있습니다.

Spring Security 설정에서 이 엔드포인트가 인증 필수로 설정되어 있는지 확인해주세요. 인증 필수라면 null 체크가 불필요할 수 있습니다.

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

# Search for User entity definition
fd -t f -e java "User.java" | head -20

Repository: tinybite-2025/tinybite-server

Length of output: 128

---

🏁 Script executed:

# Search for PartyParticipant entity definition
fd -t f -e java "PartyParticipant.java" | head -20

Repository: tinybite-2025/tinybite-server

Length of output: 141

---

🏁 Script executed:

# Also search in the repository file itself to understand the context better
cat -n src/main/java/ita/tinybite/domain/party/repository/PartyParticipantRepository.java

Repository: tinybite-2025/tinybite-server

Length of output: 2246

---

🏁 Script executed:

# Read User entity to check field name
cat -n src/main/java/ita/tinybite/domain/user/entity/User.java | head -50

Repository: tinybite-2025/tinybite-server

Length of output: 1653

---

🏁 Script executed:

# Read PartyParticipant entity
cat -n src/main/java/ita/tinybite/domain/party/entity/PartyParticipant.java | head -80

Repository: tinybite-2025/tinybite-server

Length of output: 2306

---

JPQL 파라미터 바인딩 및 필드명 불일치로 런타임 오류가 발생합니다.

• Line 35: @Param("partyStatuses")는 복수형이지만 JPQL에서 :partyStatus(단수형)를 사용하여 바인딩이 실패합니다.
• Line 30: User 엔티티의 ID 필드명은 userId이므로 JPQL에서 pp.user.id 대신 pp.user.userId를 사용해야 합니다.
• Line 3: io.lettuce.core.dynamic.annotation.Param을 import하고 있는데, Spring Data JPA 쿼리에는 org.springframework.data.jpa.repository.Param을 사용해야 합니다.

🔎 수정 제안

-import io.lettuce.core.dynamic.annotation.Param;
+import org.springframework.data.jpa.repository.Param;

 @Query("SELECT pp FROM PartyParticipant pp " +
-        "WHERE pp.user.id = :userId " +
+        "WHERE pp.user.userId = :userId " +
         "AND pp.party.status =:partyStatus " +
         "AND pp.status = :participantStatus")
 List<PartyParticipant> findActivePartiesByUserId(
         @Param("userId") Long userId,
-        @Param("partyStatuses") PartyStatus partyStatus,
+        @Param("partyStatus") PartyStatus partyStatus,
         @Param("participantStatus") ParticipantStatus participantStatus
 );

🤖 Prompt for AI Agents

In
src/main/java/ita/tinybite/domain/party/repository/PartyParticipantRepository.java
around lines 29 to 37, the JPQL and parameter annotations are mismatched and the
wrong @Param import is used; update the query to reference the correct user id
field (replace pp.user.id with pp.user.userId), make the @Param name for the
party status match the JPQL placeholder (use "partyStatus" instead of
"partyStatuses") and ensure the method parameter name matches that @Param, and
replace the imported io.lettuce.core.dynamic.annotation.Param with
org.springframework.data.jpa.repository.Param so Spring Data JPA binds
parameters correctly.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

# First, locate and read the PartyResponse.java file
find . -name "PartyResponse.java" -type f

Repository: tinybite-2025/tinybite-server

Length of output: 140

---

🏁 Script executed:

# Also locate the Party entity
find . -name "Party.java" -type f

Repository: tinybite-2025/tinybite-server

Length of output: 132

---

🏁 Script executed:

# Check file sizes before reading
wc -l ./src/main/java/ita/tinybite/domain/user/dto/res/PartyResponse.java
wc -l ./src/main/java/ita/tinybite/domain/party/entity/Party.java

Repository: tinybite-2025/tinybite-server

Length of output: 207

---

🏁 Script executed:

# Read PartyResponse.java entirely
cat -n ./src/main/java/ita/tinybite/domain/user/dto/res/PartyResponse.java

Repository: tinybite-2025/tinybite-server

Length of output: 1960

---

🏁 Script executed:

# Read Party.java to check fields
cat -n ./src/main/java/ita/tinybite/domain/party/entity/Party.java

Repository: tinybite-2025/tinybite-server

Length of output: 6650

---

🏁 Script executed:

# Check if there are any null safety patterns for host field
rg "host.*null|host.*Optional" ./src/main/java/ita/tinybite/domain/party/entity/Party.java ./src/main/java/ita/tinybite/domain/user/dto/res/PartyResponse.java

Repository: tinybite-2025/tinybite-server

Length of output: 184

---

🏁 Script executed:

# Search for other usages of party.getHost() to see if null checks are done elsewhere
rg "getHost\(\)" --type java -A 2 -B 2 | head -50

Repository: tinybite-2025/tinybite-server

Length of output: 4923

---

startDate와 createdAt이 동일한 값으로 매핑되고 있으며, NPE 방지를 위한 개선이 필요합니다.

1. startDate와 createdAt이 모두 party.getCreatedAt()으로 설정되어 있어 중복되고 의미가 불명확합니다. Party 엔티티에는 별도의 시작일(startDate) 필드가 없으므로, startDate의 의도를 명확히 하고 적절한 필드를 매핑해야 합니다. 현재 매핑은 파티 생성 시간을 두 필드에 중복으로 설정하는 것으로 보이며, 이는 DTO 설계를 재검토할 필요가 있습니다.
2. party.getHost()가 null이 될 가능성에 대비하여 null 체크를 추가하는 것을 권장합니다.

🔎 NPE 방지를 위한 수정 제안

-                .hostUsername(party.getHost().getNickname())
+                .hostUsername(party.getHost() != null ? party.getHost().getNickname() : null)

🤖 Prompt for AI Agents

In src/main/java/ita/tinybite/domain/user/dto/res/PartyResponse.java around
lines 37 to 40, startDate is incorrectly mapped to party.getCreatedAt()
(duplicating createdAt) and party.getHost() is used without a null check; change
the startDate mapping to the correct source (e.g., party.getStartDate() if the
Party entity has that field, otherwise leave startDate null or remove the
duplicate), and guard host access with a null check so the DTO uses host == null
? null : host.getNickname() to avoid NPEs.

[coderabbitai]

⚠️ Potential issue | 🟠 Major

N+1 쿼리 문제가 있습니다.

각 PartyParticipant마다 countByPartyIdAndStatus()를 호출하여 참여자 수를 조회하고 있습니다. 사용자가 N개의 활성 파티에 참여 중이라면 N+1개의 쿼리가 발생합니다. 파티 수가 많아지면 성능 저하가 발생할 수 있습니다.

🔎 개선 방안

하나의 쿼리로 파티별 참여자 수를 미리 조회하는 방법을 고려해보세요:

// Repository에 추가
@Query("SELECT pp.party.id, COUNT(pp) FROM PartyParticipant pp " +
       "WHERE pp.party.id IN :partyIds AND pp.status = :status " +
       "GROUP BY pp.party.id")
List<Object[]> countByPartyIdsAndStatus(
    @Param("partyIds") List<Long> partyIds,
    @Param("status") ParticipantStatus status
);

또는 Party 엔티티의 currentParticipants 필드가 이미 동기화되어 있다면 해당 필드를 사용할 수 있습니다.

🤖 Prompt for AI Agents

In src/main/java/ita/tinybite/domain/user/service/UserService.java around lines
74-82, the current implementation issues N+1 queries by calling
participantRepository.countByPartyIdAndStatus(...) per PartyParticipant;
instead, add a repository method that returns counts grouped by party IDs for
the list of party IDs (e.g., countByPartyIdsAndStatus), call that once with all
party IDs to produce a Map<Long,Integer> of partyId->count, and then use that
map and the already-loaded Party/host info when mapping participants to
PartyResponse; ensure you collect partyIds from participants first, invoke the
batch count query, build the lookup, and replace per-item count calls with
lookups to eliminate the per-item DB call.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

party.getHost()가 null일 경우 NPE가 발생할 수 있습니다.

호스트가 탈퇴하거나 삭제된 경우 getHost()가 null을 반환할 수 있습니다.

🔎 수정 제안

-                    boolean isHost = party.getHost().getUserId().equals(userId);
+                    boolean isHost = party.getHost() != null && party.getHost().getUserId().equals(userId);

🤖 Prompt for AI Agents

In src/main/java/ita/tinybite/domain/user/service/UserService.java around line
79, calling party.getHost().getUserId() can throw an NPE if getHost() is null;
change the logic to perform a null-safe check (e.g., assign Host host =
party.getHost(); then set isHost = host != null &&
Objects.equals(host.getUserId(), userId) or use
Optional.ofNullable(party.getHost()).map(Host::getUserId).map(id ->
id.equals(userId)).orElse(false)) so that when the host is missing isHost
becomes false and no NPE is raised.

[coderabbitai]

⚠️ Potential issue | 🟠 Major

participantRepository가 초기화되지 않아 NPE가 발생할 수 있습니다.

participantRepository 필드가 @Autowired나 @Mock으로 선언되지 않아 null 상태로 UserService에 전달됩니다. 현재 테스트에서 getActiveParties() 메서드를 호출하지 않더라도, 새로 추가된 기능에 대한 테스트 커버리지가 누락되어 있고 향후 NPE를 유발할 수 있습니다.

🔎 수정 제안

     @Autowired
     private UserRepository userRepository;
 
-    private PartyParticipantRepository participantRepository;
+    @Autowired
+    private PartyParticipantRepository participantRepository;

또는 getActiveParties() 메서드에 대한 별도의 테스트 케이스를 추가하는 것을 권장합니다.

Also applies to: 44-44