validate-module.py: line-based YAML parser gives false fails and false passes; any '*-setup' folder hijacks detection
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 76/100
調査の方向性
Start with skills/bmad-module-builder/scripts/validate-module.py and trace parse_yaml_minimal() and find_setup_skill(); inspect the validator's tests if present. Reproduce the listed YAML, BOM, and setup-folder cases against a copy of skills/. Done means those cases match the expected behavior, including deterministic handling when multiple setup-skill candidates match.
索引モデルが issue の本文から書いたものです。
説明
Describe the bug
skills/bmad-module-builder/scripts/validate-module.py:
parse_yaml_minimal()flattens everykey: valueat any indent, and drops values starting with>. Each change below was applied separately to a copy of bmb's ownskills/, which passes with an eval-runner row added:
| Change | Validator | Should be |
|---|---|---|
description: > + indented text (valid YAML) |
fail: missing or empty required field: description |
pass |
top-level code: removed, a variable block has nested code: nested-value |
pass, module_code: nested-value |
fail |
module-help.csv saved with a UTF-8 BOM (Excel default) |
fail: CSV header mismatch: missing: module; extra: \ufeffmodule |
pass |
find_setup_skill()returns the first*-setupdirectory in rawiterdir()order. It doesn't require aSKILL.md. Adding an emptyaaa-old-setup/next tobmad-bmb-setup/made the validator report 3 critical "Missing required file" findings against the wrong folder.
Steps to reproduce
cp -r skills /tmp/vm && cd /tmp/vm
mkdir aaa-old-setup
python3 <bmb>/skills/bmad-module-builder/scripts/validate-module.py .
# setup_skill: aaa-old-setup -> 3 critical findings
Expected behavior
- Load
module.yamlwith a real YAML parser (yaml.safe_load), checking top-level keys only. - Read the CSV as
utf-8-sig. - Treat only directories with
SKILL.mdandassets/module.yamlas setup-skill candidates. Error clearly when more than one matches, and sort candidates for deterministic output.
Environment
- Model(s) used: n/a (deterministic script)
- Agentic IDE used: n/a (scripts run directly)
- BMad version: bmad-builder
main@d4a921c(v2.2.2 + #118) - Project language: Python 3.13, Linux
- 主要言語
- Python
- スター
- 220
- フォーク
- 54
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
bmad-code-org/bmad-builder のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
bmad-code-org/bmad-builder#157 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
bmad-code-org/bmad-builder#153 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
bmad-code-org/bmad-builder#151 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
bmad-code-org/bmad-builder#127 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
bmad-code-org/bmad-builder#123 ·
bmad-code-org/bmad-builder の issue をすべて見る
似ている issue
-
namespace
難易度 1/5 1時間未満 初心者へのやさしさ 68/100
EclipseFdn/open-vsx.org#15604 ·
メンテナーはふだん 1 日以内に返信
-
app: scripts severity: low status: backlog type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
netbox-community/netbox-scripts#164 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 5 日以内に返信
-
Public function to read participants.tsv対応中かも @bruAristimunha が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 85/100
huggingface/tau#799 ·
メンテナーはふだん 2 日以内に返信