Skip to content

Conversation

@easyhooon
Copy link
Contributor

@easyhooon easyhooon commented Dec 9, 2025

Removed duplicate links and updated troubleshooting section.

Summary by CodeRabbit

  • 문서화
    • TroubleShooting 섹션의 참고 자료가 대폭 업데이트되었습니다. Metro 적용, Compose Stability Analyzer, Android Toast 내부 구현, Coroutine 에러 처리 패턴, 일회성 이벤트 처리, CameraX/MLKit OCR, Circuit 관련 글 등 다수 새 링크가 추가되고 항목 일부가 재배치/재추가되었습니다.
    • Development 섹션의 참조 스택이 Google ML Kit·Dagger Hilt에서 Google Cloud Vision·Metro로 교체되었습니다.
  • 기타
    • README의 Kotlin 배지 및 프로젝트 Kotlin 언어 버전이 2.2.0에서 2.2.21로 업데이트되었습니다.

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

Removed duplicate links and updated troubleshooting section.
@coderabbitai
Copy link

coderabbitai bot commented Dec 9, 2025

Walkthrough

README.md의 TroubleShooting 항목들이 재구성되고, Development 섹션에서 Google ML Kit → Google Cloud Vision 및 Dagger Hilt → Metro로 참조가 교체되었습니다. Kotlin 배지·언어 버전이 2.2.0 → 2.2.21로 업데이트되었습니다. 코드·공개 API 변경은 없습니다. (50단어 이내)

Changes

Cohort / File(s) 변경 요약
문서 업데이트
README.md
TroubleShooting 섹션의 링크 목록 재구성: 기존의 "[Compose] M3 ModalBottomSheet 드래그(터치 이벤트) 막는 법" 항목 관련 변경 및 다수 신규 항목(예: Metro 적용해보기, Compose Stability Analyzer 사용 후기, Android Toast 내부 구현 확인, Coroutine CancellationException 사례, Coroutine 에러 처리 패턴, Circuit-ImpressionEffect, Handle-One-Time-Event-As-State, OCR with CameraX/MLKit 등) 추가와 항목 순서 재정렬. Development 섹션에서 참조 스택을 Google ML Kit → Google Cloud Vision, Dagger Hilt → Metro로 교체. Kotlin 배지 및 언어 버전 2.2.0 → 2.2.21로 갱신. 문서 편집에 국한된 변경임.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • 변경 범위: 문서 단일 파일(README.md)
  • 주의사항:
    • 추가/수정된 링크의 URL 및 표시 오류 여부 확인
    • 교체된 기술 명칭(Google Cloud Vision, Metro)과 Kotlin 버전 표기의 일관성 검토

Poem

당근 들고 달려온 토끼가 말하네,
새 링크들 반짝여 길이 늘었네 🥕✨
옛 항목은 다시 만나고,
개발 숲에 새 길 닦았구나,
깡충깡충 기쁘게 읽자요!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 주요 내용인 README.md의 troubleshooting 섹션 업데이트를 명확하게 설명하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch easyhooon-patch-1

📜 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 bd2af22 and 020daf2.

📒 Files selected for processing (1)
  • README.md (4 hunks)
🧰 Additional context used
🧠 Learnings (5)
📚 Learning: 2025-07-31T23:17:40.054Z
Learnt from: easyhooon
Repo: YAPP-Github/Reed-Android PR: 88
File: feature/detail/src/main/kotlin/com/ninecraft/booket/feature/detail/book/component/RecordItem.kt:29-37
Timestamp: 2025-07-31T23:17:40.054Z
Learning: Reed-Android 프로젝트에서는 API가 준비되지 않은 상황에서 UI를 먼저 구현하고, API 연동 시점에 하드코딩된 데이터를 실제 데이터로 교체하는 개발 방식을 사용한다. RecordItem 컴포넌트의 emotionTags 매개변수도 API 연동 시점에 `text = emotionTags.joinToString(separator = "·") { "#$it" }`로 적용될 예정이다.

Applied to files:

  • README.md
📚 Learning: 2025-07-31T16:58:59.404Z
Learnt from: easyhooon
Repo: YAPP-Github/Reed-Android PR: 88
File: core/model/src/main/kotlin/com/ninecraft/booket/core/model/EmotionModel.kt:11-18
Timestamp: 2025-07-31T16:58:59.404Z
Learning: Reed-Android 프로젝트에서 core:model 모듈은 순수 Kotlin 모듈이므로 Android 리소스(R.string 등)에 접근할 수 없다.

Applied to files:

  • README.md
📚 Learning: 2025-07-31T16:58:59.404Z
Learnt from: easyhooon
Repo: YAPP-Github/Reed-Android PR: 88
File: core/model/src/main/kotlin/com/ninecraft/booket/core/model/EmotionModel.kt:11-18
Timestamp: 2025-07-31T16:58:59.404Z
Learning: Reed-Android 프로젝트는 현재 다국어 지원 계획이 없어서 모델에 한글 문자열을 직접 포함하는 것이 허용된다.

Applied to files:

  • README.md
📚 Learning: 2025-07-20T12:34:23.786Z
Learnt from: easyhooon
Repo: YAPP-Github/Reed-Android PR: 61
File: feature/webview/build.gradle.kts:17-21
Timestamp: 2025-07-20T12:34:23.786Z
Learning: Reed-Android 프로젝트에서는 `booket.android.feature` convention plugin을 사용하여 feature 모듈들의 공통 의존성을 관리한다. 이 plugin은 Circuit, Compose, 그리고 core 모듈들의 의존성을 자동으로 포함하므로, 각 feature 모듈의 build.gradle.kts에서는 특별한 의존성(예: libs.logger, libs.kakao.auth)만 별도로 선언하면 된다.

Applied to files:

  • README.md
📚 Learning: 2025-10-28T05:53:09.370Z
Learnt from: seoyoon513
Repo: YAPP-Github/Reed-Android PR: 204
File: app/src/main/kotlin/com/ninecraft/booket/ReedFirebaseMessagingService.kt:80-91
Timestamp: 2025-10-28T05:53:09.370Z
Learning: Reed-Android 프로젝트의 minSdk는 28이므로, API 26+ (Android O) 이상에서만 사용 가능한 API (예: NotificationChannel)에 대한 런타임 SDK 버전 체크를 제안하지 않아야 함

Applied to files:

  • README.md
🔇 Additional comments (3)
README.md (3)

3-3: Kotlin 버전 업데이트가 일관되게 적용되었습니다.

배지(라인 3)와 Development 섹션(라인 62)의 Kotlin 버전이 모두 2.2.21로 업데이트되어 문서의 일관성이 유지되었습니다.

Also applies to: 62-62


40-51: 이전 피드백이 성공적으로 반영되었습니다.

라인 46과 51의 중복된 링크 문제가 해결되었습니다. TroubleShooting 섹션이 재구성되어 모든 항목이 고유한 링크를 유지하고 있으며, 마크다운 형식도 일관되게 유지되었습니다.


83-84: 라이브러리 업데이트가 명확하게 문서화되었습니다.

마이그레이션 변경사항(Google ML Kit → Google Cloud Vision, Dagger Hilt → Metro)이 취소선과 화살표 형식으로 명확하게 표시되어 문서의 변경 이력을 추적하기 좋습니다.


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

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 67fd699 and 7af2cca.

📒 Files selected for processing (1)
  • README.md (1 hunks)
🔇 Additional comments (1)
README.md (1)

51-52: 라인 51-52의 의도를 명확히 해주세요.

라인 51은 라인 46과 정확히 동일한 중복된 항목이고, 라인 52는 AI 요약에서 "제거됨"이라고 표시된 링크입니다. PR 목표("중복 링크 제거")와 현재 상태가 맞지 않습니다. 다음을 확인해주세요:

  • 라인 51: 46번 줄의 중복이므로 제거되어야 함
  • 라인 52: 제거 의도였다면 남아있어야 하지 않음

@github-actions github-actions bot requested a review from seoyoon513 December 9, 2025 04:09
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: 0

🧹 Nitpick comments (1)
README.md (1)

83-84: Google Cloud Vision에 참조 링크를 추가하면 더 좋을 것 같습니다.

Development 섹션의 스택 업데이트가 좋습니다. 다만 Metro는 링크가 제공되어 있지만, Google Cloud Vision은 링크가 없어서 일관성이 떨어집니다. 사용자가 새로운 라이브러리에 쉽게 접근할 수 있도록 Google Cloud Vision에도 공식 문서 링크를 추가하는 것을 권장합니다:

- ~~Google ML Kit~~ -> Google Cloud Vision
+ ~~Google ML Kit~~ -> [Google Cloud Vision](https://cloud.google.com/vision)
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c272aef and bd2af22.

📒 Files selected for processing (1)
  • README.md (2 hunks)
🔇 Additional comments (1)
README.md (1)

40-51: 이전 리뷰 의견이 잘 반영되었습니다.

중복된 링크 문제가 성공적으로 해결되었습니다. 이전에 지적된 "[[Android] 일회성 이벤트를 StateFlow, Compose의 State로 처리할 때 주의해야할 점]" 중복이 제거되었으며, 현재 라인 46과 51은 서로 다른 항목입니다:

  • Line 46: 일회성 이벤트 처리 가이드
  • Line 51: ModalBottomSheet 드래그 처리법

TroubleShooting 섹션이 더욱 풍부한 참고 자료로 확충되어 좋습니다.

@easyhooon easyhooon merged commit 7e70487 into develop Dec 18, 2025
4 checks passed
@easyhooon easyhooon deleted the easyhooon-patch-1 branch December 18, 2025 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants