Code Review — `/code-review`

Review code tự động với adversarial rigor. Mỗi review bao gồm 3 stages bắt buộc: spec compliance, code quality, và red-team analysis tìm security holes.

Cập nhật bộ claudekit mới: Thêm /review-pr <#> --fix --reply (tự động reply vào PR) và /design-review (visual QA + auto-fix). Xem cấu trúc skill mới để biết đầy đủ.

Cú pháp

/code-review                     # Review changes hiện tại (default)
/code-review #123                # Review 1 PR cụ thể
/code-review --pending           # Review staged + unstaged changes
/code-review codebase            # Full codebase scan
/review-pr #123 --fix --reply    # Review PR + auto-fix + reply comments
/design-review                   # Visual QA + design system check

Luồng hoạt động bên trong

Khi bạn gõ /code-review #123, đây là 3 stages xảy ra:

Stage 1: Spec Compliance Review

Kiểm tra code có khớp với requirements/plan không.

Code implement đúng những gì được yêu cầu?
Có thiếu requirement nào không?
Có code thừa không được yêu cầu?

PHẢI pass Stage 1 mới được chạy Stage 2. Nếu fail → fix → re-review Stage 1.

Stage 2: Code Quality Review

Spawn code-reviewer subagent kiểm tra:

Scout edge cases trước
Standards, security, performance
Naming, readability, maintainability

Chỉ chạy SAU khi Stage 1 pass.

Stage 3: Adversarial Review (Red Team)

Spawn adversarial reviewer CỐ TÌM CÁCH PHÁ code:

Security holes, false assumptions
Resource exhaustion, race conditions
Supply chain attacks, observability gaps

Mỗi finding được đánh giá: Accept (phải fix) / Reject (false positive) / Defer (tạo GitHub issue). Critical findings CHẶN merge.Skip nếu: <= 2 files, <= 30 lines, không có security files.

Input Modes

Auto-detect từ arguments. Nếu không rõ hoặc không có argument → hỏi user.

Input	Mode	Review gì
`#123` hoặc PR URL	PR	Full PR diff qua `gh pr diff`
`abc1234` (7+ hex chars)	Commit	Single commit diff qua `git show`
`--pending`	Pending	Staged + unstaged changes qua `git diff`
(không args)	Default	Recent changes trong context
`codebase`	Codebase	Full codebase scan
`codebase parallel`	Codebase+	Parallel multi-reviewer audit

Ví dụ thực tế

Input
Output

/code-review #45

CODE REVIEW REPORT — PR #45
═══════════════════════════

Stage 1: Spec Compliance ✅ PASS
├── 5/5 requirements implemented
└── 0 extra, 0 missing

Stage 2: Code Quality
├── Correctness:   8/10
├── Security:      9/10
├── Performance:   7/10
├── Readability:   8/10
└── Test Coverage: 6/10

Stage 3: Adversarial Review
├── [ACCEPT — MUST FIX] Missing auth check in DELETE /api/users/:id
│   → Bất kỳ authenticated user nào cũng xóa được user khác
├── [REJECT] N+1 query concern in getUserOrders()
│   → Đã có dataloader, false positive
└── [DEFER → GitHub Issue] No rate limiting on login endpoint

ISSUES FOUND:
1. [CRITICAL] Missing authorization guard on delete endpoint
2. [MEDIUM] Variable name 'x' not descriptive
3. [LOW] Console.log left in production code

Overall: 7.6/10 — BLOCKED (1 critical finding)

So sánh với chat trực tiếp

Tiêu chí	`/code-review`	Chat trực tiếp
Review protocol	3 stages bắt buộc, tuần tự	1 lượt review, có thể bỏ sót
Spec compliance	Kiểm tra code vs requirements trước	Không so sánh với spec
Adversarial	Red-team CỐ PHÁ code	Chỉ review “bình thường”
Input modes	PR, commit, pending, codebase	Chỉ review code được paste
Verification	Iron Law: phải verify evidence trước khi claim	Có thể claim “looks good”
Edge cases	Scout edge cases TRƯỚC khi review	Phụ thuộc kinh nghiệm reviewer
Multi-file	Task pipeline: scout → review → adversarial → fix → verify	Review từng file tách rời

Khi nào dùng / không dùng

Dùng	Không dùng
Trước khi tạo PR	Khi cần implement feature (dùng `/cook`)
Review PR của người khác	Khi cần fix bug (dùng `/fix`)
Sau refactor lớn	Khi chỉ muốn chạy tests (dùng `/test`)
Pre-landing check	Review non-code files (docs, config)
Security audit trước deploy	Khi codebase quá lớn (dùng `codebase parallel`)

Sub-commands

Sub-command	Mô tả	Khi nào dùng
`/code-review`	Review changes hiện tại	Sau khi code xong
`/code-review #123`	Review 1 PR cụ thể	Review PR từ teammate
`/code-review --pending`	Review staged + unstaged	Trước khi commit
`/code-review codebase`	Full codebase scan	Audit toàn bộ project
`/review-pr #123 --fix --reply`	Review + auto-fix + reply PR comments	Automated review + self-fixing
`/design-review`	Visual QA + design system compliance	UI component review
`/review`	Pre-landing PR review (GStack)	Trước khi merge

Review variants

`/review-pr #123 --fix --reply` — Auto-fix + reply PR

Review PR + auto-fix issues + post comments directly to PR:

/review-pr #45 --fix --reply

Output:

PR comments with findings
Auto-fixed commit (if --fix enabled)
Inline reply to each comment (if --reply enabled)

Khi dùng: Reviewer wants to fix issues AND post feedback in one go.

`/design-review` — Visual QA + design system check

Review UI components, design system compliance, visual consistency:

/design-review

Checks:

Design system tokens match (color, spacing, typography)
Component prop usage (Button, Card, Input variants)
Responsive design correctness
Accessibility (contrast, labels, ARIA)
Brand consistency

Output: Design findings + auto-fixes for style issues. Khi dùng: UI/frontend PRs, component library updates.

`/review` — GStack Pre-landing Review

Review diff trước khi merge, focus vào:

SQL safety (injection, raw queries)
LLM prompt changes (prompt injection risks)
Breaking API changes (endpoint, response shape)
Security vulnerabilities (OWASP Top 10)
Config safety (env vars, secrets)

/review

Verification Gates — Iron Law

KHÔNG BAO GIỜ CLAIM “done” hay “fixed” MÀ KHÔNG CÓ FRESH EVIDENCE. Quy trình verify:

IDENTIFY command cần chạy
RUN full command
READ output
VERIFY output confirms claim
THEN mới claim

Red flags (dấu hiệu CHƯA verify):

“should work”, “probably fine”, “seems to pass”
Hài lòng trước khi verify
Tin agent reports mà không double-check

Chi tiết lệnh Code Review (Accordion)

/review-pr #123 --fix --reply — Auto-fix + reply

Prefix: /review-pr <#> --fix --replyReview GitHub PR + auto-fix issues + post comments to PR:

/review-pr #45 --fix --reply

Flags:

--fix — Apply auto-fixes if possible
--reply — Post inline comments to PR
Both optional but recommended

Output:

PR comments with findings
Commit with auto-fixes (if --fix)
Reply to each review comment (if --reply)

Khi dùng: Full-service PR review + feedback in one command.

/design-review — Visual QA + design system

Prefix: /design-reviewReview UI/frontend code từ design system angle:

/design-review

Checks:

Design tokens (color, spacing, typography match system)
Component usage (Button, Card variants correct?)
Responsive design correctness
Accessibility (contrast, ARIA labels)
Brand consistency
Visual hierarchy

Output: Design findings + auto-fixes for style issues.Khi dùng: UI/frontend PRs, component library updates, design system compliance.

​Code Review — /code-review

​Cú pháp

​Luồng hoạt động bên trong

​Input Modes

​Ví dụ thực tế

​So sánh với chat trực tiếp

​Khi nào dùng / không dùng

​Sub-commands

​Review variants

​/review-pr #123 --fix --reply — Auto-fix + reply PR

​/design-review — Visual QA + design system check

​/review — GStack Pre-landing Review

​Verification Gates — Iron Law

​Chi tiết lệnh Code Review (Accordion)

Code Review — `/code-review`

Cú pháp

Luồng hoạt động bên trong

Input Modes

Ví dụ thực tế

So sánh với chat trực tiếp

Khi nào dùng / không dùng

Sub-commands

Review variants

`/review-pr #123 --fix --reply` — Auto-fix + reply PR

`/design-review` — Visual QA + design system check

`/review` — GStack Pre-landing Review

Verification Gates — Iron Law

Chi tiết lệnh Code Review (Accordion)