Develop / Merge to Main

[eraser502]

Overview

• 실서버 배포

---

Related Issue

• Closes

---

PR Checklist

• [ ]

---

Additional Information

[coderabbitai]

개요

이 변경사항은 여러 DTO(데이터 전송 객체)에 isProtected 부울 필드를 추가하고, 리포지토리의 JPQL 쿼리를 업데이트하며, Swagger 설정에서 API 그룹화를 위한 다섯 개의 새로운 Bean 메서드를 추가합니다. AppVersionPageSummaryResponse, AppVersionPageSummaryWithVersionResponse, 그리고 관련 리포지토리 쿼리 생성자 표현식이 새로운 필드를 포함하도록 수정되었습니다.

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10분

• 주의 필요 영역:
  • 4개의 서로 다른 파일에 걸친 일관된 필드 추가 확인 필요
  • JPQL 쿼리의 생성자 표현식에서 필드 순서 및 매핑 정확성 검증
  • SwaggerConfig의 새로운 Bean 메서드들이 올바른 경로 패턴을 설정했는지 확인

연관된 PR

• Feature / builder / cms and site api / Add isProtected field to page DTO #121: 동일한 DTO들(AppVersionPageSummaryResponse, AppVersionPageSummaryWithVersionResponse)에 isProtected 필드를 추가하는 코드 변경이 직접적으로 겹침
• Docs / swagger api #119: SwaggerConfig에 동일한 GroupedOpenApi Bean 메서드들(cmsApi, siteApi, redotApi, adminApi, allApi)을 추가하는 부분이 중복됨
• Deploy / Merge to Main #118: 이전 PR에서 도입한 동일한 DTO와 리포지토리 쿼리 구성을 직접적으로 수정

추천 레이블

feature

시

보호 필드 하나를 더하고 ✨
쿼리들을 다시 엮으며 🐰
Swagger 설정도 정리하네,
API 문서는 더욱 빛난다 🌟
다섯 그룹으로 나뉜 길을 따라가며! 🛤️

Pre-merge checks and finishing touches

❌ Failed checks (3 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	개요 섹션에만 "실서버 배포"라는 최소한의 설명이 있지만, 변경사항의 세부내용이 거의 없어 불완전합니다.	각 변경사항(isProtected 필드 추가, Swagger API 그룹화 등)의 목적과 영향범위를 상세히 작성하고 관련 이슈를 명시해주세요.
Linked Issues check	⚠️ Warning	PR 설명에서 관련 이슈 섹션이 비어있으며 "Closes" 항목이 작성되지 않았습니다.	이 PR과 관련된 GitHub 이슈를 명시하고 "Closes #이슈번호" 형식으로 연결해주세요.
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.
Title check	❓ Inconclusive	제목이 불명확하고 구체적이지 않으며, 주요 변경사항을 명확히 설명하지 못하고 있습니다.	제목을 변경하여 isProtected 필드 추가 및 Swagger 설정 확장 등 핵심 변경사항을 구체적으로 표현해주세요.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Out of Scope Changes check	✅ Passed	주요 변경사항들(DTO 필드 추가, Swagger 설정)이 단일 목적으로 일관성 있게 연결되어 있어 범위 내에 있는 것으로 보입니다.

✨ 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 develop

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 47768cc and 0dbf2c8.

📒 Files selected for processing (4)

• src/main/java/redot/redot_server/domain/cms/site/page/dto/response/AppVersionPageSummaryResponse.java (1 hunks)
• src/main/java/redot/redot_server/domain/cms/site/page/service/dto/AppVersionPageSummaryWithVersionResponse.java (1 hunks)
• src/main/java/redot/redot_server/domain/site/page/repository/AppVersionPageRepository.java (1 hunks)
• src/main/java/redot/redot_server/global/config/SwaggerConfig.java (2 hunks)

🔇 Additional comments (5)

src/main/java/redot/redot_server/domain/cms/site/page/dto/response/AppVersionPageSummaryResponse.java (1)

3-9: LGTM!

isProtected 필드가 record에 올바르게 추가되었습니다. 관련 repository 쿼리 및 AppVersionPageSummaryWithVersionResponse와 일관성 있게 구현되어 있습니다.

src/main/java/redot/redot_server/domain/site/page/repository/AppVersionPageRepository.java (2)

25-30: LGTM!

AppVersionPageSummaryWithVersionResponse 생성자 호출도 새로운 isProtected 파라미터를 포함하도록 올바르게 업데이트되었습니다.

---

18-23: LGTM!

JPQL 생성자 표현식이 DTO 변경 사항과 일치하도록 올바르게 업데이트되었습니다. AppPage 엔티티에 isProtected 필드가 존재하며, 쿼리 내 필드 순서가 DTO 생성자 파라미터와 정확하게 매치됩니다.

src/main/java/redot/redot_server/global/config/SwaggerConfig.java (1)

32-76: LGTM!

API 문서화를 위한 그룹 분리가 잘 구성되어 있습니다. CMS, Site, Redot, Admin, All로 논리적으로 분류되어 있으며, redotApi에서 다른 그룹과의 중복을 pathsToExclude로 적절히 처리하고 있습니다.

src/main/java/redot/redot_server/domain/cms/site/page/service/dto/AppVersionPageSummaryWithVersionResponse.java (1)

5-16: LGTM!

isProtected 필드 추가와 toSummary() 메서드 업데이트가 올바르게 구현되었습니다. AppVersionPageSummaryResponse와의 매핑이 일관성 있게 유지되고 있습니다.

---

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.}