Skip to content

Conversation

@nshcr
Copy link
Contributor

@nshcr nshcr commented Feb 4, 2026

https://docs.github.com/en/copilot/concepts/agents/about-agent-skills

Adjust the Copilot SKILL installation path to the correct location according to the official documentation.

.github/copilot/skills/gitbutler -> .github/skills/gitbutler

Copilot AI review requested due to automatic review settings February 4, 2026 03:31
@vercel
Copy link

vercel bot commented Feb 4, 2026

@nshcr is attempting to deploy a commit to the GitButler Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the rust Pull requests that update Rust code label Feb 4, 2026
Copy link
Contributor

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

This PR fixes the GitHub Copilot skill installation path to match the official GitHub documentation. The path was incorrectly set to .github/copilot/skills/gitbutler but should be .github/skills/gitbutler according to the official GitHub Copilot agent skills documentation.

Changes:

  • Updated the GitHub Copilot skill format path constant in the skill command implementation

@Byron
Copy link
Collaborator

Byron commented Feb 4, 2026

Thanks a lot, great catch!

I validated this and… second Copilot here :D.

@Byron Byron merged commit bab4038 into gitbutlerapp:master Feb 4, 2026
28 of 30 checks passed
@nshcr
Copy link
Contributor Author

nshcr commented Feb 4, 2026

@Byron Thanks for merging this so quickly!

In fact, this PR only fixes the project-level Copilot skill installation path (.github/skills). For global installation, Copilot uses a different location (.copilot/skills), while but skill install currently doesn't provide separate path for installing to the global scope (it only appends to the home path for now).

So unfortunately, the global Copilot installation path is still incorrect. Since it involves more extensive changes, I may not be able to address it :(.

@krlvi
Copy link
Member

krlvi commented Feb 4, 2026

Thanks for this!

@krlvi
Copy link
Member

krlvi commented Feb 4, 2026

@Byron Thanks for merging this so quickly!

In fact, this PR only fixes the project-level Copilot skill installation path (.github/skills). For global installation, Copilot uses a different location (.copilot/skills), while but skill install currently doesn't provide separate path for installing to the global scope (it only appends to the home path for now).

So unfortunately, the global Copilot installation path is still incorrect. Since it involves more extensive changes, I may not be able to address it :(.

I will take care of this

@krlvi
Copy link
Member

krlvi commented Feb 4, 2026

#12192

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants