Skip to content

Conversation

@easyhooon
Copy link
Contributor

@easyhooon easyhooon commented Jan 16, 2026

🔗 관련 이슈

📙 작업 설명

  • compose-shadow 라이브러리 관련 코드 compose first party modifier api 사용 방식으로 전환
  • compose-shadow 라이브러리 의존성 제거

🧪 테스트 내역 (선택)

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

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

기능 미리보기 기능 미리보기
기능 설명 기능 설명

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

Summary by CodeRabbit

Summary by CodeRabbit

릴리스 노트

  • 의존성 업데이트

    • Compose BOM을 2025.08.00으로 업그레이드
    • 더 이상 사용되지 않는 그림자 라이브러리 의존성 제거
  • 리팩토링

    • UI 컴포넌트의 그림자 렌더링을 최신 Compose API로 전환
  • 최적화

    • 불필요한 전이(dependency) 제거로 모듈 빌드 경량화

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 16, 2026

Walkthrough

더 이상 유지되지 않는 서드파티 compose-shadow 의존성을 제거하고, Compose 공식 Shadow API로 MainBottomBar의 그림자 구현을 마이그레이션하며 Compose BOM 버전을 상향했습니다. (공개 API 변경 없음)

Changes

응집도 / 파일(들) 변경 요약
버전/라이브러리 매핑
gradle/libs.versions.toml
androidx-compose-bom 버전 2025.07.00 -> 2025.08.00로 업데이트; compose-shadow 버전 및 라이브러리 엔트리 제거
모듈 의존성
feature/screens/build.gradle.kts
libs.compose.shadow 의존성 제거 (implementations 블록에서 삭제)
UI 코드 마이그레이션
feature/screens/src/main/kotlin/com/ninecraft/booket/feature/screens/component/MainBottomBar.kt
서드파티 dropShadow 사용을 공식 Compose Shadow 객체/파라미터 방식으로 변경(관련 import 추가/기존 사용 제거). 퍼블릭 시그니처는 불변

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 오래된 그림자를 털어내고
새 API에 깡총 뛰어들었네
반짝이는 모서리, 부드러운 그림자,
코드도 가볍게, 마음도 가볍게
🥕✨ 나 래빗이 축하하노라

🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 compose-shadow 라이브러리 제거 및 modifier.shadow API 적용이라는 주요 변경사항을 명확하게 요약하고 있습니다.
Linked Issues check ✅ Passed PR의 모든 변경사항이 연결된 이슈 #254의 요구사항을 충족합니다: compose-shadow 라이브러리 제거, modifier.shadow API 적용, 의존성 제거 완료.
Out of Scope Changes check ✅ Passed 모든 변경사항이 연결된 이슈 #254의 범위 내에 있습니다. androidx-compose-bom 버전 업그레이드는 compose-shadow 제거와 관련된 필요한 변경입니다.

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


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@gradle/libs.versions.toml`:
- Line 79: Remove the unused explicit library declaration named
"androidx-foundation" (the entry that sets foundation = "1.10.1") from
libs.versions.toml and keep the BOM-managed "androidx-compose-foundation" entry;
ensure no references to the removed "androidx-foundation" identifier remain in
the codebase and rely on the existing BOM-managed declaration for the
androidx.compose.foundation artifact.
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4f609c4 and cd38271.

📒 Files selected for processing (3)
  • feature/screens/build.gradle.kts
  • feature/screens/src/main/kotlin/com/ninecraft/booket/feature/screens/component/MainBottomBar.kt
  • gradle/libs.versions.toml
💤 Files with no reviewable changes (1)
  • feature/screens/build.gradle.kts
⏰ 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: ci-build
  • GitHub Check: Compose Stability Check
🔇 Additional comments (2)
gradle/libs.versions.toml (1)

22-22: BOM 버전 확인 완료

dropShadow API를 위해 BOM을 2025.08.00으로 업데이트한 것이 확인되었습니다. 해당 버전은 Maven 저장소에서 사용 가능하며, dropShadow API가 도입된 Compose 1.9.0을 정확히 제공합니다.

feature/screens/src/main/kotlin/com/ninecraft/booket/feature/screens/component/MainBottomBar.kt (1)

47-58: Compose Foundation dropShadow API로의 마이그레이션이 올바르게 구현되었습니다.

dropShadow 모디파이어와 Shadow 클래스의 사용법이 적절하며, 모든 임포트 문이 올바릅니다. 프로젝트에서 사용 중인 Compose BOM 2025.08.00은 dropShadow API가 도입된 정확한 버전이므로, API는 안정적으로 지원됩니다.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

Copy link
Contributor

@seoyoon513 seoyoon513 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve~👍👍

@easyhooon easyhooon merged commit 56e75fd into develop Jan 21, 2026
4 checks passed
@easyhooon easyhooon deleted the BOOK-492-refactor/#254 branch January 21, 2026 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BOOK-492/refactor] deprecated된 compose-shadow 라이브러리 제거

3 participants