fix: 멘토링 등록 계약과 경력 입력 검증 흐름 main 백포트

[Hyeonjun0527]

🌱 연관된 이슈

• 원본 PR: fix: 멘토링 등록 계약과 경력 입력 검증 흐름을 정합하게 수정 #442

☘️ 작업 내용

• refactor/mentoring-registration-sqaush-2의 멘토링 등록 관련 수정사항을 main에 백포트했습니다.
• 공개 멘토 조회의 coreKeywords 파서를 백엔드 응답 계약(code, label)과 호환되도록 반영했습니다.
• 멘토 경력 입력에서 현재 재직 단일 선택과 종료기간 검증 흐름을 정합하게 맞췄습니다.
• 멘토링 등록 저장 완료 안내 문구 앞에 성공 표시 이모지를 추가했습니다.

🍀 참고사항

• 체리픽 커밋: 752b397d
• 원본 커밋: e718d52f
• main에 해당 변경이 없어서 별도 백포트 브랜치 backport/mentoring-registration-main으로 분리했습니다.

스크린샷 (선택)

• 없음

Summary by CodeRabbit

릴리스 노트

• 개선 사항
  • 멘토 등록 시 저장 완료 메시지에 확인 아이콘(✅)이 추가되어 시각적 피드백이 향상되었습니다.
  • 경력 항목 편집에서 "현재 직장" 체크박스 로직이 개선되어 동시에 여러 항목을 현재 직장으로 선택할 수 없도록 개선되었습니다.
  • 핵심 키워드 입력 처리의 검증 및 정규화 로직이 개선되었습니다.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
study-platform-client-dev	Error		Mar 24, 2026 0:19am

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

진행 상황 설명 (Walkthrough)

이 PR은 멘토 핵심 키워드에 대한 타입 정의를 개선하고 키워드 파싱 로직을 공유 헬퍼로 리팩토링합니다. 또한 경력 항목 에디터의 "재직 중" 상태 관리를 강화하고 UI 성공 메시지에 이모지를 추가합니다.

변경 사항 (Changes)

Cohort / File(s)	요약
타입 정의 업데이트 src/features/mentoring/api/mentor-api.types.ts	MentorReadableCoreKeywordResponseDto 유니온 타입 추가 및 ProfileResponseDto.coreKeywords 프로퍼티 타입 확장
핵심 키워드 파싱 로직 리팩토링 src/features/mentoring/api/mentor-core-keyword-contract.ts	readMentorCoreKeyword 공유 헬퍼 함수 도입 및 기존 함수들 리팩토링하여 코드 중복 제거, 타입 파싱 정규화 로직 통일
UI 및 메시징 업데이트 src/features/mentoring/model/use-mentor-registration-controller.ts, src/features/mentoring/ui/registration/mentor-career-entries-editor.tsx	저장 완료 메시지에 체크마크 이모지 추가, 경력 항목 에디터의 "현재 근무" 상태 관리 개선 및 중복 선택 방지 로직 추가

예상 코드 리뷰 노력 (Estimated Code Review Effort)

🎯 3 (중간) | ⏱️ ~22분

관련 가능성 있는 PR (Possibly Related PRs)

• 멘토링 등록 저장 흐름, 공개 노출 분기, 핵심키워드 정책 안정화 #439: 멘토 핵심 키워드 타입 및 파싱/정규화 변경사항이 PR #439에서 도입된 타입/헬퍼와 직접 겹치며 확장하고 있습니다 (동일 파일 및 연관 함수).

시 (Poem)

🐰 핵심 키워드들을 정렬하고,
경력을 꼼꼼히 다듬으며,
완료의 ✅ 마크 반짝이고,
중복은 막고 타입은 맑게,
멘토의 여정이 더욱 선명하네! 🌟

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

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 사항의 주요 내용을 명확하게 반영하고 있습니다. '멘토링 등록 계약과 경력 입력 검증 흐름 main 백포트'는 타입 정의 업데이트, 파서 로직 리팩토링, 경력 입력 검증 흐름 정렬, 그리고 성공 메시지 이모지 추가 등 실제 변경 내용을 적절히 요약합니다.

_{✏️ 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 backport/mentoring-registration-main

---

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