Demand-First Review

Označuje spekulativní nebo nevyužitý nový API povrch ještě před hodnocením kvality implementace.

od CherryHQ · CherryHQ/cherry-studio

Otestováno · Funguje ★ 8.0/10

Demand-First Review — Označuje spekulativní nebo nevyužitý nový API povrch ještě před hodnocením kvality implementace.

Co umí

Použijte při review PR, API, IPC kanálu, endpointu, parametru, typu nebo configu, který přidává nový povrch — PŘED komentováním kvality implementace. Použijte také okamžitě, jakmile review odhalí signály jako „zatím žádní konzumenti“, „nevyužitý export“, „spekulativní“, „přídavné“, „zpětně kompatibilní“ nebo „pro budoucí použití“.

Testovací report

Doslovná instalace (git clone + cp) proběhla bez jediného zásahu: jeden platný SKILL.md, striktní YAML frontmatter, žádné odkazované soubory, žádné API klíče ani externí služby. V A/B review syntetického PR (nový export + IPC kanál + config flag, všechny bez jediného konzumenta) oba běhy nezávisle odhalily klíčový nález — nová snapshot funkce exfiltruje interní flagy přes zdokumentovanou bezpečnostní hranici — a dospěly ke stejným rozhodnutím odstranit/odstranit/odložit. Běh se skillem navíc přidal svou typickou normalizaci poptávky (všechny tři povrchy vyjadřují jednu předčasnou budoucí potřebu → sloučit) a explicitní bránu v kroku 7, která zadržuje review kvality implementace, dokud povrchy „nepřežijí“ — jde spíš o skromnou výhodu v prioritizaci nad již silným baseline než o zásadní zlom.

Testováno: 2026-07-11 · Claude Code 2.x (agent harness)

Instalace

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

Příkazy a ukázkové prompty

  • /demand-first-reviewOznačuje spekulativní nebo nevyužitý nový API povrch ještě před hodnocením kvality implementace.

Skilly se spouštějí běžnými požadavky — žádné příkazy k zapamatování. Po instalaci ho aktivují prompty jako tyto (anglicky):

  • 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