Skip to content

Quote SKILL.md description so skills add can parse the frontmatter - #49

Merged
raphaelmansuy merged 1 commit into
raphaelmansuy:mainfrom
xz-tak:fix-skill-description-yaml-quoting
Oct 9, 2026
Merged

raphaelmansuy merged 1 commit into
raphaelmansuy:mainfrom
xz-tak:fix-skill-description-yaml-quoting

Conversation

@xz-tak

@xz-tak xz-tak commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Problem

npx skills add raphaelmansuy/edgeparse skips the skill with:

Skipped skills/edgeparse/SKILL.md — YAML parse error: Nested mappings are not allowed in compact mappings

The description: value ends with Install with: pip install edgeparse. The unquoted : (colon-space) makes the YAML parser interpret a nested mapping inside the scalar, so the frontmatter fails to load and the skill is dropped.

Fix

Wrap the description: value in double quotes. No wording change — just making the existing text valid YAML so skills add (and any strict frontmatter parser) accepts it.

Verification

yaml.safe_load now parses the frontmatter, and npx skills add raphaelmansuy/edgeparse installs the skill instead of skipping it.

…tter

The description value ends with 'Install with: pip install edgeparse'. The unquoted
': ' makes the YAML parser read a nested mapping inside the scalar, so the frontmatter
fails to load and `npx skills add raphaelmansuy/edgeparse` skips the skill with
'Nested mappings are not allowed in compact mappings'. Wrapping the value in double
quotes keeps the wording and makes it valid YAML.
@raphaelmansuy
raphaelmansuy merged commit 8d706e0 into raphaelmansuy:main Oct 9, 2026
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.

2 participants