refactor: Gathering 타입 정의 및 지역 상수 추가

[RookieAND]

🎯 PR 제목

Gathering 타입 정의 및 지역 상수 추가

📑 작업 상세 내역

1. OpinionForm 타입 필드 required 변경

• distanceRange, dislikedFoods, preferredMenus 필드를 optional에서 required로 변경
• 폼 제출 시 필수 입력 검증을 타입 레벨에서 보장

2. MeetingContext 타입 deprecated 표시

• @deprecated JSDoc 태그 추가
• 향후 API 응답 타입으로 대체 예정임을 명시

3. 지역 옵션 상수 정의

• src/constants/gathering/opinion/region.ts 생성
• 서울 지하철역 기반 지역 옵션 배열 추가
• DistanceStep에서 사용할 region 값 중앙 관리

🙏 리뷰 요청 사항

• OpinionForm 타입의 required 변경이 기존 코드에 breaking change를 발생시키지 않는지 확인 부탁드립니다.
• 지역 옵션 상수의 값이 실제 서비스에서 사용하는 지역명과 일치하는지 검토 부탁드립니다.

[gemini-code-assist]

Summary of Changes

Hello @RookieAND, 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은 모임 관련 데이터 모델의 견고성을 높이고, 향후 API 변경에 대비하며, 지역 선택 기능을 위한 기반 데이터를 마련합니다. 사용자 입력 폼의 필수 필드를 명확히 하여 데이터 무결성을 강화하고, 레거시 타입의 사용을 줄여 코드의 유지보수성을 개선합니다.

Highlights

• OpinionForm 타입 필드 필수화: distanceRange, dislikedFoods, preferredMenus 필드가 선택 사항에서 필수로 변경되어 폼 제출 시 유효성 검사가 타입 수준에서 보장됩니다.
• MeetingContext 타입 Deprecated 처리: MeetingContext 인터페이스에 @deprecated JSDoc 태그가 추가되어 향후 API 응답 타입으로 대체될 예정임을 명시합니다.
• 지역 옵션 상수 추가: src/constants/gathering/opinion/region.ts 파일이 새로 추가되어 서울 지하철역 기반의 지역 옵션(홍대입구역, 강남역)을 상수로 관리합니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

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은 OpinionForm 타입의 필드를 필수로 변경하고, MeetingContext 타입을 deprecated 처리하며, 지역 옵션 상수를 추가하는 리팩터링을 포함합니다. 코드 리뷰 결과, 타입 정의와 관련된 두 가지 개선점을 발견했습니다. src/constants/gathering/opinion/region.ts 파일에서 as const 사용 시 발생하는 타입 오류를 수정하고 스타일 가이드에 따라 interface 대신 type을 사용하도록 제안했습니다. 또한 src/types/gathering/index.ts 파일에서도 OpinionForm을 type으로 변경하여 코드 일관성을 높이도록 제안했습니다.

[gemini-code-assist]

현재 코드에는 as const로 생성된 readonly 타입을 변경 가능한 배열 타입(RegionOption[])에 할당하려고 하여 타입 에러가 발생합니다. 또한, 스타일 가이드(규칙 4. TypeScript Rules, 196행, 224행)에서는 interface보다 type 사용을 권장하며, 상수 객체에 as const를 사용하도록 하고 있습니다.

이 상수는 타입 추론을 활용하여 더 간결하고 타입-안전하게 정의할 수 있습니다. RegionOption 인터페이스를 제거하고 as const를 통해 TypeScript가 타입을 직접 추론하도록 하는 것을 제안합니다. 이렇게 하면 불필요한 타입 정의와 import 구문을 제거하고, 스타일 가이드에 더 잘 부합하는 코드가 됩니다.

export const REGION_OPTIONS = [
	{
		id: "HONGDAE",
		label: "홍대입구역",
	},
	{
		id: "GANGNAM",
		label: "강남역",
	},
] as const;

References

1. 스타일 가이드에서는 Props 정의에 interface보다 type을 선호한다고 명시되어 있습니다. 비록 Props가 아니더라도 일관성을 위해 type을 사용하는 것이 좋습니다. ^(link)
2. 스타일 가이드에서는 상수 객체에 as const를 사용하여 타입 안정성을 높일 것을 권장합니다. 타입 추론을 활용하면 더 간결한 코드를 작성할 수 있습니다. ^(link)

[gemini-code-assist]

스타일 가이드(규칙 4. TypeScript Rules, 196행)에서는 interface보다 type 사용을 권장하고 있습니다. OpinionForm을 type으로 변경하여 프로젝트 전반의 코드 스타일 일관성을 유지하는 것을 제안합니다.

Suggested change

	export interface OpinionForm {
	distanceRange?: DistanceRange;
	distanceRange: DistanceRange;
	dislikedFoods: FoodCategory[];
	preferredMenus: {
	first?: FoodCategory;
	second?: FoodCategory;
	third?: FoodCategory;
	first: FoodCategory;
	second: FoodCategory;
	third: FoodCategory;
	};
	}
	export type OpinionForm = {
	distanceRange: DistanceRange;
	dislikedFoods: FoodCategory[];
	preferredMenus: {
	first: FoodCategory;
	second: FoodCategory;
	third: FoodCategory;
	};
	};

References

1. 스타일 가이드에서는 Props 정의에 interface보다 type을 선호한다고 명시되어 있습니다. 일관성을 위해 다른 타입 정의에도 이 규칙을 적용하는 것이 좋습니다. ^(link)

[RookieAND]

추후 enum 상수로 치환할 예정이지만, 우선 작업의 단계적 접근을 위해 임시 조치

[RookieAND]

Mock API 구현을 위해 임시로 정의했던 Interface 에 대해서 Deprecated 라벨 부착

[youngminss]

LGTM 👍

[youngminss]

이 부분은 현재 운영에 merge 할 때 conflict 날수 있음 !