Address PR review feedback without swallowing it whole — cross-validate every claim and assumption against the code, design, and specs; give each finding an explicit justified verdict (ACCEPT, PARTIAL, REJECT, DEFER); fix accepted items at the root; reply where the feedback lives; re-submit. Use when the user says "/revise", "address the review", "here's feedback on your PR", "handle the review comments", "respond to the reviewer", or pastes a review from another agent or person.
100%
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
Low
Low-risk findings worth noting
Every finding is a hypothesis to test, not an instruction to follow. Reviewers, human or agent, often lack the project's context and this PR's priorities. A well-argued finding can still be wrong. The goal is a right PR, not a happy reviewer.
Paths are relative to this file. <plugin root> is two directories above it. Read each
file at the step named.
| File | Read at | Holds |
|---|---|---|
references/triage.md | step 2 | cross-validation per claim, the scope fence, the premature-optimization test |
references/respond.md | step 5 | the decision table, per-thread replies, ticket drafts, the report |
../../assets/writing.md | step 5, before writing any reply or the report | the writing rules for every text a person reads |
references/rationalizations.md | when tempted to just accept, silently drop, or drive-by | why each shortcut fails |
Every finding ends in exactly one verdict, with its validation evidence and a justification. Silence is never one of them.
Unsure, or missing information the decision needs? That is NEEDS-INPUT. Ask the user and hold that item. Never guess a verdict.
Invoking /revise is consent for the run. The run validates, fixes accepted findings,
replies on the PR's own review threads, and re-submits to the same PR. Creating tickets,
marking the PR ready, or force-pushing always needs fresh, explicit consent.
references/triage.md now. Test each claim against the code on
HEAD, the design docs and specs, the repo's conventions, and the PR's intent. Read the
actual code, run the cheap experiment, and open the doc the finding appeals to. Cite
what you find./submit phase 2 rules.references/respond.md and ../../assets/writing.md now. Reply where
the feedback lives, and propose a ticket for each DEFER. Run
<plugin root>/scripts/prose-lint.py on every reply and on the report before you post or
print them, and fix every finding in the text./submit mechanics take over: pre-push checks, push, PR title and
body revalidated (the revision may have shifted scope), CI green at the root.references/respond.md, the decision table first.prose-lint.py with no findings.korya/swd-skills
b1520b9
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.