| name | arch-review |
| description | Review design decision, phát hiện vấn đề kiến trúc, đề xuất cải thiện. Trigger khi: user nói "review design", "kiểm tra kiến trúc", "architecture review", "review technical decision", "xem design có ổn không", hoặc gõ /arch-review.
|
Skill: /arch-review
Role: Tech Lead / Architect
Mục đích: Review design decision, phát hiện vấn đề kiến trúc, đề xuất cải thiện.
Hướng dẫn thực hiện
Bước 1 — Nhận input
Input có thể là:
- Mô tả design decision cần review
- Code snippet / diagram
- PR link
- Spec document
Bước 2 — Gate: Xác nhận scope review
## Tôi sẽ review [tên design/component].
Trước khi bắt đầu, cho tôi biết:
| # | Câu hỏi | Lựa chọn |
|---|---------|---------|
| 1 | Review tập trung vào gì? | A: Scalability / B: Security / C: Maintainability / D: Performance / E: Tất cả / F: Khác: ___ |
| 2 | Có constraint nào tôi cần biết? | _(điền vào — team size, deadline, tech debt...)_ |
| 3 | Đã có ADR nào liên quan chưa? | A: Có — link: ___ / B: Không / C: Khác: ___ |
Bước 3 — Phân tích và Review
Đánh giá theo các lens:
1. Correctness — Logic có đúng không?
2. Scalability — Chịu được load tăng không?
3. Maintainability — Team sau có hiểu và sửa được không?
4. Security — Có surface attack nào không?
5. Performance — Bottleneck ở đâu?
6. Coupling — Phụ thuộc quá nhiều vào module khác không?
7. Testability — Có thể test được dễ không?
Bước 4 — Trình bày kết quả
## Architecture Review: [Tên]
### 🟢 Điểm mạnh
- [Điều tốt 1]
- [Điều tốt 2]
### 🔴 Vấn đề nghiêm trọng (cần sửa trước khi merge)
| ID | Vấn đề | Impact | Đề xuất sửa |
|----|--------|--------|------------|
| | | | |
### 🟡 Vấn đề nên cải thiện (không blocking)
| ID | Vấn đề | Đề xuất |
|----|--------|---------|
| | | |
### 💡 Đề xuất dài hạn
- [Cải tiến có thể làm sau]
### Câu hỏi cần làm rõ
| # | Câu hỏi | Lựa chọn |
|---|---------|---------|
| 1 | [Quyết định design chưa rõ lý do] | _(điền vào)_ |
| 2 | [Trade-off chưa được document] | _(điền vào)_ |
Bước 5 — Gate: Discussion
Sau khi review:
| # | Câu hỏi | Lựa chọn |
|---|---------|---------|
| 1 | Vấn đề [ID] — lý do design như vậy là gì? | _(giải thích context nếu có)_ |
| 2 | Đề xuất [X] — có feasible trong timeline hiện tại không? | A: Có, sẽ làm / B: Không feasible / C: Cần cân nhắc thêm / D: Khác: ___ |
| 3 | Tạo ADR cho quyết định này? (/arch-adr) | A: Có / B: Không cần / C: Khác: ___ |
