Skip to content

fix(skill-creator): warn on unquoted description with YAML special characters - #539

Open
Lubrsy706 wants to merge 1 commit into
anthropics:mainfrom
Lubrsy706:fix/validate-unquoted-description
Open

Lubrsy706 wants to merge 1 commit into
anthropics:mainfrom
Lubrsy706:fix/validate-unquoted-description

Conversation

@Lubrsy706

Copy link
Copy Markdown

Summary

  • Add pre-parse validation in quick_validate.py to detect unquoted description fields containing :
  • Prevents silent YAML parsing failures where description is truncated or split into multiple keys
  • Checks raw frontmatter text before yaml.safe_load() to catch the issue early

Example

# ❌ Now caught by validator
description: Use when: gmail, inbox

# ✅ Passes validation
description: "Use when: gmail, inbox"

Closes #338

Test plan

  • Existing valid skills (pdf, frontend-design) still pass validation
  • Unquoted description with : correctly returns error with fix suggestion
  • Quoted description with : passes validation (both single and double quotes)
  • Description without : passes validation as before

🤖 Generated with Claude Code

…ial characters

Add a pre-parse check in quick_validate.py that detects when the
description field contains ':' but is not wrapped in quotes. This
causes silent YAML parsing failures where the description is
truncated or parsed as multiple keys.

Closes anthropics#338
@Lubrsy706
Lubrsy706 force-pushed the fix/validate-unquoted-description branch from 1e84d41 to 14b2f65 Compare March 6, 2026 15:22
@Lubrsy706

Copy link
Copy Markdown
Author

Friendly ping: this PR has been ready for review since March 6, 2026. If there is anything blocking it or if you would prefer changes before merge, let me know and I can update it. Would appreciate a review when someone has time.

@Lubrsy706

Copy link
Copy Markdown
Author

Following up on this PR. The fix should be ready for review; please let me know if you'd like any updates or additional test coverage from my side.

@Lubrsy706

Copy link
Copy Markdown
Author

Following up on this PR as well. The change should be ready for review from my side; if you would prefer any updates or additional coverage before merge, let me know and I can handle it. Would appreciate review when someone has time.

@Lubrsy706

Copy link
Copy Markdown
Author

Following up on this PR. The warning behavior for unquoted YAML descriptions should be ready for review; if you would like any changes or additional edge-case coverage, let me know and I can update it. Would appreciate a review when someone has time.

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.

quick_validate.py should warn about unquoted descriptions with special YAML characters

1 participant