Repository navigation
feat(macapp): show per-reviewer status in the PR panel (#52) - #64
Merged
Merged
Conversation
The Pull Request inspector now lists each reviewer with their state — who
approved, who requested changes, who is still pending — beneath the aggregate
review-decision header, so you can tell at a glance when reviews are requested
vs in, and jump to the PR when needed.
Reviews ride the existing `gh pr list` probe via two new --json fields
(latestReviews, reviewRequests) — no extra API call. classifyPR folds them into
a typed, deduped reviewer list (user/team identity so a team can't collide with
a user; reviewRequests wins on re-request; unrecognised review states skipped).
PRPresentation.reviewers sorts and maps each to a row with a bot-aware label
("in progress" for bots like Copilot, "review requested" for humans). The
aggregate reviewDecision is retained as a header, so a branch-protected
REVIEW_REQUIRED PR with no named reviewers still shows a signal.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #52.
What
The Pull Request inspector now shows per-reviewer status — who approved, who requested changes, who is still pending — beneath the aggregate review-decision header. Tells you at a glance when reviews are requested vs in, and when to go visit the PR.
How
gh pr list --head … --jsonprobe via two new fields (latestReviews,reviewRequests) — no extra API call.classifyPRfolds them into a typed, deduped[Reviewer]:.user(login)/.team(slug)identity (a team can't collide with a user),reviewRequestswins on re-request, unrecognised review states skipped, missing fields →[], malformed →keepPrior.PRPresentation.reviewers(_:)is the single sort + map seam → rows with bot-aware labels ("in progress" for bots like Copilot, "review requested" for humans) and friendly names.reviewDecisionis kept as a header above the rows, so a branch-protectedREVIEW_REQUIREDPR with no named reviewers still shows a signal.Tests
make app-build/app-test(429 pass, +15 review tests) /app-lint— all green. Covers review parse / dedup / team / skip-unknown / decoder back-compat regression; presentation sort / state mapping / bot-aware / team / no-collision.Not in scope (tracked in
TODOS.md)[N comments]) — needs a secondgh apicall.🤖 Generated with Claude Code