Skill: qc-code

WHAT

Purpose và context

• Purpose: review code phong cách senior theo bốn mục security · performance · naming · logic — mỗi mục điểm/10 + lỗi nặng nhất + cách sửa — sinh test ĐỎ tái hiện cho mỗi bug logic, rồi ra verdict advisory PASS/CẦN SỬA kèm JSON có bằng chứng.
• Trigger (when to use):
  • User muốn một cặp mắt senior soi code trước khi chốt: "review code", "soi code", "chấm điểm code".
  • Trước commit một thay đổi đáng kể — cắm tùy chọn vào /orca-workflow trước verify-before-commit.
• Non-goals: KHÔNG dùng cho: quét bảo mật tĩnh (đó là /orca-sec-scans — Trivy) hay xử lý một sự cố đã xảy ra (đó là /orca-issue — repro-first). Không review tổng quát kiểu /code-review built-in; không mặc định toàn codebase; verdict không chặn commit.

Mental model

phạm vi (diff từ base | file user nêu) → 4 mục chấm điểm → finding [nhóm][severity] → bug logic → test qc-* ĐỎ → verdict văn xuôi + qc-verdict.json (grounding-check) → test ghi vào dự án → qc-regression.py --run auto-chạy (0-token). Đắt = LLM review gọi tay; rẻ = test tất định qua hook.

Phạm vi review (mặc định = diff hiện tại)

Mặc định soi thay đổi kể từ base (git diff từ commit/branch gốc) — đúng lúc trước commit, nhanh, đúng thứ vừa viết. User chỉ định file/thư mục thì soi cái đó. KHÔNG mặc định toàn codebase (chậm, tốn token, phần lớn không đổi).