Pr Review

Verspreidt parallelle sub-agents om een branch/PR te reviewen in de win-dev-skills repo

Door microsoft · microsoft/win-dev-skills

Werkt met setup ★ 7.6/10

Pr Review — Verspreidt parallelle sub-agents om een branch/PR te reviewen in de win-dev-skills repo

Wat het doet

Een orchestrator skill die een in-progress branch of PR reviewt door de diff vast te leggen met een PowerShell helper en zes parallelle sub-agents te verspreiden (skill content, skill-vs-tool boundary, C#/PowerShell correctness, payload provenance, docs/manifest sync, plus een different-model cross-check), en vervolgens gededupliceerde bevindingen naar stdout consolideert zonder fixes toe te passen. Het activeert op zinsneden zoals "review my PR", "review my branch", "vet my changes before pushing", en "is this ready to merge?". Specifiek gebouwd voor de microsoft/win-dev-skills repo structuur in plaats van algemene code review.

Testrapport

SKILL.md gevonden op .github/skills/pr-review/ via de GitHub tree API en ruw opgehaald. Steekproefsgewijs 3 gerefereerde bestanden gecontroleerd (collect-diff.ps1, dimensions/_shared-contract.md, dimensions/skill-tool-boundary.md) — allemaal HTTP 200; de andere 6 dimensiefragmenten zijn aanwezig in de tree. Frontmatter parset netjes (name+description+infer), geen hardcoded machine paden, geen security smells (de .ps1 draait alleen git; geen curl|sh, base64, of exfiltratie). Kon OUTPUT NIET uitvoeren: pwsh is niet geïnstalleerd (which pwsh -> not found) en collect-diff.ps1 is #Requires -Version 7.0, en de zes sub-agent dimensies van de skill zijn hard-gebonden aan de win-dev-skills repo (WinUI analyzer, winui-search.exe, binary payloads), dus er werd geen representatief artefact geproduceerd. Trigger beoordeeld 5/5: MOET-activeren — "Review my PR before I push", "Vet my branch changes before pushing", "Is this ready to merge?" (alle vermelde trigger zinsneden); MOET-NIET — "Review this function for bugs" en "Is this skill paragraph okay?" (beide expliciet uitgesloten in de 'Do not activate' sectie). Docs zijn accuraat en gedetailleerd, maar de pwsh 7 vereiste en repo-specificiteit zijn begraven in de .ps1 header in plaats van naar voren gebracht, vandaar 4/5.

Getest op: 2026-07-21 · Claude Code 2.x (agent harness)

Installatie

git clone --depth 1 https://github.com/microsoft/win-dev-skills.git /tmp/pr-review-src
mkdir -p ~/.claude/skills
cp -R /tmp/pr-review-src/.github/skills/pr-review ~/.claude/skills/pr-review
# Requires PowerShell 7 (pwsh) on PATH: collect-diff.ps1 is #Requires -Version 7.0.
# Designed for the microsoft/win-dev-skills repo layout: the 6 dimension fragments in
# dimensions/ review that repo's WinUI Roslyn analyzer, winui-search.exe, and committed
# binary payloads. It also fans out 6 parallel sub-agents via the `task` tool (mode sync),
# #6 requiring a different model family. Outside that repo the diff step still runs but the
# dimension checks lose most of their relevance.

Commando's en voorbeeldprompts

/pr-reviewVerspreidt parallelle sub-agents om een branch/PR te reviewen in de win-dev-skills repo

Skills reageren op gewone verzoeken — geen commando's om te onthouden. Na installatie activeren prompts zoals deze de skill (in het Engels):

Review my PR before I push it
Vet this branch for skill-vs-tool boundary issues
Is this PR ready to merge

Volledig rapport (Engels) →