maintainer-review
Installation
SKILL.md
Maintainer Review
Objective
Make a maintainer decision, not a generic diff summary. Separate these questions:
- Is the claimed behavior real?
- What user outcome or constraint exists independently of the reporter's proposed API or fix?
- Can supported functionality already achieve that outcome with reasonable composition or configuration?
- If a gap remains, is the proposed solution the best design and implementation layer?
- Can supported users plausibly reach the gap, and what happens when they do?
- Is it important enough to act on now?
- If this PR did not already exist, would maintainers choose to open and implement the same work?
- For a PR, is this solution worth merging and maintaining?
- Can overlapping or stale operations corrupt shared state or clean up resources owned by surviving work?
- If competing PRs exist, which single implementation path should maintainers pursue?
- What concise maintainer message should communicate closure, an evidence request, or required changes?
Treat an issue's requested field, callback, flag, class, or implementation strategy as a proposed mechanism, not as the accepted requirement. Do not begin by asking how to implement it. First establish either a concrete unmet user outcome or a violated supported contract, then prove that the proposed mechanism is better than the available alternatives.