Repository navigation
Follow analysis-workflow: AGENTS.md only, package from main - #19
Marius1311 wants to merge 2 commits into
Conversation
Claude Code reads AGENTS.md itself when there is no CLAUDE.md, so one instruction file is enough (quadbio/claude-plugins#13). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5
It no longer tags releases: the project's lock pins the commit, and `pixi update` moves it. 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: f4a5f8f9c9
ℹ️ 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".
| @@ -1,3 +0,0 @@ | |||
| # {{ cookiecutter.project_name }} agent entry point | |||
|
|
|||
| @AGENTS.md | |||
There was a problem hiding this comment.
When a generated or updated project has a CLAUDE.local.md or an ancestor project-level CLAUDE.md, Claude Code does not load its AGENTS.md by default; direct loading is also unavailable before v2.1.277. Deleting this import therefore silently drops the repository's canonical instructions—including the required analysis-workflow skill—for those sessions. Preserve the shim unless the template can enforce compatible instruction-loading configuration and a minimum Claude Code version. See the Claude Code instruction-loading rules.
AGENTS.md reference: {{cookiecutter.project_name}}/AGENTS.md:L3-L4
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Agreed, confirmed in the docs: a CLAUDE.local.md or a parent folder's CLAUDE.md hides AGENTS.md. The import stays: #19 is closed in favour of #20, which only takes analysis-workflow from main, and quadbio/claude-plugins#14 makes the import the shared rule.
|
Superseded by #20, which keeps |
1 participant
Two changes so new projects match analysis-workflow as it is now:
CLAUDE.md. It only importedAGENTS.md, which Claude Code reads by itself when there is noCLAUDE.md(docs). With one file, the template follows Write down the conventions plugins here share claude-plugins#13. research-agent-skills' project-setup skill would otherwise tell agents to merge aCLAUDE.mdintoAGENTS.mdand delete it. Linked projects lose the file on their nextcruft update.main. The tagv0.1.1was two releases old, and analysis-workflow no longer tags releases (Release 0.2.2: plugin.json holds the only version analysis-workflow#9). The line now matches its README: the project's lock pins the commit. The template uses onlymain_checkout, which is unchanged.The template CI creates a project and runs its tests and pre-commit, so it checks both changes.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5