Pr Review

Spreder parallelle sub-agenter til at gennemgå en branch/PR i win-dev-skills depotet

Af microsoft · microsoft/win-dev-skills

Virker med opsætning ★ 7.6/10

Pr Review — Spreder parallelle sub-agenter til at gennemgå en branch/PR i win-dev-skills depotet

Hvad det gør

En orkestrator-færdighed, der gennemgår en igangværende branch eller PR ved at fange diff'en med en PowerShell-hjælper og sprede seks parallelle sub-agenter (færdighedsindhold, færdighed-vs-værktøj-grænse, C#/PowerShell-korrekthed, payload-proveniens, docs/manifest-synkronisering, plus en anden-model krydscheck), og derefter konsolidere deduplikerede fund til stdout uden at anvende rettelser. Den udløses på fraser som "review my PR", "review my branch", "vet my changes before pushing" og "is this ready to merge?". Formålsbygget til microsoft/win-dev-skills depotstrukturen snarere end generel kodeanmeldelse.

Testrapport

Fandt SKILL.md på .github/skills/pr-review/ via GitHub tree API og hentede den rå. Stikprøvekontrollerede 3 refererede filer (collect-diff.ps1, dimensions/_shared-contract.md, dimensions/skill-tool-boundary.md) — alle HTTP 200; de andre 6 dimensionsfragmenter er til stede i træet. Frontmatter parses rent (name+description+infer), ingen hardkodede maskinstier, ingen sikkerhedsbrister (den .ps1 kører kun git; ingen curl|sh, base64 eller exfiltration). Kunne IKKE køre OUTPUT: pwsh er ikke installeret (which pwsh -> not found), og collect-diff.ps1 er #Requires -Version 7.0, og færdighedens seks sub-agent dimensioner er hårdt bundet til win-dev-skills depotet (WinUI analyzer, winui-search.exe, binære payloads), så ingen repræsentativ artefakt blev produceret. Udløser bedømt 5/5: SKAL-aktivere — "Review my PR before I push", "Vet my branch changes before pushing", "Is this ready to merge?" (alle listede udløserfraser); SKAL-IKKE — "Review this function for bugs" og "Is this skill paragraph okay?" (begge eksplicit udelukket i 'Do not activate'-sektionen). Dokumenter er nøjagtige og detaljerede, men pwsh 7-forudsætningen og depot-specificiteten er begravet i .ps1-headeren snarere end fremhævet, deraf 4/5.

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

Installation

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.

Kommandoer og eksempelprompter

  • /pr-reviewSpreder parallelle sub-agenter til at gennemgå en branch/PR i win-dev-skills depotet

Skills udløses af almindelige forespørgsler — ingen kommandoer at huske. Efter installationen aktiverer prompter som disse skillen (på engelsk):

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