Get the FREE Ultimate OpenClaw Setup Guide →

bmad-developer

Scanned
npx machina-cli add skill xmm/codex-bmad-skills/bmad-developer --openclaw
Files (1)
SKILL.md
5.5 KB

BMAD Developer

Trigger Intents

  • bmad:dev-story
  • bmad:code-review

Workflow Variants

  1. dev-story
  • Implement one story to completion with tests.
  1. code-review
  • Review implementation quality against story acceptance criteria.

Inputs

  • target story file docs/stories/STORY-*.md
  • architecture constraints from docs/bmad/architecture.md when present
  • repository coding and testing standards

Language Guard (Mandatory)

Enforce language selection separately for chat responses and generated artifacts.

Chat language (communication_language) fallback order:

  1. language.communication_language from bmad/project.yaml
  2. English

Rules for chat responses:

  • Use the resolved chat language for all assistant responses (questions, status updates, summaries, and handoff notes).
  • Do not switch chat language unless the user explicitly requests a different language in the current thread.

Artifact language (document_output_language) fallback order:

  1. language.document_output_language from bmad/project.yaml
  2. English

Rules for generated artifacts:

  • Use the resolved artifact language for all generated BMAD documents and structured artifacts.
  • write prose and field values in the resolved document language
  • avoid mixed-language requirement clauses with English modal verbs (for example, System shall followed by non-English text)
  • allow English acronyms/abbreviations in non-English sentences (for example, API, SLA, KPI, OAuth, WCAG)
  • Keep code snippets, CLI commands, file paths, and identifiers in their original technical form.

Mandatory Reference Load

Before executing dev-story or code-review, read REFERENCE.md first and treat it as required context. Then load focused details from:

  • resources/clean-code-checklist.md
  • resources/testing-standards.md

Architecture Guardrails = Architecture Compliance Guardrails section (source of truth for architecture constraints).

When docs/bmad/architecture.md exists, apply Architecture Guardrails.

Output Contract

  • code and test changes in repository
  • optional review artifact docs/bmad/code-review.md
  • clear pass/fail status against acceptance criteria
  • for architecture-constrained stories: concise note on how key constraints were handled

Core Workflow

  1. Read story scope and acceptance criteria.
  2. Implement minimal complete solution.
  3. Add or update tests.
  4. Run quality checks and summarize outcomes.
  5. Report residual risks and follow-up items.
  6. If architecture constraints are clearly violated, return Needs changes; otherwise report residual risks and follow-up items.

Implementation Approach

1. Understand Story and Constraints

  • Parse acceptance criteria into explicit checks.
  • Identify touched modules, dependencies, and data contracts.
  • Load architecture constraints from docs/bmad/architecture.md when present.
  • Apply Architecture Guardrails.
  • Flag ambiguities and assumptions before implementation.

2. Plan Before Coding

  • Break work into small slices:
    • data and schema changes
    • business logic
    • interfaces and adapters
    • test updates
  • Define edge cases, failure modes, and rollback-safe behavior.

3. Implement Incrementally

  • Prefer small, reviewable changes over large rewrites.
  • Preserve existing project conventions and architecture boundaries.
  • Handle errors explicitly and keep behavior deterministic.
  • Avoid hidden scope expansion; if scope grows, record it in the summary.

4. Test to Match Risk

  • Add unit tests for new logic and edge cases.
  • Add integration tests where component boundaries are crossed.
  • Add end-to-end tests for critical user-facing flows when applicable.
  • Verify acceptance criteria with test evidence, not assumptions.

5. Run Hard Quality Gates

  • Run lint and static checks:
    bash scripts/lint-check.sh
    
  • Run pre-commit checks:
    bash scripts/pre-commit-check.sh
    
  • Run coverage validation (default 80% minimum unless project requires higher):
    bash scripts/check-coverage.sh
    
  • Do not mark complete with unresolved failing checks unless explicitly approved.

6. Close with Traceable Outcome

  • Map each acceptance criterion to code and test evidence.
  • Summarize relevant architecture constraints and where they are addressed in code/tests.
  • Summarize changed files, key decisions, and residual risks.
  • For code-review, record findings using templates/code-review.template.md.

Script Selection

  • Lint checks:
    bash scripts/lint-check.sh
    
  • Pre-commit quality checks:
    bash scripts/pre-commit-check.sh
    
  • Coverage checks:
    bash scripts/check-coverage.sh
    

Template Map

  • templates/code-review.template.md
  • Why: structured review output for quality and risk communication.

Reference Map

  • REFERENCE.md

  • Must read first for developer workflow and implementation discipline.

  • resources/clean-code-checklist.md

  • Use when reviewing maintainability and readability.

  • resources/testing-standards.md

  • Use when validating test completeness and quality.

Quality Gates

  • acceptance criteria are explicitly verified
  • architecture constraints are explicitly verified (storage, auth, idempotency when applicable)
  • tests and lint pass, or failures are reported clearly
  • no hidden scope expansion without note
  • implementation remains maintainable and reviewable

Source

git clone https://github.com/xmm/codex-bmad-skills/blob/main/skills/bmad-developer/SKILL.mdView on GitHub

Overview

BMAD developer helps implement a single BMAD story end-to-end (dev-story) or review code against story acceptance criteria (code-review). It emphasizes architectural guardrails, mandatory reference loading, and delivering code with tests that meet story specs.

How This Skill Works

The workflow reads the story scope and acceptance criteria, applies architecture guardrails from docs/bmad/architecture.md when present, and plans incremental work. It then implements the minimal complete solution, adds unit/integration tests, and runs lint and quality gates, reporting pass/fail status and residual risks. Output includes code and test changes, with an optional docs/bmad/code-review.md artifact if applicable.

When to Use It

  • Starting a new BMAD story (dev-story) to deliver code and tests from story specs
  • Reviewing an existing BMAD implementation against its acceptance criteria (code-review)
  • When architecture constraints exist and must be validated against implementation
  • Adding or updating tests to cover new logic or edge cases
  • Documenting review results and residual risks after a code review

Quick Start

  1. Step 1: Open docs/stories/STORY-*.md and extract acceptance criteria and scope
  2. Step 2: Implement the minimal complete solution with tests, adhering to architecture guardrails
  3. Step 3: Run lint, static checks, and tests; report outcomes and any residual risks

Best Practices

  • Parse acceptance criteria into explicit checks before coding
  • Respect Architecture Guardrails and load constraints from docs/bmad/architecture.md
  • Implement minimal, complete, testable changes with clear scope
  • Write unit and integration tests for new logic and edge cases
  • Capture residual risks and follow-up items after the review

Example Use Cases

  • Implement a new BMAD feature for a dev-story with corresponding unit tests and verification against acceptance criteria
  • Perform a code-review to ensure the implemented story aligns with the acceptance criteria and architecture constraints
  • Update tests and documentation when a story scope is clarified or changed during review
  • Fix a regression discovered during code-review by adding a targeted test and minimal code change
  • Produce an optional code-review artifact documenting reasoning and evidence of criteria satisfaction

Frequently Asked Questions

Add this skill to your agents
Sponsor this space

Reach thousands of developers