code-review

Category: Tools & Productivity | Uploader: KirSsuRyu | Downloads: 0 | Version: v1.0（Latest）

사용자가 코드 리뷰, diff 리뷰, "내 변경사항 검토해줘", "이 PR 봐줘", "이 코드 어때", PR 열기 전 점검 등을 요청할 때 사용하세요. 정확성/설계/보안/테스트/스타일을 다루는 구조화된 언어 무관 리뷰를 수행합니다.

Changelog: Source: GitHub https://github.com/KirSsuRyu/venom

Directory Structure

Current level: tree/main/.claude/skills/code-review/

📄 SKILL.md 2.6 KB

SKILL.md

---
name: code-review
description: 사용자가 코드 리뷰, diff 리뷰, "내 변경사항 검토해줘", "이 PR 봐줘", "이 코드 어때", PR 열기 전 점검 등을 요청할 때 사용하세요. 정확성/설계/보안/테스트/스타일을 다루는 구조화된 언어 무관 리뷰를 수행합니다.
---

# 코드 리뷰 스킬

구조화된 리뷰를 수행한다. "괜찮아 보입니다"로 끝내지 않는다. 구체적으로,
줄 번호를 인용하고, 중요도순으로 분류한다.

## 입력
- diff(`git diff`, `git diff main...HEAD`), 파일, PR, 또는 디렉토리.
- 명확하지 않으면 어떤 범위를 볼지 묻는다.

## 절차

1. **변경 프레이밍.** diff를 처음부터 끝까지 한 번 읽는다. *무엇을* *왜*
   바꾸는지 한 문장으로 요약한다. 못 한다면 변경이 이미 너무 산만한 것 —
   그렇게 말한다.

2. **정적 패스 (파일별).** 건드린 각 파일에 대해 다음을 점검한다:
   - 정확성: off-by-one, null/None 처리, 에러 경로, 동시성, 자원 누수(파일/소켓/락).
   - API 설계: 이름의 정확성, 파라미터 순서, 기본값, 호환성을 깨는 변경.
   - 보안: 인젝션, 역직렬화, 경로 트래버설, 비밀, 미검증 입력 — `.claude/rules/20-security.md` 참고.
   - 성능: 명백한 N+1, 우발적 2차 복잡도, 핫패스 할당.
   - 테스트 가능성: 새 코드가 테스트에서 도달 가능한가?

3. **테스트 패스.** 새 테스트가 있는가? 변경을 커버하는가? 기존 테스트는
   여전히 통과하는가? 아직 안 돌렸다면 지금 돌린다.

4. **스타일 패스.** 주변 코드와 `.claude/rules/10-coding-standards.md`와
   비교한다. 스타일 nit은 별도 섹션에 둔다 — 진짜 발견과 섞지 않는다.

5. **메모리 패스.** `.claude/memory/mistakes.md`와 `lessons.md`를 훑는다.
   diff가 알려진 실수를 반복한다면 **HIGH**로 표시하고 항목을 링크한다.

## 출력 형식

```
## 요약
<한 단락>

## 발견 사항

### 🔴 차단 (Blocking)
- [파일:줄] <문제> — <왜 중요한가> — <제안된 수정>

### 🟡 고치는 게 좋음
- ...

### 🟢 사소한 것 (nits)
- ...

## 테스트
- 실행: <명령>
- 결과: <성공/실패/미실행>
- 변경 커버리지: <좋음 / 부분 / 없음>

## 결론
<승인 / 변경 요청 / 논의 필요>
```

## 규칙
- 끝까지 읽지 않은 변경을 승인하지 않는다.
- 줄 번호를 지어내지 않는다. 인용했다면 읽은 것이다.
- 발견 사항이 0이라면 한 번 더 본다. 코드보다 리뷰가 더 자주 틀린다.

Comments 0

Latest | Hot

Please login before commenting.

Loading comments...

code-review

Directory Structure

SKILL.md

Report

Notice