Skip to content

Commit 7ea8630

Browse files
Snow9666chenshujian
andauthored
Fix setup skill template YAML frontmatter (#55)
Co-authored-by: chenshujian <chenshujian@solaxpower.com>
1 parent 228747c commit 7ea8630

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

skills/bmad-module-builder/assets/setup-skill-template/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
name: { setup-skill-name }
2+
name: "{setup-skill-name}"
33
description: Sets up {module-name} module in a project. Use when the user requests to 'install {module-code} module', 'configure {module-name}', or 'setup {module-name}'.
44
---
55

skills/bmad-module-builder/scripts/tests/test-scaffold-setup-skill.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,12 @@ def test_skill_md_frontmatter_substitution():
9595
assert "{module-code}" not in skill_md
9696

9797

98+
def test_template_frontmatter_uses_quoted_name_placeholder():
99+
"""Test that the template frontmatter is valid before substitution."""
100+
template_skill_md = (TEMPLATE_DIR / "SKILL.md").read_text()
101+
assert 'name: "{setup-skill-name}"' in template_skill_md
102+
103+
98104
def test_generated_files_written():
99105
"""Test that module.yaml and module-help.csv contain generated content."""
100106
with tempfile.TemporaryDirectory() as tmp:
@@ -201,6 +207,7 @@ def test_missing_source_file():
201207
tests = [
202208
test_basic_scaffold,
203209
test_skill_md_frontmatter_substitution,
210+
test_template_frontmatter_uses_quoted_name_placeholder,
204211
test_generated_files_written,
205212
test_anti_zombie_replaces_existing,
206213
test_missing_target_dir,

0 commit comments

Comments
 (0)