검토
이 스킬을 사용하여 이미 존재하는 작업을 평가합니다. 검토는 절대 판단하는 변경사항을 작성하지 않습니다.
이것은 표준 검토 워크플로입니다. merge-readiness가 여기로 라우팅되며, ai-slop-cleaner는 그 안의 옵션 레인입니다.
목표
실제로 잘못된 것을 심각도 순으로 찾아내고, 조치할 수 있을 만큼 상세히 파악합니다.
워크플로
- 무엇이 변경되었고, 무엇을 의도했는지 파악합니다.
- 해당 의도에 맞춰 변경사항을 읽습니다.
- 먼저 정확성을 확인한 후, 위험, 단순화 순서로 확인합니다.
- 각 후보 발견 사항을 보고하기 전에 재현 가능한지 확인합니다.
- 가장 심각한 발견 사항부터 먼저 보고합니다.
확인할 사항
- 정확성 — 로직 결함, 엣지 케이스, 오류 경로, 동시성
- 위험 — 보안 경계, 파괴적 작업, 데이터 무결성
- 재사용 — 변경사항이 사용했어야 할 기존 유틸리티 또는 패턴
- 단순화 — 삭제하거나 병합할 수 있는 코드
- 커버리지 — 테스트되지 않은 상태로 출시되는 동작
규칙
- 별도의 레인: 검토자는 작성자와 동일한 활성 컨텍스트를 공유해서는 안 됩니다.
- 보고하기 전에 확인하세요. 그럴듯해 보이지만 재현되지 않는 발견 사항은 노이즈입니다.
- 심각도를 정직하게 표시하세요. 목록을 부풀리지 마세요.
- 작업이 문제가 없으면 "발견 사항 없음"도 유효한 결과입니다.
- 기본적으로 조언 역할: 검토는 정보를 제공할 뿐, 게이트 역할을 하지 않습니다. 하드 게이트(릴리스, 보안, 파괴적 작업)는 별도로 유지되며, 닫힌 상태에서 실패합니다.
출력
- 발견 사항, 가장 심각한 것부터 먼저, 각각 파일, 줄, 구체적인 실패 시나리오 포함
- 확인했지만 문제가 없었던 사항
- 평가할 수 없었던 사항