Skip to content

Fix invalid SKILL.md frontmatter metadata - #51457

Merged
pelikhan merged 3 commits into
mainfrom
copilot/fix-skill-md-frontmatter-defects
Aug 8, 2026
Merged

Fix invalid SKILL.md frontmatter metadata#51457
pelikhan merged 3 commits into
mainfrom
copilot/fix-skill-md-frontmatter-defects

Conversation

Copilot AI commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Five skill documents had invalid or incomplete YAML frontmatter, preventing spec-compliant consumers from reliably discovering and parsing them.

  • Required names

    • Add directory-matching name fields to error-recovery-patterns and skillz-integration.
    • Rename http_mcp_headers to http-mcp-headers.
  • Valid YAML descriptions

    • Quote colon-containing descriptions in developer-internals and developer-security.
name: http-mcp-headers
description: "Internal gh-aw architecture: validation system design, ..."

Copilot AI and others added 2 commits August 8, 2026 20:21
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix frontmatter defects in SKILL.md files Fix invalid SKILL.md frontmatter metadata Aug 8, 2026
Copilot AI requested a review from pelikhan August 8, 2026 20:22
@pelikhan
pelikhan marked this pull request as ready for review August 8, 2026 20:23
Copilot AI balanced review requested due to automatic review settings August 8, 2026 20:23
@pelikhan
pelikhan merged commit 7b52d5f into main Aug 8, 2026
@pelikhan
pelikhan deleted the copilot/fix-skill-md-frontmatter-defects branch August 8, 2026 20:23

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

Fixes invalid YAML frontmatter in five skill documents, improving standards-compliant skill discovery.

Changes:

  • Adds missing or corrected skill names.
  • Quotes descriptions containing colons.
  • Also introduces an unrelated /ponytail routing change.
Show a summary per file
File Description
.github/workflows/agentic_commands.yml Adds /ponytail routing.
.github/skills/skillz-integration/SKILL.md Adds the required name.
.github/skills/http-mcp-headers/SKILL.md Corrects the skill name.
.github/skills/error-recovery-patterns/SKILL.md Adds the required name.
.github/skills/developer-security/SKILL.md Makes the description valid YAML.
.github/skills/developer-internals/SKILL.md Makes the description valid YAML.

Review details

Tip

Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 6/6 changed files
  • Comments generated: 1
  • Review effort level: Balanced

# /nit -> pr-nitpick-reviewer [pull_request_comment,pull_request_review_comment] reaction=eyes
# /plan -> plan [discussion_comment,issue_comment] reaction=eyes
# /poem-bot -> poem-bot [issues] reaction=eyes
# /ponytail -> ponytail-reviewer [pull_request_comment,pull_request_review_comment] reaction=eyes
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.

SKILL.md frontmatter defects: missing name field, invalid name charset, and unparseable YAML in descriptions

3 participants