Fix invite share domain

[chanho0908]

No description provided.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

📋 Walkthrough

이 PR은 초대 기능의 딥링크 호스트를 keepiluv.jiyong.xyz에서 keepiluv.teamtwix.com으로 마이그레이션합니다. InviteLaunchDispatcher의 상수와 AndroidManifest의 인텐트 필터를 동기화하고, 새 도메인으로 생성되는 딥링크가 올바른지 검증하는 단위 테스트와 통합 테스트를 추가합니다. 동시에 공유 메시지 문자열에서 불필요한 플레이스홀더를 제거하고 UI 로직을 정리합니다.

🎯 Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

---

💡 상세 리뷰 의견

1️⃣ 도메인 마이그레이션 구조 — 일관성 있고 명확합니다

긍정점:

• ✅ 상수 변경(InviteLaunchDispatcher.kt)과 매니페스트 업데이트(AndroidManifest.xml)가 동기화되어 있어서 일관성이 있습니다.
• ✅ 딥링크 생성과 파싱 양쪽을 모두 검증하는 테스트(단위 + 통합)가 함께 추가되어 신뢰성이 높습니다.

제안:
이 정도 규모의 도메인 변경이라면, commit message나 PR 설명에 "이 도메인 변경이 프로덕션 URL 변경, DNS 설정 변경, 웹서버 인증서 변경 등 인프라 변경과 동기화되었는지" 명시하는 것이 좋습니다. 딥링크는 OS 레벨에서 검증되므로 불일치 시 초대 공유가 작동하지 않을 수 있습니다.

---

2️⃣ 테스트 커버리지 — 양쪽 방향을 모두 검증합니다

긍정점:

• ✅ InviteLaunchDispatcherTest.kt: buildInviteDeepLink()가 정확한 URL을 반환하는지 검증 (outbound)
• ✅ InviteLaunchDispatcherAndroidTest.kt: dispatchFromIntent()가 딥링크 intent에서 코드를 올바르게 추출하는지 검증 (inbound)
• ✅ 두 테스트 모두 새 도메인 keepiluv.teamtwix.com을 명시적으로 검증합니다.

질문:

• 향후 도메인이 다시 변경될 경우를 대비해, INVITE_WEB_HOST 상수를 테스트에도 주입받도록(예: 공유 상수 정의) 리팩터링할 여지가 있지 않을까요? 현재는 URL 문자열이 두 곳에 하드코딩되어 있습니다(InviteLaunchDispatcher.kt의 상수값과 테스트의 URL 문자열).

---

3️⃣ 공유 메시지 문자열 수정 — 문맥 확인 권장

관찰:

• onboarding_invite_share_message에서 %3$s 플레이스홀더가 제거되었습니다.
• CoupleConnectScreen에서도 Constants.PLAY_STORE_URL을 getString에 전달하지 않도록 변경되었습니다.

질문:

• 이 %3$s 자리가 원래 PLAY_STORE_URL을 표시했던 것인가요? 제거 후 사용자가 보는 공유 메시지는 "연결 코드(1번) + 안내문(2번)" 구조가 되는 건가요?
• 만약 PLAY_STORE_URL 링크가 여전히 필요하다면, 다른 방식(예: 별도 버튼)으로 제공되고 있는지 확인해주세요.

---

4️⃣ 코드 정리 — Import 제거가 깔끔합니다

긍정점:

• ✅ CoupleConnectScreen.kt에서 Constants import가 제거되어, 사용하지 않는 의존성이 정리되었습니다.
• ✅ 불필요한 상수 참조가 사라져 유지보수 부담이 줄어듭니다.

---

🔍 최종 체크리스트

• ☑️ 도메인 변경이 일관되게 적용됨 (상수 + 매니페스트)
• ☑️ 단위 + 통합 테스트로 양방향 검증
• ☑️ Import 및 불필요한 인자 정리 완료
• ❓ 확인 필요: PLAY_STORE_URL 제거 후 사용자 UI에 문제 없는지 (기획/QA 단계)
• ❓ 확인 필요: 프로덕션 도메인(teamtwix.com)에 대한 인프라 준비 완료 여부

전반적으로 명확하고 구조적인 변경입니다. 도메인 일관성과 테스트 커버리지가 좋습니다! 🎉

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경 사항의 핵심을 명확하게 요약하고 있습니다. 초대 공유 도메인 변경이라는 주요 변경 사항을 간결하고 이해하기 쉽게 표현했습니다.
Description check	✅ Passed	PR 설명이 변경 사항, 목적, 테스트 추가 내용을 명확히 설명하고 있으며, 리뷰어를 위한 추가 요청사항도 포함되어 있습니다.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/invite-share-domain-fix

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}