would you believe that I'm adding even more generics

go.mod

@@ -14,7 +14,7 @@ require (
 	github.com/gookit/color v1.4.2
 	github.com/imdario/mergo v0.3.11
 	github.com/integrii/flaggy v1.4.0
-	github.com/jesseduffield/generics v0.0.0-20220319080325-a60171f800d5
+	github.com/jesseduffield/generics v0.0.0-20220319083513-5f145a9c0677
 	github.com/jesseduffield/go-git/v5 v5.1.2-0.20201006095850-341962be15a4
 	github.com/jesseduffield/gocui v0.3.1-0.20220227022729-69f0c798eec8
 	github.com/jesseduffield/minimal/gitignore v0.3.3-0.20211018110810-9cde264e6b1e

go.sum

@@ -66,8 +66,8 @@ github.com/integrii/flaggy v1.4.0 h1:A1x7SYx4jqu5NSrY14z8Z+0UyX2S5ygfJJrfolWR3zM
 github.com/integrii/flaggy v1.4.0/go.mod h1:tnTxHeTJbah0gQ6/K0RW0J7fMUBk9MCF5blhm43LNpI=
 github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 h1:BQSFePA1RWJOlocH6Fxy8MmwDt+yVQYULKfN0RoTN8A=
 github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99/go.mod h1:1lJo3i6rXxKeerYnT8Nvf0QmHCRC1n8sfWVwXF2Frvo=
-github.com/jesseduffield/generics v0.0.0-20220319080325-a60171f800d5 h1:mZf9Ezkd4Thuw2tj5naFeoUbHkbNiD38LQFokUGSbtQ=
-github.com/jesseduffield/generics v0.0.0-20220319080325-a60171f800d5/go.mod h1:+LLj9/WUPAP8LqCchs7P+7X0R98HiFujVFANdNaxhGk=
+github.com/jesseduffield/generics v0.0.0-20220319083513-5f145a9c0677 h1:GoP06WWOE4AvTkAavXkF40nhYsg2hI09uVLDIuxzZYI=
+github.com/jesseduffield/generics v0.0.0-20220319083513-5f145a9c0677/go.mod h1:+LLj9/WUPAP8LqCchs7P+7X0R98HiFujVFANdNaxhGk=
 github.com/jesseduffield/go-git/v5 v5.1.2-0.20201006095850-341962be15a4 h1:GOQrmaE8i+KEdB8NzAegKYd4tPn/inM0I1uo0NXFerg=
 github.com/jesseduffield/go-git/v5 v5.1.2-0.20201006095850-341962be15a4/go.mod h1:nGNEErzf+NRznT+N2SWqmHnDnF9aLgANB1CUNEan09o=
 github.com/jesseduffield/gocui v0.3.1-0.20220227022729-69f0c798eec8 h1:9N08i5kjvOfkzMj6THmIM110wPTQLdVYEOHMHT2DFiI=

pkg/commands/hosting_service/hosting_service.go

@@ -111,10 +111,10 @@ func (self *HostingServiceMgr) getCandidateServiceDomains() []ServiceDomain {
 
 			serviceDefinition, ok := serviceDefinitionByProvider[provider]
 			if !ok {
-				providerNames := []string{}
-				for _, serviceDefinition := range serviceDefinitions {
-					providerNames = append(providerNames, serviceDefinition.provider)
-				}
+				providerNames := slices.Map(serviceDefinitions, func(serviceDefinition ServiceDefinition) string {
+					return serviceDefinition.provider
+				})
+
 				self.log.Errorf("Unknown git service type: '%s'. Expected one of %s", provider, strings.Join(providerNames, ", "))
 				continue
 			}

pkg/commands/loaders/commits.go

@@ -307,6 +307,7 @@ func (self *CommitLoader) getInteractiveRebasingCommits() ([]*models.Commit, err
 
 	commits := []*models.Commit{}
 	lines := strings.Split(string(bytesContent), "\n")
+
 	for _, line := range lines {
 		if line == "" || line == "noop" {
 			return commits, nil
@@ -315,12 +316,12 @@ func (self *CommitLoader) getInteractiveRebasingCommits() ([]*models.Commit, err
 			continue
 		}
 		splitLine := strings.Split(line, " ")
-		commits = append([]*models.Commit{{
+		commits = slices.Prepend(commits, &models.Commit{
 			Sha:    splitLine[1],
 			Name:   strings.Join(splitLine[2:], " "),
 			Status: "rebasing",
 			Action: splitLine[0],
-		}}, commits...)
+		})
 	}
 
 	return commits, nil

pkg/commands/loaders/remotes.go

@@ -10,7 +10,6 @@ import (
 	"github.com/jesseduffield/lazygit/pkg/commands/models"
 	"github.com/jesseduffield/lazygit/pkg/commands/oscommands"
 	"github.com/jesseduffield/lazygit/pkg/common"
-	"github.com/samber/lo"
 )
 
 type RemoteLoader struct {
@@ -43,12 +42,12 @@ func (self *RemoteLoader) GetRemotes() ([]*models.Remote, error) {
 	}
 
 	// first step is to get our remotes from go-git
-	remotes := lo.Map(goGitRemotes, func(goGitRemote *gogit.Remote, _ int) *models.Remote {
+	remotes := slices.Map(goGitRemotes, func(goGitRemote *gogit.Remote) *models.Remote {
 		remoteName := goGitRemote.Config().Name
 
 		re := regexp.MustCompile(fmt.Sprintf(`(?m)^\s*%s\/([\S]+)`, remoteName))
 		matches := re.FindAllStringSubmatch(remoteBranchesStr, -1)
-		branches := lo.Map(matches, func(match []string, _ int) *models.RemoteBranch {
+		branches := slices.Map(matches, func(match []string) *models.RemoteBranch {
 			return &models.RemoteBranch{
 				Name:       match[1],
 				RemoteName: remoteName,

pkg/commands/loaders/stash.go

@@ -5,6 +5,7 @@ import (
 	"strconv"
 	"strings"
 
+	"github.com/jesseduffield/generics/slices"
 	"github.com/jesseduffield/lazygit/pkg/commands/models"
 	"github.com/jesseduffield/lazygit/pkg/commands/oscommands"
 	"github.com/jesseduffield/lazygit/pkg/common"
@@ -65,11 +66,9 @@ outer:
 
 func (self *StashLoader) getUnfilteredStashEntries() []*models.StashEntry {
 	rawString, _ := self.cmd.New("git stash list --pretty='%gs'").DontLog().RunWithOutput()
-	stashEntries := []*models.StashEntry{}
-	for i, line := range utils.SplitLines(rawString) {
-		stashEntries = append(stashEntries, self.stashEntryFromLine(line, i))
-	}
-	return stashEntries
+	return slices.MapWithIndex(utils.SplitLines(rawString), func(line string, index int) *models.StashEntry {
+		return self.stashEntryFromLine(line, index)
+	})
 }
 
 func (c *StashLoader) stashEntryFromLine(line string, index int) *models.StashEntry {

pkg/commands/patch/patch_parser.go

@@ -191,7 +191,7 @@ func (p *PatchParser) Render(firstLineIndex int, lastLineIndex int, incLineIndic
 		return ""
 	}
 
-	renderedLines := lo.Map(p.PatchLines, func(patchLine *PatchLine, index int) string {
+	renderedLines := slices.MapWithIndex(p.PatchLines, func(patchLine *PatchLine, index int) string {
 		selected := index >= firstLineIndex && index <= lastLineIndex
 		included := lo.Contains(incLineIndices, index)
 		return patchLine.render(selected, included)

pkg/gui/app_status_manager.go

@@ -4,6 +4,7 @@ import (
 	"sync"
 	"time"
 
+	"github.com/jesseduffield/generics/slices"
 	"github.com/jesseduffield/lazygit/pkg/utils"
 )
 
@@ -23,13 +24,9 @@ func (m *statusManager) removeStatus(id int) {
 	m.mutex.Lock()
 	defer m.mutex.Unlock()
 
-	newStatuses := []appStatus{}
-	for _, status := range m.statuses {
-		if status.id != id {
-			newStatuses = append(newStatuses, status)
-		}
-	}
-	m.statuses = newStatuses
+	m.statuses = slices.Filter(m.statuses, func(status appStatus) bool {
+		return status.id != id
+	})
 }
 
 func (m *statusManager) addWaitingStatus(message string) int {

pkg/gui/context.go

@@ -14,14 +14,14 @@ import (
 )
 
 func (gui *Gui) popupViewNames() []string {
-	result := []string{}
-	for _, context := range gui.State.Contexts.Flatten() {
-		if context.GetKind() == types.PERSISTENT_POPUP || context.GetKind() == types.TEMPORARY_POPUP {
-			result = append(result, context.GetViewName())
-		}
-	}
-
-	return result
+	return slices.FilterThenMap(gui.State.Contexts.Flatten(),
+		func(c types.Context) bool {
+			return c.GetKind() == types.PERSISTENT_POPUP || c.GetKind() == types.TEMPORARY_POPUP
+		},
+		func(c types.Context) string {
+			return c.GetViewName()
+		},
+	)
 }
 
 func (gui *Gui) currentContextKeyIgnoringPopups() types.ContextKey {

pkg/gui/controllers/helpers/cherry_pick_helper.go

@@ -8,7 +8,6 @@ import (
 	"github.com/jesseduffield/lazygit/pkg/gui/context"
 	"github.com/jesseduffield/lazygit/pkg/gui/modes/cherrypicking"
 	"github.com/jesseduffield/lazygit/pkg/gui/types"
-	"github.com/samber/lo"
 )
 
 type CherryPickHelper struct {
@@ -109,7 +108,7 @@ func (self *CherryPickHelper) Reset() error {
 }
 
 func (self *CherryPickHelper) CherryPickedCommitShaSet() *set.Set[string] {
-	shas := lo.Map(self.getData().CherryPickedCommits, func(commit *models.Commit, _ int) string {
+	shas := slices.Map(self.getData().CherryPickedCommits, func(commit *models.Commit) string {
 		return commit.Sha
 	})
 	return set.NewFromSlice(shas)

pkg/gui/filetree/inode.go

@@ -200,10 +200,7 @@ func getLeaves(node INode) []INode {
 		return []INode{node}
 	}
 
-	output := []INode{}
-	for _, child := range node.GetChildren() {
-		output = append(output, getLeaves(child)...)
-	}
-
-	return output
+	return slices.FlatMap(node.GetChildren(), func(child INode) []INode {
+		return getLeaves(child)
+	})
 }

pkg/gui/presentation/graph/graph.go

@@ -67,13 +67,10 @@ func GetPipeSets(commits []*models.Commit, getStyle func(c *models.Commit) style
 
 	pipes := []*Pipe{{fromPos: 0, toPos: 0, fromSha: "START", toSha: commits[0].Sha, kind: STARTS, style: style.FgDefault}}
 
-	pipeSets := [][]*Pipe{}
-	for _, commit := range commits {
+	return slices.Map(commits, func(commit *models.Commit) []*Pipe {
 		pipes = getNextPipes(pipes, commit, getStyle)
-		pipeSets = append(pipeSets, pipes)
-	}
-
-	return pipeSets
+		return pipes
+	})
 }
 
 func RenderAux(pipeSets [][]*Pipe, commits []*models.Commit, selectedCommitSha string) []string {
@@ -115,9 +112,9 @@ func RenderAux(pipeSets [][]*Pipe, commits []*models.Commit, selectedCommitSha s
 }
 
 func getNextPipes(prevPipes []*Pipe, commit *models.Commit, getStyle func(c *models.Commit) style.TextStyle) []*Pipe {
-	maxPos := lo.Max(
-		slices.Map(prevPipes, func(pipe *Pipe) int { return pipe.toPos }),
-	)
+	maxPos := slices.MaxBy(prevPipes, func(pipe *Pipe) int {
+		return pipe.toPos
+	})
 
 	// a pipe that terminated in the previous line has no bearing on the current line
 	// so we'll filter those out

vendor/github.com/jesseduffield/generics/slices/slices.go

@@ -37,6 +37,16 @@ func Map[T any, V any](slice []T, f func(T) V) []V {
 	return result
 }
 
+// Produces a new slice, leaves the input slice untouched.
+func MapWithIndex[T any, V any](slice []T, f func(T, int) V) []V {
+	result := make([]V, 0, len(slice))
+	for i, value := range slice {
+		result = append(result, f(value, i))
+	}
+
+	return result
+}
+
 // Produces a new slice, leaves the input slice untouched.
 func FlatMap[T any, V any](slice []T, f func(T) []V) []V {
 	// impossible to know how long this slice will be in the end but the length
@@ -74,6 +84,18 @@ func Filter[T any](slice []T, test func(T) bool) []T {
 	return result
 }
 
+// Produces a new slice, leaves the input slice untouched.
+func FilterWithIndex[T any](slice []T, f func(T, int) bool) []T {
+	result := make([]T, 0, len(slice))
+	for i, value := range slice {
+		if f(value, i) {
+			result = append(result, value)
+		}
+	}
+
+	return result
+}
+
 // Mutates original slice. Intended usage is to reassign the slice result to the input slice.
 func FilterInPlace[T any](slice []T, test func(T) bool) []T {
 	newLength := 0

vendor/modules.txt

@@ -120,7 +120,7 @@ github.com/integrii/flaggy
 # github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99
 ## explicit
 github.com/jbenet/go-context/io
-# github.com/jesseduffield/generics v0.0.0-20220319080325-a60171f800d5
+# github.com/jesseduffield/generics v0.0.0-20220319083513-5f145a9c0677
 ## explicit; go 1.18
 github.com/jesseduffield/generics/maps
 github.com/jesseduffield/generics/set