Skip to content

feat(web): show repository in thread pull request rows - #13180

Closed
BeLazy167 wants to merge 4 commits into
pingdotgg:mainfrom
BeLazy167:feat/pr-panel-repo-name
Closed

BeLazy167 wants to merge 4 commits into
pingdotgg:mainfrom
BeLazy167:feat/pr-panel-repo-name

Conversation

@BeLazy167

@BeLazy167 BeLazy167 commented Sep 23, 2026 •

Copy link
Copy Markdown

What Changed

Rows in a thread's linked pull requests panel now show the repository name. It sits between the author and head → base. The owner is dropped to save room. Hovering the name shows the full host/owner/repo in the standard tooltip.

One file: apps/web/src/components/pullRequest/ThreadPullRequestsPanel.tsx.

Why

A thread often links pull requests from several repositories. The rows showed only author and branches, so rows from different repos looked the same. Branch names like feat/x → staging repeat across repos and do not tell them apart. The full pull requests page already shows the repository in the same slot.

UI Changes

All names, titles, branches, numbers, and times in the screenshots are placeholders.

Before After
before after

Checked in a local web dev build: one thread, 8 linked pull requests across 7 repositories, each row shows its repository.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes (not applicable: no motion)

Done by Claude Opus 5.5 in Claude Code.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 23, 2026
@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

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

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration

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

Review profile: CHILL

Plan: Advanced

Run ID: 5b93e052-d3ac-49c8-a3db-a08302f4f35c

📥 Commits

Reviewing files that changed from the base of the PR and between 4ac416a and 2c72e65.

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

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: 01fb5254-b5a0-481f-bb2d-0fee3b28bd9c

📥 Commits

Reviewing files that changed from the base of the PR and between 0b6e41b and 5a2008b.

📒 Files selected for processing (1)
  • apps/web/src/components/pullRequest/ThreadPullRequestsPanel.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • apps/web/src/components/pullRequest/ThreadPullRequestsPanel.tsx

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


📝 Walkthrough

Walkthrough

LinkRow now uses a styled tooltip to show the full repository path when pull-request snapshot data exists. Repository truncation, branch display, and the no-snapshot fallback remain unchanged.

Changes

Pull request context display

Layer / File(s) Summary
Snapshot repository tooltip display
apps/web/src/components/pullRequest/ThreadPullRequestsPanel.tsx
When snapshot data exists, LinkRow uses Tooltip, TooltipTrigger, and TooltipPopup for the repository name. The popup shows the full host/repository path. Other repository and branch display behavior remains unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 5a200

The repository label and full-path tooltip preserve the supported snapshot and fallback display behavior. No actionable merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. 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 and concisely describes the main change: displaying the repository in thread pull request rows.
Description check ✅ Passed The description explains what changed, why it changed, the UI impact, testing performed, and includes before/after screenshots. It is focused and mostly complete. The hover tooltip is an interaction, …
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

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

@github-actions github-actions Bot added size:S 10-29 changed lines (additions + deletions). and removed size:XS 0-9 changed lines (additions + deletions). labels Sep 23, 2026
@BeLazy167

BeLazy167 commented Sep 23, 2026 •

Copy link
Copy Markdown
Author

@juliusmarminge this should be quick

@juliusmarminge

Copy link
Copy Markdown
Member

Superseded by #13061, which landed on main and already shows the repository name on linked pull request rows.

Closing this PR as superseded.

@juliusmarminge

Copy link
Copy Markdown
Member

Superseded by #13061 (merged).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants