review

# 코드 리뷰 실행 **즉시 실행하세요. 설명하지 말고 바로 실행합니다.** ## 파이프라인 구조 ``` ┌──────────────┐ ┌──────────────┐ ┌───────────────┐ │ 리뷰 대상 │ → │ review-code │ → │ security-scan │ │ 파악 (Main) │ │ (opus) │ │ (sonnet) │ └──────────────┘ └──────────────┘ └───────────────┘ ``` --- ## 0단계: 정적 분석 (Python 파일 포함 시) **리뷰 대상에 `.py` 파일이 포함되어 있으면 ruff check를 먼저 실행합니다.** ```bash # 특정 파일 지정 시 ruff check [대상 파일 또는 디렉토리] # git diff 대상 시 (변경된 .py 파일 추출 후) git diff HEAD --name-only | grep '\.py$' | xargs ruff check 2>/dev/null ``` **결과 처리:** - ruff check 출력이 있으면: 리뷰 컨텍스트에 포함하여 review-code 에이전트에 전달 - ruff check 통과 시: "정적 분석: 통과" 메시지만 출력 - ruff가 설치되지 않은 경우: "스킵됨 (ruff 미설치)" 메시지 출력 **형식:** ``` ## 0단계: 정적 분석 (ruff) - 대상: [파일 목록 또는 "없음 (Python 파일 없음)"] - 결과: [통과 | N건 발견] - 발견된 이슈: [있을 때만 출력] ``` --- ## 0.5단계: 복잡도 flag — 하이브리드 (advisory 전용, Python 파일 포함 시) **결정론 flag → LLM 판단** 2단 하이브리드입니다. 복잡도 수치는 **리젝 사유가 아니라 리뷰 초점 신호**입니다. ```bash # mccabe 복잡도 flag (ruff 내장, 기본 임계 10) ruff check --select C901 [0단계와 동일한 대상 파일] ``` **결과 처리:** - flag된 함수가 있으면: 목록을 review-code 에이전트 컨텍스트에 다음 지시와 함께 전달 — > 아래 함수들은 복잡도 임계를 넘었다(결정론 flag). **줄수·복잡도 수치 자체를 리젝 > 사유로 삼지 말고**, 다음 관점으로만 분리 타당성을 판단하라: ① 모듈/함수 경계가 > 단일 책임인가 ② IN/OUT 계약(인자·반환·부수효과)이 명확히 분리 가능한가 > ③ 분리 시 계약이 단순해지는가(아니면 그대로가 응집적인가). 응집적이면 flag를 > 기각하고 그 근거를 명시하라. - flag 없으면: "복잡도: 통과" 한 줄만. - **절대 blocking 아님**: 이 단계는 어떤 경우에도 리뷰를 실패시키지 않는다 (전역 lint(E,F)·stop-validator와 무관 — advisory 전용). - ruff 미설치: "스킵됨 (ruff 미설치)" — fail-open....