♻️Refactor: MatchDetailResponse 응답 구조 개선

[angoroa]

#️⃣ Issue Number

• closed ✨Feat: 매칭상태 자동 전환 스케줄러 구현 #65

📝 요약(Summary)

• 매칭 상세 조회 시 시설 이름(facilityName)과 주소(facilityAddress)까지 함께 반환하도록 개선

🛠️ PR 유형

어떤 변경 사항이 있나요?

• 새로운 기능 추가
• 버그 수정
• CSS 등 사용자 UI 디자인 변경
• 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
• 코드 리팩토링
• 주석 추가 및 수정
• 문서 수정
• 테스트 추가, 테스트 리팩토링
• 빌드 부분 혹은 패키지 매니저 수정
• 파일 혹은 폴더명 수정
• 파일 혹은 폴더 삭제

📸스크린샷 (선택)

💬 공유사항 to 리뷰어

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

• 커밋 메시지 컨벤션에 맞게 작성했습니다.
• 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

Summary by CodeRabbit

새로운 기능

• 경기 상세 조회 시 운동 시설 정보(시설명, 주소)가 함께 표시되도록 개선되었습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 9146791a-f749-4a86-b3c9-bf19dec3a753

📥 Commits

Reviewing files that changed from the base of the PR and between 9875e28 and ec88607.

📒 Files selected for processing (2)

• src/main/java/com/be/sportizebe/domain/match/dto/response/MatchDetailResponse.java
• src/main/java/com/be/sportizebe/domain/match/service/MatchServiceImpl.java

---

요약

Walkthrough

매치 상세 응답(MatchDetailResponse)에 시설 이름과 주소 필드를 추가했습니다. 서비스 계층에서 스포츠 시설을 조회하여 응답 객체에 포함시키는 기능을 구현했습니다.

Changes

Cohort / File(s)	Summary
시설 정보 통합 src/main/java/com/be/sportizebe/domain/match/dto/response/MatchDetailResponse.java, src/main/java/com/be/sportizebe/domain/match/service/MatchServiceImpl.java	MatchDetailResponse에 facilityName, facilityAddress 필드 추가 및 factory 메서드 서명 변경. MatchServiceImpl에 SportsFacilityRepository 의존성 주입하여 getMatchDetail에서 시설 정보 조회 후 응답에 포함.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• ✨Feat: 운동 매칭 기능 추가 #56: MatchDetailResponse 확장 및 MatchServiceImpl.getMatchDetail 메서드 업데이트로 동일 클래스/메서드에 직접적으로 영향을 미치는 관련 PR입니다.

Poem

🐰 경기장의 이름과 주소를

응답에 담아 사용자 곁으로

시설 정보는 이제 한눈에,

더 완성된 매치 정보가 탄생했네! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (3 warnings)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	PR은 #65 이슈를 종료한다고 하지만, 실제 변경사항(facilityName/facilityAddress 추가)은 자동 상태 전환 스케줄러 요구사항과 관련이 없습니다.	PR #65의 MatchStatus 자동 전환 스케줄러 구현 요구사항을 완료하거나, 잘못된 이슈 링크를 수정하세요.
Out of Scope Changes check	⚠️ Warning	facilityName과 facilityAddress 필드 추가는 #65 이슈의 스케줄러 구현 범위 밖입니다.	스케줄러 구현 기능을 추가하거나, 이 변경사항에 대한 별도의 이슈를 생성하고 링크하세요.
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
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 MatchDetailResponse 응답 구조 개선에 관한 것이지만, 실제 변경사항은 facilityName과 facilityAddress 필드 추가로 제한적입니다.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refactor/match-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

• X
• Mastodon
• Reddit
• LinkedIn

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