Skip to content

Replace local ci-analysis skill with arcade-skills plugin#53526

Open
ellahathaway wants to merge 2 commits intodotnet:mainfrom
ellahathaway:add-sdk-skills
Open

Replace local ci-analysis skill with arcade-skills plugin#53526
ellahathaway wants to merge 2 commits intodotnet:mainfrom
ellahathaway:add-sdk-skills

Conversation

@ellahathaway
Copy link
Member

@ellahathaway ellahathaway commented Mar 18, 2026

Closes #53525

  • Remove the local ci-analysis skill from .claude/skills/
  • Configure the dotnet/arcade-skills plugin in .github/copilot/settings.json.
  • Configure the dotnet/arcade-skills plugin in .ckaude/settings.json.

Remove the local ci-analysis skill from .claude/skills/ and configure
the dotnet/arcade-skills plugin in .github/copilot/settings.json.
Add .claude/rules/setup.md with plugin installation instructions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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 migrates the repo’s AI CI-analysis capabilities from a locally maintained Claude skill to the shared dotnet/arcade-skills plugin, aligning with the goal of making skill consumption the default and reducing duplication across dotnet repos.

Changes:

  • Added GitHub Copilot plugin configuration to consume dotnet/arcade-skills.
  • Removed the local .claude/skills/ci-analysis skill (script + reference docs + skill metadata).
  • Added a Claude setup rule document describing how to install/use the arcade-skills plugin.

Reviewed changes

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

Show a summary per file
File Description
.github/copilot/settings.json Adds marketplace + enabled plugin mapping for dotnet/arcade-skills.
.claude/rules/setup.md Documents plugin installation steps for accessing shared skills (e.g., CI analysis).
.claude/skills/ci-analysis/scripts/Get-CIStatus.ps1 Removes locally hosted CI analysis script (replaced by plugin).
.claude/skills/ci-analysis/references/sql-tracking.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/manual-investigation.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/helix-artifacts.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/delegation-patterns.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/build-progression-analysis.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/binlog-comparison.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/azure-cli.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/references/azdo-helix-reference.md Removes local reference documentation (now expected to come from plugin).
.claude/skills/ci-analysis/SKILL.md Removes local skill definition (replaced by plugin skill).

You can also share your feedback on Copilot code review. Take the survey.

Move arcade-skills marketplace and plugin configuration from manual
setup instructions in .claude/rules/setup.md to .claude/settings.json,
enabling automatic discovery by Claude Code.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Automate dotnet/arcade-skills consumption in the SDK

3 participants