← ClaudeAtlas

code-reviewerlisted

Pre-landing PR 리뷰 정책 레이어. 일반 버그/품질 리뷰는 CLI 네이티브 엔진(Claude /code-review, Codex /review)에 위임하고, 네이티브가 못 하는 것 — Scope Drift 감지, 도메인 체크리스트(LLM 신뢰 경계, Enum 완전성), Fix-First 분류, Suppressions, 통합 보고서 — 를 더합니다. 네이티브 엔진이 없는 환경(Gemini 등)은 풀 경로(2-Pass + Specialist 병렬 dispatch)로 폴백. "코드 리뷰 해줘", "review", "리뷰", "PR 체크" 요청에 실행. 코드 작성 완료 시 자동 제안.
Dannykkh/skill-olympus · ★ 5 · Code & Development · score 80
Install: claude install-skill Dannykkh/skill-olympus
# Code Reviewer v4 — Policy Layer + Native Engine PR 단위 코드 리�� 오케스트레이터. v4부터 **리뷰 엔진과 정책 레이어를 분리**합니다. ``` 리뷰 요청 │ ├─ Step 0~1: 베이스 브랜치 + 리뷰 대상 확인 ├─ Step 2: 엔진 선택 (CLI 감지) │ ├─ 경로 A: Claude → 네이티브 code-review 스킬 │ ├─ 경로 B: Codex → 네이티브 codex review │ └─ 경로 C: 풀 경로 (네이티브 없음 — Gemini 등) └─ Step 3: 정책 레이어 P1~P5 (공통) P1 Scope Drift → P2 도메인 보강 패스 → P3 Suppressions → P4 Fix-First → P5 통합 보고서 ``` **설계 원칙**: 네이티브 엔진이 잘하는 일반 리뷰(버그, 보안 기본기, 성능)는 중복 구현하지 않는다. 이 스킬은 네이티브가 안 하는 것만 담당한다. ## 적용 시점 - 명시적 리뷰 요청 시 ("코드 리뷰 해줘", "review") - PR 생성 전 (`/ship` 전) - 코드 작성 완료 시 자동 제안 --- ## Step 0: 베이스 브랜치 감지 ```bash # 플랫폼 감지 _REMOTE_URL=$(git remote get-url origin 2>/dev/null || echo "unknown") # 베이스 브랜치 결정 _BASE=$(git symbolic-ref refs/remotes/origin/HEAD 2>/dev/null | sed 's|refs/remotes/origin/||') [ -z "$_BASE" ] && git rev-parse --verify origin/main &>/dev/null && _BASE="main" [ -z "$_BASE" ] && git rev-parse --verify origin/master &>/dev/null && _BASE="master" [ -z "$_BASE" ] && _BASE="main" echo "BASE: $_BASE" echo "BRANCH: $(git branch --show-current 2>/dev/null)" ``` --- ## Step 1: 리뷰 대상 확인 1. `git branch --show-current`로 현재 브랜치 확인. 2. 베이스 브랜치 위에 있으면: **"베이스 브랜치에서는 리뷰할 대상이 없습니다."** → 중단. 3. `git fetch origin $_BASE --quiet && git diff origin/$_BASE --stat` 실행. 4. diff가 없으면 동일 메시지 → 중단. --- ## Step 2: 엔진 선택 | 환경 | 감지 방법 | 경로 | |------|----------|------| | Claude Code | Skill 도구 목록에 `code-review` 존재 | **A — 네이티브 위임** | | Codex CLI | Code