Repository navigation
feat(ci): automated PR readiness checks — contributor feedback before review #750
Description
Problem
Contributors open PRs that aren't ready for review — draft state, multiple unsquashed commits, stale branch, no Copilot review approval, missing changeset, merge conflicts. Brady has to manually inspect each one and comment with what needs fixing. This doesn't scale, and contributors don't get immediate feedback on what's blocking their PR.
Proposed Solution
An automated PR readiness check that runs on pull_request events and posts/updates a checklist comment telling contributors exactly what needs to be fixed before the PR is review-ready.
Readiness Checks
The following checks should be evaluated on every PR event:
| # | Check | Description |
|---|---|---|
| 1 | Single commit (or squashable) | PR should have 1 commit, or repo should be configured for squash merge |
| 2 | Not in draft | Draft PRs shouldn't trigger review requests |
| 3 | Branch up to date | PR branch should be rebased on latest dev |
| 4 | Copilot review | copilot-pull-request-reviewer should have an approved review |
| 5 | Changeset present | A .changeset/*.md file exists in the PR diff (per changelog-gate) |
| 6 | No merge conflicts | GitHub's mergeable state is MERGEABLE |
| 7 | CI passing | All required status checks are green |
Expected Behavior
- On every PR open/sync/edit event, a GitHub Action evaluates all checks
- Posts (or updates) a single comment with a checklist showing pass/fail for each item
- Contributors see immediately what's blocking their PR
- Brady only needs to review PRs that pass all automated checks
Design Considerations
- Should complement existing
squad-ci.yml, not replace it - Lightweight — single workflow file, no external dependencies
- Must handle fork PRs (limited permissions for external contributors)
- Comment should be updated in-place (not spammed on every push)
Related
.github/PR_REQUIREMENTS.md— canonical PR requirements specCONTRIBUTING.md— contributor-facing docs- Issue Merge .ai-team-templates/ into .squad/templates/ #104 — PR completeness gates (this partially addresses that)
Activity
feat(ci): automated PR readiness checks (#750)
fix(ci): address PR review feedback on comment moderation (#751)
fix(ci): address PR review feedback on readiness checks (#750)
chore: remove PR readiness workflow cargo from spam moderation PR
refactor(ci): extract PR readiness script from workflow inline code
feat(ci): add Copilot review thread resolution gate to PR readiness
feat(ci): automated PR readiness checks (#750)
feat(ci): automated PR readiness checks (#750)
feat(ci): automated PR readiness checks (#750) (#752)
feat(ci): automated PR readiness checks (#750) (#752)