Demand-First Review
Flaggar spekulativ eller oanvänd ny API-yta innan implementationskvaliteten granskas.
Testad · Fungerar
Vad den gör
Används vid granskning av en PR, ett API, en IPC-kanal, en endpoint, en parameter, en typ eller en konfiguration som lägger till ny yta — INNAN implementationskvaliteten kommenteras. Används också omedelbart när en granskning ger signaler som 'inga konsumenter än', 'oanvänd export', 'spekulativt', 'additivt', 'bakåtkompatibelt' eller 'för framtida bruk'.
Testrapport
Installation ord för ord (git clone + cp) fungerade utan kirurgi: en giltig SKILL.md, strikt YAML-frontmatter, inga refererade filer, inga API-nycklar eller externa tjänster. I en A/B-granskning av en syntetisk PR (ny export + IPC-kanal + konfigurationsflagga, alla utan konsumenter) fångade båda grenarna oberoende av varandra det bärande fyndet — den nya snapshot-funktionen läcker interna flaggor över en dokumenterad säkerhetsgräns — och landade i identiska beslut om att ta bort/ta bort/skjuta upp. Skillens gren lade till sin karakteristiska demand-normalisering (alla tre ytor uttrycker ett och samma prematura framtida behov → slå ihop) och en uttrycklig grind vid steg 7 som håller tillbaka implementationskvalitetsgranskningen tills ytorna överlever prövningen; en måttlig prioriteringsfördel över en redan stark baslinje snarare än ett genombrott.
Testad: 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
Kommandon och exempelprompter
/demand-first-reviewFlaggar spekulativ eller oanvänd ny API-yta innan implementationskvaliteten granskas.
Skills triggas av vanliga förfrågningar — inga kommandon att memorera. Efter installationen aktiverar prompter som dessa skillen (på engelska):
Review this new endpoint for unnecessary speculative fieldsCheck if this new IPC channel has real consumersIs this config option actually needed right now