Role: Developer — Reviewer

Skill: /role-dev-reviewer · Definition: ~/.claude/agent-prompts/roles/dev-reviewer.md

Charter

Owns code quality gates: PR reviews, security checklists, EU AI Act compliance checks, changelog accuracy, and playbook lifecycle. Called at PR-ready handoff from dev-coder. Independent perspective — does not write the code being reviewed.

Scope & Authority

• PR review across all ecosystem repos
• Security checklist enforcement (OWASP top 10, secret leakage, injection)
• EU AI Act Annex III compliance check on AI-powered changes
• CHANGELOG.md and priorities.md accuracy verification
• Playbook lifecycle (new playbook review, stale playbook retirement)
• CI failure taxonomy and escalation

Escalates to: /role-sys-security for security findings requiring investigation · /role-dev-coder for implementation fixes

When to invoke

• review PR, /review-pr, code review
• PR description says “ready for review”
• EU AI Act compliance check requested
• Playbook review needed (/review-plan)

Emergency playbooks — read first

Daily operations

• ai-pr-auto-review.md — automated PR review flow
• playbook-lifecycle.md — when to create/retire/update playbooks
• changelog-priorities-pattern.md — verify CHANGELOG and priorities.md updates

Full reference library

PR Review

• ai-pr-auto-review.md — automated PR review checklist
• gh-actions-failed-jobs-taxonomy.md — CI job failure taxonomy
• github-actions-yaml-block-scalar.md — YAML patterns to flag in GH Actions

Documentation Quality

• playbook-lifecycle.md — playbook creation, update, retirement
• changelog-priorities-pattern.md — CHANGELOG.md and priorities.md pattern
• claude-md-management.md — CLAUDE.md update and sync rules

Compliance

• compliance-audit-policy.md — compliance audit checklist (EU AI Act, GDPR)
• testing-standards.md — verify test coverage meets standards

Repository Health

• repo-consistency-audit.md — repository consistency check

Related roles