Skip to content

Fix primitive front-matter for spec compliance#5

Open
ChrisRomp wants to merge 4 commits into
mainfrom
fix/style-checker-skill-frontmatter
Open

Fix primitive front-matter for spec compliance#5
ChrisRomp wants to merge 4 commits into
mainfrom
fix/style-checker-skill-frontmatter

Conversation

@ChrisRomp

@ChrisRomp ChrisRomp commented May 5, 2026

Copy link
Copy Markdown

APM and CLI are giving errors or warnings for missing front-matter in some skills and instructions. This PR aims to resolve these errors.

Expanded the description of the style-checker skill to include front matter specific use cases and areas of focus.
@ChrisRomp ChrisRomp requested review from Copilot and danielmeppiel May 5, 2026 15:24
Added a description to the design standards file.
@ChrisRomp ChrisRomp changed the title Enhance style-checker skill description Fix primitive front-matter for spec compliance May 5, 2026

Copilot AI left a comment

Copy link
Copy Markdown

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 updates APM markdown frontmatter metadata so the style-checker skill loads correctly in the CLI, and aligns the design-standards instruction with richer metadata.

Changes:

  • Added required YAML frontmatter (name, description) to .apm/skills/style-checker/SKILL.md to prevent CLI load failures.
  • Added a description field to the design standards instruction frontmatter for clearer package metadata.
Show a summary per file
File Description
.apm/skills/style-checker/SKILL.md Adds YAML frontmatter (name/description) so the skill can be discovered/loaded and better communicates intended style-check scope.
.apm/instructions/design-standards.instructions.md Adds a frontmatter description to improve instruction metadata consistency and clarity.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

@ChrisRomp

Copy link
Copy Markdown
Author

I just noticed I'm partially duplicating #3.

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.

2 participants