[REFACTOR] 챗 그리드 조회 로직 수정

[yunseo-leeo]

🎋 이슈 및 작업중인 브랜치

• [REFACTOR] 트라이브 챗 그리드 조회 로직 수정 #129

🔑 주요 내용

• 챗 그리드 조회 로직 수정

Check List

• Reviewers 등록을 하였나요?
• Assignees 등록을 하였나요?
• 라벨(Label) 등록을 하였나요?
• PR 머지하기 전 반드시 CI가 정상적으로 작동하는지 확인해주세요!

Summary by CodeRabbit

릴리스 노트

Refactor

• 채팅 그리드 조회 시스템을 개선하였습니다. 채팅 그리드 검색 과정을 간소화하여 더 효율적인 데이터 검색 성능을 제공합니다.

[coderabbitai]

📝 Walkthrough

Walkthrough

채팅 그리드 조회 플로우에서 userId 파라미터를 제거하여 컨트롤러와 서비스 계층 전반에서 사용자 범위 검증을 제거했습니다.

Changes

Cohort / File(s)	Summary
Chat Grid 조회 메서드 서명 수정 src/main/java/com/umc/nuvibe/domain/tribe/controller/ChatController.java, src/main/java/com/umc/nuvibe/domain/tribe/service/chat/ChatService.java, src/main/java/com/umc/nuvibe/domain/tribe/service/chat/ChatServiceImpl.java	getChatGridList 메서드에서 userId 파라미터를 제거했습니다. ChatServiceImpl에서 userId 기반 사용자-부족 검증 로직도 함께 제거되어 이제 tribeId만으로 유효성 검사를 수행합니다.

Possibly related issues

• [REFACTOR] 트라이브 챗 그리드 조회 로직 수정 #129: 동일한 채팅 그리드 조회 로직을 대상으로 하며, 이 PR과 반대로 사용자 검증을 추가하도록 요청합니다.

Possibly related PRs

• [REFACTOR] 트라이브 및 채팅 조회 반환값 수정 #103: 채팅 그리드 조회 로직을 수정하고 userId 사용 방식을 변경합니다 (이 PR은 userId를 제거하는 반면 해당 PR은 사용자 의존 로직을 추가함).

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 60.00% 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 제목은 챗 그리드 조회 로직 수정이라는 주요 변경사항을 명확하게 설명하며, 제공된 코드 변경사항(userId 파라미터 제거 및 getChatGridList 메서드 서명 수정)과 일치합니다.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refactor/#129_tribe-grid

---

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]

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/com/umc/nuvibe/domain/tribe/service/chat/ChatServiceImpl.java (1)

135-145: ⚠️ Potential issue | 🟠 Major

validateUserInTribe 호출 제거 — 비인가 접근 가능

기존에는 validateUserInTribe(userId, tribeId) 호출을 통해 요청자가 해당 트라이브의 멤버인지 검증했으나, 이 변경으로 해당 검증이 완전히 제거되었습니다. tribeId만 알면 누구든 해당 트라이브의 채팅 이미지 그리드를 조회할 수 있게 됩니다.

만약 비회원/비멤버에게도 그리드를 공개하는 것이 의도라면, 해당 사항을 주석이나 문서에 명시해 주세요. 그렇지 않다면, 컨트롤러에서 @AuthUser를 복원하고 서비스에서 멤버십 검증을 유지해야 합니다.