/rust-review

Rust 코드 리뷰

이 커맨드는 종합적인 Rust 전용 코드 리뷰를 위해 rust-reviewer 에이전트를 호출합니다.

이 커맨드가 하는 것

1. 자동 체크 검증: cargo check, cargo clippy -- -D warnings, cargo fmt --check, cargo test를 실행하며, 하나라도 실패하면 중단합니다.
2. Rust 변경 사항 식별: git diff HEAD~1(또는 PR의 경우 git diff main...HEAD)을 통해 수정된 .rs 파일을 찾습니다.
3. 보안 오디트 실행: 가능한 경우 cargo audit을 실행합니다.
4. 보안 스캔: unsafe 사용, 커맨드 인젝션, 하드코딩된 시크릿 등을 확인합니다.
5. 소유권 리뷰: 불필요한 clone, 수명(lifetime) 이슈, 빌림(borrowing) 패턴을 분석합니다.
6. 보고서 생성: 발견된 이슈를 심각도별로 분류합니다.

사용 시점

다음과 같은 경우에 /rust-review를 사용하세요:

• Rust 코드를 작성하거나 수정했을 때
• Rust 변경 사항을 커밋하기 전
• Rust 코드가 포함된 풀 리퀘스트를 리뷰할 때
• 새로운 Rust 코드베이스에 온보딩할 때
• 관용적인(idiomatic) Rust 패턴을 배울 때

리뷰 카테고리

치명적 (CRITICAL - 반드시 수정)

• 프로덕션 코드 경로에서 체크되지 않은 unwrap()/expect() 사용
• 불변성(invariants)을 문서화하는 // SAFETY: 주석 없는 unsafe 사용
• 쿼리 내 문자열 삽입을 통한 SQL 인젝션
• std::process::Command에서 검증되지 않은 입력을 통한 커맨드 인젝션
• 하드코딩된 자격 증명
• 원시 포인터(raw pointers)를 통한 Use-after-free

높음 (HIGH - 수정 권장)

• borrow checker를 만족시키기 위한 불필요한 .clone()
• &str 또는 impl AsRef<str>로 충분한 곳에 String 파라미터 사용
• 비동기 컨텍스트에서의 블로킹 작업 (std::thread::sleep, std::fs 등)
• 공유 타입에 대한 Send/Sync 바운드 누락
• 비즈니스 크리티컬한 enum에 대한 와일드카드 _ => 매치
• 너무 긴 함수 (50라인 초과)

중간 (MEDIUM - 검토 필요)

• 빈번하게 호출되는 경로(hot path)에서의 불필요한 할당
• 크기를 알 수 있는 경우의 with_capacity 누락
• 정당한 사유 없이 억제된 clippy 경고
• /// 문서화가 없는 public API
• 값을 무시하는 것이 버그일 가능성이 높은 비-must_use 반환 타입에 #[must_use] 고려

실행되는 자동 체크

# 빌드 게이트 (리뷰 전 반드시 통과해야 함)
cargo check

# 린트 및 제안
cargo clippy -- -D warnings

# 포맷팅
cargo fmt --check

# 테스트
cargo test

# 보안 오디트 (가능한 경우)
if command -v cargo-audit >/dev/null; then cargo audit; else echo "cargo-audit이 설치되지 않음"; fi

사용 예시

User: /rust-review

Agent:
# Rust 코드 리뷰 보고서

## 리뷰된 파일
- src/service/user.rs (수정됨)
- src/handler/api.rs (수정됨)

## 정적 분석 결과
- 빌드: 성공
- Clippy: 경고 없음
- 포맷팅: 통과
- 테스트: 모두 통과

## 발견된 이슈

[치명적] 프로덕션 경로에서 체크되지 않은 unwrap 발견
파일: src/service/user.rs:28
이슈: 데이터베이스 쿼리 결과에 `.unwrap()` 사용
```rust
let user = db.find_by_id(id).unwrap();  // 사용자가 없을 경우 패닉 발생
```
수정: 컨텍스트와 함께 에러 전파
```rust
let user = db.find_by_id(id)
    .context("사용자 정보를 가져오지 못했습니다")?;
```

[높음] 불필요한 Clone
파일: src/handler/api.rs:45
이슈: borrow checker를 만족시키기 위해 String을 복제함
```rust
let name = user.name.clone();
process(&user, &name);
```
수정: 복제를 피하도록 구조 재조정
```rust
let result = process_name(&user.name);
use_user(&user, result);
```

## 요약
- 치명적: 1
- 높음: 1
- 중간: 0

권고: 치명적 이슈가 수정될 때까지 머지를 차단하세요.

승인 기준

상태	조건
승인 (Approve)	치명적 또는 높음 이슈 없음
주의 (Warning)	중간 이슈만 있음 (주의해서 머지)
차단 (Block)	치명적 또는 높음 이슈 발견됨

다른 커맨드와의 연동

• 테스트 통과 확인을 위해 /rust-test를 먼저 사용
• 빌드 에러 발생 시 /rust-build 사용
• 커밋 전에는 반드시 /rust-review 사용
• Rust 전용이 아닌 일반적인 사항은 /code-review 사용

관련 정보

• 에이전트: agents/rust-reviewer.md
• 스킬: skills/rust-patterns/, skills/rust-testing/