review

Tổng quan về review skill

review skill làm gì

review là một review skill PR trước khi merge, dùng để kiểm tra diff so với nhánh base trước khi code được đưa vào. Skill này dành cho reviewer cần nhiều hơn một prompt chung chung: nó tập trung vào những rủi ro cấu trúc cụ thể như an toàn SQL, vi phạm trust boundary, tác dụng phụ có điều kiện, shell injection, và độ đầy đủ của enum.

Ai nên dùng

Hãy dùng review skill khi bạn sắp chốt code, trả lời “review PR này,” hoặc muốn có một workflow review for PR Review nhất quán giữa các repo. Nó đặc biệt hữu ích cho các agent cần phát hiện nhanh lỗi thực sự và tránh các comment nhiễu chỉ về style.

Điểm khác biệt

Skill này có quan điểm rõ về thứ tự review và mức độ nghiêm trọng. Nó ưu tiên các kiểm tra an toàn quan trọng trước, rồi mới đến các vấn đề mức độ thấp hơn, đồng thời hỗ trợ các hướng review chuyên biệt cho những mảng như testing, security, performance, và maintainability. Nhờ vậy, review guide trở nên hữu dụng hơn nhiều so với một prompt rộng kiểu “quét diff đi.”

Phù hợp và giới hạn

Skill này mạnh nhất với các thay đổi code có git diff rõ ràng và một nhánh base có ý nghĩa. Nó kém hữu ích hơn cho các nhận xét ở mức thiết kế tổng thể, chỉnh sửa nội dung sản phẩm, hoặc những trường hợp người dùng chỉ muốn lời khen hay một bản tóm tắt. Nếu bạn chỉ cần một ý kiến nhẹ thay vì một review ưu tiên sửa lỗi, thì prompt chung có thể đã đủ.

Cách dùng review skill

Cài đặt và kích hoạt

Cài bằng npx skills add garrytan/gstack --skill review. Skill này được thiết kế để kích hoạt khi gặp các cụm như “review this PR,” “code review,” “check my diff,” và “pre-landing review,” vì vậy đầu vào nên nêu rõ thay đổi và điểm so sánh bất cứ khi nào có thể.

Đưa cho skill một mục tiêu review thật sự

Cách dùng review usage tốt nhất là bắt đầu bằng một yêu cầu có nhận thức về diff, thay vì hỏi mơ hồ. Input tốt nên có repo, branch, và mục đích:

• “Review PR này so với origin/main; tập trung vào safety, rủi ro merge, và bất kỳ thứ gì có thể chặn việc landing.”
• “Review diff trong packages/billing về SQL injection, race conditions, và lỗi phá vỡ API contract.”

Đọc các file này trước

Để kiểm tra review install một cách thực tế, hãy bắt đầu với SKILL.md, rồi đến checklist.md, design-checklist.md, và greptile-triage.md. Nếu bạn đang điều chỉnh workflow, hãy xem thêm TODOS-format.md và thư mục specialists/ để hiểu issue nào được chuyển cho subagent và issue nào ở lại với checklist chính.

Dùng đúng workflow mà skill mong đợi

Repo này được xây dựng theo quy trình review hai lượt: kiểm tra an toàn quan trọng trước, rồi mới đến các vấn đề mang tính thông tin. Một prompt mạnh nên yêu cầu output có thể hành động được, kèm tham chiếu file/line và hướng sửa, chứ không chỉ “tìm bug.” Nếu bạn biết khu vực thay đổi, hãy nêu ra để skill có thể ưu tiên đúng chuyên gia phù hợp.

FAQ về review skill

review có tốt hơn prompt bình thường không?

Có, khi bạn cần hành vi review PR lặp lại và nhất quán. Prompt bình thường có thể bỏ sót quy tắc phạm vi, thứ tự leo thang, hoặc việc chuyển sang chuyên gia phù hợp. review skill đóng gói sẵn những yếu tố đó vào workflow, giúp giảm phần phải đoán xem nên kiểm tra gì trước.

review skill có thân thiện với người mới không?

Phần lớn là có, nếu người dùng mô tả được branch hoặc PR. Người mới hưởng lợi từ checklist có hướng dẫn, nhưng họ vẫn cần cung cấp một diff thật và đủ ngữ cảnh để phân biệt hành vi mong đợi với regression.

Khi nào không nên dùng review?

Đừng dùng nó cho câu hỏi không liên quan đến code, brainstorming, hoặc thảo luận kiến trúc trừu tượng mà không có một bộ thay đổi cụ thể. Nó cũng không phù hợp nếu bạn chỉ muốn kiểm tra nhanh cho yên tâm và không cần kết quả ở mức line-by-line.

Tôi nên kỳ vọng gì từ review for PR Review?

Hãy kỳ vọng các comment ngắn gọn, thiên về sửa lỗi, với thứ tự ưu tiên theo mức độ nghiêm trọng. Skill này được thiết kế để nêu ra các blocker trước khi landing, chứ không phải để nhắc lại toàn bộ repo hay tạo một bản tóm tắt mang tính ăn mừng.

Cách cải thiện review skill

Đưa đầu vào sắc hơn

Những input mạnh nhất cho review skill sẽ nêu rõ nhánh base, subsystem có rủi ro, và tiêu chuẩn mong muốn. Thay vì “review PR của tôi,” hãy dùng “review feature/payment-retry so với origin/main; ưu tiên data safety, idempotency, và mọi thay đổi hành vi có thể làm hỏng client.”

Cung cấp ngữ cảnh ảnh hưởng đến đánh giá

Nếu thay đổi này cố ý chấp nhận rủi ro, hãy nói rõ. Nếu project policy cho phép một pattern nào đó, hãy nói trước. Điều đó giảm false positive và giúp review tập trung vào tradeoff thực sự thay vì tranh luận lại những quyết định đã biết.

Yêu cầu đúng định dạng bạn cần

Nếu bạn muốn output theo hướng sẵn sàng để landing, hãy yêu cầu findings theo file:line, severity, và cách sửa cụ thể. Nếu bạn chỉ muốn các blocker, hãy nói rõ vậy. Nếu bạn muốn phủ chuyên gia, hãy yêu cầu thẳng để review không dừng ở lượt đầu tiên.

Lặp lại sau lượt đầu

Dùng lượt review đầu tiên để vá các vấn đề rõ ràng, rồi chạy lại review trên diff đã cập nhật. Những cải thiện chất lượng lớn nhất thường đến từ việc làm rõ phạm vi, siết chặt nhánh so sánh, và đưa đúng vùng rủi ro bạn quan tâm nhất vào skill.