Draft a PR body from OMC's paper trail — the smallest visual that proves the change, evidence consumed from the verify protocol (never re-collected), reversibility from the ADR test on the diff, glossary language throughout.
62
75%
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
View guide
Passed
No findings from the security scan
Fix and improve this skill with Tessl
tessl review fix ./skills/pr/SKILL.mdWhen work lands behind a review, the PR body is the one artifact a reviewer outside the session reads. Everything it needs already exists in the paper trail — this skill assembles it and adds nothing that was not already established. The summary-visual discipline is inspired by show-me (credited in the frontmatter).
## Summary
<one-line claim, then the smallest visual that proves it>
## Evidence
- **Before:** <output / failing check>
**After:** <output / passing check>
## Reversibility
<Rollback: cheap via <mechanism> | expensive because <reason>>
<Blast radius: one line>State the change in one sentence, then show the smallest view that makes the point. Match the visual to the change:
Show the whole block only when most of it is new, omitted context would hide ownership, or the reviewer needs a copyable target shape. One visual usually suffices; never all forms.
Consume the verify protocol's output — BUILD, TEST, LINT, FUNCTIONALITY evidence, freshness rule inherited. Present before/after pairs: the exact check that failed before and passes after, quoted from the output itself. When the change is visual, prefer a captured screenshot; qa-tester can drive the runtime for one. Never re-collect evidence here and never claim evidence the verify pass did not produce.
Run the ADR test on the diff itself — the same three questions launch applies to decisions: hard to reverse? surprising without context? the result of a real trade-off? A no on all three is a cheap rollback: say by what mechanism (revert the merge, toggle the flag). Any yes means an expensive rollback: say why. Link the ADR when the change implements or touches one. Close with the blast radius in one line — what else this change can move.
Terms come from CONTEXT.md where a glossary exists. The PR is read outside the session; the glossary is the only vocabulary the reader shares with it.
dc7ba1d
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.