Skip to content

fix(web): show repository names on linked pull requests - #13061

Merged
juliusmarminge merged 3 commits into
pingdotgg:mainfrom
Bil0000:t3code/linked-pr-repo-labels-20260922
Sep 24, 2026
Merged

juliusmarminge merged 3 commits into
pingdotgg:mainfrom
Bil0000:t3code/linked-pr-repo-labels-20260922

Conversation

@Bil0000

@Bil0000 Bil0000 commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Linked pull requests can have the same number in different repositories. Rows with a loaded PR snapshot currently hide the repository name.

Show the owner and repository on every linked PR row, with truncation and a tooltip for long names. Mobile already includes this label.

Verified with focused lint and the web typecheck. Verified in the Browser panel with two linked PRs that share #42 across different repositories.

Addresses #13050.

Made with GPT-5.6-Sol in the native Codex harness.

Linked PRs before

Linked PR repository labels after review

Summary by CodeRabbit

  • Enhancements
    • Pull request details display the linked repository name alongside branch information.
    • Long repository names are now truncated in the middle to keep both the beginning and end visible; hovering still shows the full name.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 22, 2026
@Bil0000
Bil0000 marked this pull request as ready for review September 22, 2026 15:52
@macroscopeapp

This comment has been minimized.

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Sep 22, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 720490a

Macroscope's review found this PR approvable — This is a focused one-file UI fix that adds repository labels to loaded linked pull requests using existing truncation and tooltip components. It leaves data flow, interactions, unloaded-row behavior, and production configuration unchanged.

You can add or adjust custom eligibility rules. Learn more.

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

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: 6ac7e083-4bab-4610-af52-e180722242c7

📥 Commits

Reviewing files that changed from the base of the PR and between 1163550 and 720490a.

📒 Files selected for processing (1)
  • apps/web/src/components/pullRequest/ThreadPullRequestsPanel.tsx

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


📝 Walkthrough

Walkthrough

When a snapshot is available, the pull request link row displays the repository name with middle truncation. Its tooltip still shows the full name.

Changes

Pull request metadata

Layer / File(s) Summary
Repository name display
apps/web/src/components/pullRequest/ThreadPullRequestsPanel.tsx
The snapshot-present metadata branch uses MiddleTruncate for the repository name. The tooltip continues to show the full name.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: maria-rcks

Merge Risk: ⚪ Minimal · up to 72049

This change improves repository identification while keeping the full name available on hover; no merge-blocking risk is evident.

🚥 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
Title check ✅ Passed The title clearly identifies the main change: showing repository names on linked pull requests. It is concise and specific.
Description check ✅ Passed The description explains the problem, the implemented behavior, verification steps, related discussion, and UI changes with before-and-after screenshots. It does not include the template checklist, bu…
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.
  • 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.

A tail cut on a long owner hides the repository name, so two linked PRs
from the same owner read identically. Cut in the middle like the branch
names beside it, keeping the name after the slash.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@macroscopeapp
macroscopeapp Bot dismissed their stale review September 24, 2026 06:41

Dismissing prior approval to re-evaluate 720490a

@macroscopeapp

This comment has been minimized.

1 similar comment
@macroscopeapp

This comment has been minimized.

@juliusmarminge
juliusmarminge merged commit cb1a3f3 into pingdotgg:main Sep 24, 2026
24 checks passed
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Sep 24, 2026
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: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.

2 participants