Kỹ năng rà soát mã trong kho lưu trữ agent-skills của Addy Osmani: trước khi hợp nhất (hoặc khi rà soát mã do các agent khác hoặc người khác viết), hướng dẫn agent rà soát các thay đổi dựa trên năm chiều: tính chính xác, khả năng đọc, kiến trúc, bảo mật và hiệu suất. Ưu tiên xem xét các bài kiểm tra trước khi xem xét phần triển khai, đánh dấu mức độ nghiêm trọng cho mỗi nhận xét (Critical / Bắt buộc sửa / Nit / Optional / FYI), và cuối cùng đưa ra kết luận phê duyệt hoặc yêu cầu chỉnh sửa.
Số lượt cài đặt trên skills.sh 79294 (+975 / 24h)
Kiểm toán bảo mật: Chưa kiểm toán
- Chưa quét
Lý do
- Quy trình gồm 5 bước: trước tiên làm rõ ý định của thay đổi, sau đó rà soát các bài kiểm tra trước, rà soát phần triển khai theo từng tệp dựa trên năm chiều, liệt kê các vấn đề theo mức độ nghiêm trọng, và đối chiếu với các mô tả xác minh của tác giả; trước khi báo cáo, yêu cầu đối chiếu với các bên gọi, các điều kiện bảo vệ và các bài kiểm tra để loại trừ cảnh báo giả, kết luận không có vấn đề nào cũng là hợp lệ, không được phép bịa đặt vấn đề
- Việc đánh giá hiệu quả của các bài kiểm tra dựa trên thử nghiệm: đảo ngược một điều kiện trong thay đổi và chạy các bài kiểm tra, sau đó khôi phục tệp từ bản sao; nếu các bài kiểm tra vẫn vượt qua, điều đó có nghĩa là thiếu trường hợp kiểm thử, và cần chỉ ra trường hợp nào đang thiếu
- Cung cấp tham khảo về quy mô thay đổi (khoảng 100 dòng phù hợp để rà soát trong một lần, khoảng 1000 dòng nên được chia nhỏ) và bốn phương pháp chia nhỏ; đối với việc nâng cấp phụ thuộc, cần đọc nhật ký thay đổi, nâng cấp từng gói một lần, và rà soát sự khác biệt trong tệp khóa
- Khi phát hiện mã không sử dụng, trước tiên liệt kê danh sách và hỏi xem có nên xóa hay không, không tự ý xóa; khi nêu ra các vấn đề về cấu trúc, cần đồng thời cung cấp các phương pháp tái cấu trúc cụ thể
- Kho lưu trữ được cấp phép MIT, cùng kho lưu trữ có 24 kỹ năng khác; skills.sh có 975 lượt cài đặt trong 24 giờ, tổng cộng 79294, số lượt cài đặt hàng tuần trong 8 tuần gần đây tăng từ 3503 lên 5359 (chỉ phản ánh mức độ quan tâm)
Cách sử dụng
- Lệnh cài đặt sẽ hiển thị sau khi vượt qua kiểm toán; kích hoạt khi nói “rà soát PR / diff này”, khi hoàn thành tính năng và chuẩn bị hợp nhất, hoặc khi cần đánh giá mã do các agent khác viết
- Không cần tài khoản hoặc dịch vụ bên ngoài; bước “đảo ngược điều kiện và chạy kiểm tra” yêu cầu môi trường cục bộ có thể chạy các bài kiểm tra của dự án
Rủi ro
- Chưa kiểm toán trong kỳ này, xếp hạng cố định ở mức C, không hiển thị lệnh cài đặt; SKILL.md chủ yếu là danh sách kiểm tra rà soát, hành động duy nhất liên quan đến việc sửa đổi tệp là “tạm thời đảo ngược một điều kiện để chạy kiểm tra, sau đó khôi phục từ bản sao”, phù hợp với mục đích sử dụng, nhưng cần xác nhận rằng tệp đã được khôi phục; trên skills.sh, Gen Agent Trust Hub và kiểm toán Socket có kết quả Pass, Snyk có kết quả Warn
- Các phần về bảo mật và hiệu suất tham chiếu đến các kỹ năng security-and-hardening, performance-optimization trong cùng kho lưu trữ và các tệp danh sách ở thư mục gốc của kho lưu trữ; nếu chỉ cài đặt kỹ năng này, các liên kết đó có thể không truy cập được
- Tiêu chuẩn phê duyệt khá rộng: chỉ cần cải thiện sức khỏe tổng thể của mã là được phê duyệt; các quy chuẩn nhóm như “phản hồi trong vòng một ngày làm việc” ít có ý nghĩa đối với các dự án cá nhân