Skip to content

fix(bmm): complete cleanup of epic-tech-context workflow removal#1001

Merged
bmadcode merged 2 commits intobmad-code-org:mainfrom
alexeyv:fix/phase4-epic-tech-context-cleanup
Dec 1, 2025
Merged

fix(bmm): complete cleanup of epic-tech-context workflow removal#1001
bmadcode merged 2 commits intobmad-code-org:mainfrom
alexeyv:fix/phase4-epic-tech-context-cleanup

Conversation

@alexeyv
Copy link
Copy Markdown
Collaborator

@alexeyv alexeyv commented Nov 29, 2025

Comprehensive cleanup following cd98a7f which deleted epic-tech-context and story-context workflows but left stale references throughout documentation and workflow files.

Critical Fixes

Issue #1 - Remove all epic-tech-context references:

  • Removed from 8 BMM documentation files (quick-start, faq, glossary, brownfield-guide, agents-guide, enterprise-agentic-development, workflows-solutioning, document-sharding-guide)
  • Removed 'contexted' epic status from all status flows
  • Updated Phase 4 workflow descriptions to simplified model

Issue #3 - Fix contradictory workflow guidance:

  • Aligned retrospective with sprint-planning canonical rule
  • Epic marked 'in-progress' automatically WHEN first story created
  • Not manually BEFORE creating stories

Issue #8 - Add migration for legacy 'contexted' status:

  • create-story workflow now handles legacy 'contexted' epic status
  • Automatically converts to 'in-progress' for backward compatibility
  • Prevents breaking existing sprint-status.yaml files

Major Fixes

Issue #2 - Add epic status transition documentation:

  • Added explicit transition rules to sprint-planning
  • backlog → in-progress: Automatically when first story created
  • in-progress → done: Manually when all stories complete

Issue #5 - Add epic status validation:

  • create-story now validates epic status before proceeding
  • Errors if epic is 'done' (cannot add stories to completed epic)
  • Errors if epic status is invalid/corrupted

Issue #6 - Fix code review process comments:

  • Corrected 4 misleading 'SM review' comments
  • Verified against agent definitions (dev.agent.yaml)
  • Dev runs code-review workflow (with fresh context/different LLM)

Issue #7 - Remove orphaned workflow references:

  • Removed all story-done and story-ready-for-dev references
  • Updated faq, agents-guide, atdd-checklist-template
  • Workflows deleted in cd98a7f but docs not updated

Files Changed (13 modified, 9 added)

Documentation:

  • docs/document-sharding-guide.md
  • src/modules/bmm/docs/agents-guide.md
  • src/modules/bmm/docs/brownfield-guide.md
  • src/modules/bmm/docs/enterprise-agentic-development.md
  • src/modules/bmm/docs/faq.md
  • src/modules/bmm/docs/glossary.md
  • src/modules/bmm/docs/quick-start.md
  • src/modules/bmm/docs/workflows-solutioning.md

Workflows:

  • src/modules/bmm/workflows/4-implementation/create-story/instructions.xml
  • src/modules/bmm/workflows/4-implementation/retrospective/instructions.md
  • src/modules/bmm/workflows/4-implementation/sprint-planning/instructions.md
  • src/modules/bmm/workflows/4-implementation/sprint-planning/sprint-status-template.yaml
  • src/modules/bmm/workflows/testarch/atdd/atdd-checklist-template.md

Verification

  • Grep confirms 0 epic-tech-context references in BMM scope
  • Grep confirms 0 contexted status references in BMM scope
  • All agent validation passing
  • All linting and formatting checks passing

Resolves code review issues #1, #2, #3, #5, #6, #7, #8 from commit 67f124c

@alexeyv alexeyv requested a review from bmadcode November 29, 2025 20:42
@alexeyv alexeyv self-assigned this Nov 29, 2025
@alexeyv alexeyv marked this pull request as draft November 29, 2025 20:42
- Remove references to deprecated epic-tech-context, story-context,
  validate-epic-tech-context, validate-story-context, and story-done workflows
- Simplify epic status: backlog → in-progress → done (was backlog → contexted)
- Update create-story to handle legacy 'contexted' status for backward compat
- Clean up sprint-planning instructions and status template
- Update docs: agents-guide, brownfield-guide, faq, glossary, quick-start

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@alexeyv alexeyv force-pushed the fix/phase4-epic-tech-context-cleanup branch from 305c9ac to 8d49490 Compare November 29, 2025 20:45
@alexeyv alexeyv marked this pull request as ready for review November 29, 2025 22:53
@bmadcode bmadcode merged commit fe0817f into bmad-code-org:main Dec 1, 2025
5 checks passed
@alexeyv alexeyv deleted the fix/phase4-epic-tech-context-cleanup branch December 2, 2025 12:15
thecontstruct pushed a commit to thecontstruct/BMAD-METHOD that referenced this pull request Dec 4, 2025
…d-code-org#1001)

- Remove references to deprecated epic-tech-context, story-context,
  validate-epic-tech-context, validate-story-context, and story-done workflows
- Simplify epic status: backlog → in-progress → done (was backlog → contexted)
- Update create-story to handle legacy 'contexted' status for backward compat
- Clean up sprint-planning instructions and status template
- Update docs: agents-guide, brownfield-guide, faq, glossary, quick-start

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Brian <bmadcode@gmail.com>
sanchorelaxo pushed a commit to sanchorelaxo/BMAD-METHOD that referenced this pull request Dec 7, 2025
…d-code-org#1001)

- Remove references to deprecated epic-tech-context, story-context,
  validate-epic-tech-context, validate-story-context, and story-done workflows
- Simplify epic status: backlog → in-progress → done (was backlog → contexted)
- Update create-story to handle legacy 'contexted' status for backward compat
- Clean up sprint-planning instructions and status template
- Update docs: agents-guide, brownfield-guide, faq, glossary, quick-start

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Brian <bmadcode@gmail.com>
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.

2 participants