fix(skills): quote argument-hint YAML values as strings#121
Open
thejesh23 wants to merge 1 commit into
Open
Conversation
Copilot CLI 1.0.65 tightened its skill-loader validation to require
`argument-hint` to be a string. When the value is written with bare
YAML brackets (e.g. `[takeout-zip-path]`), YAML parses it as a
one-element flow sequence — an array, not a string — so the loader
silently rejects the skill and it stops appearing in the command menu.
Wrap each buggy value in double quotes so YAML round-trips to `str`.
Verified with `yaml.safe_load` on each frontmatter.
Files:
- gemini-history-analyzer/SKILL.md
- product-analysis/SKILL.md
- feishu-doc-scraper/SKILL.md
- notify-wecom/SKILL.md
- daymade-audio/asr-transcribe-to-text/SKILL.md
- daymade-claude-code/continue-claude-work/SKILL.md
- daymade-claude-code/marketplace-dev/SKILL.md
- daymade-skill/skill-creator/SKILL.md (frontmatter reference example
that was teaching the buggy pattern; updated to model the correct
quoted form; the "Bad:" example lower in the same file is left
intact — it exists to teach users NOT to overload argument-hint
with manual flags)
Also latent on Claude Code today; see anthropics/claude-code#22161.
043372e to
7cf0ebd
Compare
Author
|
Tracking issue: #122 — 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
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.
Description
Fixes a YAML frontmatter bug in 7 skills + 1 documentation example. GitHub Copilot CLI 1.0.65 tightened its skill-loader validation to require
argument-hintto be a string. When the value is written with bare YAML brackets, YAML parses it as a one-element flow sequence — a list, not a string — so the loader silently rejects the skill and it disappears from the command menu.Verified with
yaml.safe_load— every changed frontmatter now returnsargument-hintasstr, notlist.Also latent on Claude Code today; see anthropics/claude-code#22161.
Type of Change
Files changed (8)
Real skill frontmatters (7):
gemini-history-analyzer/SKILL.mdproduct-analysis/SKILL.mdfeishu-doc-scraper/SKILL.mdnotify-wecom/SKILL.mddaymade-audio/asr-transcribe-to-text/SKILL.mddaymade-claude-code/continue-claude-work/SKILL.mddaymade-claude-code/marketplace-dev/SKILL.mdDocumentation example (1):
daymade-skill/skill-creator/SKILL.md— the "YAML Frontmatter Reference" code block (around line 235) previously showedargument-hint: [topic]as the canonical example, which taught downstream skill authors to reproduce the bug. Updated toargument-hint: "[topic]". Left intact: the "Bad:" example around line 389 — that block deliberately shows a buggy pattern to warn users NOT to overloadargument-hintwith manual flags, and rewriting it would defeat the pedagogical point.Test plan
yaml.safe_loadon each changed frontmatter —type(argument-hint) == strfor all 7 real filesfind . -name '*.md' | xargs grep '^argument-hint: \['— only the intentional "Bad:" doc example on line 389 remainscopilot skill listshows all 7 skills after install (maintainer to verify locally if desired)Related Issues
None open on this repo. Same class of latent bug on Claude Code: anthropics/claude-code#22161.