chore: drop CLAUDE.md import stubs in favour of AGENTS.md - #119
Merged
Merged
Conversation
Claude Code reads AGENTS.md directly (v2.1.277+), including nested AGENTS.md loaded on demand when it opens files in a subdirectory, so the one-line `@AGENTS.md` stubs are no longer needed. Removes the four stubs and rewrites every remaining CLAUDE.md reference — headings, cross-links and the PR-review workflow prompt — to AGENTS.md. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Codecov Report✅ All modified and coverable lines are covered by tests. |
aig-hannes
approved these changes
Sep 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Claude Code reads
AGENTS.mddirectly since v2.1.277, including nestedAGENTS.mdthat load on demand when it opens a file in a subdirectory (docs). The one-line@AGENTS.mdimport stubs were a workaround for that gap and are no longer needed.Removes the four stubs (repo root,
.github/,tests/,src/aignostics_foundry_core/) and rewrites every remainingCLAUDE.mdreference toAGENTS.md: headings, the cross-links between the nested guides, and the PR-review workflow prompt. Markdown and one workflow prompt string only — no code.This mirrors aignostics/foundry-python#431, which does the same in the template this repo is generated from. Worth landing both so a later
copier updateis a no-op here rather than a conflict.Caveats worth a look before merging
CLAUDE.local.mdsilently suppressesAGENTS.md. Per the docs, aCLAUDE.local.mdcounts as a project instruction file, so Claude reads only it and skipsAGENTS.mdentirely. Today the committed stub means such a developer still gets everything through the import; afterwards they get their local file and none of the project guidance. It is gitignored and per-developer, so this is invisible from the repo. Workaround:/config→ Project instructions →claude-md-and-agents-md.AGENTS.mdand the docs recommend keeping aCLAUDE.mdimport there. Those environments, and any Claude Code older than v2.1.277, get zero project instructions once the stub is gone.Both are the maintainers' call — flagging, not blocking.
Verification
mise run lint: 0 errors.mise run test_unit: 16 passed.CLAUDE.mdreference remains anywhere in the repo (git grepreturns nothing)..mdcross-link in all fourAGENTS.mdfiles resolves to a file that exists.Unrelated pre-existing issue noticed
AGENTS.md:171andAGENTS.md:175link toCONTRIBUTING.mdandRELEASE_WORKFLOW.md, neither of which exists in this repo — broken onmaintoo. Left alone as out of scope.🤖 Generated with Claude Code