Review một diff tốt cần nhìn gì?
Một diff không chỉ cần “đúng syntax”. Hãy chia review thành behavior, error handling, performance, security, test coverage và khả năng rollback. Claude Code có thể tạo checklist theo từng nhóm, sau đó trích dẫn vị trí cụ thể thay vì đưa nhận xét chung chung.
- Behavior có đúng acceptance criteria?
- Input lỗi có được xử lý?
- Có thay đổi quyền truy cập không?
- Test có bao phủ edge case?
- Migration có rollback được không?
Prompt review có thể tái sử dụng
Bạn có thể yêu cầu: “Review diff hiện tại như một senior engineer. Chỉ báo cáo finding có bằng chứng, ghi severity, file/dòng, tác động và cách viết test để xác minh. Nếu không có vấn đề, nói rõ phần nào đã kiểm tra”. Prompt này giúp giảm nhận xét mơ hồ và thuận tiện đưa vào checklist PR.
- Review theo diff, không quét mù toàn repo
- Yêu cầu severity và bằng chứng
- Yêu cầu test xác minh
- Tách finding khỏi suggestion
Reviewer con người phải xác nhận gì?
Reviewer cần đối chiếu yêu cầu thật, hiểu trade-off, kiểm tra dữ liệu production và cân nhắc tác động vận hành. AI không biết đầy đủ mục tiêu kinh doanh hoặc lịch sử incident nếu bạn không cung cấp context.
Câu hỏi thường gặp
Claude Code review có thay thế reviewer không?
Không. Claude Code có thể review vòng một, còn người chịu trách nhiệm phải xác nhận yêu cầu, rủi ro và quyết định merge.
Nên đưa toàn bộ repository vào review không?
Thường nên bắt đầu từ diff và các file liên quan để kết quả tập trung, sau đó mở rộng khi cần.
Review AI nên trả về format nào?
Nên có severity, vị trí file/dòng, tác động, bằng chứng và đề xuất test hoặc cách sửa.
