From 01927f35e649f50d0beb0254c016e8d1957db089 Mon Sep 17 00:00:00 2001
From: Alex Verkhovsky <alexey.verkhovsky@gmail.com>
Date: Sun, 8 Mar 2026 15:16:21 -0600
Subject: [PATCH] test: update fixtures to remove workflow.yaml references

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 test/fixtures/file-refs-csv/valid/bmm-style.csv     |  2 +-
 test/test-file-refs-csv.js                          |  2 +-
 tools/cli/installers/lib/core/manifest-generator.js | 11 ++++-------
 tools/cli/installers/lib/modules/manager.js         |  2 --
 4 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/test/fixtures/file-refs-csv/valid/bmm-style.csv b/test/fixtures/file-refs-csv/valid/bmm-style.csv
index ab870ab01..c803064ba 100644
--- a/test/fixtures/file-refs-csv/valid/bmm-style.csv
+++ b/test/fixtures/file-refs-csv/valid/bmm-style.csv
@@ -1,3 +1,3 @@
 module,phase,name,code,sequence,workflow-file,command,required,agent,options,description,output-location,outputs,
-bmm,anytime,Document Project,DP,,_bmad/bmm/workflows/document-project/workflow.yaml,bmad-bmm-document-project,false,analyst,Create Mode,"Analyze project",project-knowledge,*,
+bmm,anytime,Document Project,DP,,_bmad/bmm/workflows/document-project/workflow.md,bmad-bmm-document-project,false,analyst,Create Mode,"Analyze project",project-knowledge,*,
 bmm,1-analysis,Brainstorm Project,BP,10,_bmad/core/workflows/brainstorming/workflow.md,bmad-brainstorming,false,analyst,data=template.md,"Brainstorming",planning_artifacts,"session",
diff --git a/test/test-file-refs-csv.js b/test/test-file-refs-csv.js
index d068bd75d..c6a8e4623 100644
--- a/test/test-file-refs-csv.js
+++ b/test/test-file-refs-csv.js
@@ -58,7 +58,7 @@ test('bmm-style.csv: extracts workflow-file refs with trailing commas', () => {
   const { fullPath, content } = loadFixture('valid/bmm-style.csv');
   const refs = extractCsvRefs(fullPath, content);
   assert(refs.length === 2, `Expected 2 refs, got ${refs.length}`);
-  assert(refs[0].raw === '_bmad/bmm/workflows/document-project/workflow.yaml', `Wrong raw[0]: ${refs[0].raw}`);
+  assert(refs[0].raw === '_bmad/bmm/workflows/document-project/workflow.md', `Wrong raw[0]: ${refs[0].raw}`);
   assert(refs[1].raw === '_bmad/core/workflows/brainstorming/workflow.md', `Wrong raw[1]: ${refs[1].raw}`);
   assert(refs[0].type === 'project-root', `Wrong type: ${refs[0].type}`);
   assert(refs[0].line === 2, `Wrong line for row 0: ${refs[0].line}`);
diff --git a/tools/cli/installers/lib/core/manifest-generator.js b/tools/cli/installers/lib/core/manifest-generator.js
index 76dee258e..b02ccde9e 100644
--- a/tools/cli/installers/lib/core/manifest-generator.js
+++ b/tools/cli/installers/lib/core/manifest-generator.js
@@ -183,9 +183,7 @@ class ManifestGenerator {
               const content = rawContent.replaceAll('\r\n', '\n').replaceAll('\r', '\n');
 
               const frontmatterMatch = content.match(/^---\n([\s\S]*?)\n---/);
-              if (!frontmatterMatch) {
-                if (debug) console.log(`[DEBUG] collectSkills: skipped (no frontmatter): ${workflowPath}`);
-              } else {
+              if (frontmatterMatch) {
                 const workflow = yaml.parse(frontmatterMatch[1]);
 
                 if (!workflow || !workflow.name || !workflow.description) {
@@ -228,6 +226,8 @@ class ManifestGenerator {
                     console.log(`[DEBUG] collectSkills: claimed skill "${workflow.name}" as ${canonicalId} at ${dir}`);
                   }
                 }
+              } else {
+                if (debug) console.log(`[DEBUG] collectSkills: skipped (no frontmatter): ${workflowPath}`);
               }
             } catch (error) {
               if (debug) console.log(`[DEBUG] collectSkills: failed to parse ${workflowPath}: ${error.message}`);
@@ -334,10 +334,7 @@ class ManifestGenerator {
           // Recurse into subdirectories
           const newRelativePath = relativePath ? `${relativePath}/${entry.name}` : entry.name;
           await findWorkflows(fullPath, newRelativePath);
-        } else if (
-          entry.name === 'workflow.md' ||
-          (entry.name.startsWith('workflow-') && entry.name.endsWith('.md'))
-        ) {
+        } else if (entry.name === 'workflow.md' || (entry.name.startsWith('workflow-') && entry.name.endsWith('.md'))) {
           // Parse workflow file (both YAML and MD formats)
           if (debug) {
             console.log(`[DEBUG] Found workflow file: ${fullPath}`);
diff --git a/tools/cli/installers/lib/modules/manager.js b/tools/cli/installers/lib/modules/manager.js
index e405e0c58..8b97b1737 100644
--- a/tools/cli/installers/lib/modules/manager.js
+++ b/tools/cli/installers/lib/modules/manager.js
@@ -772,7 +772,6 @@ class ModuleManager {
     }
   }
 
-
   /**
    * Compile .agent.yaml files to .md format in modules
    * @param {string} sourcePath - Source module path
@@ -1124,7 +1123,6 @@ class ModuleManager {
     }
   }
 
-
   /**
    * Create directories declared in module.yaml's `directories` key
    * This replaces the security-risky module installer pattern with declarative config