Skip to content

Add labels for skill check findings#1627

Merged
aaronpowell merged 1 commit into
stagedfrom
add-skill-check-labels
May 6, 2026
Merged

Add labels for skill check findings#1627
aaronpowell merged 1 commit into
stagedfrom
add-skill-check-labels

Conversation

@aaronpowell
Copy link
Copy Markdown
Contributor

Summary

  • manage PR labels from the workflow_run-based skill-check comment workflow
  • add skill-check-warning and skill-check-error labels based on validator results
  • keep labels in sync as findings change

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 6, 2026 00:01
@github-actions github-actions Bot added the workflow PR touches workflow automation label May 6, 2026
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 PR-label management to the skill-check comment workflow so validator results are reflected not only in the comment body but also in repository labels. This fits into the existing GitHub Actions automation around validating changed skills/agents on pull requests.

Changes:

  • Grant the workflow issues: write so it can manage labels on PRs.
  • Add logic to create and synchronize skill-check-warning / skill-check-error labels from validator output.
  • Run label sync alongside the existing artifact-driven PR comment update flow.
Show a summary per file
File Description
.github/workflows/skill-check-comment.yml Extends the workflow-run comment job to provision and synchronize skill-check labels based on validator findings.

Copilot's findings

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

Comment on lines +123 to +127
if (exitCode !== '0' || errorCount > 0) {
desiredLabels.add('skill-check-error');
}

await syncManagedLabels(prNumber, desiredLabels);
if (exitCode !== '0' || errorCount > 0) {
desiredLabels.add('skill-check-error');
}

@aaronpowell aaronpowell merged commit ce51668 into staged May 6, 2026
14 checks passed
@aaronpowell aaronpowell deleted the add-skill-check-labels branch May 6, 2026 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow PR touches workflow automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants