feat(framework): add review checklist consumed by the reviewer agent#253
feat(framework): add review checklist consumed by the reviewer agent#253alexsoyes wants to merge 1 commit into
Conversation
alexsoyes
force-pushed
the
feat/review-checklist
branch
4 times, most recently
from
63a6cf2 to
794bd29
Compare
Add aidd_docs/review-checklist.md (review-only baseline, not in memory/). The reviewer agent loads it on code/diff reviews, alongside the validator; 02-project-init scaffolds it idempotently so downstream projects get the baseline. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: alexsoyes <contact.alexsoyes@gmail.com>
alexsoyes
force-pushed
the
feat/review-checklist
branch
from
794bd29 to
2e7a67d
Compare
|
@blafourcade je trouve ça over-engineered de dingue pour si peu. Tu en dis quoi ? |
@alexsoyes On ajoute un nouveau document encore à maintenir, pourquoi tu as voulu ajouter ça exactement ? Tu parles de default qui reviennent mais j'ai du mal à comprendre le cas de base ? |
|
@blafourcade En gros souvent on doit balancer des :
J'aimerais que ça se fasse en auto, surtout pour gérer la création de skill ou les doublons l'IA adore :) |
| @@ -0,0 +1,11 @@ | |||
| # Review Checklist | |||
There was a problem hiding this comment.
Ce fichier semble identique à aidd_docs/review-checklist.md.
On a pas moyen d'en avoir qu'un seul?
S'il faut vraiment physiquement les deux on a pas moyen de scaffolder à partir d'une source plutôt que d'avoir 2 sources?
There was a problem hiding this comment.
En fait c'est le Skill actuel de Alex qui le met dans le aidd_docs justement ! @jdm-web
Et VS le mettre dans le Prompt de l'Agent ? Trop de contenu qui risque de dilluer le context ? |
3 participants
🎯 What & why
Lors des reviews, les mêmes défauts reviennent (duplication d'info, incohérences, over-engineering). On ajoute une checklist personnalisable que l'agent reviewer applique automatiquement sur chaque review de code, avec un socle par défaut.
🛠️ How it works
aidd_docs/review-checklist.md(review-only, pas dansmemory/qui est auto-injecté partout) : socle de règles transverses + bloc## Project-specific ruleséditable.quality_scoredes validators YAML (specs). Les actions01/02restent inchangées — c'est l'agent qui porte la checklist.aidd-context:02-project-init(02-scaffold-docs.md) avec étape idempotente (préserve les règles projet au re-run), sinon le reviewer pointerait vers un fichier inexistant.🧪 How to verify
test -f aidd_docs/review-checklist.md && echo okaidd-dev:05-review(01-review-code) sur un diff contenant une duplication / de l'over-engineering → le reviewer remonte ces points enfindings.L'arbre de structure dans
aidd_docs/README.mdne liste pas (encore)review-checklist.md— laissé hors périmètre.✅ I certify
🤖 Generated with Claude Code