ด่านรีวิว: เช็คลิสต์ก่อนการ merge
ด่านรีวิว (review gate) คือรายการสั้น ๆ ของสิ่งที่ต้องเป็นจริงก่อนที่การเปลี่ยนแปลงจะถูกนำไปใช้จริง ให้คุณทำเองหรือให้ AI ยืนยันแต่ละข้อ:
- ฉันอ่าน diff แล้วและเข้าใจว่าอะไรเปลี่ยนไปและทำไม
- การเปลี่ยนแปลงมีขอบเขตตรงกับสิ่งที่ฉันขอจริง ๆ
- เทสต์ผ่านทั้งหมด (และพฤติกรรมใหม่มีเทสต์รองรับ)
- การตรวจสอบชนิดข้อมูลและ linter สะอาด
- ไม่มีความลับ log สำหรับดีบัก หรือโค้ดที่ถูกคอมเมนต์ทิ้งค้างอยู่
- ไม่มีการตรวจสอบความปลอดภัยใดถูกลบออกไปอย่างเงียบ ๆ
- ฉันได้ลองใช้งานการเปลี่ยนแปลงนี้เองแล้ว อย่างน้อยก็เส้นทางการใช้งานปกติ
ถ้าคุณติ๊กช่องไหนไม่ได้ แปลว่ายังไม่เสร็จ
เช็คลิสต์นี้จงใจทำให้สั้น เพราะด่านที่คุณข้ามไปก็ไม่ต่างจากไม่มีด่านเลย สิ่งที่ล่อใจ โดยเฉพาะตอนที่คุณกำลังเร่งความเร็วและการเปลี่ยนแปลง "ดูเหมือนจะโอเค" คือการปล่อยผ่านไปเฉย ๆ จงต้านทานสิ่งนั้นเมื่อเป็นเรื่องที่ผู้ใช้เห็นได้ เทคนิคที่มีประโยชน์คือให้ AI กรอกเช็คลิสต์แทนคุณ — วางรายการแล้วถามว่า: "ตรวจเช็คลิสต์รีวิวนี้สำหรับการเปลี่ยนแปลงที่คุณเพิ่งทำ สำหรับแต่ละข้อ บอกว่า PASS หรือ FAIL พร้อมหลักฐานหนึ่งบรรทัด" มันไม่สามารถเขียนอย่างซื่อสัตย์ว่า "เทสต์ผ่าน" ได้โดยไม่ได้รันมันจริง ๆ และการบังคับให้มันอ้างหลักฐานในแต่ละบรรทัดจะเปลี่ยนคำว่า "ดูโอเค" ที่คลุมเครือ ให้กลายเป็นสิ่งที่คุณตรวจสอบได้