Release/1.3.0 - #5
Conversation
- TapTarget.kt 파일의 CompositionLocal 관련 import 문 정리 및 상태 관리 로직 개선 - App.kt 파일에서 TapTargetCoordinator의 파라미터 구조 변경 및 인덱스 변경 로그 추가 - libs.versions.toml 파일에서 AGP, Kotlin, Compose Plugin, AndroidX 버전 업데이트 - gradle-wrapper.properties 파일에서 Gradle 버전 9.1.0에서 9.5.1로 업데이트
…t.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…t.kt Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
- TapTargetCoordinator 함수로 로직 분리 및 상태 관리 개선 - Modifier 확장 함수를 통해 TapTarget 정의 방식 변경 - TapTargetScope 및 TapTargetDefinition 클래스 추가로 컴포지션 로컬 활용 개선
index 추적 기능 및 ifTapTarget 추가
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 풀프레스트는 Kotlin 2.4.0, AGP 9.1.0, Gradle 9.5.1 등 주요 의존성 버전을 업데이트하고, TapTargetCoordinator에 onIndexChanged 콜백을 추가하며 CompositionLocalProvider를 활용한 Modifier.ifTapTarget 확장 함수를 도입했습니다. 리뷰에서는 두 가지 개선 사항이 제안되었습니다. 첫째, state.currentTargetIndex가 증가할 때 인덱스가 유효한 범위 내에 있을 때만 onIndexChanged를 호출하도록 예외 방지 처리를 적용하는 것입니다. 둘째, 성능 오버헤드를 줄이기 위해 Modifier.composed 대신 @Composable 확장 함수를 사용하여 ifTapTarget을 구현하는 것을 권장합니다.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
- gradle-wrapper.properties 파일의 distributionSha256Sum 값 변경 - 빌드 환경 일관성 유지 위한 의존성 해시 업데이트 - gradle 빌드 정상 동작을 위한 필수 수정
- Modifier.ifTapTarget 함수에 @composable 어노테이션 추가 및 로직 개선 - TapTargetDefinition, TapTargetCoordinatorState, TapTarget, TextDefinition, TapTargetStyle 데이터 클래스 정의 순서 변경 - 코드 구조 개선을 위한 파일 내부 정의 순서 조정
- TapTargetCoordinator 상태 변경 시 onIndexChanged 콜백 호출 로직 개선 - LaunchedEffect를 사용한 현재 타겟 인덱스 변경 감지 로직 추가 - CompositionLocalProvider 내부에서 타겟 좌표 계산 및 표시 로직 정리
|
/gemini review |
There was a problem hiding this comment.
Code Review
이번 풀 리퀘스트에서는 TapTargetCoordinator에 onIndexChanged 콜백을 추가하고, LocalTapTargetScope와 Modifier.ifTapTarget 확장 함수를 도입하여 사용성을 개선하였으며, Kotlin 및 Gradle 등 의존성 버전을 업데이트하였습니다. 코드 리뷰에서는 두 가지 중요한 개선 사항이 제시되었습니다. 첫째, TapTargetCoordinatorState에서 currentTargetIndex를 tapTargets 맵의 키로 직접 사용하면 비연속적인 우선순위 지정 시 가이드가 중단될 수 있으므로, 키를 정렬한 후 인덱스로 안전하게 조회하도록 수정해야 합니다. 둘째, Modifier 반환 함수에 @Composable을 직접 사용하는 대신 composed를 사용하여 Compose의 Modifier 체이닝 관례를 준수하도록 개선할 것을 권장합니다.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
- onIndexChanged를 onTargetChanged로 변경 - 타겟 변경 시 precedence 값으로 변경하여 콘솔 출력 로직 개선 - ifTapTarget Modifier 내부 로직 재구성 및 null 체크 개선
1 participant
No description provided.