AI in the PR Review Loop (Stage 5: Deploy)

1. Những thay đổi: AI trong PR Review Loop

Claude đảm nhận cả hai vai trò: vừa đưa ra nhận xét review (gives reviews) vừa tiếp nhận review (receives reviews). Nó rà soát các PR gửi đến dựa trên chính sách của tổ chức và xử lý các nhận xét review trên chính các PR của nó. Điều này cho phép các kỹ sư tập trung vào hành vi trong lượt review PR, mà bản chất cốt lõi là đánh giá chủ đích (intent) và mức độ rủi ro (risk).

2. Sơ đồ PR Review Gate

Kiến trúc PR Review Loop nhiều tầng kiểm soát chất lượng mã nguồn trước khi merge:

• Tầng 1 (L1 - Deterministic CI): Chạy unit tests, linter, phân tích tĩnh và các quy tắc kiểm tra tất định trong pipeline CI.
• Tầng 2 (L2 - Nhiều tầng review bằng Agent / Layers of Agentic Review): Chạy các review passes độc lập theo chính sách REVIEW.md (Bugs, Security, Compliance đối chiếu spec.md và plan.md), xuất bảng kiểm đếm máy đọc được (machine-readable tally) theo mức độ nghiêm trọng.
• Tầng 3 (L3 - Human Code Owner Approval): Con người rà soát các phần regulated/critical (mã nguồn trọng yếu và tuân thủ quy chuẩn). Chính sách branch protection bắt buộc sự phê duyệt từ code owner dựa trên bảng kiểm đếm và đánh giá chủ đích.

3. Bắt đầu: Cấu hình REVIEW.md & Hạ tầng

Tech lead viết chính sách review dưới dạng tệp REVIEW.md tại thư mục gốc repository, phân chia thành các lượt rà soát (passes) mà tổ chức quan tâm:

• Bugs: Lỗi logic, trường hợp biên bị hỏng (broken edge cases), suy thoái ngầm (subtle regressions).
• Security: Nguy cơ injection, lỗ hổng xác thực (authentication gaps), dữ liệu định danh cá nhân (PII) trong log.
• Compliance: Thay đổi khớp với spec.md (từ bài học requirements), plan.md (từ bài học plan mode) và các nguyên tắc thiết kế của tổ chức.

Cấu hình chuẩn REVIEW.md tại repo root:

# Review instructions
## Passes
Run three passes and tag each finding with its pass:
- Bugs: logic errors, broken edge cases, subtle regressions
- Security: injection risks, authentication gaps, PII in logs
- Compliance: the change matches spec.md, plan.md and our design principles
## What Important means here
Reserve Important for findings that would break behavior, leak data
or breach a policy. Style and naming are nits.
## Cap the nits
Report at most five nits per review; summarize the rest as a count.
## Do not report
Generated files under src/gen/ and anything CI already enforces.

4. Các bước thực thi: 6 bước review

5. Feedback Loop vào CLAUDE.md (Rule of Two)

Các phát hiện trong PR review không dừng lại ở việc sửa lỗi cục bộ mà cung cấp cơ chế phản hồi ngược (feedback loop) trực tiếp vào CLAUDE.md:

Lượt review tuân thủ (compliance pass) cũng tự động gắn cờ cảnh báo khi thay đổi mã nguồn làm các chỉ dẫn trong CLAUDE.md trở nên lỗi thời, đảm bảo tài liệu sống luôn đồng bộ với thực tế codebase.

6. Kiểm soát & Đo lường

Nguyên tắc phân tách trách nhiệm (Separation of duties) được bảo toàn tuyệt đối: agent viết code không có cách nào tự phê duyệt (approve) mã của chính mình. Chính sách review trong REVIEW.md được áp dụng đồng nhất cho mọi PR. Mọi phát hiện (findings), bản sửa lỗi (fixes), đánh giá xếp hạng (ratings) và phê duyệt (approvals) đều được ghi nhận đầy đủ trong lịch sử PR, biến bản thân PR thành hồ sơ kiểm toán (audit record). Sự phê duyệt cuối cùng đến từ con người thông qua chính sách bảo vệ nhánh (branch protection), được cung cấp đầy đủ thông tin từ các phát hiện review — trong đó con người rà soát các phần regulated/critical (mã nguồn được kiểm soát và trọng yếu).