refactor(stack): remove mergify stack skill command#1133
Merged
mergify[bot] merged 2 commits intomainfrom Apr 10, 2026
Merged
Conversation
Add `npx skills add Mergifyio/mergify-cli` as the primary installation method for AI agent skills, since it works across 40+ agents (Claude Code, Cursor, etc.) via the skills.sh ecosystem. Keep Claude Code plugin instructions as an alternative. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Change-Id: I334783fb9aca8bab59b18ab9445825cc81db9061 Claude-Session-Id: 10686d4d-ed4a-45e8-b02b-4f7e9dbcb277
Skills are now installed via `npx skills add Mergifyio/mergify-cli` or the Claude Code plugin marketplace. The `mergify stack skill` command that dumped skill content to stdout is no longer needed. - Remove `skill` subcommand from `mergify stack` CLI - Remove `mergify_cli/stack/skill.py` and `mergify_cli/stack/skill.md` - Replace symlink in `skills/mergify-stack/SKILL.md` with actual file - Remove `test_skill.py` tests Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Change-Id: I65b803f77d2baa13ac02cf09fd47b01439eecae4
Member
Author
Contributor
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🟢 ⛓️ Depends-On RequirementsWonderful, this rule succeeded.Requirement based on the presence of
🟢 🤖 Continuous IntegrationWonderful, this rule succeeded.
🟢 👀 Review RequirementsWonderful, this rule succeeded.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 ReviewsWonderful, this rule succeeded.
🟢 📕 PR descriptionWonderful, this rule succeeded.
|
There was a problem hiding this comment.
Pull request overview
Removes the now-unneeded mergify stack skill CLI subcommand (which previously printed the embedded skill markdown) now that skills are installed via npx skills add … or the Claude Code plugin marketplace.
Changes:
- Drop the
skillsubcommand frommergify stackand its associated implementation. - Remove the packaged
skill.mdresource and its loader helper. - Remove tests that validated the old embedded skill content.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
mergify_cli/stack/cli.py |
Removes the stack skill click command and the related import. |
mergify_cli/stack/skill.py |
Deletes the helper that read the packaged skill.md resource. |
mergify_cli/stack/skill.md |
Deletes the embedded skill markdown that was printed by the CLI. |
mergify_cli/tests/stack/test_skill.py |
Removes tests that exercised the deleted skill loader/content. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Base automatically changed from
devs/jd/worktree-skill-npx/I334783fb9aca8bab59b18ab9445825cc81db9061
to
main
April 10, 2026 15:12
JulianMaurin
approved these changes
Apr 10, 2026
kozlek
approved these changes
Apr 10, 2026
Contributor
Merge Queue Status
This pull request spent 4 minutes 48 seconds in the queue, including 4 minutes 32 seconds running CI. Required conditions to merge
|
36 tasks
mergify
bot
deleted the
devs/jd/worktree-skill-npx/I65b803f77d2baa13ac02cf09fd47b01439eecae4
branch
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.
Skills are now installed via
npx skills add Mergifyio/mergify-clior the Claude Code plugin marketplace. The
mergify stack skillcommand that dumped skill content to stdout is no longer needed.
skillsubcommand frommergify stackCLImergify_cli/stack/skill.pyandmergify_cli/stack/skill.mdskills/mergify-stack/SKILL.mdwith actual filetest_skill.pytestsCo-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com
Depends-On: #1120