Demand-First Review

Flagger spekulativ eller ubrukt ny API-overflate før den vurderer implementasjonskvalitet.

Bestått ★ 8.0/10

Hva den gjør

Brukes når du gjennomgår en PR, API, IPC-kanal, endepunkt, parameter, type eller config som legger til ny overflate — FØR du kommenterer implementasjonskvalitet. Brukes også umiddelbart når en gjennomgang avdekker signaler som «no consumers yet», «unused export», «speculative», «additive», «forward-compatible», eller «for future use».

Testrapport

Verbatim-installasjon (git clone + cp) fungerte uten redning: én gyldig SKILL.md, streng YAML-frontmatter, ingen refererte filer, ingen API-nøkler eller eksterne tjenester. I en A/B-gjennomgang av en syntetisk PR (ny export + IPC-kanal + config-flagg, alle uten forbrukere) fanget begge armene uavhengig av hverandre det bærende funnet — den nye snapshot-funksjonen eksfiltrerer interne flagg på tvers av en dokumentert sikkerhetsgrense — og kom fram til identiske fjern/fjern/utsett-beslutninger. Skillarmen la til sin signatur-demand-normalisering (alle tre overflatene uttrykker ett prematurt fremtidig behov → konsolider) og en eksplisitt steg-7-sperre som holder tilbake implementasjonskvalitets-gjennomgangen til overflatene består; et beskjedent prioriteringsfortrinn over en allerede sterk baseline snarere enn et transformerende ett.

Testet på: 2026-07-11 · Claude Code 2.x (agent harness)

Installer

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-reviewFlagger spekulativ eller ubrukt ny API-overflate før den vurderer implementasjonskvalitet.

Skills utløses av vanlige forespørsler — ingen kommandoer å huske. Etter installasjonen aktiverer prompter som disse skillen (på engelsk):

Review this new endpoint for unnecessary speculative fields
Check if this new IPC channel has real consumers
Is this config option actually needed right now

Full rapport →