Conversation
가격이나 수량등을 위한 숫자 입력 패턴이 아닌, 책 페이지 입력이므로 선행 0은 어느 경우에나 제거하는게 맞음
versionName: 1.1.1 -> 1.1.2 versionCode: 5 -> 6
Walkthrough숫자-only 입력 변환 로직에서 단일 "0"을 더 이상 허용하지 않고 선행 0으로 간주해 공백으로 정리하도록 변경. Gradle 버전 카탈로그에서 앱 설정 버전/SDK 값을 1.1.2(코드 6), target/compileSdk 35로 갱신. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor User as User
participant TF as TextField
participant IT as digitOnlyInputTransformation
participant VM as ViewModel/Validator
User->>TF: 입력("0" / "01" / "123")
TF->>IT: onTextChanged(raw)
alt raw == "0" or startsWith("0")
IT-->>TF: ""
else raw is non-leading-zero digits
IT-->>TF: raw
end
TF->>VM: 제출 값(정규화된 문자열)
VM-->>User: 검증 결과(빈 값이면 미입력 취급)
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changes
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
gradle/libs.versions.toml (2)
6-7: 스토어 트랙과 버전 일관성 확인versionCode 6/1.1.2가 현재 배포 트랙(내부/알파/프로덕션)에서 단조 증가 조건을 만족하는지, 기존 핫픽스/베타 브랜치와 충돌이 없는지 최종 확인해 주세요. 릴리스 노트에도 “페이지 입력 선행 0 제거 및 0 페이지 방지”가 반영되면 좋습니다.
원하시면 CHANGELOG/릴리스 노트 초안을 작성해 드리겠습니다.
8-8: packageName를 [versions] 섹션에서 분리하는 것을 제안버전 카탈로그의 [versions]는 의미상 “버전” 전용 키를 두는 것이 가독성에 좋습니다. 식별자(applicationId/namespace 등)는 별도 섹션([app] 또는 [project])로 분리하면 유지보수성이 올라갑니다. 사용처가 넓다면 영향 범위를 고려해 점진 적용을 권장합니다.
적용 예시(이 파일 내 변경):
- packageName = "com.ninecraft.booket"그리고 동일 파일 하단 등 적절한 위치에 새 섹션 추가(참고용, 별도 변경 필요):
[app] packageName = "com.ninecraft.booket"마지막으로 Gradle 스크립트에서 참조 키도
libs.versions.packageName→libs.app.packageName로 치환해야 합니다. 아래 스캔으로 참조 지점을 파악할 수 있습니다.#!/bin/bash rg -nP -C2 '\blibs\.versions\.packageName\b' -g '!**/build/**'
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/component/textfield/InputTransformation.kt(0 hunks)gradle/libs.versions.toml(1 hunks)
💤 Files with no reviewable changes (1)
- core/designsystem/src/main/kotlin/com/ninecraft/booket/core/designsystem/component/textfield/InputTransformation.kt
⏰ 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). (1)
- GitHub Check: ci-build
🔇 Additional comments (2)
gradle/libs.versions.toml (2)
2-7: 버전/SDK 업데이트 LGTM
- versionName 1.1.2 / versionCode 6으로의 증분이 PR 목적(입력 검증 수정 배포)과 일치합니다.
- targetSdk/compileSdk 35 상향도 합리적입니다. 모듈 전반에서 카탈로그 값을 참조하도록만 일관성 유지하면 됩니다.
2-5: 코드 내 하드코딩된 SDK/버전 정보 검토 결과
compileSdk/targetSdk/minSdk하드코딩된 값이 빌드 스크립트 내에 더 이상 존재하지 않습니다.versionName/versionCode하드코딩된 설정도 발견되지 않았습니다.- 이전 SDK·앱 버전(예: versionCode=5, “1.1.1”) 흔적도 남아있지 않습니다.
위 스캔 결과, 모든 모듈이
gradle/libs.versions.toml카탈로그 값을 정상 참조하고 있어 코드 레벨에서는 추가 작업이 필요 없습니다.
다만, SDK 35 타깃 시 변경된 OS 동작(권한 처리, 백그라운드 제한 등)에 대한 QA 테스트 커버리지는 별도로 확인해 주세요.
|
요건 정책 정해지면 진행하는걸루! |
2 participants
🔗 관련 이슈
📙 작업 설명
🧪 테스트 내역 (선택)
📸 스크린샷 또는 시연 영상 (선택)
💬 추가 설명 or 리뷰 포인트 (선택)
Summary by CodeRabbit