split context common from helper common

25 files changed, 67 insertions, 69 deletions

diff --git a/pkg/gui/controllers/helpers/bisect_helper.go b/pkg/gui/controllers/helpers/bisect_helper.go
index 65fb781d4..7d0fadf8a 100644
--- a/pkg/gui/controllers/helpers/bisect_helper.go
+++ b/pkg/gui/controllers/helpers/bisect_helper.go
@@ -1,23 +1,15 @@
 package helpers
 
 import (
-	"github.com/jesseduffield/lazygit/pkg/commands"
 	"github.com/jesseduffield/lazygit/pkg/gui/types"
 )
 
 type BisectHelper struct {
-	c   *types.HelperCommon
-	git *commands.GitCommand
+	c *HelperCommon
 }
 
-func NewBisectHelper(
-	c *types.HelperCommon,
-	git *commands.GitCommand,
-) *BisectHelper {
-	return &BisectHelper{
-		c:   c,
-		git: git,
-	}
+func NewBisectHelper(c *HelperCommon) *BisectHelper {
+	return &BisectHelper{c: c}
 }
 
 func (self *BisectHelper) Reset() error {
@@ -26,7 +18,7 @@ func (self *BisectHelper) Reset() error {
 		Prompt: self.c.Tr.Bisect.ResetPrompt,
 		HandleConfirm: func() error {
 			self.c.LogAction(self.c.Tr.Actions.ResetBisect)
-			if err := self.git.Bisect.Reset(); err != nil {
+			if err := self.c.Git().Bisect.Reset(); err != nil {
 				return self.c.Error(err)
 			}
 
diff --git a/pkg/gui/controllers/helpers/cherry_pick_helper.go b/pkg/gui/controllers/helpers/cherry_pick_helper.go
index 84b77a299..695a726ae 100644
--- a/pkg/gui/controllers/helpers/cherry_pick_helper.go
+++ b/pkg/gui/controllers/helpers/cherry_pick_helper.go
@@ -1,7 +1,6 @@
 package helpers
 
 import (
-	"github.com/jesseduffield/lazygit/pkg/commands"
 	"github.com/jesseduffield/lazygit/pkg/commands/models"
 	"github.com/jesseduffield/lazygit/pkg/gui/context"
 	"github.com/jesseduffield/lazygit/pkg/gui/modes/cherrypicking"
@@ -9,9 +8,7 @@ import (
 )
 
 type CherryPickHelper struct {
-	c *types.HelperCommon
-
-	git *commands.GitCommand
+	c *HelperCommon
 
 	contexts *context.ContextTree
 
@@ -22,14 +19,12 @@ type CherryPickHelper struct {
 // even if in truth we're running git cherry-pick
 
 func NewCherryPickHelper(
-	c *types.HelperCommon,
-	git *commands.GitCommand,
+	c *HelperCommon,
 	contexts *context.ContextTree,
 	rebaseHelper *MergeAndRebaseHelper,
 ) *CherryPickHelper {
 	return &CherryPickHelper{
 		c:            c,
-		git:          git,
 		contexts:     contexts,
 		rebaseHelper: rebaseHelper,
 	}
@@ -87,7 +82,7 @@ func (self *CherryPickHelper) Paste() error {
 		HandleConfirm: func() error {
 			return self.c.WithWaitingStatus(self.c.Tr.CherryPickingStatus, func() error {
 				self.c.LogAction(self.c.Tr.Actions.CherryPick)
-				err := self.git.Rebase.CherryPickCommits(self.getData().CherryPickedCommits)
+				err := self.c.Git().Rebase.CherryPickCommits(self.getData().CherryPickedCommits)
 				return self.rebaseHelper.CheckMergeOrRebase(err)
 			})
 		},
diff --git a/pkg/gui/controllers/helpers/confirmation_helper.go b/pkg/gui/controllers/helpers/confirmation_helper.go
index 04211b7ac..d7ccb151a 100644
--- a/pkg/gui/controllers/helpers/confirmation_helper.go
+++ b/pkg/gui/controllers/helpers/confirmation_helper.go
@@ -15,12 +15,12 @@ import (
 )
 
 type ConfirmationHelper struct {
-	c        *types.HelperCommon
+	c        *HelperCommon
 	contexts *context.ContextTree
 }
 
 func NewConfirmationHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	contexts *context.ContextTree,
 ) *ConfirmationHelper {
 	return &ConfirmationHelper{
diff --git a/pkg/gui/controllers/helpers/credentials_helper.go b/pkg/gui/controllers/helpers/credentials_helper.go
index 10679d7a4..0aed34110 100644
--- a/pkg/gui/controllers/helpers/credentials_helper.go
+++ b/pkg/gui/controllers/helpers/credentials_helper.go
@@ -8,11 +8,11 @@ import (
 )
 
 type CredentialsHelper struct {
-	c *types.HelperCommon
+	c *HelperCommon
 }
 
 func NewCredentialsHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 ) *CredentialsHelper {
 	return &CredentialsHelper{
 		c: c,
diff --git a/pkg/gui/controllers/helpers/diff_helper.go b/pkg/gui/controllers/helpers/diff_helper.go
index 952104f81..701df93cd 100644
--- a/pkg/gui/controllers/helpers/diff_helper.go
+++ b/pkg/gui/controllers/helpers/diff_helper.go
@@ -10,10 +10,10 @@ import (
 )
 
 type DiffHelper struct {
-	c *types.HelperCommon
+	c *HelperCommon
 }
 
-func NewDiffHelper(c *types.HelperCommon) *DiffHelper {
+func NewDiffHelper(c *HelperCommon) *DiffHelper {
 	return &DiffHelper{
 		c: c,
 	}
diff --git a/pkg/gui/controllers/helpers/files_helper.go b/pkg/gui/controllers/helpers/files_helper.go
index a0d2d4e8c..9893123cb 100644
--- a/pkg/gui/controllers/helpers/files_helper.go
+++ b/pkg/gui/controllers/helpers/files_helper.go
@@ -3,7 +3,6 @@ package helpers
 import (
 	"github.com/jesseduffield/lazygit/pkg/commands"
 	"github.com/jesseduffield/lazygit/pkg/commands/oscommands"
-	"github.com/jesseduffield/lazygit/pkg/gui/types"
 )
 
 type IFilesHelper interface {
@@ -13,13 +12,13 @@ type IFilesHelper interface {
 }
 
 type FilesHelper struct {
-	c   *types.HelperCommon
+	c   *HelperCommon
 	git *commands.GitCommand
 	os  *oscommands.OSCommand
 }
 
 func NewFilesHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	git *commands.GitCommand,
 	os *oscommands.OSCommand,
 ) *FilesHelper {
diff --git a/pkg/gui/controllers/helpers/gpg_helper.go b/pkg/gui/controllers/helpers/gpg_helper.go
index 2e287c2b4..0249c560f 100644
--- a/pkg/gui/controllers/helpers/gpg_helper.go
+++ b/pkg/gui/controllers/helpers/gpg_helper.go
@@ -9,13 +9,13 @@ import (
 )
 
 type GpgHelper struct {
-	c   *types.HelperCommon
+	c   *HelperCommon
 	os  *oscommands.OSCommand
 	git *commands.GitCommand
 }
 
 func NewGpgHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	os *oscommands.OSCommand,
 	git *commands.GitCommand,
 ) *GpgHelper {
diff --git a/pkg/gui/controllers/helpers/helpers.go b/pkg/gui/controllers/helpers/helpers.go
index 8c676f4de..c250a09b5 100644
--- a/pkg/gui/controllers/helpers/helpers.go
+++ b/pkg/gui/controllers/helpers/helpers.go
@@ -1,5 +1,21 @@
 package helpers
 
+import (
+	"github.com/jesseduffield/lazygit/pkg/common"
+	"github.com/jesseduffield/lazygit/pkg/gui/context"
+	"github.com/jesseduffield/lazygit/pkg/gui/types"
+)
+
+type HelperCommon struct {
+	*common.Common
+	types.IGuiCommon
+	IGetContexts
+}
+
+type IGetContexts interface {
+	Contexts() *context.ContextTree
+}
+
 type Helpers struct {
 	Refs           *RefsHelper
 	Bisect         *BisectHelper
diff --git a/pkg/gui/controllers/helpers/host_helper.go b/pkg/gui/controllers/helpers/host_helper.go
index edc0bc7ba..b11bc32c2 100644
--- a/pkg/gui/controllers/helpers/host_helper.go
+++ b/pkg/gui/controllers/helpers/host_helper.go
@@ -3,7 +3,6 @@ package helpers
 import (
 	"github.com/jesseduffield/lazygit/pkg/commands"
 	"github.com/jesseduffield/lazygit/pkg/commands/hosting_service"
-	"github.com/jesseduffield/lazygit/pkg/gui/types"
 )
 
 // this helper just wraps our hosting_service package
@@ -14,12 +13,12 @@ type IHostHelper interface {
 }
 
 type HostHelper struct {
-	c   *types.HelperCommon
+	c   *HelperCommon
 	git *commands.GitCommand
 }
 
 func NewHostHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	git *commands.GitCommand,
 ) *HostHelper {
 	return &HostHelper{
diff --git a/pkg/gui/controllers/helpers/merge_and_rebase_helper.go b/pkg/gui/controllers/helpers/merge_and_rebase_helper.go
index 981b85a0d..56154e6ad 100644
--- a/pkg/gui/controllers/helpers/merge_and_rebase_helper.go
+++ b/pkg/gui/controllers/helpers/merge_and_rebase_helper.go
@@ -14,14 +14,14 @@ import (
 )
 
 type MergeAndRebaseHelper struct {
-	c          *types.HelperCommon
+	c          *HelperCommon
 	contexts   *context.ContextTree
 	git        *commands.GitCommand
 	refsHelper *RefsHelper
 }
 
 func NewMergeAndRebaseHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	contexts *context.ContextTree,
 	git *commands.GitCommand,
 	refsHelper *RefsHelper,
diff --git a/pkg/gui/controllers/helpers/merge_conflicts_helper.go b/pkg/gui/controllers/helpers/merge_conflicts_helper.go
index b099a69cd..c84cf4540 100644
--- a/pkg/gui/controllers/helpers/merge_conflicts_helper.go
+++ b/pkg/gui/controllers/helpers/merge_conflicts_helper.go
@@ -7,13 +7,13 @@ import (
 )
 
 type MergeConflictsHelper struct {
-	c        *types.HelperCommon
+	c        *HelperCommon
 	contexts *context.ContextTree
 	git      *commands.GitCommand
 }
 
 func NewMergeConflictsHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	contexts *context.ContextTree,
 	git *commands.GitCommand,
 ) *MergeConflictsHelper {
diff --git a/pkg/gui/controllers/helpers/patch_building_helper.go b/pkg/gui/controllers/helpers/patch_building_helper.go
index b58812733..08bb8122d 100644
--- a/pkg/gui/controllers/helpers/patch_building_helper.go
+++ b/pkg/gui/controllers/helpers/patch_building_helper.go
@@ -13,13 +13,13 @@ type IPatchBuildingHelper interface {
 }
 
 type PatchBuildingHelper struct {
-	c        *types.HelperCommon
+	c        *HelperCommon
 	git      *commands.GitCommand
 	contexts *context.ContextTree
 }
 
 func NewPatchBuildingHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	git *commands.GitCommand,
 	contexts *context.ContextTree,
 ) *PatchBuildingHelper {
diff --git a/pkg/gui/controllers/helpers/record_directory_helper.go b/pkg/gui/controllers/helpers/record_directory_helper.go
index 4877a6b46..377e3b94a 100644
--- a/pkg/gui/controllers/helpers/record_directory_helper.go
+++ b/pkg/gui/controllers/helpers/record_directory_helper.go
@@ -2,15 +2,13 @@ package helpers
 
 import (
 	"os"
-
-	"github.com/jesseduffield/lazygit/pkg/gui/types"
 )
 
 type RecordDirectoryHelper struct {
-	c *types.HelperCommon
+	c *HelperCommon
 }
 
-func NewRecordDirectoryHelper(c *types.HelperCommon) *RecordDirectoryHelper {
+func NewRecordDirectoryHelper(c *HelperCommon) *RecordDirectoryHelper {
 	return &RecordDirectoryHelper{
 		c: c,
 	}
diff --git a/pkg/gui/controllers/helpers/refresh_helper.go b/pkg/gui/controllers/helpers/refresh_helper.go
index 9a1089de3..132c65545 100644
--- a/pkg/gui/controllers/helpers/refresh_helper.go
+++ b/pkg/gui/controllers/helpers/refresh_helper.go
@@ -21,7 +21,7 @@ import (
 )
 
 type RefreshHelper struct {
-	c                    *types.HelperCommon
+	c                    *HelperCommon
 	contexts             *context.ContextTree
 	git                  *commands.GitCommand
 	refsHelper           *RefsHelper
@@ -33,7 +33,7 @@ type RefreshHelper struct {
 }
 
 func NewRefreshHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	contexts *context.ContextTree,
 	git *commands.GitCommand,
 	refsHelper *RefsHelper,
diff --git a/pkg/gui/controllers/helpers/refs_helper.go b/pkg/gui/controllers/helpers/refs_helper.go
index c03a2b24c..ad295e59c 100644
--- a/pkg/gui/controllers/helpers/refs_helper.go
+++ b/pkg/gui/controllers/helpers/refs_helper.go
@@ -23,14 +23,14 @@ type IRefsHelper interface {
 }
 
 type RefsHelper struct {
-	c        *types.HelperCommon
+	c        *HelperCommon
 	git      *commands.GitCommand
 	contexts *context.ContextTree
 	model    *types.Model
 }
 
 func NewRefsHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	git *commands.GitCommand,
 	contexts *context.ContextTree,
 	model *types.Model,
diff --git a/pkg/gui/controllers/helpers/repos_helper.go b/pkg/gui/controllers/helpers/repos_helper.go
index ef983e856..060724a0a 100644
--- a/pkg/gui/controllers/helpers/repos_helper.go
+++ b/pkg/gui/controllers/helpers/repos_helper.go
@@ -22,13 +22,13 @@ type onNewRepoFn func(startArgs appTypes.StartArgs, reuseState bool) error
 
 // helps switch back and forth between repos
 type ReposHelper struct {
-	c                     *types.HelperCommon
+	c                     *HelperCommon
 	recordDirectoryHelper *RecordDirectoryHelper
 	onNewRepo             onNewRepoFn
 }
 
 func NewRecentReposHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	recordDirectoryHelper *RecordDirectoryHelper,
 	onNewRepo onNewRepoFn,
 ) *ReposHelper {
diff --git a/pkg/gui/controllers/helpers/snake_helper.go b/pkg/gui/controllers/helpers/snake_helper.go
index 5f53273cf..fb4d67bb4 100644
--- a/pkg/gui/controllers/helpers/snake_helper.go
+++ b/pkg/gui/controllers/helpers/snake_helper.go
@@ -5,16 +5,15 @@ import (
 	"strings"
 
 	"github.com/jesseduffield/lazygit/pkg/gui/style"
-	"github.com/jesseduffield/lazygit/pkg/gui/types"
 	"github.com/jesseduffield/lazygit/pkg/snake"
 )
 
 type SnakeHelper struct {
-	c    *types.HelperCommon
+	c    *HelperCommon
 	game *snake.Game
 }
 
-func NewSnakeHelper(c *types.HelperCommon) *SnakeHelper {
+func NewSnakeHelper(c *HelperCommon) *SnakeHelper {
 	return &SnakeHelper{
 		c: c,
 	}
diff --git a/pkg/gui/controllers/helpers/staging_helper.go b/pkg/gui/controllers/helpers/staging_helper.go
index 4b4eb2626..dee6a0fa1 100644
--- a/pkg/gui/controllers/helpers/staging_helper.go
+++ b/pkg/gui/controllers/helpers/staging_helper.go
@@ -9,13 +9,13 @@ import (
 )
 
 type StagingHelper struct {
-	c        *types.HelperCommon
+	c        *HelperCommon
 	git      *commands.GitCommand
 	contexts *context.ContextTree
 }
 
 func NewStagingHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	git *commands.GitCommand,
 	contexts *context.ContextTree,
 ) *StagingHelper {
diff --git a/pkg/gui/controllers/helpers/suggestions_helper.go b/pkg/gui/controllers/helpers/suggestions_helper.go
index 53a076f66..206fd371a 100644
--- a/pkg/gui/controllers/helpers/suggestions_helper.go
+++ b/pkg/gui/controllers/helpers/suggestions_helper.go
@@ -34,7 +34,7 @@ type ISuggestionsHelper interface {
 }
 
 type SuggestionsHelper struct {
-	c *types.HelperCommon
+	c *HelperCommon
 
 	model    *types.Model
 	contexts *context.ContextTree
@@ -43,7 +43,7 @@ type SuggestionsHelper struct {
 var _ ISuggestionsHelper = &SuggestionsHelper{}
 
 func NewSuggestionsHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	model *types.Model,
 	contexts *context.ContextTree,
 ) *SuggestionsHelper {
diff --git a/pkg/gui/controllers/helpers/tags_helper.go b/pkg/gui/controllers/helpers/tags_helper.go
index a2f2e9c77..7f17be222 100644
--- a/pkg/gui/controllers/helpers/tags_helper.go
+++ b/pkg/gui/controllers/helpers/tags_helper.go
@@ -10,11 +10,11 @@ import (
 // and the commits context.
 
 type TagsHelper struct {
-	c   *types.HelperCommon
+	c   *HelperCommon
 	git *commands.GitCommand
 }
 
-func NewTagsHelper(c *types.HelperCommon, git *commands.GitCommand) *TagsHelper {
+func NewTagsHelper(c *HelperCommon, git *commands.GitCommand) *TagsHelper {
 	return &TagsHelper{
 		c:   c,
 		git: git,
diff --git a/pkg/gui/controllers/helpers/update_helper.go b/pkg/gui/controllers/helpers/update_helper.go
index 21bf2ad83..a9a49f0f4 100644
--- a/pkg/gui/controllers/helpers/update_helper.go
+++ b/pkg/gui/controllers/helpers/update_helper.go
@@ -7,11 +7,11 @@ import (
 )
 
 type UpdateHelper struct {
-	c       *types.HelperCommon
+	c       *HelperCommon
 	updater *updates.Updater
 }
 
-func NewUpdateHelper(c *types.HelperCommon, updater *updates.Updater) *UpdateHelper {
+func NewUpdateHelper(c *HelperCommon, updater *updates.Updater) *UpdateHelper {
 	return &UpdateHelper{
 		c:       c,
 		updater: updater,
diff --git a/pkg/gui/controllers/helpers/upstream_helper.go b/pkg/gui/controllers/helpers/upstream_helper.go
index a2d8e8ae2..b4825ffaa 100644
--- a/pkg/gui/controllers/helpers/upstream_helper.go
+++ b/pkg/gui/controllers/helpers/upstream_helper.go
@@ -9,7 +9,7 @@ import (
 )
 
 type UpstreamHelper struct {
-	c     *types.HelperCommon
+	c     *HelperCommon
 	model *types.Model
 
 	getRemoteBranchesSuggestionsFunc func(string) func(string) []*types.Suggestion
@@ -25,7 +25,7 @@ type IUpstreamHelper interface {
 var _ IUpstreamHelper = &UpstreamHelper{}
 
 func NewUpstreamHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	model *types.Model,
 	getRemoteBranchesSuggestionsFunc func(string) func(string) []*types.Suggestion,
 ) *UpstreamHelper {
diff --git a/pkg/gui/controllers/helpers/view_helper.go b/pkg/gui/controllers/helpers/view_helper.go
index bbe7742cf..e1d837da5 100644
--- a/pkg/gui/controllers/helpers/view_helper.go
+++ b/pkg/gui/controllers/helpers/view_helper.go
@@ -6,11 +6,11 @@ import (
 )
 
 type ViewHelper struct {
-	c        *types.HelperCommon
+	c        *HelperCommon
 	contexts *context.ContextTree
 }
 
-func NewViewHelper(c *types.HelperCommon, contexts *context.ContextTree) *ViewHelper {
+func NewViewHelper(c *HelperCommon, contexts *context.ContextTree) *ViewHelper {
 	return &ViewHelper{
 		c:        c,
 		contexts: contexts,
diff --git a/pkg/gui/controllers/helpers/window_helper.go b/pkg/gui/controllers/helpers/window_helper.go
index 3b180ad80..8754cc41d 100644
--- a/pkg/gui/controllers/helpers/window_helper.go
+++ b/pkg/gui/controllers/helpers/window_helper.go
@@ -11,12 +11,12 @@ import (
 )
 
 type WindowHelper struct {
-	c          *types.HelperCommon
+	c          *HelperCommon
 	viewHelper *ViewHelper
 	contexts   *context.ContextTree
 }
 
-func NewWindowHelper(c *types.HelperCommon, viewHelper *ViewHelper, contexts *context.ContextTree) *WindowHelper {
+func NewWindowHelper(c *HelperCommon, viewHelper *ViewHelper, contexts *context.ContextTree) *WindowHelper {
 	return &WindowHelper{
 		c:          c,
 		viewHelper: viewHelper,
diff --git a/pkg/gui/controllers/helpers/working_tree_helper.go b/pkg/gui/controllers/helpers/working_tree_helper.go
index 17850b994..1e3c211b0 100644
--- a/pkg/gui/controllers/helpers/working_tree_helper.go
+++ b/pkg/gui/controllers/helpers/working_tree_helper.go
@@ -20,7 +20,7 @@ type IWorkingTreeHelper interface {
 }
 
 type WorkingTreeHelper struct {
-	c                     *types.HelperCommon
+	c                     *HelperCommon
 	git                   *commands.GitCommand
 	contexts              *context.ContextTree
 	refHelper             *RefsHelper
@@ -30,7 +30,7 @@ type WorkingTreeHelper struct {
 }
 
 func NewWorkingTreeHelper(
-	c *types.HelperCommon,
+	c *HelperCommon,
 	git *commands.GitCommand,
 	contexts *context.ContextTree,
 	refHelper *RefsHelper,