SkillProof tested Claude skills

Pr Review

Verteilt parallele Sub-Agenten, um einen Branch/PR im win-dev-skills Repo zu überprüfen

von microsoft · microsoft/win-dev-skills

Funktioniert mit Setup ★ 7.6/10

Pr Review — Verteilt parallele Sub-Agenten, um einen Branch/PR im win-dev-skills Repo zu überprüfen

Was es kann

Ein Orchestrator-Skill, der einen in Bearbeitung befindlichen Branch oder PR überprüft, indem er den Diff mit einem PowerShell-Helfer erfasst und sechs parallele Sub-Agenten (Skill-Inhalt, Skill-vs-Tool-Grenze, C#/PowerShell-Korrektheit, Payload-Herkunft, Docs/Manifest-Synchronisation plus einen Cross-Check mit einem anderen Modell) verteilt, dann deduplizierte Ergebnisse ohne Anwendung von Korrekturen auf stdout konsolidiert. Er wird ausgelöst bei Phrasen wie „review my PR“, „review my branch“, „vet my changes before pushing“ und „is this ready to merge?“. Speziell für die microsoft/win-dev-skills Repo-Struktur und nicht für allgemeine Code-Reviews entwickelt.

Testbericht

SKILL.md unter .github/skills/pr-review/ über die GitHub tree API gefunden und roh abgerufen. 3 referenzierte Dateien (collect-diff.ps1, dimensions/_shared-contract.md, dimensions/skill-tool-boundary.md) stichprobenartig überprüft – alle HTTP 200; die anderen 6 Dimensionsfragmente sind im Baum vorhanden. Frontmatter wird sauber geparst (name+description+infer), keine fest codierten Maschinenpfade, keine Sicherheitsmängel (die .ps1 führt nur git aus; kein curl|sh, base64 oder Exfiltration). Konnte OUTPUT NICHT ausführen: pwsh ist nicht installiert (which pwsh -> not found) und collect-diff.ps1 ist #Requires -Version 7.0, und die sechs Sub-Agenten-Dimensionen des Skills sind fest an das win-dev-skills Repo gebunden (WinUI Analyzer, winui-search.exe, binäre Payloads), so dass kein repräsentatives Artefakt erzeugt wurde. Trigger 5/5 beurteilt: SOLLTE-aktivieren – „Review my PR before I push“, „Vet my branch changes before pushing“, „Is this ready to merge?“ (alle aufgelisteten Trigger-Phrasen); SOLLTE-NICHT – „Review this function for bugs“ und „Is this skill paragraph okay?“ (beide explizit im Abschnitt „Do not activate“ ausgeschlossen). Die Dokumentation ist genau und detailliert, aber die pwsh 7-Voraussetzung und Repo-Spezifität sind im .ps1-Header vergraben statt sichtbar gemacht, daher 4/5.

Getestet am: 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.

Befehle & Beispiel-Prompts

  • /pr-reviewVerteilt parallele Sub-Agenten, um einen Branch/PR im win-dev-skills Repo zu überprüfen

Skills reagieren auf normale Anfragen — keine Slash-Befehle nötig. Nach der Installation aktivieren Prompts wie diese den Skill (auf Englisch):

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

Vollständiger Testbericht (Englisch) →