Fix Codex marketplace installs missing skills and MCP configuration - #307
Open
stbenjam wants to merge 1 commit into
Open
Fix Codex marketplace installs missing skills and MCP configuration#307stbenjam wants to merge 1 commit into
stbenjam wants to merge 1 commit into
Conversation
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.
Installing
link@link-pluginsfrom this repository's Codex marketplace reports success, but the installed plugin contains only the Claude and Codex manifest directories. The marketplace selects./plugins/link, whereskills,.mcp.json, andassetsare symlinks to files at the repository root. Codex skips these symlinks when copying the plugin into its installation cache.This surfaced while investigating skillsaw warnings about manifest paths resolving outside the plugin root.
Point the Codex marketplace at
./so it packages the existing root skills, MCP configuration, and assets. Set the root Codex manifest version to0.18.0, matchingpackages/cli/package.json, update the version sync script to maintain it, and remove the redundant nested Codex manifest.Before and after: installs directly from GitHub
Tested with Codex CLI 0.153.4, using a separate temporary
CODEX_HOMEfor each repository. These were fresh GitHub marketplace installs, not installs from a modified local checkout..mcp.jsonstripe/link-cli,main0.15.1stbenjam/link-cli,main0.18.0Both
plugin addcommands exited successfully. Inspected the actual installed files under each profile'splugins/cache/link-plugins/link/<version>/. The fixed installation includes:Also confirmed that the installed MCP configuration contains the
linkserver and both manifest icon paths resolve to a file.Additional validation
0.18.0without recreating the nested manifest, using a temporary copy.codex-plugin-json-valid,codex-plugin-structure,codex-marketplace-json-valid, andcodex-marketplace-registration.node --check scripts/sync-skill-version.jsandgit diff --checkpass.Validation covers installation and packaged contents; authentication and payment operations were not exercised.