Skip to content

feat(framework): review checklist consumed by the reviewer agent #291

Description

@blafourcade

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 rules block.
  • 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

  1. moved this to In Progress in AIDD Roadmapon Jun 18, 2026
  2. moved this from In Progress to Done in AIDD Roadmapon Jul 9, 2026
  3. closed this as completedby moving to Done in AIDD Roadmapon Jul 9, 2026
  4. blafourcade commented on Jul 10, 2026

    @blafourcade
    ContributorAuthor

    Reopening: PR #253 is still open. The issue was closed on 2026-07-09 while its implementation had not landed.

  5. blafourcade commented on Jul 13, 2026

    @blafourcade
    ContributorAuthor

    Done, landed differently. plugins/aidd-dev/agents/checker.md already applies a review checklist to every code or diff, and already extends it with the project's own:

    Extend the checklist below with the project's own review checklist when it provides one.

    Checklist — This is the behavioral baseline. Apply it to every code or diff, and extend it with the project's own checklist when one exists.

    A project supplies its checklist wherever it likes; the checker extends the baseline with it. No frozen review-checklist.md path is required. The reviewer agent this issue targeted (via PR #253) was replaced by the checker in the executor/checker redesign. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Fields

    Priority

    High

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions