Skip to content

Lite: add support for moving commits below branches#14635

Open
OliverJAsh wants to merge 1 commit into
masterfrom
push-unyousslrmkn
Open

Lite: add support for moving commits below branches#14635
OliverJAsh wants to merge 1 commit into
masterfrom
push-unyousslrmkn

Conversation

@OliverJAsh

@OliverJAsh OliverJAsh commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Related: GB-1735

Copilot AI review requested due to automatic review settings July 6, 2026 20:12

Copilot AI 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.

Pull request overview

Adds Lite workspace support for moving commits below branch rows by enriching operation resolution with HeadInfoIndex so “below branch” can resolve to the bottom of the branch’s segment rather than the branch ref itself.

Changes:

  • Extend getOperations/getOperation with an optional operation context (headInfoIndex) and use it to enable “move below branch”.
  • Update workspace UI call sites to compute/pass headInfoIndex when determining available operations (navigation filtering, drag/drop, keyboard controls, dry-run labels).
  • Implement branch “below” RelativeTo resolution via segmentBottomRelativeTo(...) using headInfoIndex branch context.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
apps/lite/ui/src/routes/project/$id/workspace/WorkspacePage.tsx Builds headInfoIndex and uses it when filtering outline items by available operations in transfer mode.
apps/lite/ui/src/routes/project/$id/workspace/OutlineTree/OutlineTree.tsx Passes headInfoIndex into dry-run operation lookup so “below branch” operations can be surfaced consistently.
apps/lite/ui/src/routes/project/$id/workspace/OperationTarget.tsx Derives headInfoIndex via query select and uses it for drag/drop operation availability and tooltip labeling.
apps/lite/ui/src/routes/project/$id/workspace/OperationControls.tsx Uses headInfoIndex when computing keyboard transfer operations.
apps/lite/ui/src/operations/operation.ts Introduces optional operation context and enables “below branch” by resolving to the bottom of the branch’s segment.

@OliverJAsh OliverJAsh requested a review from samhh July 8, 2026 09:50
@OliverJAsh OliverJAsh enabled auto-merge July 8, 2026 15:08
@OliverJAsh OliverJAsh disabled auto-merge July 8, 2026 15:08
@OliverJAsh OliverJAsh marked this pull request as draft July 8, 2026 15:08
@OliverJAsh OliverJAsh marked this pull request as ready for review July 8, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants