Repository navigation
feat(framework): review checklist consumed by the reviewer agent #291
Description
The same review defects recur: duplicated information, inconsistencies, over-engineering. A reviewer should not have to remember them.
Scope
aidd_docs/review-checklist.md: a default set of cross-cutting rules, plus an editable## Project-specific rulesblock.- The reviewer agent loads it whenever the artifact under review is code or a diff.
- Review-only: it does not live in
memory/, which is auto-injected everywhere.
Acceptance criteria
- The reviewer agent applies the checklist without being asked.
- A project can add its own rules without editing plugin source.
- The checklist is not injected into unrelated conversations.
Implemented in PR #253.
Activity
blafourcade commentedon Jul 10, 2026
Reopening: PR #253 is still open. The issue was closed on 2026-07-09 while its implementation had not landed.
blafourcade commentedon Jul 13, 2026
Done, landed differently.
plugins/aidd-dev/agents/checker.mdalready applies a review checklist to every code or diff, and already extends it with the project's own:A project supplies its checklist wherever it likes; the checker extends the baseline with it. No frozen
review-checklist.mdpath is required. The reviewer agent this issue targeted (via PR #253) was replaced by the checker in the executor/checker redesign. Closing.