📦 SKILL🛠 Dev🔧 Bash, Read, Grep
Review โค้ดก่อน Push (ผู้ช่วย Senior)
รีวิว diff ปัจจุบันแบบ senior dev หาบั๊ก, ช่องโหว่ security, edge case แล้วสรุปเป็นไทยจัดลำดับความรุนแรง — ใช้เมื่อจะ push, เปิด PR, หรือ vibe code เสร็จแล้วอยากให้ตรวจก่อนส่ง
--- name: code-reviewer-th description: รีวิว diff ปัจจุบันแบบ senior dev หาบั๊ก, ช่องโหว่ security, edge case แล้วสรุปเป็นไทยจัดลำดับความรุนแรง — ใช้เมื่อจะ push, เปิด PR, หรือ vibe code เสร็จแล้วอยากให้ตรวจก่อนส่ง allowed-tools: Bash, Read, Grep --- # Review โค้ดก่อน Push (ผู้ช่วย Senior) ## Workflow 1. หาขอบเขตที่จะรีวิว: `git diff main...HEAD` (ถ้าอยู่บน branch) หรือ `git diff --staged` / `git diff` — ถามผู้ใช้ถ้าไม่ชัดว่าจะรีวิวช่วงไหน 2. อ่านไฟล์เต็มของส่วนที่แก้ (ไม่ใช่แค่ diff) เพื่อเข้าใจ context รอบๆ — diff อย่างเดียวทำให้พลาด edge case 3. ตรวจตาม checklist ด้านล่างทีละหมวด 4. จัดผลเป็น 3 ระดับ: 🔴 Blocker (ห้าม merge) · 🟡 Should-fix (ควรแก้) · 🟢 Nitpick (เก็บได้) 5. แต่ละข้อระบุ `ไฟล์:บรรทัด` + อธิบายปัญหา + เสนอวิธีแก้พร้อม snippet สั้นๆ 6. ปิดท้ายด้วยคำตัดสิน: ✅ พร้อม push / ⚠️ แก้ก่อน — และนับจำนวนแต่ละระดับ …
ใช้ได้กับ
- vibe code มาทั้งวันอยากให้ตรวจก่อน push ขึ้น main
- จะเปิด PR แต่กลัวมีบั๊กหรือ console.log หลุดไป
- ไม่มี senior ในทีมช่วยดู อยากได้สายตาที่สอง
- เขียน API endpoint ใหม่ อยากเช็ก security/validation