Skip to content

Fix Claude plugin marketplace and agent discovery - #2272

Open
Brett Wilson (MSBrett) wants to merge 3 commits into
devfrom
MSBrett/fix-claude-plugin-schema
Open

Fix Claude plugin marketplace and agent discovery#2272
Brett Wilson (MSBrett) wants to merge 3 commits into
devfrom
MSBrett/fix-claude-plugin-schema

Conversation

@MSBrett

@MSBrett Brett Wilson (MSBrett) commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

🛠️ Description

Correct the Claude Code marketplace source so it resolves to the repository's actual plugin root, enable strict marketplace validation, and load the shared agents through Claude's standard agents/ auto-discovery. Agent files now use plain .md names so Claude exposes the intended identifiers without an .agent suffix. The MCP configuration remains declared explicitly.

Validated against Anthropic's current documentation:

A clean installation from this PR's online branch with Claude Code CLI 2.1.235 reports one skill, five correctly named agents, and one MCP server.

Fixes # N/A

📷 Screenshots

N/A

📋 Checklist

🔬 How did you test this change?

  • 🤏 Lint tests
  • 🤞 PS -WhatIf / az validate
  • 👍 Manually deployed + verified
  • 💪 Unit tests
  • 🙌 Integration tests

Additional validation:

  • claude plugin validate . --strict
  • claude plugin validate src/templates/agent-plugin --strict
  • Clean online-branch marketplace installation and component inventory with Claude Code CLI 2.1.235

📦 Deploy to test?

  • Hubs + ADX (managed)
  • Hubs + Fabric (manual) — URI:
  • Hubs (manual)
  • Hubs (no data)
  • Workbooks
  • Alerts

🙋‍♀️ Do any of the following that apply?

  • 🚨 This is a breaking change.
  • 🤏 The change is less than 20 lines of code.

📑 Did you update docs/changelog.md?

  • ✅ Updated changelog (required for dev PRs)
  • ➡️ Will add log in a future PR (feature branch PRs only)
  • ❎ Log not needed (small/internal change)

📖 Did you update documentation?

  • ✅ Public docs in docs (required for dev)
  • ✅ Public docs in docs-mslearn (required for dev)
  • ✅ Internal dev docs in docs-wiki (required for dev)
  • ✅ Internal dev docs in src (required for dev)
  • ➡️ Will add docs in a future PR (feature branch PRs only)
  • ❎ Docs not needed (small/internal change)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings August 20, 2026 21:16
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Review 👀 PR that is ready to be reviewed label Aug 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the Claude Code plugin packaging for the FinOps Toolkit so Claude’s marketplace entry points at the actual plugin root in this repo, and the Claude-specific manifest explicitly declares agents and MCP configuration (with strict mode enabled to keep the manifest authoritative).

Changes:

  • Declare Claude plugin agents and mcpServers in src/templates/agent-plugin/.claude-plugin/plugin.json.
  • Update PowerShell unit tests to validate Claude manifest paths/components and enforce strict mode in the Claude marketplace entry.
  • Fix the Claude marketplace source path to the repo’s real plugin root for Claude (./src/templates/agent-plugin) and enable strict: true.

Reviewed changes

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

File Description
src/templates/agent-plugin/.claude-plugin/plugin.json Adds explicit Claude manifest fields for agents and MCP servers, aligning with supported schema fields.
src/powershell/Tests/Unit/AgentPlugins.Tests.ps1 Updates/extends Pester tests to validate Claude manifest component declarations and strict mode configuration.
.claude-plugin/marketplace.json Points Claude marketplace source at the correct plugin root and enables strict mode for the entry.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 3 out of 3 changed files in this pull request and generated no new comments.

@MSBrett Brett Wilson (MSBrett) added this to the v15 milestone Aug 20, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@MSBrett Brett Wilson (MSBrett) changed the title Fix Claude plugin schema Fix Claude plugin marketplace and agent discovery Aug 21, 2026
@MSBrett
Brett Wilson (MSBrett) enabled auto-merge (squash) August 22, 2026 04:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 3 out of 8 changed files in this pull request and generated 1 comment.

Comment thread src/powershell/Tests/Unit/AgentPlugins.Tests.ps1
Co-authored-by: Copilot App <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

Needs: Review 👀 PR that is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants