requesting-code-review
AI 코드 리뷰 요청
또한 다음에서 사용할 수 있습니다: davila7,DMJGilbert,DYAI2025,Cygnusfear,Cycleaddict,CodingCossack,obra
개발자는 구현에 너무 가까이 있기 때문에 자신의 코드에서 문제를 놓치는 경우가 많습니다. 이 스킬은 AI 서브에이전트를 투입하여 병합 전 코드 변경사항을 요구사항에 따라 검토하고, 품질, 아키텍처, 테스트, 프로덕션 준비 상태를 확인합니다.
스킬 ZIP 다운로드
Claude에서 업로드
설정 → 기능 → 스킬 → 스킬 업로드로 이동
토글을 켜고 사용 시작
테스트해 보기
"requesting-code-review" 사용 중입니다. 인증 모듈 변경사항에 대한 코드 리뷰 요청
예상 결과:
### Strengths
- 인증 서비스 (auth.ts:1-50) 의 관심사 명확한 분리
- bcrypt 를 사용한 적절한 비밀번호 해싱 (hash.ts:15)
- 엣지 케이스에 대한 좋은 테스트 커버리지
### Issues
#### Important
1. **토큰 만료 확인 누락**
- File: auth.ts:45
- Issue: 토큰이 만료되지 않아 보안 위험 발생
- Fix: JWT 페이로드에 만료 타임스탬프 추가
#### Minor
1. **일관성 없는 오류 메시지**
- File: auth.ts:30-35
### Assessment
**Ready to merge: With fixes**
**Reasoning:** 핵심 인증 로직은 견고합니다. 프로덕션 사용 전 토큰 만료 관련 Important 이슈를 해결해야 합니다.
"requesting-code-review" 사용 중입니다. 데이터베이스 마이그레이션 변경사항 리뷰
예상 결과:
### Strengths
- 적절한 외래 키 제약조건 (migrations/002.sql:20)
- 좋은 롤백 전략
### Issues
#### Critical
1. **파괴적 작업 전 백업 없음**
- File: migrations/003.sql:15
- Issue: 백업 전략 없이 DROP TABLE 수행
- Fix: 데이터 마이그레이션 스크립트 또는 백업 단계 추가
### Assessment
**Ready to merge: With fixes**
**Reasoning:** 중요한 데이터 안전 이슈를 해결해야 합니다.
보안 감사
안전All 27 static findings are false positives. The skill uses legitimate git commands (git diff, git rev-parse) for code review - these are standard development operations. The detected 'weak cryptographic algorithm' patterns match the word 'security' in documentation headings, not actual crypto code. The 'system reconnaissance' matches are generic text. This is a safe code review workflow skill with no command injection risk - git commands use hardcoded arguments with template variable placeholders.
위험 요인
감지된 패턴
품질 점수
만들 수 있는 것
병합 전 코드 리뷰
기능 브랜치를 병합하기 전에 사용하여 일찍 이슈를 포착합니다. 서브에이전트는 모든 변경사항을 요구사항에 따라 검토하고 명확한 ready/not-ready 판정을 제공합니다.
서브에이전트 작업 검증
에이전트 주도 개발에서 각 서브에이전트가 작업을 완료한 후, 다음 작업으로 진행하기 전에 코드 리뷰어를 투입하여 작업을 검증합니다.
문제 해결이 막혔을 때
복잡한 버그나 기능 개발이 막혔을 때, 코드 리뷰를 요청하여 잠재적 이슈나 더 나은 접근 방식에 대한 AI 서브에이전트의 새로운 관점을 얻습니다.
이 프롬프트를 사용해 보세요
Request code review for the changes I just made. Use the requesting-code-review skill with: WHAT_WAS_IMPLEMENTED: [describe what you built], PLAN_OR_REQUIREMENTS: [link to plan or describe requirements], BASE_SHA: [starting commit], HEAD_SHA: [ending commit], DESCRIPTION: [brief summary]
Please review my latest feature implementation using the requesting-code-review skill. Compare against the requirements in docs/plans/[feature-name].md. Include architecture assessment and testing coverage evaluation.
Run the code-reviewer subagent to verify the task just completed. Check against the original task requirements and report any Critical or Important issues that need fixing before proceeding.
Request a code review of the current state of [component/module] before I refactor it. This will serve as a baseline to ensure the refactoring does not introduce regressions.
모범 사례
- 리뷰 요청 시 명확한 요구사항 또는 계획 참조 제공 - 서브에이전트가 이를 기준으로 구현을 비교합니다
- 에이전트 주도 개발에서 각 중요한 작업 후 리뷰를 수행하여 이슈가 누적되기 전에 포착
- Critical 및 Important 이슈는 진행 전 수정; Minor 이슈는 후속 작업에서 처리
피하기
- 변경사항이 간단해 보여서 리뷰를 건너뜀 - 작은 변경에도 미묘한 버그가 있을 수 있음
- Critical 이슈를 무시하거나 수정되지 않은 Important 이슈를 그대로 진행
- 서브에이전트가 비교할 명확한 요구사항 또는 계획 없이 리뷰 요청
자주 묻는 질문
이 스킬은 Claude 를 직접 사용하는 것과 어떻게 다릅니까?
어떤 프로그래밍 언어에도 사용할 수 있습니까?
공식 요구사항 문서가 없으면 어떻게 합니까?
이것이 인간 코드 리뷰를 대체합니까?
BASE_SHA 와 HEAD_SHA 는 어떻게 구합니까?
리뷰 기준을 커스터마이즈할 수 있습니까?
개발자 세부 정보
작성자
ZhanlinCui라이선스
MIT
리포지토리
https://github.com/ZhanlinCui/Ultimate-Agent-Skills-Collection/tree/main/requesting-code-review참조
main
파일 구조