@hyperdrive.bot/bmad-workflow 1.0.16 → 1.0.18

package/dist/commands/stories/qa.js

@@ -22,7 +22,6 @@ import { PathResolver } from '../../services/file-system/path-resolver.js';
 import { StoryParserFactory } from '../../services/parsers/story-parser-factory.js';
 import * as colors from '../../utils/colors.js';
 import { createLogger, generateCorrelationId } from '../../utils/logger.js';
-import { createSpinner } from '../../utils/progress.js';
 import { runAgentWithRetry } from '../../utils/retry.js';
 /**
  * Stories QA Command
@@ -127,11 +126,10 @@ export default class StoriesQaCommand extends Command {
             // Match story files using glob pattern
             const storyFiles = await this.matchStoryFiles(args.pattern);
             if (storyFiles.length === 0) {
-                this.log(colors.warning(`No story files matched pattern: ${args.pattern}`));
+                this.logger.info({ pattern: args.pattern }, 'No story files matched pattern');
                 return;
             }
-            this.log(colors.info(`Found ${storyFiles.length} story file(s) to QA`));
-            this.log('');
+            this.logger.info({ count: storyFiles.length }, 'Found story file(s) to QA');
             // Process stories sequentially
             const results = await this.qaStoriesSequentially(storyFiles, flags);
             // Display summary report
@@ -245,13 +243,8 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
      * Display countdown timer between stories
      */
     async displayCountdown(intervalSeconds) {
-        /* eslint-disable no-await-in-loop */
-        for (let remaining = intervalSeconds; remaining > 0; remaining--) {
-            process.stdout.write(`\r${colors.warning(`⏳ Next story in ${remaining}s...`)}`);
-            await this.sleep(1000);
-        }
-        /* eslint-enable no-await-in-loop */
-        process.stdout.write('\r' + ' '.repeat(40) + '\r'); // Clear countdown line
+        this.logger.info({ seconds: intervalSeconds }, 'Waiting before next story');
+        await this.sleep(intervalSeconds * 1000);
     }
     /**
      * Display summary report
@@ -264,45 +257,22 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
         const errorCount = results.filter((r) => !r.success).length;
         const movedToDone = results.filter((r) => r.movedTo === 'done').length;
         const movedBack = results.filter((r) => r.movedTo === 'stories').length;
-        // Box drawing
-        const boxTop = '┌─────────────────────────────────────────┐';
-        const boxDivider = '├─────────────────────────────────────────┤';
-        const boxBottom = '└─────────────────────────────────────────┘';
-        this.log('');
-        this.log(boxTop);
-        this.log('│         Story QA Summary                │');
-        this.log(boxDivider);
-        this.log(`│ ${colors.success('PASS:')}            ${passCount.toString().padEnd(20)}│`);
-        if (concernsCount > 0) {
-            this.log(`│ ${colors.warning('CONCERNS:')}        ${concernsCount.toString().padEnd(20)}│`);
-        }
-        if (failCount > 0) {
-            this.log(`│ ${colors.error('FAIL:')}            ${failCount.toString().padEnd(20)}│`);
-        }
-        if (waivedCount > 0) {
-            this.log(`│ WAIVED:           ${waivedCount.toString().padEnd(20)}│`);
-        }
-        if (errorCount > 0) {
-            this.log(`│ ${colors.error('Errors:')}          ${errorCount.toString().padEnd(20)}│`);
-        }
-        this.log(boxDivider);
-        this.log(`│ Moved to Done:    ${movedToDone.toString().padEnd(20)}│`);
-        this.log(`│ Returned to Dev:  ${movedBack.toString().padEnd(20)}│`);
-        this.log(boxDivider);
-        this.log(`│ Duration:         ${(duration / 1000).toFixed(2)}s${' '.repeat(15)}│`);
-        this.log(boxBottom);
-        // List details for non-PASS stories
         const nonPassStories = results.filter((r) => r.finalGate !== 'PASS' || !r.success);
-        if (nonPassStories.length > 0) {
-            this.log('');
-            this.log(colors.bold('Stories Requiring Attention:'));
-            for (const result of nonPassStories) {
-                const status = result.success ? result.finalGate : 'ERROR';
-                const statusColor = status === 'CONCERNS' ? colors.warning : colors.error;
-                this.log(statusColor(`  ${status}: ${result.storyNumber} - ${path.basename(result.storyPath)}${result.error ? ` (${result.error})` : ''}`));
-            }
-        }
-        this.log('');
+        this.logger.info({
+            concerns: concernsCount,
+            durationSec: (duration / 1000).toFixed(2),
+            errors: errorCount,
+            fail: failCount,
+            movedBack,
+            movedToDone,
+            nonPass: nonPassStories.map((r) => ({
+                error: r.error,
+                gate: r.success ? r.finalGate : 'ERROR',
+                story: r.storyNumber,
+            })),
+            pass: passCount,
+            waived: waivedCount,
+        }, 'Story QA Summary');
     }
     /**
      * Extract gate status from story content or gate file
@@ -393,19 +363,16 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
         for (let index = 0; index < storyFiles.length; index++) {
             const storyPath = storyFiles[index];
             const storyNum = index + 1;
-            this.log(colors.bold(`\n[${storyNum}/${total}] QA: ${path.basename(storyPath)}`));
+            this.logger.info({ storyNum, total, story: path.basename(storyPath) }, 'QA starting');
             // Parse story metadata
-            const spinner = createSpinner('Parsing story metadata...');
-            spinner.start();
             let storyMetadata;
             try {
                 storyMetadata = await this.storyParserFactory.parseStory(storyPath);
-                const storyId = storyMetadata.id;
-                spinner.succeed(colors.success(`Story ${storyId}: ${storyMetadata.title}`));
+                this.logger.info({ storyId: storyMetadata.id, title: storyMetadata.title }, 'Parsed story');
             }
             catch (error) {
                 const err = error;
-                spinner.fail(colors.error(`Failed to parse story: ${err.message}`));
+                this.logger.error({ error: err.message, storyPath }, 'Failed to parse story');
                 results.push({
                     error: `Parse error: ${err.message}`,
                     finalGate: 'UNKNOWN',
@@ -425,19 +392,8 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
                 reference: flags.reference,
             });
             results.push(result);
-            // Display result
-            if (result.finalGate === 'PASS') {
-                this.log(colors.success(`  ✓ PASSED - Moved to done`));
-            }
-            else if (result.finalGate === 'WAIVED') {
-                this.log(colors.warning(`  ⚠ WAIVED - Moved to done`));
-            }
-            else if (result.success) {
-                this.log(colors.warning(`  ⚠ ${result.finalGate} - Returned for rework`));
-            }
-            else {
-                this.log(colors.error(`  ✗ Error: ${result.error}`));
-            }
+            // Log result
+            this.logger.info({ gate: result.finalGate, movedTo: result.movedTo, story: path.basename(storyPath), success: result.success }, result.success ? `QA ${result.finalGate}` : `QA error: ${result.error}`);
             // Display countdown timer before next story (except for last story)
             if (index < storyFiles.length - 1) {
                 await this.displayCountdown(flags.interval);
@@ -456,7 +412,7 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
         this.logger.info({ storyNumber, storyPath }, 'Starting story QA workflow');
         try {
             // Phase 1: Initial QA Deep Dive
-            this.log(colors.info('  Phase 1: QA Deep Dive Review...'));
+            this.logger.info('Phase 1: QA Deep Dive Review');
             const agentTimeout = flags.timeout ?? 2_700_000;
             const agentRetries = flags['agent-retries'];
             const retryBackoff = flags['retry-backoff'];
@@ -475,11 +431,11 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
             // Read story to get gate status
             let storyContent = await this.fileManager.readFile(storyPath);
             currentGate = this.extractGateStatus(storyContent);
-            this.log(colors.info(`  Initial Gate: ${currentGate}`));
+            this.logger.info({ gate: currentGate }, 'Initial gate result');
             // Phase 2: Dev Fix-Forward Loop (if needed and retries allowed)
             while ((currentGate === 'CONCERNS' || currentGate === 'FAIL') && retriesUsed < flags['max-retries']) {
                 retriesUsed++;
-                this.log(colors.warning(`  Phase 2: Dev Fix-Forward (Retry ${retriesUsed}/${flags['max-retries']})...`));
+                this.logger.info({ retry: retriesUsed, maxRetries: flags['max-retries'] }, 'Phase 2: Dev Fix-Forward');
                 // Run Dev agent to fix issues (sequential retry loop by design)
                 const devPrompt = this.buildDevFixPrompt(storyPath, flags['dev-prompt'], flags.reference);
                 // eslint-disable-next-line no-await-in-loop
@@ -496,7 +452,7 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
                     // Continue anyway - QA will re-evaluate
                 }
                 // Phase 3: Re-run QA to validate fixes
-                this.log(colors.info(`  Phase 3: QA Re-validation (Retry ${retriesUsed})...`));
+                this.logger.info({ retry: retriesUsed }, 'Phase 3: QA Re-validation');
                 // eslint-disable-next-line no-await-in-loop
                 const reQaResult = await runAgentWithRetry(this.agentRunner, qaPrompt, {
                     agentType: 'tea',
@@ -513,7 +469,7 @@ ${result.finalGate === 'PASS' ? '3. [x] Final QA Validation - PASSED' : result.f
                 // eslint-disable-next-line no-await-in-loop
                 storyContent = await this.fileManager.readFile(storyPath);
                 currentGate = this.extractGateStatus(storyContent);
-                this.log(colors.info(`  Gate after retry ${retriesUsed}: ${currentGate}`));
+                this.logger.info({ gate: currentGate, retry: retriesUsed }, 'Gate after retry');
             }
             // Append QA workflow summary to story
             const result = {

package/dist/commands/workflow.d.ts

@@ -33,6 +33,7 @@ export default class Workflow extends Command {
         'prd-interval': import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
         model: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         prefix: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
+        'session-prefix': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         provider: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         qa: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'qa-prompt': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
@@ -50,12 +51,72 @@ export default class Workflow extends Command {
     private cancelled;
     private logger;
     private orchestrator;
+    private reporter;
+    private scaffolder;
+    private sessionDir;
     /**
      * Main command execution
      *
      * Orchestrates the complete workflow with progress tracking and summary display.
      */
     run(): Promise<void>;
+    /**
+     * Collect all errors from workflow result into a flat list
+     *
+     * @param result - Workflow execution result
+     * @returns Array of error entries with phase context
+     * @private
+     */
+    private collectErrors;
+    /**
+     * Build dashboard summary from workflow result for final display
+     *
+     * @param result - Workflow execution result
+     * @returns DashboardSummary for the reporter's final dashboard
+     * @private
+     */
+    private buildDashboardSummary;
+    /**
+     * Create callbacks object wired to scaffolder for incremental artifact writing
+     *
+     * All callbacks are wrapped in try-catch to prevent scaffolder errors from
+     * interrupting workflow execution (AC: #9).
+     * @returns WorkflowCallbacks object with scaffolder integration
+     * @private
+     */
+    private createScaffolderCallbacks;
+    /**
+     * Merge multiple callback objects into a single callbacks object
+     *
+     * When the same callback is defined in multiple sources, all handlers are called
+     * in sequence (fire-and-forget). This enables dual-channel output where scaffolder
+     * and reporter receive events independently.
+     *
+     * @param sources - Array of WorkflowCallbacks objects to merge
+     * @returns Merged WorkflowCallbacks object
+     * @private
+     */
+    private mergeCallbacks;
+    /**
+     * Derive session prefix from input filename
+     *
+     * Extracts meaningful prefix from input path:
+     * - PRD-feature.md → feature
+     * - PRD-my-project.md → my-project
+     * - epic-001.md → epic-001
+     *
+     * @param inputPath - Input file path
+     * @returns Derived prefix string
+     * @private
+     */
+    private derivePrefixFromFilename;
+    /**
+     * Display detailed failure information for each phase
+     *
+     * @param result - Workflow result with all phase data
+     * @private
+     */
+    private displayFailureDetails;
     /**
      * Display dry-run banner
      *
@@ -82,15 +143,35 @@ export default class Workflow extends Command {
      * @private
      */
     private displayPhaseHeader;
+    /**
+     * Finalize session by writing session report and workflow log
+     *
+     * Called on workflow completion (success or failure).
+     * Errors are caught and logged but don't interrupt workflow completion.
+     *
+     * @param result - Workflow execution result
+     * @param _config - Workflow configuration used (unused for now)
+     * @private
+     */
+    private finalizeSession;
     /**
      * Initialize services and dependencies
      *
      * Creates all service instances needed for workflow orchestration.
      * @param maxConcurrency - Maximum number of concurrent operations
      * @param provider - AI provider to use (claude or gemini)
+     * @param verbose - Enable verbose output mode
      * @private
      */
     private initializeServices;
+    /**
+     * Initialize session scaffolder and create session directory structure
+     *
+     * @param inputPath - Input file path for prefix derivation
+     * @param sessionPrefix - Optional explicit session prefix
+     * @private
+     */
+    private initializeSession;
     /**
      * Register SIGINT handler for graceful cancellation
      *