Skill · SKILL.md

Một bản review không đóng dấu cho qua

Một Skill tái dùng được, review một thay đổi để tìm vấn đề thật thay vì duyệt theo phản xạ.

DạngSKILL.md

Bạn nhờ agent review thay đổi trước khi ship. Nó đọc diff và bảo trông ổn — sạch, hợp lý, không có gì lo. Ship. Con bug nằm ở ca biên mà diff không phơi ra: đầu vào rỗng, lần gọi đồng thời thứ hai, cái nơi gọi duy nhất vẫn truyền hình dạng cũ. Bản review không hẳn sai. Nó chỉ chưa bao giờ nhìn vào nơi bug sống.

Một bản review chỉ xác nhận lại những gì diff phơi ra không phải review — nó là cái dấu đóng cho qua khoác thêm vài chữ. Một bản review thật đi xuống một bộ chiều cố định, lần nào cũng vậy, và chỉ được phép "không thấy gì" sau khi đã nói nó kiểm cái gì. Đóng gói thành một Skill biến sự kỹ lưỡng đó thành lặp-lại-được, thay vì phụ thuộc tâm trạng:

✕ Đóng dấu cho qua

Đọc diff từ trên xuống

▼

"Trông ổn, không có gì lo"

▼

Cái ca biên nó chưa mở ra bị ship đi

✓ Review thật

Đi qua đúng đắn → ca biên → ảnh hưởng → dễ đọc

▼

Mỗi phát hiện: file, mức độ, bản sửa nhỏ nhất

▼

Không gì lọt, vì không gì bị bỏ qua

Lưu thành một SKILL.md và gọi nó trên bất kỳ thay đổi nào đáng nhìn lần hai. Các chiều quan trọng hơn cái phán quyết: đúng đắn thì bạn thấy được, nhưng ca biên, bán kính ảnh hưởng, và người đọc kế tiếp mới là nơi những con bug đắt giá ẩn nấp. Cái luật nó không bao giờ được phá — nếu không thấy gì, nó phải nói đã kiểm cái gì và vì sao chắc. Duyệt-mặc-định chính là cách cái ca biên được ship đi.

Lưu thành một Skill

---
name: review-changes
description: Review a diff for real problems before it ships — no rubber-stamping
---
Review the change across these dimensions, in order:
1. **Correctness** — does it do what it claims? Trace one real input through the new path.
2. **Edge cases** — empty, null, huge, the second call, concurrent access. Where does it fall at the edges?
3. **Blast radius** — what else calls this? What silently changes for them?
4. **Readability** — could the next person change it without you in the room?

For each finding, report: `file:line` · severity (blocker / should-fix / nit) · why it matters · the smallest fix.
If you find nothing, say what you checked and why you're confident. Never approve by default.

Muốn agent tự cài hộ?

Thêm rule này vào CLAUDE.md của tôi, nguyên văn:

---
name: review-changes
description: Review a diff for real problems before it ships — no rubber-stamping
---
Review the change across these dimensions, in order:
1. **Correctness** — does it do what it claims? Trace one real input through the new path.
2. **Edge cases** — empty, null, huge, the second call, concurrent access. Where does it fall at the edges?
3. **Blast radius** — what else calls this? What silently changes for them?
4. **Readability** — could the next person change it without you in the room?

For each finding, report: `file:line` · severity (blocker / should-fix / nit) · why it matters · the smallest fix.
If you find nothing, say what you checked and why you're confident. Never approve by default.

Bài này giúp được bạn không?

Thẻ#kiểm chứng #quy trình #skill

Skill khác

01Kiểm chứng trước khi gọi là xong 02Plan trước, rồi mới code 03Lập bản đồ ảnh hưởng trước khi sửa 04Việc có một đáp án thì đừng để model đoán 05Chẩn đoán từ triệu chứng, không từ phỏng đoán 06Bắt nó phản biện chính kế hoạch của nó 07Bàn giao cho phiên làm việc sau 09Chạy tới cổng thì dừng, không phải soi từng bước 10Bộ khung CLAUDE.md — biến file mô tả thành luật vận hành 11Vòng review hàng tuần — năm phút biến trải nghiệm thành kỹ năng 12Chia việc thành mảnh gọn, mỗi mảnh một context sạch 13Việc này có đáng giao cho agent không — hỏi trước khi mở chat 14Thử một việc nhỏ trước khi tin agent ở miền mới

Về Skill thực chiến

Người viết

Mỗi câu chuyện ở đây gói một bài học đã trả giá để học.

craftagentmột người vừa xây vừa học

Bạn đang xây gì với agent? Muốn trao đổi, phản biện, hay cùng làm một thứ gì đó — viết cho mình một dòng.

Viết thư hello@craftagent.cloud

58bài12cụmVI·ENsong ngữ

✕ Đóng dấu cho qua

✓ Review thật

Nhận bài mới qua email