## 코드 리뷰 규칙

### 언어

- 모든 내용은 한국어를 사용할것.

### 프로그래밍의 기본베이스 관련

- SOLID 원칙을 준수하는지 체크
- clean architecture 원칙을 준수하는지 체크

### 클린아키텍쳐 관련

- [name].tsx 는 API 호출을 담당한다. 핸들러 선언을 담당한다.
- [name].view.tsx 는 UI 렌더링을 담당한다. 핸들러를 직접 선언하지 않는다. API를 직접 호출하지 않는다.

### 타입스크립트 관련

- any 타입을 사용하는지 체크

### 리액트 컴포넌트 구조 관련

- 컴포넌트안에서 css를 인라인으로 직접 선언하는지 체크

### md 파일관련하여 리뷰시 참고사항

- AI 프롬프트에에 주입되는 rules, instructions, forms 와 같은 성격의 md파일은 아래와 같은 항목을 추가적으로 검토하여 알려준다.
  - 서로 의미가 상충되는 내용이 존재하는 경우 사용자에게 알려준다. ( 단, 명시적으로 특정룰을 우선시하는 메시지가 있는 경우는 제외 )
  - 문맥적으로 오류나 오타일 가능성이 있는 부분을 알려준다.
  - 프롬프트 튜닝을 위해, 개선이 필요한 부분이 있는 경우 알려준다.
  - 그 밖에 프롬프트에 주입되는 md파일로서 체크해야할 부분들이 있다면 적절히 판단하여 알려준다.
- CHANGESET을 위한 md 파일의 검토는 아래 사항을 추가적으로 한다.
  - 수정된 코드 내용과 맞는 내용인지 검토한다.
  - 수정된 코드 내용의 누락이 있다면 알려준다.
- DOCS.md , README.md 와 같은 해당 패키지의 전체적인 가이드를 담당하는 md파일인 경우는 아래의 내용을 추가적으로 검토한다.
  - 수정 또는 추가된 내용이 잘 반영되었는지 검토한다.
- 리뷰가 불필요한 md파일의 경우는 skip 한다.

### 기타 리뷰시, 요구사항

- 공통 유틸이나 공통 컴포넌트화 해서, 일반화/추상화 가능한 코드가 있는경우 제안

- 개발적으로 더 좋은 가이드가 있으면 같이 알려줘.

- 답변은 반드시 한국어로 할것.

- 확인한 파일목록을 표현할것.

- 체크한 파일목록을 보여주고,파일명 옆에 체크리스트를 통과했는지 표시할것

- 체크리스트를 통과하지 못한 파일이 있다면, 해당 파일의 어떤 부분이 체크리스트를 통과하지 못했는지 표시할것

### ignore 파일들

- package.json

- \*.yml

- \*.lock

- dist폴더내 파일들

- node_modules 폴더내 파일들
