Repository navigation
feat(pr): Add structured PR body skill - #1297
Conversation
Port of mattpocock/skills engineering/pr (credits: Dex Horthy's show-me). Model-invoked when writing a PR body: Summary visual, before/after Evidence, Merge Danger (door + blast radius). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one): 1
Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
OpenAPI changes ⚪ No API surface changesNote This PR does not modify the API contract.
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Runs .claude/skills/pr-ready/SKILL.md through the Messages API when the ready-for-review label is applied (same trigger as CodeRabbit auto-review). Requires ANTHROPIC_API_KEY repo secret. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
🎉 This PR is included in version 1.8.0-main.27 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
Adds a model-invoked skill that shapes PR bodies. Port of
mattpocock/skillsengineering/pr, itself derived from Dex Horthy'sshow-me(Humanlayer). We had/pr-reviewfor reading PRs, nothing for writing them.Label flow, same trigger CodeRabbit uses (
.coderabbit.yaml):sequenceDiagram participant Dev participant GH as GitHub participant WF as pr-ready.yml participant Claude as Messages API Dev->>GH: add label ready-for-review GH-->>WF: pull_request.labeled GH-->>GH: CodeRabbit auto-review (existing) WF->>GH: read SKILL.md + PR template + diff WF->>Claude: claude-opus-5-5, fallbacks: default Claude-->>WF: markdown body WF->>GH: pulls.update(body), previous body in <details>Body shape it produces:
Only edits vs upstream skill:
name: pr-ready, description widened to "writing or updating a PR body".Workflow:
actions/github-script+ rawfetch, same pattern aspr-formatter.yml. Diff capped at 400K chars. Fails loud on missing key, API error, orstop_reason: refusal.Evidence
After: fix(evaluations): Compact traces for prompt #1274 body rewritten with this skill: call-tree diff, payload-shape diff, before/after test list,
Door: two-way,Blast Radius: prompt-improvementnaming the movedTraceDataimport path and the unfixedai_summarysibling. See fix(evaluations): Compact traces for prompt #1274Merge Danger
Door: two-way
One markdown file under
.claude/. Delete to revert.Blast Radius: pr-descriptions
.claude/and CI.ready-for-reviewgets its body rewritten. Previous body kept in a<details>block.pr-formatter.yml(on open), so the final shape is pr-ready's. Could retire pr-formatter later.ANTHROPIC_API_KEYrepo secret. Not present today; workflow fails with a clear message until added.Checklist
fastapi run --reload app/main.pyordocker compose upin the repository root and test.Notes
Upstream: https://github.com/mattpocock/skills/blob/main/skills/engineering/pr/SKILL.md
🤖 Generated with Claude Code