pr
Use when writing a PR body: summary with the smallest clarifying visual, before/after evidence, and merge-danger (door + blast radius).
- License 许可
- Apache-2.0
- 所属仓库
- r3dlex/skills
- 最近更新
- 2026-10-29
- Stars
- ★ 0
- Forks
- 0
01 前言 Front Matter
- name
- pr
- description
- Use when writing a PR body: summary with the smallest clarifying visual, before/after evidence, and merge-danger (door + blast radius).
03 SKILL.md 正文
Adapted from
show-meby Dex Horthy (Humanlayer): https://github.com/humanlayer/skills
Use this template for writing the PR body:
## Summary
<diagram, diff-sketch, or tree>
## Evidence
- **Before:** <screenshot/output/failing test run>
**After:** <screenshot/output/passing test run>
## Merge Danger
**Door:** <one-way or two-way>
<optional: description>
**Blast Radius:** <one-word description>
<optional: potential ramifications of merge>
Skip all preambles and keep prose brief. Use the repo's domain vocabulary from GLOSSARY.md (vendored repos: CONTEXT.md).
Summary
Pick the smallest view that makes the key point clear:
- Pseudocode for logic or an algorithm:
on(save)
if content is unchanged
return cached result
write new content
- Call tree for runtime control flow:
submitForm
createSession
persistPrompt
navigateToSession
- Component/module tree for UI structure or file responsibility:
src/
├── commands/ # parses user actions
├── sessions/ # owns session state
└── transport/ # sends API requests
- Mermaid for component interaction or data flow; diff when the point is what changes inside a shape that already exists (match the diff shape: component, file layout, call tree, or state flow). Show the whole block as code only when most of it is new or omitted context would hide ownership.
Place each visual next to the short text it supports. Keep only the calls, files, states, and boundaries needed for the current question. One visual, several, or none — use judgement and don't overwhelm the reviewer.
Evidence
Concrete evidence that the change works — before and after. Screenshots are S-tier when the change is visual and the environment supports them. Execution-based evidence (test results, console output) is A-tier: show the exact test that now fails and passes, describing its steps as pseudocode.
Merge Danger
Door — one-way or two-way. Two-way doors can be walked back through; one-way doors cannot. A PR that is cheap to roll back is lower risk; destructive actions or hard-to-reverse decisions are one-way doors.
Blast radius — the potential impact of the change: layout shift, breakage for consumers, behavior drift, mobile responsiveness, and so on. Consider all possibilities, not just the intended one.
04 同仓库更多 Skills
skill
Fixture.
stable-skill
Fixture stable-skill.
experimental-skill
Fixture experimental-skill.
deprecated-skill
Fixture deprecated-skill.
karpathy-guidelines
Apply think-before-coding, simplicity, surgical-change, and goal-driven engineering rules. Use when planning, implementing, or reviewing code.
karpathy-guidelines
Apply think-before-coding, simplicity, surgical-change, and goal-driven engineering rules. Use when planning, implementing, or reviewing code.