Skip to content

Commit 5c6c89e

Browse files
authored
docs: update PR guidance (#550)
1 parent 8f722c3 commit 5c6c89e

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

AGENTS.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -242,9 +242,8 @@ Command-only flags (like `find --first`) that don't flow to the platform layer o
242242
- Open a ready-for-review PR by default. Use a draft PR only when the user explicitly asks for one or the work is intentionally incomplete.
243243
- Run required checks for touched scope from **Testing Matrix**.
244244
- PR body must be short and include:
245-
- `## Summary`
246-
- `## Validation` written as evidence of what was checked and what passed; prefer concise prose such as "Unit tests covered X", "SkillGym confirmed Y planning", "Manual Android run showed Z before/after". Avoid dumping raw command lists unless a specific command is the useful evidence.
247-
- In the summary, describe behavior changes as before/after when that makes the fix easier to review.
245+
- `## Summary`: describe the user-visible or reviewer-relevant change as before/after when useful, and include the issue closed by the PR using `Closes #123` when applicable.
246+
- `## Validation`: answer this prompt in concise prose: "How did you verify the change, and what passed or changed on screen?" Prefer evidence over command dumps; mention the relevant check category or scenario, and include screenshots when visual/UI behavior is relevant.
248247
- Call out known gaps/follow-ups explicitly.
249248
- Include touched-file count and note if scope expanded beyond initial command family.
250249

0 commit comments

Comments
 (0)