Skip to content

fix: move SKILL.md into skills/ subdirectory to align with agentskills spec#24

Merged
tomaspozo merged 1 commit intomainfrom
tomas/move-skill-to-skills-dir
Apr 1, 2026
Merged

fix: move SKILL.md into skills/ subdirectory to align with agentskills spec#24
tomaspozo merged 1 commit intomainfrom
tomas/move-skill-to-skills-dir

Conversation

@tomaspozo
Copy link
Copy Markdown
Member

Summary

  • Move SKILL.md from repo root to skills/supabase-server/SKILL.md to align with the agentskills spec
  • npx skills add was copying the entire repo contents instead of just the skill file, because the skill discovery sets path to the directory containing SKILL.md — which was the repo root
  • Update package.json files to include skills directory

Copilot AI review requested due to automatic review settings April 1, 2026 02:09
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves the agent skill definition out of the repository root into a dedicated skills/ subdirectory to align with the agentskills discovery spec and prevent tooling from treating the entire repo as the skill payload.

Changes:

  • Relocated the Supabase server skill file to skills/supabase-server/SKILL.md.
  • Updated the skill trigger description to include supabase/server mentions.
  • Updated package.json publish whitelist to include the skills directory.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
skills/supabase-server/SKILL.md Updates skill metadata/trigger text after moving the skill under skills/.
package.json Adjusts npm files whitelist to publish the skill content from skills/.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tomaspozo tomaspozo requested a review from kallebysantos April 1, 2026 02:32
@tomaspozo tomaspozo merged commit 10c8780 into main Apr 1, 2026
10 of 11 checks passed
@tomaspozo tomaspozo deleted the tomas/move-skill-to-skills-dir branch April 1, 2026 08:04
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.

3 participants