chore: close or split PRs with excessive commit churn (40+ commits) #359
Description
Problem
Several PRs have accumulated excessive commits, indicating scope explosion:
| PR | Commits | Days Open | Review Comments |
|---|---|---|---|
| #255 | 48 | 2 days | 82 |
| #235 | 23 | 2 days | 25 |
| #199 | 21 | 4 days | 79 |
These PRs are growing instead of shipping. 48 commits on a single PR is a scope disaster.
Evidence
From PR velocity analysis:
- PR feat(github-skill): enhance skill for Claude effectiveness #255: 48 commits, 73 bot reviews, 1 human review
- Pattern: Bot responds to every AI comment with fixes instead of prioritizing human feedback
Proposed Action
For each PR with 40+ commits:
- Evaluate if salvageable (can be split into smaller PRs)
- If not salvageable, close with explanation
- Re-open as smaller, focused PRs if work is still valuable
Acceptance Criteria
- PR feat(github-skill): enhance skill for Claude effectiveness #255 evaluated and actioned (close, split, or merge)
- PR feat(agents): add mandatory memory phases to pr-comment-responder #199 evaluated and actioned
- Decision documented for each
Priority
P0 - These PRs are accumulating rebase pain daily and won't get easier to merge
🤖 Generated from PR velocity analysis session