Fix Claude plugin marketplace and agent discovery

[Brett Wilson (MSBrett)]

🛠️ 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:

• Plugin manifest schema
• Marketplace plugin entries and strict mode
• Repository-root-relative plugin sources
• Plugin subagent discovery

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)

[Copilot]

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]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Roland Krummenacher (RolandKrummenacher)]

Verified the premise empirically rather than by reading alone: installed both the dev and PR versions of the plugin into isolated CLAUDE_CONFIG_DIRs and compared claude plugin details, and reproduced the symlink-source failure in a scratch marketplace.

Confirmed:

• On dev, the .agent.md filenames cause Claude to derive agent ids from the filename stem and ignore frontmatter name:, so agents surface as azure-capacity-manager.agent, ftk-database-query.agent, etc. This PR's version correctly exposes 5 unsuffixed agents, 1 skill, 1 MCP server.
• The ./plugins/microsoft-finops-toolkit source is genuinely unusable on Windows checkouts (the git symlink materializes as a text file, so install fails with ENOTDIR).
• claude plugin validate --strict passes on both manifests, and the validator does reject unknown fields, so strict is a recognized marketplace field.
• All 24 Pester tests pass locally.

The fix works for Claude Code. My concern is what it costs on the other platform.

Renaming the agents breaks GitHub Copilot CLI discovery

(Left here rather than inline, since the five renames carry no diff hunks to anchor to.)

The root plugin.json exists specifically for Copilot and declares "agents": "./agents/". GitHub's plugin format requires NAME.agent.md, so dropping the suffix takes Copilot from 5 agents to 0 — while the changelog still describes the plugin as shared between Claude Code and Copilot CLI. Claude auto-discovers agents/ regardless of what the manifest says, so adding a second directory doesn't resolve this; the Copilot-suffixed variant probably needs to be generated at package time in Build-AgentPlugin.ps1.

README is now out of date

src/templates/agent-plugin/README.md:7 isn't in this PR, but it now contradicts both changes — it still says the plugin directory is "used by both marketplace manifests", and that Claude declares only its output style.

Remaining comments inline.