Demand-First Review
Segnala superficie API nuova, speculativa o inutilizzata prima di rivedere la qualità dell'implementazione.
Promosso
Cosa fa
Da usare quando si revisiona una PR, un'API, un canale IPC, un endpoint, un parametro, un tipo o una config che aggiunge nuova superficie — PRIMA di commentare la qualità dell'implementazione. Da usare anche subito quando una review fa emergere segnali come "no consumers yet", "unused export", "speculative", "additive", "forward-compatible" o "for future use".
Rapporto di test
Installazione verbatim (git clone + cp) senza alcun intervento: un solo SKILL.md valido, frontmatter YAML rigoroso, nessun file referenziato, nessuna chiave API né servizio esterno. In una review A/B di una PR sintetica (nuovo export + canale IPC + flag di config, tutti senza consumatori) entrambi i bracci hanno individuato indipendentemente il risultato portante — la nuova funzione di snapshot esfiltra flag interni attraverso un confine di sicurezza documentato — arrivando a decisioni identiche di rimozione/rimozione/rinvio. Il braccio con lo skill ha aggiunto la sua caratteristica normalizzazione della domanda (le tre superfici esprimono un'unica domanda futura prematura → da consolidare) e un gate esplicito al passo 7 che trattiene la review di qualità dell'implementazione finché le superfici non superano l'esame; un vantaggio di prioritizzazione modesto su una baseline già solida, non un guadagno trasformativo.
Testato il: 2026-07-11 · Claude Code 2.x (agent harness)
Installazione
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
Comandi e prompt di esempio
/demand-first-reviewSegnala superficie API nuova, speculativa o inutilizzata prima di rivedere la qualità dell'implementazione.
Gli skill si attivano con richieste in linguaggio naturale, senza comandi da ricordare. Dopo l'installazione, prompt come questi lo attivano (in inglese):
Review this new endpoint for unnecessary speculative fieldsCheck if this new IPC channel has real consumersIs this config option actually needed right now