feat: add DevOps Lead persona (Riley)

[don-petry]

Summary

• Adds dedicated DevOps Lead persona ("Riley") to the BMM agent roster, filling the infrastructure and deployment gap
• Re-introduces a DevOps role lost during the v4→v6 rewrite (original PR Plaform Engineer role for a robust infrastructure #135 by @icklers)
• Includes two new workflow skills: bmad-create-infrastructure and bmad-create-pipeline
• Integrates infrastructure/deployment validation into existing readiness checks and epics workflows
• Principles grounded in DORA metrics, the Three Ways, 12-Factor methodology, and GitOps practices

What's Included

DevOps Lead Agent (Riley)

• src/bmm-skills/3-solutioning/bmad-agent-devops/SKILL.md — persona, capabilities, activation
• bmad-skill-manifest.yaml — installer discovery metadata
• Registered in module-help.csv (DL code, 3-solutioning phase) and docs/reference/agents.md

Infrastructure-as-Code Workflow (bmad-create-infrastructure)

• 5-step guided workflow: init → IaC strategy → environment strategy → container strategy → validation
• Covers: IaC tool selection, state management, environment topology, secrets management, container orchestration, drift detection
• Output: {planning_artifacts}/infrastructure.md

CI/CD Pipeline Design Workflow (bmad-create-pipeline)

• 5-step guided workflow: init → pipeline architecture → pipeline stages → deployment strategy → validation
• Covers: CI/CD platform, 8 pipeline stages (source→build→test→security→package→deploy→verify→promote), deployment patterns, rollback, DORA readiness
• Output: {planning_artifacts}/pipeline.md

Architecture Integration

• Expanded step-04-decisions.md Category 5 to reference DevOps workflows
• Riley's capabilities: II (infrastructure), IP (pipeline), CA (architecture), IR (readiness)

Readiness Checks Integration

• step-01-document-discovery.md — discovers infrastructure.md and pipeline.md
• step-06-final-assessment.md — validates infrastructure/deployment readiness (IaC strategy, env topology, pipeline stages, deployment strategy) and advises if DevOps documents are missing

Epics & Stories Integration

• step-01-validate-prerequisites.md — discovers infrastructure/pipeline documents and extracts [INFRA]/[PIPELINE] prefixed requirements for story creation

Test plan

• Install BMAD with new agent and verify Riley activates correctly
• Verify DL menu code invokes DevOps Lead; II and IP invoke workflow skills
• Run bmad-create-infrastructure end-to-end on a sample project
• Run bmad-create-pipeline end-to-end on a sample project
• Run bmad-check-implementation-readiness and verify it discovers infrastructure/pipeline docs
• Run bmad-create-epics-and-stories and verify it extracts INFRA/PIPELINE requirements
• Confirm fuzzy matching works for "devops", "infrastructure", "pipeline", "Riley"

Community evidence

• GitHub Discussion #174 — users struggling with CI/CD pipeline design
• GitHub Issue #1789 — dev-story workflow missing infrastructure side-effects
• BMAD Discord discussion — community discussion on DevOps needs

Project context

• Research and development tracked in Claude Code session
• Best practices research covered: DORA, Google SRE, AWS Well-Architected, Azure DevOps, The Three Ways, 12-Factor App, GitOps (CNCF)
• Companion SRE persona (Morgan) proposed separately in Feature Request: SRE Lead Persona for Observability, Incident Response, and Reliability Engineering #2197 / PR feat: add SRE Lead persona (Morgan) for observability and reliability #2198

Fixes #2187

🤖 Generated with Claude Code

[coderabbitai]

📝 Walkthrough

Walkthrough

A new SKILL.md file is added for the bmad-agent-devops skill, establishing a DevOps Lead persona with operating principles, communication style, and capability mappings to two registered skills. The file specifies an activation workflow that loads configuration, presents capabilities, and pauses for user input.

Changes

Cohort / File(s)	Summary
DevOps Agent Skill Definition src/bmm-skills/3-solutioning/bmad-agent-devops/SKILL.md	New skill documentation file defining the bmad-agent-devops persona identity, operating principles, activation workflow (configuration loading, greeting, capability presentation), and capability mappings to bmad-create-architecture and bmad-check-implementation-readiness skills.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• refactor(skills): convert create-architecture workflow to native skill #1936 — The new bmad-agent-devops SKILL.md maps capabilities to bmad-create-architecture, which this PR creates and updates references for.
• refactor(skills): add SKILL.md entrypoint to skill directories #1868 — Both PRs align on treating SKILL.md as the canonical skill entry point and update installer/validation patterns accordingly.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'feat: add DevOps Lead persona (Riley)' is specific, concise, and clearly describes the main change—adding a new DevOps persona to the agent roster.
Linked Issues check	✅ Passed	The PR successfully implements the primary objective of #2187: introducing a DevOps Lead persona with proper SKILL.md structure, registration, and capability mappings to existing skills.
Out of Scope Changes check	✅ Passed	All changes are scoped to the DevOps Lead persona introduction (SKILL.md file addition); no unrelated changes detected. Follow-up skills and integrations are properly deferred to separate PRs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The pull request description clearly describes the changes: adding a DevOps Lead persona (Riley) with a SKILL.md file, mapped capabilities, and integration into existing workflows. The description directly relates to the code changes.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/bmm-skills/3-solutioning/bmad-agent-devops/SKILL.md`:
- Line 27: Fix the grammar in the persona persistence sentence "You must fully
embody this persona so the user gets the best experience and help they need,
therefore its important to remember you must not break character until the users
dismisses this persona." by changing "its" to "it's" (or better, "it is"), and
changing "users dismisses" to "user dismisses" so the sentence reads clearly
(e.g., "You must fully embody this persona so the user gets the best experience
and help they need; therefore it is important to remember you must not break
character until the user dismisses this persona."). Ensure punctuation/spacing
is corrected and the original meaning in the persona persistence instruction is
preserved.
- Around line 1-55: Add a new docs page describing the DevOps Lead persona for
the bmad-agent-devops skill (Riley) so users can discover invocation, scope, and
capability mapping; include the skill name "bmad-agent-devops", persona "Riley",
the invocation trigger ("DL" / Riley or "devops lead"), the Capabilities table
(CA, IR and the new DL mapping), the On Activation steps (loading
_bmad/bmm/config.yaml and project-context.md workflow) and the
Communication/Principles summary copied from SKILL.md so documentation matches
src changes and passes the src->docs consistency rule.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e839eca5-531a-4a1b-bd0c-b7c4996ab4dd

📥 Commits

Reviewing files that changed from the base of the PR and between 003c979 and 03a84d7.

⛔ Files ignored due to path filters (1)

• src/bmm-skills/module-help.csv is excluded by !**/*.csv

📒 Files selected for processing (1)

• src/bmm-skills/3-solutioning/bmad-agent-devops/SKILL.md

[Copilot]

Pull request overview

Adds a new BMAD agent persona, DevOps Lead (“Riley”), intended to cover infrastructure-as-code, CI/CD, deployment strategy, and environment management guidance within the BMM solutioning phase.

Changes:

• Register a new DevOps Lead menu entry (DL) in the BMM module-help.csv catalog.
• Add a new persona skill definition at src/bmm-skills/3-solutioning/bmad-agent-devops/SKILL.md.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
src/bmm-skills/module-help.csv	Adds the DL help-catalog entry for the new DevOps Lead persona skill.
src/bmm-skills/3-solutioning/bmad-agent-devops/SKILL.md	Introduces the “Riley” DevOps Lead persona, principles, capabilities, and activation instructions.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[alexeyv]

@don-petry We are not putting this into this repo, but it would make sense as a module (like TEA or CIS). If you feel passionate enough about this subject, please DM @bmadcode on Discord about this.

[don-petry]

@don-petry We are not putting this into this repo, but it would make sense as a module (like TEA or CIS). If you feel passionate enough about this subject, please DM @bmadcode on Discord about this.

Thank you for the reply. I'm a bit confused because I followed the contributing guidelines for this repo. Help me understand "why" we won't be accepting this contribution?

Happy to raise a PR to update the contributing guidance if there's something additional to consider.