SkillProof tested Claude skills

Pr Review

Sprider ut parallella underagenter för att granska en gren/PR i win-dev-skills-arkivet

av microsoft · microsoft/win-dev-skills

Fungerar med konfiguration ★ 7.6/10

Pr Review — Sprider ut parallella underagenter för att granska en gren/PR i win-dev-skills-arkivet

Vad den gör

En orkestreringsfärdighet som granskar en pågående gren eller PR genom att fånga diffen med en PowerShell-hjälpare och sprida ut sex parallella underagenter (färdighetsinnehåll, färdighet-vs-verktygsgräns, C#/PowerShell-korrekthet, payload-proveniens, docs/manifest-synkronisering, plus en korscheck med annan modell), och sedan konsolidera deduplicerade fynd till stdout utan att tillämpa fixar. Den utlöses på fraser som "review my PR", "review my branch", "vet my changes before pushing" och "is this ready to merge?". Byggd specifikt för microsoft/win-dev-skills-arkivstrukturen snarare än allmän kodgranskning.

Testrapport

Hittade SKILL.md på .github/skills/pr-review/ via GitHub tree API och hämtade den rått. Punktkontrollerade 3 refererade filer (collect-diff.ps1, dimensions/_shared-contract.md, dimensions/skill-tool-boundary.md) — alla HTTP 200; de andra 6 dimensionsfragmenten finns i trädet. Frontmatter parsas rent (name+description+infer), inga hårdkodade maskinsökvägar, inga säkerhetsbrister (ps1-filen kör endast git; ingen curl|sh, base64 eller exfiltrering). Kunde INTE köra OUTPUT: pwsh är inte installerat (which pwsh -> hittades inte) och collect-diff.ps1 är #Requires -Version 7.0, och färdighetens sex underagentdimensioner är hårt bundna till win-dev-skills-arkivet (WinUI analyzer, winui-search.exe, binära nyttolaster), så ingen representativ artefakt producerades. Trigger bedömdes 5/5: SKA-aktiveras — "Review my PR before I push", "Vet my branch changes before pushing", "Is this ready to merge?" (alla listade trigger-fraser); SKA-INTE — "Review this function for bugs" och "Is this skill paragraph okay?" (båda explicit exkluderade i avsnittet 'Do not activate'). Dokumenten är korrekta och detaljerade men pwsh 7-förutsättningen och arkivspecificiteten är begravda i ps1-huvudet snarare än synliggjorda, därav 4/5.

Testad: 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.

Kommandon och exempelprompter

  • /pr-reviewSprider ut parallella underagenter för att granska en gren/PR i win-dev-skills-arkivet

Skills triggas av vanliga förfrågningar — inga kommandon att memorera. Efter installationen aktiverar prompter som dessa skillen (på engelska):

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

Läs hela rapporten →