Skip to content

feat(web): pull request list splits what you can merge from what you cannot - #242

Merged
badcuban merged 2 commits into
mainfrom
claude/separate-owned-unowned-prs-8643c2
Sep 6, 2026
Merged

feat(web): pull request list splits what you can merge from what you cannot#242
badcuban merged 2 commits into
mainfrom
claude/separate-owned-unowned-prs-8643c2

Conversation

@badcuban

@badcuban badcuban commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

The open Pull requests list filed every PR you wrote under Yours, whether it was on your own repo or a contribution to someone else's, and called everything else Others. The rows you can actually land were mixed in with rows that are out of your hands.

The list already knows, per row, whether you can push to that repo (GitHub's viewerPermission, mapped to viewerCanWrite). This change uses it to group the Open tab as:

  1. Needs you: unchanged.
  2. Yours: your PRs on repos you can merge.
  3. Incoming: other people's PRs on repos you can merge.
  4. Contributions: your PRs on repos you can't merge.
  5. Elsewhere: other people's PRs on repos you only follow.

The Involvement filter offers the same five and each survives in the URL. A host that doesn't report permission leaves the row under Yours or Incoming, as before, so non-GitHub hosts behave as they did. A bookmarked involvement=others now falls back to All.

Gates: vp fmt, vp lint, vp run typecheck, pullRequests.logic unit tests (73), and the full web browser suite (557) all pass.

…cannot

The open list filed every pull request the viewer wrote under Yours, whether
it sat on their own repository or was a contribution to someone else's, and
called everything else Others. The rows the viewer can actually land were mixed
in with rows that are out of their hands.

The list now reads the push access it already carries per row and groups the
open tab as Needs you, Yours, Incoming (other people's work on repositories the
viewer can merge), Contributions (the viewer's work on repositories they
cannot), and Elsewhere (other people's work on repositories they only follow).
The Involvement filter offers the same five, and a host that never says whether
the viewer may push leaves a row under Yours or Incoming as before.
@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
threadlines Ready Ready Preview Sep 6, 2026 7:48am UTC

Request Review

@badcuban
badcuban enabled auto-merge September 6, 2026 07:43
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 6, 2026
@badcuban
badcuban added this pull request to the merge queue Sep 6, 2026
Merged via the queue into main with commit f854efb Sep 6, 2026
16 checks passed
@badcuban
badcuban deleted the claude/separate-owned-unowned-prs-8643c2 branch September 6, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant