Repository navigation
Adopting a repo: one AGENTS.md; release 0.2.3 - #10
Marius1311 wants to merge 1 commit into
Conversation
…; release 0.2.3 Claude Code reads AGENTS.md only when there is no CLAUDE.md, so an AGENTS.md added next to one goes unread. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b853573bd2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
| 6. The repo's instructions live in one `AGENTS.md`: rename a `CLAUDE.md` to it, or merge one in, since Claude Code | ||
| reads `AGENTS.md` only when there is no `CLAUDE.md`. In it, one short section naming the plugin and where the |
There was a problem hiding this comment.
When an adopter uses Claude Code before v2.1.277, has a CLAUDE.local.md, or launches beneath another project-level CLAUDE.md, replacing the repository's CLAUDE.md with only AGENTS.md silently leaves the new dataset and working-object instructions unloaded. Claude's instruction-loading documentation identifies these version and file-presence limitations, but this plugin declares no minimum Claude version and the checklist does not ask adopters to audit the other blockers; retain an @AGENTS.md loader or explicitly cover/version-gate these cases.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Agreed that the import must stay. Rather than fix the step, I am closing this PR: the instruction-file layout (AGENTS.md plus a CLAUDE.md importing it) belongs to analysis_template and the marketplace conventions (quadbio/claude-plugins#14), so this checklist should not restate it. Step 6 stays as on main.
|
Closing: the instruction-file layout belongs to analysis_template and quadbio/claude-plugins#14; restating it here would make a second source. |
1 participant
The adoption checklist said to edit
AGENTS.mdbut never covered a repo that has aCLAUDE.md. Claude Code then reads only theCLAUDE.md(docs), so anAGENTS.mdadded next to it goes unread. Step 6 now says to rename theCLAUDE.mdtoAGENTS.mdor merge it in, as quadbio/claude-plugins#13 asks. quadbio/analysis_template#19 drops the template'sCLAUDE.md, which only importedAGENTS.md.Releases 0.2.3.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5