viet-pr-review
Review để chặn bug và nợ kỹ thuật. Không review để khoe style guide.
Khi nào dùng
- Diff hoặc mô tả PR được dán vào chat
- Cần comment tiếng Việt để gửi team
Không dùng
- Viết lại cả PR vì “em thấy chưa clean”
- Đánh giá người thay vì code
Bước
- Đọc mục đích PR trước.
- Rà logic, bảo mật, data loss, performance hiển nhiên, test thiếu, rồi mới style.
- Mỗi comment — vị trí + vấn đề + vì sao + hướng sửa.
- Chia Must change / Nice to have / Hỏi để hiểu.
- Kết 5–8 dòng — merge được không.
Template comment
[Must] OrderService#checkout — trừ tồn kho sau khi tạo order. Nếu retry, trừ 2 lần. Sửa — cùng transaction + idempotency-key.
Checklist
- Có nhận xét runtime, không chỉ naming
- Không đòi refactor ngoài scope trừ khi chặn ship
- Test thiếu kèm scenario