history: stop spamming add() on arrow nav, only updatePath adds dirs

Iakov Senatov · Iakov Senatov · commit ee434a60d878 · 2026-03-10T17:43:20.000+01:00
diff --git a/GUI/Sources/States/AppState/SelectionManager.swift b/GUI/Sources/States/AppState/SelectionManager.swift
@@ -92,9 +92,10 @@ final class SelectionManager {
                 lastRecordedPathRight = path
         }
 
-        log.debug("[SelectionManager] history add \(panelSide): \(path)")
+        // Only setCurrent — don't add to history.
+        // History is populated in updatePath() when user actually enters a directory.
+        // Arrow-key navigation through files/dirs should NOT pollute history.
         selectionsHistory.setCurrent(to: path)
-        selectionsHistory.add(path)
     }
 
     /// Move selection up/down by step count

Original file line number	Diff line number	Diff line change
`@@ -92,9 +92,10 @@ final class SelectionManager {`
`92`	`92`	`lastRecordedPathRight = path`
`93`	`93`	`}`
`94`	`94`
`95`		`- log.debug("[SelectionManager] history add \(panelSide): \(path)")`
	`95`	`+ // Only setCurrent — don't add to history.`
	`96`	`+ // History is populated in updatePath() when user actually enters a directory.`
	`97`	`+ // Arrow-key navigation through files/dirs should NOT pollute history.`
`96`	`98`	`selectionsHistory.setCurrent(to: path)`
`97`		`- selectionsHistory.add(path)`
`98`	`99`	`}`
`99`	`100`
`100`	`101`	`/// Move selection up/down by step count`