Skip to content

fix: validate skill frontmatter as yaml#512

Merged
kaizhou-lab merged 1 commit into
mainfrom
fix-skill-yaml-frontmatter
Jun 24, 2026
Merged

fix: validate skill frontmatter as yaml#512
kaizhou-lab merged 1 commit into
mainfrom
fix-skill-yaml-frontmatter

Conversation

@kaizhou-lab

Copy link
Copy Markdown
Contributor

Summary

  • Validate skill frontmatter with serde_yaml during import/read paths instead of hand-parsing name/description prefixes.
  • Tighten SKILL.md frontmatter fence extraction and add regression coverage for invalid YAML, quoted descriptions, block scalars, and embedded builtin skills.
  • Format invalid builtin skill descriptions as folded YAML scalars.

Test Plan

  • cargo test -p aionui-extension
  • cargo clippy -p aionui-extension -- -D warnings
  • cargo fmt --all -- --check
  • just push -u origin fix-skill-yaml-frontmatter

@kaizhou-lab kaizhou-lab merged commit 6b46055 into main Jun 24, 2026
6 checks passed
@kaizhou-lab kaizhou-lab deleted the fix-skill-yaml-frontmatter branch June 24, 2026 07:52
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