feat: add codex llmdoc helper skills#22
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the llmdoc workflow by introducing dedicated Codex helper skills, llmdoc-init and llmdoc-update, to complement Claude Code commands. It also updates the documentation and project-scoped agent configurations to support these changes. Feedback was provided to correct the relative paths in the JSON examples within the README files to ensure proper plugin resolution from the home directory.
| "name": "llmdoc", | ||
| "source": { | ||
| "source": "local", | ||
| "path": "./.codex/plugins/llmdoc" |
There was a problem hiding this comment.
The path ./.codex/plugins/llmdoc in the JSON example is likely incorrect if it is intended to be relative to the home directory as stated in line 205. In standard path resolution, ./ refers to the directory containing the configuration file (~/.agents/plugins/). To correctly reference ~/.codex/plugins/llmdoc from ~/.agents/plugins/marketplace.json, the relative path should be ../../.codex/plugins/llmdoc.
| "path": "./.codex/plugins/llmdoc" | |
| "path": "../../.codex/plugins/llmdoc" |
| "name": "llmdoc", | ||
| "source": { | ||
| "source": "local", | ||
| "path": "./.codex/plugins/llmdoc" |
There was a problem hiding this comment.
The path ./.codex/plugins/llmdoc in the JSON example is likely incorrect if it is intended to be relative to the home directory as stated in line 205. In standard path resolution, ./ refers to the directory containing the configuration file (~/.agents/plugins/). To correctly reference ~/.codex/plugins/llmdoc from ~/.agents/plugins/marketplace.json, the relative path should be ../../.codex/plugins/llmdoc.
| "path": "./.codex/plugins/llmdoc" | |
| "path": "../../.codex/plugins/llmdoc" |
1 participant
No description provided.