시니어 엔지니어 코드 리뷰 어시스턴트

[여기에 코드를 붙여넣으세요] ``` ## 컨텍스트 (선택 사항): - 이 코드의 목적: [무엇을 하는지] - 더 큰 시스템의 일부: [예/아니오 - 간단한 설명] - 성능 요구 사항: [특정 요구 사항이 있는 경우] - 이것은: [새 기능 / 버그 수정 / 리팩터링] ## 포괄적인 코드 리뷰를 제공해 주세요: ### 1. 요약 간단한 전체 평가로 시작하세요: - 전체 코드 품질 (1-10 척도, 근거 포함) - 주요 장점 (2-3개 불릿 포인트) - 우선 개선 영역 (2-3개 불릿 포인트) - 승인 권장 사항: [승인 / 변경 요청 / 논의 필요] ### 2. 중요 이슈 (수정 필수) 각 중요 이슈에 대해: 🔴 **이슈 제목** - 위치: [파일:줄 또는 함수 이름] - 문제: [이슈에 대한 명확한 설명] - 영향: [무엇이 잘못될 수 있는지 - 보안, 데이터 손실, 충돌] - 해결책: [코드 예시가 포함된 구체적인 수정] ```[언어] // 이전 (문제가 있는 코드) [현재 코드] // 이후 (수정된 코드) [개선된 코드] ``` ### 3. 보안 분석 다음을 확인하세요: - [ ] 입력 검증 및 삭제 - [ ] SQL 인젝션 취약점 - [ ] XSS 취약점 - [ ] 인증/권한 부여 문제 - [ ] 민감한 데이터 노출 - [ ] 안전하지 않은 의존성 - [ ] CSRF 보호 - [ ] 속도 제한 고려 사항 각 발견 사항에 대해 공격 벡터와 완화 방법을 설명하세요. ### 4. 성능 리뷰 분석: - 알고리즘의 시간 복잡도 (Big O 표기법) - 공간 복잡도 - 데이터베이스 쿼리 효율성 (N+1 문제, 누락된 인덱스) - 메모리 누수 가능성 - 불필요한 계산 - 캐싱 기회 - Async/await 사용 해당되는 경우 벤치마킹 제안을 제공하세요. ### 5. 코드 품질 및 유지보수성 평가: - **네이밍**: 변수, 함수, 클래스가 명확하게 명명되었나요? - **단일 책임**: 각 함수가 한 가지 일을 하나요? - **DRY**: 코드 중복이 있나요? - **복잡성**: 순환 복잡성이 합리적인가요? - **오류 처리**: 오류가 적절하게 처리되나요? - **주석**: 주석이 필요하고 도움이 되나요? - **테스팅**: 이 코드는 테스트 가능한가요? 어떤 테스트가 필요한가요? ### 6. 아키텍처 및 디자인 패턴 고려 사항: - 코드베이스의 기존 패턴을 따르나요? - 이 사용 사례에 더 나은 디자인 패턴이 있나요? - 추상화 수준이 적절한가요? - 의존성이 적절하게 관리되나요? - 이 코드는 향후 요구 사항에 확장 가능한가요? ### 7. 제안 (있으면 좋은 것) 코드를 개선할 우선순위가 낮은 개선 사항: 🟡 **제안 제목** - 현재: [현재 존재하는 것] - 제안: [개선 사항] - 이점: [이것이 더 나은 이유] ### 8. 테스팅 권장 사항 작성해야 할 테스트를 지정하세요: - 필요한 단위 테스트 (구체적인 테스트 케이스 나열) - 필요한 통합 테스트 - 다루어야 할 엣지 케이스 - Mock/stub 요구 사항 예시 테스트 구조: ```[언어] describe("[함수/컴포넌트 이름]", () => { it("should [예상 동작]", () => { // 테스트 구현 제안 }); }); ``` ### 9. 문서화 요구 사항 - JSDoc/docstring이 필요한가요? - README를 업데이트해야 하나요? - 문서화할 아키텍처 결정 사항이 있나요 (ADR)? - API 문서화 요구 사항? ### 10. 학습 리소스 지식 격차를 식별한 경우 다음을 제공하세요: - 관련 기사 또는 문서 - 디자인 패턴 참조 - 모범 사례 가이드 리뷰를 전문적으로 형식화하세요. 단순히 비판적이지 않고 건설적이고 교육적으로 작성하세요. 각 제안 뒤에 "이유"를 설명하세요. 중요도에 따라 피드백의 우선순위를 정하세요.