Skill Tracer
Skills › BMAD Method › bmad-code-review › workflow.md

workflow.md

44 lines · skills/bmad-code-review/workflow.md at bd70c64

Open on GitHub

Most marks come from matching words in the file, so treat them as pointers.

1{% if workflow.review not in ("quick", "thorough") %}
2 {{ halt("workflow.review must be quick or thorough, not " ~ workflow.review) }}
3{% endif %}
4{% if workflow.interactive is not boolean %}
5 {{ halt("workflow.interactive must be true or false, not " ~ workflow.interactive) }}
6{% endif %}
7{% if workflow.fixes not in ("apply", "handoff") %}
8 {{ halt("workflow.fixes must be apply or handoff, not " ~ workflow.fixes) }}
9{% endif %}
11# Code Review Workflow
13Several independent reviewers hunt for defects in a code change; this workflow sends them the change, then verifies, triages, presents, and fixes what they found.
15Subagents are an important part of this workflow. Use them wherever a step calls for them, if you can. If you need the user's permission to run them, ask once now for the whole run.

{% if workflow.review not in ("quick", "thorough") %} {{ halt("workflow.review must be quick or thorough, not " ~ workflow.review) }} {% endif %} {% if workflow.interactive is not boolean %} {{ halt("workflow.interactive must be true or false, not " ~ workflow.interactive) }} {% endif %} {% if workflow.fixes not in ("apply", "handoff") %} {{ halt("workflow.fixes must be apply or handoff, not " ~ workflow.fixes) }} {% endif %}

Code Review Workflow

Several independent reviewers hunt for defects in a code change; this workflow sends them the change, then verifies, triages, presents, and fixes what they found.

Subagents are an important part of this workflow. Use them wherever a step calls for them, if you can. If you need the user's permission to run them, ask once now for the whole run.

Raw lines 1 to 16
Starts an agent Subagents :15
17## Conventions
19- When a step directs you to another file, read it fully and follow it. Load one step at a time, when it is reached.
20- A step that shows a menu or checkpoint halts there and waits for the user.
21{% if workflow.fixes == "handoff" %}

Conventions

  • When a step directs you to another file, read it fully and follow it. Load one step at a time, when it is reached.
  • A step that shows a menu or checkpoint halts there and waits for the user.

{% if workflow.fixes == "handoff" %}

Raw lines 17 to 22
23## No-code review
25Review by inspection only. Do not modify code, run builds or tests, or invoke quality gates.
26{% endif %}

No-code review

Review by inspection only. Do not modify code, run builds or tests, or invoke quality gates. {% endif %}

Raw lines 23 to 27
28## On activation
30Run each of these in order before step 1 (`_None._` means skip):
32{{ workflow.activation_steps_prepend }}
34Hold every entry below as fact for the whole run. Entries prefixed `file:` are paths or globs under `{project-root}` to read; the rest are facts verbatim (`_None._` means none):
36{{ workflow.persistent_facts }}
38Then run each of these in order (`_None._` means skip):
40{{ workflow.activation_steps_append }}

On activation

Run each of these in order before step 1 (_None._ means skip):

{{ workflow.activation_steps_prepend }}

Hold every entry below as fact for the whole run. Entries prefixed file: are paths or globs under {project-root} to read; the rest are facts verbatim (_None._ means none):

{{ workflow.persistent_facts }}

Then run each of these in order (_None._ means skip):

{{ workflow.activation_steps_append }}

Raw lines 28 to 41
42## First step
44Read fully and follow `{{ rendered("step-01-gather-context.md") }}`.

First step

Read fully and follow {{ rendered("step-01-gather-context.md") }}.

Raw lines 42 to 44