code-review

Systematic code review based on 8 categories

항목	내용
Invoke	/code-review
Aliases	/review, /pr:review
Category	kobic-development
Complexity	moderate
MCP Servers	serena, context7

/code-review — 코드 8개 항목 자동 점검#

한마디로#

작성한 코드를 PR(병합 요청)에 올리기 전, 숙련된 검토자가 8가지 기준으로 꼼꼼히 살펴보는 것과 같습니다. 출판 전 원고를 교정 보는 것처럼, 보안·성능·테스트 등 빠뜨리기 쉬운 부분을 자동으로 짚어줍니다.

누가·언제 쓰나요#

동료의 PR을 검토(리뷰)해야 할 때
코드 품질을 한번 점검하고 싶을 때
커밋(코드 저장)하기 전에 스스로 먼저 살펴보고 싶을 때

무엇을 해주나요#

코드를 8개 항목으로 나누어 점검하고, 별점 요약표 + 문제 목록을 만들어 줍니다.

8개 점검 항목: 아키텍처 / 상태 관리 / 보안 / 성능 / 테스트 / 가독성 / 다국어(i18n) / 접근성
발견된 문제는 심각도별로 분류 — 🔴 치명적 / 🟡 개선 / 🟢 제안
흔한 오탐(잘못된 지적)은 자동으로 걸러줌(억제 규칙). 걸러진 항목도 따로 보여줘서 빠짐없이 확인 가능
--gate-mode에서는 "통과/차단" 형태의 관문(Gate) 결과표로 정리 — PR 생성을 막을 치명적 문제와 PR 본문에 넣을 참고 사항을 분리해서 보여줌

어떻게 쓰나요#

# 특정 파일 검토
/code-review feature/auth/lib/src/presentation/bloc/login_bloc.dart

# 보안에 집중해서 검토
/code-review feature/auth/ --focus security

# 빠르게 훑어보기
/code-review . --quick

--focus : 한 항목에 집중 (예: security 보안, performance 성능)
--quick : 빠른 검토 모드 / --verbose : 더 자세한 검토 모드
--no-suppress : 오탐 걸러내기를 끄고 모든 지적을 다 보기
--gate-mode : 관문(Gate) 결과표 형태로 출력 (치명적 문제와 참고 사항 분리)

안에서 무슨 일이 벌어지나요#

오탐 걸러내기 준비 — 플러그인 기본 규칙과 프로젝트 자체 규칙(.code-review-suppress.md)을 불러와 잘못된 지적을 미리 거를 준비를 합니다.
8개 항목 점검 — 아키텍처부터 접근성까지 8개 기준으로 코드를 하나씩 살펴봅니다. (필요하면 Serena로 코드 구조를, Context7로 모범 사례를 함께 확인)
결과 정리 — 항목별 별점 요약표와 함께 문제를 🔴치명적·🟡개선·🟢제안으로 나눠 보여주고, 걸러낸 항목도 따로 표시합니다.

⚙️ 상세 옵션·실행 명세 (개발자 / AI 에이전트용)

Triggers#

When a PR review is requested
When code quality inspection is needed
For self-review before committing

Context Trigger Pattern#

/code-review {target} [--options]

Parameters#

Parameter	Required	Description	Example
`target`	✅	Review target	File path, PR link
`--focus`	❌	Focus category	`security`, `performance`
`--quick`	❌	Quick review mode
`--verbose`	❌	Detailed review mode
`--no-suppress`	❌	Disable suppression
`--gate-mode`	❌	Review Gate mode (separates Critical/Informational output)

Suppression List#

Automatically applies suppression rules during review to reduce false positives.

Loading Order#

Plugin default: cc-code-quality/rules/suppression.md
Project custom: project root .code-review-suppress.md (if exists)

Application Method#

Each review finding is compared against the suppression list
Matching items are excluded from review results
Can be disabled with --no-suppress option

Result Display#

## Suppressed (3 items excluded)
- [Readability] `*.g.dart` generated code pattern (suppression: code generation related)
- [Performance] const not applied (suppression: style/convention related)
- [Architecture] melos import path (suppression: project structure related)

Review Categories#

1. Architecture#

Clean Architecture layer separation followed
Domain → Data → Presentation dependency direction
Business logic accessed through UseCases
Repository interface separation
Feature module independence

2. State Management#

BLoC/Cubit pattern followed
Event → BLoC → State flow
Immutable state via sealed class extends Equatable + copyWith (no @freezed)
Proper error handling
Loading state management

3. Security#

No hardcoded API keys or secrets
No sensitive information in logs
User input sanitization
Authentication/authorization properly applied

4. Performance#

const widgets utilized
BlocBuilder buildWhen used
Image cacheWidth/cacheHeight applied
Resources released in dispose
Stream subscriptions cancelled

5. Testing#

UseCase unit tests
Repository tests (mocked)
BLoC tests
Edge cases covered
Arrange-Act-Assert pattern

6. Readability#

Clear and meaningful naming
Project convention followed
Appropriate file/class size
Single Responsibility Principle
Duplicate code removed

7. Internationalization (i18n)#

All UI text uses translation keys
context.t.* pattern used
Proper pluralization
Dynamic values parameterized

8. Accessibility#

Appropriate semantic labels
Minimum 48x48 touch target
WCAG color contrast criteria met

Review Output Template#

## Summary

**Overall**: ⭐⭐⭐⭐☆ (4/5)

| Category | Score | Key Issues |
|----------|-------|------------|
| Architecture | ✅ | - |
| State Management | ⚠️ | BlocBuilder optimization needed |
| Security | ✅ | - |
| Performance | ⚠️ | Image cache size not specified |
| Testing | ❌ | UseCase tests missing |
| Readability | ✅ | - |
| i18n | ✅ | - |
| Accessibility | ⚠️ | Some semantic labels missing |

### Critical Issues 🔴

1. **[Security]** API key hardcoded
   - File: `lib/core/config.dart:15`
   - Fix: Use environment variables via Envied

### Improvements 🟡

1. **[Performance]** Image cache size needs to be specified
   - File: `lib/presentation/widget/product_card.dart:42`
   - Recommended: Add `cacheWidth: 200`

### Suggestions 🟢

1. **[Readability]** Variable name improvement suggestion
   - `data` → `userProfile`

### Suppressed (N items excluded)
 >   Items excluded by suppression rules. Use `--no-suppress` to see all.

Gate Mode Output (--gate-mode)#

Structured output for workflow gate when running /code-review --gate-mode:

## Code Review Gate Result

**Status**: ✅ PASS / ❌ BLOCKED

### Critical Issues (Pass 1) 🔴
 >   Issues that block PR creation

| # | Category | Issue | File | Auto-fix |
|---|----------|-------|------|----------|
| 1 | Security | API key hardcoded | config.dart:15 | ✅ Possible |

### Informational (Pass 2) 📋
 >   Improvement suggestions to include in PR body

| # | Category | Issue | File |
|---|----------|-------|------|
| 1 | Performance | Image cache not specified | product_card.dart:42 |

### Summary
- Critical: 0 items → ✅ Gate passed
- Informational: 3 items → Included in PR body
- Suppressed: 2 items → Excluded

Automation Commands#

# Lint check
melos run analyze
melos run lint:check

# Test execution
melos run test
melos run test:with-html-coverage

# Formatting (kobic uses dcm format, not dart format)
melos run format

MCP Integration#

Step	MCP Server	Purpose
Code analysis	Serena	Symbol and reference search
Pattern verification	Context7	Best practices validation

Examples#

File review#

/code-review feature/auth/lib/src/presentation/bloc/login_bloc.dart

Security-focused review#

/code-review feature/auth/ --focus security

Quick review#

/code-review . --quick

References#

Detailed checklist: .claude/prompts/code-review.md
PR review guide: .claude/checklists/pr-review.md