refactor(ui): extract diff selection controller - #1037

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
hunk-web	Ignored	Preview	Sep 7, 2026 6:47pm UTC

Greptile Summary

This PR separates DiffPane’s persistent selection behavior and contextual action-bar rendering without changing the pane’s authority over review geometry.

• Moves pointer, keyboard, clipboard, comment, cancellation, and invalidation coordination into useDiffSelectionController.
• Extracts the contextual Comment/Copy/Clear UI into SelectionActionBar.
• Makes the pure action-bar planner return its complete bounds, including the pane-clamped width.
• Adds focused planner and component coverage while retaining existing DiffPane integration coverage.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete correctness, security, or repository-rule issues identified.

The refactor preserves existing selection semantics, keeps review geometry in DiffPane, and consistently reconnects selection painting, commands, cancellation, and action-bar interactions through the extracted controller.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  DP[DiffPane geometry and cursor state] --> CTX[Canonical CopySelectionContext]
  CTX --> CTRL[useDiffSelectionController]
  EVT[Pointer and keyboard events] --> CTRL
  CTRL --> PAINT[Selected row ranges and side]
  CTRL --> ACT[Copy, Comment, and Clear actions]
  CTRL --> VM[SelectionActionBar view model]
  VM --> BAR[SelectionActionBar]
  PAINT --> DP
  BAR --> ACT

_{Reviews (1): Last reviewed commit: "refactor(ui): extract diff selection con..." | Re-trigger Greptile}