fix(claude): quote argument-hint so it parses as string, not YAML array#3
Open
thejesh23 wants to merge 1 commit into
Open
fix(claude): quote argument-hint so it parses as string, not YAML array#3thejesh23 wants to merge 1 commit into
thejesh23 wants to merge 1 commit into
Conversation
Copilot CLI 1.0.65 requires the `argument-hint` frontmatter field to be a string, but unquoted `[X]` is parsed by YAML as a flow-style array (and unquoted `<X>` as a tag/error), which breaks command loading. Wrap all such values in double quotes so they parse as scalars. Fixes 17 command/skill files across `claude/commands/` and `claude/skills/`. Same class of bug as anthropics/claude-code#22161 (latent in Claude Code, overt in Copilot CLI).
Author
|
Tracking issue: #4 — 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
Quote every
argument-hint:frontmatter value inclaude/commands/andclaude/skills/so it parses as a YAML string rather than a flow-style array or an invalid tag.The bug
The command/skill spec requires
argument-hintto be a string. YAML parsesargument-hint: [X]as a flow sequence (a list with one elementX), andargument-hint: <X>as a node tag / parse error. Both are silently tolerated by Claude Code today, but GitHub Copilot CLI 1.0.65 enforces the string type and refuses to load any command whoseargument-hintis not a scalar string.YAML rule at play:
Same class of latent bug as anthropics/claude-code#22161.
Files fixed (17)
Commands:
claude/commands/assess/generic.mdclaude/commands/attribution.mdclaude/commands/commit.mdclaude/commands/my-code-review.mdclaude/commands/pick.mdclaude/commands/recall.mdSkills:
claude/skills/deep-plan/SKILL.mdclaude/skills/implement-next/SKILL.mdclaude/skills/implement-with-notes/SKILL.mdclaude/skills/lavish/SKILL.mdclaude/skills/memory-promote/SKILL.mdclaude/skills/next-steps/SKILL.mdclaude/skills/nuxt-baseline/SKILL.mdclaude/skills/plan-next/SKILL.mdclaude/skills/skills-map/SKILL.mdclaude/skills/transcribe/SKILL.mdclaude/skills/youtube-transcript/SKILL.mdFiles where
argument-hint:is empty (null) were left alone — a null value is valid and does not trigger the bug.Test plan
argument-hint:across the tree; every non-empty value is now a double-quoted stringargument-hintasstr(no arrays, no tag errors)/help-style command list