Skip to content

chore: remove SM agent (Bob) and migrate to Developer agent#2186

Merged
alexeyv merged 3 commits intomainfrom
chore/remove-sm-agent
Apr 2, 2026
Merged

chore: remove SM agent (Bob) and migrate to Developer agent#2186
alexeyv merged 3 commits intomainfrom
chore/remove-sm-agent

Conversation

@alexeyv
Copy link
Copy Markdown
Collaborator

@alexeyv alexeyv commented Apr 2, 2026

Summary

  • Delete the Bob (bmad-agent-sm) agent wrapper — SKILL.md and manifest
  • Add sprint planning, story creation, and retrospective capabilities (SP, CS, ER) to Amelia (bmad-agent-dev)
  • Update agent tables, tutorials, and workflow docs across EN and ZH-CN
  • Remove bmad-agent-sm from marketplace.json
  • Clean up Bob/Scrum Master references in retrospective, sprint-planning, sprint-status, correct-course, party-mode, READMEs, style guides, and upgrade docs

Follows the same pattern as the Quinn removal in #2179.

Test plan

  • grep -ri 'bmad-agent-sm' src/ docs/ website/ .claude-plugin/ | grep -v CHANGELOG → zero matches
  • grep -ri bob src/ docs/ .claude-plugin/ --include='*.md' --include='*.yaml' --include='*.json' | grep -v CHANGELOG | grep -v customize-bmad | grep -vi "l'éponge" | grep -vi sponge → zero matches
  • npm run quality passes (204 tests, 0 errors)
  • Dev agent menu shows SP/CS/ER capabilities after reinstall

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Apr 2, 2026

🤖 Augment PR Summary

Summary: Removes the Scrum Master agent (Bob) and consolidates sprint/ceremony workflows under the Developer agent (Amelia).

Changes:

  • Removed bmad-agent-sm (SKILL + agent manifest) and delisted it from .claude-plugin/marketplace.json.
  • Extended bmad-agent-dev to include Sprint Planning (SP), Create Story (CS), and Epic Retrospective (ER) capabilities.
  • Updated EN + ZH-CN docs/tutorials/READMEs to replace SM/Bob references with Developer/Amelia equivalents.
  • Reworded several implementation workflows/templates (sprint-planning, sprint-status, retrospective, correct-course) to reflect Developer facilitation.
  • Cleaned up remaining Scrum Master terminology in checklists and workflow guidance.

Technical Notes: Follows the earlier “agent removal” consolidation pattern and aims to keep sprint lifecycle workflows discoverable through a single implementation persona.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 2, 2026

📝 Walkthrough

Walkthrough

This PR removes the Scrum Master agent (bmad-agent-sm) and redistributes its workflow capabilities—sprint planning, story creation, and retrospectives—to the Developer agent. The change involves deleting agent manifest files, updating workflow role assignments, and revising documentation across English and Chinese locales.

Changes

Cohort / File(s) Summary
Scrum Master Agent Removal
.claude-plugin/marketplace.json, src/bmm-skills/4-implementation/bmad-agent-sm/*
Removed bmad-agent-sm skill entry from marketplace plugin and deleted the full agent definition files (SKILL.md and bmad-skill-manifest.yaml).
Developer Agent Capability Expansion
src/bmm-skills/4-implementation/bmad-agent-dev/SKILL.md
Added three new capabilities to the Developer agent: bmad-sprint-planning (SP), bmad-create-story (CS), and bmad-retrospective (ER).
Workflow Role Reassignment
src/bmm-skills/4-implementation/bmad-sprint-planning/..., src/bmm-skills/4-implementation/bmad-retrospective/workflow.md, src/bmm-skills/4-implementation/bmad-correct-course/..., src/bmm-skills/4-implementation/bmad-sprint-status/workflow.md
Updated workflow role descriptions and persona assignments from Scrum Master to Developer; adjusted agent coordination references for scope changes and sprint recommendations.
Skill Step Documentation Updates
src/bmm-skills/3-solutioning/bmad-check-implementation-readiness/steps/step-*.md, src/bmm-skills/3-solutioning/bmad-check-implementation-readiness/workflow.md
Removed "Scrum Master" role references from prompt instructions, leaving "Product Manager" as the retained role reinforcement.
Core Skill Party-Mode Update
src/core-skills/bmad-party-mode/SKILL.md
Updated example follow-up to invoke Amelia (Developer) instead of Bob (Scrum Master) in party-mode interactions.
English Documentation Updates
README.md, docs/_STYLE_GUIDE.md, docs/how-to/upgrade-to-v6.md, docs/reference/agents.md, docs/reference/commands.md, docs/tutorials/getting-started.md
Removed Scrum Master references from feature descriptions, agent tables, and workflow tutorials; reassigned workflows from SM to Developer agent in upgrade and getting-started guides.
Chinese Documentation Updates
README_CN.md, docs/zh-cn/_STYLE_GUIDE.md, docs/zh-cn/how-to/upgrade-to-v6.md, docs/zh-cn/reference/agents.md, docs/zh-cn/reference/commands.md, docs/zh-cn/tutorials/getting-started.md
Corresponding updates to Chinese documentation: removed Scrum Master role from feature lists, agent reference tables, and tutorials; updated workflow assignment to Developer agent.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • pbean
  • cecil-the-coder
  • muratkeremozcan
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main objective: removing the Scrum Master agent and migrating its capabilities to the Developer agent.
Description check ✅ Passed The description clearly outlines the changes made, including deletion of the SM agent, migration of capabilities to the Developer agent, documentation updates, and comprehensive test verification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-sm-agent

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

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

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 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-check-implementation-readiness/workflow.md`:
- Line 5: Replace the incorrect phrase "users product vision" in the workflow
prompt with the possessive form "user's product vision" to correct the grammar;
locate the sentence that begins "Your Role: You are an expert Product
Manager..." and update that exact fragment to read "user's product vision".
🪄 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: 6b04eb76-7dd3-440b-bc95-58c6d4c7350b

📥 Commits

Reviewing files that changed from the base of the PR and between 48c2324 and 08f1ba9.

📒 Files selected for processing (27)
  • .claude-plugin/marketplace.json
  • README.md
  • README_CN.md
  • docs/_STYLE_GUIDE.md
  • docs/how-to/upgrade-to-v6.md
  • docs/reference/agents.md
  • docs/reference/commands.md
  • docs/tutorials/getting-started.md
  • docs/zh-cn/_STYLE_GUIDE.md
  • docs/zh-cn/how-to/upgrade-to-v6.md
  • docs/zh-cn/reference/agents.md
  • docs/zh-cn/reference/commands.md
  • docs/zh-cn/tutorials/getting-started.md
  • src/bmm-skills/3-solutioning/bmad-check-implementation-readiness/steps/step-01-document-discovery.md
  • src/bmm-skills/3-solutioning/bmad-check-implementation-readiness/steps/step-02-prd-analysis.md
  • src/bmm-skills/3-solutioning/bmad-check-implementation-readiness/steps/step-03-epic-coverage-validation.md
  • src/bmm-skills/3-solutioning/bmad-check-implementation-readiness/workflow.md
  • src/bmm-skills/4-implementation/bmad-agent-dev/SKILL.md
  • src/bmm-skills/4-implementation/bmad-agent-sm/SKILL.md
  • src/bmm-skills/4-implementation/bmad-agent-sm/bmad-skill-manifest.yaml
  • src/bmm-skills/4-implementation/bmad-correct-course/checklist.md
  • src/bmm-skills/4-implementation/bmad-correct-course/workflow.md
  • src/bmm-skills/4-implementation/bmad-retrospective/workflow.md
  • src/bmm-skills/4-implementation/bmad-sprint-planning/sprint-status-template.yaml
  • src/bmm-skills/4-implementation/bmad-sprint-planning/workflow.md
  • src/bmm-skills/4-implementation/bmad-sprint-status/workflow.md
  • src/core-skills/bmad-party-mode/SKILL.md
💤 Files with no reviewable changes (5)
  • docs/reference/commands.md
  • .claude-plugin/marketplace.json
  • docs/zh-cn/reference/commands.md
  • src/bmm-skills/4-implementation/bmad-agent-sm/bmad-skill-manifest.yaml
  • src/bmm-skills/4-implementation/bmad-agent-sm/SKILL.md

Standardize Developer agent references to bmad-agent-dev (matching
installed skill directory name) and fix possessive apostrophe in
implementation-readiness workflow.
@alexeyv
Copy link
Copy Markdown
Collaborator Author

alexeyv commented Apr 2, 2026

Triage Summary

4 findings — FIX: 2, DISMISS: 2, DEFER: 0

# Severity Finding Decision
F1 HIGH Retrospective time-estimate constraint violation DISMISS — pre-existing, not introduced by this PR
F2 MEDIUM IR workflow hardcodes PM role, conflicts with Architect DISMISS — by design, not a defect
F3 MEDIUM bmad-agent-dev vs bmad-dev naming inconsistency FIX — standardized to bmad-agent-dev in reference docs
F4 LOW Grammar: "users product vision" → "user's product vision" FIX — corrected possessive apostrophe

Fixed in 8281bf5.

No longer a multi-agent development team — just one Developer agent.
Remove residual Scrum Master search patterns from retrospective.
@alexeyv alexeyv merged commit 003c979 into main Apr 2, 2026
5 checks passed
@alexeyv alexeyv deleted the chore/remove-sm-agent branch April 2, 2026 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant