refactor: 멘토 도메인 응답의 사용자 id를 siteUserId로 통일

[Gyuhyeok99]

관련 이슈

• resolves: refactor: 멘토 도메인 응답의 사용자 id를 siteUserId로 통일한다. #663

작업 내용

as is

• siteuser_id, mentor_id로 응답값으로 나감

to be

• siteuser_id만 응답값으로 나가도록 변경

특이 사항

리뷰 요구사항 (선택)

[coderabbitai]

작업 개요

이 변경사항은 멘토 식별자 체계를 내부 멘토 ID에서 사이트 사용자 ID 기반으로 전환하는 리팩토링입니다. 다음과 같은 수정사항이 포함됩니다:

1. 컨트롤러 엔드포인트 수정: /{mentor-id} 경로를 /{site-user-id}로 변경하고 해당 파라미터명 업데이트
2. DTO 응답 변경: 네 개의 응답 클래스에서 mentor.getId() → mentor.getSiteUserId() 변환
3. 서비스 로직 개선: MentorQueryService와 MentoringCommandService의 멘토 조회 방식을 사이트 사용자 ID 기반으로 리팩토링
4. 테스트 동기화: 모든 관련 테스트에서 ID 참조를 사이트 사용자 ID로 일관되게 업데이트

코드 리뷰 예상 소요 시간

🎯 3 (Moderate) | ⏱️ ~20 분

추천 리뷰어

• wibaek
• whqtker
• lsy1307
• Hexeong

🚥 Pre-merge checks | ✅ 5 | ❌ 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 (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 멘토 도메인 응답의 사용자 id를 siteUserId로 통일하는 리팩터링이라는 변경의 핵심을 명확하게 전달합니다.
Description check	✅ Passed	PR 설명에 관련 이슈와 작업 내용(as-is/to-be)이 포함되어 있으나, 특이 사항과 리뷰 요구사항 섹션은 비어있습니다.
Linked Issues check	✅ Passed	변경사항이 이슈 #663의 목표(멘토 응답에서 id를 siteUserId로 통일)를 충족하고 있습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 멘토 도메인의 id 필드를 siteUserId로 통일하는 리팩터링 범위 내에 있습니다.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into develop

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In
`@src/main/java/com/example/solidconnection/mentor/controller/MentorController.java`:
- Around line 28-33: The controller changed the path variable meaning from
{mentor-id} to {site-user-id}, which breaks clients still passing the old
mentor-id; update MentorController.getMentorDetails to accept both legacy and
new path names by adding a dual mapping and resolving the value from the path
variable map: change `@GetMapping` to `@GetMapping`({"/{site-user-id}",
"/{mentor-id}"}) and replace the single `@PathVariable` parameter with
`@PathVariable` Map<String,String> pathVars, parse the id using
pathVars.getOrDefault("site-user-id", "mentor-id") (convert to long) and pass
that resolved mentorSiteUserId into mentorQueryService.getMentorDetails (also
add a short deprecation log when the legacy key "mentor-id" is used so we can
coordinate client rollout).

🧹 Nitpick comments (2)

src/main/java/com/example/solidconnection/mentor/service/MentoringCommandService.java (1)

34-38: MentoringApplyRequest.mentorId() 필드명을 mentorSiteUserId로 변경해주세요.

현재 코드에서 다음과 같은 이름 불일치가 발생하고 있습니다:

1. DTO 필드명과 실제 의미의 괴리

   • MentoringApplyRequest의 필드는 mentorId로 정의되어 있습니다
   • 하지만 실제로는 mentor1.getSiteUserId()로 값이 전달되고 있습니다
   • 서비스 레이어에서도 mentorSiteUserId 변수로 명확하게 변환되어 사용됩니다

2. 검증 메시지의 혼동 유발

   • 현재 검증 메시지는 "멘토 id를 입력해주세요"로 되어 있습니다
   • API 사용자에게 실제로는 siteUserId를 보내야 한다는 점이 명확하지 않습니다

3. 미완성 리팩토링

   • 서비스 메서드의 변수명과 저장소 쿼리 메서드명(findBySiteUserId)이 명확하게 siteUserId를 지칭합니다
   • DTO 필드명도 이에 맞게 통일하는 것이 코드 일관성을 높입니다

필드명을 mentorSiteUserId로 변경하면 향후 유지보수와 API 이해도가 크게 개선될 것입니다.

src/test/java/com/example/solidconnection/mentor/service/MentorQueryServiceTest.java (1)

119-127: 변수명을 리팩터링 컨텍스트에 맞게 수정하면 좋겠습니다.

getMentorDetails의 첫 번째 파라미터가 이제 mentorSiteUserId를 받도록 변경되었으므로, 테스트의 변수명도 이에 맞추면 가독성이 향상됩니다.

♻️ 변수명 수정 제안

-            long notExistingMentorId = 999L;
+            long notExistingSiteUserId = 999L;

             // when & then
-            assertThatCode(() -> mentorQueryService.getMentorDetails(notExistingMentorId, siteUserFixture.사용자().getId()))
+            assertThatCode(() -> mentorQueryService.getMentorDetails(notExistingSiteUserId, siteUserFixture.사용자().getId()))

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

API 경로 변수 변경에 따른 하위 호환성 확인이 필요합니다.

1. 엔드포인트가 /{mentor-id} → /{site-user-id}로 변경되었습니다.
2. URL 패턴 자체(/mentors/{value})는 동일하지만, 클라이언트가 전달해야 하는 ID 값의 의미가 내부 mentor ID에서 siteUserId로 바뀌었습니다.

기존 클라이언트(프론트엔드, 모바일 등)가 이전 응답에서 캐싱한 mentor ID를 사용하고 있다면 호환성 문제가 발생할 수 있으니, 클라이언트 측 배포 타이밍을 함께 조율해 주세요.

🤖 Prompt for AI Agents

In
`@src/main/java/com/example/solidconnection/mentor/controller/MentorController.java`
around lines 28 - 33, The controller changed the path variable meaning from
{mentor-id} to {site-user-id}, which breaks clients still passing the old
mentor-id; update MentorController.getMentorDetails to accept both legacy and
new path names by adding a dual mapping and resolving the value from the path
variable map: change `@GetMapping` to `@GetMapping`({"/{site-user-id}",
"/{mentor-id}"}) and replace the single `@PathVariable` parameter with
`@PathVariable` Map<String,String> pathVars, parse the id using
pathVars.getOrDefault("site-user-id", "mentor-id") (convert to long) and pass
that resolved mentorSiteUserId into mentorQueryService.getMentorDetails (also
add a short deprecation log when the legacy key "mentor-id" is used so we can
coordinate client rollout).

[Gyuhyeok99]

@coderabbitai
그런 거 없는데 그냥 신경 안써도 되지 하위 호환성?

[coderabbitai]

@Gyuhyeok99 알겠습니다! 클라이언트 측 조율이나 배포 상황을 이미 파악하고 계신 것으로 이해했습니다. 불필요한 우려를 드려 죄송합니다. 👍