fix(web): share touch-visible pull request edit actions

[juliusmarminge]

Targets main independently. This PR has no dependency on V2 or another UI audit PR.

Five pull-request edit buttons duplicated hover-only visibility and remained hidden on touch screens. PullRequestEditButton now owns the pencil icon, focus behavior, motion defaults, and coarse-pointer visibility. Permissions, edit targets, handlers, and local placement remain with each consumer.

Validation: focused pull-request component tests, affected client typecheck, scoped lint, and React Doctor review. React Doctor’s existing consumer findings remain documented from the original audit; no new component findings were identified.

Code-only verification was requested; no browser/device pass or before/after screenshots are included. This remains a draft for visual review. No wire contracts, persisted data, or upgrade migrations change.

Model: GPT-6. Harness: Codex.

Summary by CodeRabbit

• UI Improvements
  • Standardized edit controls across pull request titles, descriptions, and comments.
  • Edit buttons now provide consistent styling and behavior across pull request views.
  • Improved edit control visibility for touch devices, keyboard navigation, and focused elements.
  • Added accessible labeling requirements to edit controls.

[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.5 KiB	13.5 KiB	−16 B (−0.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.1 KiB	7.1 KiB	+3 B (+0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.4 KiB	−19 B (−0.3%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	56.3 KiB	56.2 KiB	−44 B (−0.1%)	66.4 KiB	✅
Codex	Live turn messages	10	9	−1 (−10.0%)	21	✅
Claude	Total thread wire	13.5 KiB	13.5 KiB	+2 B (+0.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	0 B (0.0%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+2 B (+0.0%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.0 KiB	57.0 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	9	9	0 (0.0%)	21	✅

Baseline: 53510d4 · PR result: f613f9c · 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.

[macroscopeapp]

Approvability

Verdict: Approved at f613f9c

Macroscope's review found this PR approvable — This is a small, isolated web UI fix that centralizes existing pull-request edit buttons and makes them visible on touch devices without changing permissions, handlers, data contracts, or other production workflows. The runtime impact is limited to presentation and accessibility 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: Team

Run ID: 65689275-8f86-4985-be8a-a6aee8084e5d

📥 Commits

Reviewing files that changed from the base of the PR and between 53510d4 and f613f9c.

📒 Files selected for processing (5)

• apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx
• apps/web/src/components/pullRequest/PullRequestEditButton.tsx
• apps/web/src/components/pullRequest/PullRequestReviewAnnotation.tsx
• apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx
• apps/web/src/components/pullRequest/PullRequestTimelineTab.tsx

Limit details: You’ve used all 10 included reviews currently available.

---

📝 Walkthrough

Walkthrough

The pull request adds a shared PullRequestEditButton and uses it for title, comment, and description editing. Existing edit labels and callbacks remain unchanged.

Changes

Pull request edit controls

Layer / File(s)	Summary
Shared edit button apps/web/src/components/pullRequest/PullRequestEditButton.tsx	Adds a reusable ghost edit button with a required accessible label, pencil icon, pointer and focus visibility styles, and reduced-motion support.
Edit control adoption apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx, apps/web/src/components/pullRequest/PullRequestReviewAnnotation.tsx, apps/web/src/components/pullRequest/PullRequestSummaryTab.tsx, apps/web/src/components/pullRequest/PullRequestTimelineTab.tsx	Replaces inline edit buttons with PullRequestEditButton for pull request titles, comments, and descriptions. Removes the replaced direct PencilIcon imports. Existing callbacks and labels remain in place.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: maria-rcks

Merge Risk: ⚪ Minimal · up to f613f

The edit controls retain their existing actions and remain available through hover, focus, and coarse-pointer interactions; no merge-blocking regression was identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 5 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 summarizes the main change: shared pull-request edit actions are visible on touch devices.
Description check	✅ Passed	The description explains what changed, why it changed, scope boundaries, and validation. It also states that browser/device validation and screenshots are not included. The template headings and check…
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 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

Usage-based review receipt

• Mode: Continue automatically
• Reviewed files: 5
• Charged: $1.25
• View usage details

Note

This review was completed with usage-based billing: files reviewed beyond your plan's included limits are billed at $0.25/file. View usage-based billing.

---

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