fix: quote argument-hint so YAML parses it as string#34
Open
thejesh23 wants to merge 1 commit into
Open
Conversation
Copilot CLI 1.0.65 requires the skill frontmatter argument-hint to be a string. An unquoted bracketed value parses as a YAML sequence and the skill fails to load. Wrap the value in double quotes so it stays a string across YAML 1.1/1.2 parsers.
Author
|
Tracking issue: #35 — captures the bug diagnosis and reproducer separately for anyone searching the repo who lands there before this PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skills/organize/SKILL.mdhas:YAML parses the unquoted
[...]value as a flow sequence, not a string. Copilot CLI 1.0.65 validates skill frontmatter and rejectsargument-hintunless it is a string, so the skill fails to load there. Claude Code has been lenient about this so the bug has gone unnoticed.This PR quotes the value so every YAML 1.1/1.2 parser reads it as a plain string:
YAML rule
argument-hint: [X]-> flow sequence -> BAD. Fix:argument-hint: "[X]".argument-hint: ["X"]-> single-element sequence containing a string -> BAD. Fix:argument-hint: "X".argument-hint: '["foo"]'..claude-plugin/skills/organize.mdalready uses the quoted form (argument-hint: "[port]"), so no change needed there.Files changed
skills/organize/SKILL.md(1 line)Test plan
python3 -c 'import yaml; ...'on the patched file returnsstrforargument-hint, value[--port <number>].rg 'argument-hint:'shows only two shipped skill files; both now parse as strings. (research/datasets/routeguard_external_v0.jsonmatches are external corpus data used by the activation-scanner benchmark, not shipped skills.)Context
Same class of bug tracked upstream at anthropics/claude-code#22161 - Copilot CLI's stricter frontmatter validation has surfaced YAML-array
argument-hintfields across many public skills. This PR only patches this repo's copy.