# 변경 품질 리뷰 (ponytail-review)

> 변경을 정확성, 안전성, 부하, 테스트, 속도, 간결성 관점으로 리뷰하며 diff뿐 아니라 연결된 코드도 읽습니다.

- 카테고리: [코드 리뷰](https://www.myskills.tools/ko/categories/code-review/)
- 출처: 커뮤니티 · DietrichGebert
- 저장소: https://github.com/DietrichGebert/ponytail/tree/main/skills/ponytail-review (commit 2a1fe84)
- SKILL.md: https://github.com/DietrichGebert/ponytail/blob/main/skills/ponytail-review/SKILL.md
- 설치 명령: `npx skills add DietrichGebert/ponytail --skill ponytail-review`
- 라이선스: MIT
- 호환 도구: Claude Code, Codex, Cursor
- 권한: read-repo ✓, write-files ✗, shell ✓, network ✗
- 상태: 스캔 통과
- 규칙 스캔으로 모든 위험을 찾을 수는 없습니다. 설치 전에 이 스킬의 SKILL.md를 읽고 권한을 확인하세요.

## 핵심 규칙

모든 지적에는 어떤 입력이나 상황이 어떤 잘못된 결과로 이어지는지 구체적 사례가 필요하며, 사례가 없으면 지적하지 않습니다.

## 실행 단계

1. **먼저 변경 이해** — diff와 함께 호출자, 호출 대상 함수, 테스트, README를 읽고 데이터 흐름을 추적하며 저장소에서 예상 부하를 파악합니다.
2. **항목별 점검** — 버그, 보안·데이터 손실 위험, 규모 문제, 누락된 테스트, 속도, 줄일 수 있는 코드를 차례로 확인합니다.
3. **보고 전 검증** — 해당 줄을 다시 읽어 지적이 맞는지 확인하고 최소한의 수정을 제안합니다. 코드 삭제를 우선하며 취향이나 막연한 우려는 쓰지 않습니다.
4. **쉬운 말로 보고** — 변경이 하는 일을 먼저 설명하고, 필수·권장·선택 그룹으로 번호를 매겨 지적을 나열한 뒤 판정을 제시합니다.

## 권한 설명 (SKILL.md 기준)

| 기능 | 필요 여부 | 설명 |
|---|---|---|
| 파일 쓰기 | 필요 없음 | 문제만 나열하고 코드는 전혀 수정하지 않습니다. |
| 명령 실행 | 일부 필요 | diff 확인과 grep으로 호출자를 찾는 등 읽기 전용 명령으로 한정됩니다. |
| 네트워크 송신 | 필요 없음 | 절차에 네트워크 접근 단계가 없습니다. |

## SKILL.md 프런트매터(YAML)

```yaml
---
name: ponytail-review
description: >
  Quality review of a change: is the logic right, is it safe, does it hold
  under real load, is risky code tested, is it fast enough, and is every line
  needed. Reads the connected code, not only the diff. Each finding is
  explained in plain English. Use for "review this", "code review", "review
  the last commit", "review my PR", "is this over-engineered", /ponytail-review.
---
```

홈: https://www.myskills.tools/ko/ · URL: https://www.myskills.tools/ko/skills/ponytail-review/