Skip to content

The skills/agents check filters-in only touched files#13586

Merged
JanKrivanek merged 2 commits into
dotnet:mainfrom
JanKrivanek:dev/jsnkrivanek/check-only-changed-skills
Apr 21, 2026
Merged

The skills/agents check filters-in only touched files#13586
JanKrivanek merged 2 commits into
dotnet:mainfrom
JanKrivanek:dev/jsnkrivanek/check-only-changed-skills

Conversation

@JanKrivanek
Copy link
Copy Markdown
Member

#13537 followup

Context

The PR above introduced skills/agents files checking within PRs touching skills/agents
But it checked all the files - possibly repeating same and unactionable findings

Changes Made

We now filter only for the touched skills and agent files

@JanKrivanek JanKrivanek requested a review from a team as a code owner April 21, 2026 10:16
Copilot AI review requested due to automatic review settings April 21, 2026 10:16
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Skill Validator Results

✅ All checks passed

Scope Checked
Skills 13
Agents 1
Total 14
Severity Count
--- ---:
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

No findings were emitted by the validator.

Full validator output ```text No validator output captured. ```

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

Updates the Skill Validation GitHub Actions workflow to scope skill-validator checks to only the skills/agents modified in a pull request, reducing redundant and unactionable findings.

Changes:

  • Add a PR-only step that queries the PR file list and identifies changed .github/skills/** and .github/agents/** paths.
  • Remove unchanged skill directories / agent files from the workspace before running skill-validator.
  • Skip the skill-validator step on PRs where no skill/agent files changed, and normalize the recorded exit code when the check is skipped.
Show a summary per file
File Description
.github/workflows/skill-validation.yml Detects PR-changed skill/agent files, filters the workspace to only those paths, and conditionally runs skill-validator.

Copilot's findings

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

Comment thread .github/workflows/skill-validation.yml
Comment thread .github/workflows/skill-validation.yml
Comment thread .github/workflows/skill-validation.yml Outdated
Comment thread .github/workflows/skill-validation.yml Outdated
@JanKrivanek JanKrivanek merged commit 73b6615 into dotnet:main Apr 21, 2026
11 checks passed
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