fix: root plugin path#19
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a dedicated Codex plugin bundle directory and updates the agent marketplace config to reference that new plugin root instead of the repository root.
Changes:
- Added
plugins/make-skills-codex/.codex-plugin/plugin.jsonas a Codex plugin manifest within the new plugin bundle directory. - Updated
.agents/plugins/marketplace.jsonto install the plugin from./plugins/make-skills-codex.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
plugins/make-skills-codex/.codex-plugin/plugin.json |
Introduces a Codex manifest colocated with the plugin bundle content under plugins/. |
.agents/plugins/marketplace.json |
Points the local marketplace install path at the new plugin bundle directory. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { | ||
| "name": "make-skills", | ||
| "version": "0.1.3", | ||
| "description": "Expert skills for designing, building, and deploying Make.com automation scenarios", |
There was a problem hiding this comment.
This plugin manifest appears to be a byte-for-byte duplicate of the existing root .codex-plugin/plugin.json. Having two manifests with independent version fields is likely to drift (e.g., release tooling currently bumps .codex-plugin/plugin.json per .versionrc.json, but won’t touch this new file), which can lead to inconsistent plugin metadata depending on which path is used. Consider making a single source of truth (remove one copy, use a symlink during packaging, or update the release/version-bump process to keep both in sync).
No description provided.