Demand-First Review
Markerer spekulativ eller ubrugt ny API-overflade, før implementeringskvaliteten gennemgås.
Testet · Virker
Hvad det gør
Brug når du gennemgår en PR, API, IPC-kanal, endpoint, parameter, type eller config, der tilføjer nyt overfladeareal — FØR du kommenterer på implementeringskvaliteten. Brug også straks, når en gennemgang afslører signaler som "ingen forbrugere endnu", "ubrugt export", "spekulativ", "additiv", "fremadkompatibel" eller "til fremtidig brug".
Testrapport
Verbatim install (git clone + cp) virkede uden nogen redning: én gyldig SKILL.md, streng YAML-frontmatter, ingen refererede filer, ingen API-nøgler eller eksterne services. I en A/B-gennemgang af en syntetisk PR (ny export + IPC-kanal + config-flag, alle uden forbrugere) fangede begge arme uafhængigt af hinanden det bærende fund — den nye snapshot-funktion lækker interne flags på tværs af en dokumenteret sikkerhedsgrænse — og nåede identiske fjern/fjern/udskyd-beslutninger. Skill-armen tilføjede sin karakteristiske demand-normalisering (alle tre overflader udtrykker ét for tidligt fremtidigt behov → konsolider) og en eksplicit step-7-gate, der tilbageholder implementeringskvalitetsgennemgangen, indtil overfladerne overlever; en beskeden prioriteringsfordel frem for en allerede stærk baseline snarere end en transformerende en.
Testet: 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
Kommandoer og eksempelprompter
/demand-first-reviewMarkerer spekulativ eller ubrugt ny API-overflade, før implementeringskvaliteten gennemgås.
Skills udløses af almindelige forespørgsler — ingen kommandoer at huske. Efter installationen aktiverer prompter som disse skillen (på engelsk):
Review this new endpoint for unnecessary speculative fieldsCheck if this new IPC channel has real consumersIs this config option actually needed right now