Skip to content

feat(pull-requests): show label descriptions on hover - #6576

Open
RakshithBhat03 wants to merge 5 commits into
pingdotgg:mainfrom
RakshithBhat03:feat/pr-label-description-hover
Open

feat(pull-requests): show label descriptions on hover#6576
RakshithBhat03 wants to merge 5 commits into
pingdotgg:mainfrom
RakshithBhat03:feat/pr-label-description-hover

Conversation

@RakshithBhat03

@RakshithBhat03 RakshithBhat03 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Pull request labels in the Summary tab show their name and color, but leave the label's GitHub description inaccessible.

Labels with a non-empty description now use the shared hover-and-focus Tooltip primitive, including the standard tooltip surface and accessibility wiring. Labels whose description is missing, null, or blank remain ordinary non-interactive chips, and long unbroken descriptions wrap inside the panel.

Descriptions are optional in the shared contract so older clients and providers that do not supply them continue to work. GitHub descriptions are retained only for pull request detail responses; list and cross-repository search payloads stay unchanged.

Verification

  • vp test run packages/contracts/src/pullRequest.test.ts apps/server/src/pullRequest/gitHubPullRequestJson.test.ts — 86 tests passed.
  • vp run --filter @t3tools/contracts --filter t3 --filter @t3tools/web typecheck — passed. Existing Effect suggestions remain in unchanged server files.
  • vp fmt --check packages/contracts/src/pullRequest.ts packages/contracts/src/pullRequest.test.ts apps/server/src/pullRequest/gitHubPullRequestJson.ts apps/server/src/pullRequest/gitHubPullRequestJson.test.ts apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx — passed.
  • vp lint --report-unused-disable-directives packages/contracts/src/pullRequest.ts packages/contracts/src/pullRequest.test.ts apps/server/src/pullRequest/gitHubPullRequestJson.ts apps/server/src/pullRequest/gitHubPullRequestJson.test.ts apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx — passed with a pre-existing react(no-array-index-key) warning in PullRequestSummaryTab.tsx.
  • vp run --filter @t3tools/web build — production web build passed.
  • Manually verified the label description interaction in an isolated seeded web environment.

Screenshots

Pull request label description tooltip


Built by GPT-5.6 in T3 Code through the Codex harness.


Note

Low Risk
Backward-compatible optional field and UI-only tooltip behavior; list payloads unchanged and hosts without descriptions keep working.

Overview
Pull request label descriptions are now available on the Summary tab: chips with a non-empty description use a hover/focus tooltip (same pattern as other PR metadata), while labels without a description stay plain chips.

The shared PullRequestLabel type gains an optional description (omitted when the host does not send it, null when empty). GitHub decoding normalizes trimmed text and includes descriptions only on detail responses—list and search rows still expose name and color only, so list payloads stay unchanged.

Implementation is split across toLabels(..., includeDescription) on the server, contract/tests for backward compatibility, and a PullRequestLabelChip component in the Summary tab.

Reviewed by Cursor Bugbot for commit 3377fd4. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

[!NOTE]

Show label descriptions on hover in pull request summary tab

  • Adds an optional description field to the PullRequestLabel schema in pullRequest.ts, which is absent when the host does not report it and null when no description exists.
  • Extends the GitHub label decoder in gitHubPullRequestJson.ts to parse and normalize descriptions; detail views include descriptions, list views omit them.
  • Introduces a PullRequestLabelChip component in PullRequestSummaryTab.tsx that wraps labels with a tooltip showing the description when one is present.

Macroscope summarized 3377fd4.

Co-authored-by: GPT-5.6 <codex@openai.com>
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

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: 190d6880-22ec-4c74-a9f9-20eaf73d5f4d

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

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

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Aug 14, 2026
@RakshithBhat03
RakshithBhat03 marked this pull request as ready for review August 14, 2026 04:50
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Aug 14, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 5b2dcbc

Minor UI enhancement that adds tooltip descriptions to pull request labels on hover. The changes are additive, backward compatible, follow existing tooltip patterns in the codebase, and include comprehensive tests.

You can customize Macroscope's approvability policy. Learn more.

@macroscopeapp macroscopeapp Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One consistency finding on the new label chip: the hover description is built from Popover rather than the shared Tooltip primitive this file already uses for its other hover-only text. Details inline.

Posted via Macroscope — UI Consistency

Comment thread apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx Outdated
Co-authored-by: GPT-5.6 <codex@openai.com>
@macroscopeapp
macroscopeapp Bot dismissed their stale review August 15, 2026 03:07

Dismissing prior approval to re-evaluate 5b2dcbc

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

Labels

size:M 30-99 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.

1 participant