Demand-First Review
Signale la surface d'API nouvelle, spéculative ou inutilisée avant de revoir la qualité d'implémentation.
Testé · Fonctionne
Ce que fait
À utiliser lors de la revue d'une PR, d'une API, d'un canal IPC, d'un endpoint, d'un paramètre, d'un type ou d'une config qui ajoute une nouvelle surface — AVANT de commenter la qualité d'implémentation. À utiliser aussi immédiatement quand une revue fait apparaître des signaux comme « pas encore de consommateurs », « export inutilisé », « spéculatif », « additif », « rétrocompatible » ou « pour usage futur ».
Rapport de test
Installation verbatim (git clone + cp) sans aucun sauvetage : un seul SKILL.md valide, frontmatter en strict-YAML, aucun fichier référencé, aucune clé API ni service externe. Dans une revue A/B d'une PR synthétique (nouvel export + canal IPC + flag de config, tous sans consommateur), les deux bras ont indépendamment repéré la trouvaille porteuse — la nouvelle fonction de snapshot exfiltre des flags internes à travers une frontière de sécurité documentée — et abouti à des décisions identiques de suppression/suppression/report. Le bras avec skill a ajouté sa normalisation de demande caractéristique (les trois surfaces expriment une seule demande future prématurée → à consolider) et une porte explicite d'étape 7 qui retient la revue de qualité d'implémentation tant que les surfaces ne survivent pas à l'examen ; un avantage de priorisation modeste sur une baseline déjà solide plutôt qu'un gain transformateur.
Testé le: 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
Commandes et exemples de prompts
/demand-first-reviewSignale la surface d'API nouvelle, spéculative ou inutilisée avant de revoir la qualité d'implémentation.
Les skills se déclenchent sur des demandes en langage courant — aucune commande à retenir. Après installation, des prompts comme ceux-ci l'activent (en anglais) :
Review this new endpoint for unnecessary speculative fieldsCheck if this new IPC channel has real consumersIs this config option actually needed right now