feat(web): pull request surfaces — filters & search qualifiers, all-server listing, update branch, smarter diffs, panel follow-ups

[Bil0000]

Follow-ups to the pull request surfaces, from Discord/X feedback.

List: filters, search qualifiers, review status

• URL-driven filters mirroring GitHub's search: Draft (all / only / hide), Review (Approved / Changes requested / Review required / No reviews), Checks (Passing / Failing). On GitHub they become search qualifiers; elsewhere rows are narrowed service-side from the fields they carry (checks stays host-side; hosts that never summarise reviews aren't judged by the review filter).
• GitHub-style qualifiers typed into the search bar: label:x, label:"needs design", -label:x, author:login, draft:true|false, review:…, status:… — and any unrecognized key:value token (size:XXL, vouch:trusted) filters as that label, so each repository's own label scheme just works; quoting keeps a token literal text. Typed qualifiers win over the menu; leftover words stay ordinary search text.
• Rows show their review status ("Approved" / "Changes requested") via PullRequestListEntry.reviewDecision; listings stay ordered by recently updated.

List: every connected server

The page previously listed one environment. It now unions all connected servers that support pull requests: per-environment atoms fanned out through one merge atom, rows tagged with their server, viewer identity / truncation / cursors / project scope all environment-scoped, failures degrading to partial results. A Server filter scopes to one connection; the sidebar link shows when any server qualifies; PR links in chat resolve across servers; right-panel tabs are server-scoped so the same PR from two servers is two tabs (parts adapted from #6108 — thanks @StiensWout). Azure DevOps repositories no longer read as unavailable: the selector hands az the repository name instead of the org/project/_git/repo path.

Detail: base-branch freshness + Update branch

GitHub parity for the out-of-date banner: up to date / "out-of-date with the base branch — changes can be cleanly merged" (with behind-by count, only on a real mergeable verdict) / conflicting. Update branch split button: merge commit by default, "Update with rebase" when the host and viewer allow it (gh pr update-branch [--rebase] through the existing action pipeline; detection via the compare API's behindBy with fork-qualified head refs, since mergeStateStatus only reports BEHIND under strict checks). Capability- and permission-gated end to end; other providers untouched.

Code tab: diffs in reading order instead of alphabetical

orderDiffFiles orders each slice source → tests → generated (lockfiles, snapshots, dist/, minified). Source files follow an import graph parsed from the patch's own lines — dependencies before dependents, deterministic, directory-clustered ties. Tests sit after the implementation they name.

Right panel + summary follow-ups

• Fix/Ask/resolve-conflicts hand-offs land in the open thread's composer when the panel shows that thread's own PR — matched by project + repository + number, so a second checkout of the same repository prepares its own worktree.
• Chrome scroll collapse in the panel; smooth reopen at the top; sans font for review annotations; hidden tab-strip scrollbar.
• Summary: labels row, avatar-only attribution with hover names, capitalized review states, resolved conversations and dismissed approvals collapse (dismissal reason read from the timeline's ReviewDismissedEvent, paginated), approved/dismissed reviews don't offer "Fix in a thread".
• Correct per-file counts for withheld hunks via PullRequestDiffResult.omittedFileStats.

Verification

• tsgo --noEmit clean in contracts, client-runtime, server, web.
• 388 server, 178+ web (pullRequest/, routes/, rightPanelStore, openPullRequestLink, sidebar) and 244 contracts tests pass.
• Live read-only verification against this repository: reviewDecision via gh/GraphQL, base comparison (behindBy 1091 on fix(keybindings): let xterm handle Ctrl+Arrow natively, use CSI for Alt+Arrow #2243, 0 on feat(web): pull request surfaces — filters & search qualifiers, all-server listing, update branch, smarter diffs, panel follow-ups #6039), and the malformed-variables regression proven against the real CLI.
• All Macroscope findings across every round fixed with regression tests and resolved.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 526c80f1-7ee5-46fe-bda8-4022150be7d9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Needs human review

This PR adds multiple new features including PR list filters, review status display, import-based diff ordering, update-branch functionality, and multi-environment listing support. The scope and new logic introduced warrant human review.

^{You can customize Macroscope's approvability policy. Learn more.}

Dismissing prior approval to re-evaluate 4095917

Dismissing prior approval to re-evaluate c4e4edc

Dismissing prior approval to re-evaluate 9b66581

Dismissing prior approval to re-evaluate 1c4c08f