chore: migrate SynapseML skills to Copilot path#2559
Conversation
|
Hey @BrendanWalsh 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
There was a problem hiding this comment.
Pull request overview
Migrates the remaining repo-versioned SynapseML skill content to the Copilot CLI documented project-skill path (.github/skills/) while keeping .agents/ as a compatibility pointer directory for older tooling.
Changes:
- Updates the
code-reviewskill metadata and checklist wording under.github/skills/. - Adds pointer README files under
.agents/and.agents/skills/directing contributors/tools to.github/skills/. - Reinforces that
.agents/should not containSKILL.mdsources going forward.
Show a summary per file
| File | Description |
|---|---|
.github/skills/code-review/SKILL.md |
Improves skill frontmatter description and clarifies checklist phrasing/punctuation. |
.agents/skills/README.md |
Adds a pointer README telling tools/contributors where skills now live. |
.agents/README.md |
Documents that Copilot CLI skills are discovered from .github/skills/ and .agents/ is compatibility-only. |
Copilot's findings
- Files reviewed: 3/3 changed files
- Comments generated: 0
## Summary Move the remaining SynapseML repo skill from `.agents/skills/` to `.github/skills/` so Copilot CLI can discover all repo-versioned skills from the documented project-skill path. Add README pointers under `.agents/` for tools or agents that inspect the older convention. ## Prompting Intent The engineer asked to migrate everything to the correct Copilot CLI path and suggested keeping a generic agents pointer. The goal was to make existing skills discoverable by Copilot while avoiding future confusion about `.agents/skills`. ## Linked Sources - User request in current session: migrate everything to the correct path for Copilot CLI and keep a generic agents pointer. - Skill location reference: /home/brwals/.copilot/installed-plugins/copilot-toolkit-marketplace/common/skills/create-skill/references/REFERENCE.md - Prior merged skill PR: microsoft#2558 ## Rationale `.github/skills/<name>/` is the documented Copilot CLI project-skill location. Keeping only README pointers under `.agents/` preserves a breadcrumb for other agent conventions without leaving duplicate or stale `SKILL.md` files in a path Copilot CLI may not load.
79211a6 to
cfb12ce
Compare
Summary
Migrates the remaining SynapseML repo skill to the documented Copilot CLI project-skill path and leaves
.agentsas a pointer-only compatibility directory.What changed
code-reviewfrom.agents/skills/code-review/to.github/skills/code-review/..agents/README.mdto point agents at.github/skills/..agents/skills/README.mdso tools that inspect the old directory see the new source of truth.synapseml-local-setupunder.github/skills/from the prior merged skill PR.Validation
git diff --checkSKILL.mdfiles remain under.agents/.