CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

bmad-code-review

Review code changes with several independent reviewers in parallel, then triage and present the findings. Use when the user says "run code review" or "review this code"

Quality

86%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

SKILL.md
Quality
Evals
Security

Quality

Content

85%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

An exemplary lean dispatcher body: one exact command, explicit placeholder resolution, conditional flags with a tie-breaker, and enumerated handling of every output outcome. The only structural weakness is that the two bundled reference files (claims-check.md, deletion-check.md) are invisible from SKILL.md, and no fully-resolved example command is given.

Suggestions

Add a brief note in the body (e.g., a short list under the command) naming references/claims-check.md and references/deletion-check.md and when the printed workflow will direct agents to them, so the bundle's structure is discoverable from the entry point.

Include one fully-resolved example invocation (with real-looking project-root and skill-root paths, plus a sample flag append) so the command is copy-paste adaptable without inference.

DimensionReasoningScore

Conciseness

The ~18-line body is lean and every token earns its place: "Run the following command exactly once without changing the current working directory", terse substitution rules, conditional flag appends, and one-line outcome handling. It assumes Claude's competence, explains nothing Claude already knows, and even compresses the judgment call to "When in doubt, don't." — a clear match for the anchor "Lean and efficient; assumes Claude's competence; every token earns its place".

5 / 5

Actionability

The body gives an exact command (`uv run --no-cache "{project-root}/_bmad/scripts/render_skill.py" ...`) with unambiguous substitution rules ("the nearest folder containing `_bmad/`, from the working directory upward") and concrete conditional appends, matching "Mostly executable guidance; concrete code or commands with minor gaps". Not a 5 because the command is templated rather than copy-paste ready — no fully-resolved example invocation is shown; not a 3 because the guidance is real and executable, not pseudocode or high-level hints.

4 / 5

Workflow Clarity

This is a single-action dispatcher skill whose one action is unambiguous (the simple-skill exception applies): substitute placeholders, conditionally append flags with an explicit tie-breaker ("When in doubt, don't"), run once, then branch on every possible outcome — "HALT: <reason>: report the reason and stop", missing script → offer setup then re-run, "On any other output or failure ... report the command output and stop. Do not run any workflow source directly." Every checkpoint and error path is explicit.

5 / 5

Progressive Disclosure

The bundle contains references/claims-check.md and references/deletion-check.md, but the body never mentions or signals them — navigation is entirely delegated to the printed workflow, so from the entry point the references are present but not clearly signaled (anchor 3). Not a 2 because the body itself is a clean, short overview that appropriately keeps detail out of SKILL.md rather than inlining it; not a 4/5 because a reader of SKILL.md cannot discover the two bundled reference files at all.

3 / 5

Total

17

/

20

Passed

Description

80%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong description that cleanly answers both what the skill does and when to use it, with concrete quoted triggers. Its main weakness is distinctiveness: the trigger terms are the generic ones any code review request would use, so it risks firing for (or being shadowed by) ordinary code-review skills, and common trigger synonyms like diff/PR/changes are missing.

Suggestions

Add natural trigger variations such as "review my diff", "review this PR", or "review my changes" to the 'Use when' clause.

Signal what makes this review different from a plain code review in the description itself (e.g., mention the multi-reviewer/BMad workflow by name) so it is distinguishable from generic or built-in review skills.

DimensionReasoningScore

Specificity

"Review code changes with several independent reviewers in parallel, then triage and present the findings" names the domain and several concrete actions (parallel independent reviewers, triage, present findings), matching the anchor "Lists several specific actions; minor gaps in coverage". Not a 5 because it does not enumerate the review's actual coverage (e.g., what the reviewers check or the quick/thorough modes the body supports); not a 3 because more than 1-2 actions are named and the pipeline end-to-end is described.

4 / 5

Completeness

Both halves are explicit: what — "Review code changes with several independent reviewers in parallel, then triage and present the findings"; when — "Use when the user says 'run code review' or 'review this code'". This matches the anchor "Clearly and explicitly answers both what AND when with concrete trigger phrases" exactly.

5 / 5

Trigger Term Quality

The quoted triggers "run code review" and "review this code" are natural phrases a user would actually say, fitting "Good keyword coverage; a few natural terms missing". Not a 5 because common variations like "review my diff", "review this PR", or "review my changes" are absent; not a 3 because the terms present are natural rather than technical jargon.

4 / 5

Distinctiveness Conflict Risk

"several independent reviewers in parallel" distinguishes the mechanism, but the trigger phrases are exactly the generic words any code-review skill (or the environment's built-in review) would claim, so it "could still overlap with similar skills". Not a 4 because nothing in the triggers or wording carves out a niche distinct from a plain code review; not a 2 because the multi-reviewer/triage framing is more specific than a broad blanket claim.

3 / 5

Total

16

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

bmad-code-org/BMAD-METHOD

Path

skills/bmad-code-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.