bmad-code-review

Run adversarial multi-layer code review on branch diffs, pull requests, and commit ranges.

4|1|Updated Sep 30, 2024
One-click install
npx skills add https://github.com/co-cddo/ndx --skill bmad-code-review-co-cddo
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: bmad-code-review
Source: https://github.com/co-cddo/ndx/tree/main/_bmad/bmm/workflows/4-implementation/bmad-code-review
Command: npx skills add https://github.com/co-cddo/ndx --skill bmad-code-review-co-cddo

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill enables adversarial, multi-layer code review of changes to surface issues, ensure alignment with intent, and produce actionable findings.

Core Features & Use Cases

  • Parallel review layers: Blind Hunter, Edge Case Hunter, and Acceptance Auditor operate on diffs to surface diverse issues.
  • Structured triage: Findings are normalized, deduplicated, classified into patch, defer, reject, or other categories, and presented for decision.
  • Context aware: Supports reviewing branch diffs, PRs, and specific commit ranges, with optional spec and context when provided.
  • Usage scenarios: Use when evaluating risky changes, security-sensitive patches, or complex feature rollouts to reduce risk before merging.

Quick Start

Provide the code diff you want reviewed and say 'run code review'.

Frequently Asked Questions about bmad-code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I perform an adversarial code review on a pull request?

Adversarial code review on a pull request is performed by running parallel review layers—Blind Hunter, Edge Case Hunter, and Acceptance Auditor—on the diff to surface issues. You provide the code diff and trigger the review process.

What is the best way to find edge cases in a branch diff before merging?

To find edge cases in a branch diff, use a multi-layer code review approach that includes an Edge Case Hunter. It analyzes the changes alongside integration concerns to surface potential boundary failures before merging.

Can I check if my code changes align with acceptance criteria automatically?

Yes, you can check code changes against acceptance criteria using an Acceptance Auditor layer. It evaluates the provided diff and optional specifications to ensure the implementation aligns with the intended requirements.

How are code review findings triaged after analyzing a commit range?

Code review findings from a commit range are triaged by normalizing and deduplicating the results. They are then classified into categories like patch, defer, or reject to present actionable decisions.

Does this multi-layer code review work without providing a specification?

Yes, this code review works without a specification by focusing on the provided diff. However, supplying optional specs and context enhances the Acceptance Auditor layer's ability to verify alignment with intent.

When should I use an adversarial review for my code changes?

Adversarial review should be used when evaluating risky changes, security-sensitive patches, or complex feature rollouts. It reduces risk by orchestrating parallel review layers to surface hidden issues before merging.