fix(sagemaker-ai): quote model-evaluation skill description to fix YAML parse error#175
Open
youtae wants to merge 1 commit into
Open
Conversation
…ML parse error
The model-evaluation SKILL.md description is an unquoted YAML plain scalar that
contains a colon-space sequence ("two evaluation types: LLM-as-Judge"). The YAML
parser reads that colon as a mapping indicator and rejects the frontmatter with
"mapping values are not allowed here". When this happens the skill loads with
empty metadata (name, description, and version are silently dropped), so the
skill no longer triggers and "claude plugin validate" reports an error.
Wrapping the description in single quotes makes it a valid quoted scalar while
preserving the text verbatim, including the inner double quotes and the colon.
This is the same class of fix as awslabs#10.
youtae
requested review from
a team,
krokoko,
scottschreckengaust and
theagenticguy
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.
Fixes a YAML frontmatter parse error in the
sagemaker-aiplugin'smodel-evaluationskill.Related
Same class of fix as #10. No existing issue or PR addresses this skill.
Changes
The
descriptioninplugins/sagemaker-ai/skills/model-evaluation/SKILL.mdis an unquoted YAML plain scalar that contains a colon followed by a space (two evaluation types: LLM-as-Judge). A YAML parser treats that colon as a mapping indicator inside a block scalar and rejects the frontmatter withmapping values are not allowed here.Effect: the skill loads with empty metadata (the
name,description, andmetadata.versionfields are silently dropped), so the skill does not trigger, andclaude plugin validate plugins/sagemaker-aifails with:Fix: wrap the
descriptionvalue in single quotes. This turns it into a valid quoted scalar while preserving the text verbatim, including the inner double quotes and the colon. The change is a single line; no wording is altered.I scanned all 28
SKILL.mdfiles in the repo with a YAML parser; this was the only one that failed to parse. After the change,claude plugin validate plugins/sagemaker-aireports✔ Validation passed.Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.