fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (closes #1) - #2
Conversation
… skills `argument-hint: [foo]` YAML-parses as a flow sequence (array), not a string. Downstream slash-command loaders that validate `argument-hint` as a string — notably GitHub Copilot CLI ≥ 1.0.65 — silently reject the skill on load, and the command disappears from the CLI menu. Wrap the value in double quotes so it parses as a string. No behaviour change on Claude Code.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThree command markdown files update YAML front-matter ChangesCommand argument-hint normalization
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Closes #1.
Summary
Purely mechanical single-character transform to 3 file(s): wrap the value after
argument-hint:in double quotes so YAML parses it as a string instead of a flow-sequence array. Fixes GitHub Copilot CLI ≥ 1.0.65 silently dropping the skill.Bare
[...]is YAML flow-sequence syntax → the loader receives["foo"](a list), fails itsstrvalidation, and rejects the SKILL without an error. Claude Code, VS Code Agent Skills, and every other loader documentargument-hintas a string; the quoted-string form is what all reference docs show.Files (3)
command/debug.mdcommand/review.mdcommand/subagent-driven-development.mdVerification
yaml.safe_loadon every changed frontmatter reportsargument-hintasstr"or\, so bare double-quote wrapping is safeSources
argument-hintdocumented as a stringSummary by cubic
Quote the
argument-hintfrontmatter values so YAML parses them as strings, preventing GitHub Copilot CLI ≥ 1.0.65 from silently dropping these skills.Written for commit f2286a8. Summary will update on new commits.
Summary by CodeRabbit