Skip to content

feat: add skill validation CI job#18

Merged
CybotTM merged 1 commit intomainfrom
feat/add-skill-validation-ci
Feb 25, 2026
Merged

feat: add skill validation CI job#18
CybotTM merged 1 commit intomainfrom
feat/add-skill-validation-ci

Conversation

@CybotTM
Copy link
Member

@CybotTM CybotTM commented Feb 25, 2026

Summary

  • Add skill validation job calling reusable workflow from skill-repo-skill
  • Checks SKILL.md frontmatter, word count, composer.json, plugin.json, file presence

Test plan

  • CI passes with new validation job

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@CybotTM CybotTM requested a review from Copilot February 25, 2026 14:11
@CybotTM CybotTM merged commit 0b3576b into main Feb 25, 2026
5 checks passed
@CybotTM CybotTM deleted the feat/add-skill-validation-ci branch February 25, 2026 14:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a CI “Skill Validation” job to the existing lint workflow by calling a reusable workflow from netresearch/skill-repo-skill, to validate SKILL.md and related skill/package metadata.

Changes:

  • Introduce a new validate job to run skill repository validation via reusable workflow.

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


validate:
name: Skill Validation
uses: netresearch/skill-repo-skill/.github/workflows/validate.yml@main
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

This reusable workflow is referenced by branch name (@main). That makes CI behavior mutable and increases supply-chain risk; it also diverges from the rest of this workflow which pins actions by commit SHA. Pin the reusable workflow to a specific tag or commit SHA (and ideally document the version in a comment) so runs are reproducible.

Suggested change
uses: netresearch/skill-repo-skill/.github/workflows/validate.yml@main
# validate workflow v1.0.0
uses: netresearch/skill-repo-skill/.github/workflows/validate.yml@0123456789abcdef0123456789abcdef01234567

Copilot uses AI. Check for mistakes.
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