Skip to content

feat(setup): install skill stub to .agents/skills/ for multi-agent support#1104

Merged
mergify[bot] merged 1 commit intomainfrom
devs/jd/worktree-slack-other-agents/Icbd17bb8b0291c911b9014d76e12a1927355534b
Apr 10, 2026
Merged

feat(setup): install skill stub to .agents/skills/ for multi-agent support#1104
mergify[bot] merged 1 commit intomainfrom
devs/jd/worktree-slack-other-agents/Icbd17bb8b0291c911b9014d76e12a1927355534b

Conversation

@jd
Copy link
Copy Markdown
Member

@jd jd commented Apr 9, 2026

Move the canonical SKILL.md stub from .claude/skills/ to .agents/skills/
which is discoverable by Cursor, Copilot, Cline, and other coding agents.
Claude Code gets a relative symlink from .claude/skills/.

Existing real files at .claude/skills/ are automatically migrated to
symlinks on next setup.

Copilot AI review requested due to automatic review settings April 9, 2026 15:22
@mergify mergify bot had a problem deploying to Mergify Merge Protections April 9, 2026 15:22 Failure
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 9, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success=ci-gate

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify mergify bot requested a review from a team April 9, 2026 15:24
@jd jd marked this pull request as ready for review April 9, 2026 15:26
…pport

Move the canonical SKILL.md stub from .claude/skills/ to .agents/skills/
which is discoverable by Cursor, Copilot, Cline, and other coding agents.
Claude Code gets a relative symlink from .claude/skills/.

Existing real files at .claude/skills/ are automatically migrated to
symlinks on next setup.

Change-Id: Icbd17bb8b0291c911b9014d76e12a1927355534b
Claude-Session-Id: 2801865e-cacd-4237-ab2d-ba1144939a49
@jd jd force-pushed the devs/jd/worktree-slack-other-agents/Icbd17bb8b0291c911b9014d76e12a1927355534b branch from e0e3506 to 7a38587 Compare April 9, 2026 15:27
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 canonical Mergify stack skill stub to the agent-agnostic .agents/skills/ location, while keeping Claude Code compatibility via a .claude/skills/ symlink created during setup.

Changes:

  • Update skill stub install paths from .claude/skills/ to .agents/skills/ (project + global).
  • Add symlink installation/migration logic to keep .claude/skills/.../SKILL.md pointing at the canonical stub.
  • Extend CLI status output and tests to cover symlink installation and migration scenarios.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
mergify_cli/stack/setup.py Switch stub location to .agents/skills/, add symlink install + status reporting, and wire into setup/status flows.
mergify_cli/stack/cli.py Update status output and --global help text to reflect .agents/skills/ plus Claude symlink status.
mergify_cli/tests/stack/test_skill.py Update stub path assertions and add symlink/migration/idempotency tests.
Comments suppressed due to low confidence (1)

mergify_cli/stack/setup.py:470

  • The _get_claude_hooks_status() docstring says the return contains only scripts/settings/skill_stub, but the function now also returns skill_symlink (and the settings keys are named settings_installed/settings_path). Please update the docstring so it matches the actual returned structure used by the CLI status printer.
    """Get detailed status of Claude hooks for display.

    Returns:
        Dictionary with 'scripts', 'settings', and 'skill_stub' status info.
    """

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

@mergify mergify bot deployed to Mergify Merge Protections April 9, 2026 15:27 Active
@mergify mergify bot requested a review from a team April 9, 2026 15:29
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 10, 2026

Merge Queue Status

  • Entered queue2026-04-10 07:54 UTC · Rule: default
  • Checks skipped · PR is already up-to-date
  • Merged2026-04-10 07:54 UTC · at 7a38587a794dab6a2f7c8e814564c2726146c1f3

This pull request spent 11 seconds in the queue, including 2 seconds running CI.

Required conditions to merge

@mergify mergify bot merged commit 5838f30 into main Apr 10, 2026
11 checks passed
@mergify mergify bot deleted the devs/jd/worktree-slack-other-agents/Icbd17bb8b0291c911b9014d76e12a1927355534b branch April 10, 2026 07:54
@mergify mergify bot added queued and removed queued labels Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants