Skip to content

fix: 메시지 검색 결과에 unreadCount, isMuted 필드 추가#555

Merged
Soundbar91 merged 1 commit intodevelopfrom
fix/message-matches-response
Apr 16, 2026
Merged

fix: 메시지 검색 결과에 unreadCount, isMuted 필드 추가#555
Soundbar91 merged 1 commit intodevelopfrom
fix/message-matches-response

Conversation

@Soundbar91
Copy link
Copy Markdown
Collaborator

🔍 개요


🚀 주요 변경 내용

  • roomMatches와 동일하게 messageMatches 응답에도 읽지 않은 메시지 수와 알림 뮤트 여부를 포함하도록 수정

💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

roomMatches와 동일하게 messageMatches 응답에도
읽지 않은 메시지 수와 알림 뮤트 여부를 포함하도록 수정

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 41c7ce60-870f-42ce-b51a-38bc4ecd6901

📥 Commits

Reviewing files that changed from the base of the PR and between 845049c and 3330418.

📒 Files selected for processing (1)
  • src/main/java/gg/agit/konect/domain/chat/dto/ChatMessageMatchResult.java
📜 Recent review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Analyze (java-kotlin)
  • GitHub Check: coverage
🧰 Additional context used
📓 Path-based instructions (2)
src/main/java/**/*.java

⚙️ CodeRabbit configuration file

src/main/java/**/*.java: 아래 원칙으로 리뷰 코멘트를 작성한다.

  • 코멘트는 반드시 한국어로 작성한다.
  • 반드시 수정이 필요한 항목만 코멘트로 남기고, 단순 취향 차이는 지적하지 않는다.
  • 각 코멘트 첫 줄에 심각도를 [LEVEL: high|medium|low] 형식으로 반드시 표기한다.
  • 심각도 기준: high=운영 장애 가능, medium=품질 저하, low=개선 권고.
  • 각 코멘트는 "문제 -> 영향 -> 제안" 순서로 3문장 이내로 간결하게 작성한다.
  • 가능하면 재현 조건 및 실패 시나리오도 포함한다.
  • 제안은 현재 코드베이스(Spring Boot + JPA + Flyway) 패턴과 일치해야 한다.
  • 보안, 트랜잭션 경계, 예외 처리, N+1, 성능 회귀 가능성을 우선 점검한다.
  • 가독성: 변수/메서드 이름이 의도를 바로 드러내는지, 중첩과 메서드 길이가 과도하지 않은지 점검한다.
  • 단순화: 불필요한 추상화, 중복 로직, 과한 방어 코드가 있으면 더 단순한 대안을 제시한다.
  • 확장성: 새 요구사항 추가 시 변경 범위가 최소화되는 구조인지(하드코딩 분기/값 여부 포함) 점검한다.

Files:

  • src/main/java/gg/agit/konect/domain/chat/dto/ChatMessageMatchResult.java
**/*

⚙️ CodeRabbit configuration file

**/*: 공통 리뷰 톤 가이드:

  • 모든 코멘트는 첫 줄에 [LEVEL: ...] 태그를 포함한다.
  • 과장된 표현 없이 사실 기반으로 작성한다.
  • 한 코멘트에는 하나의 이슈만 다룬다.
  • 코드 예시가 필요하면 최소 수정 예시를 제시한다.
  • 가독성/단순화/확장성 이슈를 발견하면 우선순위를 높여 코멘트한다.

Files:

  • src/main/java/gg/agit/konect/domain/chat/dto/ChatMessageMatchResult.java

📝 Walkthrough

Walkthrough

ChatMessageMatchResult 레코드에 unreadCountisMuted 필드가 추가되었습니다. 팩토리 메서드 from()이 업데이트되어 ChatRoomSummaryResponse 객체에서 이들 새로운 필드 값을 자동으로 추출하여 할당합니다.

Changes

Cohort / File(s) Summary
ChatMessageMatchResult Record Enhancement
src/main/java/gg/agit/konect/domain/chat/dto/ChatMessageMatchResult.java
unreadCount (Integer)와 isMuted (Boolean) 필드 추가. from() 팩토리 메서드가 room.unreadCount()room.isMuted()에서 값을 가져오도록 수정됨.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Suggested labels

버그

Poem

🐰 새로운 필드가 쏙쏙 들어왔네요,
미읽은 메시지와 조용한 방들,
채팅방이 이제 더 똑똑해졌어요!
작은 변화, 큰 완성이랍니다. 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Title check ✅ Passed 풀 리퀘스트 제목이 변경 사항의 주요 내용을 명확하게 요약하고 있습니다. unreadCount와 isMuted 필드를 messageMatches 응답에 추가하는 것이 정확히 표현되어 있습니다.
Description check ✅ Passed 풀 리퀘스트 설명이 변경의 목적과 내용을 명확하게 설명하고 있으며, messageMatches 응답 스펙 확장에 대해 관련 정보를 제공하고 있습니다.

✏️ 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 fix/message-matches-response

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

🧪 JaCoCo Coverage Report (Changed Files)

변경된 main Java 소스 파일에 대한 JaCoCo 데이터가 없습니다.

📊 View Workflow Run

@Soundbar91 Soundbar91 merged commit f1c0dab into develop Apr 16, 2026
5 checks passed
@Soundbar91 Soundbar91 deleted the fix/message-matches-response branch April 16, 2026 08:39
@Soundbar91 Soundbar91 self-assigned this Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant