fix: quote the four remaining argument-hint values (follow-up to #69) - #73
Open
Flospe0712 wants to merge 1 commit into
Open
fix: quote the four remaining argument-hint values (follow-up to #69)#73Flospe0712 wants to merge 1 commit into
Flospe0712 wants to merge 1 commit into
Conversation
…lderLAB#69, closes rest of XBuilderLAB#68) cheat-learn-from, cheat-predict, cheat-publish and cheat-retro still had an unquoted `argument-hint:` starting with `<...>` and containing `[... | ...]`. Strict YAML parsers (PyYAML safe_load) reject these frontmatters; loaders that validate the field as a string drop the skill silently (see XBuilderLAB#68). Same mechanical fix as XBuilderLAB#69: wrap the value in double quotes. No value contained `"` or `\`. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0181v451FvyoNNZnKWmn9UiD
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.
Follow-up to #69 (closes the remaining part of #68).
Summary
Four skills still carry an unquoted
argument-hint:value whose text starts with<…>and contains[… | …]— the flow-sequence / anchor-looking syntax that made loaders drop skills in #68:skills/cheat-learn-from/SKILL.mdskills/cheat-predict/SKILL.mdskills/cheat-publish/SKILL.mdskills/cheat-retro/SKILL.mdSame mechanical fix as #69: wrap the value in double quotes so every YAML parser reads it as a plain string.
Why it matters
Strict YAML parsers (PyYAML
safe_load, and the frontmatter lint some of us run over~/.claude/skills) choke on these four files, while Claude Code happens to tolerate them — so the breakage is invisible until a stricter loader (Copilot CLI ≥ 1.0.65, see #68) silently skips the skill. Found by a frontmatter lint pass on 2026-09-07 over a clone at 4123941; the other seven skills fixed in #69 parse cleanly.Verification
yaml.safe_loadover the frontmatter of everyskills/*/SKILL.md(and the rootSKILL.md) succeeds after the change, and each changed file returnsargument-hintasstr"or\, so plain double-quote wrapping is safecheat-bump,cheat-score,cheat-score-blind,cheat-shoot) still have unquoted hints, but those contain no:/[at the start and parse as plain scalars — left untouched to keep the diff minimal. The actual parser trap in the four fixed files is the[— mode: v1|v2]part::inside a plain scalar reads as a nested mapping.🤖 Generated with Claude Code
https://claude.ai/code/session_0181v451FvyoNNZnKWmn9UiD