Update AGENTS.md#1701
Merged
st0012 merged 2 commits intoruby:masterfrom May 7, 2026
Merged
Conversation
Add a rule under the 'Notes for AI Agents' section requiring concise PR descriptions (2-4 short paragraphs covering context, rationale, and notable side effects), with no 'Test plan' section and no Claude Code session link or other AI attribution. This overrides the default Claude Code PR template so cloud sessions working on this repo produce uniform descriptions.
st0012
requested a deployment
to
fork-preview-protection
GitHub Actions
Waiting
There was a problem hiding this comment.
Pull request overview
Updates the contributor guidance for AI-assisted work by adding expectations for pull request descriptions and fork workflow hygiene, aiming to reduce stale/noisy PR metadata and minimize merge conflict risk when developing on forks.
Changes:
- Adds a rule requiring concise PR descriptions and explicitly disallowing a “Test plan” section and AI attribution/session links.
- Adds guidance to fast-forward a fork’s
mastertoruby/rdoc:masterbefore creating feature branches.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
Test plansection generated by Claude, whether in my own PR or in other people's. They rarely provide real value and can easily become stale (e.g. it'd list CI passing as one checklist, but can't automatically tick it off when the CI passed)