fix: 독서 기록 화면 감상평 빈 값일 경우 하단 패딩 제거

[easyhooon]

🔗 관련 이슈

• Close [BOOK-416/fix] 기록 상세 화면 Spacing 간격 조정 #212

📙 작업 설명

• "" 로 들어가도 Text의 높이를 그대로 차지함, 감상평이 빈 값일 경우 Text Composable 사용 X
• Review가 빈값(Empty)일때 프리뷰 추가

🧪 테스트 내역 (선택)

• 주요 기능 정상 동작 확인
• 브라우저/기기에서 동작 확인
• 엣지 케이스 테스트 완료
• 기존 기능 영향 없음

📸 스크린샷 또는 시연 영상 (선택)

Before	After

💬 추가 설명 or 리뷰 포인트 (선택)

• 최하단 RecordDetailUiSpacer는 높이가 좁은 화면을 대응하기 위한 패딩입니다.

Summary by CodeRabbit

변경사항

• 스타일

  • 리뷰 항목의 수직 패딩 조정으로 목록 간격이 변경됨

• 버그 수정

  • 빈 리뷰 텍스트는 더 이상 화면에 표시되지 않음

• 새로운 기능

  • 빈 리뷰 상태를 보여주는 미리보기/디자인 추가로 빈 상태 UI 확인 가능

[coderabbitai]

Warning

Rate limit exceeded

@easyhooon has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 46 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between aeea6fa and 0a43a7d.

📒 Files selected for processing (3)

• core/designsystem/stability/designsystem.stability (4 hunks)
• feature/detail/stability/detail.stability (2 hunks)
• gradle/libs.versions.toml (1 hunks)

Walkthrough

RecordDetailUi.kt에 빈 리뷰 상태의 새로운 미리보기 함수가 추가되었고, ReviewItem.kt에서 세로 패딩이 조정되며 리뷰 텍스트의 조건부 렌더링 및 빈 상태 미리보기 함수가 추가되었습니다. 또한, detail.stability 파일에서 도서 상세 관련 Compose 안정성 항목들이 대거 제거되고, RecordDetailContent 컴포저블의 안정성 메타데이터가 런타임 체크로 변경되는 등 안정성 관련 리팩터링이 진행되었습니다.

Changes

코호트 / 파일(s)	변경 요약
리뷰 상세 UI 미리보기 개선 feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/record/RecordDetailUi.kt	빈 리뷰 필드를 가진 새로운 ComponentPreview 함수 ReviewDetailEmptyPreview 추가
리뷰 아이템 조건부 렌더링 및 스타일 조정 feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/record/component/ReviewItem.kt	세로 패딩 spacing3 → spacing4 조정, 리뷰 텍스트 조건부 렌더링(blank 체크), 빈 상태 미리보기 함수 ReviewBoxEmptyPreview 추가
Compose 안정성 메타데이터 및 공개 API 정리 feature/detail/stability/detail.stability	도서 상세 관련 Compose 안정성 블록 다수 제거, RecordDetailContent의 안정성 상태를 런타임 체크로 변경, 여러 공개/내보내기 콤포저블 및 API 제거, 안정성 플래그 리팩터링

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

• detail.stability 파일에서 대규모 공개 API 항목 제거 및 안정성 플래그 변경으로 인한 복잡한 검토 필요
• RecordDetailContent 안정성 상태 변경에 따른 영향 분석 요구
• UI 변경과 안정성 리팩터링이 혼재되어 있어 다방면 검토 필요

Poem

🐰 새 미리보기로 빈 공간 채워주고,
패딩은 살짝 키워 따스함 더하네.
안정성도 새 옷 입고 조용히 정리하며,
깔끔한 변화로 UI 꽃 피운다! 🌸

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	PR 설명에 따르면 이슈 #213을 닫는다고 했으나, 연결된 이슈 #213의 요구사항은 "도서 상세 화면의 감정 뱃지 컬러를 각 감정에 매칭되는 색상으로 수정"입니다. 반면 실제 코드 변경사항(ReviewItem.kt, RecordDetailUi.kt)은 감정 뱃지 색상 수정이 아닌 감상평 텍스트 렌더링 조건 처리와 패딩 제거에만 집중하고 있습니다. 제공된 정보에서 감정 뱃지 컬러 변경 관련 코드 수정이 발견되지 않아, PR이 연결된 이슈의 요구사항을 충족하지 못하고 있습니다.	PR이 실제로 구현하는 변경사항(감상평 빈 값 시 패딩 제거)과 연결된 이슈 #213의 요구사항(감정 뱃지 컬러 수정)이 일치하지 않습니다. 연결된 이슈를 재검토하거나, 감정 뱃지 컬러 수정 코드가 포함되어야 하는지 확인이 필요합니다.
Out of Scope Changes Check	❓ Inconclusive	ReviewItem.kt와 RecordDetailUi.kt의 변경사항은 PR 목표인 감상평 빈 값 시 패딩 제거와 직접적으로 연관되어 있습니다. 그러나 feature/detail/stability/detail.stability 파일의 변경사항은 북 상세 화면의 많은 공개 API 제거 및 레코드 관련 항목들의 안정성 플래그 업데이트를 포함하고 있으며, 이는 원래 PR 목표의 범위를 벗어나는 광범위한 구조적 변경으로 보입니다.	stability.detail 파일의 광범위한 변경사항(공개 API 제거 및 안정성 메타데이터 업데이트)이 감상평 패딩 제거 수정과의 연관성을 명확히 해야 합니다. 이러한 변경사항이 Compose 컴파일러의 자동 생성 결과인지, 아니면 의도적인 API 리팩토링인지 검토가 필요합니다.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목 "fix: 독서 기록 화면 감상평 빈 값일 경우 하단 패딩 제거"는 변경사항의 주요 목표를 명확하게 설명합니다. 코드 변경사항에서 ReviewItem.kt의 리뷰 텍스트를 조건부로 렌더링하고 패딩을 조정한 것이 정확히 이 제목과 일치하며, RecordDetailUi.kt의 빈 리뷰 상태 미리보기 추가도 이를 뒷받침합니다. 제목은 간결하고 명확하며 오류 유도적이지 않습니다.

---

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

[seoyoon513]

요건 처음보네요 👀

[seoyoon513]

LGTM~