Skip to content

feat(web): show required pull request checks - #151

Merged
incognitojam merged 2 commits into
mainfrom
t3code/show-required-checks
Aug 15, 2026
Merged

feat(web): show required pull request checks#151
incognitojam merged 2 commits into
mainfrom
t3code/show-required-checks

Conversation

@incognitojam

@incognitojam incognitojam commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

Note

Shows GitHub repository requirements directly in the Checks tab and promotes Auto-merge when a pull request is blocked but otherwise mergeable.

What changed

  • Added GitHub repository-policy-aware merge readiness to pull request details.
  • Marked reported required checks and synthesized waiting rows for required checks that have not reported yet.
  • Read effective repository and organization rulesets alongside classic branch protection, with graceful GitHub Enterprise degradation.
  • Replaced the primary Merge action with Auto-merge when GitHub says the pull request is blocked and the viewer may enable auto-merge.
  • Kept unknown merge states and behind-base branches on their existing merge/update flows.

Why

A green or red job list does not explain whether GitHub will actually allow a merge. Required checks and GitHub’s merge-readiness verdict make the Checks tab answer that question directly, while Auto-merge becomes the obvious action when requirements are still outstanding.

UI changes

Captured against this pull request while its required checks were running.

Before: reported jobs only After: repository requirements
Before: the Checks tab shows generic job progress and a Merge button After: the Checks tab shows merge readiness, Required badges, and an Auto-merge button

Verification

  • vp test run apps/server/src/pullRequest/GitHubPullRequestCli.test.ts apps/server/src/pullRequest/GitHubPullRequestProvider.test.ts apps/server/src/pullRequest/PullRequestService.test.ts apps/server/src/pullRequest/gitHubPullRequestJson.test.ts apps/server/src/pullRequest/pullRequestChecks.test.ts apps/web/src/components/pullRequest/PullRequestChecksTab.test.tsx apps/web/src/components/pullRequest/pullRequestChecks.test.tsx apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts
  • vp run t3#typecheck
  • vp run @t3tools/web#typecheck
  • vp run @t3tools/contracts#typecheck
  • Targeted formatting and lint checks

Checklist

  • This PR is focused on GitHub merge requirements and the action they imply
  • I explained what changed and why
  • I included before/after screenshots for the UI change
  • No video is needed because this change adds no motion or timing behavior

Written by an agent (T3 Code, gpt-5.6-sol).

@incognitojam
incognitojam merged commit 91532eb into main Aug 15, 2026
9 checks passed
@incognitojam
incognitojam deleted the t3code/show-required-checks branch August 15, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant