Skip to content

Fix parsing issue with WebGPU testing skill#29237

Merged
adrastogi merged 1 commit into
mainfrom
adrastogi/webgpu-skill-fix
Jun 23, 2026
Merged

Fix parsing issue with WebGPU testing skill#29237
adrastogi merged 1 commit into
mainfrom
adrastogi/webgpu-skill-fix

Conversation

@adrastogi

Copy link
Copy Markdown
Contributor

Description

The  webgpu-local-testing skill is failing to load because of invalid YAML in its frontmatter. The unquoated description: value contained colon-space sequences ( SCOPE: lavapipe ,  e.g.: ), and according to Copilot, in YAML a plain (unquoted) scalar can't contain ": ". The parser reads it as a nested mapping key and aborts with:

ScannerError: mapping values are not allowed here

It was the only one of the 8 skills with this pattern, which is why every other skill loaded fine. The fix is to wrap the description value in double quotes and adjust SCOPE:  to SCOPE - so the colons are treated as literal text. The frontmatter now parses, with both required keys (name, description) intact.

Motivation and Context

The Copilot CLI was flagging this skill as failing to load, so this change attempts to resolve that error.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a YAML frontmatter parsing failure in the webgpu-local-testing agent skill by making the description field a quoted scalar so colon-space sequences are treated as literal text and the skill can load correctly.

Changes:

  • Wrap the description frontmatter value in double quotes to prevent YAML scanner errors.
  • Adjust the SCOPE: text within the description to avoid : sequences in the rendered string.
Show a summary per file
File Description
.agents/skills/webgpu-local-testing/SKILL.md Quotes the YAML description frontmatter (and tweaks the embedded “SCOPE” text) to make the skill’s frontmatter parse cleanly.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@titaiwangms

Copy link
Copy Markdown
Contributor

Should we add this into one of the SKILL.md or somewhere as a reminder?

@adrastogi

Copy link
Copy Markdown
Contributor Author

Should we add this into one of the SKILL.md or somewhere as a reminder?

Yeah, good question- I don't see a natural place to slot this, unfortunately. If this becomes a more systemic problem, perhaps we can re-evaluate if we need to add explicit instructions.

@adrastogi adrastogi merged commit 67aa0aa into main Jun 23, 2026
91 of 93 checks passed
@adrastogi adrastogi deleted the adrastogi/webgpu-skill-fix branch June 23, 2026 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants