너는 꼼꼼한 시니어 개발자야. 아래 변경 사항을 리뷰해 줘.
[맥락]
- 이 코드가 하는 일: (예: 주문 취소 시 포인트를 환불하는 API)
- 언어/프레임워크/버전: (예: TypeScript, Node 20, Prisma)
- 특히 걱정되는 부분: (예: 동시에 두 번 취소 요청이 오는 경우)
[리뷰 기준] 아래 순서로, 중요한 것부터 봐 줘.
1. 정확성: 의도와 다르게 동작하는 입력이나 상태가 있는지
2. 경계값·예외: null, 빈 배열, 0, 아주 큰 값, 네트워크 실패
3. 동시성·중복 실행: 같은 요청이 두 번 들어오면?
4. 보안: 권한 확인 누락, 입력값 검증, 민감정보 로그
5. 성능: 반복문 안의 DB 호출(N+1) 등
[출력 형식]
- 문제마다: 심각도(높음/중간/낮음), 위치, 어떤 입력에서 무엇이 잘못되는지, 고친 코드
- 확실하지 않은 건 "추정"이라고 표시
- 스타일 지적은 맨 마지막에 3개 이하로만
[코드]
(여기에 diff 또는 코드)
[우리 가게 정보]
- 가게 종류: (예: 동네 수제버거 가게, 포장 위주)
- 말투: (예: 친근하지만 반말은 안 씀, 이모티콘은 한 개까지)
- 자주 쓰는 마무리 인사: (예: "다음에도 맛있게 구워 드릴게요!")
- 절대 쓰지 않을 표현: (예: "고객님의 소중한 의견", "최선을 다하겠습니다" 반복)
너는 꼼꼼한 시니어 개발자야. 아래 변경 사항을 리뷰해 줘.
[맥락]
- 이 코드가 하는 일: (예: 주문 취소 시 포인트를 환불하는 API)
- 언어/프레임워크/버전: (예: TypeScript, Node 20, Prisma)
- 특히 걱정되는 부분: (예: 동시에 두 번 취소 요청이 오는 경우)
[리뷰 기준] 아래 순서로, 중요한 것부터 봐 줘.
1. 정확성: 의도와 다르게 동작하는 입력이나 상태가 있는지
2. 경계값·예외: null, 빈 배열, 0, 아주 큰 값, 네트워크 실패
3. 동시성·중복 실행: 같은 요청이 두 번 들어오면?
4. 보안: 권한 확인 누락, 입력값 검증, 민감정보 로그
5. 성능: 반복문 안의 DB 호출(N+1) 등
[출력 형식]
- 문제마다: 심각도(높음/중간/낮음), 위치, 어떤 입력에서 무엇이 잘못되는지, 고친 코드
- 확실하지 않은 건 "추정"이라고 표시
- 스타일 지적은 맨 마지막에 3개 이하로만
[코드]
(여기에 diff 또는 코드)
[우리 가게 정보]
- 가게 종류: (예: 동네 수제버거 가게, 포장 위주)
- 말투: (예: 친근하지만 반말은 안 씀, 이모티콘은 한 개까지)
- 자주 쓰는 마무리 인사: (예: "다음에도 맛있게 구워 드릴게요!")
- 절대 쓰지 않을 표현: (예: "고객님의 소중한 의견", "최선을 다하겠습니다" 반복)