fix(web): multi-PR badges open the linked pull requests panel

[flamboh]

Note

🤖 Claude Opus 5.5 on behalf of Oliver

ELI5

When a thread has several linked pull requests, clicking its PR badge now shows all of them instead of just the newest one.

Problem

A thread with multiple unrelated linked PRs shows a +N badge, but clicking it opened only the most recently linked PR. The badge promises a set and delivers one item; the other links were only reachable through the panel's tab bar.

Fix

Stacked linked PRs already open the thread's Pull requests tab from their badge; this matches that convention, so any badge standing for more than one PR now opens the tab too. A single PR badge is unchanged: it still opens that PR and still supports cmd/ctrl-click to the host. The sidebar and composer footer share this control, so both follow. Mobile's badge is not tappable, so nothing changes there. onOpenStack became onOpenList since it no longer only serves stacks.

UI Changes

Before: clicking +2 opens a single PR.

After: clicking +2 opens the Pull requests tab with both links.

---

Made with Claude Opus 5.5 in Claude Code (via T3 Code). Screenshots by GPT-6 Luna in Codex.

Summary by CodeRabbit

• Bug Fixes
  • PR badges with multiple linked pull requests now open the thread’s pull-request list. Badges linked to a single PR continue to open that PR.

[macroscopeapp]

Approvability

Verdict: Approved at df8fdc3

Macroscope's review found this PR approvable — This small web UI fix changes only aggregate pull-request badge navigation: single PRs remain external links, while multi-PR badges open the existing linked pull-requests panel. The callback renames preserve existing stack behavior, and the remaining documentation update has no runtime impact.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 5113878b-4a90-4878-a613-33746a717491

📥 Commits

Reviewing files that changed from the base of the PR and between aca3c87 and df8fdc3.

📒 Files selected for processing (4)

• apps/web/src/components/BranchToolbarBranchSelector.tsx
• apps/web/src/components/Sidebar.tsx
• apps/web/src/components/ThreadStatusIndicators.tsx
• docs/user/source-control.md

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

Pull-request badges now open the thread’s pull-request list when they represent a stack or have additional linked pull requests. Single pull-request badges retain their existing link behavior.

Changes

Pull-request badge list behavior

Layer / File(s)	Summary
Badge list callback and click behavior apps/web/src/components/ThreadStatusIndicators.tsx, apps/web/src/components/Sidebar.tsx, apps/web/src/components/BranchToolbarBranchSelector.tsx	The badge control uses onOpenList for stacks and badges with additional linked pull requests. Those clicks prevent default navigation and propagation, then open the list. Other badges retain the single-PR click handler. Callers pass the renamed callback.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to df8fd

Multi-PR badges open the linked pull-requests list for the clicked thread. No issue requiring a fix before merge was identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 files. (1 skipped: 1…	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly identifies the web fix: multi-PR badges now open the linked pull requests panel.
Description check	✅ Passed	The description explains the problem, fix, behavior for single and multiple PR badges, shared controls, mobile behavior, and UI changes with before/after screenshots. It does not use the template's ex…

Full details: Docstring Coverage

Explanation

Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 files. (1 skipped: 1 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

---

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