| name | code-review-skill |
| description | Use when: running pre-merge quality reviews for correctness, architecture fit, maintainability, and risk. |
Skill: Code Review và Quality Gate
Mục tiêu
Thiết lập quality gate nhất quán trước khi merge.
Khi dùng
- Mọi PR trước merge.
- Đặc biệt khi thay đổi auth/network/router/providers.
Trục review bắt buộc
- Đúng yêu cầu nghiệp vụ.
- Đúng kiến trúc và dependency direction.
- Dễ đọc, dễ bảo trì.
- Error handling và edge cases.
- Rủi ro bảo mật/hiệu năng.
Phân loại nhận xét
- Critical: bắt buộc sửa.
- Important: nên sửa trước merge.
- Suggestion: cải tiến tùy chọn.
Quy tắc Beacon
- Không approve nếu còn hardcode route/endpoint/storage key.
- Không approve nếu bỏ qua verify tối thiểu (
flutter analyze).
Done khi
- Không còn Critical.
- Người review và người sửa thống nhất phạm vi còn lại.