Skip to content

Commit 6471c85

Browse files
authored
fix: CI should validate SKILL.md instead of arbitrary .md files (#2)
The validation step used `find ... -name '*.md' | head -1` to pick the first .md file in each skill directory. On Linux (GitHub Actions), `find` returns files in inode order — not alphabetical — so auxiliary files like CREATION-LOG.md or code-quality-reviewer-prompt.md could be picked instead of SKILL.md. These auxiliary files have no frontmatter, causing 4 false-positive validation errors on every CI run. Fix: directly target SKILL.md (the canonical skill definition file) and error if it's missing rather than silently skipping.
1 parent 6046c4c commit 6471c85

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@ jobs:
2323
run: |
2424
ERRORS=0
2525
for dir in skills/*/; do
26-
SKILL_FILE=$(find "$dir" -maxdepth 1 -name '*.md' | head -1)
27-
if [ -z "$SKILL_FILE" ]; then
28-
echo "::warning::No .md file in $dir"
26+
SKILL_FILE="${dir}SKILL.md"
27+
if [ ! -f "$SKILL_FILE" ]; then
28+
echo "::error::Missing SKILL.md in $dir"
29+
ERRORS=$((ERRORS + 1))
2930
continue
3031
fi
3132

0 commit comments

Comments
 (0)