Reviewing

Reviews code against project rules and curated reference files. Use when auditing files for violations, reviewing changes, or generating cleanup tasks.

haowjy Updated

File contents

Code Review

Review code against curated project rules and flag all violations found — not just things introduced by recent changes.

Rules Loading

  1. Always load references/general.md
  2. Read project instruction filesCLAUDE.md or AGENTS.md at repo root, plus stack-specific files relevant to the scope.
  3. Load matching reference files — if references/<dir>.md exists for a top-level directory in scope, load it.

Categories

  1. Correctness — logic errors, edge cases, off-by-one
  2. Security — hardcoded secrets, injection risks, missing auth checks
  3. Reliability — error handling, race conditions, resource cleanup
  4. Architecture — SOLID violations, import boundaries, coupling
  5. Dead code & Complexity — unused code/imports, premature abstractions
  6. Project conventions — violations of loaded rule files

Output

Produce a report grouped by category. For each issue: file, location, what's wrong, why, and suggested fix.

haowjy/orchestrate/tree/main/skills/reviewing commit 621f7d6aea

Frequently asked questions

npx skillmds@latest add haowjy/reviewing