Skip to content

Fix skill-creator scaffold frontmatter#345

Open
wsk-builds wants to merge 1 commit intoopenai:mainfrom
wsk-builds:fix/skill-creator-init-skill-template
Open

Fix skill-creator scaffold frontmatter#345
wsk-builds wants to merge 1 commit intoopenai:mainfrom
wsk-builds:fix/skill-creator-init-skill-template

Conversation

@wsk-builds
Copy link
Copy Markdown

@wsk-builds wsk-builds commented Apr 13, 2026

Problem

skills/.system/skill-creator/scripts/init_skill.py generates a SKILL.md whose frontmatter description placeholder is written as a YAML list:

description: [TODO: ...]

That means a freshly generated skill fails validation immediately instead of producing a valid starter template.

Root Cause

The scaffold uses square-bracket YAML syntax for the placeholder, so description is parsed as a list rather than a string. The validator then rejects the generated file with:

Description must be a string, got list

Fix

  • quote the generated description placeholder so it is parsed as a YAML string
  • add a regression test that generates a new skill and validates it immediately

Validation

  • python3 skills/.system/skill-creator/tests/test_init_skill.py
  • python3 skills/.system/skill-creator/scripts/init_skill.py repro-skill-final --path /tmp
  • python3 skills/.system/skill-creator/scripts/quick_validate.py /tmp/repro-skill-final

@wsk-builds wsk-builds requested a review from a team April 13, 2026 14:42
@wsk-builds
Copy link
Copy Markdown
Author

Minimal repro for the current bug:

python3 skills/.system/skill-creator/scripts/init_skill.py repro-skill --path /tmp
python3 skills/.system/skill-creator/scripts/quick_validate.py /tmp/repro-skill

Before this change, the second command fails with:

Description must be a string, got list

The generated SKILL.md contains this frontmatter entry:

description: [TODO: ...]

which YAML parses as a list instead of a string.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant