From 6580a0c1329a8a0d478aa1db517b1f42dd79b85c Mon Sep 17 00:00:00 2001 From: Jonah Schulte Date: Sun, 25 Jan 2026 21:43:58 -0500 Subject: [PATCH] feat: add implementation readiness check to batch-super-dev MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit **New Step 2.5: Implementation Readiness Check** Validates story quality BEFORE execution mode selection: - Checks story file exists (or BACKLOG for creation) - Validates BMAD format (12 sections required) - Checks gap analysis markers (✅/❌) - Verifies minimum tasks (≥3) - Verifies minimum ACs (≥3) - Checks Current State content (≥100 words) **Benefits:** - Catches quality issues before starting execution - User can remove/fix invalid stories - Prevents wasted time on incomplete stories - Ensures hospital-grade story quality **User Control:** - Validation failures: Option to remove and continue - Validation warnings: Option to fix or proceed - Full transparency on quality issues --- package-lock.json | 4 +- .../batch-super-dev/instructions.md | 112 ++++++++++++++++++ 2 files changed, 114 insertions(+), 2 deletions(-) diff --git a/package-lock.json b/package-lock.json index 4749f4a9..aa424d03 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,12 +1,12 @@ { "name": "@jonahschulte/bmad-method", - "version": "6.3.0-alpha.23", + "version": "6.4.0-alpha.0", "lockfileVersion": 3, "requires": true, "packages": { "": { "name": "@jonahschulte/bmad-method", - "version": "6.3.0-alpha.23", + "version": "6.4.0-alpha.0", "license": "MIT", "dependencies": { "@clack/prompts": "^0.11.0", diff --git a/src/modules/bmm/workflows/4-implementation/batch-super-dev/instructions.md b/src/modules/bmm/workflows/4-implementation/batch-super-dev/instructions.md index 2fdec9f5..b83b9f79 100644 --- a/src/modules/bmm/workflows/4-implementation/batch-super-dev/instructions.md +++ b/src/modules/bmm/workflows/4-implementation/batch-super-dev/instructions.md @@ -577,6 +577,118 @@ Only the first {{max_stories}} will be processed. + + +━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ +🔍 IMPLEMENTATION READINESS CHECK +━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ + +Validating selected stories for quality and completeness... + + + For each selected story: + + + Read story file: {{story_file_path}} + + + + ⚠️ {{story_key}}: No story file (BACKLOG) - will create during execution + Mark story as needs_creation + Continue to next story + + + + ❌ {{story_key}}: Story file MISSING but status is ready-for-dev (inconsistent state) + Add to validation_failures list + Continue to next story + + + + + Validate story completeness: + - Count sections (need 12) + - Check Current State word count (need ≥100) + - Check gap analysis markers (✅/❌) + - Count Acceptance Criteria (need ≥3) + - Count unchecked tasks (need ≥3) + + + + ❌ {{story_key}}: INSUFFICIENT TASKS ({{task_count}}/3 minimum) + Add to validation_failures: "{{story_key}}: Only {{task_count}} tasks" + + + + ⚠️ {{story_key}}: Story incomplete ({{sections_found}}/12 sections{{#if !gap_analysis}}, no gap analysis{{/if}}) + Add to validation_warnings: "{{story_key}}: Needs regeneration" + + + + ✅ {{story_key}}: Valid and ready + Add to validated_stories list + + + + + + +❌ **Validation Failures ({{validation_failures.length}}):** + +{{#each validation_failures}} + - {{this}} +{{/each}} + +These stories CANNOT be processed. Options: +1. Remove them from selection +2. Fix them manually +3. Cancel batch execution + + + Remove failed stories and continue? (yes/no): + + + Remove validation_failures from selected_stories + ✅ Removed {{validation_failures.length}} invalid stories. Continuing with {{selected_stories.length}} valid stories. + + + + ❌ Batch processing cancelled. Please fix story validation issues first. + Exit workflow + + + + + +⚠️ **Validation Warnings ({{validation_warnings.length}}):** + +{{#each validation_warnings}} + - {{this}} +{{/each}} + +These stories have quality issues but can still be processed. +Recommend regenerating with /create-story-with-gap-analysis for better quality. + + + Continue with these stories anyway? (yes/no): + + + ❌ Batch processing cancelled. + Exit workflow + + + + +━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ +✅ Implementation Readiness: PASS +━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ +**Validated:** {{validated_stories.length}} stories +**Needs Creation:** {{needs_creation.length}} stories (BACKLOG) +**Quality:** All stories meet minimum standards +━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ + + + ╔═══════════════════════════════════════════════════════════════════╗