Addy Osmani의 agent-skills 저장소에 있는 코드 리뷰 스킬: 병합 전(또는 다른 에이전트나 타인이 작성한 코드를 리뷰할 때) 에이전트가 정확성, 가독성, 아키텍처, 보안, 성능의 5가지 차원에서 변경 사항을 검토하도록 합니다. 테스트를 먼저 확인한 후 구현을 검토하며, 각 의견에 심각도 등급(Critical / 필수 수정 / Nit / 선택 사항 / 참고)을 표시하고, 최종적으로 승인 또는 수정 요청 결론을 제시합니다.
skills.sh 설치 횟수 79294(+975 / 24h)
보안 감사: 감사되지 않음
- 아직 스캔되지 않음
이유
- 프로세스는 5단계로 구성됩니다. 먼저 변경 의도를 파악한 후, 테스트를 먼저 검토하고 파일별로 5가지 차원에서 구현을 검토하며, 문제를 등급별로 나열하고, 작성자의 검증 설명을 확인합니다. 보고서를 작성하기 전에 호출자, 가드 및 테스트를 참조하여 오탐지를 배제해야 하며, 문제가 전혀 없는 것도 유효한 결론으로 인정되므로 억지로 문제를 만들어내서는 안 됩니다.
- 테스트의 유효성 판단은 실험을 통해 이루어집니다. 변경 사항 중 하나의 조건을 반전시켜 테스트를 실행한 후, 사본에서 파일을 복원합니다. 테스트가 여전히 통과한다면 테스트 케이스가 누락된 것이므로, 어떤 케이스가 누락되었는지 지적해야 합니다.
- 변경 규모에 대한 참고 기준(약 100줄은 한 번에 검토하기에 적합하고, 약 1000줄은 분할해야 함)과 4가지 분할 방법을 제시합니다. 의존성 업그레이드의 경우 변경 로그를 읽고, 한 번에 하나의 패키지만 업그레이드하며, 잠금 파일(lock file)의 차이를 검토해야 합니다.
- 불필요한 코드를 발견하면 먼저 목록을 작성한 후 삭제 여부를 문의하며, 임의로 삭제하지 않습니다. 구조적 문제를 제기할 때는 구체적인 리팩토링 방법을 함께 제시해야 합니다.
- 저장소는 MIT 라이선스를 따르며, 동일 저장소에는 24개의 다른 스킬이 있습니다. skills.sh에서 24시간 동안 975회 설치되었으며, 누적 설치 횟수는 79294회입니다. 최근 8주 동안 주간 설치 횟수가 3503회에서 5359회로 증가했습니다(이는 관심도를 나타냅니다).
사용법
- 설치 명령어는 감사 통과 후 표시됩니다. “이 PR / diff를 리뷰해줘”, 기능 개발 완료 후 병합 준비 중이거나 다른 에이전트가 작성한 코드를 평가해야 할 때 트리거됩니다.
- 계정이나 외부 서비스가 필요하지 않습니다. “조건을 반전시켜 테스트 실행” 단계에서는 로컬에서 프로젝트 테스트를 실행할 수 있어야 합니다.
리스크
- 이번 호에서는 감사되지 않았으므로 고정 C등급으로 분류되며 설치 명령어가 표시되지 않습니다. SKILL.md는 주로 리뷰 체크리스트로 구성되어 있으며, 파일을 수정하는 유일한 부분은 “임시로 하나의 조건을 반전시켜 테스트를 실행한 후 사본에서 복원”하는 것입니다. 이는 용도와 일치하지만, 파일이 복원되었는지 확인해야 합니다. skills.sh에서 Gen Agent Trust Hub 및 Socket 감사는 Pass로 평가되었으나, Snyk은 Warn으로 평가되었습니다.
- 보안 및 성능 부분은 동일 저장소의 security-and-hardening, performance-optimization 스킬과 저장소 루트 디렉터리의 체크리스트 파일을 참조합니다. 이 스킬만 설치할 경우 이러한 링크를 읽을 수 없을 수 있습니다.
- 승인 기준이 다소 느슨합니다. 전체적으로 코드 건강성을 개선하면 승인됩니다. “영업일 1일 내 회신”과 같은 팀 규정은 1인 프로젝트에는 큰 의미가 없습니다.