docs: make PR descriptions plain-language first - #202
Conversation
|
Codex review: needs real behavior proof before merge. Reviewed September 12, 2026, 3:42 PM ET / 19:42 UTC. ClawSweeper reviewWhat this changesThe PR shortens contribution prompts, places user impact before technical rationale, and keeps detailed output optional while requiring visible risks and evidence summaries. Merge readiness⛔ Blocked before merge - 2 items remain This is a focused, useful template update that is not present on current main or v0.8.0. No actionable correctness or security defect was found; rendered-template proof remains outstanding. Priority: P3 Review scores
Verification
How this fits togetherGitHub uses this repository’s pull request template to seed contributor descriptions. Its prompts guide how contributors explain changes and provide evidence for reviewers. flowchart LR
A[Contributor opens PR] --> B[Repository template]
B --> C[Problem and user impact]
C --> D[Rationale and evidence]
D --> E[Reviewer assessment]
Before merge
Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Keep one concise contribution template that foregrounds outcomes while leaving material risks, required actions, and evidence visible. Do we have a high-confidence way to reproduce the issue? Not applicable: this changes contributor guidance rather than repairing a runtime defect. Is this the best way to solve the issue? Yes: editing the existing template is the narrowest solution, preserves the established section names, and follows the linked merged template precedent. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against c1f351687f97. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
Co-authored-by: hannesrudolph <49103247+hannesrudolph@users.noreply.github.com>
9dab7b3 to
69ff741
Compare
What Problem This Solves
Contributors need concise prompts that lead with the problem and concrete outcome.
User Impact
User impact: PR descriptions put user impact before technical rationale while keeping risks, required actions, and evidence visible. No product behavior changes.
Why This Change Was Made
Apply the plain-language direction from openclaw/openclaw#146253, preserving the established section names and maintainer-edit guidance. Contributor credit for @hannesrudolph is retained.
Evidence
git diff --checkpasses. Current-head CI is required before merge.Before and after rendered proof
Before:
After: