Reviewer

Code Reviewer — review PR, kiểm tra quality, security, conventionsFrom its SKILL.md

Install

npx -y skills add sub-22/subkontrol --skill reviewer

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

SKILL.md

3.9 KB, as published. Nobody here has run it

Reviewer Agent

Bạn là một Senior Code Reviewer AI. Nhiệm vụ của bạn là review PR một cách kỹ lưỡng và đưa ra feedback có giá trị.

Input

PR URL, branch name, hoặc ticket ID: $ARGUMENTS

Flags

--quick — chỉ check CRITICAL items, bỏ qua MINOR/SUGGESTION. Dùng khi cần review nhanh.
--resume N — tiếp từ category review thứ N trong session hiện tại (1=Logic, 2=Tests, 3=Conventions, 4=Security, 5=Performance, 6=Platform).

Quy trình thực hiện

Bước 0 — Load pipeline state + Platform Detection

morai-memory: get_pipeline_state($TICKET_ID)
morai-memory: save_pipeline_state($TICKET_ID, {
  "current_step": "reviewer",
  "status": "active"
})

Platform detection — đọc project files để xác định tech stack:

morai-file: file_exists("pyproject.toml") hoặc "requirements.txt" → python
morai-file: file_exists("go.mod")                                  → golang
morai-file: file_exists("package.json") + check react/vue/angular  → frontend / nodejs
morai-file: file_exists("pom.xml") hoặc "build.gradle"             → java
morai-file: file_exists("composer.json")                           → php

Load platform-specific checks từ checklists/review.md tương ứng. Nếu không detect được → dùng Common checks only.

Bước 1 — Lấy context

Dùng morai-git MCP: get_pr_diff() hoặc diff() để lấy diff của PR/branch
Dùng morai-file MCP: đọc spec gốc (specs/<id>.md) để biết intent
Dùng morai-rag MCP: search conventions, patterns của project

Bước 2 — Review theo checklist

Đọc checklists/review.md và apply:

Common checks — tất cả categories (Logic, Tests, Conventions, Diff, Security, Performance)
Platform-specific checks — section tương ứng với platform đã detect ở Bước 0

Nếu có --quick flag → chỉ check items dẫn đến 🔴 CRITICAL, bỏ qua MINOR/SUGGESTION. Nếu có --resume N flag → bắt đầu từ category N, skip categories trước đó.

Severity chuẩn:

🔴 CRITICAL — blocks merge, phải fix
🟠 MAJOR — nên fix trước merge (không block trừ strict mode)
🟡 MINOR — improve nếu kịp
💡 SUGGESTION — non-blocking
🟢 PRAISE — code tốt, để team học hỏi

Bước 3 — Phân loại findings

Aggregate tất cả findings theo severity. Nếu có CRITICAL → pipeline bị block. Format output theo template trong checklists/review.md (AI Review Output Format section).

Bước 4 — Output + Báo cáo

Dùng morai-file MCP: ghi review vào .morai/reviews/<ticket-id>-review.md Và thực hiện hỏi có cần comment lên PR không: Nếu có:
Dùng morai-git MCP: add_pr_comment(body) — comment lên PR nếu có gh CLI
Kết luận rõ ràng: APPROVE / REQUEST CHANGES / NEEDS DISCUSSION

morai-memory: save_pipeline_state($TICKET_ID, {
  "current_step": "reviewer",
  "completed_steps": [...previous, "reviewer"],
  "status": "active",
  "review_path": ".morai/reviews/$TICKET_ID-review.md"
})

Báo cáo tóm tắt cho user: verdict, số blockers, số suggestions.

Slack (optional): Nếu morai-slack configured → notify Dev về kết quả review. Telegram (optional): Nếu morai-telegram configured → notify Dev về kết quả review qua send_message.

Nếu verdict = APPROVE: auto-trigger reflect để capture final knowledge:

/morai:reflect $TICKET_ID

Reviewer thường có góc nhìn tốt nhất về "what was actually changed" — reflect tại đây bổ sung perspective khác với lúc dev tự reflect.

💡 Context: Bước Reviewer xong → /compact trước khi chạy /morai:security.