bmad-fh 6.0.0-alpha.23 → 6.0.0-alpha.23.3b00cb36

package/tools/cli/installers/lib/modules/manager.js

@@ -416,7 +416,9 @@ class ModuleManager {
       if (needsDependencyInstall || wasNewClone || nodeModulesMissing) {
         const installSpinner = ora(`Installing dependencies for ${moduleInfo.name}...`).start();
         try {
-          execSync('npm install --production --no-audit --no-fund --prefer-offline --no-progress', {
+          // Remove lockfile first - it may reference devDeps that don't exist
+          execSync('rm -f package-lock.json', { cwd: moduleCacheDir, stdio: 'pipe' });
+          execSync('npm install --omit=dev --ignore-scripts --no-package-lock --no-audit --no-fund --prefer-offline --no-progress', {
             cwd: moduleCacheDir,
             stdio: 'pipe',
             timeout: 120_000, // 2 minute timeout
@@ -441,7 +443,9 @@ class ModuleManager {
         if (packageJsonNewer) {
           const installSpinner = ora(`Installing dependencies for ${moduleInfo.name}...`).start();
           try {
-            execSync('npm install --production --no-audit --no-fund --prefer-offline --no-progress', {
+            // Remove lockfile first - it may reference devDeps that don't exist
+            execSync('rm -f package-lock.json', { cwd: moduleCacheDir, stdio: 'pipe' });
+            execSync('npm install --omit=dev --ignore-scripts --no-package-lock --no-audit --no-fund --prefer-offline --no-progress', {
               cwd: moduleCacheDir,
               stdio: 'pipe',
               timeout: 120_000, // 2 minute timeout

package/tools/cli/scripts/migrate-workflows.js

@@ -1,13 +1,12 @@
-
 /**
  * Workflow Migration Script
- * 
+ *
  * Updates workflow.yaml files to support the multi-scope system.
  * Primarily updates test_dir and other path variables to use scope-aware paths.
- * 
+ *
  * Usage:
  *   node migrate-workflows.js [--dry-run] [--verbose]
- * 
+ *
  * Options:
  *   --dry-run   Show what would be changed without making changes
  *   --verbose   Show detailed output
@@ -28,48 +27,42 @@ const PATH_MIGRATIONS = [
   {
     pattern: /\{output_folder\}\/tests/g,
     replacement: '{scope_tests}',
-    description: 'test directory to scope_tests'
+    description: 'test directory to scope_tests',
   },
   {
     pattern: /\{config_source:implementation_artifacts\}\/tests/g,
     replacement: '{config_source:scope_tests}',
-    description: 'implementation_artifacts tests to scope_tests'
+    description: 'implementation_artifacts tests to scope_tests',
   },
   // Planning artifacts
   {
     pattern: /\{output_folder\}\/planning-artifacts/g,
     replacement: '{config_source:planning_artifacts}',
-    description: 'output_folder planning to config_source'
+    description: 'output_folder planning to config_source',
   },
-  // Implementation artifacts  
+  // Implementation artifacts
   {
     pattern: /\{output_folder\}\/implementation-artifacts/g,
     replacement: '{config_source:implementation_artifacts}',
-    description: 'output_folder implementation to config_source'
-  }
+    description: 'output_folder implementation to config_source',
+  },
 ];
 
 // Variables that indicate scope requirement
-const SCOPE_INDICATORS = [
-  '{scope}',
-  '{scope_path}',
-  '{scope_tests}',
-  '{scope_planning}',
-  '{scope_implementation}'
-];
+const SCOPE_INDICATORS = ['{scope}', '{scope_path}', '{scope_tests}', '{scope_planning}', '{scope_implementation}'];
 
 /**
  * Find all workflow.yaml files
  */
 async function findWorkflowFiles(basePath) {
   const files = [];
-  
+
   async function walk(dir) {
     const entries = await fs.readdir(dir, { withFileTypes: true });
-    
+
     for (const entry of entries) {
       const fullPath = path.join(dir, entry.name);
-      
+
       if (entry.isDirectory()) {
         // Skip node_modules and hidden directories
         if (!entry.name.startsWith('.') && entry.name !== 'node_modules') {
@@ -80,7 +73,7 @@ async function findWorkflowFiles(basePath) {
       }
     }
   }
-  
+
   await walk(basePath);
   return files;
 }
@@ -89,7 +82,7 @@ async function findWorkflowFiles(basePath) {
  * Check if a workflow already uses scope variables
  */
 function usesScope(content) {
-  return SCOPE_INDICATORS.some(indicator => content.includes(indicator));
+  return SCOPE_INDICATORS.some((indicator) => content.includes(indicator));
 }
 
 /**
@@ -101,20 +94,20 @@ function analyzeWorkflow(content, filePath) {
     needsMigration: false,
     alreadyScoped: false,
     suggestions: [],
-    currentVariables: []
+    currentVariables: [],
   };
-  
+
   // Check if already uses scope
   if (usesScope(content)) {
     analysis.alreadyScoped = true;
     return analysis;
   }
-  
+
   // Find variables that might need migration
   const variablePattern = /\{[^}]+\}/g;
   const matches = content.match(variablePattern) || [];
   analysis.currentVariables = [...new Set(matches)];
-  
+
   // Check each migration pattern
   for (const migration of PATH_MIGRATIONS) {
     if (migration.pattern.test(content)) {
@@ -122,21 +115,21 @@ function analyzeWorkflow(content, filePath) {
       analysis.suggestions.push({
         description: migration.description,
         pattern: migration.pattern.toString(),
-        replacement: migration.replacement
+        replacement: migration.replacement,
       });
     }
   }
-  
+
   // Check for test_dir variable
   if (content.includes('test_dir:') || content.includes('test_dir:')) {
     analysis.needsMigration = true;
     analysis.suggestions.push({
       description: 'Has test_dir variable - may need scope_tests',
       pattern: 'test_dir',
-      replacement: 'scope_tests via config_source'
+      replacement: 'scope_tests via config_source',
     });
   }
-  
+
   return analysis;
 }
 
@@ -146,14 +139,14 @@ function analyzeWorkflow(content, filePath) {
 function migrateWorkflow(content) {
   let migrated = content;
   let changes = [];
-  
+
   for (const migration of PATH_MIGRATIONS) {
     if (migration.pattern.test(migrated)) {
       migrated = migrated.replace(migration.pattern, migration.replacement);
       changes.push(migration.description);
     }
   }
-  
+
   return { content: migrated, changes };
 }
 
@@ -163,12 +156,12 @@ function migrateWorkflow(content) {
 function addScopeMarker(content) {
   try {
     const parsed = yaml.parse(content);
-    
+
     // Add scope_required if not present
     if (!parsed.scope_required) {
       parsed.scope_required = false; // Default to false for backward compatibility
     }
-    
+
     return yaml.stringify(parsed, { lineWidth: 120 });
   } catch {
     // If YAML parsing fails, return original
@@ -183,37 +176,37 @@ async function main() {
   const args = new Set(process.argv.slice(2));
   const dryRun = args.has('--dry-run');
   const verbose = args.has('--verbose');
-  
+
   console.log(chalk.bold('\nWorkflow Migration Script'));
   console.log(chalk.dim('Updating workflow.yaml files for multi-scope support\n'));
-  
+
   if (dryRun) {
     console.log(chalk.yellow('DRY RUN MODE - No changes will be made\n'));
   }
-  
+
   // Find all workflow files
   console.log(chalk.blue('Scanning for workflow.yaml files...'));
   const files = await findWorkflowFiles(SRC_PATH);
   console.log(chalk.green(`Found ${files.length} workflow.yaml files\n`));
-  
+
   // Analysis results
   const results = {
     analyzed: 0,
     alreadyScoped: 0,
     migrated: 0,
     noChanges: 0,
-    errors: []
+    errors: [],
   };
-  
+
   // Process each file
   for (const filePath of files) {
     const relativePath = path.relative(SRC_PATH, filePath);
     results.analyzed++;
-    
+
     try {
       const content = await fs.readFile(filePath, 'utf8');
       const analysis = analyzeWorkflow(content, filePath);
-      
+
       if (analysis.alreadyScoped) {
         results.alreadyScoped++;
         if (verbose) {
@@ -221,7 +214,7 @@ async function main() {
         }
         continue;
       }
-      
+
       if (!analysis.needsMigration) {
         results.noChanges++;
         if (verbose) {
@@ -229,16 +222,16 @@ async function main() {
         }
         continue;
       }
-      
+
       // Apply migration
       const { content: migrated, changes } = migrateWorkflow(content);
-      
+
       if (changes.length > 0) {
         console.log(chalk.cyan(`  ● ${relativePath}`));
         for (const change of changes) {
           console.log(chalk.dim(`      → ${change}`));
         }
-        
+
         if (!dryRun) {
           await fs.writeFile(filePath, migrated, 'utf8');
         }
@@ -246,13 +239,12 @@ async function main() {
       } else {
         results.noChanges++;
       }
-      
     } catch (error) {
       results.errors.push({ file: relativePath, error: error.message });
       console.log(chalk.red(`  ✗ ${relativePath} - Error: ${error.message}`));
     }
   }
-  
+
   // Print summary
   console.log(chalk.bold('\n─────────────────────────────────────'));
   console.log(chalk.bold('Summary'));
@@ -265,17 +257,17 @@ async function main() {
     console.log(chalk.red(`  Errors:             ${results.errors.length}`));
   }
   console.log();
-  
+
   if (dryRun && results.migrated > 0) {
     console.log(chalk.yellow('Run without --dry-run to apply changes\n'));
   }
-  
+
   // Exit with error code if there were errors
   process.exit(results.errors.length > 0 ? 1 : 0);
 }
 
 // Run
-main().catch(error => {
+main().catch((error) => {
   console.error(chalk.red('Fatal error:'), error.message);
   process.exit(1);
 });

package/.github/workflows/publish-multi-artifact.yaml

@@ -1,50 +0,0 @@
-name: Publish Multi-Artifact
-
-on:
-  push:
-    branches:
-      - feat/multi-artifact-support
-
-permissions:
-  contents: read
-  packages: write
-
-jobs:
-  publish:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version-file: ".nvmrc"
-          cache: npm
-          registry-url: https://registry.npmjs.org
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Get version
-        id: version
-        run: |
-          echo "version=$(node -p "require('./package.json').version")" >> $GITHUB_OUTPUT
-
-      - name: Publish to NPM
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-        run: |
-          echo "Publishing bmad-fh@${{ steps.version.outputs.version }} with tag multi-artifact"
-          npm publish --tag multi-artifact
-
-      - name: Summary
-        run: |
-          echo "## Published bmad-fh@${{ steps.version.outputs.version }}" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo "### Installation" >> $GITHUB_STEP_SUMMARY
-          echo "\`\`\`bash" >> $GITHUB_STEP_SUMMARY
-          echo "npx bmad-fh@multi-artifact install" >> $GITHUB_STEP_SUMMARY
-          echo "\`\`\`" >> $GITHUB_STEP_SUMMARY