@hyperdrive.bot/bmad-workflow 1.0.12 → 1.0.14

package/dist/services/orchestration/workflow-orchestrator.d.ts

@@ -390,10 +390,11 @@ export declare class WorkflowOrchestrator {
     /**
      * Execute QA phase
      *
-     * Runs QA workflow on all stories in the QA folder.
+     * Runs QA workflow on stories matching the workflow prefix.
      * Dynamically imports and delegates to StoriesQaCommand.
      *
      * @param config - Workflow configuration
+     * @param detection - Input detection result for prefix resolution
      * @returns PhaseResult with success count, failures, and duration
      * @private
      */
@@ -402,6 +403,7 @@ export declare class WorkflowOrchestrator {
      * Execute QA phase if needed
      *
      * @param config - Workflow configuration
+     * @param detection - Input detection result for prefix resolution
      * @param devPhase - Dev phase result
      * @param shouldExecute - Whether to execute QA phase
      * @returns QA phase result

package/dist/services/orchestration/workflow-orchestrator.js

@@ -112,7 +112,7 @@ export class WorkflowOrchestrator {
             return this.buildFailureResult(startTime, epicPhase);
         }
         const { devPhase, storyPhase } = await this.executeStoryAndDevPhases(config, detection, epicPhase, phaseFlags, startTime);
-        const qaPhase = await this.executeQaPhaseIfNeeded(config, devPhase, phaseFlags.shouldExecuteQaPhase);
+        const qaPhase = await this.executeQaPhaseIfNeeded(config, detection, devPhase, phaseFlags.shouldExecuteQaPhase);
         return this.buildSuccessResult(startTime, epicPhase, storyPhase, devPhase, qaPhase);
     }
     /**
@@ -1126,14 +1126,15 @@ Write output to: ${outputPath}`;
     /**
      * Execute QA phase
      *
-     * Runs QA workflow on all stories in the QA folder.
+     * Runs QA workflow on stories matching the workflow prefix.
      * Dynamically imports and delegates to StoriesQaCommand.
      *
      * @param config - Workflow configuration
+     * @param detection - Input detection result for prefix resolution
      * @returns PhaseResult with success count, failures, and duration
      * @private
      */
-    async executeQaPhase(config) {
+    async executeQaPhase(config, detection) {
         const startTime = Date.now();
         const failures = [];
         let successCount = 0;
@@ -1143,8 +1144,9 @@ Write output to: ${outputPath}`;
         try {
             // Get QA story directory
             const qaStoryDir = await this.pathResolver.getQaStoryDir();
-            // Find all stories in QA folder
-            const storyPattern = '*.md';
+            // Find stories matching the workflow prefix
+            const prefix = this.resolvePrefix(config, detection);
+            const storyPattern = `${prefix}-story-*.md`;
             const storyFiles = await this.fileManager.listFiles(qaStoryDir, storyPattern);
             if (storyFiles.length === 0) {
                 this.logger.info('No stories found in QA folder, skipping QA phase');
@@ -1159,9 +1161,9 @@ Write output to: ${outputPath}`;
             this.logger.info({ storyCount: storyFiles.length }, 'Found stories for QA phase');
             // Dynamically import QA command to avoid circular dependencies
             const { default: StoriesQaCommand } = await import('../../commands/stories/qa.js');
-            // Process each story through QA
+            // Process each story through QA (storyFile is already an absolute path from listFiles)
             for (const storyFile of storyFiles) {
-                const storyPath = `${qaStoryDir}/${storyFile}`;
+                const storyPath = storyFile;
                 this.logger.info({ storyPath }, 'Running QA workflow for story');
                 try {
                     // Build args for QA command
@@ -1229,12 +1231,13 @@ Write output to: ${outputPath}`;
      * Execute QA phase if needed
      *
      * @param config - Workflow configuration
+     * @param detection - Input detection result for prefix resolution
      * @param devPhase - Dev phase result
      * @param shouldExecute - Whether to execute QA phase
      * @returns QA phase result
      * @private
      */
-    async executeQaPhaseIfNeeded(config, devPhase, shouldExecute) {
+    async executeQaPhaseIfNeeded(config, detection, devPhase, shouldExecute) {
         if (!shouldExecute || !devPhase || devPhase.success === 0) {
             return this.createSkippedPhaseResult('qa');
         }
@@ -1242,7 +1245,7 @@ Write output to: ${outputPath}`;
             this.logger.info('[DRY RUN] Would execute QA phase');
             return this.createSkippedPhaseResult('qa');
         }
-        return this.executeQaPhase(config);
+        return this.executeQaPhase(config, detection);
     }
     /**
      * Execute dev phase in sequential mode
@@ -1373,9 +1376,15 @@ Write output to: ${outputPath}`;
                     this.logger.warn({ epicNumber: epic.number }, 'Epic file not found, skipping');
                     return [];
                 }
-                const epicContent = await this.fileManager.readFile(epicFilePath);
-                const stories = this.epicParser.parseStories(epicContent, epicFilePath);
-                return stories;
+                try {
+                    const epicContent = await this.fileManager.readFile(epicFilePath);
+                    const stories = this.epicParser.parseStories(epicContent, epicFilePath);
+                    return stories;
+                }
+                catch (error) {
+                    this.logger.warn({ epicNumber: epic.number, error: error.message }, 'Failed to parse stories from epic, skipping');
+                    return [];
+                }
             }));
             // Flatten the array of story arrays
             allStories.push(...epicStories.flat());

package/dist/services/parsers/epic-parser.js

@@ -137,13 +137,10 @@ export class EpicParser {
                 this.logger.debug({ matches: matches.length, patternName: pattern.name }, 'Pattern matched stories');
             }
         }
-        // Check if any stories were found
+        // If no stories found, return empty array (epic may legitimately have no stories)
         if (allMatches.length === 0) {
-            this.logger.error({ epicPath }, 'No stories found in epic');
-            throw new ParserError('No stories found in epic. Expected format: `### Story 1.1: Title` or similar', {
-                filePath: epicPath,
-                suggestion: 'Ensure epic has story headers like: ### Story 1.1: Initialize Project',
-            });
+            this.logger.warn({ epicPath }, 'No stories found in epic, returning empty array');
+            return [];
         }
         // Sort by position (natural document order)
         allMatches.sort((a, b) => a.position - b.position);

package/dist/services/parsers/prd-parser.d.ts

@@ -53,6 +53,14 @@ export declare class PrdParser {
      *
      * CRITICAL: Pattern order matches prd-tmpl.yaml output format
      * Template generates: ## Epic 1 Title (space-separated)
+     *
+     * Patterns support both H2 (##) and H3 (###) epic headers.
+     * PRDs may use H3 for epics when H2 is used for document sections
+     * (e.g., ## 6. Epic Details → ### Epic 1: Title).
+     *
+     * Generic patterns (N: and N.) are only used as fallback when no
+     * explicit "Epic N" patterns match — they can false-positive on
+     * numbered document sections like "## 1. Goals".
      */
     private readonly patterns;
     /**

package/dist/services/parsers/prd-parser.js

@@ -32,12 +32,32 @@ export class PrdParser {
      *
      * CRITICAL: Pattern order matches prd-tmpl.yaml output format
      * Template generates: ## Epic 1 Title (space-separated)
+     *
+     * Patterns support both H2 (##) and H3 (###) epic headers.
+     * PRDs may use H3 for epics when H2 is used for document sections
+     * (e.g., ## 6. Epic Details → ### Epic 1: Title).
+     *
+     * Generic patterns (N: and N.) are only used as fallback when no
+     * explicit "Epic N" patterns match — they can false-positive on
+     * numbered document sections like "## 1. Goals".
      */
     patterns = [
         {
             name: 'Epic N Nested (N. Epic M:)',
             regex: /^##\s+\d+\.\s+Epic\s+(\d+)\s*:\s*(.+?)$/gim,
         },
+        {
+            name: 'H3 Epic N: Title',
+            regex: /^###\s+Epic\s+(\d+)\s*:\s*(.+?)$/gim,
+        },
+        {
+            name: 'H3 Epic N - Title',
+            regex: /^###\s+Epic\s+(\d+)\s+-\s+(.+?)$/gim,
+        },
+        {
+            name: 'H3 Epic N Title (space)',
+            regex: /^###\s+Epic\s+(\d+)\s+(?![:-])(.+?)$/gim,
+        },
         {
             name: 'Epic N: Title',
             regex: /^##\s+Epic\s+(\d+)\s*:\s*(.+?)$/gim,
@@ -106,16 +126,27 @@ export class PrdParser {
         // Collect matches from all patterns - patterns are ordered by specificity
         // More specific "Epic N" patterns come first in the array
         const allMatches = [];
+        let hasExplicitEpicMatches = false;
         for (const pattern of this.patterns) {
             // Skip "N. Title" pattern if nested format found (N. is used for sections)
             if (hasNestedEpicFormat && pattern.name === 'N. Title') {
                 this.logger.debug({ patternName: pattern.name }, 'Skipping N. pattern - nested Epic format found in PRD');
                 continue;
             }
+            // Skip generic number-only patterns when explicit "Epic N" patterns already matched
+            // Generic patterns like "## 1. Goals" false-positive on document section headers
+            if (hasExplicitEpicMatches && (pattern.name === 'N: Title' || pattern.name === 'N. Title')) {
+                this.logger.debug({ patternName: pattern.name }, 'Skipping generic pattern - explicit Epic patterns already matched');
+                continue;
+            }
             const matches = this.matchPattern(pattern, lines, prdContent);
             if (matches.length > 0) {
                 allMatches.push(...matches);
                 this.logger.debug({ matches: matches.length, patternName: pattern.name }, 'Pattern matched epics');
+                // Track if we found explicit "Epic N" patterns (any pattern with "Epic" in the name)
+                if (pattern.name.includes('Epic')) {
+                    hasExplicitEpicMatches = true;
+                }
             }
         }
         // Check if any epics were found
@@ -212,12 +243,8 @@ export class PrdParser {
             const position = this.findLineNumber(lines, fullMatch);
             // Extract fullNumber based on pattern type
             let fullNumber;
-            if (pattern.name.includes('Nested')) {
-                // For nested format "## N. Epic M:", use "Epic M"
-                fullNumber = `Epic ${epicNumber}`;
-            }
-            else if (pattern.name.startsWith('Epic N')) {
-                // For Epic patterns, use "Epic N"
+            if (pattern.name.includes('Nested') || pattern.name.includes('Epic N') || pattern.name.includes('H3 Epic')) {
+                // For explicit Epic patterns (H2, H3, nested), use "Epic N"
                 fullNumber = `Epic ${epicNumber}`;
             }
             else {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@hyperdrive.bot/bmad-workflow",
   "description": "AI-driven development workflow orchestration CLI for BMAD projects",
-  "version": "1.0.12",
+  "version": "1.0.14",
   "author": {
     "name": "DevSquad",
     "email": "marcelo@devsquad.email",