DYN-10279 Add workflow - validate-skills#16961
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-10279
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Adds a GitHub Actions workflow to validate changes under .agents/skills/ using an external skill-validator tool, with a companion bash script to determine which skill folders changed and emit markdown results into the job summary.
Changes:
- Introduces a new PR-scoped workflow that runs when files under
.agents/skills/**change. - Adds a bash script to compute changed skill directories (via git diff) and run
skill-validatorper skill. - Emits validator output to both GitHub Actions annotations and the job summary.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
.github/workflows/validate-skills.yml |
New workflow to run Go-based skill-validator on PRs touching .agents/skills/**. |
.github/scripts/validate-skill.sh |
New script to detect changed skill directories and run validation with markdown summary output. |
You can also share your feedback on Copilot code review. Take the survey.
| @@ -74,13 +74,13 @@ Load the relevant reference when working on that content type. | |||
|
|
|||
| | Content Type | When to Use | Reference | | |||
| |-------------|-------------|-----------| | |||
| | **UI content** | Error messages, notifications, labels, tooltips | [UI content guidelines](references/ui-content.md) | | |||
There was a problem hiding this comment.
I believe references is the folder following agent skill standard
There was a problem hiding this comment.
I followed the pattern of dynamo codebase patterns agent here. We might have to create a skill for this as well and create agent wrapper.
QilongTang
left a comment
QilongTang
left a comment
There was a problem hiding this comment.
One comment then LGTM
|
Purpose
Declarations
Check these if you believe they are true
Release Notes
N/A
Reviewers
(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)
(FILL ME IN, optional) Any additional notes to reviewers or testers.
FYIs
(FILL ME IN, Optional) Names of anyone else you wish to be notified of