[TESTERS NEEDED] Rewrite AGENTS.md with community-first agent guidelines#8604
[TESTERS NEEDED] Rewrite AGENTS.md with community-first agent guidelines#8604SableRaf wants to merge 5 commits intoprocessing:mainfrom
Conversation
Revise AGENTS.md to target p5.js: replace Processing-specific references with p5.js equivalents.
|
Thanks @SableRaf ! This update seems like a positive improvement. |
|
|
||
| ## How to help | ||
|
|
||
| ### 0. Welcome them and find out where they are |
There was a problem hiding this comment.
Not super important but both this and the next point are labelled as 0, is that intentional?
There was a problem hiding this comment.
Ah good catch! I fixed it
|
Note that Claude CLI tends to ignore I have managed to correct this behavior locally by adding a
However, I have resisted including vendor-specific prompt files in the root of the project so far, as it feels like free advertising or even endorsement. Would it be worth it in this case? Curious what others think. |
Update .github/copilot-instructions.md to explicitely require consulting AGENTS.md BEFORE any other actions.
|
I'm also a little hesitant, at least at first, to add instructions for specific tools, for the reasons you describe and also because it seems like we might be playing whack-a-mole as new tools come up. But it's also true that new tools, and even commonly used ones like Claude, may just ignore our instructions. To a certain extent, I feel like maybe we do what we can via methods like this, but then beyond a reasonable amount of effort on our part, fall back on our AI usage policy to close PRs that ignore them? |
|
Based on experience from several test runs. I didn’t have Claude at hand, so the experiments were carried out with Codex 5.3 in xhigh mode. Overall, the result matches the desired behavior, except for three scenarios:
I had the idea of trying a kind of prompt injection to force the agent to read the instructions file, but that did not produce much of a result: even when the warning about the need to read AGENTS.md was surfaced as ESLint warnings, it could still ignore them without issue, while saying something like "it's default warning, nothing to worry about". It seems that advanced agents are smart enough to ignore commands coming from tools (ls/dir with instructions in file names, warnings from eslint, and so on). I'm agree, I think it’s not worth trying to force an AI agent to familiarize itself with the instructions when this is its owner responsibility. Instead of adding support for every vendor’s file format, maybe it would be better to make that process a bit easier for AI users. For example, it may be worth adding a special section under “AI Usage Policy” in README.md (agents sometimes tend to read this before starting a job, even while ignoring AGENTS.md) with an onboarding prompt, since it seems that the initial user command cannot be reliably overridden. Something like:
It'll make things easier for human contributors. A bit. |





Note
See more context in the similar PR in processing/processing4#1453
This PR is a rewrite of the
AGENTS.mdfirst introduced in processing/p5.js#8194.What's new?
How to test this PR
The best way to test this is to check out this PR an agentic coding environment (Claude Code, Copilot, Cursor, etc.) and try a few contributor scenarios:
Test the same input multiple times and check whether the responses are consistently helpful for the particular scenario.
Note any misaligned behavior and share it in the comments below. Please specify which model, version, and environment was used.
Note that this is largely subjective so use your judgment when you evaluate agent behavior. Discussion in the comments is encouraged.
Common failure-cases for agent behavior
AGENTS.mdfileOther changes
copilot-instructions.mdto direct copilot to read AGENTS.md, since it tended to skip it.