Skip to content

chore: Kakao iOS SDK 의존성 추가 - #103

Merged
gomminjae merged 1 commit into
developfrom
feat/kakao-sdk
Jun 2, 2026
Merged

gomminjae merged 1 commit into
developfrom
feat/kakao-sdk

Conversation

@gomminjae

Copy link
Copy Markdown
Collaborator

개요

카카오 로그인 도입을 위한 첫 단계 — SDK 의존성만 추가합니다. (로그인 로직/UI 없음)

변경

  • Tuist/Package.swift: kakao-ios-sdk 추가 (resolved 2.27.3), KakaoSDKCommon/Auth/User productType=framework
  • Modules/App/Project.swift: App 타겟에 KakaoSDKCommon/Auth/User 링크
  • 생성물(App.xcodeproj/Newdok.xcworkspace/Package.resolved) 갱신

검증

  • tuist install → Kakao SDK 2.27.3 해석
  • tuist generate + xcodebuild -scheme Newdok build → BUILD SUCCEEDED (Kakao 프레임워크 링크 확인)

범위 밖 (후속 PR)

  • KakaoSDK.initSDK(appKey:) 초기화 — 카카오 네이티브 앱키 필요
  • Info.plist: CFBundleURLSchemes kakao{앱키}, LSApplicationQueriesSchemes
  • onOpenURL에서 AuthController.handleOpenUrl
  • 백엔드 소셜 로그인 엔드포인트 연동 (KakaoAuthProviding 격리 + KakaoLoginUseCase + Repository/API)

필요 확인 (후속)

  • 카카오 네이티브 앱키
  • 백엔드 카카오 로그인 엔드포인트 req/res 규약
  • 신규 유저 처리(자동가입 vs 추가정보 입력)

- Package.swift에 kakao-ios-sdk(2.27.3) 추가, KakaoSDKCommon/Auth/User productType framework
- App 타겟에 KakaoSDKCommon/Auth/User 링크
- SDK 의존성/링크만 추가 (initSDK·Info.plist URL스킴·로그인 로직은 앱키/백엔드 확정 후 후속 PR)

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request integrates the Kakao iOS SDK (including KakaoSDKCommon, KakaoSDKAuth, and KakaoSDKUser) into the project using Tuist. The feedback suggests aligning the Kakao SDK version in Package.swift with the resolved version in Package.resolved to prevent unintended downgrades. Additionally, it is recommended to refactor the dependencies by moving KakaoSDKAuth and KakaoSDKUser to the Auth feature module, keeping only KakaoSDKCommon in the main App target for initialization.

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.

Comment thread Tuist/Package.swift
.package(url: "https://github.com/onevcat/Kingfisher.git", from: "8.0.0"),
.package(url: "https://github.com/exyte/PopupView.git", from: "4.1.13"),
.package(url: "https://github.com/firebase/firebase-ios-sdk.git", from: "11.0.0"),
.package(url: "https://github.com/kakao/kakao-ios-sdk.git", from: "2.22.0"),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

현재 Package.resolved에는 2.27.3 버전으로 명시되어 있으나, Package.swift에는 from: "2.22.0"으로 설정되어 있습니다. 최소 버전을 실제 사용 및 검증된 버전인 2.27.3으로 일치시켜 다른 환경이나 의존성 업데이트 시 의도치 않게 하위 버전이 설치되는 것을 방지하는 것이 좋습니다.

Suggested change
.package(url: "https://github.com/kakao/kakao-ios-sdk.git", from: "2.22.0"),
.package(url: "https://github.com/kakao/kakao-ios-sdk.git", from: "2.27.3"),

Comment thread Modules/App/Project.swift
Comment on lines +107 to +109
.external(name: "KakaoSDKCommon"),
.external(name: "KakaoSDKAuth"),
.external(name: "KakaoSDKUser")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

현재 KakaoSDKCommon, KakaoSDKAuth, KakaoSDKUser 의존성이 App 타겟에 직접 추가되어 있습니다. 프로젝트의 모듈화 구조를 고려할 때, 카카오 로그인 관련 비즈니스 로직과 UI는 Auth 피처 모듈 내부에서 구현될 예정이므로, KakaoSDKAuth와 KakaoSDKUser 의존성은 App 타겟이 아닌 Auth 피처 모듈의 Project.swift에 정의하는 것이 적절합니다. App 타겟에서는 SDK 초기화(KakaoSDK.initSDK)를 위해 KakaoSDKCommon만 필요하므로, 역할에 따라 의존성을 분리하여 모듈 간의 결합도를 낮추는 것을 권장합니다.

Suggested change
.external(name: "KakaoSDKCommon"),
.external(name: "KakaoSDKAuth"),
.external(name: "KakaoSDKUser")
.external(name: "KakaoSDKCommon")

@gomminjae
gomminjae merged commit 90e7f6a into develop Jun 2, 2026
1 check passed
@gomminjae
gomminjae deleted the feat/kakao-sdk branch June 2, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant