-
Couldn't load subscription status.
- Fork 226
Rename CLAUDE.MD to AGENTS.md to support other agentic tools #3449
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
That one is nice, if lots of agends support it. @sebthom would Copilot in https://github.com/eclipse-platform/eclipse.platform.ui/blob/master/.github/copilot-instructions.md also support this? If yes, please update the PR to use the same file, would be nice to have only one source of truth. |
|
@vogella GitHub Copilot supports AGENTS.md directly according https://docs.github.com/en/copilot/how-tos/configure-custom-instructions/add-repository-instructions It is also listed at https://agents.md/ as supporting agent. |
|
Thanks @sebthom, in this case https://github.com/eclipse-platform/eclipse.platform.ui/blob/master/.github/copilot-instructions.md can be deleted? If yes, please update the PR with this change. I moved from Copilot to Claude a few weeks ago but would like to give @laeubi the time to test this, AFAIK he is a heavy Copilot user. |
|
Btw, that for educating me here. I'm happy to see Gemini and codex also in the list of https://agents.md/, so this should cover all our user needs. Sad that Claude needs a special treatment. |
I think the file can be deleted (but I am not a copilot user). Also the files currently differ in content. What to do about that? |
IIRC I created both using the AI tools. As it was requested to streamline the documentation of the agents, I suggest to remove it and rely on the new agent file. Copilot could update it later if it need to add information to it. |
|
Btw. I think the AGENTS.md file contains some irrelevant information (such as |
It would great if you remove such information if, based on your experience, these not not helpful to the AI tools. I currently cannot ask Claude why it added these, as I'm already in my 4 hour token limit from using Claude for a Flutter app. |
Test Results 3 018 files 3 018 suites 2h 30m 45s ⏱️ Results for commit 275b8ee. ♻️ This comment has been updated with latest results. |
According to https://code.visualstudio.com/docs/copilot/customization/custom-instructions I assume the /.github/copilot-instructions.md can be deleted if you have an AGENTS.md. At least I would understand it that way |
|
LGTM, planning to merge soon |
|
Thanks @sebthom |
|
Would it be possible to even further unify these files and have one or at least a common one in a common place, which is used for multiple repositories. E.g. in the |
I am not sure if centralization makes much sense since it gives context about the specific repo. |
@vogella