🐛 Fix: 미판매 + 구매제안 -> 채팅방 생성 가능

[Chungs0604]

🔗 관련 이슈

• 🐛 Fix 채팅방 생성시 미판매 + 구매제안 상태라면 채팅방 생성 가능 #402

📌 작업 내용

• HIDDEN(미판매) 상태 상품에 SENT 상태 구매 제안이 존재하는 경우, 채팅방 생성 허용하도록
  로직 수정
• 기존: ON_SALE / TRADING 상태만 채팅방 생성 가능
• 변경: HIDDEN + SENT 구매 제안 있음 → 채팅방 생성 허용 / SOLD + SENT 구매 제안 있음 →
  기존과 동일하게 409 반환

📌 변경된 API

• POST /chats — 채팅방 생성 로직 수정 (ChatRoomService.findOrCreateChatRoom)

📌 테스트 결과

• ON_SALE 상품 → 채팅방 생성 ✅
• 기존 채팅방 존재 + 상품 HIDDEN 전환 → 채팅방 입장 ✅
• HIDDEN + 구매 제안 없음 → 409 ✅
• HIDDEN + SENT 구매 제안 있음 → 채팅방 생성 ✅
• SOLD + SENT 구매 제안 있음 → 409 ✅

📌 확인 필요 사항

• 구매 제안 수락 후 TRADING 전환 시 채팅 흐름 정상 동작 여부

✅ 체크리스트

• 기능 요구사항을 충족했는가?
• API 응답 형식이 통일되어 있는가?
• 예외 상황이 적절히 처리되었는가?
• 인증/인가 검증이 누락되지 않았는가?
• Entity를 직접 응답하지 않았는가?
• 중복 코드가 과도하지 않은가?
• 메서드와 클래스의 역할이 명확한가?
• 테스트가 필요한 핵심 로직에 테스트가 작성되었는가?

[github-actions]

Gemini 파일 단위 요약 리뷰

src/main/java/com/tenure/domain/chat/service/ChatRoomService.java

변경 요약

• 채팅방 생성 및 조회 시, 상품이 ON_SALE(판매중) 또는 TRADING(거래중) 상태가 아니더라도 구매자가 보낸 구매 제안(SENT 상태)이 존재하고 상품이 SOLD(판매완료)가 아니라면 채팅방 조회가 가능하도록 로직을 확장했습니다.
• PurchaseOfferStatus.SENT를 Static Import로 변경하여 코드 가독성을 개선했습니다.

영향 범위

• 기능: 채팅방 생성 및 조회 API(findOrCreateChatRoom)에 영향을 미치며, 미판매 상태이지만 구매 제안이 진행 중인 아이템에 대해서도 채팅방 진입이 가능해집니다.
• 실행 흐름/DB 호출: 상품이 판매중/거래중 상태가 아닐 경우, purchaseOfferRepository 및 productRepository에 대한 추가적인 조회가 동기적으로 발생합니다.

확인 필요

• 차단 관계 검증 누락 리스크: enterChatroom 메서드 하단에는 상대방의 차단 여부를 확인하는 로직이 존재하지만, 채팅방을 새로 생성하거나 조회하는 findOrCreateChatRoom 단계에서는 구매자(buyer)와 판매자(owner) 간의 차단 관계 검증이 누락되어 있습니다. 차단된 상대방과의 채팅방 생성을 방지해야 하는지 확인이 필요합니다.
• buyerId의 신뢰성: findOrCreateChatRoom에 전달되는 buyerId가 컨트롤러 수준에서 CurrentUserProvider.getCurrentUserId()를 거쳐 검증된 값인지 확인이 필요합니다.

제안

• DB 호출 최적화 제안: orElseGet 블록 내부에서 purchaseOfferRepository.findIdBy...와 productRepository.findByItemId가 순차적으로 호출되고 있습니다. 이미 존재하지 않는 상태를 검증하기 위해 여러 번의 쿼리가 발생하므로, 해당 아이템의 정보와 제안 상태를 Join 쿼리 한 번으로 가져오거나, 필요 최소한의 컬럼만 조회하도록 레포지토리 메서드를 최적화하는 방안을 제안합니다.