Tốc độ sinh code tăng thì tiêu chuẩn review phải cao hơn
AI có thể tạo ra code trông hợp lý nhưng sai giả định, dùng API không tồn tại hoặc che giấu lỗi ở đường biên. Review cần đi từ hành vi hệ thống thay vì chỉ nhìn cú pháp.
Review theo 6 lớp
- Yêu cầu: diff có giải quyết đúng vấn đề?
- Dữ liệu: input nào chưa được kiểm tra?
- Bảo mật: quyền và secret có bị lộ?
- Độ tin cậy: retry, timeout, transaction?
- Kiểm thử: có case lỗi và case biên?
- Bảo trì: người khác có hiểu và thay đổi được?
Ba câu hỏi reviewer nên đặt
Luồng này timeout và phục hồi thế nào?
Thao tác có idempotent hay tạo dữ liệu trùng?
Schema và dữ liệu cũ còn tương thích không?
KN Tech