← ClaudeAtlas

code-reviewlisted

(agent-internal — code-reviewer 에이전트 전용, 사용자 요청으로 직접 호출 금지) code-reviewer 에이전트가 호출하는 코드리뷰 실행 엔진. mode(general|diff-branch)와 scope를 입력받아 코드 수집 → 자동화 선행검증 → C.L.E.A.R./Layered/Checklist/심각도 분류 리뷰 → 포맷된 리포트 생성까지 수행한다.
ridsync/devoks-team-harness · ★ 1 · Code & Development · score 61
Install: claude install-skill ridsync/devoks-team-harness
# code-review — 리뷰 실행 엔진 **호출 주체:** `code-reviewer` 에이전트 (subagent 안에서 실행) **메인 루프 직접 호출 금지** — 커맨드에서 직접 사용하지 않는다. --- ## 입력 파라미터 | 파라미터 | 타입 | 설명 | |---------|------|------| | `mode` | `general` \| `diff-branch` | 리뷰 방식 | | `scope` | 파일/폴더 경로 (`general`) 또는 base 브랜치 이름 (`diff-branch`) | 커맨드에서 이미 확정된 리뷰 대상 | --- ## Steps ### 1. 코드 수집 **mode=general** - 지정된 scope(파일/폴더 경로)의 파일을 읽는다. - 동작 이해에 필요한 **직접 의존 코드만** 최소 범위로 추가 확인한다. **mode=diff-branch** - 현재 브랜치와 base를 확인하고 변경 파일·diff를 수집한다: ```bash git branch --show-current git log --oneline <base>..HEAD git diff <base>...HEAD --name-only git diff <base>...HEAD ``` - diff를 기준으로 **실제 변경된 부분만** 집중 검토한다. - diff 파일이 5개 이상이거나 변경 줄이 300줄 초과 시 레이어(구조 → 로직 → 보안) 또는 파일 단위로 분할 순차 검토한다. - 본 리뷰 전 아래를 확인한다. - [ ] 커밋 메시지·PR/이슈로 원본 요구사항 또는 변경 목적을 확인했다. - [ ] 변경된 파일 목록과 의존 관계를 파악했다. - [ ] 이 변경이 시스템(아키텍처·기존 모듈)에서 차지하는 위치를 이해했다. - [ ] 외부 유입 코드(서드파티·외부 기여)가 포함된 경우, 격리 검토 대상으로 표시했다. ### 2. 자동화 도구 선행 검증 수동 리뷰 전에 최소 품질 게이트를 먼저 확인한다. - **Lint/Format**: 프로젝트의 lint/format 도구 결과를 먼저 확인한다. **lint 오류가 있는 상태로 리뷰 진행 금지.** - **타입 검사**: 프로젝트의 타입 검사 도구가 있다면 컴파일 오류 없음을 확인한다. - 자동화 도구 통과 여부를 리포트 `📋 리뷰 대상` 헤더에 명시한다. ### 3. 리뷰 가이드 적용 `.claude/refs/code-review.md`의 C.L.E.A.R., Layered Examination, Component Type Checklists, Severity Classification(§8)을 기준으로 리뷰를 수행한다. **필수 체크리스트 (5개 항목 — 리뷰 중 하나도 빠뜨리지 않는다):** - [ ] 잠재적 버그는 없는가? - [ ] 예외 처리가 빠진 곳은 없는가? - [ ] 성능상 문제는 없는가? - [ ] 더 읽기 좋은 코드가 있는가? - [ ] 보안상 위험은 없는가? 각 항목에서 발견된 이슈는 아래 "주요