fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (closes #5626)#5627
fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (closes #5626)#5627thejesh23 wants to merge 1 commit into
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.
|
@thejesh23 is attempting to deploy a commit to the Sim Team on Vercel. A member of the Team first needs to authorize it. |
PR SummaryLow Risk Overview Unquoted values like Reviewed by Cursor Bugbot for commit 65e2714. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR fixes command frontmatter so bracketed argument hints load as strings.
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (1): Last reviewed commit: "fix: quote argument-hint YAML values so ..." | Re-trigger Greptile |
Closes #5626.
Summary
Purely mechanical single-character transform to 9 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 (9)
.claude/commands/cleanup.md.claude/commands/ship.md.claude/commands/you-might-not-need-a-memo.md.claude/commands/you-might-not-need-an-effect.md.claude/commands/you-might-not-need-state.md.claude/commands/react-query-best-practices.md.claude/commands/emcn-design-review.md.claude/commands/you-might-not-need-a-callback.md.claude/commands/you-might-not-need-url-state.mdVerification
yaml.safe_loadon every changed frontmatter reportsargument-hintasstrargument-hint:lines remain in these files"or\, so bare double-quote wrapping is safeSources
argument-hintdocumented as a string