From bf74468e0ded77fddcc3dff607d6c941f8a4aa75 Mon Sep 17 00:00:00 2001 From: Brian Madison Date: Mon, 13 Apr 2026 00:58:25 -0500 Subject: [PATCH] fix: rename setup-skill-template to _setup-skill-template The BMM installer's manifest generator walks all directories looking for SKILL.md files. The template directory contains a SKILL.md with placeholder name "{setup-skill-name}" which doesn't match the directory name, producing a spurious error during installation. Prefixing with underscore makes the scanner skip it, matching the existing convention for non-scannable directories. --- .../{setup-skill-template => _setup-skill-template}/SKILL.md | 0 .../assets/module-help.csv | 0 .../assets/module.yaml | 0 .../scripts/cleanup-legacy.py | 0 .../scripts/merge-config.py | 0 .../scripts/merge-help-csv.py | 0 skills/bmad-module-builder/scripts/scaffold-setup-skill.py | 2 +- .../scripts/tests/test-scaffold-setup-skill.py | 2 +- 8 files changed, 2 insertions(+), 2 deletions(-) rename skills/bmad-module-builder/assets/{setup-skill-template => _setup-skill-template}/SKILL.md (100%) rename skills/bmad-module-builder/assets/{setup-skill-template => _setup-skill-template}/assets/module-help.csv (100%) rename skills/bmad-module-builder/assets/{setup-skill-template => _setup-skill-template}/assets/module.yaml (100%) rename skills/bmad-module-builder/assets/{setup-skill-template => _setup-skill-template}/scripts/cleanup-legacy.py (100%) rename skills/bmad-module-builder/assets/{setup-skill-template => _setup-skill-template}/scripts/merge-config.py (100%) rename skills/bmad-module-builder/assets/{setup-skill-template => _setup-skill-template}/scripts/merge-help-csv.py (100%) diff --git a/skills/bmad-module-builder/assets/setup-skill-template/SKILL.md b/skills/bmad-module-builder/assets/_setup-skill-template/SKILL.md similarity index 100% rename from skills/bmad-module-builder/assets/setup-skill-template/SKILL.md rename to skills/bmad-module-builder/assets/_setup-skill-template/SKILL.md diff --git a/skills/bmad-module-builder/assets/setup-skill-template/assets/module-help.csv b/skills/bmad-module-builder/assets/_setup-skill-template/assets/module-help.csv similarity index 100% rename from skills/bmad-module-builder/assets/setup-skill-template/assets/module-help.csv rename to skills/bmad-module-builder/assets/_setup-skill-template/assets/module-help.csv diff --git a/skills/bmad-module-builder/assets/setup-skill-template/assets/module.yaml b/skills/bmad-module-builder/assets/_setup-skill-template/assets/module.yaml similarity index 100% rename from skills/bmad-module-builder/assets/setup-skill-template/assets/module.yaml rename to skills/bmad-module-builder/assets/_setup-skill-template/assets/module.yaml diff --git a/skills/bmad-module-builder/assets/setup-skill-template/scripts/cleanup-legacy.py b/skills/bmad-module-builder/assets/_setup-skill-template/scripts/cleanup-legacy.py similarity index 100% rename from skills/bmad-module-builder/assets/setup-skill-template/scripts/cleanup-legacy.py rename to skills/bmad-module-builder/assets/_setup-skill-template/scripts/cleanup-legacy.py diff --git a/skills/bmad-module-builder/assets/setup-skill-template/scripts/merge-config.py b/skills/bmad-module-builder/assets/_setup-skill-template/scripts/merge-config.py similarity index 100% rename from skills/bmad-module-builder/assets/setup-skill-template/scripts/merge-config.py rename to skills/bmad-module-builder/assets/_setup-skill-template/scripts/merge-config.py diff --git a/skills/bmad-module-builder/assets/setup-skill-template/scripts/merge-help-csv.py b/skills/bmad-module-builder/assets/_setup-skill-template/scripts/merge-help-csv.py similarity index 100% rename from skills/bmad-module-builder/assets/setup-skill-template/scripts/merge-help-csv.py rename to skills/bmad-module-builder/assets/_setup-skill-template/scripts/merge-help-csv.py diff --git a/skills/bmad-module-builder/scripts/scaffold-setup-skill.py b/skills/bmad-module-builder/scripts/scaffold-setup-skill.py index 34d132b..010ebb7 100644 --- a/skills/bmad-module-builder/scripts/scaffold-setup-skill.py +++ b/skills/bmad-module-builder/scripts/scaffold-setup-skill.py @@ -50,7 +50,7 @@ def main() -> int: ) args = parser.parse_args() - template_dir = Path(__file__).resolve().parent.parent / "assets" / "setup-skill-template" + template_dir = Path(__file__).resolve().parent.parent / "assets" / "_setup-skill-template" setup_skill_name = f"{args.module_code}-setup" target = Path(args.target_dir) / setup_skill_name diff --git a/skills/bmad-module-builder/scripts/tests/test-scaffold-setup-skill.py b/skills/bmad-module-builder/scripts/tests/test-scaffold-setup-skill.py index 6f38912..3b82a5d 100644 --- a/skills/bmad-module-builder/scripts/tests/test-scaffold-setup-skill.py +++ b/skills/bmad-module-builder/scripts/tests/test-scaffold-setup-skill.py @@ -11,7 +11,7 @@ from pathlib import Path SCRIPT = Path(__file__).resolve().parent.parent / "scaffold-setup-skill.py" -TEMPLATE_DIR = Path(__file__).resolve().parent.parent.parent / "assets" / "setup-skill-template" +TEMPLATE_DIR = Path(__file__).resolve().parent.parent.parent / "assets" / "_setup-skill-template" def run_scaffold(tmp: Path, **kwargs) -> tuple[int, dict]: