Viet Pr Review

Review pull request theo giọng team Việt Nam. Dùng khi user dán diff, link PR, hoặc hỏi code review trước khi merge.

gonegirl07 Updated

File contents

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

  1. Đọc mục đích PR trước.
  2. Rà logic, bảo mật, data loss, performance hiển nhiên, test thiếu, rồi mới style.
  3. Mỗi comment — vị trí + vấn đề + vì sao + hướng sửa.
  4. Chia Must change / Nice to have / Hỏi để hiểu.
  5. 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

gonegirl07/awesome-vn-agent-skills/tree/main/skills/viet-pr-review commit 93db0fef8b

Frequently asked questions

npx skillmds@latest add gonegirl07/viet-pr-review