Conversation
xl-openai
force-pushed
the
xl/plugins
branch
2 times, most recently
from
e68bb01 to
ff0b16c
Compare
| - `collaborationMode/list` — list available collaboration mode presets (experimental, no pagination). This response omits built-in developer instructions; clients should either pass `settings.developer_instructions: null` when setting a mode to use Codex's built-in instructions, or provide their own instructions explicitly. | ||
| - `skills/list` — list skills for one or more `cwd` values (optional `forceReload`). | ||
| - `plugin/list` — list discovered plugin marketplaces and plugin state, including effective marketplace install/auth policy metadata. `interface.category` uses the marketplace category when present; otherwise it falls back to the plugin manifest category. Pass `forceRemoteSync: true` to refresh curated plugin state before listing (**under development; do not call from production clients yet**). | ||
| - `plugin/list` — list discovered plugin marketplaces and plugin state, including marketplace-level `displayName` plus effective marketplace install/auth policy metadata. Plugin `interface.category` uses the marketplace category when present; otherwise it falls back to the plugin manifest category. Pass `forceRemoteSync: true` to refresh curated plugin state before listing (**under development; do not call from production clients yet**). |
There was a problem hiding this comment.
do we really need this in the readme? seems a bit too schema-specific, up to you
| } | ||
| validate_plugin_segment(&plugin_version, "plugin version") | ||
| .map_err(PluginStoreError::Invalid)?; | ||
| let disallowed_skill_dirs = disallowed_skill_dirs(source_path.as_path(), product)?; |
There was a problem hiding this comment.
do we need to also key the install cache by product now? it seems like a ChatGPT or API install could be overwritten by a later Codex refresh/sync, so the cache would serve the wrong values. LMK if im misunderstanding
xl-openai
changed the title
|
Repurpose to support add marketplace display names to plugin/list only as product-aware plugin support requires more thinking. |
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
2 participants
Add display_name support to marketplace.json.