Skip to content

meta: add a root-level AGENTS.md - #65493

Open
jasnell wants to merge 4 commits into
nodejs:mainfrom
jasnell:jasnell/agents-md
Open

meta: add a root-level AGENTS.md#65493
jasnell wants to merge 4 commits into
nodejs:mainfrom
jasnell:jasnell/agents-md

Conversation

@jasnell

@jasnell jasnell commented Aug 22, 2026

Copy link
Copy Markdown
Member

We're seeing a number of repeated mistakes in AI-generated contributions, including agents using Signed-off-by, creating what appear to be unsupervised PRs, appearing to respond to comments without human oversight, etc. Having an AGENTS.md file with specific instructions to the agent on what to avoid should help.

@jasnell
jasnell requested a review from a team August 22, 2026 23:03
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. labels Aug 22, 2026
We're seeing a number of repeated mistakes in AI-generated
contributions, including agents using `Signed-off-by`, creating
what appear to be unsupervised PRs, appearing to respond to
comments without human oversight, etc. Having an AGENTS.md
file with specific instructions to the agent on what to avoid
should help.

Signed-off-by: James M Snell <jasnell@gmail.com>
@jasnell
jasnell force-pushed the jasnell/agents-md branch from ed9ce39 to 37e80dd Compare August 22, 2026 23:07
Comment thread AGENTS.md Outdated
@jasnell jasnell added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Aug 22, 2026
@jasnell jasnell added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Aug 23, 2026
@cjihrig
cjihrig requested a review from BethGriggs August 23, 2026 00:35
@legendecas

Copy link
Copy Markdown
Member

There is an ealier PR for AGENTS.md: #65463

Comment thread AGENTS.md

@MikeMcC399 MikeMcC399 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to see the experience of receiving and responding to AI-assisted / -generated PRs has resulted in a proposal for explicit coverage in an AGENTS.md directive!

#62612 disabled linting for AGENTS.md. Consider re-enabling, then re-formatting and responding to linting findings. I noticed because #65463 from @BethGriggs had done this.

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md
Comment thread AGENTS.md Outdated
@joyeecheung

joyeecheung commented Aug 23, 2026

Copy link
Copy Markdown
Member

The document uses a tone as if it's speaking to a human, even though it is not supposed to be read by human? e.g. the distinction of collaborators or non-collaborators is pointless to an agent unless you give it permisssions where the distinction makes a difference, but then the document goes on saying the agent shouldn't use any of these permissions, which means the agent is not supposed to make use of that distinction in the first place.

Co-authored-by: Brian Muenzenmeyer <brian.muenzenmeyer@gmail.com>
Comment thread AGENTS.md Outdated
@jasnell
jasnell requested a review from mcollina August 23, 2026 20:35
@jasnell

jasnell commented Aug 23, 2026

Copy link
Copy Markdown
Member Author

The document uses a tone as if it's speaking to a human, even though it is not supposed to be read by human? e.g. the distinction of collaborators or non-collaborators is pointless to an agent unless you give it permisssions where the distinction makes a difference, but then the document goes on saying the agent shouldn't use any of these permissions, which means the agent is not supposed to make use of that distinction in the first place.

Agents are perfectly capable of correctly interpretting it as written.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.