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.