fix(skills): codex-fleet and omp-fleet frontmatter fails YAML parsing - #2
Open
UBRN wants to merge 1 commit into
Open
fix(skills): codex-fleet and omp-fleet frontmatter fails YAML parsing#2UBRN wants to merge 1 commit into
UBRN wants to merge 1 commit into
Conversation
Both descriptions were unquoted plain YAML scalars containing ": "
(colon-space). YAML reads that as a mapping indicator, so the block
fails to parse:
mapping values are not allowed in this context
A skill whose frontmatter does not parse is silently dropped — no
error, no warning, it just never loads. Anyone installing these two
from the repo got a skill that does nothing.
Three colon-space sequences per file, not one:
codex-fleet cols 129 ("describes): "), 371 ("locked: "), 562 ("on: ")
omp-fleet cols 241, 446, 527
Converted all three to >- folded blocks. Descriptions round-trip
byte-identical (882 / 686 / 423 chars), so every trigger keyword is
preserved verbatim.
gptpro was not broken but was the same fragile construct — one future
": " away from the identical failure. Hardened along with them.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
Both
codex-fleetandomp-fleetcurrently fail to load. Theirdescriptionvalues are unquoted plain YAML scalars containing": "(colon-space), which YAML reads as a mapping indicator:The failure is silent — a SKILL.md whose frontmatter does not parse is skipped with no error and no warning, so the skill simply never triggers. It looks identical to a skill that was loaded but did not match.
Three colon-space sequences per file, not one:
skills/codex-fleet/SKILL.mddescribes):), 371 (locked:), 562 (on:)skills/omp-fleet/SKILL.mdFix
Converted both to
>-folded block scalars. No wording changed — the descriptions round-trip byte-identical (882 and 686 chars), so every trigger keyword is preserved verbatim.skills/gptpro/SKILL.mdwas not broken, but its description is the same unquoted plain-scalar construct — one future": "away from the identical failure. Hardened alongside, same zero-content-change treatment (423 chars, unchanged).Verification
Fails on the two files before this change, clean after. All 10 skills parse;
namematches directory in every case; every description is under the 1024-char limit.Note
Happy to follow up with a small CI check (
ruby scripts/validate-skills.rb+ a workflow) that parses everySKILL.mdand fails the build on malformed frontmatter, if that is wanted — kept out of this PR to keep it to one concern. This bug class is invisible without one.🤖 Generated with Claude Code