Demand-First Review

Flaggar spekulativ eller oanvänd ny API-yta innan implementationskvaliteten granskas.

av CherryHQ · CherryHQ/cherry-studio

Testad · Fungerar ★ 8.0/10

Demand-First Review — Flaggar spekulativ eller oanvänd ny API-yta innan implementationskvaliteten granskas.

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 fields
  • Check if this new IPC channel has real consumers
  • Is this config option actually needed right now