Add .codex-plugin/plugin.json manifests for Codex CLI plugin install#726
Merged
AbhitejJohn merged 1 commit intoJun 9, 2026
Merged
Conversation
AbhitejJohn
requested review from
a team,
JanKrivanek,
ManishJayaswal,
Redth,
ViktorHofer,
dbreshears,
jfversluis and
timheuer
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds Codex CLI–required per-plugin manifest entry points (.codex-plugin/plugin.json) across all plugins so codex plugin add <plugin>@dotnet-agent-skills can install successfully, and updates the Codex marketplace manifest to include missing plugin entries.
Changes:
- Added
plugins/<plugin>/.codex-plugin/plugin.jsonfor all 14 plugin directories to satisfy Codex CLI’s manifest discovery expectations. - Updated
.agents/plugins/marketplace.jsonto adddotnet-blazoranddotnet11entries, keeping the Codex marketplace list aligned with the other marketplace manifests.
Show a summary per file
| File | Description |
|---|---|
| plugins/dotnet/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet. |
| plugins/dotnet-ai/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-ai. |
| plugins/dotnet-aspnet/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-aspnet. |
| plugins/dotnet-blazor/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-blazor. |
| plugins/dotnet-data/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-data. |
| plugins/dotnet-diag/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-diag. |
| plugins/dotnet-experimental/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-experimental. |
| plugins/dotnet-maui/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-maui. |
| plugins/dotnet-msbuild/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-msbuild (includes MCP server config). |
| plugins/dotnet-nuget/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-nuget. |
| plugins/dotnet-template-engine/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-template-engine. |
| plugins/dotnet-test/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-test (includes agents list). |
| plugins/dotnet-upgrade/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet-upgrade. |
| plugins/dotnet11/.codex-plugin/plugin.json | Adds Codex plugin manifest for dotnet11. |
| .agents/plugins/marketplace.json | Adds missing dotnet-blazor and dotnet11 marketplace entries for Codex discovery parity. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 15/15 changed files
- Comments generated: 14
Member
|
Note that skill-validator validates the plugin.json schema but nothing does that for this newly introduced code-plugin variant. Not a big deal I guess but I'm still a little split on how far we should go in regards to supporting clients that don't support the marketplace + plugin schema. |
Member
I would go minimal support/effort so no validation or eval of quality for these. We would wait for user bug/suggestion reports |
The Codex CLI requires .codex-plugin/plugin.json as the plugin manifest entry point. Without it, 'codex plugin add' fails with 'missing plugin.json' even though the marketplace listing works. This adds .codex-plugin/plugin.json to all 14 plugin directories, with paths relative to the plugin root per the Codex docs. Also updates the agents marketplace to use dotnet-aspnetcore (per #711 rename) and adds missing dotnet-blazor and dotnet11 entries. Fixes #578 Fixes #724 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
AbhitejJohn
force-pushed
the
abhitejjohn/fix-plugin-install-missing-plugin-json
branch
from
557d8d0 to
b6d58f0
Compare
ViktorHofer
approved these changes
Jun 9, 2026
AbhitejJohn
deleted the
abhitejjohn/fix-plugin-install-missing-plugin-json
branch
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
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.
Summary
The Codex CLI requires
.codex-plugin/plugin.jsonas the plugin manifest entry point (docs). Without it,codex plugin addfails withmissing plugin.jsoneven though marketplace listing works fine.This adds
.codex-plugin/plugin.jsonto all 14 plugin directories, with paths relative to the plugin root per the Codex documentation. Also adds missingdotnet-blazoranddotnet11entries to.agents/plugins/marketplace.jsonfor parity with the other marketplace files.What changed
plugins/<name>/.codex-plugin/plugin.jsonfor every plugin.agents/plugins/marketplace.json— addeddotnet-blazoranddotnet11entriesUseful Context
PR #555 added the
.agents/plugins/marketplace.jsonenabling Codex to discover plugins. But each plugin directory still only had a root-levelplugin.json(used by Claude Code and Cursor). The Codex CLI specifically looks for.codex-plugin/plugin.jsoninside the plugin directory pointed to by the marketplacesourcepath.Validation
.codex-plugin/plugin.json"./prefix relative to plugin root per docs: "Keep manifest paths relative to the plugin root and start them with./"Fixes #578
Fixes #724