code-reviewer

Expert Code Review

Lean, pragmatic code reviews with scope discipline. Stack-specific PE sub-agents do the heavy lifting; the parent skill orchestrates dispatch and consolidates findings.

---

Phase 1: Scope Analysis

Determine review type from the user's request:

User Request	Review Type	What to Review
"Review my branch" / "Review this PR"	Branch Diff	git diff main...HEAD
GitHub PR URL or "Review PR #N"	PR Review	git diff <target>...<source> (auto-checkouts source branch)
"Review staged changes"	Staged Diff	git diff --cached
"Review these files: X, Y, Z"	Specific Files	Read and review entire files
"Review the src/services folder"	Folder	Read and review all files in folder
"Review this code" + pasted code	Ad-hoc	Review the provided code

Branch / Staged

git diff main...HEAD      # branch diff
git diff --cached         # staged changes

if current_branch == "main":
  ask user which branch to review (do NOT proceed)

PR

gh pr view <PR_NUMBER> --json number,url,headRefName,baseRefName,author
git checkout <source_branch>
git pull
git diff <target_branch>...<source_branch>

Capture: source branch, target branch, PR number, PR URL, author. Store for Phase 6.

Critical: Review files MUST be committed to the source branch, never to main.

Files / Folders / Ad-hoc

Read the files directly. No diff context — review entire content.

Review Name (used for output filename)

if branch matches "<type>/<issue-id>-<description>":
  name = issue_id                 # e.g., "983"
elif on a branch (no issue ID):
  name = slugify(branch)          # e.g., "fix-code-reviewer-cleanup"
else:
  ask user for a short descriptive name

PE Routing

git diff main...HEAD --stat | tail -1          # branch reviews (informational)
git diff <target>...<source> --stat | tail -1  # PR reviews (informational)

matching_pes = PEs whose Stack Map paths intersect with files in the diff

if matching_pes is empty:
  approach = "generic"     # primary runs generic three-pass (no PE expertise available)
elif len(matching_pes) == 1:
  approach = "single_pe"   # dispatch the one matching PE
else:
  approach = "multi_pe"    # dispatch all matching PEs in parallel