Package revdiff skills for Codex marketplace installs - #349
Merged
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The changes are low-risk packaging/documentation updates with CI validation added; the remaining feedback is minor and non-blocking.
Pull request overview
Packages the Codex-specific revdiff skills as a marketplace-installable Codex plugin (while keeping revdiff-planning as the separate automatic Plan-mode plugin), and updates documentation + CI checks to match the new install flow.
Changes:
- Add a Codex plugin manifest under
plugins/codex/.codex-plugin/and a Codex marketplace mapping in.agents/plugins/marketplace.json. - Update docs (README + site + Codex plugin docs) to use
codex plugin marketplace add/codex plugin addinstalls instead of manual skill copying. - Add CI validation to ensure Codex marketplace entries, local sources, and plugin manifests stay aligned.
File summaries
| File | Description |
|---|---|
| site/llms.txt | Updates Codex integration description and install guidance. |
| site/index.html | Switches Codex install instructions to marketplace/plugin commands. |
| site/docs.html | Updates Codex plugin install section to marketplace-based installs and clarifies plugin split. |
| README.md | Makes codex plugin add revdiff@revdiff the canonical install for manual skills; clarifies revdiff-planning. |
| plugins/codex/skills/revdiff/SKILL.md | Updates script-path resolution guidance to derive from installed plugin root. |
| plugins/codex/skills/revdiff/references/install.md | Documents marketplace-based Codex plugin installation. |
| plugins/codex/skills/revdiff-plan/SKILL.md | Updates script-path resolution guidance to derive from installed plugin root. |
| plugins/codex/README.md | Updates installation instructions and documents new path-resolution approach. |
| plugins/codex/.codex-plugin/plugin.json | Adds the Codex plugin manifest exposing skills from ./skills/. |
| CLAUDE.md | Updates internal contributor docs for Codex plugin packaging + marketplace metadata location. |
| .github/workflows/ci.yml | Adds a CI step to validate Codex marketplace/manifests. |
| .github/scripts/validate-codex-marketplace.py | New validator ensuring Codex marketplace entries match local plugin manifests + packaged files. |
| .agents/plugins/marketplace.json | New Codex marketplace mapping of revdiff → plugins/codex and revdiff-planning → planning plugin. |
Review details
- Files reviewed: 13/13 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
paskal
force-pushed
the
package-codex-skills
branch
from
September 1, 2026 22:04
6a4d7d4 to
21f309f
Compare
Document the four plugin manifests that must stay aligned with their marketplace entries. Remove unsupported `allowed-tools` frontmatter from both Codex skills and tell manual-install users to remove the superseded skill directories after plugin installation.
Add manual-skill cleanup to the website install section so every published installation surface gives the same migration instruction.
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.
Previously, Codex used the shared Claude marketplace entry, so
revdiff@revdiffexposed the Claude skill while the maintainedrevdiffandrevdiff-planCodex skills required manual installation.After this change,
.agents/plugins/marketplace.jsonmapsrevdifftoplugins/codexand preservesrevdiff-planningas the separate automatic Plan-mode plugin. Both skills resolve their bundled scripts from the installed plugin root.With Codex 0.152.0,
codex plugin add revdiff@revdiffexposes both skills and resolves both bundled launchers from the plugin cache. CI pins the exact plugin set, source paths, versions, and packaged skill files.Resolves #348.