Skip to content

fix(skill): quote the description so SKILL.md frontmatter parses - #32

Merged
ChanMeng666 merged 1 commit into
masterfrom
fix/skill-frontmatter-yaml
Oct 21, 2026
Merged

ChanMeng666 merged 1 commit into
masterfrom
fix/skill-frontmatter-yaml

Conversation

@ChanMeng666

Copy link
Copy Markdown
Owner

Summary

#31 shortened the skill description, and the new text contained an unquoted : (echook (audio-hooks): sound, ...). That is invalid YAML. Claude Code drops a skill whose frontmatter fails to parse without any warning, so after updating to 7.1.0 audio-hooks:audio-hooks was missing from /context and from a fresh claude -p session.

  • The description is now double-quoted. The text is unchanged.
  • New tests/test_skill_frontmatter.py covers this case:
    • parses the frontmatter with PyYAML when it is installed;
    • also runs a dependency-free check for unquoted : .
    • Verified that it fails on the old file and passes on the fixed one.

Test plan

  • pytest tests: 840 passed, 2 skipped
  • scripts/build-plugin.sh --check: in sync

🤖 Generated with Claude Code

The shortened description contained an unquoted ': ', which is invalid YAML;
Claude Code silently dropped the skill. Quote it and add a frontmatter test
that fails on the old file.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ChanMeng666
ChanMeng666 merged commit f6173c7 into master Oct 21, 2026
12 checks passed
@ChanMeng666
ChanMeng666 deleted the fix/skill-frontmatter-yaml branch October 21, 2026 09:10
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