Skip to content

Add static eval via skill-validator#1195

Merged
aaronpowell merged 2 commits into
github:stagedfrom
JanKrivanek:dev/jankrivanek/static-eval-v2
Mar 30, 2026
Merged

Add static eval via skill-validator#1195
aaronpowell merged 2 commits into
github:stagedfrom
JanKrivanek:dev/jankrivanek/static-eval-v2

Conversation

@JanKrivanek
Copy link
Copy Markdown
Contributor

(replacement of #1186)

Motivation

Add static non-LLM evaluation of skills by skill-validator from dotnet/skills
As a:

  • nonblocking PR gate
  • nightly run creating discussion item with report

Prereq

  1. go to https://github.com/github/awesome-copilot/discussions/categories

  2. Click New category

  3. Set:

    • Name: Skill Quality Reports
    • Description: Automated nightly skills eval reports
    • Format: Announcement (so only the workflow bot can post, others can comment)
    • (+ optional - emoji)
  4. Click Create

Copy link
Copy Markdown
Contributor

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 GitHub Actions workflows to run the skill-validator tool for static (non-LLM) evaluation of skills/agents: one workflow posts a nightly report to Discussions (with an Issue fallback), and another posts a non-blocking comment on PRs affecting skills/agents.

Changes:

  • Added a nightly scheduled workflow that runs skill-validator across all skills/agents and publishes a formatted report (Discussion preferred, Issue fallback).
  • Added a PR-gate workflow that runs skill-validator on changed skills/agents and posts/updates a single PR comment with the results.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/skill-quality-report.yml Nightly scheduled scan that generates and publishes a “Skill Quality Report” discussion (or issue fallback).
.github/workflows/skill-check.yml PR workflow that validates changed skills/agents and posts the results as a PR comment (non-blocking).

Comment thread .github/workflows/skill-check.yml
Comment thread .github/workflows/skill-check.yml
Comment thread .github/workflows/skill-quality-report.yml
@aaronpowell aaronpowell merged commit 85d6909 into github:staged Mar 30, 2026
8 checks passed
@JanKrivanek JanKrivanek mentioned this pull request Mar 30, 2026
JanKrivanek added a commit to dotnet/msbuild that referenced this pull request Apr 15, 2026
### Context
Let's use the skill-validator to check for basic problems of our skills
and agents
This is inspired and sourced from similar integrations:
 * github/awesome-copilot#1195
 * dotnet/runtime#126814

### Changes Made
- Added pipeline using the skill-validator
- Fixed the current findings (renames of folders and agent file)

### Testing
- skill-validator is the test harness for the skills
PureWeen pushed a commit to dotnet/maui that referenced this pull request Apr 21, 2026
<!-- Please let the below note in for people that find this PR -->
> [!NOTE]
> Are you waiting for the changes in this PR to be merged?
> It would be very helpful if you could [test the resulting
artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from
this PR and let us know in a comment if this change resolves your issue.
Thank you!

### Description of Change

Adopting skills-validator

This is inspired and sourced from similar integrations:
 * github/awesome-copilot#1195
 * dotnet/runtime#126814
 * dotnet/msbuild#13537

One important difference is that in this case we run as well in the LLM
judging mode

cc: @PureWeen - for awareness and for help with iterating on this :)
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