[FIX] 매직넘버 리턴값 code에서 phone으로 변경

[seoyeonjin]

Related Issue 🚀

• closed [FIX] 매직넘버 리턴값을 code에서 phone으로 변경 #206

Work Description ✏️

• 매직넘버 리턴값 code에서 phone으로 변경

PR Point 📸

[height]

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

[coderabbitai]

Summary by CodeRabbit

• 버그 수정
  • 매직 로그인 인증 완료 시 표시/반환되던 값이 인증 코드에서 전화번호로 바로잡혀, 계정 식별 정보가 일관되고 정확하게 제공됩니다. 이로써 인증 결과 해석 과정에서의 혼선을 줄이고, 이후 단계(자동 로그인/계정 연동 등)에서 잘못된 코드 표시로 인한 오류 가능성을 낮춥니다. 처리 흐름과 오류 동작에는 변화가 없습니다.

Walkthrough

VerifyVerificationService.verify의 매직 인증 분기에서 VerifyVerificationResult의 두 번째 값이 magicLoginProperty.code()에서 magicLoginProperty.phone()으로 변경되었으며, 나머지 흐름과 예외 처리는 그대로입니다.

Changes

Cohort / File(s)	Summary
Auth verification return payload tweak src/main/java/sopt/makers/authentication/application/service/auth/VerifyVerificationService.java	매직 인증 경로에서 VerifyVerificationResult 생성 시 두 번째 필드를 code() → phone()으로 교체. 다른 로직/흐름 변경 없음.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

깡충, 코드 대신 전화가 뿅! 📱
매직문 열리면 숫자 아닌 폰이 쏙—
리턴 값만 살짝, 흐름은 그대로
새벽 빌드에 달빛이 미소 짓고
토끼는 수염 정리, “머지 완료!” 🐇✨

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title Check	✅ Passed	제목이 변경된 주된 내용을 정확히 요약하여 magic-number 반환 키를 code에서 phone으로 변경한 핵심 변경점을 명확히 전달하고 있어 적절합니다.
Linked Issues Check	✅ Passed	이 PR은 VerifyVerificationService.verify 메서드에서 magic-number 반환 필드를 code에서 phone으로 변경하여 이슈 #206의 주요 요구사항을 정확히 충족하고 있습니다.
Out of Scope Changes Check	✅ Passed	변경 내역이 오직 magic-number 반환 키 변경에만 국한되어 있으며 다른 기능이나 파일에는 불필요한 수정이 없습니다.
Description Check	✅ Passed	제공된 설명이 레포지토리 템플릿의 Related Issue, Work Description, PR Point 섹션을 모두 포함하고 있어 기본 요구사항을 충족하며 각 섹션이 올바르게 작성되어 있습니다.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/#206

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 856ad84 and f31bec6.

📒 Files selected for processing (1)

• src/main/java/sopt/makers/authentication/application/service/auth/VerifyVerificationService.java (1 hunks)

🔇 Additional comments (1)

src/main/java/sopt/makers/authentication/application/service/auth/VerifyVerificationService.java (1)

27-27: 매직 로그인 분기에서도 전화번호가 반환되도록 맞춰주셔서 좋습니다.

일반 인증 플로우와 동일하게 VerifyVerificationResult 두 번째 인자가 전화번호로 유지되어 응답 스키마가 일관됩니다.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[JungYoonShin]

수고하셨습니다~~