From 24b0d8c86cfbd485466b8892492c086afb1c3f8a Mon Sep 17 00:00:00 2001 From: Larry Ewing Date: Fri, 8 May 2026 17:53:19 -0500 Subject: [PATCH 1/2] Move breaking-change-doc scripts into scripts/ subdirectory Aligns the breaking-change-doc skill with the agentskills.io recommended layout: executable code goes in scripts/, not at the skill root. Get-VersionInfo.ps1 -> scripts/Get-VersionInfo.ps1 Build-IssueComment.ps1 -> scripts/Build-IssueComment.ps1 Updated all references: - 4 invocation paths in SKILL.md - 1 usage example comment in each script (2 total) Pure file moves preserved via git mv (similarity 100% before the in-script comment edit). No behavioral change. --- .github/skills/breaking-change-doc/SKILL.md | 8 ++++---- .../{ => scripts}/Build-IssueComment.ps1 | 2 +- .../breaking-change-doc/{ => scripts}/Get-VersionInfo.ps1 | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) rename .github/skills/breaking-change-doc/{ => scripts}/Build-IssueComment.ps1 (97%) rename .github/skills/breaking-change-doc/{ => scripts}/Get-VersionInfo.ps1 (98%) diff --git a/.github/skills/breaking-change-doc/SKILL.md b/.github/skills/breaking-change-doc/SKILL.md index ced83b253d7e50..0f2e78469ca2d0 100644 --- a/.github/skills/breaking-change-doc/SKILL.md +++ b/.github/skills/breaking-change-doc/SKILL.md @@ -44,8 +44,8 @@ migration guidance. This skill automates that process: | `.github/workflows/breaking-change-doc.md` | gh-aw workflow — triggers on PR merge/label | | `.github/workflows/breaking-change-doc.lock.yml` | Compiled workflow (generated by `gh aw compile`) | | `.github/skills/breaking-change-doc/SKILL.md` | This skill | -| `.github/skills/breaking-change-doc/Get-VersionInfo.ps1` | `gh` CLI-based .NET version detection | -| `.github/skills/breaking-change-doc/Build-IssueComment.ps1` | Builds PR comment with URL-encoded issue creation link | +| `.github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1` | `gh` CLI-based .NET version detection | +| `.github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1` | Builds PR comment with URL-encoded issue creation link | --- @@ -102,7 +102,7 @@ include the actual area label in the description. Run the helper script to determine the .NET version context: ``` -pwsh .github/skills/breaking-change-doc/Get-VersionInfo.ps1 -PrNumber +pwsh .github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 -PrNumber ``` **You MUST display the complete script output.** The script outputs JSON that includes: @@ -239,7 +239,7 @@ Pass the PR assignees so they are `@`-mentioned in the comment and receive a GitHub notification: ```bash -pwsh .github/skills/breaking-change-doc/Build-IssueComment.ps1 \ +pwsh .github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 \ -IssueDraftPath artifacts/docs/breakingChanges/issue-draft.md \ -Title "" \ -Assignees "@user1 @user2" \ diff --git a/.github/skills/breaking-change-doc/Build-IssueComment.ps1 b/.github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 similarity index 97% rename from .github/skills/breaking-change-doc/Build-IssueComment.ps1 rename to .github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 index 6fb00936c92ab8..361419244d1b7e 100644 --- a/.github/skills/breaking-change-doc/Build-IssueComment.ps1 +++ b/.github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 @@ -7,7 +7,7 @@ # - An email reminder # # Usage: -# pwsh .github/skills/breaking-change-doc/Build-IssueComment.ps1 ` +# pwsh .github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 ` # -IssueDraftPath issue-draft.md ` # -Title "[Breaking change]: Something changed" ` # -Assignees "@user1 @user2" ` diff --git a/.github/skills/breaking-change-doc/Get-VersionInfo.ps1 b/.github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 similarity index 98% rename from .github/skills/breaking-change-doc/Get-VersionInfo.ps1 rename to .github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 index 76332081aa3d38..7004ae77200bb1 100644 --- a/.github/skills/breaking-change-doc/Get-VersionInfo.ps1 +++ b/.github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 @@ -2,7 +2,7 @@ # Determines the .NET version context for a merged PR using the GitHub CLI (gh). # # Usage: -# pwsh .github/skills/breaking-change-doc/Get-VersionInfo.ps1 -PrNumber 114929 +# pwsh .github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 -PrNumber 114929 # # Output: JSON object with LastTagBeforeMerge, FirstTagWithChange, EstimatedVersion From 6324891b60a29fcca2c5d2b6790521099c879895 Mon Sep 17 00:00:00 2001 From: Larry Ewing Date: Fri, 8 May 2026 18:17:09 -0500 Subject: [PATCH 2/2] Use bare skill-relative script paths to match the spec example The agentskills.io spec shows skill-relative paths (e.g. scripts/foo.py) rather than repo-relative ones. Switch the two pwsh invocation examples in SKILL.md to that form. The 'files in this skill' table on lines 46-48 keeps repo-relative paths because it also lists files outside the skill (workflow files), and mixing styles in one table would be jarring. --- .github/skills/breaking-change-doc/SKILL.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/skills/breaking-change-doc/SKILL.md b/.github/skills/breaking-change-doc/SKILL.md index 0f2e78469ca2d0..2208ee72979290 100644 --- a/.github/skills/breaking-change-doc/SKILL.md +++ b/.github/skills/breaking-change-doc/SKILL.md @@ -102,7 +102,7 @@ include the actual area label in the description. Run the helper script to determine the .NET version context: ``` -pwsh .github/skills/breaking-change-doc/scripts/Get-VersionInfo.ps1 -PrNumber +pwsh scripts/Get-VersionInfo.ps1 -PrNumber ``` **You MUST display the complete script output.** The script outputs JSON that includes: @@ -239,7 +239,7 @@ Pass the PR assignees so they are `@`-mentioned in the comment and receive a GitHub notification: ```bash -pwsh .github/skills/breaking-change-doc/scripts/Build-IssueComment.ps1 \ +pwsh scripts/Build-IssueComment.ps1 \ -IssueDraftPath artifacts/docs/breakingChanges/issue-draft.md \ -Title "" \ -Assignees "@user1 @user2" \