Skip to content

Agents: tighten the pull request description rules - #339

Merged
ericholscher merged 5 commits into
mainfrom
claude/amazing-turing-7m4a0y-pr-descriptions
Nov 9, 2026
Merged

ericholscher merged 5 commits into
mainfrom
claude/amazing-turing-7m4a0y-pr-descriptions

Conversation

@ericholscher

@ericholscher ericholscher commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Update the agents file to follow the pattern from Simon here: https://simonwillison.net/2026/Nov/9/details-summary/

Claude Fable 5.1

Agent-written PR descriptions were accurate and too long to read: 150–200 words, roadmap, open questions. The rules said what to cover but not how long or where to stop.

Descriptions now open with one or two sentences a human wrote or chose, and the agent's write-up follows inside a <details> block whose <summary> is the model name — that line replaces a separate AI-generated note. Inside: problem, approach, what the reviewer must do; decisions stated, not asked; links and context welcome. No word cap — the fold is what keeps the opener short.

After merge, run sync-agents.sh --apply to push the file to the sibling repos.

Sessions: https://claude.ai/code/session_019qNu2BuxS1RBohnVdGovpF · https://claude.ai/code/session_0172yK2bcCT8DhqXFDfR2irP


Generated by Claude Code

Give the description a fixed order (problem, approach, what the reviewer
must do), a length cap, and a scope rule so agents stop writing roadmap
context and open questions into PR bodies.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019qNu2BuxS1RBohnVdGovpF
…tails

The human's one or two sentences lead; the agent's write-up follows inside a
<details> block whose <summary> is the model name, which is also the note that
the rest is agent-written. The ordering, length and scope rules now describe
that write-up. Pattern from https://simonwillison.net/2026/Nov/9/details-summary/

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0172yK2bcCT8DhqXFDfR2irP
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0172yK2bcCT8DhqXFDfR2irP
@ericholscher
ericholscher marked this pull request as ready for review November 9, 2026 13:39
@ericholscher
ericholscher requested a review from a team as a code owner November 9, 2026 13:39
@ericholscher
ericholscher requested a review from stsewd November 9, 2026 13:39
@ericholscher
ericholscher merged commit e8ac3d6 into main Nov 9, 2026
1 check passed
@ericholscher
ericholscher deleted the claude/amazing-turing-7m4a0y-pr-descriptions branch November 9, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants