Run full Squad team review on a pull request before merge
Every PR in this repository MUST receive a full Squad team review before merging (AD-005). No exceptions, regardless of PR size. This skill defines the review process.
github-mcp-server-pull_request_read with method get to get PR metadataget_files to list changed filesget_diff to read the actual changesview to read the full content of changed files when diff context is insufficientEach reviewer's perspective is defined by their charter:
.squad/agents/avery/charter.md — Lead/Architect.squad/agents/morgan/charter.md — C# Generator Developer.squad/agents/quinn/charter.md — DevOps/Scripts Engineer.squad/agents/sage/charter.md — AI/Prompt Engineer.squad/agents/parker/charter.md — QA/Tester.squad/agents/reeve/charter.md — Documentation EngineerAlso read .squad/decisions.md for project decisions that apply.
Not every reviewer reviews every PR. Route based on changed files:
| Changed files | Required reviewers |
|---|---|
*.cs (non-test) | Morgan + Avery |
*.Tests/*.cs | Parker + Avery |
*.ps1, *.sh, Dockerfile, *.yml | Quinn + Avery |
prompts/, GenerativeAI/ | Sage + Avery |
docs/, README.md, *.md (non-generated) | Reeve + Avery |
| Any code change | Parker (test coverage review) |
| Any PR | Reeve (documentation gate per AD-004) |
Avery always reviews. Parker and Reeve always review code changes.
For each relevant reviewer, provide a review in this format:
### {Name} ({Role}) — {APPROVED or CHANGES REQUESTED}
{2-3 sentences reviewing from their domain perspective}
**Issues found:** (if any)
- {specific issue with file and line reference}
Review guidelines:
Post the complete review as a single PR comment using gh pr comment.
Format:
## Squad Team Review — PR #{number}
{Individual reviews from each relevant reviewer}
---
**Summary: {X} APPROVED, {Y} CHANGES REQUESTED.**
{If changes requested: "Address {reviewer} blockers before merge."}
{If all approved: "Ready to merge."}
Small fix (2 files, URL change): Still gets Avery (architecture) + Reeve (docs) + relevant domain reviewer.
Large feature (10+ files, new test project): Gets full team: Avery + Morgan + Parker + Sage + Reeve.