Skip to content

refactor(core): 카카오 토큰 헬퍼를 core:common 으로 올린다 (#2194) - #2201

Merged
OnlyReadOneBook merged 1 commit into
developfrom
refactor/2194-core-kakao-token-helper
Oct 4, 2026
Merged

OnlyReadOneBook merged 1 commit into
developfrom
refactor/2194-core-kakao-token-helper

Conversation

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator

먼저 볼 파일: core/common/.../auth/KakaoAccessToken.kt, core/common/build.gradle.kts, feature/setting/.../account/ConnectedAccountsScreen.kt입니다.

Closes #2194

무엇을

  • 설정 KakaoLinkHelper.kt 의 requestKakaoAccessToken 을 core:common com.afternote.core.common.auth 로 옮겼다. 본문은 한 글자도 바꾸지 않았다(앱 로그인 우선, 취소는 CoreAuthFailure.UserCancelledAuth, 그 밖의 오류는 계정 로그인 폴백). KDoc 은 더 자세했던 온보딩 쪽 설명을 바탕으로 썼다.
  • core:common 에 core:domain(CoreAuthFailure)과 카카오 SDK 두 개 의존을 더했다. core:domain 은 core:model 에만 의존해 순환이 없다.
  • 설정은 ConnectedAccountsScreen 의 import 한 줄만 바꿨고, 더는 쓰지 않는 카카오 SDK 의존 두 줄을 걷었다.

왜

온보딩 KakaoLoginHelper 와 설정 KakaoLinkHelper 는 d91bbdd 가 복사한 같은 코드다. 설정이 온보딩을 의존할 수 없어 생긴 사본이고, 85033b0 의 CoreAuthFailure 전환 때 두 벌을 똑같이 고쳐야 했다. 헬퍼가 Activity 와 SDK 를 쓰므로 JVM 모듈인 core:domain 이 아니라, 이미 생체 인증 관문·영상 실행기 같은 플랫폼 래퍼가 있는 core:common 에 둔다.

범위 밖

검증

  • :core:common:testDebugUnitTest(47), :feature:setting:presentation:testDebugUnitTest(101), 두 모듈 ktlintCheck, :app:compileDebugUnitTestKotlin, :konsist:test --rerun-tasks(56) 통과
  • 설정 스크린샷 렌더 19장이 변경 전후 바이트 단위로 같다. 로컬 validateDebugScreenshotTest 는 base 에서도 같은 12건이 실패해(로컬 렌더와 CI 골든의 차이) 판정은 CI 결과로 한다.
  • ConnectedAccountsScreen.kt 는 교체 대상(refactor(setting): 남은 이벤트 스트림 2곳을 UiState·MVI로 전환하고 재유입을 막는다 #1502)이지만 import 한 줄만 바뀌어 rewrite_guard 가 기계적 변경으로 통과시켰다.

CI Test Plan

{"androidTest": {"mode": "none", "reason": "카카오 토큰 헬퍼의 모듈 이동과 설정 화면의 import 교체뿐이다. 함수 본문과 화면·네비게이션·계측 테스트 파일은 바뀌지 않았고 스크린샷 렌더도 같다."}}

설정 KakaoLinkHelper.kt 의 requestKakaoAccessToken 을 core:common auth 패키지로 옮기고
동작·예외 문구·취소 판정(CoreAuthFailure.UserCancelledAuth)은 그대로 둔다.
연결된 계정 화면은 import 만 core 로 바꾸고, 설정 모듈에서 쓰지 않게 된 카카오 SDK 의존을 걷는다.
@github-actions github-actions Bot added the sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다 label Oct 1, 2026
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

열린 PR 중 이 PR 과 같은 파일을 서로 다르게 고친 것이 있다. 어느 쪽이 먼저 머지되든 나중 쪽은 base 를 병합해야 통과한다 (#1750).

머지 자체는 GitHub 이 막는다 — 한쪽이 들어가면 다른 쪽은 CONFLICTING 이 되어 merge queue 에 못 들어간다.
이 라벨은 그 전에 순서를 정하라는 신호다. 먼저 갈 쪽을 정하고, 나중 쪽은 그 머지 뒤에 base 를 병합한다.
상대 PR 에도 같은 안내가 달린다.

이 코멘트는 리컨사일러가 매 실행 현재 상태로 고쳐 쓴다. 해소되면 sibling-conflict 라벨은 자동으로 떨어진다.

@Sadturtleman Sadturtleman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

함수 본문이 그대로 옮겨졌고 core:common → core:domain 의존에 순환이 없는 것을 확인했습니다. 승인합니다.

@OnlyReadOneBook
OnlyReadOneBook added this pull request to the merge queue Oct 4, 2026
Merged via the queue into develop with commit cf42c1d Oct 4, 2026
28 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the refactor/2194-core-kakao-token-helper branch October 4, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sibling-conflict 열린 형제 PR 과 같은 파일에서 충돌 — 나중에 머지되는 쪽이 base 를 병합해야 한다. 해소되면 자동으로 떨어진다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(core): 카카오 토큰 헬퍼를 core:common 으로 올린다

3 participants