Demand-First Review
Markiert spekulative oder ungenutzte neue API-Oberfläche, bevor die Implementierungsqualität geprüft wird.
Getestet · Funktioniert
Was es kann
Einsatz beim Review eines PR, einer API, eines IPC-Channels, Endpoints, Parameters, Typs oder einer Config, die neue Oberfläche hinzufügt — BEVOR die Implementierungsqualität kommentiert wird. Auch sofort einsetzen, wenn ein Review Signale wie „noch keine Consumer“, „ungenutzter Export“, „spekulativ“, „additiv“, „zukunftskompatibel“ oder „für künftige Verwendung“ liefert.
Testbericht
Verbatim-Installation (git clone + cp) funktionierte ohne jede Rettung: eine gültige SKILL.md, strenges YAML-Frontmatter, keine referenzierten Dateien, keine API-Keys oder externen Dienste. Im A/B-Review eines synthetischen PR (neuer Export + IPC-Channel + Config-Flag, alle ohne Consumer) fanden beide Arme unabhängig voneinander den tragenden Befund — die neue Snapshot-Funktion exfiltriert intern-only-Flags über eine dokumentierte Sicherheitsgrenze hinweg — und kamen zu identischen Remove/Remove/Defer-Entscheidungen. Der Skill-Arm ergänzte seine charakteristische Demand-Normalisierung (alle drei Oberflächen drücken eine vorzeitige künftige Anforderung aus → konsolidieren) sowie ein explizites Gate in Schritt 7, das die Implementierungsqualitäts-Review zurückhält, bis die Oberflächen Bestand haben; ein moderater Priorisierungsvorteil gegenüber einer bereits starken Baseline statt eines transformativen.
Getestet am: 2026-07-11 · Claude Code 2.x (agent harness)
Installation
git clone https://github.com/CherryHQ/cherry-studio cd cherry-studio mkdir -p ~/.claude/skills cp -r .agents/skills/demand-first-review ~/.claude/skills/demand-first-review
Befehle & Beispiel-Prompts
/demand-first-reviewMarkiert spekulative oder ungenutzte neue API-Oberfläche, bevor die Implementierungsqualität geprüft wird.
Skills reagieren auf normale Anfragen — keine Slash-Befehle nötig. Nach der Installation aktivieren Prompts wie diese den Skill (auf Englisch):
Review this new endpoint for unnecessary speculative fieldsCheck if this new IPC channel has real consumersIs this config option actually needed right now