feat: Allow hover to file

Author: lazysegtree

src/cmd/main.go

@@ -120,9 +120,9 @@ func Run(content embed.FS) {
 
 func spfAppAction(_ context.Context, c *cli.Command) error {
 	// If no args are called along with "spf" use current dir
-	firstFilePanelDirs := []string{""}
+	firstPanelPaths := []string{""}
 	if c.Args().Present() {
-		firstFilePanelDirs = c.Args().Slice()
+		firstPanelPaths = c.Args().Slice()
 	}
 
 	variable.UpdateVarFromCliArgs(c)
@@ -131,7 +131,7 @@ func spfAppAction(_ context.Context, c *cli.Command) error {
 
 	firstUse := checkFirstUse()
 
-	p := tea.NewProgram(internal.InitialModel(firstFilePanelDirs, firstUse),
+	p := tea.NewProgram(internal.InitialModel(firstPanelPaths, firstUse),
 		tea.WithAltScreen(), tea.WithMouseCellMotion())
 	if _, err := p.Run(); err != nil {
 		utils.PrintfAndExitf("Alas, there's been an error: %v", err)

src/internal/config_function.go

@@ -25,7 +25,7 @@ import (
 // themes) setted up. Processes input directories and returns toggle states.
 
 // This is the only usecase of named returns, distinguish between multiple return values
-func initialConfig(firstFilePanelDirs []string) (toggleDotFile bool, //nolint: nonamedreturns // See above
+func initialConfig(firstPanelPaths []string) (toggleDotFile bool, //nolint: nonamedreturns // See above
 	toggleFooter bool, zClient *zoxidelib.Client) {
 	// Open log stream
 	file, err := os.OpenFile(variable.LogFile, os.O_RDWR|os.O_CREATE|os.O_APPEND, 0666)
@@ -80,9 +80,9 @@ func initialConfig(firstFilePanelDirs []string) (toggleDotFile bool, //nolint: n
 		}
 	}
 
-	updateFirstFilePanelDirs(firstFilePanelDirs, cwd, zClient)
+	updateFirstFilePanelPaths(firstPanelPaths, cwd, zClient)
 
-	slog.Debug("Directory configuration", "cwd", cwd, "start_directories", firstFilePanelDirs)
+	slog.Debug("Directory configuration", "cwd", cwd, "start_paths", firstPanelPaths)
 	printRuntimeInfo()
 
 	toggleDotFile = utils.ReadBoolFile(variable.ToggleDotFile, false)
@@ -91,27 +91,27 @@ func initialConfig(firstFilePanelDirs []string) (toggleDotFile bool, //nolint: n
 	return toggleDotFile, toggleFooter, zClient
 }
 
-func updateFirstFilePanelDirs(firstFilePanelDirs []string, cwd string, zClient *zoxidelib.Client) {
-	for i := range firstFilePanelDirs {
-		if firstFilePanelDirs[i] == "" {
-			firstFilePanelDirs[i] = common.Config.DefaultDirectory
+func updateFirstFilePanelPaths(firstPanelPaths []string, cwd string, zClient *zoxidelib.Client) {
+	for i := range firstPanelPaths {
+		if firstPanelPaths[i] == "" {
+			firstPanelPaths[i] = common.Config.DefaultDirectory
 		}
-		originalPath := firstFilePanelDirs[i]
-		firstFilePanelDirs[i] = utils.ResolveAbsPath(cwd, firstFilePanelDirs[i])
-		if _, err := os.Stat(firstFilePanelDirs[i]); err != nil {
+		originalPath := firstPanelPaths[i]
+		firstPanelPaths[i] = utils.ResolveAbsPath(cwd, firstPanelPaths[i])
+		if _, err := os.Stat(firstPanelPaths[i]); err != nil {
 			slog.Error("cannot get stats for firstFilePanelDir", "error", err)
 			// In case the path provided did not exist, use zoxide query
 			// else, fallback to home dir
 			if common.Config.ZoxideSupport && zClient != nil {
 				path, err := attemptZoxideForInitPath(originalPath, zClient)
 				if err != nil {
 					slog.Error("Zoxide query error", "originalPath", originalPath, "error", err)
-					firstFilePanelDirs[i] = variable.HomeDir
+					firstPanelPaths[i] = variable.HomeDir
 				} else {
-					firstFilePanelDirs[i] = path
+					firstPanelPaths[i] = path
 				}
 			} else {
-				firstFilePanelDirs[i] = variable.HomeDir
+				firstPanelPaths[i] = variable.HomeDir
 			}
 		}
 	}

src/internal/default_config.go

@@ -20,15 +20,15 @@ import (
 // TODO: Move the configuration parameters to a ModelConfig struct.
 // Something like `RendererConfig` struct for `Renderer` struct in ui/renderer package
 func defaultModelConfig(toggleDotFile, toggleFooter, firstUse bool,
-	firstFilePanelDirs []string, zClient *zoxidelib.Client) *model {
+	firstPanelPaths []string, zClient *zoxidelib.Client) *model {
 	return &model{
 		filePanelFocusIndex: 0,
 		focusPanel:          nonePanelFocus,
 		processBarModel:     processbar.New(),
 		sidebarModel:        sidebar.New(),
 		fileMetaData:        metadata.New(),
 		fileModel: fileModel{
-			filePanels:  filePanelSlice(firstFilePanelDirs),
+			filePanels:  filePanelSlice(firstPanelPaths),
 			filePreview: preview.New(),
 			width:       10,
 		},

src/internal/model.go

@@ -38,9 +38,9 @@ var (
 // is passed to tea.NewProgram() which accepts tea.Model
 // Either way type 'model' is not exported, so there is not way main package can
 // be aware of it, and use it directly
-func InitialModel(firstFilePanelDirs []string, firstUseCheck bool) tea.Model {
-	toggleDotFile, toggleFooter, zClient := initialConfig(firstFilePanelDirs)
-	return defaultModelConfig(toggleDotFile, toggleFooter, firstUseCheck, firstFilePanelDirs, zClient)
+func InitialModel(firstPanelPaths []string, firstUseCheck bool) tea.Model {
+	toggleDotFile, toggleFooter, zClient := initialConfig(firstPanelPaths)
+	return defaultModelConfig(toggleDotFile, toggleFooter, firstUseCheck, firstPanelPaths, zClient)
 }
 
 // Init function to be called by Bubble tea framework, sets windows title,
@@ -698,50 +698,82 @@ func getMaxW(s string) int {
 func (m *model) getFilePanelItems() {
 	focusPanel := m.fileModel.filePanels[m.filePanelFocusIndex]
 	for i, filePanel := range m.fileModel.filePanels {
-		var fileElement []element
 		nowTime := time.Now()
-		// Check last time each element was updated, if less then 3 seconds ignore
-		if !filePanel.isFocused && nowTime.Sub(filePanel.lastTimeGetElement) < 3*time.Second {
-			// TODO : revisit this. This feels like a duct tape solution of an actual
-			// deep rooted problem. This feels very hacky.
-			if !m.updatedToggleDotFile {
-				continue
-			}
-		}
-
-		focusPanelReRender := false
-
-		if len(focusPanel.element) > 0 {
-			if filepath.Dir(focusPanel.element[0].location) != focusPanel.location {
-				focusPanelReRender = true
-			}
-		} else {
-			focusPanelReRender = true
-		}
-
-		reRenderTime := int(float64(len(filePanel.element)) / 100)
-
-		if filePanel.isFocused && !focusPanelReRender &&
-			nowTime.Sub(filePanel.lastTimeGetElement) < time.Duration(reRenderTime)*time.Second {
+		if m.shouldSkipPanelUpdate(filePanel, focusPanel, nowTime) {
 			continue
 		}
 
-		// Get file names based on search bar filter
-		if filePanel.searchBar.Value() != "" {
-			fileElement = returnDirElementBySearchString(filePanel.location, m.toggleDotFile,
-				filePanel.searchBar.Value(), filePanel.sortOptions.data)
-		} else {
-			fileElement = returnDirElement(filePanel.location, m.toggleDotFile, filePanel.sortOptions.data)
-		}
+		// Load elements for this panel (with/without search filter)
+		fileElement := m.getElementsForPanel(filePanel)
+
 		// Update file panel list
-		filePanel.element = fileElement
 		m.fileModel.filePanels[i].element = fileElement
 		m.fileModel.filePanels[i].lastTimeGetElement = nowTime
+
+		// Position cursor on target file when provided
+		m.applyTargetFileCursor(i, fileElement)
 	}
 
 	m.updatedToggleDotFile = false
 }
 
+// Helper to decide whether to skip updating a panel this tick.
+func (m *model) shouldSkipPanelUpdate(filePanel filePanel, focusPanel filePanel, nowTime time.Time) bool {
+	// Throttle non-focused panels unless dotfile toggle changed
+	if !filePanel.isFocused && nowTime.Sub(filePanel.lastTimeGetElement) < 3*time.Second {
+		if !m.updatedToggleDotFile {
+			return true
+		}
+	}
+
+	focusPanelReRender := m.focusPanelNeedsReRender(focusPanel)
+	reRenderTime := int(float64(len(filePanel.element)) / 100)
+	if filePanel.isFocused && !focusPanelReRender &&
+		nowTime.Sub(filePanel.lastTimeGetElement) < time.Duration(reRenderTime)*time.Second {
+		return true
+	}
+	return false
+}
+
+// Checks whether the focus panel directory changed and forces a re-render.
+func (m *model) focusPanelNeedsReRender(focusPanel filePanel) bool {
+	if len(focusPanel.element) > 0 {
+		return filepath.Dir(focusPanel.element[0].location) != focusPanel.location
+	}
+	return true
+}
+
+// Retrieves elements for a panel based on search bar value and sort options.
+func (m *model) getElementsForPanel(filePanel filePanel) []element {
+	if filePanel.searchBar.Value() != "" {
+		return returnDirElementBySearchString(
+			filePanel.location,
+			m.toggleDotFile,
+			filePanel.searchBar.Value(),
+			filePanel.sortOptions.data,
+		)
+	}
+	return returnDirElement(filePanel.location, m.toggleDotFile, filePanel.sortOptions.data)
+}
+
+// Applies targetFile cursor positioning, if configured for the panel.
+func (m *model) applyTargetFileCursor(panelIndex int, elements []element) {
+	if tf := m.fileModel.filePanels[panelIndex].targetFile; tf == "" {
+		return
+	} else {
+		for idx, el := range elements {
+			if el.name == tf {
+				m.fileModel.filePanels[panelIndex].cursor = idx
+				if idx > 0 {
+					m.fileModel.filePanels[panelIndex].render = idx - 1
+				}
+				break
+			}
+		}
+		m.fileModel.filePanels[panelIndex].targetFile = ""
+	}
+}
+
 // Close superfile application. Cd into the current dir if CdOnQuit on and save
 // the path in state direcotory
 func (m *model) quitSuperfile(cdOnQuit bool) {

src/internal/model_test.go

@@ -95,6 +95,27 @@ func TestBasic(t *testing.T) {
 	})
 }
 
+func TestInitialFilePathPositionsCursor(t *testing.T) {
+	curTestDir := t.TempDir()
+	dir1 := filepath.Join(curTestDir, "dir1")
+	file1 := filepath.Join(dir1, "file1.txt")
+	file2 := filepath.Join(dir1, "file2.txt")
+	file3 := filepath.Join(dir1, "file3.txt")
+
+	utils.SetupDirectories(t, curTestDir, dir1)
+	utils.SetupFiles(t, file1, file2, file3)
+
+	// Initialize model with a file path; cursor should land on that file
+	m := defaultTestModel(dir1, file2)
+	p := NewTestTeaProgWithEventLoop(t, m)
+	// Cause an blocking Update()
+	p.SendDirectly(nil)
+	// Populate elements for the panel and apply target cursor
+	require.Len(t, m.fileModel.filePanels, 2)
+	assert.Equal(t, m.fileModel.filePanels[0].location, dir1)
+	assert.Equal(t, m.fileModel.filePanels[1].getSelectedItem().location, file2)
+}
+
 func TestQuit(t *testing.T) {
 	// Test
 	// 1 - Normal quit

src/internal/type.go

@@ -167,6 +167,7 @@ type filePanel struct {
 	renaming           bool
 	searchBar          textinput.Model
 	lastTimeGetElement time.Time
+	targetFile         string // filename to position cursor on after load
 }
 
 // Sort options

src/internal/type_utils.go

@@ -2,6 +2,8 @@ package internal
 
 import (
 	"fmt"
+	"os"
+	"path/filepath"
 
 	"github.com/yorukot/superfile/src/internal/common"
 	"github.com/yorukot/superfile/src/internal/utils"
@@ -36,21 +38,29 @@ func (m *model) validateLayout() error {
 
 // ================ filepanel
 
-func filePanelSlice(dir []string) []filePanel {
-	res := make([]filePanel, len(dir))
-	for i := range dir {
+func filePanelSlice(paths []string) []filePanel {
+	res := make([]filePanel, len(paths))
+	for i := range paths {
 		// Making the first panel as the focussed
 		isFocus := i == 0
-		res[i] = defaultFilePanel(dir[i], isFocus)
+		res[i] = defaultFilePanel(paths[i], isFocus)
 	}
 	return res
 }
 
-func defaultFilePanel(dir string, focused bool) filePanel {
+func defaultFilePanel(path string, focused bool) filePanel {
+	targetFile := ""
+	panelPath := path
+	// If path refers to a file, switch to its parent and remember the filename
+	if stat, err := os.Stat(panelPath); err == nil && !stat.IsDir() {
+		targetFile = filepath.Base(panelPath)
+		panelPath = filepath.Dir(panelPath)
+	}
+
 	return filePanel{
 		render:   0,
 		cursor:   0,
-		location: dir,
+		location: panelPath,
 		sortOptions: sortOptionsModel{
 			width:  20,
 			height: 4,
@@ -69,6 +79,7 @@ func defaultFilePanel(dir string, focused bool) filePanel {
 		isFocused:        focused,
 		directoryRecords: make(map[string]directoryRecord),
 		searchBar:        common.GenerateSearchBar(),
+		targetFile:       targetFile,
 	}
 }