lsm / neokai / 25255886703

82%

Build Type

push

github

Committed by web-flow

Commit Message

Block unresolved PR conversations before merge (#1763)

* Block handoff with unresolved PR conversations

Ensure coding handoffs and post-approval merges check unresolved GitHub review threads so PRs do not remain blocked after fixes land.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* Address review thread gate feedback

Extend review-thread guidance to all affected PR workflows and tighten gate/merge instructions so agents can resolve or escalate unresolved conversations deterministically.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* Paginate review thread checks instead of hard-failing at 100

Replace the single-page hasNextPage hard-failure with cursor-based pagination that iterates all review thread pages and accumulates unresolved counts across pages, so large PRs with 100+ resolved threads are no longer permanently blocked.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* Fail closed when GraphQL returns errors or incomplete reviewThreads data

The review-thread pagination loop now checks for `.errors` in the
GraphQL response and for a missing `data.repository.pullRequest.reviewThreads`
node. Both cases cause the gate to fail with a descriptive error instead
of silently treating missing data as "no unresolved threads."

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* Add cursor pagination to post-approval merge template review-thread check

The merge template's GraphQL query now accepts a $cursor variable and
includes endCursor in pageInfo, with explicit pagination guidance so
the post-approval session fetches all review thread pages instead of
stopping at the first 100. Added test coverage for the pagination
tokens and guidance text.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* Route GraphQL review-thread queries to the correct GH host

Extract the hostname from the PR URL and pass --hostname to
gh api graphql when it differs from github.com. This ensures
GitHub Enterprise PRs qu... (continued)

Coverage Stats

7801 of 10201 branches covered (76.47%)

Branch coverage included in aggregate %.

104 of 104 new or added lines in 2 files covered. (100.0%)

1 existing line in 1 file now uncovered.

60443 of 84509 relevant lines covered (71.52%)

255.81 hits per line