fix(skills): move mirror note below frontmatter in budget-negotiator - #452
Merged
cobusgreyling merged 1 commit intoAug 3, 2026
Merged
Conversation
The HTML comment sat above the opening `---`, so the YAML frontmatter was never recognised. Claude Code's skill registry loaded the skill with no `name` and advertised `description: <!--`, so the skill was effectively undiscoverable. Move the note below the frontmatter block in both the skill and its template mirror, and drop the trailing whitespace on the comment lines so the two files remain byte-identical as the note itself requires. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
cobusgreyling
approved these changes
Aug 3, 2026
cobusgreyling
left a comment
cobusgreyling
left a comment
Owner
There was a problem hiding this comment.
Correct and minimal: frontmatter must start on line 1 for skill discovery. Moving the mirror note below the closing --- (and keeping skill/template byte-identical) is the right fix. Approving; waiting on required CI after first-time-contributor workflow approval.
Owner
|
Merged. Thanks @Tarikv1 — frontmatter now starts on line 1 so |
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
2 participants
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.
Problem
skills/budget-negotiator/SKILL.mdopens with an HTML comment above the---frontmatter delimiter:Because
---is not the first line, the YAML frontmatter is never recognised. Claude Code's skill registry loads the skill with nonameand advertises its description as the literal string<!--, sobudget-negotiatoris effectively undiscoverable — the agent can't tell what it's for.It's the only
SKILL.mdin the repo with this layout; every other skill starts with---on line 1.Fix
Move the note below the frontmatter block. Same content, same file, just after the closing
---.Applied to both
skills/budget-negotiator/SKILL.mdandtemplates/SKILL.md.budget-negotiator, since the note itself requires the two to stay byte-identical. Trailing whitespace on the comment lines is dropped in both so they still match exactly (also satisfiesmarkdownlintMD009).Verification
Both files parse:
And remain byte-identical —
md5sumon both:7715893e316710fc26b8fb4faa7a937a.Body content is unchanged; the diff is header reordering only.
🤖 Generated with Claude Code