pr 精选
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.
安装方式
cp -r skills/pr ~/.claude/skills/pr把该 skill 目录放进客户端 skills 目录。不同客户端的目录可能不同,Claude Code 默认是 ~/.claude/skills/
/plugin marketplace add Yeachan-Heo/oh-my-claudecode⚠️ 该命令形态随客户端版本变化,仅在检测到 .claude-plugin/marketplace.json 时展示;不可用时请用「目录复制」。
https://raw.githubusercontent.com/Yeachan-Heo/oh-my-claudecode/8262f9f790eed35e6cbd66572b36c2acba18f679/skills/pr/SKILL.md下载后另存为 ~/.claude/skills/pr/SKILL.md。链接锁定采集时的 commit,内容与本站展示一致。
整仓一次性获取:git clone --depth 1 https://github.com/Yeachan-Heo/oh-my-claudecode.git
依赖与权限
- 无需 API Key
- 需要联网访问外部服务
- 无外部命令依赖
- 以上为按正文与 frontmatter 自动推断的信号,仅供参考;安装前建议先阅读原文。
SKILL.md 正文
PR Body
When 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).
Template
## 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>
Sections
Summary
State the change in one sentence, then show the smallest view that makes the point. Match the visual to the change:
- logic or an algorithm → pseudocode
- runtime control flow → a call tree
- UI structure → a component tree
- file responsibility or a broad refactor → a shallow file tree
- component interaction or data flow → a Mermaid diagram
- the delta itself → a diff shaped like the change (component, layout, call tree, or state machine)
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.
Evidence
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.
Reversibility
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. Classify the door first: a two-way door (undo is cheap and ordinary) needs no ceremony beyond naming the undo; a one-way door says so explicitly and names what makes the step irreversible. Link the ADR when the change implements or touches one. Close with the blast radius in one line — what else this change can move.
Language
Terms come from GLOSSARY.md where a glossary exists (legacy yards not yet migrated keep it in CONTEXT.md). The PR is read outside the session; the glossary is the only vocabulary the reader shares with it.
Triggers
- launch Phase 5, when the platform reviews through PRs: the completion report carries a PR body drafted by this skill.
- Any session that opens or materially updates a PR.
Output
- The PR body, ready to paste
- The paper-trail pointers it relies on (verify evidence, ADRs, spec)
质量与来源
- 所属仓库
- Yeachan-Heo/oh-my-claudecode
- 仓库星标
- 4.0 万
- 最近更新
- 今天
- 采集时间
- 今天
- commit
- 8262f9f790ee
- 内容指纹
- sha256:c80af1215263a74
- 文件路径
- skills/pr/SKILL.md