refactor: update story naming to match epic:story scheme from PRD and fix typos#7
Merged
bmadcode merged 1 commit intobmad-code-org:mainfrom Apr 25, 2025
Conversation
Contributor
alchemydc
commented
Apr 21, 2025
alchemydc
commented
- Change story file naming from -.story.md to story-{epicnumber.storynumber}.md
- Update example story paths to reflect new naming pattern
- Clarify story completion requirements for test verification
- Fix minor typos and improve document formatting
- Update Product Owner document title
- Change story file naming from <story>-<n>.story.md to story-{epicnumber.storynumber}.md
- Update example story paths to reflect new naming pattern
- Clarify story completion requirements for test verification
- Fix minor typos and improve document formatting
- Update Product Owner document title
Collaborator
|
Thank you for the contribution! |
Contributor
|
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
alvin-chang
pushed a commit
to alvin-chang/BMAD-METHOD
that referenced
this pull request
Sep 15, 2025
…ry_numbers_to_PRD refactor: update story naming to match epic:story scheme from PRD and fix typos
alexeyv
referenced
this pull request
in alexeyv/BMAD-METHOD
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
3 tasks
rotationalphysics495
added a commit
to rotationalphysics495/BMAD-METHOD
that referenced
this pull request
Jan 26, 2026
Implements the final two improvements from bmad_improvements_v2.md: ## Structured JSON Output (Improvement bmad-code-org#6) - New module: scripts/epic-execute-lib/json-output.sh - Functions for extracting and parsing JSON from Claude output - Unified check_phase_completion() with JSON + text fallback - Updated prompts to request JSON result blocks - Added --legacy-output flag to disable JSON parsing ## Test-First Flow (Improvement bmad-code-org#7) - New module: scripts/epic-execute-lib/tdd-flow.sh - execute_test_spec_phase() - Generates BDD specs from acceptance criteria - execute_test_impl_phase() - Creates failing tests from specs - execute_test_verification_phase() - Verifies tests fail correctly - Integration with dev phase for TDD context - Added --skip-tdd, --skip-test-spec, --skip-test-impl flags All 7 improvements from the analysis are now complete. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
bmadcode
pushed a commit
that referenced
this pull request
Feb 11, 2026
…ns (#1608) * fix: remove output_folder/story_dir aliases, flatten variables sections (#1602) Drop pointless alias variables (output_folder, story_dir, story_directory) from Phase 4, Quick Flow, and QA workflows. Replace all references with the canonical {implementation_artifacts} or {planning_artifacts} variables. Also flatten unnecessary `variables:` YAML nesting in all affected workflow.yaml files — the workflow engine treats all keys as top-level, so the nesting added complexity with no semantic value. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: add missing config declarations, remove remaining aliases and dead variables - Add missing document_output_language and user_skill_level to create-story (referenced in instructions.xml but never declared) - Remove retrospectives_folder alias, replace with canonical implementation_artifacts - Remove unused sprint_status and duplicate validation alias from correct-course - Remove unused date, planning_artifacts, tracking_system from sprint-status Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: address PR review findings and rename shadowed output_folder variable - Fix single/double brace mismatch for {implementation_artifacts} in create-story instructions.xml (F1) - Remove escaped asterisks in glob patterns in retrospective instructions.md (F2) - Eliminate redundant {config_source} re-resolution for story_location in sprint-planning workflow.yaml (F5) - Add explicit instruction to discover previous_story_num by scanning artifacts instead of leaving it undefined (#7) - Rename output_folder to project_knowledge in document-project workflows to stop shadowing the canonical core config variable (#13) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: use plural retrospectives in previous retro search instructions The glob pattern can match multiple retrospective files for the same epic (e.g., partial mid-sprint retro and full completion retro). Use plural "retrospectives" to make clear the LLM should load all matches. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Feb 23, 2026
MartenAngner
added a commit
to whiteport-collective/whiteport-design-studio
that referenced
this pull request
Feb 24, 2026
Workshop fixes (issues bmad-code-org#4-bmad-code-org#12): - Add output-discipline block to compiler activation (bmad-code-org#12) - Rename synthesize steps to create-document (bmad-code-org#5) - Add context carry-forward to prevent repetitive questions (bmad-code-org#4) - Add length constraints to scenario outlines (bmad-code-org#10) - Add completeness gate for dream mode trigger map (bmad-code-org#6) - Add data extraction + cross-validation to hub generation (bmad-code-org#7) - Extend dream-up architecture for Phase 3 scenarios (bmad-code-org#8) - Add dream/suggest mode-conditional rules to all 15 UX steps (bmad-code-org#11) Stale reference cleanup: - Remove orphaned Mimir agent guides and presentations - Update Mimir references to Saga in templates and step files - Rename UX Scenario Architect to Facilitator across all scenario files - Fix strategic-documentation.md step 12 heading Workflow audit fixes: - Fix broken explore section routing links in alignment workflow - Fix FILE-INDEX.md wrong parent path in agentic-development - Fix routing tables in design-system and product-evolution workflows - Clarify dream mode routing notation in ux-design workflow - Remove empty steps-d/ directory Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Feb 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.