fix(web): show pointer cursors on pull request controls

[shivamhwp]

What changed

Pull request rows, linked CI checks and timeline commits now show a pointer cursor when hovered. Checks without a URL keep the default cursor. The Summary, Timeline and Code tabs already inherit pointer styling from Toggle.

Why

These controls opened a review, check or commit while displaying the ordinary cursor, making their click targets harder to recognize.

Fixes #7606

UI changes

Before

After

Validation

Verified the live PR row changes from computed cursor default to pointer; screenshots include the actual cursor in an isolated Linux window. Web typecheck passes. Scoped lint has one existing array-index-key warning.

The detail panel returned a GitHub CLI error in this environment, so linked-check and timeline cursor changes were inspected in source but not exercised in that panel. Web and desktop share these components. Mobile uses a separate touch UI. No action handlers, contracts, provider behavior or connection handling changed.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for UI changes

Model: GPT-6. Harness: Codex in T3 Code.

Summary by CodeRabbit

• Style
  • Updated hover cursors across pull request rows, checks with available links, and timeline commit events to clearly indicate clickable elements.
  • Preserved the default cursor for checks without links.
  • Improved pull request title-row indicators by displaying approval status and check details alongside the title.
  • Approved pull requests now use the approval glyph, while “Changes requested” remains highlighted in amber.

[macroscopeapp]

Approvability

Verdict: Approved at 3c01874

Macroscope's review found this PR approvable — This three-file change only improves pointer-cursor affordances on existing pull request controls. It does not alter handlers, disabled states, data flow, or other runtime behavior.

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 6012e1cc-87f9-4821-9156-1741fcda4c69

📥 Commits

Reviewing files that changed from the base of the PR and between f2dd98d and 3c01874.

📒 Files selected for processing (2)

• apps/web/src/components/pullRequest/PullRequestRow.tsx
• apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx

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

---

📝 Walkthrough

Walkthrough

Pull request rows now display approval and checks indicators in the title row. Interactive pull request controls now use pointer cursors. Checks without URLs retain the default cursor.

Changes

Pull request presentation updates

Layer / File(s)	Summary
Title-row approval and checks indicators apps/web/src/components/pullRequest/PullRequestRow.tsx	Approved verdicts use PullRequestApprovalGlyph. The checks popover and verdict display move to the title row.
Clickable control cursor states apps/web/src/components/pullRequest/PullRequestRow.tsx, apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx, apps/web/src/components/pullRequest/PullRequestTimelineTab.tsx	Pull request rows, checks with URLs, and commit event buttons now use cursor-pointer. Checks without URLs retain cursor-default.

Priority: ⬇️ Low

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

Change: Bug fix · Severity of issue fixed: Low

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 3c018

The change is limited to pull request presentation and cursor styling, with typecheck passing.

🚥 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 3 functions across 3 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 and concisely describes the main change: adding pointer cursors to pull request controls.
Description check	✅ Passed	The description includes the change, rationale, UI screenshots, validation details, issue reference, and checklist. It is focused and matches the pull request objectives.
Linked Issues check	✅ Passed	Issue #7606 requires pointer cursors for clickable controls in pull request screens. The reviewed diff adds cursor-pointer to pull request rows, checks that have URLs, and timeline commit buttons. C…
Out of Scope Changes check	✅ Passed	The reviewed changes address cursor feedback for the pull request controls named by issue #7606. The focused diff confirms that PullRequestRow.tsx changes only the row utility classes. The other rep…

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/web-pr-pointer-cursors

---

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

Dismissing prior approval to re-evaluate 3c01874

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.6 KiB	13.6 KiB	0 B (0.0%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	−2 B (−0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.5 KiB	+2 B (+0.0%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	57.0 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	8	8	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.6 KiB	+11 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	−1 B (−0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.5 KiB	+12 B (+0.2%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.8 KiB	+44 B (+0.1%)	66.4 KiB	✅
Claude	Live turn messages	8	9	+1 (+12.5%)	21	✅

Baseline: 7bd7f99 · PR result: 3c01874 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 113.9 KiB
• Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.