Add .claude-plugin/marketplace.json so Claude Code can load the marketplace#1747
Open
freezscholte wants to merge 2 commits into
Open
Add .claude-plugin/marketplace.json so Claude Code can load the marketplace#1747freezscholte wants to merge 2 commits into
freezscholte wants to merge 2 commits into
Conversation
…tplace
Claude Code resolves a marketplace manifest only at
`.claude-plugin/marketplace.json`, but this repo ships it at the repo
root (`/marketplace.json`). As a result `claude plugin marketplace add
microsoft/azure-container-apps` fails with:
Marketplace file not found at .../.claude-plugin/marketplace.json
This adds the manifest at the path Claude Code expects. The repo already
follows this convention for the plugin manifest
(`plugin/.claude-plugin/plugin.json`), so this aligns the marketplace
manifest with the same convention. The root `marketplace.json` is left
untouched so GitHub Copilot CLI is unaffected.
It also drops the redundant `skills` array from the plugin entry: skill
discovery is already handled by the plugin's own manifest
(`plugin/.claude-plugin/plugin.json` -> "skills": "./skills/"). The
marketplace-level `skills` path was resolved relative to the plugin
source dir, producing a doubled `plugin/plugin/skills/aca-sandboxes` and
causing the plugin to fail to load even after the manifest was found.
Verified locally: `claude plugin marketplace add` + `install` now load
the `sandboxes` plugin and discover the `aca-sandboxes` skill (status:
enabled).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Author
|
@microsoft-github-policy-service agree |
Three documented forms hard-error on `aca 1.0.0-preview.1`; corrected to the forms the CLI actually accepts (verified live against the CLI): - egress allow-list: `--host-allow "<host>"` → `--rule "<pattern>:Allow"` (the CLI's own `egress set --help` example and `egress init` template use `--rule`; `--host-allow` returns "unexpected argument"). Fixed in the SKILL.md egress rows + the copilot scenario in scenarios.md. Added an explicit anti-cue. - sandbox labels: `--labels "k=v,k2=v2"` → `--label k=v --label k2=v2` (repeatable; the CLI suggests `--label` and rejects `--labels`). - group secrets: `secret upsert --name X --values "K=V"` → `secret set --name X --key K --value V` (there is no `upsert` subcommand; `secret` exposes list/show/set/remove/delete). The skill was internally inconsistent (one "Try asking" row already used `--rule`), which confirms drift from the shipped CLI rather than a version pin.
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
1 participant
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.
Problem
claude plugin marketplace add microsoft/azure-container-appsfails:Claude Code resolves a marketplace manifest only at
.claude-plugin/marketplace.json, but this repo ships the manifest at the repo root (/marketplace.json). So Claude Code users can't add the marketplace at all, even though the README advertises support for "coding agents (Copilot CLI, Claude Code)".Fix
Add
.claude-plugin/marketplace.json— the path Claude Code expects. The repo already uses this convention for the plugin manifest (plugin/.claude-plugin/plugin.json), so this just aligns the marketplace manifest with the same convention. The rootmarketplace.jsonis left untouched so GitHub Copilot CLI is unaffected.Drop the redundant
skillsarray from the plugin entry. Skill discovery is already handled by the plugin's own manifest (plugin/.claude-plugin/plugin.json→"skills": "./skills/"). The marketplace-levelskills: ["./plugin/skills/aca-sandboxes"]was resolved relative to the plugin source dir, producing a doubled pathplugin/plugin/skills/aca-sandboxesand causing the plugin to fail to load even after the manifest was found:Verification (local)
Note
The root
marketplace.jsoncarries the same redundant/incorrectskillsarray. I left it untouched to avoid changing Copilot CLI behavior, but maintainers may want to apply the same cleanup there.🤖 Generated with Claude Code