strategic-code-reviewer

DRY, KISS, SRP 원칙에 대해서 코드 리뷰를 수행합니다. 코드 품질 검토, 리팩터링 기회 식별, 중복 코드 감지, 복잡한 로직 단순화에 특화되어 있습니다.

Stars 163

Forks 31

Install this agent skill to your Project

npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/development/strategic-code-reviewer-devstefancho-claude-code-book-sam

SKILL.md

전략적 코드 리뷰

세 가지 검토 원칙

DRY (Don't Repeat Yourself) - 중복 제거

목표: 유지보수 비용 감소

동일한 코드가 여러 곳에 반복됨
심각도: 3회 이상 반복 = 높음, 2회 = 중간
개선 방향: 함수 추출 → Hook/모듈화 → 라이브러리화

KISS (Keep It Simple, Stupid) - 단순화

목표: 인지 부하 감소 및 버그 방지

깊은 중첩(3단계+), 복잡한 조건식, 매개변수 과다(4개+)
심각도: 순환 복잡도 > 10 = 높음, 7-10 = 중간
개선 방향: 조기 반환(guard clause) → 룩업 테이블 → 함수 분리

SRP (Single Responsibility Principle) - 단일 책임 원칙

목표: 변경 영향 최소화 및 유지보수성 향상

하나의 함수/클래스가 여러 책임 수행 (데이터 처리 + UI 업데이트 + 로깅 등)
심각도: 3개 이상 책임 = 높음, 2개 = 중간, 사소한 개선 = 낮음
개선 방향: 책임 분리 → 함수/클래스 분할 → 모듈화

🔍 이슈 분류

심각도	설명
🔴 높음	즉시 수정 필요 (유지보수성 또는 신뢰성 영향)
🟡 중간	조만간 개선 필요 (인지 부하 증가)
🟢 낮음	선택적 개선 (스타일 및 명확성)

Examples

examples/ 폴더에 각 원칙별 Good/Bad 패턴 예시가 있다.

examples/dry.md: DRY 원칙 위반/준수 예시
examples/kiss.md: KISS 원칙 위반/준수 예시
examples/srp.md: SRP 원칙 위반/준수 예시

리뷰 중 특정 원칙 위반을 발견했을 때, 해당 예시 파일을 Read 도구로 참조하여 구체적인 개선 방향을 제시한다.