Cổng rà soát (review gate): danh sách kiểm tra trước khi merge
Một cổng rà soát (review gate) là một danh sách ngắn những điều phải đúng trước khi một thay đổi được đưa vào hoạt động thật. Tự bạn chạy qua danh sách này, hoặc để AI xác nhận từng mục:
- Tôi đã đọc diff và hiểu những gì đã thay đổi cùng lý do vì sao
- Thay đổi được giới hạn đúng trong phạm vi tôi thực sự yêu cầu
- Các bài kiểm thử pass (và hành vi mới có một bài kiểm thử)
- Trình kiểm tra kiểu dữ liệu và linter đều sạch
- Không còn sót lại bí mật, log debug, hay mã bị comment ra
- Không có bước kiểm tra an toàn nào bị lặng lẽ gỡ bỏ
- Tôi đã tự mình thử thay đổi này, ít nhất là đường đi thuận
Nếu bạn không thể đánh dấu một ô, bạn chưa xong.
Danh sách kiểm tra này cố tình ngắn gọn vì một cổng mà bạn bỏ qua thì không phải là cổng gì cả. Sự cám dỗ, đặc biệt là khi bạn đang tiến nhanh và thay đổi "trông có vẻ ổn," là vẫy tay cho qua. Hãy cưỡng lại điều đó với bất cứ thứ gì hướng tới người dùng. Một mẹo hữu ích là để AI tự điền vào giúp bạn — dán danh sách và hỏi: "Hãy đi qua cổng rà soát này cho thay đổi bạn vừa thực hiện. Với mỗi mục, nói PASS hoặc FAIL kèm một dòng bằng chứng." Nó không thể trung thực viết "tests pass" mà không thực sự chạy chúng, và việc buộc nó phải trích dẫn bằng chứng cho từng dòng biến một câu "trông ổn" mơ hồ thành thứ bạn có thể kiểm toán được.