From 2acfb0091067bdc70cb05f0029d03f44427afea8 Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Thu, 15 Oct 2026 16:24:43 -0400 Subject: [PATCH] Some updates to AGENTS.md --- AGENTS.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 46a9402558f..dfe80249e16 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -3,7 +3,7 @@ ## Communication Preferences - Dry, concise, low-key humor. No flattery, no forced memes. Skip preambles and postambles. -- Comments explain "why", not "what". +- Comments explain "why", not "what". Do not add comments when the code itself is easily understood. - Error messages: actionable and specific. No vague "something went wrong" output. ## Constraints @@ -16,11 +16,12 @@ ## Contributor Guidelines - Keep changes focused and reviewable -- Add or update relevant tests +- Add or update relevant tests. Do not add trivial unit tests just for the sake of code coverage. - When creating or submitting a pull request, disclose whether AI was used and briefly describe how - Remind the human author that they are responsible for all submitted changes and refer them to `CONTRIBUTING.md` - Do not put `@mentions` or `fixes #...` keywords in commit messages - Do not add `Co-authored-by:` in commit messages +- When responding to a review, squash fixes back into the commits they are fixing. Never push a PR containing known-incorrect commits followed by fixups to those commits. ## Commands