Repository navigation
fix: quote argument-hint in the 4 skills missed by #69, and pair the quotes inside description - #75
EndlessBlueWJL wants to merge 1 commit into
Conversation
Correction to my earlier description: I originally claimed these four files
carry TWO invalid frontmatter values. That was wrong, and I verified it:
- argument-hint unquoted with "key: value" inside a flow sequence
-> the actual and only cause of the parse failure
- description containing raw ASCII double quotes
-> VALID YAML, not a fault
Evidence (yaml package parser, upstream main):
original -> fails
fix description only -> still fails
fix argument-hint only -> parses, all fields present
fix both -> parses
So �rgument-hint alone is sufficient; the description's ASCII quotes parse
fine (checked with the quotes left in place: description length 127, 10 ASCII
double quotes, no error).
This PR therefore changes only the four argument-hint values, and additionally
pairs the ASCII quotes in description as a cosmetic consistency pass. The
cosmetic part is optional and can be dropped without affecting the fix.
Upstream main still has the unquoted argument-hint in:
skills/cheat-learn-from/SKILL.md
skills/cheat-predict/SKILL.md
skills/cheat-publish/SKILL.md
skills/cheat-retro/SKILL.md
b13febe to
ee3d1ff
Compare
|
Closing this in favour of the smaller existing PRs — #73 and #74 already make this exact fix, and they are more minimal than mine. I also want to correct my own description above. I originally claimed these four files carry two invalid frontmatter values. That was wrong, and I verified it before closing: Testing with the
So That means this PR's extra change (pairing those quotes into #73 and #74 each change only the four Sorry for the duplicate; the correct fix was already in the queue and I should have checked open PRs before opening my own. |
PR #69 quoted
argument-hintforSKILL.mdandskills/cheat-init/SKILL.mdonly. Four skill filesstill carry invalid YAML frontmatter, so strict frontmatter parsers reject the file and the host drops
the skill silently — it never appears in the catalog and there is no per-skill diagnostic, only a
host-log warning.
skills/cheat-learn-from/SKILL.md"indescription+ unquotedargument-hintskills/cheat-predict/SKILL.mdskills/cheat-publish/SKILL.mdskills/cheat-retro/SKILL.mdWhy each one fails
Both faults are "a value YAML cannot read as a plain scalar":
descriptioninterleaves raw ASCII double quotes with plain text:The plain scalar ends early; the parser then reports
Unexpected flow-seq-start at node end.argument-hintis unquoted and containskey: valueinside a flow sequence:→
Nested mappings are not allowed in compact mappings.Because both live in the frontmatter, the skill body never gets a chance to load.
Fix
Same approach as #69 — only the two header values change, the body is untouched:
description: raw ASCII quotes → paired Chinese quotes (“…”), alternating open/close so theinterleaved pattern
"A"/"B"/"C"becomes“A”/“B”/“C”.argument-hint: wrap the whole value in double quotes, exactly as fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (closes #68) #69 did forSKILL.md/skills/cheat-init/SKILL.md.Verification
Reproduced with the
yamlpackage parser that hosts ship:Nested mappings are not allowed in compact mappings at line 3, column 16name/descriptionpresent, Chinese quotes balanced(no orphan
“or”)the other 12 parse fine
Affected hosts
Any host that validates frontmatter strictly rather than tolerantly. Confirmed on Copilot CLI ≥1.0.65
(the surface #69 targeted) and on DeepSeek Harness. Tolerant readers (some Claude Code versions) load
these files anyway, which is why the bug stayed hidden.
Files changed
8 changed lines total (2 per file).