Develop

[g-hyxn]

💡 배경 및 개요

PR을 하게 된 문제상황, 배경 및 개요에 대해서 작성해주세요!

퍼블리싱의 경우 스크린샷/동영상도 추가해주면 좋아요!

Resolves: #{이슈번호}

📃 작업내용

PR에서 한 작업을 작성해주세요!

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!

또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

• 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
• 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
• 작업한 코드가 정상적으로 동작하나요?
• Merge 대상 브랜치가 올바른가요?
• PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

[vercel]

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

Project	Deployment	Review	Updated (UTC)
cklob-gami	Ready	Preview, Comment	Dec 29, 2025 7:24am

[gemini-code-assist]

Summary of Changes

Hello @g-hyxn, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 사용자 프로필 및 회원가입 페이지에서 사용되는 전공(major) 카테고리를 확장하고 관련 매핑을 업데이트하여 시스템의 유연성을 높이고 데이터 일관성을 확보합니다. 이를 통해 더 다양한 전공 분야를 지원하고, 사용자 경험을 개선하는 기반을 마련합니다.

Highlights

• 전공 목록 확장: 사용자 프로필(MemberProfile) 인터페이스의 major 타입에 'MOBILE_ROBOTICS', 'DEVOPS', 'AI', 'IT_NETWORK', 'FLUTTER', 'CYBER_SECURITY', 'GAME_DEVELOP', 'CLOUD_COMPUTING' 등 새로운 전공 옵션들이 추가되었습니다.
• 전공-관심사 매핑 업데이트: majorToInterestMap 객체에 새로 추가된 전공들에 대한 매핑이 추가되었으며, 기존 'ANDROID'의 매핑이 'AOS'에서 'Android'로 변경되었습니다.
• 전공 ID 일관성 유지: 회원가입 페이지(SignupPage.tsx)에서 'Game Development' 전공의 ID가 GAME_DEVELOPMENT에서 GAME_DEVELOP로 수정되어, 확장된 전공 목록과의 일관성을 맞췄습니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

이번 PR은 선택 가능한 전공 목록을 확장하는 내용을 담고 있습니다. 변경 사항 자체는 올바르게 적용되었으나, 이로 인해 MyPage.tsx, SignupPage.tsx 등 여러 파일에 걸쳐 전공 데이터가 중복으로 관리되고 있는 구조적인 문제가 드러났습니다. 코드의 유지보수성을 크게 향상시키고 비효율적인 로직을 개선하기 위해, 전공 데이터를 한 곳에서 관리하는 '단일 정보 소스(Single Source of Truth)' 방식으로 리팩토링하는 상세한 방법을 리뷰 코멘트로 제안드렸습니다.

[gemini-code-assist]

전공(major) 목록이 여러 곳에 중복으로 정의되어 있어 유지보수성을 저해하고 있습니다.

1. 이 파일 내에서의 중복: MemberProfile 인터페이스의 major 타입과 majorToInterestMap 객체에 전공 목록이 중복으로 하드코딩되어 있습니다.
2. 파일 간의 중복: src/pages/signup/SignupPage.tsx의 interestList와도 사실상 동일한 데이터가 중복되고 있습니다.

이러한 중복을 해결하고 코드의 효율성과 유지보수성을 높이기 위해 전공 데이터를 한 곳에서 관리하는 '단일 정보 소스(Single Source of Truth)'를 만드는 것을 강력히 권장합니다. 예를 들어, src/assets/shared/ 디렉토리에 majors.ts 같은 파일을 만들어 전공 데이터를 중앙에서 관리할 수 있습니다.

제안하는 리팩토링:

1. 공유 데이터 파일 생성 (src/assets/shared/majors.ts):

   export const majors = [
     { id: 'FRONTEND', label: 'FE' },
     { id: 'BACKEND', label: 'BE' },
     { id: 'ANDROID', label: 'Android' },
     { id: 'IOS', label: 'iOS' },
     { id: 'MOBILE_ROBOTICS', label: 'Mobile Robotics' },
     { id: 'DESIGN', label: 'Design' },
     { id: 'DEVOPS', label: 'DevOps' },
     { id: 'AI', label: 'AI' },
     { id: 'IT_NETWORK', label: 'IT Network' },
     { id: 'FLUTTER', label: 'Flutter' },
     { id: 'CYBER_SECURITY', label: 'Cyber Security' },
     { id: 'GAME_DEVELOP', label: 'Game Development' },
     { id: 'CLOUD_COMPUTING', label: 'Cloud Computing' },
   ] as const;
   
   export type MajorId = typeof majors[number]['id'];
   export type MajorLabel = typeof majors[number]['label'];
   
   export const majorToInterestMap = Object.fromEntries(
     majors.map(m => [m.id, m.label])
   ) as Record<MajorId, MajorLabel>;
   
   export const interestToMajorMap = Object.fromEntries(
     majors.map(m => [m.label, m.id])
   ) as Record<MajorLabel, MajorId>;

2. MyPage.tsx 수정:

   • major 타입을 MajorId로 교체하고, majorToInterestMap과 interestToMajorMap을 import하여 사용합니다.
   • 비효율적인 역방향 탐색 로직(Object.keys(...).find(...))을 interestToMajorMap을 사용한 O(1) 탐색으로 개선할 수 있습니다.

   // handleCategorySave 함수 내부
   import { interestToMajorMap } from '@/assets/shared/majors';
   
   const majorKey = interestToMajorMap[tempSelectedInterest];

3. SignupPage.tsx 수정:

   • interestList를 하드코딩하는 대신 majors를 import하여 사용합니다.

이렇게 변경하면 향후 전공 목록이 변경될 때 공유 파일 하나만 수정하면 되므로 코드 관리가 훨씬 쉬워지고, 성능도 개선됩니다.