Demand-First Review
Wyłapuje spekulacyjną lub nieużywaną nową powierzchnię API, zanim ocenisz jakość implementacji.
Testowano · Działa
Co robi ten skill
Używaj przy przeglądzie PR, API, kanału IPC, endpointu, parametru, typu lub konfiguracji, które dodają nową powierzchnię — ZANIM skomentujesz jakość implementacji. Używaj też natychmiast, gdy przegląd ujawni sygnały typu „brak konsumentów jeszcze”, „nieużywany eksport”, „spekulacyjne”, „addytywne”, „kompatybilne wstecznie na przyszłość” albo „do przyszłego użytku”.
Raport z testu
Dosłowny install (git clone + cp) zadziałał bez żadnej ingerencji: jeden poprawny SKILL.md, ścisły YAML we frontmatterze, brak plików referencyjnych, brak kluczy API czy usług zewnętrznych. W przeglądzie A/B syntetycznego PR (nowy eksport + kanał IPC + flaga konfiguracyjna, wszystkie bez konsumentów) oba warianty niezależnie wyłapały kluczowe znalezisko — nowa funkcja snapshot wynosi na zewnątrz flagi przeznaczone tylko do użytku wewnętrznego, przekraczając udokumentowaną granicę bezpieczeństwa — i doszły do identycznych decyzji usuń/usuń/odłóż. Wariant ze skillem dodał swoją charakterystyczną normalizację popytu (wszystkie trzy powierzchnie wyrażają jedno przedwczesne przyszłe zapotrzebowanie → skonsoliduj) oraz jawną bramkę na kroku 7, która wstrzymuje przegląd jakości implementacji, dopóki powierzchnie nie przetrwają — skromna przewaga w priorytetyzacji nad już mocnym baseline'em, a nie przełomowa różnica.
Testowano: 2026-07-11 · Claude Code 2.x (agent harness)
Instalacja
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
Komendy i przykładowe prompty
/demand-first-reviewWyłapuje spekulacyjną lub nieużywaną nową powierzchnię API, zanim ocenisz jakość implementacji.
Skille uruchamiają się na zwykłe polecenia — bez komend do zapamiętania. Po instalacji aktywują go prompty takie jak te (po angielsku):
Review this new endpoint for unnecessary speculative fieldsCheck if this new IPC channel has real consumersIs this config option actually needed right now