Remove the misspelled unreleased-dependancy skill; point at the marketplace - #8
Open
ebellamy-bh wants to merge 1 commit into
Open
Remove the misspelled unreleased-dependancy skill; point at the marketplace#8ebellamy-bh wants to merge 1 commit into
ebellamy-bh wants to merge 1 commit into
Conversation
…tplace The directory name `unreleased-dependancy` did not match the frontmatter `name: unreleased-dependency`, which the Agent Skills spec requires, so the skill silently never loaded. The merged, maintained version now lives in mieweb/agent-skills (PR #4 there) and ships through its plugin marketplace. copilot-instructions gains a short "Shared Agent Skills" section: do not copy skills into repos created from this template; MIE staff install the marketplace once at user level; repo-local skills must keep directory and frontmatter name equal. Refs mieweb/agent-skills#3. AI-Agent: GitHub Copilot / claude-fable-5.1 AI-Sessions: 1 Atlas-Layer: governance Work-Type: maintenance
There was a problem hiding this comment.
Pull request overview
Removes a misspelled, non-loading repo-local Agent Skill from the template and updates the template’s Copilot instructions to point contributors toward the shared mieweb/agent-skills marketplace approach instead.
Changes:
- Deleted the
.github/skills/unreleased-dependancy/skill content that would never load due to the directory/frontmatter name mismatch. - Added a “Shared Agent Skills” section to
.github/copilot-instructions.mdexplaining marketplace usage and the directory-name == frontmatter-name requirement for repo-local skills.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/skills/unreleased-dependancy/SKILL.md | Removes the misspelled, non-functional skill from the template. |
| .github/copilot-instructions.md | Documents the recommended shared marketplace approach and naming requirements for repo-local skills. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+101
to
+103
| - **Do not copy skills into this repo.** Reusable engineering skills (commit attribution, unreleased-dependency vendoring, package weight gating, …) live in the `mieweb/agent-skills` plugin marketplace and are catalogued at https://skills.mieweb.org. | ||
| - **MIE staff**: add `"chat.plugins.marketplaces": ["mieweb/agent-skills"]` to your VS Code user settings once; the `mieweb` plugin then follows you into every repository and updates itself. | ||
| - Repository-specific skills belong in `.github/skills/<name>/SKILL.md`, with the directory name equal to the frontmatter `name` — a mismatch makes the skill silently fail to load. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of mieweb/agent-skills#3.
Removes
.github/skills/unreleased-dependancy/. The directory name never matched the frontmattername: unreleased-dependency, so per the Agent Skills spec the skill silently never loaded in any repo created from this template. The merged, maintained version is now inmieweb/agent-skills(their #4, with hooks, CI and cleanup scripts) and ships via its plugin marketplace (their #8).Adds a "Shared Agent Skills" section to
copilot-instructions.md— three lines: don't copy skills into template-derived repos; MIE staff install themieweb/agent-skillsmarketplace once at user level so it follows them everywhere; repo-local skills must keep directory name == frontmattername.Deliberately not added: a
.github/copilot/settings.jsonrecommending the marketplace. This template is public and the marketplace is private; a workspace recommendation would prompt every external user of the template to install something they can't access. The user-level setting covers MIE staff across all repos without that cost. If the marketplace ever goes public, the recommendation can be added then.