Skip to content

Route copied StructId skill files to .agents/skills#113

Merged
kzu merged 1 commit intomainfrom
copilot/update-skill-md-directory
May 8, 2026
Merged

Route copied StructId skill files to .agents/skills#113
kzu merged 1 commit intomainfrom
copilot/update-skill-md-directory

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 8, 2026

The package was copying SKILL.md into .github/skills, which does not match the expected skill root for consuming repositories. This updates the packaged MSBuild target to place the file under .agents/skills instead.

  • Copy target path

    • Changed CopyStructIdSkill to emit SKILL.md to .agents/skills/structid/SKILL.md
    • Preserved the existing repo root detection and opt-out behavior
  • Target documentation

    • Updated the inline target comment so the documented destination matches the actual copy path
<Copy SourceFiles="$(MSBuildThisFileDirectory)..\skills\structid\SKILL.md"
      DestinationFiles="$(_StructIdSkillRepoRoot).agents\skills\structid\SKILL.md"
      SkipUnchangedFiles="true"
      Condition="'$(_StructIdSkillRepoRoot)' != '' and Exists('$(MSBuildThisFileDirectory)..\skills\structid\SKILL.md')"
      ContinueOnError="true" />

Copilot AI assigned Copilot and kzu May 8, 2026
@kzu kzu marked this pull request as ready for review May 8, 2026 20:34
@kzu kzu enabled auto-merge (rebase) May 8, 2026 20:34
@kzu
Copy link
Copy Markdown
Member

kzu commented May 8, 2026

81 passed 81 passed
81 passed 81 passed
81 passed 81 passed

🧪 Details on macOS Unix 15.7.4
🧪 Details on Ubuntu 24.04.4 LTS
🧪 Details on Microsoft Windows 10.0.26100

from retest v1.1.0 on .NET 10.0.7 with 💜 by @devlooped

@kzu kzu merged commit 565e526 into main May 8, 2026
6 checks passed
@kzu kzu deleted the copilot/update-skill-md-directory branch May 8, 2026 20:35
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.

2 participants