Skip to content

feat: 앱 소셜 로그인에 게스트 토큰 동봉 - #724

Merged
iOdiO89 merged 2 commits into
devfrom
feat/670-social-login-guest-token
Oct 10, 2026
Merged

iOdiO89 merged 2 commits into
devfrom
feat/670-social-login-guest-token

Conversation

@iOdiO89

@iOdiO89 iOdiO89 commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

작업 요약

  • 앱에서 게스트로 토너먼트에 참여한 뒤 소셜 로그인하면, 새 계정이 생기지 않고 게스트 기록이 회원 계정으로 이어지도록 했습니다.

작업 내용

  • postSocialLogin 이 저장된 access token 이 있으면 Authorization 헤더로 함께 전송
  • useSocialLogin 이 로그인 요청 직전 TokenStorage 의 토큰을 꺼내 전달 — 웹(clientApi 인터셉터)은 이미 보내던 것을 네이티브만 빠져 있던 것을 맞춤
  • 서버(core#1087)가 게스트 세션을 인지해 승격(같은 userId 유지)·기존 회원으로 플레이 승계가 동작
  • 참고: JS 변경뿐이라 머지 후 runtime 1.2.3 production OTA 로 배포 예정 (development 채널 OTA 로 승격 흐름 확인 완료)

연관 이슈

closes #670

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 새 기능
    • 소셜 로그인 요청에 저장된 게스트 액세스 토큰이 포함됩니다. 토큰이 있는 경우 인증 정보로 전달되며, 토큰이 없거나 비어 있으면 인증 헤더를 보내지 않습니다.
    • 소셜 로그인 성공 후 토큰 저장과 성공 메시지 표시 방식은 기존과 동일합니다.

iOdiO89 and others added 2 commits September 16, 2026 15:54
- postSocialLogin 이 저장된 access token 을 Authorization 헤더로 함께 전송
- 웹(clientApi 요청 인터셉터)은 이미 보내던 것을 네이티브만 빠져 있어 맞춤
- 서버가 게스트 세션을 인지해 승격(같은 userId 유지)·플레이 승계가 동작

closes #670
@iOdiO89 iOdiO89 self-assigned this Oct 10, 2026
@vercel

vercel Bot commented Oct 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
piki Ready Ready Preview Oct 10, 2026 10:19am UTC
piki-notify Ready Ready Preview Oct 10, 2026 10:19am UTC

@github-actions github-actions Bot added APP Good for newcomers feature New feature or request labels Oct 10, 2026
@github-actions
github-actions Bot requested a review from soyeong0115 October 10, 2026 10:19
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c47e67fd-dbb0-4ae4-ba57-d068f23e4e7b

📥 Commits

Reviewing files that changed from the base of the PR and between a9a13e9 and f2a7ffc.


📒 Files selected for processing (2)
  • apps/app/apis/postSocialLogin.ts
  • apps/app/hooks/useSocialLogin.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Walkthrough

Walkthrough

소셜 로그인 요청 전에 저장된 게스트 액세스 토큰을 조회합니다. 토큰이 있으면 요청의 Authorization 헤더에 Bearer 토큰으로 전달합니다.

Changes

게스트 토큰 소셜 로그인 전달

Layer / File(s) Summary
게스트 토큰 조회와 요청 전달
apps/app/hooks/useSocialLogin.ts, apps/app/apis/postSocialLogin.ts
useSocialLogin이 TokenStorage에서 게스트 액세스 토큰을 읽어 postSocialLogin에 전달합니다. 토큰이 truthy인 경우 요청에 Authorization: Bearer ... 헤더를 추가합니다.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature · Severity of issue fixed: Medium

Suggested reviewers: soyeong0115


Merge Risk: ⚪ Minimal · up to f2a7f

Native social login forwards a stored guest token to support guest-session promotion and omits the header when no token is available. No concrete client-side merge blocker was identified.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed PR 제목은 앱 소셜 로그인 요청에 게스트 토큰을 포함하는 핵심 변경 사항을 정확하고 간결하게 설명합니다.
Linked Issues check Passed 직접 연결된 #670의 핵심 요구사항을 충족합니다. useSocialLogin은 로그인 요청 전에 TokenStorage.getAccessToken()으로 저장 토큰을 읽습니다. postSocialLogin은 토큰이 있을 때 Authorization: Bearer <token> 헤더를 추가하고, 없을 때 헤더를 추가하지 않습니다. 이 동작은 …
Out of Scope Changes check Passed 변경 범위는 apps/app/apis/postSocialLogin.ts의 게스트 토큰 헤더 전달과 apps/app/hooks/useSocialLogin.ts의 토큰 조회·전달로 제한됩니다. 두 변경은 #670의 게스트 데이터 승계 목적에 직접 연결됩니다. 검토 가능한 변경에서 무관한 기능 변경은 확인되지 않았습니다.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR


🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

apps/app/apis/postSocialLogin.ts

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.


apps/app/hooks/useSocialLogin.ts

ESLint skipped: the matched ESLint configuration already failed (missing-dependency).



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.

@iOdiO89
iOdiO89 merged commit e2b45fa into dev Oct 10, 2026
11 checks passed
@iOdiO89
iOdiO89 deleted the feat/670-social-login-guest-token branch October 10, 2026 10:25

This branch was successfully deployed

2 active deployments
Preview – piki — f2a7ffc5 Deployed Oct 10, 2026 by vercel[bot]
Preview – piki-notify — f2a7ffc5 Deployed Oct 10, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

APP Good for newcomers feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

네이티브 앱 소셜 로그인에 게스트 토큰을 함께 보내야 한다 (게스트 데이터 승계)

2 participants