Skip to content

Fix skill frontmatter YAML - #11

Merged
songxuchen merged 1 commit into
mainfrom
codex/fix-video-to-music-yaml
Aug 16, 2026
Merged

Fix skill frontmatter YAML#11
songxuchen merged 1 commit into
mainfrom
codex/fix-video-to-music-yaml

Conversation

@songxuchen

Copy link
Copy Markdown
Contributor

What changed

  • Quote compatibility frontmatter values that contain credentials: ... so GitHub and YAML parsers can read the skill metadata.
  • Add a validator check for unquoted : in flat frontmatter values so this class of issue is caught in CI.

Why

GitHub was showing mapping values are not allowed in this context for video-to-music/SKILL.md because YAML plain scalars cannot contain : unless the value is quoted or folded. The same pattern existed in several Sonilo skill files.

Validation

  • python3 tests/validate.py
  • python3 + PyYAML parse of every */SKILL.md frontmatter
  • git diff --check

@songxuchen
songxuchen marked this pull request as ready for review August 16, 2026 02:01
@sapient-app

sapient-app Bot commented Aug 16, 2026

Copy link
Copy Markdown

Lightsage docs evals

Waiting for the staging docs URL before running evals.

Lightsage will start the selected PR evals automatically when GitHub reports a successful docs deployment for this PR. This usually happens within 15 minutes.

Commit: 8685b97
Status: waiting for staging docs URL

@songxuchen
songxuchen merged commit a0658f2 into main Aug 16, 2026
1 check passed
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