Skip to content

Rebase hooks#21586

Merged
niik merged 5 commits intodevelopmentfrom
rebase-hooks
Feb 10, 2026
Merged

Rebase hooks#21586
niik merged 5 commits intodevelopmentfrom
rebase-hooks

Conversation

@niik
Copy link
Member

@niik niik commented Feb 9, 2026

Closes #[issue number]

Description

Adds support for intercepting pre-rebase and post-rewrite hooks during rebasing.

Screenshots

Release notes

Notes:

niik added 5 commits February 6, 2026 16:18
Introduce RebaseInteractiveOptions and update rebaseInteractive to accept an options object instead of multiple positional args. Pull GIT_EDITOR from opts, forward hook callbacks (onHookProgress/onHookFailure/onTerminalOutputAvailable) into git execution options, and conditionally add --no-verify. Update callers (reorder and squash) to pass the new options shape. Also add HookCallbackOptions import and a minor formatting tweak to getRebaseSnapshot signature. This centralizes interactive rebase configuration and enables hook/no-verify support.
Change continueRebase/rebaseInteractive to accept a RebaseInteractiveOptions object instead of multiple positional args. Move progressCallback and gitEditor into the options object and wire through additional hooks and flags (onTerminalOutputAvailable, onHookFailure, onHookProgress, noVerify). Update callers in AppStore and unit tests to pass the new options object. Also include minor formatting and destructuring cleanups across app-store.ts for line-wrapping and consistency.
@niik niik mentioned this pull request Feb 9, 2026
Copy link
Member

@sergiou87 sergiou87 left a comment

Choose a reason for hiding this comment

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

:shipit:

@niik niik merged commit 07bde9c into development Feb 10, 2026
8 checks passed
@niik niik deleted the rebase-hooks branch February 10, 2026 08:36
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.

2 participants